From c784d2b2d3f072e49e507797ab15c50a50e1c48b Mon Sep 17 00:00:00 2001
From: valentina <pek5892@gmail.com>
Date: Wed, 16 Oct 2024 08:48:39 +0200
Subject: [PATCH] fix: gestione movimenti articoli

---
 modules/articoli/actions.php      | 16 +---------------
 modules/articoli/src/Articolo.php |  5 ++---
 2 files changed, 3 insertions(+), 18 deletions(-)

diff --git a/modules/articoli/actions.php b/modules/articoli/actions.php
index f6b97b837..6f105475d 100755
--- a/modules/articoli/actions.php
+++ b/modules/articoli/actions.php
@@ -84,7 +84,7 @@ switch (post('op')) {
         if (!empty(post('qta'))) {
             $data_movimento = new Carbon();
             $sede = post('sede');
-            $articolo->movimenta(post('qta'), tr('Carico manuale'), $data_movimento->format('Y-m-d'), true, $sede);
+            $articolo->movimenta(post('qta'), tr('Carico manuale'), $data_movimento->format('Y-m-d'), true, ['idsede' => $sede]);
         }
 
         $id_record = $articolo->id;
@@ -170,20 +170,6 @@ switch (post('op')) {
         // Aggiornamento delle varianti per i campi comuni
         Combinazione::sincronizzaVarianti($articolo);
 
-        // Leggo la quantità attuale per capire se l'ho modificata
-        // TODO: gestire la movimentazione manuale per sede
-        $id_sede = 0;
-        $old_qta = $articolo->getGiacenze(post('data_movimento'));
-        $movimento = $qta - $old_qta[$id_sede][0];
-
-        $qta_manuale = post('qta_manuale');
-        if (!empty($qta_manuale)) {
-            $descrizione_movimento = post('descrizione_movimento');
-            $data_movimento = post('data_movimento');
-
-            $articolo->movimenta($movimento, $descrizione_movimento, $data_movimento, true);
-        }
-
         // Salvataggio info componente (campo `contenuto`)
         if (!empty($componente)) {
             $contenuto_precedente_esistente = !empty($articolo->contenuto);
diff --git a/modules/articoli/src/Articolo.php b/modules/articoli/src/Articolo.php
index 5c5f546b9..017c07887 100755
--- a/modules/articoli/src/Articolo.php
+++ b/modules/articoli/src/Articolo.php
@@ -71,10 +71,10 @@ class Articolo extends Model
      *
      * @return bool
      */
-    public function movimenta($qta, $descrizone = null, $data = null, $manuale = false, $id_sede = null, $array = [], )
+    public function movimenta($qta, $descrizone = null, $data = null, $manuale = false, $array = [])
     {
         $data = ($data ?: date('Y-m-d H:i:s'));
-        $id = $this->registra($qta, $descrizone, $data, $manuale, $array, $id_sede);
+        $id = $this->registra($qta, $descrizone, $data, $manuale, $array, $array['idsede']);
 
         if (empty($this->servizio)) {
             $this->qta += $qta;
@@ -112,7 +112,6 @@ class Articolo extends Model
                 'movimento' => $descrizone,
                 'data' => $data,
                 'manuale' => $manuale,
-                'idsede' => $id_sede ?: 0,
                 'idutente' => $user->id,
             ]));
         }