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

Proposed by jftempo
Status: Merged
Merged at revision: 5369
Proposed branch: lp:~julie-w/unifield-server/US-5619
Merge into: lp:unifield-server
Diff against target: 84 lines (+46/-4)
2 files modified
bin/addons/analytic_distribution/account.py (+1/-0)
bin/addons/msf_audittrail/audittrail_invoice_data.yml (+45/-4)
To merge this branch: bzr merge lp:~julie-w/unifield-server/US-5619
Reviewer Review Type Date Requested Status
UniField Reviewer Team Pending
Review via email: mp+366902@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/addons/analytic_distribution/account.py'
2--- bin/addons/analytic_distribution/account.py 2018-09-18 12:25:20 +0000
3+++ bin/addons/analytic_distribution/account.py 2019-05-03 09:52:11 +0000
4@@ -31,6 +31,7 @@
5 _name = 'account.destination.link'
6 _description = 'Destination link between G/L and Analytic accounts'
7 _order = 'name, id'
8+ _trace = True
9
10 def _get_tuple_name(self, cr, uid, ids, name=False, args=False, context=None):
11 """
12
13=== modified file 'bin/addons/msf_audittrail/audittrail_invoice_data.yml'
14--- bin/addons/msf_audittrail/audittrail_invoice_data.yml 2017-01-25 14:42:56 +0000
15+++ bin/addons/msf_audittrail/audittrail_invoice_data.yml 2019-05-03 09:52:11 +0000
16@@ -256,23 +256,64 @@
17 rule_id = self.search(cr, uid, [('object_id', 'in', object_ids)], context=context)
18 if object_ids:
19 # Create the rule
20- fields = ['state', 'category', 'code', 'complete_name', 'cost_center_ids', 'date', 'date_start', 'name', 'parent_id']
21+ fields = ['state', 'category', 'code', 'complete_name', 'cost_center_ids', 'date', 'date_start', 'name', 'parent_id',
22+ 'type', 'for_fx_gain_loss', 'instance_id', 'tuple_destination_account_ids', 'description', 'destination_ids']
23
24 fields_ids = self.pool.get('ir.model.fields').search(cr, uid, [('model', '=' ,'account.analytic.account'), ('name', 'in', fields)], context=context)
25
26 vals = {
27 'name': name,
28 'object_id': object_ids[0],
29- 'log_write': False,
30+ 'log_write': True,
31+ 'log_unlink': True,
32+ 'log_create': True,
33+ 'field_ids': [(6, 0, fields_ids)],
34+ }
35+ if not rule_id:
36+ rule_id = self.create(cr, uid, vals, context=context)
37+ elif rule_id:
38+ self.write(cr, uid, rule_id, vals, context=context)
39+
40+ # Subscribe to the rule
41+ self.subscribe(cr, uid, rule_id)
42+-
43+ For "Destination link between G/L and Analytic accounts" (account.destination.link), track only the modification of the field "disabled"
44+-
45+ !python {model: audittrail.rule}: |
46+ name = 'Destinations Links'
47+ object_ids = self.pool.get('ir.model').search(cr, uid, [('model', '=', 'account.destination.link')], context=context)
48+ rule_id = self.search(cr, uid, [('object_id', 'in', object_ids)], context=context)
49+ if object_ids:
50+ # track only the field "disabled", at Destination level (i.e. on the account.analytic.account object)
51+ # cf. when an account is removed from a Dest., it is disabled but not unlinked (so still attached to the dest.)
52+ fields = ['disabled']
53+ fields_ids = self.pool.get('ir.model.fields').search(cr, uid, [('model', '=' , 'account.destination.link'), ('name', 'in', fields)], context=context)
54+ field_name = self.pool.get('ir.model.fields').search(cr, uid, [('model', '=', 'account.destination.link'), ('name', '=', 'account_id')], context=context)
55+ field_parent = self.pool.get('ir.model.fields').search(cr, uid, [('model', '=', 'account.destination.link'), ('name', '=', 'destination_id')], context=context)
56+
57+ name_id = False
58+ parent_id = False
59+
60+ if field_parent:
61+ parent_id = field_parent[0]
62+ if field_name:
63+ name_id = field_name[0]
64+
65+ vals = {
66+ 'name': name,
67+ 'object_id': object_ids[0],
68+ 'log_write': True,
69 'log_unlink': False,
70- 'log_create': True,
71+ 'log_create': False,
72 'field_ids': [(6, 0, fields_ids)],
73+ 'parent_field_id': parent_id,
74+ 'name_get_field_id': name_id,
75 }
76+
77 if not rule_id:
78 rule_id = self.create(cr, uid, vals, context=context)
79 elif rule_id:
80 self.write(cr, uid, rule_id, vals, context=context)
81-
82 # Subscribe to the rule
83 self.subscribe(cr, uid, rule_id)
84 -

Subscribers

People subscribed via source and target branches