Bug 1397035 - Add 'is empty' & 'is not empty' match types.
Summary: Add 'is empty' & 'is not empty' match types.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: Internal Tools
Version: 4.4
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 4.4
Assignee: Matt Tyson 🤬
QA Contact: tools-bugs
URL:
Whiteboard:
: 1413383 (view as bug list)
Depends On: 1406266
Blocks:
TreeView+ depends on / blocked
 
Reported: 2016-11-21 12:59 UTC by Romana Cadova
Modified: 2018-12-09 06:29 UTC (History)
8 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-01-30 01:00:45 UTC
Embargoed:


Attachments (Terms of Use)

Description Romana Cadova 2016-11-21 12:59:04 UTC
Getting:

field	value
cf_conditional_nak	Hardware Upstream Capacity Patch Design Reproducer


when creating clear_devel_conditional_nak rule in BRE.
https://partner-bugzilla.redhat.com/page.cgi?id=ruleengine/details/index.html&rule_name=clear_devel_conditional_nak

In step2 I set the condition:

Devel Conditional NAK is any of - and here I would need to check everything

also tried with "contains any of the strings" "Hardware, Upstream, ..."

In step3 I set:

Devel Conditional NAK - remove - check everything

also tried with
- unset- in step3

I was able to save this rule only with "Hardware" checked.

What am I doing wrong?
The same happens in prod bz too.

Thank you.

Comment 1 Matt Tyson 🤬 2016-11-23 04:13:37 UTC
The fix here is to add a new custom field match type

'is empty'
'is not empty'

This way you can create a new match type

'Devel Conditional NAK' 'is not empty'

On page three, you should be selecting 'Unset'.  A bug in the recursive action checker was causing this not to work.

Comment 2 Rony Gong 🔥 2016-12-20 09:16:24 UTC
'is empty'
'is not empty'
Added, and waiting for bug 1406266 to be fixed

Comment 3 Rony Gong 🔥 2016-12-21 04:24:41 UTC
Tested on QA environment(5.0.3-rh20)
Result: Pass
It works now.

Comment 4 Rony Gong 🔥 2017-01-09 09:37:03 UTC
Tested on QA environment(4.4.12065-1.el6)
Result: Pass
It works now.

Comment 5 Jeff Fearn 🐞 2017-01-15 22:31:13 UTC
*** Bug 1413383 has been marked as a duplicate of this bug. ***

Comment 6 Matt Tyson 🤬 2017-01-30 01:00:45 UTC
This change is now live. If there are any issues, do not reopen this bug.                                                                                                           
Instead, you should create a new bug and reference this bug.


Note You need to log in before you can comment on or make changes to this bug.