Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035805Openbravo ERPA. Platformpublic2017-03-30 16:222017-04-24 13:45
ReporterJONHM 
Assigned Toinigosanchez 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0PR17Q2Fixed in Version3.0PR17Q2 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-02-11
Regression introduced in release3.0PR16Q2
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/670d9903e7053700dad4cd2321b3df166571e122 [^]
Triggers an Emergency PackNo
Summary0035805: Fields redraws in grid view when a numeric field is referenced by a read only logic
DescriptionWhen editing a numeric field in grid view which is referenced by read only logic by some other field of the same table, it redraws and format its value automatically
Steps To ReproduceAs System Administrator:
1) Open 'Tables and columns' window
2) Select C_OrderLine table and PriceActual field
3) Edit its Read Only Logic adding the reference to the PriceList column:
 @Processed@='Y' | @GROSSPRICE@='Y' & @PriceList@!0
4) Then, make the same step with the PriceList field, referencing the PriceActual column:
 @Processed@='Y' & @PriceActual@!0

As 'F&B International Group Admin':
4) Open Sales Order window and create a new record
5) Create a new record in grid view into [Lines] subtab also change its 'Net Unit Price'
--> The net unit price will be redrawed inmediately
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0035658 closed inigosanchez Fields redraws in grid view when a numeric field is referenced by a read only logic 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-04-20 10:57inigosanchezTypedefect => backport
2017-04-20 10:57inigosanchezTarget Version => 3.0PR17Q2
2017-04-21 13:32inigosanchezReview Assigned To => caristu
2017-04-21 13:34hgbotCheckin
2017-04-21 13:34hgbotNote Added: 0096102
2017-04-21 13:34hgbotStatusscheduled => resolved
2017-04-21 13:34hgbotResolutionopen => fixed
2017-04-21 13:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR17Q2/rev/adcbc4652225d0994466160d6d9c21ab77809e18 [^]
2017-04-24 12:04hgbotCheckin
2017-04-24 12:04hgbotNote Added: 0096120
2017-04-24 13:45caristuNote Added: 0096133
2017-04-24 13:45caristuStatusresolved => closed
2017-04-24 13:45caristuFixed in Version => 3.0PR17Q2

Notes
(0096102)
hgbot   
2017-04-21 13:34   
Repository: erp/backports/3.0PR17Q2
Changeset: adcbc4652225d0994466160d6d9c21ab77809e18
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Fri Apr 21 13:33:24 2017 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR17Q2/rev/adcbc4652225d0994466160d6d9c21ab77809e18 [^]

Fixed issue 35805: Fields redraw instantly in grid under some circumstances

The problem was that numeric fields redraws instantly in grid view when a numeric field
was referenced by a read only logic. This reported problem was caused by the partial fix
of 0032202 regression.

The fix of 0032202 was partial because the added condition did not taken into account the
grid view properly. This problem has been resolved by take into account form and grid view.

Besides another problem was found and resolved with this fix. Since this regression was
introduced, the 0023136 issue was broken too. This problem has been resolved too with this solution.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-number.js
---
(0096120)
hgbot   
2017-04-24 12:04   
Repository: erp/backports/3.0PR17Q2
Changeset: e17feb0f32b4e4cfd45e698b78bd0ee3f4b6198e
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Mon Apr 24 12:03:31 2017 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR17Q2/rev/e17feb0f32b4e4cfd45e698b78bd0ee3f4b6198e [^]

Related with issue 35805: Codereview take into account all the cases

Now it is take into account when a parameter window with grid is opened from the menu.
Rest of the cases sucha as grid, form and parameter window from a button is working as
expected too.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-number.js
---
(0096133)
caristu   
2017-04-24 13:45   
Code reviewed + tested OK.