Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0019992Openbravo ERPA. Platformpublic2012-03-09 10:502012-04-01 20:39
ReporterSandrahuguet 
Assigned Todbaz 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS20OS VersionUbuntu 11.10
Product Versionpi 
Target VersionpiFixed in Version3.0MP10 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browserGoogle Chrome, Mozilla Firefox
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0019992: 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
Additional Information
TagsNo tags attached.
Relationships
related to defect 0012538 closed alostale onChange action is executed for red only drop down fields 
depends on defect 0016960 closed Sandrahuguet Initial Org Setup - Read only logic on Currency not working properly 
Attached Filespng Pantallazo del 2012-03-09 09:36:31.png (27,051) 2012-03-09 10:50
https://issues.openbravo.com/file_download.php?file_id=5075&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2012-03-09 10:50SandrahuguetNew Issue
2012-03-09 10:50SandrahuguetAssigned To => dbaz
2012-03-09 10:50SandrahuguetFile Added: Pantallazo del 2012-03-09 09:36:31.png
2012-03-09 10:50SandrahuguetWeb browser => Google Chrome, Mozilla Firefox
2012-03-09 10:50SandrahuguetModules => Core
2012-03-09 10:51SandrahuguetRelationship addeddepends on 0016960
2012-03-09 10:53SandrahuguetWeb browserGoogle Chrome, Mozilla Firefox => Google Chrome, Mozilla Firefox
2012-03-09 10:53SandrahuguetProposed Solution updated
2012-03-09 10:58SandrahuguetTag Attached: Regression
2012-03-09 13:48dbazRelationship addedrelated to 0012538
2012-03-09 13:56hgbotCheckin
2012-03-09 13:56hgbotNote Added: 0046232
2012-03-09 13:56hgbotStatusnew => resolved
2012-03-09 13:56hgbotResolutionopen => fixed
2012-03-09 13:56hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/96262cde8de7e7cfe4c0c478916a2712d8d15a4b [^]
2012-03-09 14:17dbazNote Added: 0046233
2012-03-09 14:20dbazNote Added: 0046234
2012-03-12 11:45alostaleTag Detached: Regression
2012-03-16 10:54alostaleNote Added: 0046506
2012-03-16 10:54alostaleStatusresolved => closed
2012-03-16 10:54alostaleFixed in Version => 3.0MP11
2012-03-16 10:54alostaleStatusclosed => new
2012-03-16 10:54alostaleResolutionfixed => open
2012-03-16 10:54alostaleFixed in Version3.0MP11 =>
2012-03-16 10:54alostaleStatusnew => scheduled
2012-03-16 10:54alostalefix_in_branch => pi
2012-03-16 10:54alostaleStatusscheduled => resolved
2012-03-16 10:54alostaleResolutionopen => fixed
2012-03-16 10:54alostaleStatusresolved => closed
2012-03-16 10:54alostaleFixed in Version => 3.0MP10
2012-04-01 20:39hudsonbotCheckin
2012-04-01 20:39hudsonbotNote Added: 0047078

Notes
(0046232)
hgbot   
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   
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   
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   
2012-03-16 10:54   
Code reviewed and tested on pi@d14e8752f4e6
(0047078)
hudsonbot   
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