Closed
Bug 313020
Opened 19 years ago
Closed 19 years ago
Implement the ability to add individual bugs to saved searches
Categories
(Bugzilla :: Query/Bug List, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.22
People
(Reporter: LpSolit, Assigned: LpSolit)
References
(Blocks 1 open bug)
Details
Attachments
(2 files, 8 obsolete files)
13.37 KB,
patch
|
bugreport
:
review+
|
Details | Diff | Splinter Review |
9.69 KB,
patch
|
cso
:
review+
|
Details | Diff | Splinter Review |
The footer should allow you to add individual bugs to existing or new saved searches, for instance using a UI similar to http://landfill.bugzilla.org/qa220rc2/attachment.cgi?id=347 These saved searches would have a special status (i.e. needs a new column in the namedqueries table) to only accept a list of bug IDs. Of course, the drop-down menu would only shows these saved searches. They can then be used as normal saved searches, for instance for whining and can also be edited.
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Bugzilla 2.22
Assignee | ||
Comment 1•19 years ago
|
||
Attachment #200123 -
Flags: review?(bugreport)
Assignee | ||
Comment 2•19 years ago
|
||
minor update to prevent a crash when no query name is given
Assignee | ||
Updated•19 years ago
|
Attachment #200123 -
Attachment is obsolete: true
Attachment #200149 -
Flags: review?(bugreport)
Assignee | ||
Updated•19 years ago
|
Attachment #200123 -
Flags: review?(bugreport)
Comment 3•19 years ago
|
||
Comment on attachment 200149 [details] [diff] [review] patch, v1.1 cool nit.... >+ $new_query = "bug_id=" . join(',', keys %bug_ids); May as well sort the list. This should make it easier to manage when loading one of these searches into "edit this search" I dont think this wants to be required on every user's footer. Please add a user pref for the footer display. Also factor it out into a distinct template file that is called by Useful-links.html.tmpl if the user pref is enabled. That will make it merge with other skins much more easily. There are a whole bunch of RFEs I have for this, but they can be additional bugs. The first of those is I would like to have buglists displayed with checkboxes like the change-multiple form to add/remove bugs from the saved queries.
Attachment #200149 -
Flags: review?(bugreport) → review-
Assignee | ||
Comment 4•19 years ago
|
||
- bug IDs sorted before being stored; - separate global/per-bug-queries.html.tmpl template; - 'per-bug-queries' user pref added to enable/disable this feature.
Attachment #200149 -
Attachment is obsolete: true
Attachment #200343 -
Flags: review?(bugreport)
Comment 5•19 years ago
|
||
Bug 95722 seems to be a dupe of this one (read the description, not the summary).
Assignee | ||
Comment 6•19 years ago
|
||
Not exactly a dupe as the goal of my patch is to be able to add individual bugs to some given list of bugs, independently of any query result. Bug 95722 is the next step in the process, i.e. to be able to add some selected bug IDs from a bug list.
Blocks: 95722
Comment 7•19 years ago
|
||
Comment on attachment 200343 [details] [diff] [review] patch, v2 The more I play with this, the more RFEs I have. Let's start with this and start down the RFE path.
Attachment #200343 -
Flags: review?(bugreport) → review+
Comment 8•19 years ago
|
||
Comment on attachment 200343 [details] [diff] [review] patch, v2 One nit (on checkin)... Please check for the user pref in useful links before calling the template that makes the form. That way, it can be called from elsewhere (like saved searches) even if the pref for the footer is off.
Assignee | ||
Comment 9•19 years ago
|
||
Why this request? If the param is off, I see no reason to display it in some pages.
Flags: approval?
Comment 10•19 years ago
|
||
(In reply to comment #7) > (From update of attachment 200343 [details] [diff] [review] [edit]) > The more I play with this, the more RFEs I have. Let's start with this and > start down the RFE path. hear hear! (and hence my approval anyway, despite the reservations I had on IRC, as we can work on preference wording later)
Flags: approval? → approval+
Assignee | ||
Comment 11•19 years ago
|
||
Checking in buglist.cgi; /cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v <-- buglist.cgi new revision: 1.315; previous revision: 1.314 done Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.445; previous revision: 1.444 done Checking in Bugzilla/Constants.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/Constants.pm,v <-- Constants.pm new revision: 1.29; previous revision: 1.28 done Checking in Bugzilla/User.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v <-- User.pm new revision: 1.92; previous revision: 1.91 done Checking in Bugzilla/DB/Schema.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Schema.pm,v <-- Schema.pm new revision: 1.41; previous revision: 1.40 done Checking in skins/standard/global.css; /cvsroot/mozilla/webtools/bugzilla/skins/standard/global.css,v <-- global.css new revision: 1.16; previous revision: 1.15 done RCS file: /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/per-bug-queries.html.tmpl,v done Checking in template/en/default/global/per-bug-queries.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/per-bug-queries.html.tmpl,v <-- per-bug-queries.html.tmpl initial revision: 1.1 done Checking in template/en/default/global/setting-descs.none.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/setting-descs.none.tmpl,v <-- setting-descs.none.tmpl new revision: 1.6; previous revision: 1.5 done Checking in template/en/default/global/useful-links.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/useful-links.html.tmpl,v <-- useful-links.html.tmpl new revision: 1.43; previous revision: 1.42 done Checking in template/en/default/global/user-error.html.tmpl; /cvsroot/mozilla/webtools/bugzilla/template/en/default/global/user-error.html.tmpl,v <-- user-error.html.tmpl new revision: 1.139; previous revision: 1.138 done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•19 years ago
|
Flags: documentation?
Comment 12•19 years ago
|
||
Attachment #207895 -
Flags: review?(documentation)
Comment 13•19 years ago
|
||
Added to the Bugzilla 2.22 Release Notes in bug 322960. I think this should be "off" by default, in the preferences. We'll see, though, how people react.
Keywords: relnote
Comment 14•18 years ago
|
||
Comment on attachment 207895 [details] [diff] [review] docs patch v1 > On the bug list page, you'll see 'Remember search' button, you'll see _a_ > and subsequently text box. subsequently is a nice SAT vocab word, but it's not something one would normally use. instead "'Remember search' button' followed by a text box." > You can enter some text and press this button to save this condition "condition" doesn't match anything until this point, in technical terms, i'd use "query", since i've already said don't use query, how about "search's conditions" > so you can query the same condition easily. Don't use "query" :). "so you can easily search for the same conditions again. > You can delete this by clicking 'Forget Search' link or on your prefs page. You can _later_ delete /it/ /using the/ 'Forget Search' link or on your prefs page. -- I don't want to tell people who don't use "clicking" devices to click :) > As of Bugzilla 2.21.2, individual bugs can be saved and added as Saved Search. change "as Saved Search" to "to a Saved Search" But do you need the As of? isn't the documentation for a specific version, or isn't there some markup that tells the xml parser that we mean 2.21.2+?
Attachment #207895 -
Flags: review?(documentation) → review-
Comment 15•18 years ago
|
||
only for tip, for others should not include + <para> + As of Bugzilla 2.22, individual bugs can be saved and added + to a Saved Search. + </para>
Attachment #207895 -
Attachment is obsolete: true
Attachment #211923 -
Flags: review?(documentation)
Comment 16•18 years ago
|
||
Comment on attachment 211923 [details] [diff] [review] docs patch for tip v2 >Index: docs/xml/using.xml >+ <para> >+ As of Bugzilla 2.22, individual bugs can be saved and added >+ to a Saved Search. >+ </para> I was expecting this to actually include documentation of the feature - like how to use it etc.
Attachment #211923 -
Flags: review?(documentation) → review-
Comment 17•18 years ago
|
||
Attachment #211923 -
Attachment is obsolete: true
Attachment #223599 -
Flags: review?(documentation)
Comment 18•18 years ago
|
||
Comment on attachment 223599 [details] [diff] [review] docs patch for tip v3 Better :) >Index: docs/xml/using.xml >+ <section id="savedsearch"> >+ <title>Saved Search</title> >+ <para> >+ On the bug list page, you'll see a <quote>Remember search</quote> >+ button followed by a text box. You can enter some text and press >+ this button to save this search condition so you can easily >+ search the same condition again. search the same condition again -> repeat the search again. >+ You can later delete it using >+ the <quote>Forget Search</quote> link or on your prefs page. >+ </para> or on your prefs page -> or on the <quote>Saved Searches</quote> section of your Prefs page. >+ <para> >+ As of Bugzilla 2.22, individual bugs can be saved and added >+ to a Saved Search. If your >+ <quote>Add individual bugs to saved searches</quote> is >+ <quote>On</quote>, you'll see <quote>Add</quote> button and >+ followed by the button, 2 text box, and if you have this type >+ of the Saved Searches, select box will also appear on the footer. >+ To use this, enter bugs' number or aliases of the bugs in the >+ first text box, select the Saved Searches or enter the name >+ of the new Saved Searches in the second text box, and press >+ <quote>Add</quote> button. This doesn't make much sense to me... some general comments first: An admin can disable this facility, so it might not exist. My "Add individual bugs to saved searches" what? You'll see *an* Add button 2 text *boxes* I really don't like the description of where the feature is located and the layout of the feature. "To use this feature, you should enter either the number or aliases of the bug or bugs you wish to add to a sarch in the first text box. You should then either select the existing Saved Search you wish to add them to in the select box or type the name of a new Saved Search in the second text box and then press the Add button." >+ Note that you CANNOT add individual bugs to the normal type >+ of the Saved Searches like default query configured by the >+ <quote>mybugstemplate</quote> parameter. "Note that you cannot add individual bugs to Saved Searches created from the results of a query" or something would be better.
Attachment #223599 -
Flags: review?(documentation) → review-
Comment 19•18 years ago
|
||
Attachment #223599 -
Attachment is obsolete: true
Attachment #223609 -
Flags: review?(documentation)
Comment 20•18 years ago
|
||
Comment on attachment 223609 [details] [diff] [review] docs patch for tip v4 + aliases of the bug or bugs you wish to add to a sarch in the s/sarch/search/ + created from the results of a query A final period would probably be nice here.
Comment 21•18 years ago
|
||
Comment on attachment 223609 [details] [diff] [review] docs patch for tip v4 you'll see an <quote>Add</quote> button and + followed by the button, 2 text boxes, and if you have this type + of the Saved Searches, select box will also appear on the footer. This doesn't make much sense like Colin said. I think part of the problem is throwing too many "and"s in only one sentence. Maybe we should go along the path of breaking this up in 2 sentences or so. Nits from comment #20 would also be welcomed in a new patch.
Attachment #223609 -
Flags: review?(documentation) → review-
Assignee | ||
Comment 22•18 years ago
|
||
Attachment #223609 -
Attachment is obsolete: true
Attachment #246427 -
Flags: review?(documentation)
Assignee | ||
Comment 23•18 years ago
|
||
I forgot a </section> in v5
Attachment #246427 -
Attachment is obsolete: true
Attachment #246428 -
Flags: review?(documentation)
Attachment #246427 -
Flags: review?(documentation)
Assignee | ||
Comment 24•18 years ago
|
||
I forgot a <title>foo</title> after a <section>, and I also fixed some nits reported by Colin on IRC.
Attachment #246428 -
Attachment is obsolete: true
Attachment #246431 -
Flags: review?
Attachment #246428 -
Flags: review?(documentation)
Updated•18 years ago
|
Attachment #246431 -
Flags: review? → review+
Assignee | ||
Comment 25•18 years ago
|
||
tip: Checking in docs/xml/using.xml; /cvsroot/mozilla/webtools/bugzilla/docs/xml/using.xml,v <-- using.xml new revision: 1.56; previous revision: 1.55 done 2.22.1: Checking in docs/xml/using.xml; /cvsroot/mozilla/webtools/bugzilla/docs/xml/using.xml,v <-- using.xml new revision: 1.37.2.14; previous revision: 1.37.2.13 done
Flags: documentation?
Flags: documentation2.22+
Flags: documentation+
Comment 26•18 years ago
|
||
*** Bug 361899 has been marked as a duplicate of this bug. ***
You need to log in
before you can comment on or make changes to this bug.
Description
•