openerp-india team mailing list archive

["Amit Parik \(OpenERP\)" <amp@xxxxxxxxxxx>]

Hello Raphael alla,

Thanks for your response.

I have tested your scenario and I also faced the space are not working
on thousands separator as I mention in my comment#2 If we set a space as
a thousands separator then current behaviour is not printed the space as
a thousands separator but It's printed a comma instead of space.

You have suggested that in in French, space  are used as the thousands separator.
I have also checked this with your given patch but also it's not working as you mention. 
Indeed, as per the http://en.wikipedia.org/wiki/Decimal_mark link I also agree with you that we have to provided a space as a thousands separator.

That's why I am confirming this issue.

Thanks again!

** Changed in: openobject-server
   Importance: Undecided => Low

** Changed in: openobject-server
       Status: Incomplete => Confirmed

** Changed in: openobject-server
     Assignee: (unassigned) => OpenERP's Framework R&D (openerp-dev-framework)

-- 
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:
  Confirmed

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