From 046bfee9f90c30328b74e4bc95291c9cc7162c47 Mon Sep 17 00:00:00 2001
From: OpenXE <>
Date: Wed, 18 Sep 2024 20:12:54 +0200
Subject: [PATCH] seriennummern rework checks
---
www/lib/class.erpapi.php | 28 ++++-
www/pages/auftrag.php | 2 +
www/pages/lieferschein.php | 2 +-
www/pages/seriennummern.php | 228 +++++++++++++++++++-----------------
www/pages/versandpakete.php | 5 +
www/pages/wareneingang.php | 24 +++-
6 files changed, 173 insertions(+), 116 deletions(-)
diff --git a/www/lib/class.erpapi.php b/www/lib/class.erpapi.php
index 48380c9a..b07a9d64 100644
--- a/www/lib/class.erpapi.php
+++ b/www/lib/class.erpapi.php
@@ -2821,20 +2821,36 @@ public function NavigationHooks(&$menu)
}
}
- function SeriennummernCheckWarnung(int $artikel_id) {
+ function SeriennummernCheckBenachrichtigung(int $artikel_id) {
$obj = $this->LoadModul('seriennummern');
- $obj->seriennummern_check_and_message_stock_added($artikel_id);
+ return($obj->seriennummern_check_and_notification_stock_added($artikel_id));
}
-
- function SeriennummernCheckLieferscheinWarnung(int $lieferschein_id, bool $notification) {
+
+ function SeriennummernCheckLieferscheinBenachrichtigung(int $lieferschein_id) {
$obj = $this->LoadModul('seriennummern');
- return($obj->seriennummern_check_and_message_delivery_note($lieferschein_id, $notification));
+ return($obj->seriennummern_check_and_notification_delivery_note($lieferschein_id));
+ }
+
+ function SeriennummernCheckLieferscheinWarnung(int $lieferschein_id) {
+ $obj = $this->LoadModul('seriennummern');
+ return($obj->seriennummern_check_and_message_delivery_notes($lieferschein_id));
}
function SeriennummernCheckLieferschein($lieferschein_id = null, $ignore_date = false, $only_missing = true, $group_lieferschein = false) {
$obj = $this->LoadModul('seriennummern');
return ($obj->seriennummern_check_delivery_notes($lieferschein_id, $ignore_date, $only_missing, $group_lieferschein));
}
+
+ function SeriennummernCheckWareneingangWarnung(int $wareneingang_id) {
+ $obj = $this->LoadModul('seriennummern');
+ return($obj->seriennummern_check_and_message_incoming_goods($wareneingang_id));
+ }
+
+ function SeriennummernCheckWareneingang($wareneingang_id = null, $ignore_date = false, $only_missing = true, $group_wareneingang = false) {
+ $obj = $this->LoadModul('seriennummern');
+ return ($obj->seriennummern_check_incoming_goods($wareneingang_id, $ignore_date, $only_missing, $group_wareneingang));
+ }
+
// @refactor in Lager Modul
function ArtikelAnzahlLagerPlatzMitSperre($artikel,$lager_platz)
@@ -20479,7 +20495,7 @@ function ChargenMHDAuslagern($artikel, $menge, $lagerplatztyp, $lpid,$typ,$wert,
$this->RunHook('LagerEinlagern_after',7, $artikel, $menge, $regal, $projekt, $grund, $doctype,$doctypeid);
- $this->SeriennummernCheckWarnung($artikel);
+ $this->SeriennummernCheckBenachrichtigung($artikel);
}
function CreateLagerPlatzInhaltVPE($artikel, $menge, $gewicht, $laenge, $breite, $hoehe, $menge2 = 0, $gewicht2 = 0, $laenge2 = 0, $breite2 = 0, $hoehe2 = 0)
diff --git a/www/pages/auftrag.php b/www/pages/auftrag.php
index 1436153c..f7f09ac9 100644
--- a/www/pages/auftrag.php
+++ b/www/pages/auftrag.php
@@ -5794,6 +5794,8 @@ Die Gesamtsumme stimmt nicht mehr mit ursprünglich festgelegten Betrag '.
$nurRestmenge
);
+ $this->app->erp->SeriennummernCheckLieferscheinBenachrichtigung($lieferschein);
+
$sql = "SELECT id FROM kommissionierung k WHERE k.auftrag = '".$id."'";
$vorkommissionierung = $this->app->DB->Select($sql);
diff --git a/www/pages/lieferschein.php b/www/pages/lieferschein.php
index b2a5d241..3d3b40a6 100644
--- a/www/pages/lieferschein.php
+++ b/www/pages/lieferschein.php
@@ -1869,7 +1869,7 @@ class Lieferschein extends GenLieferschein
$seriennummern_aktiv = !empty($seriennummern);
- $seriennummern_check_result = $this->app->erp->SeriennummernCheckLieferscheinWarnung($id, false);
+ $seriennummern_check_result = $this->app->erp->SeriennummernCheckLieferscheinWarnung($id);
$seriennummern_ok = empty($seriennummern_check_result);
diff --git a/www/pages/seriennummern.php b/www/pages/seriennummern.php
index c2b88781..19501fd1 100644
--- a/www/pages/seriennummern.php
+++ b/www/pages/seriennummern.php
@@ -554,77 +554,7 @@ class Seriennummern {
$this->app->erp->MenuEintrag("index.php?module=seriennummern&action=wareneingaenge_list", "Wareneingänge");
// $this->app->erp->MenuEintrag("index.php", "Zurück");
}
-
- function seriennummern_check_and_message($artikel_id) {
- $check_seriennummern = $this->seriennummern_check_serials($artikel_id);
-
- if (!empty($check_seriennummern)) {
- $artikel_minus_id_links = array();
- $artikel_plus_id_links = array();
- foreach ($check_seriennummern as $artikel_id) {
- if ($artikel_id['menge_nummern'] < $artikel_id['menge_auf_lager']) {
- $artikel_minus_id_links[] = ''.$artikel_id['nummer'].'';
- }
- else if ($artikel_id['menge_nummern'] > $artikel_id['menge_auf_lager']) {
- $artikel_plus_id_links[] = ''.$artikel_id['nummer'].'';
- }
- }
- if (!empty($artikel_minus_id_links)) {
- $this->app->YUI->Message('warning','Seriennummern fehlen für Artikel: '.implode(', ',$artikel_minus_id_links));
- }
- if (!empty($artikel_plus_id_links)) {
- $this->app->YUI->Message('warning','Seriennummern Überschuss für Artikel: '.implode(', ',$artikel_plus_id_links));
- }
- }
- }
-
- function seriennummern_delivery_note_check_and_message($lieferschein_id) {
-
- $check_incoming_goods = $this->seriennummern_check_delivery_notes($lieferschein_id, group_lieferschein: true);
- if (!empty($check_delivery_notes)) {
- $lieferschein_minus_links = array();
- $lieferschein_plus_links = array();
- foreach ($check_delivery_notes as $check_delivery_note) {
- if ($check_delivery_note['anzahl_nummern'] < $check_delivery_note['menge_lieferschein']) {
- $lieferschein_minus_links[] = ''.$check_delivery_note['belegnr'].'';
- }
- else if ($check_delivery_note['anzahl_nummern'] > $check_delivery_note['menge']) {
- $lieferschein_plus_links[] = ''.$check_delivery_note['belegnr'].'';
- }
- }
- if (!empty($lieferschein_minus_links)) {
- $this->app->YUI->Message('warning','Seriennummern fehlen für Lieferschein: '.implode(', ',$lieferschein_minus_links));
- }
- if (!empty($lieferschein_plus_links)) {
- $this->app->YUI->Message('warning','Seriennummern Überschuss für Lieferschein: '.implode(', ',$lieferschein_plus_links));
- }
- }
- }
-
- function seriennummern_check_incoming_goods_and_message($wareneingang_id) {
-
- $check_incoming_goods = $this->seriennummern_check_incoming_goods($wareneingang_id, group_wareneingang: true);
-
- if (!empty($check_incoming_goods)) {
- $wareneingang_minus_links = array();
- $wareneingang_plus_links = array();
- foreach ($check_incoming_goods as $check_delivery_note) {
- if ($check_delivery_note['anzahl_nummern'] < $check_delivery_note['menge_wareneingang']) {
- $wareneingang_minus_links[] = ''.$check_delivery_note['belegnr'].'';
- }
- else if ($check_delivery_note['anzahl_nummern'] > $check_delivery_note['menge']) {
- $wareneingang_plus_links[] = ''.$check_delivery_note['belegnr'].'';
- }
- }
- if (!empty($wareneingang_minus_links)) {
- $this->app->YUI->Message('warning','Seriennummern fehlen für Wareneingang: '.implode(', ',$wareneingang_minus_links));
- }
- if (!empty($wareneingang_plus_links)) {
- $this->app->YUI->Message('warning','Seriennummern Überschuss für Wareneingang: '.implode(', ',$wareneingang_plus_links));
- }
- }
- }
-
+
function seriennummern_nummern_list() {
$this->seriennummern_menu();
@@ -675,8 +605,8 @@ class Seriennummern {
function seriennummern_menu_checks() {
$this->seriennummern_check_and_message(null);
- $this->seriennummern_delivery_note_check_and_message(null);
- $this->seriennummern_check_incoming_goods_and_message(null);
+ $this->seriennummern_check_and_message_delivery_notes(null);
+ $this->seriennummern_check_and_message_incoming_goods(null);
}
function seriennummern_artikel_list() {
@@ -1044,6 +974,9 @@ class Seriennummern {
}
+ /* --------------------------------------------
+ CHECKS
+ -------------------------------------------- */
/*
* Check if all serial numbers are given
* Return array of article ids
@@ -1095,7 +1028,7 @@ class Seriennummern {
* Check if all incoming goods notes have serials
* Return array of incoming goods note positions and head information
*/
- public function seriennummern_check_incoming_goods($wareneingang_id = null, $ignore_date = false, $only_missing = true, $group_wareneingang = false) : array {
+ public function seriennummern_check_incoming_goods($wareneingang_id = null, $only_missing = true, $group_wareneingang = false) : array {
if ($group_wareneingang) {
$sql_we = "''";
@@ -1132,16 +1065,7 @@ class Seriennummern {
) spd ON spd.paketdistribution = pd.id
WHERE
(a.seriennummern <> 'keine')
- AND (
- pa.datum >=(
- SELECT
- COALESCE(DATE(MIN(datum)),CURRENT_DATE())
- FROM
- seriennummern
- WHERE
- artikel = a.id
- ) OR ('".$ignore_date."' <> '')
- )
+ AND pa.status <> 'abgeschlossen'
AND (pa.id = '".$wareneingang_id."' OR '".$wareneingang_id."' = '')
GROUP BY
pa.id
@@ -1222,6 +1146,9 @@ class Seriennummern {
return(empty($result)?array():$result);
}
+ /* --------------------------------------------
+ NOTIFICATIONS
+ -------------------------------------------- */
protected function seriennummern_create_notification_artikel($artikel_id, $action, $title = 'Seriennummern', $message = 'Meldung', $button = 'Ok')
{
// Notification erstellen
@@ -1265,32 +1192,131 @@ class Seriennummern {
$notification = $this->app->Container->get('NotificationService');
$notification->createFromData($this->app->User->GetID(), $notification_message);
}
-
+
+ /* --------------------------------------------
+ CHECKS AND NOTIFICATIONS
+ -------------------------------------------- */
/*
* Check if new numbers need to be entered, if yes, create notification
*/
- public function seriennummern_check_and_message_stock_added(int $artikel_id) {
+ public function seriennummern_check_and_notification_stock_added(int $artikel_id) {
$check_seriennummern = $this->seriennummern_check_serials($artikel_id);
if ($check_seriennummern[0]['menge_nummern'] < $check_seriennummern[0]['menge_auf_lager']) {
$this->seriennummern_create_notification_artikel($artikel_id, 'enter', 'Seriennummern','Bitte Seriennummern für Einlagerung erfassen','Zur Eingabe');
}
- }
-
+ }
+
/*
* Check if numbers need to be entered after stock removal, if yes, create notification or message
*/
- public function seriennummern_check_and_message_delivery_note(int $lieferschein_id, bool $notification) {
+ public function seriennummern_check_and_notification_delivery_note(int $lieferschein_id) {
$check_delivery_notes = $this->seriennummern_check_delivery_notes($lieferschein_id);
if (!empty($check_delivery_notes)) {
- if ($notificiation) {
- $this->seriennummern_create_notification_lieferschein($lieferschein_id, 'enter', 'Seriennummern','Bitte Seriennummern für Lieferschein erfassen','Zur Eingabe');
- } else {
- $this->app->Tpl->Add('MESSAGE',"Seriennummern unvollständig!
");
- }
+ $this->seriennummern_create_notification_lieferschein($lieferschein_id, 'enter', 'Seriennummern','Bitte Seriennummern für Lieferschein erfassen','Zur Eingabe');
}
return($check_delivery_notes);
}
+ /* --------------------------------------------
+ CHECKS AND MESSAGES MODULE HEADER
+ -------------------------------------------- */
+ function seriennummern_check_and_message($artikel_id) {
+ $check_seriennummern = $this->seriennummern_check_serials($artikel_id);
+
+ if (!empty($check_seriennummern)) {
+ $artikel_minus_id_links = array();
+ $artikel_plus_id_links = array();
+ foreach ($check_seriennummern as $artikel_id) {
+ if ($artikel_id['menge_nummern'] < $artikel_id['menge_auf_lager']) {
+ $artikel_minus_id_links[] = ''.$artikel_id['nummer'].'';
+ }
+ else if ($artikel_id['menge_nummern'] > $artikel_id['menge_auf_lager']) {
+ $artikel_plus_id_links[] = ''.$artikel_id['nummer'].'';
+ }
+ }
+ if (!empty($artikel_minus_id_links)) {
+ $this->app->YUI->Message('warning','Seriennummern fehlen für Artikel: '.implode(', ',$artikel_minus_id_links));
+ }
+ if (!empty($artikel_plus_id_links)) {
+ $this->app->YUI->Message('warning','Seriennummern Überschuss für Artikel: '.implode(', ',$artikel_plus_id_links));
+ }
+ }
+ }
+
+ function seriennummern_check_and_message_incoming_goods($wareneingang_id) {
+
+ $check_incoming_goods = $this->seriennummern_check_incoming_goods($wareneingang_id, group_wareneingang: true);
+
+ if (!empty($check_incoming_goods)) {
+ if (empty($wareneingang_id)) {
+ $wareneingang_minus_links = array();
+ $wareneingang_plus_links = array();
+ foreach ($check_incoming_goods as $check_delivery_note) {
+ if ($check_delivery_note['anzahl_nummern'] < $check_delivery_note['menge_wareneingang']) {
+ $wareneingang_minus_links[] = ''.$check_delivery_note['belegnr'].'';
+ }
+ else if ($check_delivery_note['anzahl_nummern'] > $check_delivery_note['menge']) {
+ $wareneingang_plus_links[] = ''.$check_delivery_note['belegnr'].'';
+ }
+ }
+ if (!empty($wareneingang_minus_links)) {
+ $this->app->YUI->Message('warning','Seriennummern fehlen für Wareneingang: '.implode(', ',$wareneingang_minus_links));
+ }
+ if (!empty($wareneingang_plus_links)) {
+ $this->app->YUI->Message('warning','Seriennummern Überschuss für Wareneingang: '.implode(', ',$wareneingang_plus_links));
+ }
+ }
+ else {
+ $this->app->Tpl->Add('MESSAGE',"Seriennummern unvollständig!
");
+ }
+ }
+ return($check_incoming_goods);
+ }
+
+ function seriennummern_check_and_message_delivery_notes($lieferschein_id) {
+ $check_delivery_notes = $this->seriennummern_check_delivery_notes($lieferschein_id, group_lieferschein: true);
+ if (!empty($check_delivery_notes)) {
+ if (empty($lieferschein_id)) {
+ $lieferschein_minus_links = array();
+ $lieferschein_plus_links = array();
+ foreach ($check_delivery_notes as $check_delivery_note) {
+ if ($check_delivery_note['anzahl_nummern'] < $check_delivery_note['menge_lieferschein']) {
+ $lieferschein_minus_links[] = ''.$check_delivery_note['belegnr'].'';
+ }
+ else if ($check_delivery_note['anzahl_nummern'] > $check_delivery_note['menge']) {
+ $lieferschein_plus_links[] = ''.$check_delivery_note['belegnr'].'';
+ }
+ }
+ if (!empty($lieferschein_minus_links)) {
+ $this->app->YUI->Message('warning','Seriennummern fehlen für Lieferschein: '.implode(', ',$lieferschein_minus_links));
+ }
+ if (!empty($lieferschein_plus_links)) {
+ $this->app->YUI->Message('warning','Seriennummern Überschuss für Lieferschein: '.implode(', ',$lieferschein_plus_links));
+ }
+ } else {
+ $this->app->Tpl->Add('MESSAGE',"Seriennummern unvollständig!
");
+ }
+ }
+ return($check_delivery_notes);
+ }
+
+ /*
+ MINIDETAILS
+ */
+ public function seriennummern_minidetail($parsetarget='',$menu=true) {
+ $id = $this->app->Secure->GetGET('id');
+
+ if($parsetarget=='')
+ {
+ $tmp = new EasyTable($this->app);
+ $tmp->Query("SELECT l.belegnr AS Lieferschein, ".$this->app->erp->FormatDate('l.datum')." AS Datum, a.name AS Adresse FROM lieferschein l INNER JOIN adresse a ON a.id = l.adresse INNER JOIN lieferschein_position lp ON l.id = lp.lieferschein INNER JOIN seriennummern_lieferschein_position slp ON slp.lieferschein_position = lp.id WHERE slp.seriennummer ='$id' ",0,"");
+ $tmp->DisplayNew('TAB1',"Adresse","noAction");
+
+ $this->app->Tpl->Output('emptytab.tpl');
+ $this->app->ExitXentral();
+ }
+ }
+
public function seriennummern_lieferscheinpos_minidetail($parsetarget='',$menu=true) {
$id = $this->app->Secure->GetGET('id');
@@ -1304,18 +1330,6 @@ class Seriennummern {
$this->app->ExitXentral();
}
}
+
- public function seriennummern_minidetail($parsetarget='',$menu=true) {
- $id = $this->app->Secure->GetGET('id');
-
- if($parsetarget=='')
- {
- $tmp = new EasyTable($this->app);
- $tmp->Query("SELECT l.belegnr AS Lieferschein, ".$this->app->erp->FormatDate('l.datum')." AS Datum, a.name AS Adresse FROM lieferschein l INNER JOIN adresse a ON a.id = l.adresse INNER JOIN lieferschein_position lp ON l.id = lp.lieferschein INNER JOIN seriennummern_lieferschein_position slp ON slp.lieferschein_position = lp.id WHERE slp.seriennummer ='$id' ",0,"");
- $tmp->DisplayNew('TAB1',"Adresse","noAction");
-
- $this->app->Tpl->Output('emptytab.tpl');
- $this->app->ExitXentral();
- }
- }
}
diff --git a/www/pages/versandpakete.php b/www/pages/versandpakete.php
index e29bf326..58975674 100644
--- a/www/pages/versandpakete.php
+++ b/www/pages/versandpakete.php
@@ -814,6 +814,7 @@ class Versandpakete {
$this->app->erp->MenuEintrag("index.php?module=versandpakete&action=edit&id=".$id, "Details");
$this->app->Tpl->SetText('KURZUEBERSCHRIFT2', 'Artikel hinzufügen');
if (empty($id)) {
+
$lieferschein = $this->app->Secure->GetGET('lieferschein');
if (empty($lieferschein)) {
$msg = $this->app->erp->base64_url_encode("Kein Lieferschein angegeben.
");
@@ -874,8 +875,12 @@ class Versandpakete {
}
$lieferschein_belegnr = $this->app->erp->ReplaceLieferschein(false, $lieferschein, false); // Parameters: Target db?, value, from form?
$input['lieferschein'] = $lieferschein;
+ } else {
+
}
}
+
+ $this->app->erp->SeriennummernCheckLieferscheinWarnung(lieferschein_id: $lieferschein);
// Check if other users are editing this id
if($this->app->erp->DisableModul('versandpakete',$id))
diff --git a/www/pages/wareneingang.php b/www/pages/wareneingang.php
index 17153829..71288b5b 100644
--- a/www/pages/wareneingang.php
+++ b/www/pages/wareneingang.php
@@ -2014,14 +2014,29 @@ class Wareneingang {
$id = $this->app->Secure->GetGET('id');
$this->app->erp->MenuEintrag('index.php?module=wareneingang&action=distriinhalt&id='.$id, 'Details');
+
+
$this->app->Tpl->Add('KURZUEBERSCHRIFT', ' Paketannahme / Leistungserfassung');
$cmd = $this->app->Secure->GetGET('cmd');
$lsnr = $this->app->Secure->GetPOST('lsnr');
$renr = $this->app->Secure->GetPOST('renr');
- $bemerkung = $this->app->Secure->GetPOST('bemerkung');
-
+ $bemerkung = $this->app->Secure->GetPOST('bemerkung');
$bemerkung = str_replace(array('\r\n', '\r', '\n'), "\n", $bemerkung);
+ $seriennummern = $this->app->erp->SeriennummernCheckWareneingang(
+ wareneingang_id: $id,
+ ignore_date: true,
+ only_missing: false,
+ group_wareneingang: true);
+
+ $seriennummern_aktiv = !empty($seriennummern);
+
+ if ($seriennummern_aktiv) {
+ $this->app->erp->MenuEintrag('index.php?module=seriennummern&action=enter&wareneingang='.$id, 'Seriennummern');
+ $seriennummern_check_result = $this->app->erp->SeriennummernCheckWareneingangWarnung($id, false);
+ $seriennummern_ok = empty($seriennummern_check_result);
+ }
+
$this->app->User->SetParameter('table_wareneingang_lieferant_ausfuellen', '');
// Load from DB
@@ -2633,6 +2648,11 @@ class Wareneingang {
$this->app->YUI->Message('info','Wareneingang noch nicht abgeschlossen');
}
}
+
+ if ($seriennummern_aktiv && !$seriennummern_ok) {
+ $this->app->Tpl->Set('ABSCHLIESSENHIDDEN','hidden');
+ }
+
$this->app->Tpl->Parse('PAGE', 'wareneingang_paketinhalt.tpl');
}