Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0032668Openbravo ERP07. Sales managementpublic2016-04-14 14:222016-06-17 19:37
Reporterngarcia 
Assigned Toalostale 
PriorityimmediateSeveritymajorReproducibilityunable to reproduce
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR16Q1 
Target VersionFixed in Version3.0PR16Q3 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2015-07-16
Regression introduced in release3.0PR15Q4
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/8860b648abb2 [^]
Triggers an Emergency PackNo
Summary0032668: Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices
DescriptionWrong 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)
Steps To ReproduceIn order to easily reproduce in a local instance, set browser's network throttling to ie. Regular 3G (100ms, 750kb/s, 250kb/s)

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.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0031901 closed caristu decimal value can't be changed by trigger/observer after manual edition in form view 
related to defect 0020474 closed mtaal When you save a register in grid view, amounts fields change to the previous value 
depends on backport 00327793.0PR16Q2 closed platform Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices 
depends on backport 00327803.0PR16Q1.1 closed platform Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices 
depends on backport 00327813.0PR15Q4.6 closed platform Wrong Total Gross Amount, Total Net Amount and Tax information is obtained in order and invoices 
caused by feature request 0028366 closed adrianromero Add new functionality to calculate taxes per product BOM 
related to defect 0036454 closed alostale editing in grid record can be saved before callouts are applied 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-04-14 14:22ngarciaNew Issue
2016-04-14 14:22ngarciaAssigned To => Triage Finance
2016-04-14 14:22ngarciaModules => Core
2016-04-14 14:22ngarciaTriggers an Emergency Pack => No
2016-04-14 14:22ngarciaIssue Monitored: networkb
2016-04-25 11:41ngarciaNote Added: 0085954
2016-04-25 13:13ngarciaResolution time => 1464127200
2016-04-25 18:16ngarciaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=11859#r11859
2016-04-25 18:38ngarciaAssigned ToTriage Finance => platform
2016-04-25 18:38ngarciaPriorityurgent => immediate
2016-04-25 18:38ngarciaversion => 3.0PR16Q1
2016-04-26 08:39egoitzIssue Monitored: egoitz
2016-04-26 08:42alostaleRelationship addedrelated to 0031901
2016-04-26 08:42alostaleRelationship addedcaused by 0028366
2016-04-26 08:50alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=11860#r11860
2016-04-26 08:56alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=11861#r11861
2016-04-26 08:56alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=11862#r11862
2016-04-26 09:30alostaleRelationship addedrelated to 0020474
2016-04-26 09:50alostaleStatusnew => scheduled
2016-04-26 09:50alostaleAssigned Toplatform => alostale
2016-04-26 10:38alostaleRegression level => Production - Confirmed Stable
2016-04-26 10:38alostaleRegression date => 2015-07-16
2016-04-26 10:38alostaleRegression introduced in release => 3.0PR15Q4
2016-04-26 10:38alostaleRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/8860b648abb2 [^]
2016-04-26 10:38alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=11863#r11863
2016-04-26 11:05alostaleReview Assigned To => caristu
2016-04-26 12:18jfrancesIssue Monitored: jfrances
2016-04-26 15:59hgbotCheckin
2016-04-26 15:59hgbotNote Added: 0085995
2016-04-26 15:59hgbotStatusscheduled => resolved
2016-04-26 15:59hgbotResolutionopen => fixed
2016-04-26 15:59hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/e5e6126fc21b6f2e5ad3ecaed5bc0429335342fb [^]
2016-04-26 17:30caristuNote Added: 0086001
2016-04-26 17:30caristuStatusresolved => closed
2016-04-26 17:30caristuFixed in Version => 3.0PR16Q3
2016-05-04 08:31alostaleNote Added: 0086169
2016-06-17 19:37hudsonbotCheckin
2016-06-17 19:37hudsonbotNote Added: 0087492
2017-07-11 09:06alostaleRelationship addedrelated to 0036454

Notes
(0085954)
ngarcia   
2016-04-25 11:41   
Sometimes, the difference is the total amount of the previously created record. It seems the record is created with a value different from 0 in the total fields of the header
(0085995)
hgbot   
2016-04-26 15:59   
Repository: erp/devel/pi
Changeset: e5e6126fc21b6f2e5ad3ecaed5bc0429335342fb
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Tue Apr 26 10:47:23 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/e5e6126fc21b6f2e5ad3ecaed5bc0429335342fb [^]

fixed bug 32668: 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
---
(0086001)
caristu   
2016-04-26 17:30   
Code review + testing OK, after following the steps the status bar is in new status
(0086169)
alostale   
2016-05-04 08:31   
Test link: https://testlink.openbravo.com/testlink/linkto.php?tprojectPrefix=Communit&item=testcase&id=Communit-10018 [^]
(0087492)
hudsonbot   
2016-06-17 19:37   
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/0dc7be081b1c [^]
Maturity status: Test