299211 - whine.pl fails if derive_groups is required

Status: RESOLVED FIXED

(Bugzilla :: Whining, defect)

Description

[:glob ✱]

whine.pl fails if derive_groups is required, reports

"Attempted to lock a table without releasing previous lock first."

the locked tables are

whine_schedules WRITE,
whine_events READ,
profiles READ,
groups READ,
user_group_map READ

and the requested locks are

profiles WRITE,
user_group_map WRITE,
group_group_map READ,
groups READ

Comment 1

[Max Kanat-Alexander]

The solution is probably to lock those tables inside of whine.pl, and tell
Bugzilla::User that you've already locked the tables, somehow. (There's a way to
do it, as I recall.)

Comment 2

[Marc Schumann [:Wurblzap]]

Attachment: Patch

Comment 3

[Frédéric Buclin]

Comment on attachment 188344 [details] [diff] [review]
Patch

r=LpSolit

Comment 4

[Frédéric Buclin]

Checking in whine.pl;
/cvsroot/mozilla/webtools/bugzilla/whine.pl,v  <--  whine.pl
new revision: 1.13; previous revision: 1.12
done
Checking in Bugzilla/Constants.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Constants.pm,v  <--  Constants.pm
new revision: 1.25; previous revision: 1.24
done
Checking in Bugzilla/Flag.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Flag.pm,v  <--  Flag.pm
new revision: 1.44; previous revision: 1.43
done

Comment 5

[:glob ✱]

*** Bug 302407 has been marked as a duplicate of this bug. ***