[6.0 RC2] constraints can not be altered
Bug #700451 reported by
Ferdinand
This bug affects 6 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Odoo Server (MOVED TO GITHUB) |
Fix Released
|
Medium
|
OpenERP's Framework R&D |
Bug Description
pls see/try extra-trunk/
and see discussion here
https:/
may be because of product_product inheritance ?
Related branches
Changed in openobject-server: | |
assignee: | nobody → OpenERP's Framework R&D (openerp-dev-framework) |
importance: | Undecided → Medium |
status: | New → Confirmed |
To post a comment you must log in.
Reported by Dr. Ferdinand Gassauer:
"constraints can not be altered ..."
Indeed, when we define an expression like
class bar(osv.osv):
_inherit = 'bar.bar'
def _check_foo(self, cr, uid, ids, context):
return True
_constraints = [ (_check_foo, "Foo failed!", ['foo']) ]
... it means that _check_foo will be passed as an *object* to the model's
structure of _constraints. Therefore, it would be unequal and just append
the list of any existing constraints. So, an older (_check_foo, , ['foo'])
would always remain active using the previous code. This has to do with
the _check_foo being an unbound (ie. not inheritable) function.
Now, we check the /string name/ of the function, too. We say that if the
inherited class's constraint function has the same name "_check_foo", the
old ones shall be replaced.
Note: this MAY introduce unpredictable results, if several modules try
to override the same inherited constraint. There is no guaranteed order
of inheritance. Please avoid using this feature unless necessary.
---
bin/osv/osv.py | 7 ++++++-
1 files changed, 6 insertions(+), 1 deletions(-)
diff --git a/bin/osv/osv.py b/bin/osv/osv.py
exist = False
for c2 in range(len(new)):
#For _constraints, we should check field and methods as well new[c2] [0],'__ name__' , True) == \ c[0],'_ _name__ ', False)):
new[ c2] = c
exist = True
break
index 411dc0f..21a7e05 100644
--- a/bin/osv/osv.py
+++ b/bin/osv/osv.py
@@ -372,7 +372,12 @@ class osv(osv_base, orm.orm):
- if new[c2][2]==c[2] and new[c2][0]==c[0]:
+ if new[c2][2]==c[2] and (new[c2][0] == c[0] \
+ or getattr(
+ getattr(
+ # If new class defines a constraint with
+ # same function name, we let it override
+ # the old one.
--
1.7.1