Merge lp:~fabien-morin/unifield-server/fm-us-3090 into lp:unifield-server

Proposed by jftempo
Status: Merged
Merged at revision: 4451
Proposed branch: lp:~fabien-morin/unifield-server/fm-us-3090
Merge into: lp:unifield-server
Diff against target: 76 lines (+15/-2)
4 files modified
bin/addons/sync_client/sync_client.py (+5/-0)
bin/addons/sync_server/data/audittrail_sync_server.yml (+1/-1)
bin/addons/sync_server/sync_server.py (+6/-0)
bin/addons/update_server/entity.xml (+3/-1)
To merge this branch: bzr merge lp:~fabien-morin/unifield-server/fm-us-3090
Reviewer Review Type Date Requested Status
UniField Reviewer Team Pending
Review via email: mp+326741@code.launchpad.net
To post a comment you must log in.
Revision history for this message
jftempo (jfb-tempo-consulting) :
Revision history for this message
Fabien MORIN (fabien-morin) wrote :

On mardi 18 juillet 2017 07:06:58 CEST you wrote:
> search is not needed here, the job is done by check_validated
>

Thanks, I removed it.

--
Fabien MORIN
TeMPO Consulting
20, avenue de la Paix
67000 Strasbourg
France

http://www.tempo-consulting.fr
Tel : +33 3 88 56 82 16
Fax : +33 9 70 63 35 46

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/addons/sync_client/sync_client.py'
2--- bin/addons/sync_client/sync_client.py 2017-05-08 09:51:06 +0000
3+++ bin/addons/sync_client/sync_client.py 2017-07-18 07:49:42 +0000
4@@ -295,6 +295,11 @@
5 entity = self.get_entity(cr, uid, context=context)
6 proxy = self.pool.get("sync.client.sync_server_connection").get_connection(cr, uid, "sync.server.entity")
7 proxy.end_synchronization(entity.identifier, self._hardware_id)
8+ cr.execute('SHOW server_version')
9+ result = cr.fetchone()
10+ pg_version = result and result[0] or 'pgversion not found'
11+ proxy.set_pg_version(entity.identifier, self._hardware_id,
12+ pg_version)
13 except SkipStep:
14 # res failed but without exception
15 assert is_step, "Cannot have a SkipTest error outside a sync step process!"
16
17=== modified file 'bin/addons/sync_server/data/audittrail_sync_server.yml'
18--- bin/addons/sync_server/data/audittrail_sync_server.yml 2016-11-09 16:33:24 +0000
19+++ bin/addons/sync_server/data/audittrail_sync_server.yml 2017-07-18 07:49:42 +0000
20@@ -56,7 +56,7 @@
21 object_ids = self.pool.get('ir.model').search(cr, uid, [('model', '=', 'sync.server.entity')], context=context)
22 rule_id = self.search(cr, uid, [('name', '=', name)], context=context)
23 if object_ids:
24- fields = ['name', 'email', 'state', 'children_ids', 'parent_id', 'hardware_id', 'identifier', 'email', 'version_id']
25+ fields = ['name', 'email', 'state', 'children_ids', 'parent_id', 'hardware_id', 'identifier', 'email', 'version_id', 'pgversion']
26 fields_ids = self.pool.get('ir.model.fields').search(cr, uid, [('model', '=' ,'sync.server.entity'), ('name', 'in', fields)], context=context)
27 vals = {'name': name,
28 'object_id': object_ids[0],
29
30=== modified file 'bin/addons/sync_server/sync_server.py'
31--- bin/addons/sync_server/sync_server.py 2017-02-24 14:15:56 +0000
32+++ bin/addons/sync_server/sync_server.py 2017-07-18 07:49:42 +0000
33@@ -230,6 +230,7 @@
34 'mission': fields.char('Mission', size=64),
35 'latitude': fields.float('Latitude',digits=(16,6)),
36 'longitude': fields.float('Longitude', digits=(16,6)),
37+ 'pgversion': fields.char('Postgres Version', size=64),
38 }
39 _defaults = {
40 'version': lambda *a: 0,
41@@ -479,6 +480,11 @@
42 self._send_invalidation_email(cr, uid, entity, ids_to_validate, context=context)
43 return (True, "Instance %s are now invalidated" % ", ".join(uuid_list))
44
45+ @check_validated
46+ def set_pg_version(self, cr, uid, entity, pg_version, context=None):
47+ self.write(cr, 1, entity.id, {'pgversion': pg_version}, context=context)
48+ return True
49+
50 def validate_action(self, cr, uid, ids, context=None):
51 if not context:
52 context = {}
53
54=== modified file 'bin/addons/update_server/entity.xml'
55--- bin/addons/update_server/entity.xml 2012-07-09 16:04:00 +0000
56+++ bin/addons/update_server/entity.xml 2017-07-18 07:49:42 +0000
57@@ -10,6 +10,7 @@
58 <field name="arch" type="xml">
59 <field name="identifier" position="after">
60 <field name="version_id" ></field>
61+ <field name="pgversion" ></field>
62 </field>
63 </field>
64 </record>
65@@ -22,8 +23,9 @@
66 <field name="arch" type="xml">
67 <field name="state" position="after">
68 <field name="version_id" ></field>
69+ <field name="pgversion" ></field>
70 </field>
71 </field>
72 </record>
73 </data>
74- </openerp>
75\ No newline at end of file
76+ </openerp>

Subscribers

People subscribed via source and target branches