626658 - Add (take) link to bug edit page to allow quick assigning to the current user

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, enhancement)

Description

[David Lawrence [:dkl]]

Attachment: Patch to add (take) link to the assignee field (v1)

Add a new (take) link next to the (edit) link for the bug's assignee. This will allow the current user to quickly add their own email address to the assignee field using javascript functionality. If the user is already the assignee of the bug, the (take) like will not be displayed.

Patch attached.
Dave

Comment 1

[Max Kanat-Alexander]

Comment on attachment 504717 [details] [diff] [review]
Patch to add (take) link to the assignee field (v1)

Cool! Looks good, except that we shouldn't be doing things like "document.changeform.assigned_to". Instead this should probably have a separate function that it calls, that uses document.getElementById.

Comment 2

[David Lawrence [:dkl]]

Attachment: Patch to add (take) link to the assignee field (v2)

New patch with Max's suggested changes. I added a new argument to showEditableField that allows overriding and setting the input fields value.

Please review. If you think this would be better reviewed by Guy then feel free to change the requestee.

Dave

Comment 3

[Reed Loden [:reed]]

Comment on attachment 505101 [details] [diff] [review]
Patch to add (take) link to the assignee field (v2)

>- * var ContainerInputArray[0]: the conainer that will be hidden usually shows the (edit) text
>+ * var ContainerInputArray[0]: the conainer that will be hidden usually shows the (edit) or (take) text

container.

Comment 4

[Max Kanat-Alexander]

Comment on attachment 505101 [details] [diff] [review]
Patch to add (take) link to the assignee field (v2)

Looks good and works well. :-)

Comment 5

[David Lawrence [:dkl]]

Committing to: bzr+ssh://[e-mail]/bugzilla      /trunk/
modified js/field.js
modified template/en/default/bug/edit.html.tmpl                           Committed revision 7656.