Closed
Bug 901589
Opened 12 years ago
Closed 12 years ago
isempty operator does not work for custom multiselect fields
Categories
(Bugzilla :: Query/Bug List, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 5.0
People
(Reporter: pami.ketolainen, Assigned: pami.ketolainen)
References
Details
Attachments
(1 file)
|
571 bytes,
patch
|
glob
:
review+
|
Details | Diff | Splinter Review |
The _multiselect_isempty() method in Search.pm ignores custom multiselect fields so the isempty operator does not work for those.
The attached patch adds an elsif step to process also custom multiselect fields and see also field, which works the same way.
Attachment #785826 -
Flags: review?(glob)
Comment 1•12 years ago
|
||
confirmed
Status: UNCONFIRMED → ASSIGNED
Depends on: 345194
Ever confirmed: true
Target Milestone: --- → Bugzilla 5.0
Version: unspecified → 4.5
Comment on attachment 785826 [details] [diff] [review]
custom_multiselect_isempty.diff
r=glob nice, thanks
Attachment #785826 -
Flags: review?(glob) → review+
Flags: approval4.4+
Flags: approval+
Target Milestone: Bugzilla 5.0 → Bugzilla 4.4
Comment 3•12 years ago
|
||
The isempty operator doesn't exist in 4.4 nor the code which this patch is about.
Flags: approval4.4+
Target Milestone: Bugzilla 4.4 → Bugzilla 5.0
Comment 4•12 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Search.pm
Committed revision 8726.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•