From: Alex M. <ale...@cl...> - 2003-10-25 18:55:45
|
On Wed, Oct 22, 2003 at 11:18:50AM -0700, Mike Schilli wrote: > Kevin Goess wrote on 10/21/2003, 10:39 PM: > > > Do you remember the rationale for returning a string instead of just > > undef? Maybe we should at least make that configurable? What do you > > think? > > Think the rationale was that if the configuration file specifies a > layout with an MDC field and it's not defined in the code, > 'uninitialized variable' warnings are going to be all over the place. If > it causes problems somewhere else, it's more consistent to just return > undef. Fixed and checked in ... But now it fails with the message : DBD::Pg::st execute failed: ERROR: pg_atoi: zero-length string at /usr/local/lib/perl5/site_perl/5.8.0/Log/Log4perl/Appender/DBI.pm line 96. When I look at the query using DBI->trace(2), I see that it actually tries to insert a zero-length string instead of the expected NULL value. I've tried to figure out where the undef value is changed to a zero-length string in the code, but didn't success for now. -- Alex Marandon CLARISYS Informatique http://clarisys.fr |