dabo Commit
Revision 4520
Date: 2008-09-10 08:35:51 -0700 (Wed, 10 Sep 2008)
Author: Johnf
Trac: http://svn.dabodev.com/trac/dabo/changeset/4520

Changed:
U   trunk/dabo/db/dCursorMixin.py

Log:
changed isinstance to issubclass to catch a bug reported by John Fabiani that 
came from using 
self.addField("apvend.cvendno as cvendor") in a bizobj class.
The fix came from Paul McNett

Diff:
Modified: trunk/dabo/db/dCursorMixin.py
===================================================================
--- trunk/dabo/db/dCursorMixin.py       2008-09-09 17:10:01 UTC (rev 4519)
+++ trunk/dabo/db/dCursorMixin.py       2008-09-10 15:35:51 UTC (rev 4520)
@@ -927,7 +927,7 @@
                                dtStrings = ("<type 'DateTime'>", "<type 
'Date'>", "<type 'datetime.datetime'>")
                                if str(fldType) in dtStrings and 
isinstance(val, basestring):
                                        ignore = True
-                               elif isinstance(fldType, basestring) and 
isinstance(val, basestring):
+                               elif issubclass(fldType, basestring) and 
isinstance(val, basestring):
                                        ignore = True
                                elif val is None or fldType is type(None):
                                        # Any field type can potentially hold 
None values (NULL). Ignore these.
@@ -1965,6 +1965,7 @@
                """ Set the where clause of the sql statement."""
                self.sqlManager._whereClause = 
self.sqlManager.BackendObject.setWhereClause(clause,
                                        autoQuote=self.AutoQuoteNames)
+               return
 
 
        def addWhere(self, exp, comp="and"):




_______________________________________________
Post Messages to: [email protected]
Subscription Maintenance: http://leafe.com/mailman/listinfo/dabo-dev
Searchable Archives: http://leafe.com/archives/search/dabo-dev
This message: http://leafe.com/archives/byMID/[EMAIL PROTECTED]

Reply via email to