[Zope-Checkins] CVS: Zope/lib/python/Shared/DC/ZRDB - Aqueduct.py:1.49.4.3

[Andreas Jung]

Update of /cvs-repository/Zope/lib/python/Shared/DC/ZRDB
In directory cvs.zope.org:/tmp/cvs-serv32146

Modified Files:
      Tag: Zope-2_4-branch
	Aqueduct.py 
Log Message:
Fixed strange behaviour of _argdata() where ZSQL method call with
optional parameters never worked as supposed to work.


=== Zope/lib/python/Shared/DC/ZRDB/Aqueduct.py 1.49.4.2 => 1.49.4.3 ===
                         except: missing.append(name)
                     
-        if missing:
-            raise self.MissingArgumentError,  \
-                "The following arguments were omitted " \
-                " from the ZSQL method call: %s" % str(missing)
+        # Note: the code above tries to check if an argument of the
+        # ZSQL method above has the "optional" flag set (in case the
+        # argument is omitted from the ZSQL function call). But there
+        # is neither corresponding code inside the parse() function to 
+        # check for the "optional" parameter nor any documentation. 
+        # So we omit the check for the optional parameter. There will
+        # be probably no code break but there will be hopefully more code
+        # to work as supposed to work.
 
+#        if missing:
+#            raise self.MissingArgumentError,  \
+#                "The following arguments were omitted " \
+#                " from the ZSQL method call: %s" % str(missing)
+#
         return r
 
     _col=None