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

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0041598
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2019-08-07 13:412019-08-21 15:17
ReporterumartirenaView Statuspublic 
Assigned Torqueralta 
PriorityimmediateResolutionfixedFixed in VersionRR19Q4
StatusclosedFix in branchFixed in SCM revisione2331459b611
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0041598: Create a new shipping address and assign to a receipt returns a JS error

DescriptionCreate a new shipping address and assign to a receipt returns a JS error.
Steps To ReproduceLogin into https://livebuilds.openbravo.com/retail_pi_pgsql/web/org.openbravo.retail.posterminal/?terminal=VBS-1 [^]
 * Click on the assigned shipping address
 * Click on new address
 * Set an address name
 * Uncheck Invoicing Address
 * Save
 * Click on Assign to Receipt (Ship.)
 * JS error is raised

Uncaught TypeError: Cannot read property 'push' of undefined
    at Object.tap (74e906cc00922c64cb590e2f300f59a8.js:105548)
    at Object.<anonymous> (underscore-1.3.3.min.js:23)
    at Object.origTap (underscore-1.3.3.min.js:23)
    at Object.tap (underscore-1.3.3.min.js:23)
    at Object.dispatch (enyo.js:628)
    at Object.dispatchEvent (enyo.js:612)
    at Object.enyo.kind.inherited (enyo.js:410)
    at Object.dispatchEvent (enyo.js:1338)
    at Object.dispatchBubble (enyo.js:616)
    at Object.bubble (enyo.js:604)
TagsNo tags attached.
Attached Filespng file icon Screenshot from 2019-08-07 13-37-07.png [^] (311,594 bytes) 2019-08-07 13:41

- Relationships [ Relation Graph ] [ Dependency Graph ]
depends on backport 0041601RR19Q3 closedrqueralta Create a new shipping address and assign to a receipt returns a JS error 
caused by defect 0040880 closedrqueralta Creation and Assigning of New Shipment Address for customer is not working properly 

-  Notes
(0113887)
hgbot (developer)
2019-08-08 05:02

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: c4a80a14233d9da4439c738d5969b4530174f0f9
Author: Rafael Queralta <rafaelcuba81 <at> gmail.com>
Date: Wed Aug 07 10:53:59 2019 -0400
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c4a80a14233d9da4439c738d5969b4530174f0f9 [^]

Fixed issue 41598: Create a new shipping address and assign to a receipt returns
a JS error

- Initialized locations array before insertion if undefined

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customeraddress/customeraddrdetailview.js
---
(0113902)
marvintm (manager)
2019-08-08 12:08

 Still doesn't work fully correctly.

In both local and remote mode, now there is no javascript error, but when clicking on "Assign to receipt (ship)", nothing happens, and the previous address is kept instead.
(0113926)
hgbot (developer)
2019-08-09 12:26

 Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: fc1f033b3629682fc501653423873bd2c3d4d50a
Author: Rafael Queralta <rafaelcuba81 <at> gmail.com>
Date: Thu Aug 08 18:56:03 2019 -0400
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fc1f033b3629682fc501653423873bd2c3d4d50a [^]

Fixed issue 41598: Create a new shipping address and assign to a receipt returns
a JS error

- The saveLocModel function was overwriting the value of the new shipping address
with the old value

---
M web/org.openbravo.retail.posterminal/js/model/order.js
M web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customeraddress/customeraddrdetailview.js
---
(0113931)
aaroncalero (manager)
2019-08-09 14:52

 The issue seems to be not fully fixed yet. There are cases in which the new address is still not assigned:
 * Click on the assigned shipping address
 * Click on new address
 * Set an address name
 * Uncheck Invoicing Address
 * Save
 * Click on Assign to Receipt (Ship.)
 * Click again on the assigned shipping address
 * Click on the assigned shipping address
 * Click on new address
 * Set an address name
 * Uncheck Invoicing Address
 * Save
 * Click on Assign to Receipt (Ship.)
==> The assigned address seems to be a random shipping address from the customer. Without checking the code, looks like a query without the correct ID filter (only filtering by bp id)
(0114022)
hgbot (developer)
2019-08-15 15:01

 Repository: retail/backports/3.0RR19Q3/org.openbravo.retail.posterminal
