Merge pull request #72 from exciler/sendcloud

Versandarten Bugfixes and stability improvements, some cleanup

Author: OpenXE-ERP

www/lib/class.erpapi.php

@@ -28104,31 +28104,15 @@ function GetVersandartAuftrag($projekt = 0)
       $modul = "";
 
       $tmp = array(
-           'DHL'=>'DHL','DPD'=>'DPD',
-           'express_dpd'=>'Express DPD',
-           'export_dpd'=>'Export DPD',
-           'gls'=>'GLS',
            'keinversand'=>'Kein Versand',
            'selbstabholer'=>'Selbstabholer',
-           'versandunternehmen'=>'Sonstige',
            'spedition'=>'Spedition',
-           'Go'=>'GO!',
-           'post'=>'Post'
        );
       foreach($tmp as $key=>$value)
       {
-        if($key == 'DHL'){$modul='intraship';}
-        if($key == 'DPD'){$modul='dpdapi';}
-        if($key == 'express_dpd'){$modul='';}
-        if($key == 'export_dpd'){$modul='';}
-        if($key == 'gls'){$modul='glsapi';}
         if($key == 'keinversand'){$modul='';}
         if($key == 'selbstabholer'){$modul='selbstabholer';}
-        if($key == 'versandunternehmen'){$modul='';}
         if($key == 'spedition'){$modul='';}
-        if($key == 'Go'){$modul='';}
-        if($key == 'post'){$modul='post';}
-
 
         $this->app->DB->Insert("INSERT INTO versandarten (id,type,bezeichnung,aktiv,modul) VALUES ('','$key','$value','1','$modul')");
       }

www/lib/class.versanddienstleister.php

@@ -153,14 +153,14 @@ public function GetAdressdaten($id, $sid): array
         lp.zollwaehrung
       FROM lieferschein_position lp
       JOIN artikel a on lp.artikel = a.id
-      LEFT JOIN auftrag_position ap on lp.auftrag_position_id = ap.id
-      LEFT JOIN rechnung_position rp on ap.id = rp.auftrag_position_id
-      LEFT JOIN rechnung r on rp.rechnung = r.id
+      LEFT OUTER JOIN auftrag_position ap on lp.auftrag_position_id = ap.id
+      LEFT OUTER JOIN rechnung_position rp on ap.id = rp.auftrag_position_id
+      LEFT OUTER JOIN rechnung r on rp.rechnung = r.id
       WHERE lp.lieferschein = $lieferscheinId
       AND a.lagerartikel = 1
       AND r.status != 'storniert'
       ORDER BY lp.sort";
-    $ret['positions'] = $this->app->DB->SelectArr($sql);
+    $ret['positions'] = $this->app->DB->SelectArr($sql) ?? [];
 
     if ($sid === "lieferschein") {
       $standardkg = $this->app->erp->VersandartMindestgewicht($lieferscheinId);