From: Sven Schöling <s.schoeling@linet-services.de>
Date: Thu, 21 Oct 2010 12:48:39 +0000 (+0200)
Subject: Multibox: Keine Variablen in den rows speichern
X-Git-Tag: release-2.6.2beta1~92^2~4
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=4dbd40c0260327ca136a5c0f49c67822e837f451;p=kivitendo-erp.git

Multibox: Keine Variablen in den rows speichern

2 Gründe:

1. Die rows können Objekte sein, und sobald die entweder nicht auf Hashref
   basieren, oder per AUTOLOAD ihre methoden sauber prüfen gibt das Chaos.
2. Wenn keine Daten da reingespeichert werden, bruachen die multiboxes nicht
   mit INCLUDE aufgerufen werden, sondern können mit PROCESS den stack clone
   umgehen.
---

diff --git a/templates/webpages/generic/multibox.html b/templates/webpages/generic/multibox.html
index b26f86f0d..72d895747 100644
--- a/templates/webpages/generic/multibox.html
+++ b/templates/webpages/generic/multibox.html
@@ -35,15 +35,6 @@
 [%-
   name       = (select_name != '' and ! show_text) ? select_name : name
 -%]
-[%- FOREACH row = DATA %]
-  [%-
-       row.id       = row.$id_key     != ''  ? row.$id_key    : $id_sub(row)
-       row.label    = row.$label_key  != ''  ? row.$label_key
-                    : $label_sub(row) != ''  ? $label_sub(row)
-                    :                          row.id
-       row.selected = default == row.id
-  -%]
-[%- END -%]
 [%- IF show_text %]
 <input type="text"
  [%- IF name     %] name="[%     HTML.escape(name)     %]"[% END -%]
@@ -67,7 +58,11 @@
   <option value=""></option>
   [%- END %]
   [%- FOREACH row = DATA %]
-  <option value="[% row.id | html %]"[% IF row.selected %] selected[% END %]>[% HTML.escape(row.label) %]</option>
+  <option value="[% row.$id_key != '' ? row.$id_key : $id_sub(row) | html %]"[% IF default == (row.$id_key != '' ? row.$id_key : $id_sub(row)) %] selected[% END %]>[%
+                      row.$label_key  != ''  ? row.$label_key
+                    : $label_sub(row) != ''  ? $label_sub(row)
+                    : row.$id_key     != ''  ? row.$id_key
+                    :                          $id_sub(row) | html %]</option>
   [%- END %]
 </select>
 [%- END %]