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

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

Description of the change

Hello,

 - Fix the issue of commas are not used as a thousands separate in pos.

Thanks,
FKA

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

[MERGE] with trunk

Unmerged revisions

8523. By Foram Katharotiya (OpenERP)

[MERGE] with trunk

8522. By Foram Katharotiya (OpenERP)

[FIX] use comma as thousand seprator

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'point_of_sale/static/src/js/widgets.js'
--- point_of_sale/static/src/js/widgets.js 2013-01-14 15:25:33 +0000
+++ point_of_sale/static/src/js/widgets.js 2013-02-01 12:57:19 +0000
@@ -310,7 +310,7 @@
310 this.payment_line.bind('change', this.changedAmount, this);310 this.payment_line.bind('change', this.changedAmount, this);
311 },311 },
312 changeAmount: function(event) {312 changeAmount: function(event) {
313 var newAmount = event.currentTarget.value;313 var newAmount = event.currentTarget.value.replace(/,/g,'');
314 if (newAmount && !isNaN(newAmount)) {314 if (newAmount && !isNaN(newAmount)) {
315 this.amount = parseFloat(newAmount);315 this.amount = parseFloat(newAmount);
316 this.payment_line.set_amount(this.amount);316 this.payment_line.set_amount(this.amount);

Subscribers

People subscribed via source and target branches

to all changes: