From 87a884bd8a22226947e234a9348136a130134769 Mon Sep 17 00:00:00 2001
From: OpenXE <>
Date: Tue, 25 Apr 2023 11:26:09 +0200
Subject: [PATCH] Bugfix auftrag vorkasse_ok

---
 www/lib/class.erpapi.php | 58 +++++++++++++++++-----------------------
 www/pages/auftrag.php    | 20 ++------------
 2 files changed, 27 insertions(+), 51 deletions(-)

diff --git a/www/lib/class.erpapi.php b/www/lib/class.erpapi.php
index cfc2a612..c4f22fc2 100644
--- a/www/lib/class.erpapi.php
+++ b/www/lib/class.erpapi.php
@@ -12683,9 +12683,8 @@ function SendPaypalFromAuftrag($auftrag, $test = false)
       //$this->app->DB->Update("UPDATE auftrag_position ap, artikel a SET ap.preis='0' WHERE ap.auftrag='$auftrag' AND a.id=ap.artikel AND a.porto='1'");
     }
 
-
-  //Vorkasse Check
-  //ist genug geld da? zusammenzaehlen der kontoauszuege_zahlungseingang
+  // Vorkasse Check    
+  $auftrag_ist_vorkasse = false;
   $summe_eingang = 0;
   $auftrag_gesamtsumme = 0;
   $zahlungsweise = '';
@@ -12694,43 +12693,36 @@ function SendPaypalFromAuftrag($auftrag, $test = false)
     $auftrag_gesamtsumme = $auftragarr['gesamtsumme'];// $this->app->DB->Select("SELECT gesamtsumme FROM auftrag WHERE id='$auftrag' LIMIT 1");
     $zahlungsweise = $auftragarr['zahlungsweise'];// $this->app->DB->Select("SELECT zahlungsweise FROM auftrag WHERE id='$auftrag' LIMIT 1");
     $vorabbezahltmarkieren = $auftragarr['vorabbezahltmarkieren'];// $this->app->DB->Select("SELECT vorabbezahltmarkieren FROM auftrag WHERE id='$auftrag' LIMIT 1");
+    $waehrung = $auftragarr['waehrung'];
   }
-  //TODO zahlungsweisemodul
   $zahlungsweise = strtolower($zahlungsweise);
 
