Korjattu tarkistusta. Ei tehdä turhaa querya.

devlab-oy · Apr 27, 2016 · 097cd7f · 097cd7f
1 parent 6bb59d6
commit 097cd7f
Showing 1 changed file with 9 additions and 32 deletions.
diff --git a/tilauskasittely/verkkolasku.php b/tilauskasittely/verkkolasku.php
@@ -463,41 +463,11 @@ function pp($muuttuja, $round="", $rmax="", $rmin="") {
 
     $query_ale_lisa = generoi_alekentta('M');
 
-    if ($syotetty > $tanaan and $yhtiorow['laskutus_tulevaisuuteen'] == 'S') {
-      $query = "SELECT lasku.tunnus,
-                count(tuote.tunnus) AS tuotteita,
-                sum(if(tuote.ei_saldoa != '', 1, 0)) AS saldottomat
-                FROM lasku
-                {$lasklisa_eikateiset}
-                JOIN tilausrivi on (tilausrivi.yhtio = lasku.yhtio and tilausrivi.otunnus = lasku.tunnus and tilausrivi.tyyppi = 'L' and tilausrivi.var not in ('P','J','O','S'))
-                JOIN tuote ON (tuote.yhtio = tilausrivi.yhtio AND tuote.tuoteno = tilausrivi.tuoteno)
-                WHERE lasku.yhtio  = '{$kukarow['yhtio']}'
-                and lasku.tila     = 'L'
-                and lasku.alatila  = 'D'
-                and lasku.viite    = ''
-                and lasku.chn     != '999'
-                {$lasklisa}
-                GROUP BY 1";
-      $saldoton_chk_res = pupe_query($query);
-
-      $_not_in = array();
-
-      while ($saldoton_chk_row = mysql_fetch_assoc($saldoton_chk_res)) {
-        if ($saldoton_chk_row['saldottomat'] != $saldoton_chk_row['tuotteita']) {
-          $_not_in[] = $saldoton_chk_row['tunnus'];
-        }
-      }
-
-      if (!empty($_not_in)) {
-        $lasklisa .= " and lasku.tunnus not in (".implode(',', $_not_in).") ";
-        $tulos_ulos .= "<br>\n".t("Tuotevirheet").":<br>\n".t("Tilausta")." ".implode(',', $_not_in)." ".t("ei voida laskuttaa, koska tilauksien kaikki tuotteet eivät olleet saldottomia")."!<br>\n";
-      }
-    }
-
     // saldovirhe_esto_laskutus-parametri 'H', jolla voidaan estää tilauksen laskutus, jos tilauksen yhdeltäkin tuotteelta saldo menee miinukselle
     // kehahinvirhe_esto_laskutus-parametri 'N', Estetaan laskutus mikali keskihankintahinta on 0.00 tai tuotteen kate on negatiivinen
     // Eutukäteeen lmaksettu verkkokauppatilaus ($editil_cli) laskutetaan vaikka saldo ei ihan riittäisikään
-    if (empty($editil_cli) and ($yhtiorow['saldovirhe_esto_laskutus'] == 'H' or $yhtiorow['kehahinvirhe_esto_laskutus'] == 'N')) {
+    // Mikäli halutaan laskuttaa tulevaisuuteen niin kaikki tilauksen tuotteet täytyy olla saldottomia
+    if (empty($editil_cli) and ($yhtiorow['saldovirhe_esto_laskutus'] == 'H' or $yhtiorow['kehahinvirhe_esto_laskutus'] == 'N' or $yhtiorow['laskutus_tulevaisuuteen'] == 'S')) {
 
       $query = "SELECT
                 tilausrivi.tuoteno,
@@ -519,6 +489,13 @@ function pp($muuttuja, $round="", $rmax="", $rmin="") {
       $lasku_chk_res = pupe_query($query);
 
       while ($lasku_chk_row = mysql_fetch_assoc($lasku_chk_res)) {
+
+        # Mikäli halutaan laskuttaa tulevaisuuteen niin kaikki tilauksen tuotteet täytyy olla saldottomia
+        if ($syotetty > $tanaan and $yhtiorow['laskutus_tulevaisuuteen'] == 'S') {
+          $lasklisa .= " and lasku.tunnus not in ({$lasku_chk_row['tunnukset']}) ";
+          $tulos_ulos .= "<br>\n".t("Tuotevirheet").":<br>\n".t("Tilausta")." {$lasku_chk_row['tunnukset']} ".t("ei voida laskuttaa, koska tilauksien kaikki tuotteet eivät olleet saldottomia")."!<br>\n";
+        }
+
         // Mikäli laskutuksessa tuotteen varastosaldo vähenee negatiiviseksi, hylätään KAIKKI tilaukset, joilla on kyseistä tuotetta
         if ($yhtiorow['saldovirhe_esto_laskutus'] == 'H') {
           $query = "SELECT sum(saldo) saldo