suppress some more unnecessary default values
authorDenis Ovsienko <denis@ovsienko.info>
Fri, 19 May 2017 09:45:38 +0000 (10:45 +0100)
committerDenis Ovsienko <denis@ovsienko.info>
Fri, 19 May 2017 09:45:38 +0000 (10:45 +0100)
Amend respective test to fail the right way.

* usePreparedDeleteBlade()
* usePreparedUpdateBlade()

tests/EmptySQLWhereTest.php
wwwroot/inc/database.php

index 89e2f3d9a02f18cb32c213baf4a48ba878038b56..432f3b95c53e2778d023bc053bb9f229261dde1f 100644 (file)
@@ -21,7 +21,7 @@ class EmptySQLWhereTest extends PHPUnit_Framework_TestCase
         */
        public function testMalformedDelete ()
        {
-               usePreparedDeleteBlade ('TagTree');
+               usePreparedDeleteBlade ('TagTree', NULL);
        }
 
        /**
@@ -30,7 +30,7 @@ class EmptySQLWhereTest extends PHPUnit_Framework_TestCase
         */
        public function testMalformedUpdate1 ()
        {
-               usePreparedUpdateBlade ('TagTree', array ('is_assignable' => 'yes'));
+               usePreparedUpdateBlade ('TagTree', array ('is_assignable' => 'yes'), NULL);
        }
 
        /**
@@ -39,7 +39,7 @@ class EmptySQLWhereTest extends PHPUnit_Framework_TestCase
         */
        public function testMalformedUpdate2 ()
        {
-               usePreparedUpdateBlade ('TagTree');
+               usePreparedUpdateBlade ('TagTree', NULL, NULL);
        }
 }
 
index 82e24f28556627becf02fa2c04d0e6a5654e9626..fa55b2fa27eb5c41784c27e5a76dfbec9e19c31a 100644 (file)
@@ -3981,7 +3981,7 @@ function makeWhereSQL ($where_columns, $conjunction, &$params = array())
 // This swiss-knife blade deletes any number of records from the specified table
 // using the specified key names and values.
 // returns integer - affected rows count. Throws exception on error
-function usePreparedDeleteBlade ($tablename, $columns = array(), $conjunction = 'AND')
+function usePreparedDeleteBlade ($tablename, $columns, $conjunction = 'AND')
 {
        global $dbxlink;
        if (! count ($columns))
@@ -4018,7 +4018,7 @@ function usePreparedSelectBlade ($query, $args = array())
 }
 
 // returns integer - affected rows count. Throws exception on error
-function usePreparedUpdateBlade ($tablename, $set_columns = array(), $where_columns = array(), $conjunction = 'AND')
+function usePreparedUpdateBlade ($tablename, $set_columns, $where_columns, $conjunction = 'AND')
 {
        global $dbxlink;
        if (! count ($set_columns))