Merge pull request #363 from akretion/10-no_auto_post

[10.0] Add 'post_move' option on account.payment.mode

account_payment_order/__manifest__.py

@@ -9,7 +9,7 @@
 
 {
     'name': 'Account Payment Order',
-    'version': '10.0.1.1.1',
+    'version': '10.0.1.1.2',
     'license': 'AGPL-3',
     'author': "ACSONE SA/NV, "
               "Therp BV, "

account_payment_order/models/account_payment_mode.py

@@ -73,6 +73,7 @@ class AccountPaymentMode(models.Model):
         ('date', 'One move per payment date'),
         ('line', 'One move per payment line'),
         ], string='Move Option', default='date')
+    post_move = fields.Boolean(string='Post Move', default=True)
 
     @api.multi
     @api.constrains(

account_payment_order/models/account_payment_order.py

@@ -433,6 +433,7 @@ class AccountPaymentOrder(models.Model):
         """
         self.ensure_one()
         am_obj = self.env['account.move']
+        post_move = self.payment_mode_id.post_move
         # prepare a dict "trfmoves" that can be used when
         # self.payment_mode_id.move_option = date or line
         # key = unique identifier (date or True or line.id)
@@ -459,4 +460,5 @@ class AccountPaymentOrder(models.Model):
             mvals['line_ids'].append((0, 0, trf_ml_vals))
             move = am_obj.create(mvals)
             blines.reconcile_payment_lines()
-            move.post()
+            if post_move:
+                move.post()

account_payment_order/views/account_payment_mode.xml

@@ -40,6 +40,7 @@
                     attrs="{'invisible': [('offsetting_account', '!=', 'transfer_account')], 'required': [('offsetting_account', '=', 'transfer_account')]}"/>
                 <field name="move_option"
                     attrs="{'invisible': [('generate_move', '=', False)], 'required': [('generate_move', '=', True)]}"/>
+                <field name="post_move"/>
             </group>
         </group>
     </field>