From: Bernd Bleßmann <bernd@kivitendo-premium.de>
Date: Fri, 31 Jan 2020 11:52:57 +0000 (+0100)
Subject: Individuelle Lieferadresse hinzufügen: cp_gender nicht vergessen
X-Git-Tag: release-3.5.6.1~377
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=bca91008eddc6a1734a75fcd555f10d1c8ac65f6;p=kivitendo-erp.git

Individuelle Lieferadresse hinzufügen: cp_gender nicht vergessen

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
---

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}) {