Bug 26857 - Anpassungen an das Dojo-Framework
Anpassungen an das Dojo-Framework
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P1 enhancement (vote)
: UCS 3.1
Assigned To: Alexander Kläser
Jascha Geerds
: interim-1
: 27942 28515 (view as bug list)
Depends on:
Blocks: 25476 26784 28010 28073 28551 28635
  Show dependency treegraph
 
Reported: 2012-04-20 18:57 CEST by Alexander Kläser
Modified: 2012-12-12 21:11 CET (History)
5 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): Release Goal
Max CVSS v3 score:


Attachments
jslint output (2.54 KB, text/plain)
2012-09-17 14:06 CEST, Jascha Geerds
Details
appliance tabs (55.96 KB, image/png)
2012-09-17 14:08 CEST, Jascha Geerds
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-04-20 18:57:26 CEST
Dojo 1.7 bringt einige Änderungen mit, die wir auch in unsere Paket zeitnah mit übernehmen sollten. Dazu zählen insbesondere:

* AMD-Module-Loader
* Änderungen des Event-Handlings (dojo.connect, genau zu überprüfen)
* Änderungen des Buildsystems (?)
* ComboBoxen etc. unterstützen ObjectStores (anstelle von DataStores)
* Dem i18n-Modul kann sehr bequem über einen AMD-Modul-Parameter der URL der i18n-Dateien angegeben werden; so könnte innerhalb eines Modul einfach eine _()-Methode ohne den Umweg über this._() bereitgestellt werden.
* Es gibt ein neues DataGrid ("dgrid"), was wesentlich performanter ist und nach Aussage des Dojo-Teams auch besser getestet

Die derzeitige API wird noch bis 2.0 unterstützt, allerdings wäre es wünschenswert, alte Dojo-Sachen frühzeitig zu entfernen.

Hier die Release-Notes:

  http://dojotoolkit.org/reference-guide/1.7/releasenotes/1.7.html

Hier die Projektseite des dgrids (wurde in ein eigenes Projekt ausgegliedert):

  https://github.com/SitePen/dgrid
