Skip to content

Filter by "IS NOT" NULL #1124

Closed
Closed

Description

@renschler

Details for the issue

This is an enhancement request. I want to be able to filter by IS NOT NULL for a column in the DB Browser. I understand the syntax would be "<>NULL". This does not seem to be working. This seems related to #588 but for "IS NOT" instead of "IS".

Many Thanks!
Patrick

Useful extra information

I'm opening this issue because:

  • DB4S is crashing
  • DB4S has a bug
  • DB4S needs a feature
  • DB4S has another problem

I'm using DB4S on:

  • Windows: ( version: ___ )
  • Linux: ( distro: ___ )
  • Mac OS: ( _version:10.12.6 )
  • Other: ___

I'm using DB4S version:

  • 3.10.0
  • 3.10.0-beta*
  • 3.9.1
  • Other: ___

I have also:

This issue is similar: #588
but it only covers the =NULL case, not <>NULL (afaik)

Activity

  1. justinclift commented on Sep 9, 2017

    @justinclift
    Member

    Ahhh yeah. Makes sense, and should be reasonably easy to implement too.

    Good thinking @renschler. 😄

  2. added a commit that references this issue on Sep 10, 2017
  3. MKleusberg commented on Sep 10, 2017

    @MKleusberg
    Member

    Oh, I wonder how we (or I) missed that while implementing #588 😉 But I've just implemented the '<>NULL' syntax and it should be working in the next nightly build. Can you download that tomorrow (from here) and give it a try, @renschler? 😄

  4. renschler commented on Sep 10, 2017

    @renschler
    Author
  5. renschler commented on Sep 11, 2017

    @renschler
    Author

    I played with the Sept-11 nightly build. Looks good to me 👍

  6. justinclift commented on Sep 11, 2017

    @justinclift
    Member

    Excellent. Thanks for testing and confirming @renschler. 😄

    Well done @MKleusberg. 😁

  7. chrisjlocke commented on Sep 11, 2017

    @chrisjlocke
    Member

    Just a thought, but <>null isn't valid SQL. I appreciate these are filters and not SQL, but having two conventions in one product seems a bit odd. "type values in here, but they're not valid here". It wouldn't be hard to 'convert' 'is null' into '=NULL', 'is not NULL' to '<>NULL', etc.

    On a side point, '=NULL' works, but '=null' doesn't - its case sensitive. Is this by design?
    NULLs aren't mentioned in the Wiki at all. I'll amend that tomorrow.

  8. MKleusberg commented on Sep 12, 2017

    @MKleusberg
    Member

    Yeah, the conventions are a bit odd. The idea was to make them as unambiguous as possible, i.e. make sure it's clear what's a special comparison operator and what is not. If we allowed 'is null' as a filter value and treated it like '=NULL' that would mean that it's not possible to filter for an 'is null' string in a text column (except by searching for '=is null' but that's not obvious either and other than 'is null' this is case-sensitive).

    So yes, it's all intentional and by design. But you can definitely question the design 😉 Not sure what the best compromise would be. Either way, you're right, it's probably a good idea to document this in the Wiki 😄

  9. added a commit that references this issue on Sep 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions