Merge branch 'develop'

common/Alias/Model.php

@@ -6,9 +6,9 @@ use ProVM\Common\Factory\Model as ModelFactory;
 interface Model {
   public function getTable(): string;
   public function setFactory(ModelFactory $factory): Model;
-  public function parentOf(string $child_model_class, array $relation_definitions): array;
-  public function childOf(string $parent_model_class, array $relation_definitions): Model;
-  public function siblingOf(string $sibling_model_class, string $connecting_table, array $relation_definitions): array;
+  public function parentOf(string $child_model_class, array $relation_definitions): ?array;
+  public function childOf(string $parent_model_class, array $relation_definitions): ?Model;
+  public function siblingOf(string $sibling_model_class, string $connecting_table, array $relation_definitions): ?array;
 
   public function toArray(): array;
 }

common/Define/Model.php

@@ -37,7 +37,7 @@ abstract class Model extends BaseModel implements ModelInterface {
     }
     return $definitions;
   }
-  public function parentOf(string $child_model_class, array $relation_definitions): array {
+  public function parentOf(string $child_model_class, array $relation_definitions): ?array {
     $relation_definitions = $this->checkDefinitions($relation_definitions, [
       Model::SELF_KEY,
       Model::CHILD_KEY
@@ -54,7 +54,7 @@ abstract class Model extends BaseModel implements ModelInterface {
       ])
       ->many();
   }
-  public function childOf(string $parent_model_class, array $relation_definitions): ModelInterface {
+  public function childOf(string $parent_model_class, array $relation_definitions): ?ModelInterface {
     $relation_definitions = $this->checkDefinitions($relation_definitions, [
       Model::SELF_KEY,
       Model::PARENT_KEY
@@ -72,7 +72,7 @@ abstract class Model extends BaseModel implements ModelInterface {
       ])
       ->one();
   }
-  public function siblingOf(string $sibling_model_class, string $connecting_table, array $relation_definitions): array {
+  public function siblingOf(string $sibling_model_class, string $connecting_table, array $relation_definitions): ?array {
     $relation_definitions = $this->checkDefinitions($relation_definitions, [
       Model::SELF_KEY,
       Model::SIBLING_KEY,

common/Factory/Model.php

@@ -22,8 +22,20 @@ class Model {
     }
     return $this;
   }
-  public function create(string $model_class, array $data = null): Model {
-    return BaseModel::factory($model_class)->create($data);
+  public function create(string $model_class, array $data = null): ModelInterface {
+    if ($data !== null) {
+      $model = $this->find($model_class);
+      foreach ($data as $f => $v) {
+        $model = $model->where([[$f, $v]]);
+      }
+      $model = $model->one();
+      if ($model !== null) {
+        return $model;
+      }
+    }
+    $model = BaseModel::factory($model_class)->create($data);
+    $model->setFactory($this);
+    return $model;
   }
   protected $class;
   public function find(string $model_class): Model {
@@ -332,20 +344,29 @@ class Model {
     return $orm->offset($this->offset);
   }
 
-  public function one($id = null): ModelInterface {
+  public function one($id = null): ?ModelInterface {
     $result = $this->build()->findOne($id);
+    if (!$result) {
+      return null;
+    }
     $result->setFactory($this);
     return $result;
   }
-  public function many(): array {
+  public function many(): ?array {
     $results = $this->build()->findMany();
+    if (!$results) {
+      return null;
+    }
     foreach ($results as &$r) {
       $r->setFactory($this);
     }
     return $results;
   }
-  public function array(): array {
+  public function array(): ?array {
     $results = $this->build()->findArray();
+    if (!$results) {
+      return null;
+    }
     return $results;
   }