From 5948b728d2e459cb5d2665690bdb8ef7f41ec570 Mon Sep 17 00:00:00 2001
From: OpenXE <>
Date: Wed, 17 Jan 2024 17:32:14 +0100
Subject: [PATCH] verbindlichkeit improvements

---
 www/pages/content/verbindlichkeit_edit.tpl    |  4 +--
 .../content/verbindlichkeit_positionen.tpl    | 20 +++++++++-----
 www/pages/verbindlichkeit.php                 | 27 +++++++++----------
 3 files changed, 29 insertions(+), 22 deletions(-)

diff --git a/www/pages/content/verbindlichkeit_edit.tpl b/www/pages/content/verbindlichkeit_edit.tpl
index 31547e41..319ca0c1 100644
--- a/www/pages/content/verbindlichkeit_edit.tpl
+++ b/www/pages/content/verbindlichkeit_edit.tpl
@@ -100,7 +100,7 @@
                                                     </tr>                                                                                                                                              
                                                     <tr>
                                                         <td>
-                                                            {|Zahlbarbis|}:
+                                                            {|Zahlbar bis|}:
                                                         </td>
                                                         <td>
                                                             <input type="text" name="zahlbarbis" id="zahlbarbis" value="[ZAHLBARBIS]" size="20" [SAVEDISABLED]>
@@ -116,7 +116,7 @@
                                                     </tr>
                                                     <tr>
                                                         <td>
-                                                            {|Skontobis|}:
+                                                            {|Skonto bis|}:
                                                         </td>
                                                         <td>
                                                             <input type="text" name="skontobis" id="skontobis" value="[SKONTOBIS]" size="20" [SAVEDISABLED]>
diff --git a/www/pages/content/verbindlichkeit_positionen.tpl b/www/pages/content/verbindlichkeit_positionen.tpl
index fc5635c3..a75989ff 100644
--- a/www/pages/content/verbindlichkeit_positionen.tpl
+++ b/www/pages/content/verbindlichkeit_positionen.tpl
@@ -29,9 +29,12 @@
                     <fieldset>
                         <table width="100%" border="0" class="mkTableFormular">
                             <legend>{|Aktionen|}</legend>
+                            <tr>
+                                <td><input type="checkbox" id="auswahlallewareneingaenge" onchange="allewareneingaengeauswaehlen();" />{|alle markieren|}</td>
+                            </tr>                          
                             <tr>
                                 <td><input type="checkbox" name="bruttoeingabe" value="1" />Bruttopreise eingeben</td>
-                            </tr>      
+                            </tr>                                  
                             <tr>
                                 <td><button [SAVEDISABLED] name="submit" value="positionen_hinzufuegen" class="ui-button-icon" style="width:100%;">Hinzuf&uuml;gen</button></td>
                             </tr>
@@ -55,17 +58,17 @@
                     <fieldset>
                         <table width="100%" border="0" class="mkTableFormular">
                             <legend>{|Aktionen|}</legend>
-                            <tr [POSITIONHINZUFUEGENHIDDEN]>
-                                <td><input type="checkbox" id="auswahlalle" onchange="alleauswaehlen();" />&nbsp;{|alle markieren|}</td>
+                            <tr [SACHKONTOCHANGEHIDDEN]>
+                                <td><input type="checkbox" id="auswahlalle" onchange="alleauswaehlen();" />{|alle markieren|}</td>
                             </tr>                          
                             <tr [POSITIONHINZUFUEGENHIDDEN]>
                                 <td><button [SAVEDISABLED] name="submit" value="positionen_entfernen" class="ui-button-icon" style="width:100%;">Entfernen</button></td>
                             </tr>
-                            <tr [POSITIONHINZUFUEGENHIDDEN]>
+                            <tr [SACHKONTOCHANGEHIDDEN]>
                                 <td><input type="text" name="positionen_sachkonto" id="positionen_sachkonto" value="" size="20"></td>
                             </tr>
-                            <tr [POSITIONHINZUFUEGENHIDDEN]>
-                                <td><button [SAVEDISABLED] name="submit" value="positionen_kontorahmen_setzen" class="ui-button-icon" style="width:100%;">Sachkonto setzen</button></td>
+                            <tr [SACHKONTOCHANGEHIDDEN]>
+                                <td><button name="submit" value="positionen_kontorahmen_setzen" class="ui-button-icon" style="width:100%;">Sachkonto setzen</button></td>
                             </tr>
                         </table>
                     </fieldset>
@@ -75,6 +78,11 @@
     </div>
 </form>
 <script>
