Merge lp:~julie-w/unifield-server/US-4169 into lp:unifield-server

[jftempo]

=== modified file 'bin/addons/account_hq_entries/hq_entries.py'
--- bin/addons/account_hq_entries/hq_entries.py	2018-02-26 09:49:40 +0000
+++ bin/addons/account_hq_entries/hq_entries.py	2018-04-17 09:12:01 +0000
@@ -260,9 +260,9 @@
 
         for line in self.browse(cr, uid, ids, context=context):
             res.add(line.id)
-            if line.is_original:
+            if line.is_original and line.split_ids:
                 add_split(line)
-            if line.is_split:
+            if line.is_split and line.original_id:
                 # add original one
                 res.add(line.original_id.id)
                 # then other split lines
@@ -282,7 +282,9 @@
         # Prepare some values
         res = set()
         for line in self.browse(cr, uid, ids, context=context):
-            if line.user_validated == False and (line.is_original or line.is_split):
+            line_original = line.is_original and line.split_ids
+            line_split = line.is_split and line.original_id
+            if not line.user_validated and (line_original or line_split):
                 # First add original and split linked lines
                 for el in self.get_linked_lines(cr, uid, [line.id]):
                     res.add(el)
@@ -466,8 +468,13 @@
 
         #US-921: Only save the user_validated value if the update comes from sync!
         if context.get('sync_update_execution', False):
-            if 'user_validated' in  vals:
-                return super(hq_entries, self).write(cr, uid, ids, {'user_validated': vals['user_validated']}, context)
+            sync_vals = {}
+            if 'user_validated' in vals:
+                sync_vals.update({'user_validated': vals['user_validated']})
+            if 'is_original' in vals:  # US-4169 also enable to sync the is_original tag
+                sync_vals.update({'is_original': vals['is_original']})
+            if sync_vals:
+                return super(hq_entries, self).write(cr, uid, ids, sync_vals, context)
             return True
 
         if 'account_id' in vals:
@@ -480,6 +487,7 @@
 
     def unlink(self, cr, uid, ids, context=None):
         """
+        At synchro. only delete the entries having the tag is_split (= sync of an unsplit done in coordo). Otherwise:
         Do not permit user to delete:
          - validated HQ entries
          - split entries
@@ -487,10 +495,18 @@
         """
         if isinstance(ids, (int, long)):
             ids = [ids]
+        if context is None:
+            context = {}
+        if context.get('sync_update_execution', False):
+            new_ids = []
+            for hq_entry in self.browse(cr, uid, ids, fields_to_fetch=['is_split'], context=context):
+                if hq_entry.is_split:
+                    new_ids.append(hq_entry.id)
+            ids = new_ids
         if not context.get('from', False) or context.get('from') != 'code' and ids:
             if self.search(cr, uid, [('id', 'in', ids), ('user_validated', '=', True)]):
                 raise osv.except_osv(_('Error'), _('You cannot delete validated HQ Entries lines!'))
-            if self.search(cr, uid, [('id', 'in', ids), ('is_split', '=', True)]):
+            if self.search(cr, uid, [('id', 'in', ids), ('is_split', '=', True)]) and not context.get('sync_update_execution'):
                 raise osv.except_osv(_('Error'), _('You cannot delete split entries!'))
             if self.search(cr, uid, [('id', 'in', ids), ('is_original', '=', True)]):
                 raise osv.except_osv(_('Error'), _('You cannot delete original entries!'))

=== modified file 'bin/addons/account_hq_entries/wizard/hq_entries_unsplit.py'
--- bin/addons/account_hq_entries/wizard/hq_entries_unsplit.py	2015-05-27 15:54:46 +0000
+++ bin/addons/account_hq_entries/wizard/hq_entries_unsplit.py	2018-04-17 09:12:01 +0000
@@ -61,10 +61,10 @@
             split_ids = []
             original_ids = []
             for line in wiz.process_ids:
-                if line.is_original:
+                if line.is_original and line.split_ids:
                     original_ids.append(line.id)
                     continue
-                if line.is_split:
+                if line.is_split and line.original_id:
                     split_ids.append(line.id)
                     continue
             # Process

=== modified file 'bin/addons/account_hq_entries/wizard/hq_entries_validation.py'
--- bin/addons/account_hq_entries/wizard/hq_entries_validation.py	2018-04-03 10:18:51 +0000
+++ bin/addons/account_hq_entries/wizard/hq_entries_validation.py	2018-04-17 09:12:01 +0000
@@ -261,10 +261,10 @@
         #+ original ones
         #+ split ones
         for line in lines:
-            if line.is_original:
+            if line.is_original and line.split_ids:
                 original_lines.add(line)
                 all_lines.add(line.id)
-            elif line.is_split:
+            elif line.is_split and line.original_id:
                 original_lines.add(line.original_id)
                 all_lines.add(line.original_id.id)
         # Create the original line as it is (and its reverse)
@@ -434,7 +434,7 @@
                     self.write(cr, uid, [wiz.id], {'running': False})
                     raise osv.except_osv(_('Warning'), _('Invalid analytic distribution!'))
                 # UTP-760: Do other modifications for split lines
