Bug #89601 for SQL-Abstract: It's not clear how to wrap an arbitrary expression with "NOT (...)"

Fri Oct 18 05:45:29 2013 TIMB [...] cpan.org - Ticket created

Subject:

It's not clear how to wrap an arbitrary expression with "NOT (...)"

Given an SQLA data structure representing a complex expression, I want to express that the expression should not be true. It's not clear how to do that from the docs (assuming it can be done - I'm still experimenting with -not_bool).

Fri Oct 18 06:02:33 2013 TIMB [...] cpan.org - Correspondence added

It turned out very simple of course: perl -MSQL::Abstract -we 'warn SQL::Abstract->new->where({ -not_bool => { x => "", y => ""} })' WHERE ( (NOT ( f = ? AND y = ? )) ) at -e line 1. So the point of this ticket is that the "Unary operators: bool" section of the docs only gives examples of applying -bool and -not_bool to simple boolean fields. I suggest changing the second example to include a more complex expression. Something like: my %where = ( -and => [ -bool => 'one', -bool => 'two', -not_bool => 'three', -not_bool => { priority => [ { '=', 2 }, { '>', 5 } ] }, ], );

Wed Dec 18 08:53:52 2013 ribasushi [...] leporine.io - Correspondence added

On Fri Oct 18 06:02:33 2013, TIMB wrote: Show quoted text

> It turned out very simple of course: > > perl -MSQL::Abstract -we 'warn SQL::Abstract->new->where({ -not_bool > => { x => "", y => ""} })' > WHERE ( (NOT ( f = ? AND y = ? )) ) at -e line 1. > > So the point of this ticket is that the "Unary operators: bool" > section of the docs only gives examples of applying -bool and > -not_bool to simple boolean fields. > > I suggest changing the second example to include a more complex > expression. Something like: > > my %where = ( > -and => [ > -bool => 'one', > -bool => 'two', > -not_bool => 'three', > -not_bool => { priority => [ { '=', 2 }, { '>', 5 } ] }, > ], > );

Good idea, patched as per: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=dbsrgits/SQL-Abstract.git;a=commitdiff;h=23401b8 Thanks!

Wed Dec 18 08:53:54 2013 The RT System itself - Status changed from 'new' to 'open'

Wed Dec 18 08:53:55 2013 ribasushi [...] leporine.io - Status changed from 'open' to 'patched'

Sat Dec 28 15:33:57 2013 frioux [...] gmail.com - Correspondence added

On Wed Dec 18 08:53:52 2013, RIBASUSHI wrote: Show quoted text

> On Fri Oct 18 06:02:33 2013, TIMB wrote:

> > It turned out very simple of course: > > > > perl -MSQL::Abstract -we 'warn SQL::Abstract->new->where({ -not_bool > > => { x => "", y => ""} })' > > WHERE ( (NOT ( f = ? AND y = ? )) ) at -e line 1. > > > > So the point of this ticket is that the "Unary operators: bool" > > section of the docs only gives examples of applying -bool and > > -not_bool to simple boolean fields. > > > > I suggest changing the second example to include a more complex > > expression. Something like: > > > > my %where = ( > > -and => [ > > -bool => 'one', > > -bool => 'two', > > -not_bool => 'three', > > -not_bool => { priority => [ { '=', 2 }, { '>', 5 } ] }, > > ], > > );

> > Good idea, patched as per: > http://git.shadowcat.co.uk/gitweb/gitweb.cgi?p=dbsrgits/SQL- > Abstract.git;a=commitdiff;h=23401b8 > > Thanks!

Sat Dec 28 15:33:59 2013 frioux [...] gmail.com - Status changed from 'patched' to 'resolved'