-/*
+    // Check behaviour
+    $zahlungsweisenmodule = $this->app->DB->SelectArr("SELECT id, modul, verhalten FROM zahlungsweisen WHERE type = '".$this->app->DB->real_escape_string($zahlungsweise)."' AND (projekt = '$projekt' OR projekt = 0) ORDER BY projekt = '$projekt' DESC LIMIT 1");
+    if (!empty($zahlungsweisenmodule))   {
+        if ($zahlungsweisenmodule[0]['verhalten'] == 'vorkasse') {
+            $auftrag_ist_vorkasse = true;
+        }
+    } else if ($zahlungsweise == 'vorkasse') {
+        $auftrag_ist_vorkasse = true;    
+    }
 
-    OLD CODE REPLACED BY FUNCTION IN auftrag.php / fibu_buchungen
+   // Check fibu_buchungen saldo
+    $saldo = $this->app->erp->GetSaldoDokument($auftrag,'auftrag');
+    if (!empty($saldo)) {
+        if ($saldo['waehrung'] == $waehrung && $saldo['betrag'] >= $auftrag_gesamtsumme) {
+            $saldo_ok = true;
+        } 
+    }  
 
-  if($zahlungsweisenmodule = $this->app->DB->SelectArr("SELECT id, modul, verhalten FROM zahlungsweisen WHERE type = '".$this->app->DB->real_escape_string($zahlungsweise)."' AND
-   (projekt = '$projekt' OR projekt = 0) ORDER BY projekt = '$projekt' DESC LIMIT 1
-  "))
-  {
-    if( ( $zahlungsweisenmodule[0]['verhalten'] == 'vorkasse' && (($summe_eingang + $toleranz>=$auftrag_gesamtsumme) || $vorabbezahltmarkieren)) || ($zahlungsweisenmodule[0]['verhalten'] == 'lastschrift') || ($zahlungsweisenmodule[0]['verhalten'] == '') || ($zahlungsweisenmodule[0]['verhalten'] == 'rechnung') )
-    {
-      $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='1' WHERE id='$auftrag' LIMIT 1");
-    }elseif($summe_eingang > 0)
-    {
-      $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='2' WHERE id='$auftrag' LIMIT 1");
-    }else{
-      $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='0' WHERE id='$auftrag' LIMIT 1");
+    // Decide condition
+    if ($auftrag_ist_vorkasse && !$saldo_ok && !$vorabbezahltmarkieren) {
+        $vorkasse_ok = '0';
+    } else {
+        $vorkasse_ok = '1';
     }
-  }else{
-    if($summe_eingang+$toleranz>=$auftrag_gesamtsumme || ($zahlungsweise=="rechnung" || $zahlungsweise=="amazon" || $zahlungsweise=="amazon_bestellung" || $zahlungsweise=="secupay"
-          || $zahlungsweise=="nachnahme" || $zahlungsweise=="einzugsermaechtigung" || $zahlungsweise=="lastschrift" || $zahlungsweise=="bar") || $auftrag_gesamtsumme==0 || $vorabbezahltmarkieren=="1")
-    {
-      //TODO ok bei amazon und amazon_bestellung nur wenn transaktionsnummer vorhanden?
-      $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='1' WHERE id='$auftrag' LIMIT 1");
-    }
-    else if ($summe_eingang > 0) {
-      $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='2' WHERE id='$auftrag' LIMIT 1");
-    }
-    else {
-      $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='0' WHERE id='$auftrag' LIMIT 1");
-    }
-  }
 
-*/
+    $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok='$vorkasse_ok' WHERE id='$auftrag'");
 
   //nachnahme gebuehr check!!!!
   //$nachnahme = $this->app->DB->Select("SELECT id FROM auftrag_position WHERE auftrag='$auftrag' AND nummer='200001' LIMIT 1");
diff --git a/www/pages/auftrag.php b/www/pages/auftrag.php
index 869ea9fe..766a39a3 100644
--- a/www/pages/auftrag.php
+++ b/www/pages/auftrag.php
@@ -6713,27 +6713,11 @@ Die Gesamtsumme stimmt nicht mehr mit urspr&uuml;nglich festgelegten Betrag '.
   {
 
      // refresh all open items
-    $openids = $this->app->DB->SelectArr("SELECT id, gesamtsumme, waehrung from auftrag WHERE status <> 'abgeschlossen'");
-
+    $openids = $this->app->DB->SelectArr("SELECT id from auftrag WHERE status <> 'abgeschlossen'");
     foreach ($openids as $openid) {
-        $saldo = $this->app->erp->GetSaldoDokument($openid['id'],'auftrag');
-        if (!empty($saldo)) {
-            if ($saldo['waehrung'] == $openid['waehrung'] && $saldo['betrag'] >= $openid['gesamtsumme']) {
-                $sql = "UPDATE 
-                            auftrag
-                        SET
-                            vorkasse_ok = 1
-                        WHERE id=".$openid['id'];
-                $this->app->DB->Update($sql);
-                continue;
-            } 
-        }
-        else {
-            $this->app->DB->Update("UPDATE auftrag SET vorkasse_ok = 0 WHERE id=".$openid['id']);        
-        }
+        $this->app->erp->AuftragAutoversandBerechnen($openid['id']);
     }  
 
-
     if($this->app->Secure->GetPOST('ausfuehren') && $this->app->erp->RechteVorhanden('auftrag', 'edit'))
     {
       $drucker = $this->app->Secure->GetPOST('seldrucker');