From 8736e198c8c28a0257fc850a80b4dace85b4e9e3 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Bernd=20Ble=C3=9Fmann?= <bernd@kivitendo-premium.de>
Date: Wed, 7 Nov 2018 11:25:44 +0100
Subject: [PATCH] =?utf8?q?Inventur:=20Fehlermeldung=20an=20Benutzer,=20wen?=
 =?utf8?q?n=20kein=20Artikel=20ausgew=C3=A4hlt=20wurde.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

---
 SL/Controller/Inventory.pm | 6 +++++-
 locale/de/all              | 1 +
 locale/en/all              | 1 +
 3 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/SL/Controller/Inventory.pm b/SL/Controller/Inventory.pm
index b417f69ce..e317381f0 100644
--- a/SL/Controller/Inventory.pm
+++ b/SL/Controller/Inventory.pm
@@ -498,6 +498,9 @@ sub action_stocktaking {
 sub action_save_stocktaking {
   my ($self) = @_;
 
+  return $self->js->flash('error', t8('Please choose a part.'))->render()
+    if !$::form->{part_id};
+
   return $self->js->flash('error', t8('A target quantitiy has to be given'))->render()
     if $::form->{target_qty} eq '';
 
@@ -607,6 +610,7 @@ sub action_stocktaking_journal {
 sub action_stocktaking_get_warn_qty_threshold {
   my ($self) = @_;
 
+  return $_[0]->render(\ !!0, { type => 'text' }) if !$::form->{part_id};
   return $_[0]->render(\ !!0, { type => 'text' }) if $::form->{target_qty} eq '';
   return $_[0]->render(\ !!0, { type => 'text' }) if 0 == $::instance_conf->get_stocktaking_qty_threshold;
 
@@ -724,7 +728,7 @@ sub sanitize_target {
 }
 
 sub load_part_from_form {
-  $_[0]->part(SL::DB::Manager::Part->find_by_or_create(id => $::form->{part_id}));
+  $_[0]->part(SL::DB::Manager::Part->find_by_or_create(id => $::form->{part_id}||undef));
 }
 
 sub load_unit_from_form {
diff --git a/locale/de/all b/locale/de/all
index 9a3d123cc..290a649fa 100755
--- a/locale/de/all
+++ b/locale/de/all
@@ -2264,6 +2264,7 @@ $self->{texts} = {
   'Please Check the bank information for each vendor:' => 'Bitte überprüfen Sie die Kontoinformationen der Lieferanten:',
   'Please ask your administrator to create warehouses and bins.' => 'Bitten Sie Ihren Administrator, dass er Lager und Lagerplätze anlegt.',
   'Please change the partnumber of the following parts and run the update again:' => 'Bitte ändern Sie daher die Artikelnummer folgender Artikel:',
+  'Please choose a part.'       => 'Bitte wählen Sie einen Artikel aus.',
   'Please choose for which categories the taxes should be displayed (otherwise remove the ticks):' => 'Bitte wählen Sie für welche Kontoart die Steuer angezeigt werden soll (ansonsten einfach die Häkchen entfernen)',
   'Please choose the action to be processed for your target quantity:' => 'Bitte wählen Sie eine Aktion, die mit Ihrer gezählten Zielmenge durchgeführt werden soll:',
   'Please contact your administrator or a service provider.' => 'Bitte kontaktieren Sie Ihren Administrator oder einen Dienstleister.',
diff --git a/locale/en/all b/locale/en/all
index c2ee36627..6a0002908 100644
--- a/locale/en/all
+++ b/locale/en/all
@@ -2264,6 +2264,7 @@ $self->{texts} = {
   'Please Check the bank information for each vendor:' => '',
   'Please ask your administrator to create warehouses and bins.' => '',
   'Please change the partnumber of the following parts and run the update again:' => '',
+  'Please choose a part.'       => '',
   'Please choose for which categories the taxes should be displayed (otherwise remove the ticks):' => '',
   'Please choose the action to be processed for your target quantity:' => '',
   'Please contact your administrator or a service provider.' => '',
-- 
2.20.1