Merge lp:~openerp-dev/openobject-addons/trunk-bug-1099411-fka into lp:openobject-addons

Proposed by Foram Katharotiya (OpenERP)
Status: Needs review
Proposed branch: lp:~openerp-dev/openobject-addons/trunk-bug-1099411-fka
Merge into: lp:openobject-addons
Diff against target: 33 lines (+3/-1)
2 files modified
point_of_sale/static/src/js/models.js (+1/-0)
point_of_sale/static/src/js/widgets.js (+2/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-addons/trunk-bug-1099411-fka
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+145830@code.launchpad.net

Description of the change

Hello,

 - fix the issue of clicking on +/- after entering a payment amount manually with keyboard resets amount to 0 in pos.

Thanks,
FKA

To post a comment you must log in.
8528. By Foram Katharotiya (OpenERP)

[MERGE] with trunk

Unmerged revisions

8528. By Foram Katharotiya (OpenERP)

[MERGE] with trunk

8527. By Foram Katharotiya (OpenERP)

[IMP] improve code

8526. By Foram Katharotiya (OpenERP)

[IMP] add amount in buffer state

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/models.js'
2--- point_of_sale/static/src/js/models.js 2013-01-11 14:42:19 +0000
3+++ point_of_sale/static/src/js/models.js 2013-02-01 12:52:20 +0000
4@@ -515,6 +515,7 @@
5 },
6 //sets the amount of money on this payment line
7 set_amount: function(value){
8+ self.screen_selector.current_screen.numpadState.set({buffer: value.toString()});
9 this.amount = value;
10 this.trigger('change');
11 },
12
13=== modified file 'point_of_sale/static/src/js/widgets.js'
14--- point_of_sale/static/src/js/widgets.js 2013-01-14 15:25:33 +0000
15+++ point_of_sale/static/src/js/widgets.js 2013-02-01 12:52:20 +0000
16@@ -122,8 +122,8 @@
17 console.warn('TODO should not get there...?');
18 return;
19 }
20+ self.pos_widget.screen_selector.set_current_screen('payment');
21 self.pos.get('selectedOrder').addPaymentLine(self.cashRegister);
22- self.pos_widget.screen_selector.set_current_screen('payment');
23 });
24 },
25 });
26@@ -319,6 +319,7 @@
27 changedAmount: function() {
28 if (this.amount !== this.payment_line.get_amount())
29 this.renderElement();
30+ this.amount = this.payment_line.get_amount()
31 },
32 renderElement: function() {
33 var self = this;

Subscribers

People subscribed via source and target branches

to all changes: