Merge lp:~openerp-dev/openobject-addons/trunk-bug-1169629-Pooja into lp:openobject-addons

Proposed by Nimesh Contractor(Open ERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1169629-Pooja
Merge into: lp:openobject-addons
Diff against target: 14 lines (+3/-1)
1 file modified
point_of_sale/static/src/js/screens.js (+3/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1169629-Pooja
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+166469@code.launchpad.net

Description of the change

Hello,

Fix the virtual numpad "backspace button" error when there is no payment line.

To reproduce the issue.

1. Do an order on the POS.
2. Click a Payment mode.
3. In the payment widget, Cancel the payment (with the cross on the right of the payment)
   Then, when there's no more line of payment
4. Click on the Backspace or on a digit in the virtual numpad, you'll get an error.

Thank You.

To post a comment you must log in.

Unmerged revisions

8728. By Pooja Zankhariya(OpenERP)

[FIX] Fixed bug after cancelling payment

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'point_of_sale/static/src/js/screens.js'
2--- point_of_sale/static/src/js/screens.js 2013-04-09 14:35:36 +0000
3+++ point_of_sale/static/src/js/screens.js 2013-05-30 09:34:30 +0000
4@@ -1003,7 +1003,9 @@
5 this.numpadState.set({mode: 'payment'});
6 },
7 set_value: function(val) {
8- this.currentPaymentLines.last().set_amount(val);
9+ if (this.currentPaymentLines.last()) {
10+ this.currentPaymentLines.last().set_amount(val);
11+ }
12 },
13 });
14 }

Subscribers

People subscribed via source and target branches

to all changes: