Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
I think is good to have exclude functionality for product which we don't want under promotion.
Comment | File | Size | Author |
---|---|---|---|
#7 | inline_conditions-add_new_logical_operator_not-2133189-7.patch | 7.11 KB | jkuma |
#6 | 2133189-6.patch | 6.66 KB | roderik |
#2 | inline_conditions-add_new_logical_operator_not-2133189-2.patch | 3.38 KB | jkuma |
Comments
Comment #1
jkuma CreditAttribution: jkuma commentedThanks for your idea, I have already merged the idea of logical operators (OR and AND) for inline conditions. So I think I can add a new logical operator such as NOT to cover your needs. Let me move this feature request to the correct place.
Comment #2
jkuma CreditAttribution: jkuma commentedHere is my first attempt to add this new logical operator to inline conditions.
Comment #3
jkuma CreditAttribution: jkuma commentedI've tested in many different configuration and it seems to work fine to me.
Comment #4
jkuma CreditAttribution: jkuma commentedMerged in the dev branch. commit
Comment #6
roderikReopening issue, as it's not too old yet.
This way of adding NOT doesn't really work, because NOT applies to a single condition. Right now:
The attached patch implements NOT as a separate control (checkbox) for each condition.
Remarks:
Comment #7
jkuma CreditAttribution: jkuma commentedHello Roderik,
Thanks for re-opening this issue, your approach is better than mine, I've just added a CSS rule and i've fixed up the default value of the "negate" form item.
Comment #8
jkuma CreditAttribution: jkuma commentedMerged in the dev branch.