From 4303bd8bf896754f12a6e0afdf76a0d4f2477d6c Mon Sep 17 00:00:00 2001
From: NicNal <nn.euganeo.inf@gmail.com>
Date: Mon, 17 Jun 2019 11:10:32 +0200
Subject: [PATCH] Fix calcolo giacenze fra sedi della propria azienda

---
 .../articoli/plugins/articoli.giacenze.php    | 23 ++++++++-----------
 1 file changed, 10 insertions(+), 13 deletions(-)

diff --git a/modules/articoli/plugins/articoli.giacenze.php b/modules/articoli/plugins/articoli.giacenze.php
index 5e6c86348..c665c103e 100644
--- a/modules/articoli/plugins/articoli.giacenze.php
+++ b/modules/articoli/plugins/articoli.giacenze.php
@@ -1,7 +1,6 @@
 <?php
 
-$sedi = $dbo->fetchArray("SELECT SUM(mg_movimenti.qta)AS qta, IF(mg_movimenti.idsede_azienda= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_azienda WHERE mg_movimenti.idarticolo=".prepare($id_record).' GROUP BY idsede_azienda');
-
+$sedi = $dbo->fetchArray( '(SELECT "0" AS id, "Sede legale" AS nomesede) UNION (SELECT id, CONCAT(nomesede, " - ", citta ) AS nomesede FROM an_sedi WHERE idanagrafica='.prepare(setting('Azienda predefinita')).')' );
 ?>
 
 <div class="row">
@@ -12,11 +11,6 @@ $sedi = $dbo->fetchArray("SELECT SUM(mg_movimenti.qta)AS qta, IF(mg_movimenti.id
       </div>
       
       <div class="panel-body">
-        <?php
-        if (empty($sedi)) {
-            echo '<div class="alert alert-info">'.tr('Non ci sono ancora movimenti').'</div>';
-        } else {
-            ?>
           <table class="table table-striped table-condensed table-bordered">
             <thead>
               <tr>
@@ -28,17 +22,20 @@ $sedi = $dbo->fetchArray("SELECT SUM(mg_movimenti.qta)AS qta, IF(mg_movimenti.id
             <tbody>
               <?php
                     foreach ($sedi as $sede) {
-                        echo '
+                      // Lettura movimenti della sede
+                      $qta_azienda = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_azienda= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_azienda WHERE mg_movimenti.idarticolo=".prepare($id_record).' AND idsede_azienda='.prepare($sede['id']).' GROUP BY idsede_azienda');
+
+                      // Lettura eventuali movimenti ad una propria sede
+                      $qta_controparte = $dbo->fetchOne("SELECT SUM(mg_movimenti.qta) AS qta, IF(mg_movimenti.idsede_controparte= 0,'Sede legale',(CONCAT_WS(' - ',an_sedi.nomesede,an_sedi.citta))) as sede FROM mg_movimenti LEFT JOIN an_sedi ON an_sedi.id = mg_movimenti.idsede_controparte WHERE mg_movimenti.idarticolo=".prepare($id_record)." AND idsede_controparte=".prepare($sede['id'])." GROUP BY idsede_controparte");
+
+                      echo '
                         <tr>
-                            <td>'.$sede['sede'].'</td>
-                            <td class="text-right">'.Translator::numberToLocale($sede['qta']).'</td>
+                            <td>'.$sede['nomesede'].'</td>
+                            <td class="text-right">'.Translator::numberToLocale($qta_azienda['qta'] - $qta_controparte['qta'] ).'</td>
                         </tr>';
                     } ?>
             </tbody>
           </table>
-        <?php
-        }
-        ?>
       </div>
     </div>
   </div>