Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0032950 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] 02. Master data management | major | always | 2016-05-13 11:19 | 2016-09-02 14:28 | |||
| Reporter | gorkaion | View Status | public | |||||
| Assigned To | gorkaion | |||||||
| Priority | normal | Resolution | fixed | Fixed in Version | 3.0PR16Q3 | |||
| Status | closed | Fix in branch | Fixed in SCM revision | ea18c2c210a6 | ||||
| Projection | none | ETA | none | Target Version | pi | |||
| OS | Linux 32 bit | Database | Oracle | Java version | 1.5.0_15 | |||
| OS Version | Gentoo | Database version | 11g | Ant version | 1.7.0 | |||
| Product Version | SCM revision | |||||||
| Merge Request Status | ||||||||
| Review Assigned To | aferraz | |||||||
| OBNetwork customer | OBPS | |||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Support ticket | ||||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0032950: VariantChDescUpdateProcess executed several times for the same product | |||||||
| Description | The ProductCharacteristicValueEventHandler event handler calls the VariantChDescUpdateProcess process when a characteristic value related to a product is changed to update the characteristic description field of the product. This handler should check all the products affected during the transaction and at when the transactions completes successfully call the process to update the products. But there is an issue and in the same transaction several characteristic values are added to a single product the event handler will call the update process one time for each value added/updated. This is wrong as it should be called only once. | |||||||
| Steps To Reproduce | Create a process that adds several characteristic values to a product debug it to check that the process is called several times. | |||||||
| Proposed Solution | Store the productIds in a Set instead of a List so it does not add duplicates. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files | ||||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
||||||||
|
||||||||
 Relationships |
|
Notes |
|
|
(0086439) hgbot (developer) 2016-05-13 11:28 |
Repository: erp/devel/pi Changeset: ea18c2c210a6216be3b82301047617103d804460 Author: Gorka Ion Damián <gorkaion.damian <at> openbravo.com> Date: Fri May 13 11:26:47 2016 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/ea18c2c210a6216be3b82301047617103d804460 [^] Fixed issue 32950.Use HashSet instead of ArrayList to avoid duplicates The product ids are now stored using HashSet as this implementation does not allow duplicates. The method to create the vars in some scenarios is throwing a OBException that is properly catch. It has been changed the try to avoid the OBException and remove the error message from the logs. --- M src/org/openbravo/event/ProductCharacteristicValueEventHandler.java --- |
|
(0086466) aferraz (viewer) 2016-05-16 14:24 |
Code review OK |
|
(0087538) hudsonbot (viewer) 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 |
|
(0089690) hgbot (developer) 2016-09-02 08:49 |
Repository: erp/devel/pi Changeset: 992caa87c40b1a96d5fb5bd39711e928b608b061 Author: Gorka Ion Damián <gorkaion.damian <at> openbravo.com> Date: Wed Aug 31 15:02:29 2016 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/992caa87c40b1a96d5fb5bd39711e928b608b061 [^] Fixed issue 33210.Related to issue 32950. Use import entry instead of ProcessBundle Performance fixes on EventHandler. - Use import entry instead of ProcessBundle. --- M src-db/database/sourcedata/AD_REF_LIST.xml M src/org/openbravo/event/ProductCharacteristicValueEventHandler.java A src/org/openbravo/materialmgmt/VariantChDescUpdateProcessor.java --- |
|
(0089710) hudsonbot (viewer) 2016-09-02 14:28 |
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/0f7167eecae1 [^] Maturity status: Test |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2016-05-13 11:19 | gorkaion | New Issue | |
| 2016-05-13 11:19 | gorkaion | Assigned To | => gorkaion |
| 2016-05-13 11:19 | gorkaion | OBNetwork customer | => No |
| 2016-05-13 11:19 | gorkaion | Modules | => Core |
| 2016-05-13 11:19 | gorkaion | Triggers an Emergency Pack | => No |
| 2016-05-13 11:27 | gorkaion | Review Assigned To | => aferraz |
| 2016-05-13 11:28 | hgbot | Checkin | |
| 2016-05-13 11:28 | hgbot | Note Added: 0086439 | |
| 2016-05-13 11:28 | hgbot | Status | new => resolved |
| 2016-05-13 11:28 | hgbot | Resolution | open => fixed |
| 2016-05-13 11:28 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/ea18c2c210a6216be3b82301047617103d804460 [^] |
| 2016-05-16 14:24 | aferraz | Note Added: 0086466 | |
| 2016-05-16 14:24 | aferraz | Status | resolved => closed |
| 2016-05-16 14:24 | aferraz | Fixed in Version | => 3.0PR16Q3 |
| 2016-05-16 16:30 | aferraz | OBNetwork customer | No => Yes |
| 2016-06-17 19:37 | hudsonbot | Checkin | |
| 2016-06-17 19:37 | hudsonbot | Note Added: 0087538 | |
| 2016-08-31 13:21 | gorkaion | Relationship added | related to 0033210 |
| 2016-09-02 08:49 | hgbot | Checkin | |
| 2016-09-02 08:49 | hgbot | Note Added: 0089690 | |
| 2016-09-02 14:28 | hudsonbot | Checkin | |
| 2016-09-02 14:28 | hudsonbot | Note Added: 0089710 | |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |