From: Moritz Bunkus Date: Thu, 26 Jun 2014 07:12:38 +0000 (+0200) Subject: Einkauf/Verkauf: Lieferadressenfelder nie aus Stammdaten vorbelegen X-Git-Tag: release-3.2.0beta~411^2~4 X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=b6213d3539ccd179cd1f21b9afc54b8de8970774;p=kivitendo-erp.git Einkauf/Verkauf: Lieferadressenfelder nie aus Stammdaten vorbelegen Das neue Verhalten ist wie folgt: - Weder die shipto_id (die Drop-Down-Box in den Belegmasken) noch die individuellen shipto*-Felder werden weder beim Neuanlegen eines Beleges noch bei Wechsel des Kunden aus den Datenbanken belegt. - Beim Ausdruck werden die shipto*-Felder nicht mehr aus der Mandantenkonfiguration vorbelegt, wenn keine Lieferadresse gesetzt ist. - Beim Ausdruck werden die shipto*-Felder mit den Daten aus den Kundenstammdaten belegt, wenn die shipto_id (die Drop-Down-Box in den Belegmasken) gesetzt ist. Die ursprüngliche Intention war, möglichst viele Fälle abzudecken. Ganz ursprünglich war es nämlich in den Druckvorlagen gar nicht möglich, Kontrollstrukturen zu benutzen und damit die Ausgabe konditional zu steuern. Es konnte also rein in den Druckvorlagen nicht unterschieden werden zwischen »der Benutzer hat keine Lieferadresse eingegeben« und »der Benutzer hat eine eingegeben oder ausgewählt«. Daher wurde die ganze Logik immer im Perl-Code abgehandelt. Das macht aber erhebliche Probleme für den Benutzer, für den es absolut intransparent ist, wann welche Lieferadresse wie vorbelegt wird. Hinzu kommt, dass in den Belegmasken nicht ersichtlich ist, dass eine individuelle Lieferadresse eingetragen wurde. Hinzu kommt, dass die Druckvorlagen inzwischen verschiedene Mechanismen zur Verfügung haben, um Fallunterscheidungen zu treffen (z.B. die kivitendo-Mechanismen $(if shipto…)$ oder die LaTeX-eigenen \IfThenElse{\equal{$(shipto…)$}{…}}}…). Leider war es mit dem vorherigen Code für die Druckvorlagen nicht mehr möglich festzustellen, ob der Benutzer nun eine Lieferadresse eingegeben hat oder nicht. Die neue Situation ist recht einfach: Steht in »shiptoname« oder »shiptostreet« ein nicht leerer Wert, so ist eine Lieferadresse vorhanden, ansonsten nicht. Für die »nicht«-Fall kann dann jede Vorlage selber entscheiden, was zu tun ist. Für Vorlagen im Verkaufsbereich sinnvollerweise gar keine Lieferadresse ausgeben (oder einfach die Lieferadresse aus den Kundenrechnungsdaten), für Vorlagen im Einkaufsbereich ebenfalls keine Lieferadresse oder die Adresse aus der Mandantenkonfiguration. Behebt #2345, #2400. --- diff --git a/SL/DO.pm b/SL/DO.pm index 1da5a8e2d..0960ad71f 100644 --- a/SL/DO.pm +++ b/SL/DO.pm @@ -391,10 +391,10 @@ sub save { conv_i($form->{id})); do_query($form, $dbh, $query, @values); - # add shipto $form->{name} = $form->{ $form->{vc} }; $form->{name} =~ s/--$form->{"$form->{vc}_id"}//; + # add shipto if (!$form->{shipto_id}) { $form->add_shipto($dbh, $form->{id}, "DO"); } diff --git a/SL/Form.pm b/SL/Form.pm index 9f0cf11bc..f35327873 100644 --- a/SL/Form.pm +++ b/SL/Form.pm @@ -3337,11 +3337,10 @@ sub prepare_for_printing { $self->{"employee_${_}"} = $defaults->$_ for qw(address businessnumber co_ustid company duns sepa_creditor_id taxnumber); } - # set shipto from billto unless set - my $has_shipto = any { $self->{"shipto$_"} } qw(name street zipcode city country contact); - if (!$has_shipto && ($self->{type} =~ m/^(?:purchase_order|request_quotation)$/)) { - $self->{shiptoname} = $defaults->company; - $self->{shiptostreet} = $defaults->address; + # Load shipping address from database if shipto_id is set. + if ($self->{shipto_id}) { + my $shipto = SL::DB::Shipto->new(id => $self->{shipto_id})->load; + $self->{$_} = $shipto->$_ for grep { m{^shipto} } map { $_->name } @{ $shipto->meta->columns }; } my $language = $self->{language} ? '_' . $self->{language} : ''; diff --git a/SL/IR.pm b/SL/IR.pm index 301fe3b64..931df6b44 100644 --- a/SL/IR.pm +++ b/SL/IR.pm @@ -720,9 +720,10 @@ sub post_invoice { } - # add shipto $form->{name} = $form->{vendor}; $form->{name} =~ s/--\Q$form->{vendor_id}\E//; + + # add shipto $form->add_shipto($dbh, $form->{id}, "AP"); # delete zero entries @@ -1126,16 +1127,6 @@ sub get_vendor { } $sth->finish(); - # get shipto if we do not convert an order or invoice - if (!$params->{shipto}) { - delete @{$params}{qw(shiptoname shiptostreet shiptozipcode shiptocity shiptocountry shiptocontact shiptophone shiptofax shiptoemail)}; - - $query = qq|SELECT * FROM shipto WHERE (trans_id = ?) AND (module= 'CT')|; - $ref = selectfirst_hashref_query($form, $dbh, $query, $vid); - @{$params}{keys %$ref} = @{$ref}{keys %$ref}; - map { $params->{$_} = $ref->{$_} } keys %$ref; - } - if (!$params->{id} && $params->{type} !~ /_(order|quotation)/) { # setup last accounts used $query = diff --git a/SL/IS.pm b/SL/IS.pm index 64afc530a..63420a544 100644 --- a/SL/IS.pm +++ b/SL/IS.pm @@ -1145,10 +1145,10 @@ sub post_invoice { do_query($form, $dbh, qq|UPDATE ar SET paid = amount WHERE id = ?|, conv_i($form->{"id"})); } - # add shipto $form->{name} = $form->{customer}; $form->{name} =~ s/--\Q$form->{customer_id}\E//; + # add shipto if (!$form->{shipto_id}) { $form->add_shipto($dbh, $form->{id}, "AR"); } @@ -1615,12 +1615,6 @@ sub retrieve_invoice { $form->{exchangerate} = $form->get_exchangerate($dbh, $form->{currency}, $form->{invdate}, "buy"); - # get shipto - $query = qq|SELECT * FROM shipto WHERE (trans_id = ?) AND (module = 'AR')|; - $ref = selectfirst_hashref_query($form, $dbh, $query, $id); - delete $ref->{id}; - map { $form->{$_} = $ref->{$_} } keys %{ $ref }; - foreach my $vc (qw(customer vendor)) { next if !$form->{"delivery_${vc}_id"}; ($form->{"delivery_${vc}_string"}) = selectrow_query($form, $dbh, qq|SELECT name FROM customer WHERE id = ?|, $id); @@ -1837,19 +1831,6 @@ sub get_customer { } $sth->finish; - # get shipto if we did not converted an order or invoice - if (!$form->{shipto}) { - map { delete $form->{$_} } - qw(shiptoname shiptodepartment_1 shiptodepartment_2 - shiptostreet shiptozipcode shiptocity shiptocountry - shiptocontact shiptophone shiptofax shiptoemail); - - $query = qq|SELECT * FROM shipto WHERE trans_id = ? AND module = 'CT'|; - $ref = selectfirst_hashref_query($form, $dbh, $query, $cid); - delete $ref->{id}; - map { $form->{$_} = $ref->{$_} } keys %$ref; - } - # setup last accounts used for this customer if (!$form->{id} && $form->{type} !~ /_(order|quotation)/) { $query = diff --git a/SL/OE.pm b/SL/OE.pm index b6f653aed..8b073b9c5 100644 --- a/SL/OE.pm +++ b/SL/OE.pm @@ -609,10 +609,10 @@ sub save { $form->{ordtotal} = $amount; - # add shipto $form->{name} = $form->{ $form->{vc} }; $form->{name} =~ s/--\Q$form->{"$form->{vc}_id"}\E//; + # add shipto if (!$form->{shipto_id}) { $form->add_shipto($dbh, $form->{id}, "OE"); } diff --git a/bin/mozilla/do.pl b/bin/mozilla/do.pl index 3e5e9dedc..c16f772d1 100644 --- a/bin/mozilla/do.pl +++ b/bin/mozilla/do.pl @@ -179,7 +179,6 @@ sub order_links { 'ids' => $form->{id}); $form->backup_vars(qw(payment_id language_id taxzone_id salesman_id taxincluded cp_id intnotes delivery_term_id currency)); - $form->{shipto} = 1 if $form->{id} || $form->{convert_from_oe_ids}; # get customer / vendor if ($form->{vc} eq 'vendor') { @@ -801,7 +800,6 @@ sub invoice { require "bin/mozilla/$form->{script}"; my $currency = $form->{currency}; - $form->{shipto} = 1 if $form->{convert_from_do_ids}; invoice_links(); if ($form->{ordnumber}) { diff --git a/bin/mozilla/io.pl b/bin/mozilla/io.pl index 7fc1ef1b8..97218877b 100644 --- a/bin/mozilla/io.pl +++ b/bin/mozilla/io.pl @@ -869,8 +869,6 @@ sub order { } $form->{script} = 'oe.pl'; - $form->{shipto} = 1; - $form->{rowcount}--; $form->{cp_id} *= 1; @@ -935,8 +933,6 @@ sub quotation { $form->{script} = 'oe.pl'; - $form->{shipto} = 1; - $form->{rowcount}--; require "bin/mozilla/$form->{script}"; @@ -1417,28 +1413,6 @@ sub print_form { $form->get_shipto(\%myconfig); } - my @a = qw(name department_1 department_2 street zipcode city country contact phone fax email); - - my $shipto = 1; - - # if there is no shipto fill it in from billto - foreach my $item (@a) { - if ($form->{"shipto$item"}) { - $shipto = 0; - last; - } - } - - if ($shipto) { - if ( $form->{formname} eq 'purchase_order' - || $form->{formname} eq 'request_quotation') { - $form->{shiptoname} = $defaults->company; - $form->{shiptostreet} = $defaults->address; - } else { - map { $form->{"shipto$_"} = $form->{$_} } @a; - } - } - $form->{notes} =~ s/^\s+//g; delete $form->{printer_command}; diff --git a/bin/mozilla/is.pl b/bin/mozilla/is.pl index ef8f5996e..b6c6298a0 100644 --- a/bin/mozilla/is.pl +++ b/bin/mozilla/is.pl @@ -155,7 +155,6 @@ sub invoice_links { taxincluded currency cp_id intnotes id shipto_id delivery_term_id)); - $form->{shipto} = 1 if $editing || $form->{convert_from_oe_ids} || $form->{convert_from_do_ids}; IS->get_customer(\%myconfig, \%$form); #quote all_customer Bug 133 @@ -962,7 +961,6 @@ sub credit_note { $form->{id} = ''; $form->{rowcount}--; - $form->{shipto} = 1; $form->{title} = $locale->text('Add Credit Note'); diff --git a/bin/mozilla/oe.pl b/bin/mozilla/oe.pl index 4fff4f953..9a8aea41e 100644 --- a/bin/mozilla/oe.pl +++ b/bin/mozilla/oe.pl @@ -257,7 +257,6 @@ sub order_links { $form->{"$form->{vc}_id"} ||= $form->{"all_$form->{vc}"}->[0]->{id} if $form->{"all_$form->{vc}"}; $form->backup_vars(qw(payment_id language_id taxzone_id salesman_id taxincluded cp_id intnotes shipto_id delivery_term_id currency)); - $form->{shipto} = 1 if $form->{id} || $form->{convert_from_oe_ids}; # get customer / vendor IR->get_vendor(\%myconfig, \%$form) if $form->{type} =~ /(purchase_order|request_quotation)/; @@ -1429,7 +1428,6 @@ sub invoice { $form->{convert_from_oe_ids} = $form->{id}; $form->{transdate} = $form->{invdate} = $form->current_date(\%myconfig); $form->{duedate} = $form->current_date(\%myconfig, $form->{invdate}, $form->{terms} * 1); - $form->{shipto} = 1; $form->{defaultcurrency} = $form->get_default_currency(\%myconfig); delete @{$form}{qw(id closed)}; @@ -1662,7 +1660,6 @@ sub check_for_direct_delivery_yes { $form->{direct_delivery_checked} = 1; delete @{$form}{grep /^shipto/, keys %{ $form }}; map { s/^CFDD_//; $form->{$_} = $form->{"CFDD_${_}"} } grep /^CFDD_/, keys %{ $form }; - $form->{shipto} = 1; $form->{CFDD_shipto} = 1; purchase_order(); $main::lxdebug->leave_sub();