Merge lp:~camptocamp/ocb-server/ocb-7.0-no-sql-constraint-abstract-model-1151703 into lp:ocb-server

Proposed by Alexandre Fayolle - camptocamp
Status: Merged
Merged at revision: 4956
Proposed branch: lp:~camptocamp/ocb-server/ocb-7.0-no-sql-constraint-abstract-model-1151703
Merge into: lp:ocb-server
Diff against target: 13 lines (+2/-1)
1 file modified
openerp/osv/orm.py (+2/-1)
To merge this branch: bzr merge lp:~camptocamp/ocb-server/ocb-7.0-no-sql-constraint-abstract-model-1151703
Reviewer Review Type Date Requested Status
Raphaël Valyi - http://www.akretion.com Approve
Stefan Rijnhart (Opener) Approve
Review via email: mp+157132@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) :
review: Approve
Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) :
review: Approve
Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

Just as a question.

This dont have a bug related?

I understand technically and perfect the change, but it should be cool if the explanation is related to a bug. What do you think?

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Hello Nhomar, it is related to lp:1151703, you can see it in the merge proposal informations ('Related bugs and blueprints')

Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

Ok cool sorry I didnt see it

Written from my android
On Apr 22, 2013 1:33 AM, "Guewen Baconnier @ Camptocamp" <
<email address hidden>> wrote:

> Hello Nhomar, it is related to lp:1151703, you can see it in the merge
> proposal informations ('Related bugs and blueprints')
> --
>
> https://code.launchpad.net/~camptocamp/ocb-server/ocb-7.0-no-sql-constraint-abstract-model-1151703/+merge/157132
> Your team OpenERP Community Backports Team is subscribed to branch
> lp:ocb-server.
>

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openerp/osv/orm.py'
2--- openerp/osv/orm.py 2013-03-27 13:52:28 +0000
3+++ openerp/osv/orm.py 2013-04-04 15:10:31 +0000
4@@ -3207,7 +3207,8 @@
5
6 cr.commit() # start a new transaction
7
8- self._add_sql_constraints(cr)
9+ if getattr(self, '_auto', True):
10+ self._add_sql_constraints(cr)
11
12 if create:
13 self._execute_sql(cr)

Subscribers

People subscribed via source and target branches

to status/vote changes: