Skip Menu |

This queue is for tickets about the SQL-Abstract CPAN distribution.

Report information
The Basics
Id: 58490
Status: resolved
Priority: 0/
Queue: SQL-Abstract
People
Owner: Nobody in particular
Requestors: TIMB [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 1.67_01

History Show all quoted text
  Thu Jun 17 11:06:18 2010 TIMB [...] cpan.org - Ticket created
Subject: The docs don't discuss undef/NULL sufficiently
Some people expect a \%where of { foo => undef } to match NULLs. Others are smarter. Neither group is catered for in the docs. Currently the only mention of NULL in the docs is in the Literal SQL section where it's casually used in an example without comment. The docs should point out the gotcha for the novice and discuss alternative approaches. You could also call this ticket a request for better handling of undef/NULL values in SQL::Abstract. Specifically some way to indicate that a particular clause should be "undef capable". See the NULL Value section of http://search.cpan.org/~timb/DBI-1.611/DBI.pm#Placeholders_and_Bind_Values
  Mon Jul 12 06:31:20 2010 ribasushi [...] leporine.io - Correspondence added
On Thu Jun 17 11:06:18 2010, TIMB wrote: Show quoted text
> Some people expect a \%where of { foo => undef } to match NULLs. > Others are smarter. Neither group is catered for in the docs. > > Currently the only mention of NULL in the docs is in the Literal SQL > section where it's casually used in an example without comment. > > The docs should point out the gotcha for the novice and discuss > alternative approaches. > > You could also call this ticket a request for better handling of > undef/NULL values in SQL::Abstract. > Specifically some way to indicate that a particular clause should be > "undef capable". > See the NULL Value section of http://search.cpan.org/~timb/DBI- > 1.611/DBI.pm#Placeholders_and_Bind_Values
Patched http://dev.catalystframework.org/svnweb/bast/revision?rev=9615
  Mon Jul 12 06:31:21 2010 The RT System itself - Status changed from 'new' to 'open'
  Mon Jul 12 06:31:21 2010 ribasushi [...] leporine.io - Status changed from 'open' to 'patched'
  Sun Sep 12 03:31:43 2010 ribasushi [...] leporine.io - Status changed from 'patched' to 'resolved'
  Sun Sep 12 03:31:45 2010 ribasushi [...] leporine.io - Fixed in 1.67_01 added