WIP remove explicit table everywhere

Author: mvorisek

docs/model.rst

@@ -532,7 +532,7 @@ Populating Data
 
     The method will still convert the data needed and operate with joined
     tables as needed. If you wish to access tables directly, you'll have to look
-    into Persistence::insert($m, $data, $table);
+    into Persistence::insert($m, $data);
 
 
 

src/Persistence/Array_.php

@@ -57,7 +57,7 @@ private function assertNoIdMismatch($idFromRow, $id): void
         }
     }
 
-    private function saveRow(Model $model, array $row, $id, string $table): void
+    private function saveRow(Model $model, array $row, $id): void
     {
         if ($model->id_field) {
             $idField = $model->getField($model->id_field);
@@ -68,7 +68,7 @@ private function saveRow(Model $model, array $row, $id, string $table): void
             }
         }
 
-        $this->data[$table][$id] = $row;
+        $this->data[$model->table][$id] = $row;
     }
 
     private function addIdToLoadRow(Model $model, array &$row, $id): void
@@ -140,34 +140,37 @@ public function add(Model $model, array $defaults = []): Model
 
     public function tryLoad(Model $model, $id, string $table = null): ?array
     {
-        $table = $table ?? $model->table;
-        if (!isset($this->data[$table])) {
+        if ($table !== null) {
+            throw new \Error('debug xxx!! ' . $table . ' vs. ' . $model->table);
+        }
+
+        if (!isset($this->data[$model->table])) {
             throw (new Exception('Table was not found in the array data source'))
-                ->addMoreInfo('table', $table);
+                ->addMoreInfo('table', $model->table);
         }
 
         if ($id === self::ID_LOAD_ONE || $id === self::ID_LOAD_ANY) {
-            if (count($this->data[$table]) === 0) {
+            if (count($this->data[$model->table]) === 0) {
                 return null;
-            } elseif ($id === self::ID_LOAD_ONE && count($this->data[$table]) !== 1) {
+            } elseif ($id === self::ID_LOAD_ONE && count($this->data[$model->table]) !== 1) {
                 throw (new Exception('Ambiguous conditions, more than one record can be loaded.'))
                     ->addMoreInfo('model', $model)
                     ->addMoreInfo('id', null);
             }
 
-            $id = array_key_first($this->data[$table]);
+            $id = array_key_first($this->data[$model->table]);
 
-            $row = $this->tryLoad($model, $id, $table);
+            $row = $this->tryLoad($model, $id);
             $model->setId($id); // @TODO is it needed?
 
             return $row;
         }
 
-        if (!isset($this->data[$table][$id])) {
+        if (!isset($this->data[$model->table][$id])) {
             return null;
         }
 
-        $row = $this->data[$table][$id];
+        $row = $this->data[$model->table][$id];
         $this->addIdToLoadRow($model, $row, $id);
 
         return $this->typecastLoadRow($model, $row);
@@ -180,13 +183,15 @@ public function tryLoad(Model $model, $id, string $table = null): ?array
      */
     public function insert(Model $model, array $data, string $table = null)
     {
-        $table = $table ?? $model->table;
+        if ($table !== null) {
+            throw new \Error('debug xxx!! ' . $table . ' vs. ' . $model->table);
+        }
 
         $data = $this->typecastSaveRow($model, $data);
 
-        $id = $data[$model->id_field] ?? $this->generateNewId($model, $table);
+        $id = $data[$model->id_field] ?? $this->generateNewId($model);
 
-        $this->saveRow($model, $data, $id, $table);
+        $this->saveRow($model, $data, $id);
 
         return $id;
     }
@@ -200,11 +205,13 @@ public function insert(Model $model, array $data, string $table = null)
      */
     public function update(Model $model, $id, array $data, string $table = null)
     {
-        $table = $table ?? $model->table;
+        if ($table !== null) {
+            throw new \Error('debug xxx!! ' . $table . ' vs. ' . $model->table);
+        }
 
         $data = $this->typecastSaveRow($model, $data);
 
-        $this->saveRow($model, array_merge($this->data[$table][$id] ?? [], $data), $id, $table);
+        $this->saveRow($model, array_merge($this->data[$model->table][$id] ?? [], $data), $id);
 
         return $id;
     }
@@ -216,9 +223,11 @@ public function update(Model $model, $id, array $data, string $table = null)
      */
     public function delete(Model $model, $id, string $table = null)
     {
-        $table = $table ?? $model->table;
+        if ($table !== null) {
+            throw new \Error('debug xxx!! ' . $table . ' vs. ' . $model->table);
+        }
 
-        unset($this->data[$table][$id]);
+        unset($this->data[$model->table][$id]);
     }
 
     /**
@@ -228,13 +237,15 @@ public function delete(Model $model, $id, string $table = null)
      */
     public function generateNewId(Model $model, string $table = null)
     {
-        $table = $table ?? $model->table;
+        if ($table !== null) {
+            throw new \Error('debug xxx!! ' . $table . ' vs. ' . $model->table);
+        }
 
         $type = $model->id_field ? $model->getField($model->id_field)->type : 'integer';
 
         switch ($type) {
             case 'integer':
-                $ids = $model->id_field ? array_keys($this->data[$table]) : [count($this->data[$table])];
+                $ids = $model->id_field ? array_keys($this->data[$model->table]) : [count($this->data[$model->table])];
 
                 $id = $ids ? max($ids) + 1 : 1;
 
@@ -248,7 +259,7 @@ public function generateNewId(Model $model, string $table = null)
                     ->addMoreInfo('type', $type);
         }
 
-        $this->lastInsertIds[$table] = $id;
+        $this->lastInsertIds[$model->table] = $id;
         $this->lastInsertIds['$'] = $id;
 
         return $id;
@@ -310,7 +321,7 @@ public function initAction(Model $model, array $fields = null): \Atk4\Data\Actio
     }
 
     /**
-     * Will set limit defined inside $m onto data.
+     * Will set limit defined inside $model onto data.
      */
     protected function setLimitOrder(Model $model, \Atk4\Data\Action\Iterator $action)
     {

src/Persistence/Csv.php

@@ -326,7 +326,7 @@ public function insert(Model $model, array $data)
      *
      * @param mixed $id
      */
-    public function update(Model $model, $id, array $data, string $table = null)
+    public function update(Model $model, $id, array $data)
     {
         throw new Exception('Updating records is not supported in CSV persistence.');
     }
@@ -336,7 +336,7 @@ public function update(Model $model, $id, array $data, string $table = null)
      *
      * @param mixed $id
      */
-    public function delete(Model $model, $id, string $table = null)
+    public function delete(Model $model, $id)
     {
         throw new Exception('Deleting records is not supported in CSV persistence.');
     }
@@ -346,8 +346,10 @@ public function delete(Model $model, $id, string $table = null)
      *
      * @return string
      */
-    public function generateNewId(Model $model, string $table = null)
+    public function generateNewId(Model $model)
     {
+        $ids = array_keys($this->data[$model->table]);
+
         throw new Exception('Not implemented');
     }