Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0032781Openbravo ERP07. Sales managementpublic2016-04-14 14:222016-04-26 17:29
ngarcia 
platform 
immediatemajorunable to reproduce
closedfixed 
5
3.0PR16Q1 
3.0PR15Q4.63.0PR15Q4.6 
caristu
Core
No
0032781: Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices
Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices. Before 3.0PR15Q4 the problem could be solved by reactivating and processing the document. After the following commit

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

the only option is to delete the whole document (header and lines)
0. Login as group admin role
1. Create a Sales Order
2. Add a line to it and click on Book button
3. Before booking it keep the mouse in the New toolbar button
4. Press enter and at the same time click on the New button sometimes
    -> ERROR 1: Check the new header is created with the values of the old record in the status bar fields
            * Status: Booked
            * Total Amounts: The values in previous document
6. Fill in mandatory fields in header
7. Save header. See different behavior depending on the version:
   7.1. 3.0PR15Q4-3.0PR16Q1
     -> ERROR 2: status bar values:
          * Status: Draft (OK)
          * Total Amounts: Previous values are kept (ERROR)
   7.2. >=3.0PR16Q2
      -> OK: status bar values are reset:
          * Status: Draft
          * Total Amounts: 0


Error 7.2 is caused because in the save request numeric values are sending an outdated textualValue property, this property was removed in 16Q2 by fix for 0031901.
No tags attached.
blocks defect 0032668 closed alostale Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices 
Issue History
2016-04-26 09:50alostaleTypedefect => backport
2016-04-26 09:50alostaleTarget Version => 3.0PR15Q4.6
2016-04-26 11:06alostaleReview Assigned To => caristu
2016-04-26 11:43priyamIssue Monitored: priyam
2016-04-26 17:06hgbotCheckin
2016-04-26 17:06hgbotNote Added: 0085996
2016-04-26 17:06hgbotCheckin
2016-04-26 17:06hgbotNote Added: 0085997
2016-04-26 17:06hgbotStatusscheduled => resolved
2016-04-26 17:06hgbotResolutionopen => fixed
2016-04-26 17:06hgbotFixed in SCM revision => http://code.openbravo.com/erp/backports/3.0PR15Q4.6/rev/4a287889b6d1ab2ead088e0e2418aab3229045b2 [^]
2016-04-26 17:29caristuNote Added: 0085999
2016-04-26 17:29caristuStatusresolved => closed
2016-04-26 17:29caristuFixed in Version => 3.0PR15Q4.6

Notes
(0085996)
hgbot   
2016-04-26 17:06   
Repository: erp/backports/3.0PR15Q4.6
Changeset: 3ef47ab7dd19b0e71b9f3b8a8d0e5295c3607c3e
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Apr 26 10:50:48 2016 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR15Q4.6/rev/3ef47ab7dd19b0e71b9f3b8a8d0e5295c3607c3e [^]

related to bug 32781: wrong values displayed in status bar on record creation

  When a new record is created just after processing a previous one, status bar
  displayed valued of the previous record instead of being in New mode.

  Fixed by forcing status bar reset on FIC response for NEW.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
---
(0085997)
hgbot   
2016-04-26 17:06   
Repository: erp/backports/3.0PR15Q4.6
Changeset: 4a287889b6d1ab2ead088e0e2418aab3229045b2
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Apr 26 10:54:46 2016 +0200
URL: http://code.openbravo.com/erp/backports/3.0PR15Q4.6/rev/4a287889b6d1ab2ead088e0e2418aab3229045b2 [^]

fixed bug 32781: wrong amounts on new record just after processing another one

  When a new record was created (ie. for Order or Invoice) just after completing
  a previous one, status bar values could get not updated. In case of numeric values
  they preserved an obsolete textual value, which makes to update to previous value
  when saving, even if the status bar reflects the new state.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/ob-view-form.js
---
(0085999)
caristu   
2016-04-26 17:29   
Code review + testing OK, after following the steps:
- The status bar is in new status
- The new header is created with the new values (old values from previous order are not kept)