303899 - Allow changing Status Whiteboard when when editing multiple bugs

Status: RESOLVED FIXED

(Bugzilla :: Creating/Changing Bugs, enhancement)

Description

[Matisse Enzer]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.8) Gecko/20050511 Firefox/1.0.4
Build Identifier: 

This is a Request for Enhancement.

I'm providing a patch against template/en/default/list/edit-multiple.html.tmpl
revision   1.17.2.4

This idea is to simple add  status_whiteboard   to the list of things you can
change when doing an edit-multiple.

Reproducible: Always

Steps to Reproduce:

Comment 1

[Matisse Enzer]

Attachment: Proposed patch to template/en/default/list/edit-multiple.html.tmpl

Adds Status Whiteboard to fields you can change when doing an edit-multiple.
Pathc was made against template/en/default/list/edit-multiple.html.tmpl
revision 1.17.2.4

Comment 2

[Max Kanat-Alexander]

Make sure you ask for review for your patch:

http://www.bugzilla.org/docs/contributor.html#review

Comment 3

[Matisse Enzer]

Thanbks, done - set review flag to ? for [e-mail]

Comment 4

[Joel Peshkin]

I've reviewed this and tested it and it looks OK, but I wonder if we should
provide this option all the time or only if the status whiteboard is displayed
in the column list.

Comment 5

[Matisse Enzer]

(In reply to comment #4)
> I wonder if we should
> provide this option all the time or only if the status whiteboard is displayed
> in the column list.

That's an interesting idea. It is different though from the behavior for
Product, Platform, Component, Milestone, etc. which all are changeable in
edit-multiple without having to be present in the column list.

Comment 6

[A. Karl Kornel]

Is this a duplicate of bug 193277, or perhaps the other way 'round?

Comment 7

[Matisse Enzer]

(In reply to comment #6)
> Is this a duplicate of bug 193277, or perhaps the other way 'round?

Yup, I think this one is a dup - so if someone wants to resolve this as a
duplicate that's fine with me.

Comment 8

[Jouni Heikniemi]

Comment on attachment 191970 [details] [diff] [review]
Proposed patch to template/en/default/list/edit-multiple.html.tmpl

r=jouni

FWIW, I don't think it's necessary to make the visibility of the field
dependent on the column settings. Although it does make sense from a certain
point of view, there's no particular need to be inconsistent here - the
situation is perhaps even worse with keywords.

Comment 9

[Jouni Heikniemi]

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

Comment 10

[Matisse Enzer]

So I see this has been assigned to me, which is fine, but I don't know what to
do next - do I have commit priveleges to the CVS repository? And if so, which
branch do I commit to?

Comment 11

[Dave Miller [:justdave]]

The assignment to you is for record-keeping at this point, because that way it
shows the author of the patch when you look up the bug.  Someone else will be
happy to check it in for you since you don't have check-in privs.

Comment 12

[Jouni Heikniemi]

(In reply to comment #10)
> So I see this has been assigned to me, which is fine, but I don't know what to
> do next - do I have commit priveleges to the CVS repository? And if so, which
> branch do I commit to?

The usual behavior is that you can ask the reviewer to check the patch in; in
this case I was going to, but you were fast enough to ask first. Fixed now
anyway. Thanks for the patch!

Checking in template/en/default/list/edit-multiple.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/edit-multiple.html.tmpl,v
 <--  edit-multiple.html.tmpl
new revision: 1.27; previous revision: 1.26
done

Comment 13

[Max Kanat-Alexander]

Added to the Bugzilla 2.22 Release Notes in bug 322960.