amount_to_text_en.py incorrect in structure

Registered by Ivan Gudym

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 keep 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 example of 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

Blueprint information

Status:
Started
Approver:
None
Priority:
Undefined
Drafter:
None
Direction:
Needs approval
Assignee:
None
Definition:
Discussion
Series goal:
Proposed for trunk
Implementation:
Needs Code Review
Milestone target:
milestone icon 6.0
Started by
Ivan Gudym

Related branches

Sprints

Whiteboard

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.