Revision b6213d35
Von Moritz Bunkus vor mehr als 10 Jahren hinzugefügt
| SL/IS.pm | ||
|---|---|---|
| 1145 | 1145 |
do_query($form, $dbh, qq|UPDATE ar SET paid = amount WHERE id = ?|, conv_i($form->{"id"}));
|
| 1146 | 1146 |
} |
| 1147 | 1147 |
|
| 1148 |
# add shipto |
|
| 1149 | 1148 |
$form->{name} = $form->{customer};
|
| 1150 | 1149 |
$form->{name} =~ s/--\Q$form->{customer_id}\E//;
|
| 1151 | 1150 |
|
| 1151 |
# add shipto |
|
| 1152 | 1152 |
if (!$form->{shipto_id}) {
|
| 1153 | 1153 |
$form->add_shipto($dbh, $form->{id}, "AR");
|
| 1154 | 1154 |
} |
| ... | ... | |
| 1615 | 1615 |
|
| 1616 | 1616 |
$form->{exchangerate} = $form->get_exchangerate($dbh, $form->{currency}, $form->{invdate}, "buy");
|
| 1617 | 1617 |
|
| 1618 |
# get shipto |
|
| 1619 |
$query = qq|SELECT * FROM shipto WHERE (trans_id = ?) AND (module = 'AR')|; |
|
| 1620 |
$ref = selectfirst_hashref_query($form, $dbh, $query, $id); |
|
| 1621 |
delete $ref->{id};
|
|
| 1622 |
map { $form->{$_} = $ref->{$_} } keys %{ $ref };
|
|
| 1623 |
|
|
| 1624 | 1618 |
foreach my $vc (qw(customer vendor)) {
|
| 1625 | 1619 |
next if !$form->{"delivery_${vc}_id"};
|
| 1626 | 1620 |
($form->{"delivery_${vc}_string"}) = selectrow_query($form, $dbh, qq|SELECT name FROM customer WHERE id = ?|, $id);
|
| ... | ... | |
| 1837 | 1831 |
} |
| 1838 | 1832 |
$sth->finish; |
| 1839 | 1833 |
|
| 1840 |
# get shipto if we did not converted an order or invoice |
|
| 1841 |
if (!$form->{shipto}) {
|
|
| 1842 |
map { delete $form->{$_} }
|
|
| 1843 |
qw(shiptoname shiptodepartment_1 shiptodepartment_2 |
|
| 1844 |
shiptostreet shiptozipcode shiptocity shiptocountry |
|
| 1845 |
shiptocontact shiptophone shiptofax shiptoemail); |
|
| 1846 |
|
|
| 1847 |
$query = qq|SELECT * FROM shipto WHERE trans_id = ? AND module = 'CT'|; |
|
| 1848 |
$ref = selectfirst_hashref_query($form, $dbh, $query, $cid); |
|
| 1849 |
delete $ref->{id};
|
|
| 1850 |
map { $form->{$_} = $ref->{$_} } keys %$ref;
|
|
| 1851 |
} |
|
| 1852 |
|
|
| 1853 | 1834 |
# setup last accounts used for this customer |
| 1854 | 1835 |
if (!$form->{id} && $form->{type} !~ /_(order|quotation)/) {
|
| 1855 | 1836 |
$query = |
Auch abrufbar als: Unified diff
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.