Merge pull request #273 from avbdr/master

Support for separate key definition in hasOne relation

Author: avbdr

dbObject.php

@@ -139,17 +139,16 @@ public function __get ($name) {
             $modelName = $this->relations[$name][1];
             switch ($relationType) {
                 case 'hasone':
+                    $key = isset ($this->relations[$name][2]) ? $this->relations[$name][2] : $name;
                     $obj = new $modelName;
                     $obj->returnType = $this->returnType;
-                    $this->data[$name] = $obj->byId($this->data[$name]);
-                    return $this->data[$name];
+                    return $this->data[$name] = $obj->byId($this->data[$key]);
                     break;
                 case 'hasmany':
                     $key = $this->relations[$name][2];
                     $obj = new $modelName;
                     $obj->returnType = $this->returnType;
-                    $this->data[$name] = $obj->where($key, $this->data[$this->primaryKey])->get();
-                    return $this->data[$name];
+                    return $this->data[$name] = $obj->where($key, $this->data[$this->primaryKey])->get();
                     break;
                 default:
                     break;

tests/dbObjectTests.php

@@ -129,6 +129,11 @@ function createTable ($name, $data) {
     exit;
 }
 
+$product = product::with('user')->byId(5);
+if (!is_object ($product->data['user'])) {
+    echo "Error in with processing in getOne object";
+    exit;
+}
 
 $products = product::ArrayBuilder()->with('userId')->get(2);
 if (!is_array ($products[0]['userId']) || !is_array ($products[1]['userId'])) {

tests/models/product.php

@@ -16,7 +16,8 @@ class product extends dbObject {
         'productName' => Array ('text','required')
     );
     protected $relations = Array (
-        'userId' => Array ("hasOne", "user")
+        'userId' => Array ("hasOne", "user"),
+        'user' => Array ("hasOne", "user", "userId")
     );
 
     public function last () {