Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0041267Retail ModulesWeb POSpublic2019-07-01 09:582019-08-06 16:12
Reporteraaroncalero 
Assigned Toranjith_qualiantech_com 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR19Q3Fixed in VersionRR19Q3 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerGold
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0041267: OB.UTIL.clone always copies arrays by reference
DescriptionOB.UTIL.clone always copies arrays by reference, instead of generating a new array and cloning the values.
Steps To ReproduceCheck the code that copies array properties:
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/tip/web/org.openbravo.mobile.core/source/utils/ob-utilities.js#l1057 [^]

Instead of checking if the property is an array, the code is checking if the whole object is an array, which is wrong
Proposed SolutionIn the linked code, change from
} else if (_.isArray(source)) {
to
} else if (_.isArray(source.get(key))) {
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0041207 closed ranjith_qualiantech_com OB.UTIL.clone always copies arrays by reference 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2019-07-03 19:04marvintmTypedefect => backport
2019-07-03 19:04marvintmTarget Version => RR19Q3
2019-07-18 11:11hgbotCheckin
2019-07-18 11:11hgbotNote Added: 0113506
2019-07-18 11:11hgbotCheckin
2019-07-18 11:11hgbotNote Added: 0113507
2019-07-18 11:11hgbotStatusscheduled => resolved
2019-07-18 11:11hgbotResolutionopen => fixed
2019-07-18 11:11hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/b963e8c9401e7a87f8758f6a08b89677a435a2cf [^] => http://code.openbravo.com/retail/backports/3.0RR19Q3/org.openbravo.mobile.core/rev/c3f4351df806aab962230ff302aa0e6386712d75 [^]
2019-08-06 16:12marvintmReview Assigned To => marvintm
2019-08-06 16:12marvintmStatusresolved => closed
2019-08-06 16:12marvintmFixed in Version => RR19Q3

Notes
(0113506)
hgbot   
2019-07-18 11:11   
Repository: retail/backports/3.0RR19Q3/org.openbravo.retail.posterminal
Changeset: cf8a6258cf114e96e6dea5d58dac67c6a455ba5d
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Jul 18 14:36:48 2019 +0530
URL: http://code.openbravo.com/retail/backports/3.0RR19Q3/org.openbravo.retail.posterminal/rev/cf8a6258cf114e96e6dea5d58dac67c6a455ba5d [^]

Related to issue 41267 : Shipping Address should be same as invoice address when loading paid receipt

* When splitting lines with promotions, line lineQtyOffer should be updated instead of creating new lines to apply promotions

---
M web/org.openbravo.retail.posterminal/js/model/order.js
---
(0113507)
hgbot   
2019-07-18 11:11   
Repository: retail/backports/3.0RR19Q3/org.openbravo.mobile.core
Changeset: c3f4351df806aab962230ff302aa0e6386712d75
Author: Ranjith S R <ranjith <at> qualiantech.com>
Date: Thu Jul 18 14:36:40 2019 +0530
URL: http://code.openbravo.com/retail/backports/3.0RR19Q3/org.openbravo.mobile.core/rev/c3f4351df806aab962230ff302aa0e6386712d75 [^]

Fixed issue 41267 : Update OB.UTIL.clone to check the property is an Array

* When cloning object from array, if object is backbone the it should cloned recursively
if object is not backbone, then copy of object should be pushed to target object

---
M web/org.openbravo.mobile.core/source/utils/ob-utilities.js
---