284273 - Move productResponsibilities from editusers.cgi to Bugzilla::User.pm

Status: RESOLVED FIXED

(Bugzilla :: Administration, task, P3)

Description

[Marc Schumann [:Wurblzap]]

(In reply to bug 119485, comment #36)
> >+# Retrieve product responsibilities, usable for both display and verification.
> >+sub productResponsibilities {
> 
>   We should probably have this in Bugzilla::User, so it will be easier to
> update if we ever create more "responsibilities" for the user.

Comment 1

[Marc Schumann [:Wurblzap]]

Attachment: Patch

Very straightforward.

Comment 2

[Max Kanat-Alexander]

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

Yeah, very straightforward. :-)

The code is fine. But the POD docs need a description of the return format.

Also, we should really have a cache for this information just like we have for
all the other functions. ($self->{product_responsibilities})

Comment 3

[Marc Schumann [:Wurblzap]]

Attachment: Patch 2

o Has a cache now.
o Nice POD.
o Removed responsibility hash from confirm-delete interface because it's now
part
  of the user object.

Comment 4

[Max Kanat-Alexander]

Comment on attachment 187133 [details] [diff] [review]
Patch 2

Yep, this looks quite good on inspection. You've tested it?

Comment 5

[Marc Schumann [:Wurblzap]]

> You've tested it?

Yes, I tested

1 deleting a user without responsibilites
2 trying to delete a user
  2.1 with one responsibility
  2.2 being initial owner and initial QA contact in the same product
  2.3 being initial owner for several products, and initial QA contact for
      several products
3 trying to delete a user even though the confirm-delete template tries to keep
  me from doing it by not displaying the "delete" button

Comment 6

[Dave Miller [:justdave]]

under normal circumstances, I would say to hold this for 2.22, but since it's
blocking stuff that needs to go in 2.20, I guess it can go in :)

Comment 7

[Frédéric Buclin]

Checking in editusers.cgi;
/cvsroot/mozilla/webtools/bugzilla/editusers.cgi,v  <--  editusers.cgi
new revision: 1.90; previous revision: 1.89
done
Checking in Bugzilla/User.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v  <--  User.pm
new revision: 1.57; previous revision: 1.56
done
Checking in template/en/default/admin/users/confirm-delete.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/admin/users/confirm-delete.html.tmpl,v
 <--  confirm-delete.html.tmpl
new revision: 1.4; previous revision: 1.3
done