jQuery Grid Plugin - jqGrid - Topic: bug in unformatter of the currency http://www.trirand.com/blog/?page_id=393/bugs/bug-in-unformatter-of-the-currency Simple:Press Version 5.7.5.3 tony on bug in unformatter of the currency http://www.trirand.com/blog/?page_id=393/bugs/bug-in-unformatter-of-the-currency#p25963 Bugs http://www.trirand.com/blog/?page_id=393/bugs/bug-in-unformatter-of-the-currency#p25963 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 

]]> Thu, 16 Feb 2012 08:28:08 +0200 OlegK on bug in unformatter of the currency http://www.trirand.com/blog/?page_id=393/bugs/bug-in-unformatter-of-the-currency#p25950 Bugs http://www.trirand.com/blog/?page_id=393/bugs/bug-in-unformatter-of-the-currency#p25950 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

]]> Wed, 15 Feb 2012 14:14:40 +0200