Merge lp:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1178696-l10n_ca_account_check_writing into lp:openerp-canada

[Joao Alfredo Gama Batista]

Force 'Canadian Dollars' as default currency for the canadian checks.

[lp:~savoirfairelinux-openerp/openerp-canada/7.0-bug-1178696-l10n_ca_account_check_writing]

12. By Joao Alfredo Gama Batista on 2013-05-10

[FIX] Don't remove 'Dollars' when printing

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

LGTM

Maybe an automated test would help?

review: Approve (code review, no test)

[Joao Alfredo Gama Batista (joao-gama)]

> LGTM
>
> Maybe an automated test would help?

It definitely would! ;)

=== modified file 'l10n_ca_account_check_writing/account_voucher.py'
--- l10n_ca_account_check_writing/account_voucher.py	2013-04-25 21:23:42 +0000
+++ l10n_ca_account_check_writing/account_voucher.py	2013-05-10 16:04:25 +0000
@@ -20,11 +20,33 @@
 ##############################################################################
 
 from openerp.osv import orm, fields
-
+from openerp.tools.amount_to_text_en import amount_to_text
 
 class account_voucher(orm.Model):
     _inherit = 'account.voucher'
 
+    def onchange_amount(self, cr, uid, ids, amount, rate, partner_id,
+                        journal_id, currency_id, ttype, date,
+                        payment_rate_currency_id, company_id, context=None):
+        """ Inherited - add amount_in_word and allow_check_writting in returned value dictionary """
+        if not context:
+            context = {}
+        default = super(account_voucher, self).onchange_amount(
+            cr, uid, ids, amount, rate, partner_id, journal_id, currency_id,
+            ttype, date, payment_rate_currency_id, company_id, context=context)
+        if 'value' in default:
+            amount = 'amount' in default['value'] and default['value']['amount'] or amount
+
+            #TODO : generic amount_to_text is not ready yet, otherwise language (and country) and currency can be passed
+            #amount_in_word = amount_to_text(amount, context=context)
+            amount_in_word = amount_to_text(amount, currency='Canadian Dollars')
+            default['value'].update({'amount_in_word':amount_in_word})
+            if journal_id:
+                allow_check_writing = self.pool.get('account.journal').browse(
+                    cr, uid, journal_id, context=context).allow_check_writing
+                default['value'].update({'allow_check': allow_check_writing})
+        return default
+
     def print_check(self, cr, uid, ids, context=None):
         if not ids:
             return  {}

=== modified file 'l10n_ca_account_check_writing/report/l10n_ca_check_print.py'
--- l10n_ca_account_check_writing/report/l10n_ca_check_print.py	2013-04-25 21:23:42 +0000
+++ l10n_ca_account_check_writing/report/l10n_ca_check_print.py	2013-05-10 16:04:25 +0000
@@ -35,8 +35,8 @@
             'fill_stars' : self.fill_stars,
         })
     def fill_stars(self, amount):
-        amount = amount.replace('Dollars','')
-        if len(amount) < 100:
+        #amount = amount.replace('Dollars','')
+        if len(amount) < 90:
             stars = 100 - len(amount)
             return ' '.join([amount,'*'*stars])