Univention Bugzilla – Bug 26780
MultiObjectSelect-Widget formatter und staticValues im Form-Widget
Last modified: 2012-12-12 21:08:06 CET
Created attachment 4320 [details] umc-frontend.patch Das MultiObjectSelect-Widget sollte im formatter String-ID's in ein {label,id}-dict umwandeln. Wenn in einem Form-Widget staticValues anstatt dynamicValues benutzt werden, wird eine Exception geworfen, weil statische Werte sofort initialisiert sind und zu dem Zeitpunkt der Initialisierung die "Klassenvariable" für das Form noch nicht existiert. Im Anhang ist ein Patch, der beide Probleme behebt.
Den Patch sollten wir im Rahmen der UCS-Entwicklung einspielen :) .
*** Bug 26781 has been marked as a duplicate of this bug. ***
eingespielt in univention-management-console-frontend (2.0.0-1) Paket muss noch bauen
Ein else-Zweig wäre hier noch notwendig so weit ich das sehe: ==================== if (dojo.isString(id)) { ids[i] = {label: id, id: id}; } // else { // ids[i] = id; // } ====================
(In reply to comment #4) > Ein else-Zweig wäre hier noch notwendig so weit ich das sehe: > ==================== > if (dojo.isString(id)) { > ids[i] = {label: id, id: id}; > } > // else { > // ids[i] = id; > // } > ==================== Nein, ids[i] ist ja schon id.
(In reply to comment #5) > Nein, ids[i] ist ja schon id. Ah ja, das stimmt.
> formatter: function(ids) { > array.forEach(ids, function(id, i) { > if (typeof id == "string") { > ids[i] = {label: id, id: id}; > } > }); > return ids; > }, Besser wäre es hier array.map() verwenden und so den initialen array mit IDs nicht zu verändern (das kann sonst zu Seiteneffekten führen).
(In reply to comment #7) > Besser wäre es hier array.map() verwenden und so den initialen array mit IDs > nicht zu verändern (das kann sonst zu Seiteneffekten führen). Ja, geändert in univention-management-console-frontend 2.0.57-1
onValuesInitialized wurde von buildRendering nach startup verlagert. Neuer Formatter funktioniert. Changelog OK.
UCS 3.1-0 has been released: http://forum.univention.de/viewtopic.php?f=54&t=2125 If this error occurs again, please use "Clone This Bug".