-                if line.is_original or line.is_split:
+                if (line.is_original and line.split_ids) or (line.is_split and line.original_id):
                     split_change.append(line)
                     continue
                 if not line.user_validated:

=== modified file 'bin/addons/msf_sync_data_server/data/sync_server.sync_rule.csv'
--- bin/addons/msf_sync_data_server/data/sync_server.sync_rule.csv	2018-03-15 14:46:24 +0000
+++ bin/addons/msf_sync_data_server/data/sync_server.sync_rule.csv	2018-04-17 09:12:01 +0000
@@ -38,7 +38,8 @@
 msf_sync_data_server.fp_distribution_line_intermission,FALSE,TRUE,FALSE,FALSE,bidirectional,Bidirectional,"[('cost_center_id' , 'in' , ('account.analytic.account' , 'id' , [('code','=','cc-intermission')]))]","['amount', 'analytic_id/id', 'cost_center_id/id', 'currency_id/id', 'date', 'destination_id/id', 'distribution_id/id', 'name', 'percentage', 'source_date']",COORDINATIONS,funding.pool.distribution.line,,FP Distribution Line - Intermission ,Valid,,211
 msf_sync_data_server.free_1_distribution_line,TRUE,TRUE,TRUE,FALSE,bidirectional,Bidirectional,[],"['amount', 'analytic_id/id', 'currency_id/id', 'date', 'destination_id/id', 'distribution_id/id', 'name', 'percentage', 'source_date']",HQ + MISSION,free.1.distribution.line,,Free 1 Distribution Line,Valid,,220
 msf_sync_data_server.free_2_distribution_line,TRUE,TRUE,TRUE,FALSE,bidirectional,Bidirectional,[],"['amount', 'analytic_id/id', 'currency_id/id', 'date', 'destination_id/id', 'distribution_id/id', 'name', 'percentage', 'source_date']",HQ + MISSION,free.2.distribution.line,,Free 2 Distribution Line,Valid,,221
-msf_sync_data_server.hq_entries,TRUE,TRUE,FALSE,FALSE,bidirectional,Bidirectional-Private,[],"['account_id/id', 'account_id_first_value/id', 'amount', 'analytic_id/id', 'analytic_id_first_value/id', 'analytic_state', 'cost_center_id/id', 'cost_center_id_first_value/id', 'currency_id/id', 'date', 'destination_id/id', 'destination_id_first_value/id', 'document_date', 'free_1_id/id', 'free_2_id/id', 'name', 'partner_txt', 'period_id/id', 'ref', 'user_validated']",OC,hq.entries,cost_center_id,HQ Entries,Valid,,250
+msf_sync_data_server.hq_entries,TRUE,TRUE,FALSE,FALSE,bidirectional,Bidirectional-Private,[],"['account_id/id', 'account_id_first_value/id', 'amount', 'analytic_id/id', 'analytic_id_first_value/id', 'analytic_state', 'cost_center_id/id', 'cost_center_id_first_value/id', 'currency_id/id', 'date', 'destination_id/id', 'destination_id_first_value/id', 'document_date', 'free_1_id/id', 'free_2_id/id', 'name', 'partner_txt', 'period_id/id', 'ref', 'user_validated', 'is_split', 'is_original']",OC,hq.entries,cost_center_id,HQ Entries,Valid,,250
+msf_sync_data_server.hq_entries_deletion,TRUE,TRUE,TRUE,FALSE,bidirectional,Up,"[('id', '=', 0)]","['name']",OC,hq.entries,,HQ Entries deletion,Valid,,251
 msf_sync_data_server.gl_journal_non_registers,TRUE,TRUE,FALSE,TRUE,bidirectional,Up,"[('type', '!=', 'cash'),('type', '!=', 'cheque'),('type', '!=', 'bank')]","['bank_journal_id/id','analytic_journal_id/id', 'code', 'currency/id', 'default_credit_account_id/id', 'default_debit_account_id/id', 'name', 'type','instance_id/id']",OC,account.journal,,GL Journal (Non Registers),Valid,,260
 msf_sync_data_server.gl_journals_registers_hq_bank,FALSE,TRUE,FALSE,TRUE,bidirectional,Down,"[('type', '=', 'bank'),('instance_id.level', '=', 'section')]","['bank_journal_id/id','analytic_journal_id/id', 'code', 'currency/id', 'default_credit_account_id/id', 'default_debit_account_id/id', 'name', 'type','instance_id/id']",OC,account.journal,,GL Journals – Bank Registers (HQ),Valid,,261
 msf_sync_data_server.gl_journals_registers_coordo_bank,FALSE,TRUE,FALSE,TRUE,bidirectional,Down,"[('type', '=', 'bank'),('instance_id.level', '=', 'coordo')]","['bank_journal_id/id','analytic_journal_id/id', 'code', 'currency/id', 'default_credit_account_id/id', 'default_debit_account_id/id', 'name', 'type','instance_id/id']",MISSION,account.journal,,GL Journals – Bank Registers (Coordo),Valid,,262