Changeset: e2331459b6115b68e8120072283d7501d0fc2309
Author: Rafael Queralta <rafaelcuba81 <at> gmail.com>
Date: Thu Aug 15 09:00:25 2019 -0400
URL: http://code.openbravo.com/retail/backports/3.0RR19Q3/org.openbravo.retail.posterminal/rev/e2331459b6115b68e8120072283d7501d0fc2309 [^]

Fixed issue 41598: Create a new shipping address and assign to a receipt returns
a JS error

- The saveLocModel function was overwriting the value of the new shipping address
with the old value

---
M web/org.openbravo.retail.posterminal/js/model/order.js
M web/org.openbravo.retail.posterminal/js/pointofsale/view/subwindows/customeraddress/customeraddrdetailview.js
---
(0114080)
marvintm (manager)
2019-08-19 11:55

 We haven't been able to reproduce the issue reported by Aarón. If we manage to reproduce it at some point, we will report a new issue for it.
(0114110)
hgbot (developer)
2019-08-21 15:17

 Repository: tools/automation/pi-mobile
Changeset: 1aac6f62be3a5f09daa8196b8ac04d21a9460b5b
Author: Alejandro <alekosmp86 <at> gmail.com>
Date: Mon Aug 19 11:35:59 2019 -0400
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/1aac6f62be3a5f09daa8196b8ac04d21a9460b5b [^]

Related to issue 41598: Added automated test

---
M src-test/org/openbravo/test/mobile/core/utils/AllowedErrorsHelper.java
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/customer/I41598_VerifyNewShipmentAddressNotRaiseJSError.java
---

- Issue History
Date Modified Username Field Change
2019-08-07 13:41 umartirena New Issue
2019-08-07 13:41 umartirena Assigned To => Retail
2019-08-07 13:41 umartirena File Added: Screenshot from 2019-08-07 13-37-07.png
2019-08-07 13:41 umartirena Triggers an Emergency Pack => No
2019-08-07 13:43 umartirena Relationship added caused by 0040880
2019-08-07 14:23 guillermogil Resolution time => 1566338400
2019-08-07 14:42 marvintm Status new => scheduled
2019-08-07 14:42 marvintm Assigned To Retail => rqueralta
2019-08-08 05:02 hgbot Checkin
2019-08-08 05:02 hgbot Note Added: 0113887
2019-08-08 05:02 hgbot Status scheduled => resolved
2019-08-08 05:02 hgbot Resolution open => fixed
2019-08-08 05:02 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c4a80a14233d9da4439c738d5969b4530174f0f9 [^]
2019-08-08 12:08 marvintm Note Added: 0113902
2019-08-08 12:08 marvintm Status resolved => new
2019-08-08 12:08 marvintm Resolution fixed => open
2019-08-09 12:26 hgbot Checkin
2019-08-09 12:26 hgbot Note Added: 0113926
2019-08-09 12:26 hgbot Status new => resolved
2019-08-09 12:26 hgbot Resolution open => fixed
2019-08-09 12:26 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/c4a80a14233d9da4439c738d5969b4530174f0f9 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fc1f033b3629682fc501653423873bd2c3d4d50a [^]
2019-08-09 13:03 marvintm Review Assigned To => marvintm
2019-08-09 13:03 marvintm Status resolved => closed
2019-08-09 13:03 marvintm Fixed in Version => RR19Q4
2019-08-09 14:52 aaroncalero Note Added: 0113931
2019-08-09 14:52 aaroncalero Status closed => new
2019-08-09 14:52 aaroncalero Resolution fixed => open
2019-08-09 14:52 aaroncalero Fixed in Version RR19Q4 =>
2019-08-15 15:01 hgbot Checkin
2019-08-15 15:01 hgbot Note Added: 0114022
2019-08-15 15:01 hgbot Status new => resolved
2019-08-15 15:01 hgbot Resolution open => fixed
2019-08-15 15:01 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/fc1f033b3629682fc501653423873bd2c3d4d50a [^] => http://code.openbravo.com/retail/backports/3.0RR19Q3/org.openbravo.retail.posterminal/rev/e2331459b6115b68e8120072283d7501d0fc2309 [^]
2019-08-19 11:55 marvintm Note Added: 0114080
2019-08-19 11:55 marvintm Status resolved => closed
2019-08-19 11:55 marvintm Fixed in Version => RR19Q4
2019-08-21 15:17 hgbot Checkin
2019-08-21 15:17 hgbot Note Added: 0114110

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker