From: David T. <no...@op...> - 2007-10-03 20:45:48
|
On Wed, 2007-10-03 at 11:22 +0200, Andres Freund wrote: > Hi, > > On Tuesday 02 October 2007, Oleg Broytmann wrote in "Re: [SQLObject] SQLObject > 0.9.2b1": > > Hello! > > > > On Tue, Oct 02, 2007 at 09:34:08AM +0200, Markus Gritsch wrote: > > > the new version 0.9.2b1 added a try except block around the conversion > > > to unicode of the query in mysqlconnection.py: > > > > > > if self.need_unicode and not isinstance(query, unicode): > > > try: > > > query = unicode(query, self.encoding) > > > except UnicodeError: > > > pass > > Finally David Turner dived into the problem and, using test_blob.py and > > test_unicode.py tests, fixed the problem; the try/except is the part of the > > solution. > Perhaps it would be nice to include that description or the one of David in a > comment? Those statements surely raise a warning light in the head of many > people... I was thinking that it might make more sense to just remove that whole section. I understand that whoever wrote that code thought it was necessary to convert to Unicode, but I think it is not. Can you comment on this idea? |