From: Jan Büren <jan@kivitendo-premium.de>
Date: Thu, 17 Aug 2017 08:38:16 +0000 (+0200)
Subject: Fixt: Auslagern über Standardlagerplatz ohne Auslagern von Dienstleistungen
X-Git-Tag: release-3.5.4~883
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=8797da6a0a0330c63f85f2be0cb9d600c4ff7900;p=kivitendo-erp.git

Fixt: Auslagern über Standardlagerplatz ohne Auslagern von Dienstleistungen

Die Prüfung, ob es sich um eine Dienstleistung handelt, wird
über part_type gemacht.
Andernfalls wird die auszulagernde Menge in diesem Fall bei
allen Positionen auf 0 gesetzt.
Zusätzlich die Prüfung auf gesetzte Menge über das Fragezeichen-Feld
entfernt (ist mit ActionBar reingekommen).
---

diff --git a/bin/mozilla/do.pl b/bin/mozilla/do.pl
index d1f112963..d4b2f9e75 100644
--- a/bin/mozilla/do.pl
+++ b/bin/mozilla/do.pl
@@ -296,7 +296,7 @@ sub setup_do_action_bar {
         action => [
           t8('Transfer out via default'),
           submit   => [ '#form', { action => "transfer_out_default" } ],
-          checks   => [ @req_trans_desc, @transfer_qty ],
+          checks   => [ @req_trans_desc ],
           disabled => $::form->{delivered} ? t8('This record has already been delivered.') : undef,
           only_if  => $is_customer && $::instance_conf->get_transfer_default,
         ],
@@ -310,7 +310,7 @@ sub setup_do_action_bar {
         action => [
           t8('Transfer in via default'),
           submit   => [ '#form', { action => "transfer_in_default" } ],
-          checks   => [ @req_trans_desc, @transfer_qty ],
+          checks   => [ @req_trans_desc ],
           disabled => $::form->{delivered} ? t8('This record has already been delivered.') : undef,
           only_if  => !$is_customer && $::instance_conf->get_transfer_default,
         ],
@@ -1808,7 +1808,7 @@ sub transfer_in_out_default {
       # ... and do not create a hash entry in %qty_parts below (will skip check for bins for the transfer == out case)
       # ... and push only a empty (undef) element to @all_requests (will skip check for bin_id and warehouse_id and will not alter the row)
 
-      $qty = 0 if (!$::instance_conf->get_transfer_default_services && !defined($part_info_map{$form->{"id_$i"}}->{inventory_accno_id}) && !$part_info_map{$form->{"id_$i"}}->{assembly});
+      $qty = 0 if (!$::instance_conf->get_transfer_default_services && $part_info_map{$form->{"id_$i"}}->{part_type} eq 'service');
       $qty_parts{$form->{"id_$i"}} += $qty;
       if ($qty == 0) {
         delete $qty_parts{$form->{"id_$i"}} unless $qty_parts{$form->{"id_$i"}};