Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0019992
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformmajoralways2012-03-09 10:502012-04-01 20:39
ReporterSandrahuguetView Statuspublic 
Assigned Todbaz 
PriorityimmediateResolutionfixedFixed in Version3.0MP10
StatusclosedFix in branchpiFixed in SCM revision96262cde8de7
ProjectionnoneETAnoneTarget Versionpi
OSLinux 32 bitDatabasePostgreSQLJava version1.6.0_26
OS VersionUbuntu 11.10Database version8.4Ant version1.8.2
Product VersionpiSCM revision 
Review Assigned To
Web browserGoogle Chrome, Mozilla Firefox
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0019992: Wrong behavior in readOnlyLogicElement in utils.js

DescriptionIn initial Organization Setup, if 'include accounting' is checked, Currency field should be editable. Else, that is, if include accounting is not checked, currency field should be read only.

If the check is checked a few times, there are inconsistencies.
Steps To Reproduce- Go to 'initial organization setup'
- Realize that 'include accounting' is checked and currency is editable.
- Uncheck it. Currency turns into readonly. That is correct.
- Check it. Currency remains not editable. That is not correct.
- Uncheck it. Currency turns editable. That is not correct.
Proposed SolutionIt should change the function readOnlyLogicElement in utils.js as shown in the screenshot
TagsNo tags attached.
Attached Filespng file icon Pantallazo del 2012-03-09 09:36:31.png [^] (27,051 bytes) 2012-03-09 10:50

- Relationships Relation Graph ] Dependency Graph ]
related to defect 0012538 closedalostale onChange action is executed for red only drop down fields 
depends on defect 0016960 closedSandrahuguet Initial Org Setup - Read only logic on Currency not working properly 

-  Notes
(0046232)
hgbot (developer)
2012-03-09 13:56

Repository: erp/devel/pi
Changeset: 96262cde8de7e7cfe4c0c478916a2712d8d15a4b
Author: David Baz Fayos <david.baz <at> openbravo.com>
Date: Fri Mar 09 13:55:59 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/96262cde8de7e7cfe4c0c478916a2712d8d15a4b [^]

Fixed issue 19992: 'Initial Org Setup' 'Include accounting' check now works ok
while enabling/disabling other fields

---
M src/org/openbravo/erpCommon/security/Login.html
M src/org/openbravo/erpCommon/security/Login_F1.html
M web/js/utils.js
---
(0046233)
dbaz (developer)
2012-03-09 14:17

In case of backport to 2.50, omit the following, since Chrome is not supported in 2.50:

    3.16 + obj.removeAttribute('readOnly'); // To avoid in Chrome the dropdown arrow be light-greyed although it is not readonly anymore
    3.17 + // The previous statements are already needed since lower IE versions has problems handling "removeAttribute" function
(0046234)
dbaz (developer)
2012-03-09 14:20

The issue was caused by this issue

https://issues.openbravo.com/view.php?id=12538 [^]

particularly this:
https://code.openbravo.com/erp/devel/pi/rev/2c71d76b9c313bea91ae1292c010d353eae06295#l2.27 [^]

so it never has worked in 3.00, and it is more than two years old issue.
(0046506)
alostale (manager)
2012-03-16 10:54

Code reviewed and tested on pi@d14e8752f4e6
(0047078)
hudsonbot (developer)
2012-04-01 20:39

A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/640de9acc2d5 [^]

Maturity status: Test

- Issue History
Date Modified Username Field Change
2012-03-09 10:50 Sandrahuguet New Issue
2012-03-09 10:50 Sandrahuguet Assigned To => dbaz
2012-03-09 10:50 Sandrahuguet File Added: Pantallazo del 2012-03-09 09:36:31.png
2012-03-09 10:50 Sandrahuguet Web browser => Google Chrome, Mozilla Firefox
2012-03-09 10:50 Sandrahuguet Modules => Core
2012-03-09 10:51 Sandrahuguet Relationship added depends on 0016960
2012-03-09 10:53 Sandrahuguet Web browser Google Chrome, Mozilla Firefox => Google Chrome, Mozilla Firefox
2012-03-09 10:53 Sandrahuguet Proposed Solution updated
2012-03-09 10:58 Sandrahuguet Tag Attached: Regression
2012-03-09 13:48 dbaz Relationship added related to 0012538
2012-03-09 13:56 hgbot Checkin
2012-03-09 13:56 hgbot Note Added: 0046232
2012-03-09 13:56 hgbot Status new => resolved
2012-03-09 13:56 hgbot Resolution open => fixed
2012-03-09 13:56 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/96262cde8de7e7cfe4c0c478916a2712d8d15a4b [^]
2012-03-09 14:17 dbaz Note Added: 0046233
2012-03-09 14:20 dbaz Note Added: 0046234
2012-03-12 11:45 alostale Tag Detached: Regression
2012-03-16 10:54 alostale Note Added: 0046506
2012-03-16 10:54 alostale Status resolved => closed
2012-03-16 10:54 alostale Fixed in Version => 3.0MP11
2012-03-16 10:54 alostale Status closed => new
2012-03-16 10:54 alostale Resolution fixed => open
2012-03-16 10:54 alostale Fixed in Version 3.0MP11 =>
2012-03-16 10:54 alostale Status new => scheduled
2012-03-16 10:54 alostale fix_in_branch => pi
2012-03-16 10:54 alostale Status scheduled => resolved
2012-03-16 10:54 alostale Resolution open => fixed
2012-03-16 10:54 alostale Status resolved => closed
2012-03-16 10:54 alostale Fixed in Version => 3.0MP10
2012-04-01 20:39 hudsonbot Checkin
2012-04-01 20:39 hudsonbot Note Added: 0047078


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker