Merge lp:~therp-nl/openupgrade-server/6.1-reverse_model_type_check into lp:openupgrade-server/6.1

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 4005
Proposed branch: lp:~therp-nl/openupgrade-server/6.1-reverse_model_type_check
Merge into: lp:openupgrade-server/6.1
Diff against target: 17 lines (+5/-2)
1 file modified
openerp/modules/loading.py (+5/-2)
To merge this branch: bzr merge lp:~therp-nl/openupgrade-server/6.1-reverse_model_type_check
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Review via email: mp+176046@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'openerp/modules/loading.py'
--- openerp/modules/loading.py 2012-07-17 12:14:55 +0000
+++ openerp/modules/loading.py 2013-07-21 10:34:28 +0000
@@ -172,8 +172,11 @@
172 if not model._name: # new in 6.1172 if not model._name: # new in 6.1
173 return173 return
174174
175 # persistent models only175 # persistent models only. Note that transient models that
176 if isinstance(model, osv.orm.TransientModel):176 # inherit from persistent models cause those models to be
177 # an instance of TransientModel too (e.g. email.template)
178 # Therefore, check for a negation of Model instantion
179 if not isinstance(model, osv.orm.Model):
177 return180 return
178181
179 model_registry = local_registry.setdefault(182 model_registry = local_registry.setdefault(

Subscribers

People subscribed via source and target branches

to status/vote changes: