r3775 useSelectBlade(): sayonara, superceded by safer prepared version
authorDenis Ovsienko <infrastation@yandex.ru>
Wed, 9 Jun 2010 14:52:45 +0000 (14:52 +0000)
committerDenis Ovsienko <infrastation@yandex.ru>
Wed, 9 Jun 2010 14:52:45 +0000 (14:52 +0000)
useInsertBlade(): idem
useDeleteBlade(): idem

inc/database.php

index d41e22cc966321311dfff858fb0b6e37b6931c04..3233a540af8d7ea5b51126a7a4bc9730234e992d 100644 (file)
@@ -2137,24 +2137,6 @@ function commitUseupPort ($port_id = 0)
 // This is a swiss-knife blade to insert a record into a table.
 // The first argument is table name.
 // The second argument is an array of "name" => "value" pairs.
-// The function returns either TRUE or FALSE (we expect one row
-// to be inserted).
-function useInsertBlade ($tablename, $values)
-{
-       global $dbxlink;
-       $namelist = $valuelist = '';
-       foreach ($values as $name => $value)
-       {
-               $namelist = $namelist . ($namelist == '' ? "(${name}" : ", ${name}");
-               $valuelist = $valuelist . ($valuelist == '' ? "(${value}" : ", ${value}");
-       }
-       $query = "insert into ${tablename} ${namelist}) values ${valuelist})";
-       $result = $dbxlink->exec ($query);
-       if ($result != 1)
-               return FALSE;
-       return TRUE;
-}
-
 function usePreparedInsertBlade ($tablename, $columns)
 {
        global $dbxlink;
@@ -2189,15 +2171,8 @@ function usePreparedInsertBlade ($tablename, $columns)
        }
 }
 
-// This swiss-knife blade deletes one record from the specified table
-// using the specified key name and value.
-function useDeleteBlade ($tablename, $keyname, $keyvalue)
-{
-       global $dbxlink;
-       return 1 === $dbxlink->exec ("delete from ${tablename} where ${keyname}=${keyvalue} limit 1");
-}
-
-// Prepared version of above, but note the difference in returned value.
+// This swiss-knife blade deletes any number of records from the specified table
+// using the specified key names and values.
 function usePreparedDeleteBlade ($tablename, $columns, $conjunction = 'AND')
 {
        global $dbxlink;
@@ -2232,13 +2207,6 @@ function usePreparedDeleteBlade ($tablename, $columns, $conjunction = 'AND')
        }
 }
 
-function useSelectBlade ($query)
-{
-       global $dbxlink;
-       $result = $dbxlink->query ($query);
-       return $result;
-}
-
 function usePreparedSelectBlade ($query, $args = array())
 {
        global $dbxlink;
@@ -3297,7 +3265,7 @@ function commitUpdateFile ($file_id = 0, $new_name = '', $new_type = '', $new_co
 function commitUnlinkFile ($link_id)
 {
        if (usePreparedDeleteBlade ('FileLink', array ('id' => $link_id)) === FALSE)
-               return __FUNCTION__ . ': useDeleteBlade() failed';
+               return __FUNCTION__ . '(): query failed';
        return '';
 }
 
@@ -3305,7 +3273,7 @@ function commitDeleteFile ($file_id)
 {
        destroyTagsForEntity ('file', $file_id);
        if (usePreparedDeleteBlade ('File', array ('id' => $file_id)) === FALSE)
-               return __FUNCTION__ . ': useDeleteBlade() failed';
+               return __FUNCTION__ . '(): query failed';
        return '';
 }