Partical fix for a dbObject usage with setPrefix()

Author: avbdr

dbObject.php

@@ -282,7 +282,7 @@ public function delete () {
      * @return dbObject|array
      */
     private function byId ($id, $fields = null) {
-        $this->db->where ($this->dbTable . '.' . $this->primaryKey, $id);
+        $this->db->where (MysqliDb::$prefix . $this->dbTable . '.' . $this->primaryKey, $id);
         return $this->getOne ($fields);
     }
 
@@ -365,7 +365,8 @@ private function join ($objectName, $key = null, $joinType = 'LEFT') {
         $joinObj = new $objectName;
         if (!$key)
             $key = $objectName . "id";
-        $joinStr = "{$this->dbTable}.{$key} = {$joinObj->dbTable}.{$joinObj->primaryKey}";
+        $joinStr = MysqliDb::$prefix . $this->dbTable . ".{$key} = " .
+                    MysqliDb::$prefix . "{$joinObj->dbTable}.{$joinObj->primaryKey}";
         $this->db->join ($joinObj->dbTable, $joinStr, $joinType);
         return $this;
     }
@@ -601,7 +602,7 @@ private static function dbObjectAutoload ($classname) {
      *
      * Calling autoload() without path will set path to dbObjectPath/models/ directory
      *
-     * @param string $path 
+     * @param string $path
      */
     public static function autoload ($path = null) {
         if ($path)

tests/dbObjectTests.php

@@ -4,6 +4,8 @@
 require_once ("../dbObject.php");
 
 $db = new Mysqlidb('localhost', 'root', '', 'testdb');
+$prefix = 't_';
+$db->setPrefix($prefix);
 dbObject::autoload ("models");
 
 $tables = Array (
@@ -91,8 +93,8 @@ function createTable ($name, $data) {
 
 // rawQuery test
 foreach ($tables as $name => $fields) {
-    $db->rawQuery("DROP TABLE " . $name);
-    createTable ($name, $fields);
+    $db->rawQuery("DROP TABLE " . $prefix . $name);
+    createTable ($prefix . $name, $fields);
 }
 
 foreach ($data as $name => $datas) {
@@ -134,7 +136,7 @@ function createTable ($name, $data) {
     exit;
 }
 
-$depts = product::join('user')->orderBy('products.id', 'desc')->get(5);
+$depts = product::join('user')->orderBy('t_products.id', 'desc')->get(5);
 foreach ($depts as $d) {
     if (!is_object($d)) {
         echo "Return should be an object\n";
@@ -244,21 +246,21 @@ function createTable ($name, $data) {
 if (!is_array (user::ArrayBuilder()->byId(1)))
     echo "wrong return type2";
 
-if (!is_array (product::join('user')->orderBy('products.id', 'desc')->get(2)))
+if (!is_array (product::join('user')->orderBy('t_products.id', 'desc')->get(2)))
     echo "wrong return type2";
 
-if (!is_array (product::orderBy('products.id', 'desc')->join('user')->get(2)))
+if (!is_array (product::orderBy('t_products.id', 'desc')->join('user')->get(2)))
     echo "wrong return type2";
 
 $u = new user;
 if (!$u->byId(1) instanceof user)
     echo "wrong return type2";
 
 $p = new product;
-if (!is_array ($p->join('user')->orderBy('products.id', 'desc')->get(2)))
+if (!is_array ($p->join('user')->orderBy('t_products.id', 'desc')->get(2)))
     echo "wrong return type2";
 
-if (!is_array ($p->orderBy('products.id', 'desc')->join('user')->get(2)))
+if (!is_array ($p->orderBy('t_products.id', 'desc')->join('user')->get(2)))
     echo "wrong return type2";
 
 echo "All done";