Changeset 1437 for trunk/system/libraries/drivers/Database_Mysql.php

Timestamp:

12/06/2007 08:15:37 PM (13 months ago)

Author:

Shadowhand

Message:

Changes to core:

• date::timestamp() variable name typo
  
• form::input() no longer puts an id on array-keyed inputs
  
• form::label() no longer
  
• added arr::unshift_assoc()
  
• replace a couple of internal find_all() calls with load_result(TRUE) to avoid an extra method call
  
• fixed a bug in Mysql_Result::result_array() that made multiple calls fail after the first call do to the pointer not being reset
  
• small code and comment cleanups
  

Files:

• trunk/system/libraries/drivers/Database_Mysql.php (modified) (2 diffs)

trunk/system/libraries/drivers/Database_Mysql.php

@@ -393 +393 @@
                 }
 
-                while ($row = $fetch($this->result, $type))
-                {
-                        $rows[] = $row;
-                }
-
-                return $rows;
+                if (mysql_num_rows($this->result))
+                {
+                        // Reset the pointer location to make sure things work properly
+                        mysql_data_seek($this->result, 0);
+
+                        while ($row = $fetch($this->result, $type))
+                        {
+                                $rows[] = $row;
+                        }
+                }
+
+                return isset($rows) ? $rows : array();
         }
 
@@ -476 +482 @@
         {
                 // Check to see if the requested offset exists.
-                if (!$this->offsetExists($offset))
+                if ( ! $this->offsetExists($offset))
                         return FALSE;