Skip Menu |

This queue is for tickets about the Class-DBI-Sybase CPAN distribution.

Report information
The Basics
Id: 38425
Status: new
Priority: 0/
Queue: Class-DBI-Sybase

People
Owner: theothermike [...] gmail.com
Requestors: mbarry [...] cos.com
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.5
Fixed in: (no value)



Subject: update TEXT columns to blank value
updates will fail if you try to set a TEXT column to a blank value. Looking at the _column_placeholder method, I believe the check should be if( defined($data) ... not if( $data ... patch is attached.
Subject: cdbi_sybase.patch
*** src/Sybase.pm 2008-08-12 11:59:59.000000000 -0400 --- patch/Sybase.pm 2008-08-12 12:00:29.000000000 -0400 *************** *** 141,147 **** my @text_columns = $self->columns('TEXT'); # if its a text column, we need to $dbh -> quote() it, rather than using a placeholder, limitation of Sybase TDS libraries ! if ( $data && grep { $_ eq $column } @text_columns ) { return $self->db_Main->quote($data); } --- 141,147 ---- my @text_columns = $self->columns('TEXT'); # if its a text column, we need to $dbh -> quote() it, rather than using a placeholder, limitation of Sybase TDS libraries ! if ( defined($data) && grep { $_ eq $column } @text_columns ) { return $self->db_Main->quote($data); }