Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0037393Retail ModulesWeb POSpublic2017-11-27 13:512018-01-23 16:36
Reporterjavietxe 
Assigned Tojorge-garcia 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR18Q1 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0037393: [SERQA 3550]The hooks in JS for the Undo actions have the SelectedLines arg missing.
DescriptionOnly the order comes to the hooks of Undo actions:
OBPOS_PreUndo_ + action
OBPOS_PostUndo_ + action

Regarding the wiki, the SelectedLines have to came also in the args for these hooks to be able to identify the lines that are being affected by the Undo action.
Steps To ReproduceCreate a hook and see that args.selectedLines is always undefined.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-11-27 13:51javietxeNew Issue
2017-11-27 13:51javietxeAssigned To => Retail
2017-11-27 13:51javietxeResolution time => 1512946800
2017-11-27 13:51javietxeTriggers an Emergency Pack => No
2017-12-01 13:01jorge-garciaStatusnew => scheduled
2017-12-01 13:01jorge-garciaAssigned ToRetail => jorge-garcia
2017-12-02 09:43hgbotCheckin
2017-12-02 09:43hgbotNote Added: 0100902
2017-12-02 09:43hgbotStatusscheduled => resolved
2017-12-02 09:43hgbotResolutionopen => fixed
2017-12-02 09:43hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b7df9fb170a2330e37a0be56221999c18cd0da51 [^]
2017-12-04 15:46migueldejuanaReview Assigned To => migueldejuana
2017-12-04 15:46migueldejuanaNote Added: 0100917
2017-12-04 15:46migueldejuanaStatusresolved => closed
2017-12-04 15:46migueldejuanaFixed in Version => RR18Q1
2018-01-23 16:36hgbotCheckin
2018-01-23 16:36hgbotNote Added: 0101917

Notes
(0100902)
hgbot   
2017-12-02 09:43   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: b7df9fb170a2330e37a0be56221999c18cd0da51
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Dec 01 13:00:53 2017 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/b7df9fb170a2330e37a0be56221999c18cd0da51 [^]

Fixed issue 37393: [SERQA 3550]The hooks in JS for the Undo actions have
the SelectedLines arg missing

The selected lines set by the hook are the one set in the undo action.

---
M web/org.openbravo.retail.posterminal/js/pointofsale/view/scan.js
---
(0100917)
migueldejuana   
2017-12-04 15:46   
Tested and reviewed
(0101917)
hgbot   
2018-01-23 16:36   
Repository: tools/automation/pi-mobile
Changeset: bcffc4a23cac03e4420ae428cbb1c916bab1fe07
Author: Alejandro <alekosmp86 <at> gmail.com>
Date: Mon Jan 22 08:57:41 2018 -0500
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/bcffc4a23cac03e4420ae428cbb1c916bab1fe07 [^]

Verifies issue 37393: Added automated test 'I37393_VerifyVarsInUndoHooks'

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/receipts/I37393_VerifyVarsInUndoHooks.java
---