Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035807Openbravo ERPA. Platformpublic2017-03-30 16:222017-04-24 13:47
ReporterJONHM 
Assigned Toinigosanchez 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0PR16Q4.4Fixed in Version3.0PR16Q4.4 
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
Summary0035807: 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.0PR16Q4.4
2017-04-21 13:25inigosanchezReview Assigned To => caristu
2017-04-21 13:28hgbotCheckin
2017-04-21 13:28hgbotNote Added: 0096100
2017-04-21 13:28hgbotStatusscheduled => resolved
2017-04-21 13:28hgbotResolutionopen => fixed
2017-04-21 13:28hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR16Q4.4/rev/88cd2fb173126a9b027718440e32fd63e448d5f0 [^]
2017-04-24 11:59hgbotCheckin
2017-04-24 11:59hgbotNote Added: 0096118
2017-04-24 13:47caristuNote Added: 0096135
2017-04-24 13:47caristuStatusresolved => closed
2017-04-24 13:47caristuFixed in Version => 3.0PR16Q4.4

Notes
(0096100)
hgbot   
2017-04-21 13:28   
Repository: erp/backports/3.0PR16Q4.4
Changeset: 88cd2fb173126a9b027718440e32fd63e448d5f0
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Fri Apr 21 13:27:48 2017 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q4.4/rev/88cd2fb173126a9b027718440e32fd63e448d5f0 [^]

Fixed issue 35807: 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 32202 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 23136 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
---
(0096118)
hgbot   
2017-04-24 11:59   
Repository: erp/backports/3.0PR16Q4.4
Changeset: 4ea62d44a9751d03283b349ed404386216f7d445
Author: Inigo Sanchez <inigo.sanchez <at> openbravo.com>
Date: Mon Apr 24 11:58:21 2017 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR16Q4.4/rev/4ea62d44a9751d03283b349ed404386216f7d445 [^]

Related with issue 35807: 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
---
(0096135)
caristu   
2017-04-24 13:47   
Code reviewed + tested OK.