From 8fdab0a1bfc6d80f17ba03349df64f46d1116527 Mon Sep 17 00:00:00 2001
From: Luca Haid <luca@haid.in>
Date: Thu, 10 Oct 2024 09:34:24 +0200
Subject: [PATCH] fixed createLocations Action not working

---
 .../WarehouseAdministrationController.php           | 13 +++++++++----
 .../WarehouseLocation/WarehouseLocationModel.php    |  2 +-
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/application/WarehouseAdministration/WarehouseAdministrationController.php b/application/WarehouseAdministration/WarehouseAdministrationController.php
index 5eef0df1c..4ec4b44b7 100644
--- a/application/WarehouseAdministration/WarehouseAdministrationController.php
+++ b/application/WarehouseAdministration/WarehouseAdministrationController.php
@@ -28,6 +28,7 @@ class WarehouseAdministrationController extends mfBaseController {
     protected function createLocationsAction(): void {
         $existingLocations = WarehouseLocationModel::getAll();
         $companyCars = TimerecordingCarModel::getAll();
+        $beforeCount = WarehouseLocationModel::count();
 
 
         $wantedCarLocations = [];
@@ -73,7 +74,7 @@ class WarehouseAdministrationController extends mfBaseController {
                     "title" => $wantedCarLocation,
                     "description" => "Automatisch erstellt",
                     "assignedTo" => $assignedTo,
-                    "createdBy" => $this->me->id,
+                    "createBy" => $this->me->id,
                     "create" => time()
                 ]);
             }
@@ -97,14 +98,18 @@ class WarehouseAdministrationController extends mfBaseController {
                     "title" => $user->name . "'s Lagerort",
                     "description" => "Automatisch erstellt",
                     "assignedTo" => $user->id,
-                    "createdBy" => $this->me->id,
+                    "createBy" => $this->me->id,
                     "create" => time()
                 ]);
             }
         }
 
-        var_dump($existingLocations);
-        die();
+        $afterCount = WarehouseLocationModel::count();
 
+        if ($afterCount - $beforeCount === 0) {
+            $this->returnJSON(["success" => true, "message" => "Alle Lagerorte existieren bereits"]);
+        } else {
+            $this->returnJSON(["success" => true, "message" => "Es wurden " . ($afterCount - $beforeCount) . " neue Lagerorte erstellt"]);
+        }
     }
 }
\ No newline at end of file
diff --git a/application/WarehouseLocation/WarehouseLocationModel.php b/application/WarehouseLocation/WarehouseLocationModel.php
index 698780440..44eab9211 100644
--- a/application/WarehouseLocation/WarehouseLocationModel.php
+++ b/application/WarehouseLocation/WarehouseLocationModel.php
@@ -5,6 +5,6 @@ class WarehouseLocationModel extends TTCrudBaseModel {
     public string $title;
     public string $description;
     public int $assignedTo;
-    public int $createdBy;
+    public int $createBy;
     public int $create;
 }
\ No newline at end of file