From eb0b223d2716d3fcffb9a3d806506ca4e396f4f8 Mon Sep 17 00:00:00 2001
From: "Martin Helmling martin.helmling@octosoft.eu"
 <martin.helmling@octosoft.eu>
Date: Mon, 12 Aug 2019 10:18:02 +0200
Subject: [PATCH] Inventory Helper: Artikel laden und Fehlermeldung korrigiert

Damit beim Fehler der Name des Artikels kommt, muss dieser geladen
werden.
Im Fehlerstring wird zuerst der Fehler , dann der Text ausgegeben
---
 SL/Helper/Inventory.pm | 5 ++---
 SL/X.pm                | 4 ++--
 locale/de/all          | 2 ++
 3 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/SL/Helper/Inventory.pm b/SL/Helper/Inventory.pm
index feb57e449..5fc15ea0e 100644
--- a/SL/Helper/Inventory.pm
+++ b/SL/Helper/Inventory.pm
@@ -214,10 +214,9 @@ sub allocate {
 
     last if $rest_qty == 0;
   }
-
   if ($rest_qty > 0) {
     die SL::X::Inventory::Allocation->new(
-      error => 'not enough to allocate',
+      error => t8('not enough to allocate'),
       msg => t8("can not allocate #1 units of #2, missing #3 units", $qty, $part->displayable_name, $rest_qty),
     );
   } else {
@@ -243,7 +242,7 @@ sub allocate_for_assembly {
   my @allocations;
 
   for my $part_id (keys %parts_to_allocate) {
-    my $part = SL::DB::Part->new(id => $part_id);
+    my $part = SL::DB::Part->load_cached($part_id);
     push @allocations, allocate(%params, part => $part, qty => $parts_to_allocate{$part_id});
   }
 
diff --git a/SL/X.pm b/SL/X.pm
index 4acd3bbab..c3533e2ef 100644
--- a/SL/X.pm
+++ b/SL/X.pm
@@ -33,12 +33,12 @@ use Exception::Class (
   'SL::X::Inventory' => {
     isa                 => 'SL::X::Base',
     fields              => [ qw(msg error) ],
-    defaults            => { error_template => [ '%s: %s', qw(msg error) ] },
+    defaults            => { error_template => [ '%s: %s', qw(msg) ] },
   },
   'SL::X::Inventory::Allocation' => {
     isa                 => 'SL::X::Base',
     fields              => [ qw(msg error) ],
-    defaults            => { error_template => [ '%s: %s', qw(msg error) ] },
+    defaults            => { error_template => [ '%s: %s', qw(msg) ] },
   },
 );
 
diff --git a/locale/de/all b/locale/de/all
index b0954aa52..ead66118f 100755
--- a/locale/de/all
+++ b/locale/de/all
@@ -4121,6 +4121,7 @@ $self->{texts} = {
   'brutto'                      => 'brutto',
   'building data'               => 'Verarbeite Daten',
   'building report'             => 'Erstelle Bericht',
+  'can not allocate #1 units of #2, missing #3 units' => 'Kann keine #1 Einheiten von #2 belegen, es fehlen #3 Einheiten',
   'can only parse a pdf file'   => 'Kann nur eine gültige PDF-Datei verwenden.',
   'cash'                        => 'Ist-Versteuerung',
   'chargenumber #1'             => 'Chargennummer #1',
@@ -4249,6 +4250,7 @@ $self->{texts} = {
   'no tax_id in acc_trans'      => 'Keine tax_id in acc_trans',
   'not configured'              => 'nicht konfiguriert',
   'not delivered'               => 'nicht geliefert',
+  'not enough to allocate'      => 'Keine Verfügbarkeit',
   'not executed'                => 'nicht ausgeführt',
   'not running'                 => 'läuft nicht',
   'not set'                     => 'nicht gesetzt',
-- 
2.20.1