+    function allewareneingaengeauswaehlen()
+    {
+      var wert = $('#auswahlallewareneingaenge').prop('checked');
+      $('#verbindlichkeit_paketdistribution_list').find(':checkbox').prop('checked',wert);
+    }
     function alleauswaehlen()
     {
       var wert = $('#auswahlalle').prop('checked');
diff --git a/www/pages/verbindlichkeit.php b/www/pages/verbindlichkeit.php
index b005c441..3453bee1 100644
--- a/www/pages/verbindlichkeit.php
+++ b/www/pages/verbindlichkeit.php
@@ -217,10 +217,10 @@ class Verbindlichkeit {
                 $bestellnummer = $verbindlichkeit['belegnr'];
                 $rechnung = $verbindlichkeit['rechnung'];
            
-                $heading = array('Paket-Nr.','Paket-Pos.', 'Bestellung', 'Lieferschein', 'Rechnung', 'Artikel-Nr.','Artikel','Bemerkung','Menge','Menge offen','Eingabe','Preis','Steuer','Sachkonto','');
-                $width = array(  '1%',        '1%',        '5%',         '5%',           '5%',       '5%',         '20%',    '20%',       '2%',   '1%',         '1%',     '1%',   '1%',    '1%',       '1%');  
+                $heading = array('',  'Paket-Nr.','Paket-Pos.', 'Bestellung', 'Lieferschein', 'Rechnung', 'Artikel-Nr.','Artikel','Bemerkung','Menge','Menge offen','Eingabe','Preis','Steuer','Sachkonto','');
+                $width = array(  '1%','1%',        '1%',        '5%',         '5%',           '5%',       '5%',         '20%',    '20%',       '2%',   '1%',         '1%',     '1%',   '1%',    '1%',       '1%');  
 
-                $findcols = array('pa','auswahl','belegnr','lsnr','renr','artikelnummer','name_de','bemerkung','menge','offen_menge','offen_menge','preis','steuer','sachkonto','pa');
+                $findcols = array('id','pa','id','belegnr','lsnr','renr','artikelnummer','name_de','bemerkung','menge','offen_menge','offen_menge','preis','steuer','sachkonto','pa');
                 $searchsql = array('p.nummer', 'p.name', 'p.bemerkung');
 
                 $alignright = array(9,10);
@@ -235,10 +235,9 @@ class Verbindlichkeit {
                         ))+0";
 
                 $auswahl = array (
-                    '<input type=\"text\" name=\"ids[]\" value=\"',                   
+                    '<input type=\"checkbox\" name=\"ids[]\" value=\"',                   
                     ['sql' => 'pd.id'],
-                    '" hidden/>',
-                    ['sql' => 'pd.id']
+                    '"/>'
                 );              
 
                 $werte = array (
@@ -272,7 +271,7 @@ class Verbindlichkeit {
                     ['sql' => 'pa.id'],
                     '</a>'
                 );
-        
+                      
                 $where = "offen_menge > 0";
 
                 // Toggle filters
@@ -298,19 +297,18 @@ class Verbindlichkeit {
 
                 $more_data1 = $this->app->Secure->GetGET("more_data1");
                 if ($more_data1 == 1) {
-                    $innerwhere .= " AND (b.belegnr LIKE '%".$bestellnummer."%' OR pa.renr LIKE '%".$rechnung."%')";
+                    $innerwhere .= " AND ((b.belegnr LIKE '%".$bestellnummer."%' AND '".$bestellnummer."' <> '') OR (pa.renr LIKE '%".$rechnung."%' AND pa.renr <> ''))";
                 } else {
                 }               
-                // END Toggle filters
-
-                
+                // END Toggle filters              
 
                 $sql = "
                     SELECT SQL_CALC_FOUND_ROWS * FROM (
                         SELECT 
                             pa.id pa_id,
-                            ".$this->app->erp->ConcatSQL($paketlink)." pa,
                             ".$this->app->erp->ConcatSQL($auswahl)." AS auswahl,
+                            ".$this->app->erp->ConcatSQL($paketlink)." pa,
+                            pd.id,
                             if(b.belegnr LIKE '%".$bestellnummer."%',CONCAT('<b>',b.belegnr,'</b>'),b.belegnr) AS belegnr,
                             pa.lsnr,
                             if(pa.renr LIKE '%".$rechnung."%',CONCAT('<b>',pa.renr,'</b>'),pa.renr) AS renr,
@@ -801,7 +799,7 @@ class Verbindlichkeit {
             break;
             case 'positionen_kontorahmen_setzen':                                
                 $freigabe = $this->app->DB->SelectArr("SELECT rechnungsfreigabe, freigabe FROM verbindlichkeit WHERE id =".$id)[0];
-                if ($freigabe['rechnungsfreigabe'] || $freigabe['freigabe']) {
+                if ($freigabe['rechnungsfreigabe']) {
                     break;
                 }
                 // Process multi action
@@ -986,6 +984,7 @@ class Verbindlichkeit {
         if ($verbindlichkeit_from_db['rechnungsfreigabe']) {
             $this->app->Tpl->Set('FREIGABEBUCHHALTUNGHIDDEN','hidden');
             $this->app->Tpl->Set('RUECKSETZENEINKAUFHIDDEN','hidden');
+            $this->app->Tpl->Set('SACHKONTOCHANGEHIDDEN','hidden'); 
         } else {
             $this->app->Tpl->Set('RUECKSETZENBUCHHALTUNGHIDDEN','hidden');
         }                    
@@ -1087,7 +1086,7 @@ class Verbindlichkeit {
         $freigabe = $this->app->DB->SelectArr("SELECT rechnungsfreigabe, freigabe FROM verbindlichkeit WHERE id =".$id)[0];       
         if ($freigabe['rechnungsfreigabe'] && $freigabe['freigabe']) {
             $this->app->Tpl->Set('SAVEDISABLED','disabled');
-            $this->app->Tpl->Set('SACHKONTOSAVEDISABLED','disabled');      
+            $this->app->Tpl->Set('SACHKONTOSAVEDISABLED','disabled');
         } else if ($freigabe['freigabe']) {
             $this->app->Tpl->Set('SAVEDISABLED','disabled');   
             if ($submit != '')