Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034203Retail ModulesWeb POSpublic2016-10-13 17:092016-10-27 13:06
Reporteraaroncalero 
Assigned Tomario_castello 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q1 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerOBPS
Support ticket
Regression levelProduction - QA Approved
Regression date2016-05-19
Regression introduced in releaseRR16Q3.2
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/aefe9683c6dd07fb2eb0521b54960400126d8681 [^]
Triggers an Emergency PackNo
Summary0034203: [SERQA 1940] The OBPOS_PreDeleteCurrentOrder hook is called when tickets loaded from backend are removed
DescriptionWhen a layaway or receipt that has been loaded form backend to the web pos is removed/closed, the OBPOS_PreDeleteCurrentOrder is executed and it should not.
Steps To ReproduceLogin in web pos.
Open the console/dev tools and find the call to the OBPOS_PreDeleteCurrentOrder hook (OB.UTIL.HookManager.executeHooks('OBPOS_PreDeleteCurrentOrder' ...).
Set a breakpoint on that line.
Go to the menu > Layaways and load a layaway.
>The delete button will change to a X
Click on the delete button.
The code stops on the call to the OBPOS_PreDeleteCurrentOrder hook.
Proposed Solution
Additional Information
TagsSER-QA
Relationships
caused by defect 0033587 closed jorge-garcia PreDeleteCurrentOrderHook is not called either from the cashup or from the logout 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-10-13 17:09aaroncaleroNew Issue
2016-10-13 17:09aaroncaleroAssigned To => Retail
2016-10-13 17:09aaroncaleroOBNetwork customer => Yes
2016-10-13 17:09aaroncaleroResolution time => 1477519200
2016-10-13 17:09aaroncaleroRegression level => Production - QA Approved
2016-10-13 17:09aaroncaleroRegression date => 2016-05-19
2016-10-13 17:09aaroncaleroRegression introduced in release => RR16Q3.2
2016-10-13 17:09aaroncaleroRegression introduced by commit => https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/aefe9683c6dd07fb2eb0521b54960400126d8681 [^]
2016-10-13 17:09aaroncaleroTriggers an Emergency Pack => No
2016-10-13 17:15aaroncaleroSummaryThe OBPOS_PreDeleteCurrentOrder hook is called when tickets loaded from backend are removed => [SERQA 1940] The OBPOS_PreDeleteCurrentOrder hook is called when tickets loaded from backend are removed
2016-10-13 17:15aaroncaleroRelationship addedcaused by 0033587
2016-10-13 17:15aaroncaleroTag Attached: SER-QA
2016-10-17 15:16mario_castelloAssigned ToRetail => mario_castello
2016-10-17 15:17mario_castelloStatusnew => acknowledged
2016-10-17 16:09mario_castelloStatusacknowledged => scheduled
2016-10-20 16:36hgbotCheckin
2016-10-20 16:36hgbotNote Added: 0090731
2016-10-20 16:36hgbotStatusscheduled => resolved
2016-10-20 16:36hgbotResolutionopen => fixed
2016-10-20 16:36hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/02271b55949cbbd701b19a281b982ee1c9f04ad6 [^]
2016-10-27 13:06marvintmReview Assigned To => marvintm
2016-10-27 13:06marvintmStatusresolved => closed
2016-10-27 13:06marvintmFixed in Version => RR17Q1

Notes
(0090731)
hgbot   
2016-10-20 16:36   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 02271b55949cbbd701b19a281b982ee1c9f04ad6
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Mon Oct 17 08:29:49 2016 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/02271b55949cbbd701b19a281b982ee1c9f04ad6 [^]

Fixed issue 34203: Added validation on deleteOrder function to execute OBPOS_PreDeleteCurrentOrder hook for editable receipts only

---
M web/org.openbravo.retail.posterminal/js/model/order.js
---