From bca91008eddc6a1734a75fcd555f10d1c8ac65f6 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Bernd=20Ble=C3=9Fmann?= <bernd@kivitendo-premium.de>
Date: Fri, 31 Jan 2020 12:52:57 +0100
Subject: [PATCH] =?utf8?q?Individuelle=20Lieferadresse=20hinzuf=C3=BCgen:?=
 =?utf8?q?=20cp=5Fgender=20nicht=20vergessen?=
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

shiptocp_gender darf bei der Prüfung, ob die Lieferadresse hinzugefügt werden
soll (also nicht leer ist), nicht berücksichtigt werden, da shiptocp_gender nie
leer ist.
Allerdings muss shiptocp_gender sehr wohl beim INSERT oder UPDATE
berücksichtig werden.

Der Fehler kam mit
"Individuelle Lieferadresse hinzufügen: cp_gender nicht auf leer prüfen …"
(commit b8fa93b5d2cfff453fd8a82206d1a288407a8be6) rein.

Bezieht sich auch auf #410 (redmine)
Refs #410
---
 SL/Form.pm | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/SL/Form.pm b/SL/Form.pm
index dac601630..bf126cb55 100644
--- a/SL/Form.pm
+++ b/SL/Form.pm
@@ -1876,6 +1876,12 @@ sub add_shipto {
 
   return if !$shipto;
 
+  # shiptocp_gender only makes sense, if any other shipto attribute is set.
+  # Because shiptocp_gender is set to 'm' by default in forms
+  # it must not be considered above to decide if shiptos has to be added or
+  # updated, but must be inserted or updated as well in case.
+  push(@values, $self->{shiptocp_gender});
+
   my $shipto_id = $self->{shipto_id};
 
   if ($self->{shipto_id}) {
-- 
2.20.1