From: Martin Helmling martin.helmling@octosoft.eu <martin.helmling@octosoft.eu>
Date: Thu, 19 Mar 2020 16:25:31 +0000 (+0100)
Subject: Inventory Helper: Bessere Fehlermeldung bei fehlenden Parametern
X-Git-Tag: kivitendo-mebil_0.1-0~9^2~600
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=0845ca9d3bd7adc7689a34d562c3ff0371711514;p=kivitendo-erp.git

Inventory Helper: Bessere Fehlermeldung bei fehlenden Parametern

Alle Fehler im allocate() des Inventory-Helpers werden nun per
SL::X::Inventory::Allocation Fehlermethode erzeugt,
damit es eine einheitliche Prüfung geben kann.

verbessert egw 12413
---

diff --git a/SL/Helper/Inventory.pm b/SL/Helper/Inventory.pm
index 538876666..cd5090780 100644
--- a/SL/Helper/Inventory.pm
+++ b/SL/Helper/Inventory.pm
@@ -167,8 +167,17 @@ sub get_onhand {
 sub allocate {
   my (%params) = @_;
 
-  my $part = $params{part} or Carp::croak('allocate needs a part');
-  my $qty  = $params{qty}  or Carp::croak('allocate needs a qty');
+  die SL::X::Inventory::Allocation->new(
+    error => 'allocate needs a part',
+    msg => t8("Method allocate needs the parameter 'part'"),
+  ) unless $params{part};
+  die SL::X::Inventory::Allocation->new(
+    error => 'allocate needs a qty',
+    msg => t8("Method allocate needs the parameter 'qty'"),
+  ) unless $params{qty};
+
+  my $part = $params{part};
+  my $qty  = $params{qty};
 
   return () if $qty <= 0;
 
diff --git a/locale/de/all b/locale/de/all
index 4f3384afd..202108d70 100755
--- a/locale/de/all
+++ b/locale/de/all
@@ -1963,6 +1963,8 @@ $self->{texts} = {
   'Meta tag keywords'           => 'Metatag Keywords',
   'Meta tag title'              => 'Metatag Titel',
   'Method'                      => 'Verfahren',
+  'Method allocate needs the parameter \'part\'' => 'Es fehlt der Parameter \'part\'',
+  'Method allocate needs the parameter \'qty\'' => 'Es fehlt der Parameter \'qty\' oder die Menge ist 0',
   'Microfiche'                  => 'Mikrofilm',
   'Minimum Amount'              => 'Mindestbetrag',
   'Miscellaneous'               => 'Verschiedenes',