Fix some broken logging.

Remove (broken and incorrect) optimisation added to set_inner
multilink handling.

Author: Richard Jones

roundup/backends/back_mysql.py

@@ -66,10 +66,10 @@ def db_nuke(config):
             # stupid MySQL bug requires us to drop all the tables first
             for table in tables:
                 command = 'DROP TABLE `%s`'%table[0]
-                self.log_debug(command)
+                logging.debug(command)
                 cursor.execute(command)
             command = "DROP DATABASE %s"%config.RDBMS_NAME
-            self.log_info(command)
+            logging.info(command)
             cursor.execute(command)
             conn.commit()
         conn.close()
@@ -83,7 +83,7 @@ def db_create(config):
     conn = MySQLdb.connect(**kwargs)
     cursor = conn.cursor()
     command = "CREATE DATABASE %s"%config.RDBMS_NAME
-    self.log_info(command)
+    logging.info(command)
     cursor.execute(command)
     conn.commit()
     conn.close()

roundup/backends/rdbms_common.py

@@ -1740,12 +1740,6 @@ def set_inner(self, nodeid, **propvalues):
 
                 # handle additions
                 for id in value:
-                    # If this node is in the cache, then we do not need to go to
-                    # the database.  (We don't consider this an LRU hit, though.)
-                    if self.cache.has_key((classname, nodeid)):
-                        # Return 1, not True, to match the type of the result of
-                        # the SQL operation below.
-                        return 1
                     if not self.db.getclass(link_class).hasnode(id):
                         raise IndexError, '%s has no node %s'%(link_class, id)
                     if id in l: