jQuery Grid Plugin - jqGridForum

Topic RSS

Related Topics sp_TopicIcon

bug in unformatter of the currency

Tags: unformatter currency

15/02/2012
14:14

OlegK

Germany

Member

Members

Forum Posts: 1255

Member Since:
10/08/2009

Offline

Hello Tony,

the unformatter of currency removes prefix and suffix in the wrong way. The current implementation

ret = $(cellval).text().replace(stripTag,'').replace(op.decimalSeparator,'.')
.replace(op.prefix,'').replace(op.suffix,'');

produce "200.00€" if it unformats the string "200.00 €" where suffix: ' €' and thousandsSeparator: ' '. First all blanks will be removed from the string and then the suffix ' €' will be not more found.

The correct way will be first remove the prefix from the beginning of the string and remove the suffix from the end of the string. After that one can remove thousandsSeparator and replace decimalSeparator to '.'. So the code can be fixed as

ret = $(cellval).text();
if (op.prefix) {
ret = ret.substr(op.prefix.length);
}
if (op.suffix) {
ret = ret.substr(0, ret.length - op.suffix.length);
}
ret = ret.replace(stripTag,'').replace(op.decimalSeparator,'.');

Best regards
Oleg

16/02/2012
08:28

tony

Sofia, Bulgaria

Moderator

Members

Moderators

Forum Posts: 7721

Member Since:
30/10/2007

Offline

Oleg,

Your code is correct. The formatter first add spaces and separators and the prefix and suffix - i.e the unformat should be in reverse order - remove prefix and sufix and then for with separators.

Also fixed.

Thanks

For professional UI suites for Java Script and PHP visit us at our commercial products site - guriddo.net - by the very same guys that created jqGrid.

All RSS

Forum Timezone: Europe/Sofia

Most Users Ever Online: 715

Currently Online:
31 Guest(s)

Currently Browsing this Page:
1 Guest(s)