Merge lp:~therp-nl/ocb-addons/7.0-lp941817-autocompute_taxes_on_proforma into lp:ocb-addons

[Stefan Rijnhart (Opener)]

[FIX] lp:941817, taxes not computed automatically on proforma

Same as https://code.launchpad.net/~therp-nl/ocb-addons/6.1-lp941817-autocompute_taxes_on_proforma/+merge/159168, but for 7.0

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

LGTM

review: Approve (code review, no test)

[Holger Brunn (Therp) (hbrunn)]

review: Approve (code review, no test)

=== modified file 'account/account_invoice.py'
--- account/account_invoice.py	2013-04-04 15:59:27 +0000
+++ account/account_invoice.py	2013-04-18 18:36:32 +0000
@@ -1066,6 +1066,23 @@
                         (ref, move_id))
         return True
 
+    def action_proforma(self, cr, uid, ids, context=None):
+        """
+        Check if all taxes are present with the correct base amount
+        on creating a proforma invoice. This leaves room for manual
+        corrections of the tax amount.
+        """
+        if not ids:
+            return True
+        if isinstance(ids, (int, long)):
+            ids = [ids]
+        ait_obj = self.pool.get('account.invoice.tax')
+        for inv in self.browse(cr, uid, ids, context=context):
+            compute_taxes = ait_obj.compute(cr, uid, inv.id, context=context)
+            self.check_tax_lines(cr, uid, inv, compute_taxes, ait_obj)
+        return self.write(
+            cr, uid, ids, {'state': 'proforma2'}, context=context)
+
     def action_cancel(self, cr, uid, ids, context=None):
         if context is None:
             context = {}

=== modified file 'account/account_invoice_view.xml'
--- account/account_invoice_view.xml	2013-03-13 12:43:17 +0000
+++ account/account_invoice_view.xml	2013-04-18 18:36:32 +0000
@@ -214,7 +214,7 @@
                                 <field name="amount_untaxed" widget="monetary" options="{'currency_field': 'currency_id'}"/>
                                 <div>
                                     <label for="amount_tax"/>
-                                    <button name="button_reset_taxes" states="draft,proforma2"
+                                    <button name="button_reset_taxes" states="draft"
                                         string="(update)" class="oe_link oe_edit_only"
                                         type="object" help="Recompute taxes and total"/>
                                 </div>
@@ -369,7 +369,7 @@
                                 <field name="amount_untaxed" widget="monetary" options="{'currency_field': 'currency_id'}"/>
                                 <div>
                                     <label for="amount_tax"/>
-                                    <button name="button_reset_taxes" states="draft,proforma2"
+                                    <button name="button_reset_taxes" states="draft"
                                         string="(update)" class="oe_link oe_edit_only"
                                         type="object" help="Recompute taxes and total"/>
                                 </div>

=== modified file 'account/account_invoice_workflow.xml'
--- account/account_invoice_workflow.xml	2012-05-25 15:17:40 +0000
+++ account/account_invoice_workflow.xml	2013-04-18 18:36:32 +0000
@@ -17,7 +17,7 @@
         <record id="act_proforma2" model="workflow.activity">
             <field name="wkf_id" ref="wkf"/>
             <field name="name">proforma2</field>
-            <field name="action">write({'state':'proforma2'})</field>
+            <field name="action">action_proforma()</field>
             <field name="kind">function</field>
         </record>