Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0025569Retail ModulesWeb POSpublic2014-01-27 18:462014-01-31 13:49
Reporterjecharri 
Assigned Tomalsasua 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product VersionRMP30.1 
Target VersionRMP31Fixed in VersionRMP31 
Merge Request Status
Review Assigned Toguilleaer
OBNetwork customer
Support ticket
Regression levelProduction - Confirmed Stable
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackYes
Summary0025569: Impossible to do a cash up
DescriptionImpossible to do a cash up
Steps To Reproduce-Log in in https://livebuilds.openbravo.com/retail_pi_pgsql [^]

-Go to "Pos terminal type" window and choose "VBS POS Terminal Type"
-Go to payment method tab and uncheck in all methods the checkbox "Automated movement to other account"

-Go to Web POS and verify that it is impossible to do a cash up. You will see that the system never move the quantity to cash up financial account.
Proposed SolutionIt seems that we should change the following lines
if (model.get('paymentMethod').automatemovementtoother === false) {
        model.set('qtyToKeep', null);

in file modules/org.openbravo.retail.posterminal/web/org.openbravo.retail.posterminal/js/closecash/model/cashup-model.js

We may set to 0 instead of null
Additional Information
TagsNo tags attached.
Relationships
related to defect 0026469RR14Q2.1 closed migueldejuana Cash up is not working fine if you have a payment method with checkbox automated movement unchecked 
Attached Filesdiff issue25569.diff (1,887) 2014-01-29 13:42
https://issues.openbravo.com/file_download.php?file_id=6665&type=bug
Issue History
Date ModifiedUsernameFieldChange
2014-01-27 18:46jecharriNew Issue
2014-01-27 18:46jecharriAssigned To => marvintm
2014-01-27 18:46jecharriResolution time => 1391295600
2014-01-27 18:46jecharriTriggers an Emergency Pack => No
2014-01-27 18:47jecharriRegression level => Production - Confirmed Stable
2014-01-27 18:47jecharriTriggers an Emergency PackNo => Yes
2014-01-28 18:21malsasuaFile Added: issue25569.diff
2014-01-29 13:42malsasuaFile Deleted: issue25569.diff
2014-01-29 13:42malsasuaFile Added: issue25569.diff
2014-01-31 10:17mtaalAssigned Tomarvintm => malsasua
2014-01-31 13:46malsasuaStatusnew => scheduled
2014-01-31 13:46malsasuafix_in_branch => pi
2014-01-31 13:46malsasuaNote Added: 0063828
2014-01-31 13:46malsasuaStatusscheduled => resolved
2014-01-31 13:46malsasuaFixed in Version => RMP31
2014-01-31 13:46malsasuaResolutionopen => fixed
2014-01-31 13:49guilleaerReview Assigned To => guilleaer
2014-01-31 13:49guilleaerNote Added: 0063829
2014-01-31 13:49guilleaerStatusresolved => closed
2014-05-08 19:25dmitry_mezentsevRelationship addedrelated to 0026469

Notes
(0063828)
malsasua   
2014-01-31 13:46   
fixed by commits:
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/4cf8ba2457c5 [^]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a12cdbd42c37 [^]
(0063829)
guilleaer   
2014-01-31 13:49   
Verified and ready to be included in RMP31