Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0034077
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSminorhave not tried2016-09-24 13:562016-09-28 09:11
ReportermtaalView Statuspublic 
Assigned Tomtaal 
PrioritynormalResolutionfixedFixed in VersionRR16Q4
StatusclosedFix in branchFixed in SCM revision4a9084e1f26f
ProjectionnoneETAnoneTarget VersionRR16Q4
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned ToSandrahuguet
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034077: Unnecessary set of json property with wrong value when saving cusotmer(address)

DescriptionThere are several places in the customer address code where the json property is set in the wrong way and unnecessarily as it was already set correctly.

Steps To ReproducePut breakpoints in the changed locations and check what is being set in the object's json property
Proposed Solutionremove the unnecessary line or use JSON.stringify to correctly set the value
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
depends on backport 0034026RR16Q3.2 closedmtaal JSON error unexpected token in remote BP data when saving address through customer - remove unnecessary json set 

-  Notes
(0090226)
hgbot (developer)
2016-09-25 11:49

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 4a9084e1f26fcbdaf885aefb77cb78fb6408de94
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun Sep 25 11:49:27 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4a9084e1f26fcbdaf885aefb77cb78fb6408de94 [^]

Fixes issue 34077: Unnecessary set of json property with wrong value when saving cusotmer(address)
Remove code which has no impact as it is set after saving the object and the logic is incorrect anyway

---
M web/org.openbravo.retail.posterminal/js/data/datacustomeraddrsave.js
M web/org.openbravo.retail.posterminal/js/data/datacustomersave.js
---
(0090324)
Sandrahuguet (developer)
2016-09-28 09:11

Reviewed and tested

- Issue History
Date Modified Username Field Change
2016-09-24 13:56 mtaal New Issue
2016-09-24 13:56 mtaal Assigned To => mtaal
2016-09-24 13:56 mtaal Triggers an Emergency Pack => No
2016-09-24 14:14 mtaal Summary Json property should always be set using stringified json => Unnecessary set of json property with wrong value when saving cusotmer(address)
2016-09-24 14:14 mtaal Description Updated View Revisions
2016-09-24 14:14 mtaal Steps to Reproduce Updated View Revisions
2016-09-24 14:14 mtaal Proposed Solution updated
2016-09-25 11:49 hgbot Checkin
2016-09-25 11:49 hgbot Note Added: 0090226
2016-09-25 11:49 hgbot Status new => resolved
2016-09-25 11:49 hgbot Resolution open => fixed
2016-09-25 11:49 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4a9084e1f26fcbdaf885aefb77cb78fb6408de94 [^]
2016-09-25 11:50 mtaal Review Assigned To => Sandrahuguet
2016-09-25 12:18 mtaal Relationship added depends on 0034026
2016-09-28 09:11 Sandrahuguet Note Added: 0090324
2016-09-28 09:11 Sandrahuguet Status resolved => closed
2016-09-28 09:11 Sandrahuguet Fixed in Version => RR16Q4


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker