Hello! And now a thing I want to discuss.

On Wed, Sep 27, 2006 at 09:44:51AM -0400, Nathan Edwards wrote:
> >>>      ProgrammingError: (1064, "You have an error in your SQL syntax; 
> >>check the manual that corresponds to your MySQL server version for the 
> >>right syntax to use near 'NULL)\n)' at line 3")
> >
> >   MySQL doesn't allow NULLs in EnumCol. There is a patch in the
> >SF tracker:
> >http://sourceforge.net/tracker/index.php?func=detail&aid=1496014&group_id=74338&atid=540672
> >and it would be helpful if you review and comment it.
> 
> Works fine.

   But if the user really have asked for NULL in EnumCol - wouldn't it be
better to raise an exception instead of silently remove None's from the
list? It would be surprising for the user to not to see NULLs in the
database without any hint from the SQLObject.
   So I think the way to handle this is: fix the tests to not pass None's
for MySQL, but do not change the handling of EnumCol. What do you think?

Oleg.
-- 
     Oleg Broytmann            http://phd.pp.ru/            [EMAIL PROTECTED]
           Programmers don't die, they just GOSUB without RETURN.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
sqlobject-discuss mailing list
sqlobject-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sqlobject-discuss

Reply via email to