c2c-oerpscenario team mailing list archive
-
c2c-oerpscenario team
-
Mailing list archive
-
Message #24229
[Bug 522192] Re: Incorrect amount_to_text_en.py
In my opinion this should have been set up a base module, with the
possibility of inheriting l10n localization modules for this purpose
--
You received this bug notification because you are a member of C2C
OERPScenario, which is subscribed to the OpenERP Project Group.
https://bugs.launchpad.net/bugs/522192
Title:
Incorrect amount_to_text_en.py
Status in OpenERP Server:
Confirmed
Bug description:
amount_to_text_en.py incorrect in structure. It shouldn't redefine so called "generic" functions and variables from amount_to_text.py, but should extend it in proper way - by using add_amount_to_text_function function.
To avoid confusion i propose to leave in amount_to_text.py "generic" functions only, and all localisation staff turn into amount_to_text_fr.py and so on.
Modules shouldn't direct use any "amount_to_text_XX" function, instead it should call generic amount_to_text.
Here is incorrect use from /account_voucher/report/report_voucher.py :
38 def convert(self,amount, cur):
39 amt_en = amount_to_text_en.amount_to_text(amount,'en',cur);
40 return amt_en
Follow ups