Merge lp:~openerp-dev/openobject-server/6.0-bug-732613-xrg into lp:openobject-server/6.0

Proposed by xrg
Status: Merged
Merged at revision: 3363
Proposed branch: lp:~openerp-dev/openobject-server/6.0-bug-732613-xrg
Merge into: lp:openobject-server/6.0
Diff against target: 12 lines (+1/-1)
1 file modified
bin/osv/orm.py (+1/-1)
To merge this branch: bzr merge lp:~openerp-dev/openobject-server/6.0-bug-732613-xrg
Reviewer Review Type Date Requested Status
Jay Vora (Serpent Consulting Services) (community) Approve
Anup(SerpentCS) (community) Approve
OpenERP buildbot (community) Approve
Review via email: mp+52981@code.launchpad.net
To post a comment you must log in.
Revision history for this message
OpenERP buildbot (openerp-buildbot) :
review: Approve
Revision history for this message
xrg (xrg) wrote :

On Friday 11 March 2011, you wrote:
> Review: Approve

Well, we still need some test for that case.

Revision history for this message
Anup(SerpentCS) (anup-serpent) wrote :

Hello,

    It is a typo correction and it is crashing the system so it needs to be fixed as soon as possible.

Thanks.

review: Approve
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bin/osv/orm.py'
2--- bin/osv/orm.py 2011-03-03 14:24:25 +0000
3+++ bin/osv/orm.py 2011-03-11 08:34:05 +0000
4@@ -3078,7 +3078,7 @@
5 for group in groups:
6 module = group.split(".")[0]
7 grp = group.split(".")[1]
8- cr.execute("select count(*) from res_groups_users_rel where gid IN (select res_id from ir_model_data where name=%s and module=%s and model=%s) and uid=%s" \
9+ cr.execute("select count(*) from res_groups_users_rel where gid IN (select res_id from ir_model_data where name=%s and module=%s and model=%s) and uid=%s", \
10 (grp, module, 'res.groups', user))
11 readonly = cr.fetchall()
12 if readonly[0][0] >= 1: