DeliveryOrder->new_from: kein $custom_shipto-Objekt zurückgeben
Falls das Quellobjekt eine individuelle Lieferadresse besaß, wurden bei new_from() zwei Objekte zurückgegeben: das neue Lieferscheinobjekt und ein Clone der individuellen Lieferadresse. Diese waren nicht verknüpft. Der Aufrufer musste daher zuerst das Lieferscheinobjekt speichern, dessen ID beim gecloneten Lieferadressenobjekt hinterlegen und das anschließend speichern.
Dies ist umständlich und fehlerträchtig. So hat z.B. der einzige bisherige Nutzer dieses Interfaces, SL::DB::Order->convert_to_delivery_order, das bereits falsch gemacht und vergessen, beim Lieferadressenobjekt die ID des neuen Lieferscheinobjektes einzutragen. Somit wurden Lieferadressen erzeugt, die keinerlei Verknüpfung hatten.
Das geänderte Interface hinterlegt das Objekt für die individuelle Lieferadresse schlicht in $new_delivery_order->custom_shipto. Dort wird das Objekt gespeichert, wenn der Lieferschein selber gespeichert wird.
DeliveryOrder->new_from: kein $custom_shipto-Objekt zurückgeben
Falls das Quellobjekt eine individuelle Lieferadresse besaß, wurden bei
new_from() zwei Objekte zurückgegeben: das neue Lieferscheinobjekt und
ein Clone der individuellen Lieferadresse. Diese waren nicht verknüpft.
Der Aufrufer musste daher zuerst das Lieferscheinobjekt speichern,
dessen ID beim gecloneten Lieferadressenobjekt hinterlegen und das
anschließend speichern.
Dies ist umständlich und fehlerträchtig. So hat z.B. der einzige
bisherige Nutzer dieses Interfaces,
SL::DB::Order->convert_to_delivery_order, das bereits falsch gemacht und
vergessen, beim Lieferadressenobjekt die ID des neuen
Lieferscheinobjektes einzutragen. Somit wurden Lieferadressen erzeugt,
die keinerlei Verknüpfung hatten.
Das geänderte Interface hinterlegt das Objekt für die individuelle
Lieferadresse schlicht in $new_delivery_order->custom_shipto. Dort wird
das Objekt gespeichert, wenn der Lieferschein selber gespeichert wird.