Closed Bug 1190601 Opened 4 years ago Closed 4 years ago

Should be able to edit comments made from editing a current attachment

Categories

(bugzilla.mozilla.org :: Extensions, defect)

Production
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dkl, Assigned: dkl)

Details

Attachments

(1 file)

Related to request from bug 1190501. I was unable to edit the comments content as it was a comment type == 6 and not 0 or 5 like the extension template looks for. I do not see a reason we cannot edit 6 as well.

Patch coming

dkl
Attached patch 1190601_1.patchSplinter Review
Attachment #8642669 - Flags: review?(glob)
Comment on attachment 8642669 [details] [diff] [review]
1190601_1.patch

Review of attachment 8642669 [details] [diff] [review]:
-----------------------------------------------------------------

r=glob, with fix-on-commit

::: extensions/EditComments/template/en/default/hook/bug/comments-a_comment-end.html.tmpl
@@ +8,4 @@
>  
>  [% IF Param('edit_comments_group')
>     && user.in_group(Param('edit_comments_group'))
> +   && (comment.type == 0 || comment.type == 5 || comment.type == 6)

change these to use the CMT_* constants instead of integers.
we should also allow editing of CMT_DUPE_OF
Attachment #8642669 - Flags: review?(glob) → review+
To ssh://gitolite3@git.mozilla.org/webtools/bmo/bugzilla.git
   0e478b8..9465a77  master -> master
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Component: Extensions: EditComments → Extensions
You need to log in before you can comment on or make changes to this bug.