Comment 1 Alexander Kläser univentionstaff 2012-06-18 12:34:27 CEST
*** Bug 25476 has been marked as a duplicate of this bug. ***
Comment 2 Alexander Kläser univentionstaff 2012-07-25 16:28:19 CEST
(In reply to comment #0)
> Dojo 1.7 bringt einige Änderungen mit, die wir auch in unsere Paket zeitnah mit
> übernehmen sollten. Dazu zählen insbesondere:
> ...

Mittlerweile gibt es auch schon Dojo 1.8.
Comment 3 Alexander Kläser univentionstaff 2012-07-25 17:11:16 CEST
In diesem Zusammenhang kann auch das Grid überarbeitet werden → Bug 28073
Comment 4 Florian Best univentionstaff 2012-09-07 14:44:09 CEST
Fehler beim schließen des sysinfo-Moduls über den [x]-Button in der Tabliste:
(in umc/widgets/Page in uninitialize() ist this._subscriptionHandle null).

Im grid mit checkbox-plugin wird beim Klick auf eine Zeile standardmäßig nicht mehr selektiert.

Der Tooltip ist momentan nicht mehr zentriert (das ist wahrscheinlich CSS bedingt (Bug #26826)?).
Comment 5 Stefan Gohmann univentionstaff 2012-09-11 06:49:54 CEST
Aufgefallen sind mir die folgenden Punkte (ggf. hängt das aber auch an Bug #26826):

- Benutzername und Passwort im Login Dialog sind in Firefox 15 verschoben (demo.univention.de ist mit FF 15 OK)

- Auf der Übersichtsseite fehlt das UMC Icon

- Bei einem DC kann für "Forward zone for DNS entry" keine IP ausgewählt werden

- Im Benutzer Dialog sehen die Checkboxen falsch aus, beispielsweise  "Override password history"

- Nach dem Anlegen eines Benutzers muss in der Benutzersuche nochmal auf Suchen geklickt werden

- Das IP change Modul wird angezeigt
Comment 6 Stefan Gohmann univentionstaff 2012-09-11 07:59:43 CEST
Weitere Punkte:

- Rechte Maustaste "Bearbeiten" in der Navigations-Tree-View bleibt ohne Wirkung

- Beim Benutzer bearbeiten können keine weiteren Gruppen ausgewählt werden.
Comment 7 Stefan Gohmann univentionstaff 2012-09-11 13:55:37 CEST
Der "Anzeigen" Button in UVMM (Auflistung der VMs) ist nicht vorhanden.
Comment 8 Stefan Gohmann univentionstaff 2012-09-12 08:09:04 CEST
(In reply to comment #7)
> Der "Anzeigen" Button in UVMM (Auflistung der VMs) ist nicht vorhanden.

Die Geräte (Festplatten, CDROM, usw.) werden beim Bearbeiten von VMs nicht angezeigt. Netzwerk-Interfaces hingegen schon. Das blockt derzeit die QA für andere interim-1 Bugs.
Comment 9 Stefan Gohmann univentionstaff 2012-09-12 10:16:01 CEST
Wenn ich mit UVMM eine Instanz live migriere, dann kommt UVMM nicht oder erst sehr viel später mit einer Fehlermeldung zurück, dass die Verbindung abgebrochen wurde.

Aus der Console:
Uncaught TypeError: Cannot read property 'form' of undefined
_f96._onClickdojo.js:2
_111fdojo.js:2
_18f2._onClickdojo.js:2
lang.hitchdojo.js:2
_feedojo.js:2
Comment 10 Alexander Kläser univentionstaff 2012-09-12 17:46:31 CEST
(In reply to comment #4)
> Fehler beim schließen des sysinfo-Moduls über den [x]-Button in der Tabliste:
> (in umc/widgets/Page in uninitialize() ist this._subscriptionHandle null).

→ scheint soweit behoben

> Im grid mit checkbox-plugin wird beim Klick auf eine Zeile standardmäßig nicht
> mehr selektiert.

→ das ist gut :)

> Der Tooltip ist momentan nicht mehr zentriert (das ist wahrscheinlich CSS
> bedingt (Bug #26826)?).Bug 26826

(In reply to comment #5)
> Aufgefallen sind mir die folgenden Punkte (ggf. hängt das aber auch an Bug
> #26826):
> 
> - Benutzername und Passwort im Login Dialog sind in Firefox 15 verschoben
> (demo.univention.de ist mit FF 15 OK)Bug 26826

> - Auf der Übersichtsseite fehlt das UMC Icon

→ das wurde korrigiert

> - Bei einem DC kann für "Forward zone for DNS entry" keine IP ausgewählt werden

→ TODO

> - Im Benutzer Dialog sehen die Checkboxen falsch aus, beispielsweise  "Override
> password history"Bug 26826

> - Nach dem Anlegen eines Benutzers muss in der Benutzersuche nochmal auf Suchen
> geklickt werden

→ das wurde korrigiert

> - Das IP change Modul wird angezeigt

→ das wurde korrigiert

(In reply to comment #6)
> Weitere Punkte:
> 
> - Rechte Maustaste "Bearbeiten" in der Navigations-Tree-View bleibt ohne
> Wirkung

→ das scheint jetzt zu funktionieren

> - Beim Benutzer bearbeiten können keine weiteren Gruppen ausgewählt werden.

→ TODO

(In reply to comment #8)
> > Der "Anzeigen" Button in UVMM (Auflistung der VMs) ist nicht vorhanden.
>
> Die Geräte (Festplatten, CDROM, usw.) werden beim Bearbeiten von VMs nicht
> angezeigt. Netzwerk-Interfaces hingegen schon. Das blockt derzeit die QA für
> andere interim-1 Bugs.

→ TODO

(In reply to comment #9)
> Wenn ich mit UVMM eine Instanz live migriere, dann kommt UVMM nicht oder erst
> sehr viel später mit einer Fehlermeldung zurück, dass die Verbindung
> abgebrochen wurde.
> 
> Aus der Console:
> Uncaught TypeError: Cannot read property 'form' of undefined
> _f96._onClickdojo.js:2
> _111fdojo.js:2
> _18f2._onClickdojo.js:2
> lang.hitchdojo.js:2
> _feedojo.js:2

→ TODO

Desweiteren gibt es noch folgende Probleme:
* UDM-Modulen mit Superordinates
* DHCP-/DNS-Einträge können derzeit für Rechner nicht vorgenommen werden
Comment 11 Alexander Kläser univentionstaff 2012-09-13 00:43:22 CEST
(In reply to comment #10)
> ...
> Desweiteren gibt es noch folgende Probleme:
> * UDM-Modulen mit Superordinates
> * DHCP-/DNS-Einträge können derzeit für Rechner nicht vorgenommen werden

→ Das funktioniert jetzt
Comment 12 Alexander Kläser univentionstaff 2012-09-13 01:18:01 CEST
(In reply to comment #10)
> ...
> > - Bei einem DC kann für "Forward zone for DNS entry" keine IP ausgewählt werden
> 
> → TODO

→ Das geht jetzt
Comment 13 Dirk Wiesenthal univentionstaff 2012-09-13 10:04:47 CEST
*** Bug 27942 has been marked as a duplicate of this bug. ***
Comment 14 Alexander Kläser univentionstaff 2012-09-13 11:13:58 CEST
Layout-Probleme bei Recherner und DHCP-/DNS-Einstellungen waren ebenfalls JS-bedingt und sind jetzt korrigiert.
Comment 15 Alexander Kläser univentionstaff 2012-09-13 12:23:34 CEST
(In reply to comment #10)
> ...
> > > Der "Anzeigen" Button in UVMM (Auflistung der VMs) ist nicht vorhanden.
> >
> > Die Geräte (Festplatten, CDROM, usw.) werden beim Bearbeiten von VMs nicht
> > angezeigt. Netzwerk-Interfaces hingegen schon. Das blockt derzeit die QA für
> > andere interim-1 Bugs.
> 
> → TODO

Das ist jetzt auch behoben

> (In reply to comment #9)
> > Wenn ich mit UVMM eine Instanz live migriere, dann kommt UVMM nicht oder erst
> > sehr viel später mit einer Fehlermeldung zurück, dass die Verbindung
> > abgebrochen wurde.
> > 
> > Aus der Console:
> > Uncaught TypeError: Cannot read property 'form' of undefined
> > _f96._onClickdojo.js:2
> > _111fdojo.js:2
> > _18f2._onClickdojo.js:2
> > lang.hitchdojo.js:2
> > _feedojo.js:2
> 
> → TODO

Das ist jetzt ebenfalls behoben
Comment 16 Alexander Kläser univentionstaff 2012-09-13 13:16:59 CEST
(In reply to comment #10)
> ...
> > - Beim Benutzer bearbeiten können keine weiteren Gruppen ausgewählt werden.
> 
> → TODO

Das funktioniert jetzt auch wieder (behoben im UDM-Modul).

Es stehen noch Probleme mit IE8 aus.
Comment 17 Alexander Kläser univentionstaff 2012-09-13 20:10:30 CEST
(In reply to comment #16)
> ...
> Es stehen noch Probleme mit IE8 aus.

Das wurde nun auch behoben, IE8 scheint nun nach einem ersten Test zu funktionieren: 

→ univention-management-console-frontend (2.0.53-1)
Comment 18 Alexander Kläser univentionstaff 2012-09-13 20:20:13 CEST
Weitere Bugs können ausgelagert werden, wie z.B. Bug 28498
Comment 19 Alexander Kläser univentionstaff 2012-09-14 11:43:33 CEST
Für die QA ist eine vollständige Überprüfung aller Module nicht sinnvoll. Bitte daher die folgenden Punkte testen:

* Update von 3.0-2 via UMC
* System-Setup im Appliance-Modus (beim Booten)
* Changelog
* Grundsätzliche Integration von Dojo 1.8 verifizieren (umc-frontend, univention-dojo)
* mit jslint alle UMC-Module (ucs/*/*/umc/js) auf gravierende Fehler testen
* die unterstützten Browser einmal antesten
Comment 20 Alexander Kläser univentionstaff 2012-09-14 12:11:59 CEST
Noch aufgefallen → Bug 28514
Comment 21 Alexander Kläser univentionstaff 2012-09-14 12:58:06 CEST
*** Bug 28198 has been marked as a duplicate of this bug. ***
Comment 22 Dirk Wiesenthal univentionstaff 2012-09-14 14:21:37 CEST
Dinge, sie mir so aufgefallen sind:

Es gibt in Firefox einen Fehler: TypeError: cs is null in dojo.js:2
Ich kann das erstaunlicherweise nicht im debug-Modus nachstellen. Ich dachte erst an Firebug, aber der Fehler scheint auch ohne Firebug zu kommen.

Die Lokalisierungen (die jetzt übrigens alle sofort geladen werden...) laden jetzt immer app.json mit herunter (also gut 20 mal). Das wurde vorher nur einmal zu Beginn geladen.

Nicht schlimm, aber schon seit Monaten nervt mich so ein 404 bei branding.json. Wer lädt das eigentlich runter? Kann man das nicht unterbinden, wenn man schon mal dabei ist?

Schlimmer:
ack-grep "this\.\(" liefert die veraltete Lokalisierung in den Templates (werden gesondert angefasst), luga (wurde zurückgezogen) sowie im Frontend in app.js und dialog.js
Comment 23 Jascha Geerds univentionstaff 2012-09-14 15:43:12 CEST
Der Changelog ist in Ordnung

Update von 3.0-2 via UMC:
Irgendwann während des Updates läuft die Session aus. Nachdem man sich erneut eingelogt hat, erscheint ein 404 Error im Login Dialog (wohl aufgrund des neuen Timestamps). Direkt im Anschluss erscheint ein Dialog mit folgender Meldung:


Sie sind nicht authorisiert, diese Aktion durchzuführen.
Fehlernachricht des Servers:
Forbidden

(QA ist noch nicht beendet)
Comment 24 Jascha Geerds univentionstaff 2012-09-14 15:56:20 CEST
Mit dem iPad machte die UMC auch einen vernünftigen Eindruck. Ich konnte keine funktionalen Fehler feststellen, welche direkt den Javascript-Teil betreffen (habe jedoch nur flüchtig eine halbe Stunde rumgeklickt und getestet)
Comment 25 Alexander Kläser univentionstaff 2012-09-14 16:11:02 CEST
(In reply to comment #22)
> Dinge, sie mir so aufgefallen sind:
> ...

→ ausgelagert an Bug 28551
Comment 26 Florian Best univentionstaff 2012-09-17 12:24:58 CEST
für die QA: Ich fände es noch wichtig einmal kurz zu gucken, ob es eventuell Probleme macht, dass die modul-Ordner jetzt nicht mehr mit _ beginnen (im system-setup gab es UCR Variablen die Pfade enthalten haben).
Comment 27 Jascha Geerds univentionstaff 2012-09-17 14:06:43 CEST
Created attachment 4672 [details]
jslint output

(In reply to comment #19)
> * mit jslint alle UMC-Module (ucs/*/*/umc/js) auf gravierende Fehler testen

Im Anfang befindet sich die gefilterte Ausgabe von jslint. Bei allen Punkten denke ich, dass diese gefixed bzw. genauer angeschaut werden sollten.
Comment 28 Jascha Geerds univentionstaff 2012-09-17 14:08:28 CEST
Created attachment 4673 [details]
appliance tabs

(In reply to comment #19)
> * System-Setup im Appliance-Modus (beim Booten)

Bei meinem Test wurde das Basis-Setup-Modul 2x geöffnet
Comment 29 Jascha Geerds univentionstaff 2012-09-17 14:52:55 CEST
(In reply to comment #26)
> für die QA: Ich fände es noch wichtig einmal kurz zu gucken, ob es eventuell
> Probleme macht, dass die modul-Ordner jetzt nicht mehr mit _ beginnen (im
> system-setup gab es UCR Variablen die Pfade enthalten haben).

debian/univention-system-setup.postinst:if [ "$(univention-config-registry get 'system/setup/boot/help')" = "/usr/share/univention-management-console-frontend/js/umc/modules/_setup/:help_default.html" ]; then

Das müsste an der Stelle 'setup' statt '_setup' heißen.
Comment 30 Jascha Geerds univentionstaff 2012-09-17 15:00:39 CEST
(In reply to comment #29)
> debian/univention-system-setup.postinst:if [ "$(univention-config-registry get
> 'system/setup/boot/help')" =
> "/usr/share/univention-management-console-frontend/js/umc/modules/_setup/:help_default.html"
> ]; then
> 
> Das müsste an der Stelle 'setup' statt '_setup' heißen.

Das kann ignoriert werden, mein Fehler. Ist vollkommen richtig so :)
Comment 31 Alexander Kläser univentionstaff 2012-09-18 10:44:37 CEST
(In reply to comment #27)
> Created an attachment (id=4672) [details]
> jslint output
> 
> (In reply to comment #19)
> > * mit jslint alle UMC-Module (ucs/*/*/umc/js) auf gravierende Fehler testen
> 
> Im Anfang befindet sich die gefilterte Ausgabe von jslint. Bei allen Punkten
> denke ich, dass diese gefixed bzw. genauer angeschaut werden sollten.

→ super! → Bug 28551

(In reply to comment #29)
> debian/univention-system-setup.postinst:if [ "$(univention-config-registry get
> 'system/setup/boot/help')" =
> "/usr/share/univention-management-console-frontend/js/umc/modules/_setup/:help_default.html"
> ]; then
> 
> Das müsste an der Stelle 'setup' statt '_setup' heißen.

→ müssen wir uns nochmal genauer ansehen → Bug 28551

(In reply to comment #28)
> Bei meinem Test wurde das Basis-Setup-Modul 2x geöffnet

→ dachte, dass ich das bereits vermerkt habe, aber wohl nicht → Bug 28551
Comment 32 Jascha Geerds univentionstaff 2012-09-18 10:49:56 CEST
Alle hier beschriebenen Probleme wurden erfolgreich in andere Bugs geschoben, Verified!
Comment 33 Florian Best univentionstaff 2012-10-19 14:45:00 CEST
*** Bug 28515 has been marked as a duplicate of this bug. ***
Comment 34 Stefan Gohmann univentionstaff 2012-12-12 21:11:04 CET
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".