← Back to team overview

openerp-india team mailing list archive

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

 

@raphael alla

This bug report is very interesting... thanks for investing this problem
raphael !

I have set space as decimal separator for res_lang fr_FR (via SQL,
because if you do it with the Gtk client it removes the trailing
spaces). I have applied your patch on my OpenERP server 6.1. But it
doesn't display the space as decimal separator in the Gtk client nor on
the default invoice report.

Raphael, could you tell us more who we can make your patch work ? I
would love to help you on this bug report.

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


Follow ups

References