Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Unless I'm mistaken, the string filter operator 'Is not empty (NULL)' should read 'Is not empty (NOT NULL)'...
Made a patch if that is indeed the case.
Comment | File | Size | Author |
---|---|---|---|
#5 | views-422434-5.patch | 2.34 KB | BWPanda |
#2 | views_422434_2.patch | 734 bytes | BWPanda |
views_handler_filter_string.inc_.patch | 475 bytes | BWPanda | |
Comments
Comment #1
dawehneri think the NULL is the description for empty and not for not empty,
but i'm not native
Comment #2
BWPanda CreditAttribution: BWPanda commentedIn that case, maybe it should be 'Is not empty/NULL'... Patch attached.
Comment #3
dawehnerAs I said above NULL is just a description for EMPTY.
with / some people may see a difference in their php brain.
empty(0) <=> TRUE
0 === NULL <=> FALSE
But this is not php its sql :)
Comment #4
merlinofchaos CreditAttribution: merlinofchaos commentedI like (NOT NULL) but I just noticed that there are at least 4 occurances of the string that all need to be caught:
Comment #5
BWPanda CreditAttribution: BWPanda commentedHere's an updated patch for the latest 2.x-dev:
Comment #6
dawehnerI'm now convinced. This string makes more sense.
Comment #7
merlinofchaos CreditAttribution: merlinofchaos commentedCommitted to all 3 branches. Thanks!