← Back to team overview

openerp-india team mailing list archive

[Bug 966083] [NEW] Space as thousands separator does not show on reports

 

Public bug reported:

the current code deletes all spaces from floats in reports. This was
reported ages ago in #615403 but never fixed. The fix is extremely
simple there is no reason not too fix it.

Here is a diff of the fix to the bug (we just need to remove the code that removes the white space)
=== modified file 'openerp/addons/base/res/res_lang.py'
--- openerp/addons/base/res/res_lang.py	2012-01-24 13:17:05 +0000
+++ openerp/addons/base/res/res_lang.py	2012-03-27 10:06:16 +0000
@@ -214,11 +214,6 @@
                 parts[0], seps = intersperse(parts[0], eval_lang_grouping, thousands_sep)
 
             formatted = decimal_point.join(parts)
-            while seps:
-                sp = formatted.find(' ')
-                if sp == -1: break
-                formatted = formatted[:sp] + formatted[sp+1:]
-                seps -= 1
         elif percent[-1] in 'diu':
             if grouping:
                 formatted = intersperse(formatted, eval_lang_grouping, thousands_sep)[0]

** Affects: openobject-server
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of OpenERP
Indian Team, which is subscribed to OpenERP Server.
https://bugs.launchpad.net/bugs/966083

Title:
  Space as thousands separator does not show on reports

Status in OpenERP Server:
  New

Bug description:
  the current code deletes all spaces from floats in reports. This was
  reported ages ago in #615403 but never fixed. The fix is extremely
  simple there is no reason not too fix it.

  Here is a diff of the fix to the bug (we just need to remove the code that removes the white space)
  === modified file 'openerp/addons/base/res/res_lang.py'
  --- openerp/addons/base/res/res_lang.py	2012-01-24 13:17:05 +0000
  +++ openerp/addons/base/res/res_lang.py	2012-03-27 10:06:16 +0000
  @@ -214,11 +214,6 @@
                   parts[0], seps = intersperse(parts[0], eval_lang_grouping, thousands_sep)
   
               formatted = decimal_point.join(parts)
  -            while seps:
  -                sp = formatted.find(' ')
  -                if sp == -1: break
  -                formatted = formatted[:sp] + formatted[sp+1:]
  -                seps -= 1
           elif percent[-1] in 'diu':
               if grouping:
                   formatted = intersperse(formatted, eval_lang_grouping, thousands_sep)[0]

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-server/+bug/966083/+subscriptions


Follow ups

References