Closed
Bug 578240
Opened 14 years ago
Closed 14 years ago
Can't use "owner" to search for "assigned_to" in Quicksearch anymore
Categories
(Bugzilla :: Query/Bug List, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 3.6
People
(Reporter: stechz, Assigned: reed)
References
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
Comment 1•14 years ago
|
||
owner is a shortcut for owner_idle_time (i.e. time since assignee touched), not a shortcut for assignee. What you want is either: assignee:dolske, or @dolske, which is even shorter.
Comment 2•14 years ago
|
||
Indeed, there is no shortcut for owner. If there ever was, it's possible that I removed it as part of the 3.6 update. However, there are full docs on what field names are available, I believe, using the question mark link next to the quicksearch box.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → INVALID
Comment 4•14 years ago
|
||
"Internal server error" is not a reasonable error message.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Comment 5•14 years ago
|
||
By the way, did "owner:" used to work? It wouldn't be too hard to add back. But I agree with Jesse, we should not be sending a 500 for this.
OS: Mac OS X → All
Hardware: x86 → All
Summary: Internal error while quicksearching for owner → Quicksearch throws an "Internal Error" if you use an invalid field name
Target Milestone: --- → Bugzilla 3.6
Updated•14 years ago
|
Flags: blocking3.6.2+
Assignee | ||
Updated•14 years ago
|
Assignee: general → query-and-buglist
Component: Bugzilla-General → Query/Bug List
Version: unspecified → 3.6.1
Comment 6•14 years ago
|
||
Yes, "owner:" was an alias/shortcut for "assignee:" in previous versions.
Comment 7•14 years ago
|
||
(In reply to comment #5)
> By the way, did "owner:" used to work? It wouldn't be too hard to add back.
It was working till Bugzilla 3.4 included, but we stopped documenting and officially supporting owner: since Bugzilla 2.20, see bug 76507 (so Bugzilla 2.18 was the last release where owner: was documented). This gave 5 years to people to update their queries. IMO, there is no reason to bring it back.
Status: REOPENED → NEW
Comment 8•14 years ago
|
||
It's fewer characters. People like me don't "update their queries", they type new Bugzilla searches many times per day.
Assignee | ||
Comment 9•14 years ago
|
||
It costs us nothing to support it again, so I don't see why we shouldn't.
Assignee: query-and-buglist → reed
Status: NEW → ASSIGNED
Attachment #457882 -
Flags: review?(mkanat)
Assignee | ||
Comment 10•14 years ago
|
||
er, this doesn't actually fix this bug. I guess I should file a separate bug for owner:.
Assignee: reed → query-and-buglist
Status: ASSIGNED → NEW
Comment 11•14 years ago
|
||
FYI, "assi" will also work now. Any abbreviation is accepted that uniquely matches a field name.
Comment 12•14 years ago
|
||
Comment on attachment 457882 [details] [diff] [review]
patch - v1
Sorry, but no. Now you break owner as a shortcut for owner_idle_time. We removed all special shortcuts on purpose. I would be fine restoring owner if it didn't match any other field name, but that's not the case here.
Attachment #457882 -
Flags: review?(mkanat) → review-
Updated•14 years ago
|
Summary: Quicksearch throws an "Internal Error" if you use an invalid field name → Quicksearch throws an "Internal Error" if you use a string for the owner_idle_time field
Updated•14 years ago
|
Summary: Quicksearch throws an "Internal Error" if you use a string for the owner_idle_time field → Can't use "owner" to search for "assigned_to" in Quicksearch anymore
Comment 13•14 years ago
|
||
Comment on attachment 457882 [details] [diff] [review]
patch - v1
I've changed my mind, I'm going to allow this. I don't see any reason not to allow "owner".
The actual ISE is about putting a non-date value into the owner_idle_time field, which isn't a bug I want to fix on 3.6, and is already fixed on 4.0.
(Sorry about overriding your r- LpSolit, but I think it's OK to allow "owner" as an alias--I don't think it's really that important to search owner_idle_time.)
Attachment #457882 -
Flags: review+
Updated•14 years ago
|
Assignee: query-and-buglist → reed
Flags: approval4.0+
Flags: approval3.6+
Flags: approval+
Updated•14 years ago
|
Severity: normal → minor
Comment 14•14 years ago
|
||
Also, I didn't actually remove all special shortcuts--there are still a lot of them, actually.
Updated•14 years ago
|
Flags: approval?
Flags: approval4.0?
Flags: approval4.0+
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Comment 15•14 years ago
|
||
hmm.... what's the use case for owner_idle_time? My initial thinking on this is that's sort of an obscure field, and would probably be sufficient to let people go to the advanced search to get that.
On the other hand, I suppose the folks most interested in searching that field would probably be management types who may find the advanced search a bit daunting. I'd see no issue with making a different shortcut for owner_idle_time (like "oit" or just spelling it all the way out or something).
Comment 16•14 years ago
|
||
I guess more to the point, being able to search for owner from quicksearch seems pretty useful to me, and it ought to work.
Comment 17•14 years ago
|
||
Comment on attachment 457882 [details] [diff] [review]
patch - v1
So Dave has spoken.
Attachment #457882 -
Flags: review-
Updated•14 years ago
|
Flags: approval?
Flags: approval4.0?
Flags: approval4.0+
Flags: approval3.6?
Flags: approval3.6+
Flags: approval+
Assignee | ||
Comment 18•14 years ago
|
||
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Search/Quicksearch.pm
Committed revision 7398.
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/4.0/
modified Bugzilla/Search/Quicksearch.pm
Committed revision 7342.
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/
modified Bugzilla/Search/Quicksearch.pm
Committed revision 7145.
Status: NEW → RESOLVED
Closed: 14 years ago → 14 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•14 years ago
|
Blocks: bmo-regressions-1007
Keywords: regression
You need to log in
before you can comment on or make changes to this bug.
Description
•