From: Bernd Bleßmann <bernd@kivitendo-premium.de>
Date: Fri, 5 Jul 2019 12:07:15 +0000 (+0200)
Subject: Auftrags-Controller: item-ids nach Speichern richtig setzen
X-Git-Tag: release-3.5.4~46
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=b36180dee0d6dd13d7def595f374cbe117d9bae5;p=kivitendo-erp.git

Auftrags-Controller: item-ids nach Speichern richtig setzen

Vergessen, den idx in jedem Fall weiterzuzählen. Dadurch konnte es passieren,
das Positionen aus dem Auftrag gelöscht und evtl. Langtexte und Werte der
2. Zeile falsch zugeordnet wurden.

Nachtrag zu:
commit 7749e0e61e5d431a59e33cf497addf9ad682b8af
Author: Bernd Bleßmann <bernd@kivitendo-premium.de>
Date:   Wed May 15 15:18:21 2019 +0200

    OrderController: nach Speichern hiddens der ids wieder setzen, …

    … falls die Makse nicht neu geladen wird.
---

diff --git a/SL/Controller/Order.pm b/SL/Controller/Order.pm
index 250027022..f6f6986a1 100644
--- a/SL/Controller/Order.pm
+++ b/SL/Controller/Order.pm
@@ -1070,6 +1070,7 @@ sub js_reset_order_and_item_ids_after_save {
       ->val ('[name="orderitem_ids[+]"][value="' . $form_item_id . '"]', $self->order->items_sorted->[$idx]->id)
       ->val ('#item_' . $form_item_id, $self->order->items_sorted->[$idx]->id)
       ->attr('#item_' . $form_item_id, "id", 'item_' . $self->order->items_sorted->[$idx]->id);
+  } continue {
     $idx++;
   }
   $self->js->val('[name="converted_from_orderitems_ids[+]"]', '');