Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0017926Openbravo ERPA. Platformpublic2011-07-07 12:112011-07-08 12:49
Reporterplujan 
Assigned Toiperdomo 
PriorityimmediateSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
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
Summary0017926: Using the Product Selector in Purchase Invoice lines shows an error on saving
DescriptionSmoke tests have detected an error that blocks execution. When saving a line of a Purchase Invoice, it shows an error. The error is shown when using the magnifier icon to open the selector. If selecting by typing it works fine.
See steps.
Steps To Reproduce1. Enter as QAAdmin
2. Go to Purchase Invoice
3. Create a header for Vendor A
4. Save
5. Go to Lines and create a new line in Form mode
6. Using the magnifier icon, open the Product Selector and choose Raw Material A
7. Go to Invoiced Quantity and fill it with 11.2
8. Save. An error is shown (see attached screenshot)
9. Cancel and repeat from step 5, but this time typing RMA in the selector. Saving shows no error.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 00178683.0MP1 closed iperdomo The goods shipment attribute set shows an UUID 
Attached Filesjpg Saving_Purchase_Invoice_Lines.JPG (109,243) 2011-07-07 12:11
https://issues.openbravo.com/file_download.php?file_id=4301&type=bug
jpg
Issue History
Date ModifiedUsernameFieldChange
2011-07-07 12:11plujanNew Issue
2011-07-07 12:11plujanAssigned To => alostale
2011-07-07 12:11plujanFile Added: Saving_Purchase_Invoice_Lines.JPG
2011-07-07 12:11plujanWeb browser => Google Chrome, Mozilla Firefox
2011-07-07 12:11plujanModules => Core
2011-07-07 12:11plujanAssigned Toalostale => shuehner
2011-07-07 12:14shuehnerNote Added: 0038916
2011-07-07 12:14shuehnerAssigned Toshuehner => iperdomo
2011-07-07 12:17iperdomoRelationship addedrelated to 0017868
2011-07-07 12:17iperdomoStatusnew => scheduled
2011-07-07 12:17iperdomofix_in_branch => pi
2011-07-07 16:40hgbotCheckin
2011-07-07 16:40hgbotNote Added: 0038923
2011-07-07 16:40hgbotStatusscheduled => resolved
2011-07-07 16:40hgbotResolutionopen => fixed
2011-07-07 16:40hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/970ccf1a7b974a559d3100d67886204991ab8397 [^]
2011-07-07 21:40hudsonbotCheckin
2011-07-07 21:40hudsonbotNote Added: 0038930
2011-07-08 12:49shuehnerNote Added: 0038939
2011-07-08 12:49shuehnerStatusresolved => closed

Notes
(0038916)
shuehner   
2011-07-07 12:14   
Debugging this case shows that the 'save' request of the InvoiceLine is sending the uomid-value of the selected product as the productid.
(0038923)
hgbot   
2011-07-07 16:40   
Repository: erp/devel/pi
Changeset: 970ccf1a7b974a559d3100d67886204991ab8397
Author: Iván Perdomo <ivan.perdomo <at> openbravo.com>
Date: Thu Jul 07 16:33:50 2011 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/970ccf1a7b974a559d3100d67886204991ab8397 [^]

Fixes issue 17926: Use the fieldName instead of calculating it

---
M modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-item.js
---
(0038930)
hudsonbot   
2011-07-07 21:40   
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/bfc4b784e2e4 [^]

Maturity status: Test
(0038939)
shuehner   
2011-07-08 12:49   
Reviewed/tested in pi/pgsql/chromium (rev id: 435d6c2196c8). Testing on that pi revision does not reproduce the described bug any and also the smoke test execution now passed with that fix applied, additionally reporter tested in pi-mp1 branch and works for him. Code change does not try to calculate fieldname which failed in case of multi part fieldnames like (product.uOM.id) but directly uses fieldname available in the outFields definition so not writing the outfields value into some unrelated fields.