Bug 28551 - Anpassungen an das Dojo-Framework (2)
Anpassungen an das Dojo-Framework (2)
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P3 normal (vote)
: UCS 3.1
Assigned To: Dirk Wiesenthal
Florian Best
: interim-3
Depends on: 26857
Blocks:
  Show dependency treegraph
 
Reported: 2012-09-14 16:10 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):
Max CVSS v3 score:


Attachments
Patch für umc.tools.parseError() (1.00 KB, patch)
2012-10-02 09:59 CEST, Alexander Kläser
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-09-14 16:10:00 CEST
Hier können weitere Fehler gesammelt werden, die mit dem Update auf Dojo 1.8 noch auffallen.

Momentan wirft umc/dialog:confirm() eine Exception beim Schließen des Dialogs.

Ansonsten sind noch die folgenden Bugs aufgefallen: Bug 28514, Bug 28498
Comment 1 Alexander Kläser univentionstaff 2012-09-14 16:30:14 CEST
Aus Bug 26857, Comment 22:
> 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.

Stimmt, schwierig zu sagen was das genau ist, habe einen Thread dazu gefunden:

https://github.com/maqetta/maqetta/issues/3279

Das scheint zunächst kein direktes Problem zu erzeugen. In Chrome wird nichts dergleichen angezeigt.

> 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?

Stimmt, die JSON-Dateien werden noch nicht gecachet, das fehlt noch.

> 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

Stimmt!
Comment 2 Alexander Kläser univentionstaff 2012-09-14 19:18:41 CEST
In umc/modules/setup/RolePage.js fehlt ein lang.hitch:

> this._form.getWidget('server/role').watch('value', function...

Dadurch wird derzeit im Appliance-Modus der Hilfetext nicht aktualisiert.
Comment 3 Alexander Kläser univentionstaff 2012-09-14 19:23:52 CEST
Tracebacks werden derzeit noch nicht wie früher angezeigt, stattdessen kommt wird ein JavaScript-Fehler in dem Dialog angezeigt ("null element" oder so).
Comment 4 Florian Best univentionstaff 2012-09-17 08:34:49 CEST
Falls ein this.own um foo.watch gehört müssen noch noch folgende Zeilen angepasst werden:
svn/dev/branches/ucs-3.1/ucs $ find -name '*.js' -exec grep 'watch(' {} + | grep -v this\.own
Comment 5 Florian Best univentionstaff 2012-09-17 09:16:39 CEST
this._( → _(

./management/univention-management-console-frontend/umc/app.js:                                 var certType = this._('SSL root certificate');
./management/univention-management-console-frontend/umc/app.js:                                         certType = this._('SSL host certificate');
./management/univention-management-console-frontend/umc/app.js:                                         overviewPage.addNote( this._( 'The %s will expire in %d days and should be renewed!', certType, days ) );
Comment 6 Stefan Gohmann univentionstaff 2012-09-17 12:07:20 CEST
Passwort und Passwort (Wiederholung) sind nicht mehr in einer Reihe, beispielsweise beim Anlegen eines Benutzers.
Comment 7 Florian Best univentionstaff 2012-09-17 13:28:00 CEST
Object [Widget umc.widgets.TimeBox, umc_widgets_TimeBox_0] has no method 'adopt'
Comment 8 Jascha Geerds univentionstaff 2012-09-17 13:33:15 CEST
(In reply to comment #6)
> Passwort und Passwort (Wiederholung) sind nicht mehr in einer Reihe,
> beispielsweise beim Anlegen eines Benutzers.

Das fällt vermutlich eher in den Bereich von Bug #28516
Comment 9 Florian Best univentionstaff 2012-09-17 13:46:51 CEST
Object [Widget umc.widgets.LoginDialog, umc_widgets_LoginDialog_0] has no method '_getFocusItems'
Comment 10 Jascha Geerds univentionstaff 2012-09-17 14:23:14 CEST
"In System-Setup stimmt etwas auf der
Netzwerk-Seite nicht. Die DHCP-Query wird merkwürdig angezeigt."
Comment 11 Alexander Kläser univentionstaff 2012-09-17 15:06:14 CEST
Aufgefallen im UVMM-Paket:

  ./umc/js/uvmm/SnapshotGrid.js:  "umc/i18n!umc/modules/"
  ./umc/js/uvmm/DriveWizard.js:   "umc/i18n!umc/modules/"
Comment 12 Jascha Geerds univentionstaff 2012-09-17 15:42:40 CEST
Die UVMM Status-Icons fehlen, dadurch ist das Grid von UVMM eigentlich kaum benutzbar.
Comment 13 Lukas Walter univentionstaff 2012-09-17 15:52:50 CEST
Im System Setup fehlen noch ein paar Anpassungen, siehe Bug #27949, Kommentar 4.
Comment 14 Florian Best univentionstaff 2012-09-17 15:57:24 CEST
[UDM]
Navigation öffnen → rechtsklick auf einen container
udm.js:690 Uncaught TypeError: Cannot read property 'target' of undefined
udm.js:663 Uncaught TypeError: Cannot read property 'objectType' of null
Comment 15 Florian Best univentionstaff 2012-09-17 16:00:35 CEST
ca 30 Warnungnen beim anlegen eines neuen containers in der Navigation: 
WARNING in render.layout: The widget 'name' is not defined in the argument 
'widgets'. Ignoring error.
Comment 16 Dirk Wiesenthal univentionstaff 2012-09-17 16:52:48 CEST
Im Appliance Modus:
TypeError: this.switchDescription is not a function
  this.switchDescription();    SystemRolePage.js (Zeile 130)

Außerdem scheint die Warnung bei leeren Passwort nicht zu funktionieren?
Comment 17 Dirk Wiesenthal univentionstaff 2012-09-17 17:14:40 CEST
Das Join-Modul scheint auch im gejointen Status immer nur die "Beitrittsmaske" anzuzeigen. Sowohl bei gejointem Slave als auch beim Master selbst!
Comment 18 Dirk Wiesenthal univentionstaff 2012-09-18 09:39:10 CEST
In bestimmten Widgets werden jetzt HTML-Titel gesetzt z.B. title="Druckerverwaltung" oder in UDM title="Benutzer: Administrator" oder bei Fehlern sogar title="<span style='color:red'...>". Das wird dann vom Browser überall über das Widget gelegt.
Comment 19 Alexander Kläser univentionstaff 2012-09-18 10:28:05 CEST
(In reply to comment #8)
> (In reply to comment #6)
> > Passwort und Passwort (Wiederholung) sind nicht mehr in einer Reihe,
> > beispielsweise beim Anlegen eines Benutzers.
> 
> Das fällt vermutlich eher in den Bereich von Bug #28516

Ich denke nicht, gleiches Problem wie bei MultiInputs, via JS wird noch die Breite "One" gesetzt, dadurch brechen die Elemente um.
Comment 20 Dirk Wiesenthal univentionstaff 2012-09-18 10:36:28 CEST
[UDM]
Navigation öffnen → rechtsklick auf einen container → Bearbeiten
TypeError: this._navContextItem is null
  this.createDetailPage(this._navContextItem.objectType, this._navContextItem.id)
  udm.js (Zeile 663)
Comment 21 Alexander Kläser univentionstaff 2012-09-18 10:44:53 CEST
Noch in Bug 26857 aufgefallen:

jslint-Hinweise:

> ./univention-management-console-module-packages/umc/js/packages/AppCenterPage.js:109:39:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-module-packages/umc/js/packages/store.js:73:30:Missing semicolon.
> ./univention-management-console-module-packages/umc/js/packages/store.js:157:20:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-module-packages/umc/js/packages/store.js:242:34:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-module-udm/umc/js/udm/MultiObjectSelect.js:72:18:Unnecessary semicolon.
> ./univention-management-console-module-udm/umc/js/udm.js:658:28:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-frontend/umc/app.js:423:34:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-frontend/umc/widgets/MultiObjectSelect.js:332:32:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-frontend/umc/widgets/Grid.js:173:1:implied global variable 'require'
> ./univention-management-console-frontend/umc/widgets/CategoryPane.js:60:50:Line breaking error '</div>'.
> ./univention-management-console-frontend/umc/widgets/CategoryPane.js:60:56:Missing semicolon.
> ./univention-management-console-frontend/umc/widgets/Form.js:392:1:implied global variable 'require'
> ./univention-management-console-frontend/umc/widgets/Form.js:392:39:Line breaking error ')'.
> ./univention-management-console-frontend/umc/widgets/Form.js:392:40:Missing semicolon.
> ./univention-management-console-frontend/umc/widgets/MultiInput.js:279:41:Line breaking error ')'.
> ./univention-management-console-frontend/umc/widgets/MultiInput.js:279:42:Missing semicolon.
> ./univention-management-console-frontend/umc/store.js:208:20:Missing 'new' prefix when invoking a constructor.
> ./univention-management-console-frontend/umc/render.js:105:46:Missing semicolon.
> ./univention-management-console-module-vnc/umc/js/vnc.js:243:1:implied global variable 'window'
> ./univention-virtual-machine-manager-daemon/umc/js/uvmm/DomainPage.js:312:32:Missing 'new' prefix when invoking a constructor.
> ./univention-virtual-machine-manager-daemon/umc/js/uvmm/DomainPage.js:327:36:Missing 'new' prefix when invoking a constructor.
> ./univention-virtual-machine-manager-daemon/umc/js/uvmm/DomainWizard.js:55:32:Missing 'new' prefix when invoking a constructor.
> ./univention-system-setup/umc/js/setup/LanguagePage.js:192:1:implied global variable 'default_locale_default'
> ./univention-system-setup/umc/js/setup/NetworkPage.js:258:34:Use '===' to compare with ''.

In System-Setup:

> 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.

System-Setup-Appliance:

> Bei meinem Test wurde das Basis-Setup-Modul 2x geöffnet
Comment 22 Jascha Geerds univentionstaff 2012-09-18 10:47:31 CEST
(In reply to comment #21)
> In System-Setup:
> 
> > 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.
Comment 23 Florian Best univentionstaff 2012-09-19 07:57:37 CEST
In Chromium funktioniert der Dateiupload über das Widget, welches im Lizenz und Benutzerfoto-upload dialog verwendet wird nicht (es gibt auch keine JS-Fehler, etc.).
Comment 24 Florian Best univentionstaff 2012-09-19 07:58:54 CEST
(In reply to comment #23)
> In Chromium funktioniert der Dateiupload über das Widget, welches im Lizenz und
> Benutzerfoto-upload dialog verwendet wird nicht (es gibt auch keine JS-Fehler,
> etc.).
In keinem Browser.
Comment 25 Alexander Kläser univentionstaff 2012-09-20 10:59:14 CEST
Wenn durch einen UMC-Server-Neustart die Session abgelaufen ist, erscheint bei einem Reload der Webseite ein Fehlerdialog hinter dem LoginDialog.
Comment 26 Stefan Gohmann univentionstaff 2012-09-21 08:55:26 CEST
Wenn ich bei Shares die Optionen ändere, beispielsweise die Option Samba entferne, dann bleiben die Tabs für Samba weiterhin sichtbar. In 3.0 werden die Tabs direkt entfernt.
Comment 27 Alexander Kläser univentionstaff 2012-09-21 12:16:02 CEST
(In reply to comment #26)
> Wenn ich bei Shares die Optionen ändere, beispielsweise die Option Samba
> entferne, dann bleiben die Tabs für Samba weiterhin sichtbar. In 3.0 werden die
> Tabs direkt entfernt.

Das Anzeigen/Verstecken der Formular-Elemente auf einer UDM-Detailseite funktioniert generell nicht, das kann noch einmal genauer unter die Lupe genommen werden.
Comment 28 Florian Best univentionstaff 2012-09-24 12:20:21 CEST
(In reply to comment #3)
> Tracebacks werden derzeit noch nicht wie früher angezeigt, stattdessen kommt
> wird ein JavaScript-Fehler in dem Dialog angezeigt ("null element" oder so).
fixed, QA: auch umc.widgets.Uploader überprüfen (für success und fail)

(In reply to comment #1)
…
> > 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
Alle this._( wurden durch _( ersetzt.

(In reply to comment #2)
> In umc/modules/setup/RolePage.js fehlt ein lang.hitch:
> 
> > this._form.getWidget('server/role').watch('value', function...
> 
> Dadurch wird derzeit im Appliance-Modus der Hilfetext nicht aktualisiert.
gefixt, univention-system-setup (6.0.34-1)

(In reply to comment #4)
> Falls ein this.own um foo.watch gehört müssen noch noch folgende Zeilen
> angepasst werden:
> svn/dev/branches/ucs-3.1/ucs $ find -name '*.js' -exec grep 'watch(' {} + |
> grep -v this\.own
Wurde für alle .watch( aufrufe gemacht.

(In reply to comment #5)
Same as comment #1 unten

(In reply to comment #11)
> Aufgefallen im UVMM-Paket:
> 
>   ./umc/js/uvmm/SnapshotGrid.js:  "umc/i18n!umc/modules/"
>   ./umc/js/uvmm/DriveWizard.js:   "umc/i18n!umc/modules/"
wurde erweitert

(In reply to comment #13)
> Im System Setup fehlen noch ein paar Anpassungen, siehe Bug #27949, Kommentar
> 4.
Wurde gemacht.
(In reply to comment #21)
> Noch in Bug 26857 aufgefallen:
> 
> jslint-Hinweise:
Sämtliche JSLINT Sachen gefixt.
 
(In reply to comment #22)
> Das kann ignoriert werden.
Genau, das ist updatecode.
Comment 29 Dirk Wiesenthal univentionstaff 2012-09-24 19:03:22 CEST
Neu aufgetretenes Problem:

Error: declare umc.widgets.LoginDialog: base class is not a callable constructor.
...ion err(msg, cls){ throw new Error("declare" + (cls ? " " + cls : "") + ": " + m...
dojo.j...ssed.js (Zeile 50518)
Comment 30 Florian Best univentionstaff 2012-09-25 08:47:01 CEST
(In reply to comment #29)
> Neu aufgetretenes Problem:
> 
> Error: declare umc.widgets.LoginDialog: base class is not a callable
> constructor.
> ...ion err(msg, cls){ throw new Error("declare" + (cls ? " " + cls : "") + ": "
> + m...
> dojo.j...ssed.js (Zeile 50518)
fixed
Comment 31 Alexander Kläser univentionstaff 2012-09-25 17:00:04 CEST
Bitte auch mal schauen, ob Bug 28635 im Zuge der Dojo-Umstellung sich eingeschlichen hat.
Comment 32 Alexander Kläser univentionstaff 2012-09-26 14:39:58 CEST
(In reply to comment #31)
> Bitte auch mal schauen, ob Bug 28635 im Zuge der Dojo-Umstellung sich
> eingeschlichen hat.

→ Nein, ist ein anderer Fehler, da das Problem auch unter 3.0-2 auftrat.
Comment 33 Florian Best univentionstaff 2012-09-28 10:20:10 CEST
(In reply to comment #16)
> Im Appliance Modus:
> TypeError: this.switchDescription is not a function
>   this.switchDescription();    SystemRolePage.js (Zeile 130)
Wurde gefixt: univention-system-setup (6.0.36-1)
> Außerdem scheint die Warnung bei leeren Passwort nicht zu funktionieren?
Die hat bei mir funktioniert.

(In reply to comment #17)
> Das Join-Modul scheint auch im gejointen Status immer nur die "Beitrittsmaske"
> anzuzeigen. Sowohl bei gejointem Slave als auch beim Master selbst!
Wurde gefixt: univention-join (5.0.11-1); svn r35927
Comment 34 Florian Best univentionstaff 2012-09-28 10:23:19 CEST
(In reply to comment #20)
entspricht comment #14
Comment 35 Alexander Kläser univentionstaff 2012-09-28 16:51:35 CEST
Auch noch aufgefallen in umc/tools:parseError():

  status: parseInt(error.data.status, 10) || error.status,

Hier ist es besser vorsichtiger mit error umzugehen. Wird eine HTML-HTTP-Fehlerseite (bspw. von Apache) zurückgegeben, dann versucht Dojo diese als JSON zu interpretieren. Das funktioniert nicht und parseError() wird als error ein SyntaxFehler übergeben.

Der Error-Status kann in diesem Fall über error.response.status und die eigentliche HTML-Fehler-Seite über error.response.text erreicht werden. (Ggf. kann <title>...</title> aus dem text geparst werden.)

Der Fehler ist aufgetreten, wenn in System-Setup für ein Netzwerkgerät die IP-Adresse gelöscht und auf DHCP gesetzt wird.
Comment 36 Alexander Kläser univentionstaff 2012-09-28 18:20:48 CEST
(In reply to comment #28)
> (In reply to comment #3)
> > Tracebacks werden derzeit noch nicht wie früher angezeigt, stattdessen kommt
> > wird ein JavaScript-Fehler in dem Dialog angezeigt ("null element" oder so).
> fixed, QA: auch umc.widgets.Uploader überprüfen (für success und fail)

Ja, das funktioniert soweit. Der Dialog wird derzeit noch nicht neu zentriert, wenn der Traceback ausgeklappt wird.
Comment 37 Alexander Kläser univentionstaff 2012-09-28 18:46:33 CEST
Der folgende Traceback wurde nicht korrekt angezeigt (keine Möglichkeit, den
Traceback auszuklappen):

> {"status": 591, "message": "Execution of command packages/app_center/get has failed:
> 
> Traceback (most recent call last):
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/__init__.py&quot;, line 204, in execute
>     func( request )
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/decorators.py&quot;, line 182, in _response
>     return function(self, request)
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/udm/udm_ldap.py&quot;, line 142, in wrapper_func
>     ret = func( *args, **kwargs )
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/decorators.py&quot;, line 288, in _response
>     result = _multi_response(self, request)
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/decorators.py&quot;, line 390, in _response
>     for res in function(self, iterator, *nones):
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/decorators.py&quot;, line 272, in _fake_func
>     yield function(self, *args)
>   File &quot;/usr/lib/pymodules/python2.6/univention/management/console/modules/packages/__init__.py&quot;, line 120, in app_center_get
>     return application.to_dict_detail(package_manager, udm_license)
> NameError: global name package_manager is not defined
> "}

→ in diesem Fall ist in parseError() error.data.message noch ein String, der
von Dojo anscheinend nicht geparset wurde.
Comment 38 Florian Best univentionstaff 2012-10-01 17:59:42 CEST
(In reply to comment #35)
> Auch noch aufgefallen in umc/tools:parseError():
> 
>   status: parseInt(error.data.status, 10) || error.status,
> 
> Hier ist es besser vorsichtiger mit error umzugehen. Wird eine
> HTML-HTTP-Fehlerseite (bspw. von Apache) zurückgegeben, dann versucht Dojo
> diese als JSON zu interpretieren. Das funktioniert nicht und parseError() wird
> als error ein SyntaxFehler übergeben.
> 
> Der Error-Status kann in diesem Fall über error.response.status und die
> eigentliche HTML-Fehler-Seite über error.response.text erreicht werden. (Ggf.
> kann <title>...</title> aus dem text geparst werden.)
> 
> Der Fehler ist aufgetreten, wenn in System-Setup für ein Netzwerkgerät die
> IP-Adresse gelöscht und auf DHCP gesetzt wird.
Wird nun so gemacht. Wenn kein JSON returned wird, wird der html <title> genommen.
univention-management-console-frontend (2.0.81-1)
Comment 39 Alexander Kläser univentionstaff 2012-10-02 09:59:07 CEST
Created attachment 4696 [details]
Patch für umc.tools.parseError()

(In reply to comment #38)
> Wird nun so gemacht. Wenn kein JSON returned wird, wird der html <title>
> genommen.
> univention-management-console-frontend (2.0.81-1)

Super! Achtung, es wird auf error.status / error.text zugegriffen und müsste aber auf error.response.* zugegriffen werden. Anbei ein Änderungsvorschlag.
Comment 40 Florian Best univentionstaff 2012-10-08 14:48:24 CEST
(In reply to comment #0)
> Momentan wirft umc/dialog:confirm() eine Exception beim Schließen des Dialogs.
Wirft bei mir keine Exception mehr.
> Ansonsten sind noch die folgenden Bugs aufgefallen: Bug 28514, Bug 28498
Sind an den jeweiligen Bugs auf FIXED

(In reply to comment #1)
> Aus Bug 26857, Comment 22:
> …
> Das scheint zunächst kein direktes Problem zu erzeugen. In Chrome wird nichts
> dergleichen angezeigt.
Nicht bearbeitet.

> > 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.
> …
> Stimmt, die JSON-Dateien werden noch nicht gecachet, das fehlt noch.
Caching wurd einegebaut.

> > 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
Wurde gefixt.

(In reply to comment #7)
> Object [Widget umc.widgets.TimeBox, umc_widgets_TimeBox_0] has no method
> 'adopt'
this.adopt wurde durch this.own getauscht.

(In reply to comment #9)
> Object [Widget umc.widgets.LoginDialog, umc_widgets_LoginDialog_0] has no
> method '_getFocusItems'
bereits gefixt.

TODO:
Comment #10
Comment #12
Comment #14
Comment #15
Comment #18
Comment #19
Comment #23
Comment #25
Comment #26
Comment #27
Comment #36
Comment 41 Florian Best univentionstaff 2012-10-09 11:07:30 CEST
(In reply to comment #19)
> (In reply to comment #8)
> > (In reply to comment #6)
> > > Passwort und Passwort (Wiederholung) sind nicht mehr in einer Reihe,
> > > beispielsweise beim Anlegen eines Benutzers.
> > 
> > Das fällt vermutlich eher in den Bereich von Bug #28516
> 
> Ich denke nicht, gleiches Problem wie bei MultiInputs, via JS wird noch die
> Breite "One" gesetzt, dadurch brechen die Elemente um.
Ja, das war noch ein Dojo-Bug, der in Dojo 1.8.1 behoben ist. Der patch wurde für univention-dojo (6.0.4-1) übernommen.

(In reply to comment #37)
> Der folgende Traceback wurde nicht korrekt angezeigt (keine Möglichkeit, den
> Traceback auszuklappen):
> → in diesem Fall ist in parseError() error.data.message noch ein String, der
> von Dojo anscheinend nicht geparset wurde.
Nein, der String wurde geparsed. Das Problem lag daran, dass es noch newlines gab, die vorher durch <br> ersetzt wurden. Das wurde behoben.

(In reply to comment #9)
> Object [Widget umc.widgets.LoginDialog, umc_widgets_LoginDialog_0] has no
> method '_getFocusItems'
Der Login widget erbt nicht von dijit.Dialog wird aber als solcher übergeben:
Dialog._DialogLevelManager.hide(this)
Der LoginDialog erbt jetzt von dijit.Dialog.

(In reply to comment #14)
> [UDM]
> Navigation öffnen → rechtsklick auf einen container
> udm.js:690 Uncaught TypeError: Cannot read property 'target' of undefined
> udm.js:663 Uncaught TypeError: Cannot read property 'objectType' of null
fixed, univention-management-console-module-udm (3.0.20-1)
Comment 42 Florian Best univentionstaff 2012-10-09 14:57:22 CEST
(In reply to comment #36)
> (In reply to comment #28)
> > (In reply to comment #3)
> > > Tracebacks werden derzeit noch nicht wie früher angezeigt, stattdessen kommt
> > > wird ein JavaScript-Fehler in dem Dialog angezeigt ("null element" oder so).
> > fixed, QA: auch umc.widgets.Uploader überprüfen (für success und fail)
> 
> Ja, das funktioniert soweit. Der Dialog wird derzeit noch nicht neu zentriert,
> wenn der Traceback ausgeklappt wird.
gefixt

Comment #15 und Comment #18 können ignoriert werden.

Comment #25 tritt nicht mehr auf

Comment #27
verstecken / anzeigen von Formularelementen funktioniert, aber Comment #26 noch nicht.

Comment #23
Der Dateiupload wurde gefixt.

TODO sind folglich:
Comment #10
Comment #12
Comment #26
Comment 43 Alexander Kläser univentionstaff 2012-10-09 15:08:55 CEST
In umc/modules/pkgdb/SearchForm:onSubmit() fehlt für die letzte Zeile der Funktion der Rückgabewert:

> return this.inherited(arguments);
Comment 44 Florian Best univentionstaff 2012-10-09 15:28:44 CEST
(In reply to comment #43)
> In umc/modules/pkgdb/SearchForm:onSubmit() fehlt für die letzte Zeile der
> Funktion der Rückgabewert:
> 
> > return this.inherited(arguments);
univention-pkgdb (6.0.5-1)
Comment 45 Florian Best univentionstaff 2012-10-09 16:21:39 CEST
(In reply to comment #12)
> Die UVMM Status-Icons fehlen, dadurch ist das Grid von UVMM eigentlich kaum
> benutzbar.
fixed in univention-virtual-machine-manager-daemon (2.0.8-1)
Comment 46 Florian Best univentionstaff 2012-10-10 16:00:34 CEST
Comment #26 ist gefixt in univention-management-console-module-udm (3.0.24-1) und
 univention-management-console-frontend (2.0.95-1).

Jetzt steht noch Comment #10 aus.
Comment 47 Florian Best univentionstaff 2012-10-10 17:24:33 CEST
(In reply to comment #46)
> Comment #26 ist gefixt in univention-management-console-module-udm (3.0.24-1)
> und
>  univention-management-console-frontend (2.0.95-1).
> 
> Jetzt steht noch Comment #10 aus.
FIXED: 
univention-management-console-frontend (2.0.96-1)
Comment 48 Alexander Kläser univentionstaff 2012-10-11 11:15:42 CEST
Bitte in umc.tools noch einmal nachschauen, dojo/_base/connect ist deprecated, on() war schon richtig. Da es sich hier nicht um ein Widget handelt, sind die Events case-sensitive, d.h. das Event muss 'End' heißen (von 'onEnd'):

titlePane._wipeIn.on('End', function() { dialog.centerAlertDialog(); } );
titlePane._wipeOut.on('End', function() { dialog.centerAlertDialog(); } );
Comment 49 Florian Best univentionstaff 2012-10-11 11:55:49 CEST
(In reply to comment #48)
> Bitte in umc.tools noch einmal nachschauen, dojo/_base/connect ist deprecated,
> on() war schon richtig. Da es sich hier nicht um ein Widget handelt, sind die
> Events case-sensitive, d.h. das Event muss 'End' heißen (von 'onEnd'):
> 
> titlePane._wipeIn.on('End', function() { dialog.centerAlertDialog(); } );
> titlePane._wipeOut.on('End', function() { dialog.centerAlertDialog(); } );
Done
Comment 50 Alexander Kläser univentionstaff 2012-10-11 18:40:35 CEST
Ein Problem ist noch mit umc/widgets/LinkList aufgefallen, der Zugriff auf die dynamische geladenen Items war obsolete, so dass die Referenzen von Policies nicht mehr angezeigt wurden.

Fixed in: univention-management-console-frontend (2.0.101-1)
Comment 51 Florian Best univentionstaff 2012-10-16 10:26:04 CEST
Was noch aufgefallen ist:
Die MAC-Adresse im DHCP Eintrag eines Rechners ist manchmal/initial nicht auswählbar.
Nach dem klick auf "Änderungen speichern" wird die Seite nicht geschlossen.
Comment 52 Florian Best univentionstaff 2012-10-16 14:16:12 CEST
(In reply to comment #51)
> Was noch aufgefallen ist:
> Die MAC-Adresse im DHCP Eintrag eines Rechners ist manchmal/initial nicht
> auswählbar.
> Nach dem klick auf "Änderungen speichern" wird die Seite nicht geschlossen.
Ausgelagert an Bug #28797
Comment 53 Alexander Kläser univentionstaff 2012-10-19 14:34:59 CEST
Es ist noch ein Problem mit umc/tools:parseError aufgefallen. Durch ein Problem in System-Setup wurde beim Setzen der Netzwerkeinstellungen ein 401-Fehler zurück gegeben. Nach dem Aufruf von parseError, war status nicht gesetzt. Der Aufbau des von Dojo zurück gegebenen Errors war wie folgt:

> {
>    "message":"Unable to load /umcp/command/setup/finished?request.preventCache=1350647695267 status: 401",
>    "response":{
>       "url":"/umcp/command/setup/finished?request.preventCache=1350647695267",
>       "options":{
>          "headers":{
>             "Content-Type":"application/json"
>          },
>          "method":"POST",
>          "data":"{\"options\":{}}",
>          "preventCache":true,
>          "handleAs":"json",
>          "timeout":40000
>       },
>       "xhr":{
>          "onerror":null,
>          "readyState":4,
>          "onabort":null,
>          "onload":null,
>          "withCredentials":false,
>          "response":"{\"status\": 401, \"message\": \"None\"}",
>          "statusText":"Unauthorized",
>          "responseText":"{\"status\": 401, \"message\": \"None\"}",
>          "responseXML":null,
>          "responseType":"",
>          "onreadystatechange":null,
>          "onloadend":null,
>          "onloadstart":null,
>          "status":401,
>          "upload":{
>             "onprogress":null,
>             "onerror":null,
>             "onabort":null,
>             "onload":null,
>             "onloadend":null,
>             "onloadstart":null
>          },
>          "onprogress":null
>       },
>       "loaded":34,
>       "total":34,
>       "status":401,
>       "text":"{\"status\": 401, \"message\": \"None\"}",
>       "data":{
>          "status":401,
>          "message":"None"
>       }
>    }
> }
Comment 54 Alexander Kläser univentionstaff 2012-10-19 14:52:39 CEST
(In reply to comment #53)
> Es ist noch ein Problem mit umc/tools:parseError aufgefallen. Durch ein Problem
> in System-Setup wurde beim Setzen der Netzwerkeinstellungen ein 401-Fehler
> zurück gegeben. Nach dem Aufruf von parseError, war status nicht gesetzt. Der
> Aufbau des von Dojo zurück gegebenen Errors war wie folgt:

Bzgl. des Aufbaus des Dojo-Reponse-Objektes siehe auch:

http://livedocs.dojotoolkit.org/dojo/request
Comment 55 Alexander Kläser univentionstaff 2012-10-19 14:57:58 CEST
(In reply to comment #54)
> (In reply to comment #53)
> > Es ist noch ein Problem mit umc/tools:parseError aufgefallen. Durch ein Problem
> > in System-Setup wurde beim Setzen der Netzwerkeinstellungen ein 401-Fehler
> > zurück gegeben. Nach dem Aufruf von parseError, war status nicht gesetzt. Der
> > Aufbau des von Dojo zurück gegebenen Errors war wie folgt:
> 
> Bzgl. des Aufbaus des Dojo-Reponse-Objektes siehe auch:
> 
> http://livedocs.dojotoolkit.org/dojo/request

In parseError müsste daher die Abfrage 

  if (error.data)

besser so sein (?):

  if (error.response.data)
Comment 56 Florian Best univentionstaff 2012-10-19 15:22:25 CEST
(In reply to comment #55)
> (In reply to comment #54)
> > (In reply to comment #53)
> > > Es ist noch ein Problem mit umc/tools:parseError aufgefallen. Durch ein Problem
> > > in System-Setup wurde beim Setzen der Netzwerkeinstellungen ein 401-Fehler
> > > zurück gegeben. Nach dem Aufruf von parseError, war status nicht gesetzt. Der
> > > Aufbau des von Dojo zurück gegebenen Errors war wie folgt:
> > 
> > Bzgl. des Aufbaus des Dojo-Reponse-Objektes siehe auch:
> > 
> > http://livedocs.dojotoolkit.org/dojo/request
> 
> In parseError müsste daher die Abfrage 
> 
>   if (error.data)
> 
> besser so sein (?):
> 
>   if (error.response.data)
Sollte behoben sein…
Comment 57 Dirk Wiesenthal univentionstaff 2012-10-19 16:29:49 CEST
(In reply to comment #56)
> (In reply to comment #55)
> > (In reply to comment #54)
> > > (In reply to comment #53)
> > > > Es ist noch ein Problem mit umc/tools:parseError aufgefallen. Durch ein Problem
> > > > in System-Setup wurde beim Setzen der Netzwerkeinstellungen ein 401-Fehler
> > > > zurück gegeben. Nach dem Aufruf von parseError, war status nicht gesetzt. Der
> > > > Aufbau des von Dojo zurück gegebenen Errors war wie folgt:
> > > 
> > > Bzgl. des Aufbaus des Dojo-Reponse-Objektes siehe auch:
> > > 
> > > http://livedocs.dojotoolkit.org/dojo/request
> > 
> > In parseError müsste daher die Abfrage 
> > 
> >   if (error.data)
> > 
> > besser so sein (?):
> > 
> >   if (error.response.data)
> Sollte behoben sein…

Nein, Problem scheint error.data.message zu sein, denn error.data ist bereits undefined. Hier error.response.data benutzen (auch die Zeile darunter)
Comment 58 Dirk Wiesenthal univentionstaff 2012-10-19 17:47:16 CEST
Ich brauchte es kurz und habe das in
  univention-management-console-frontend 2.0.123-1.478.201210191730

gefixt. Wenn Du damit zufrieden bist (habe ich auf die Schnelle auch nichts übersehen?), kann das wieder auf RESOLVED
Comment 59 Florian Best univentionstaff 2012-10-22 08:30:03 CEST
(In reply to comment #58)
> Ich brauchte es kurz und habe das in
>   univention-management-console-frontend 2.0.123-1.478.201210191730
> 
> gefixt. Wenn Du damit zufrieden bist (habe ich auf die Schnelle auch nichts
> übersehen?), kann das wieder auf RESOLVED
natürlich ;)
Comment 60 Dirk Wiesenthal univentionstaff 2012-10-31 14:39:17 CET
univention-printerserver scheint noch ziemlich überarbeitungsbedürftig:
 * Das Grid für Quota scheint sich nicht automatisch am Anfang zu aktualisieren. Deshalb wird ein Dialog nicht initialisiert.
 * Der QuotaDialog sendet einen GET-Request, d.h. die Seite wird neu geladen
 * Es gibt einen Button, auf dem "nothing" steht, und der sendet einen GET-Request

Es gibt noch so ein paar Ecken und Kanten. Vieles kann damit zusammenhängen, dass ich die ganze Geschichte nicht richtig konfiguriert habe. So beschwert sich ein Programm über fehlende Rechte bei postgres.
Comment 61 Stefan Gohmann univentionstaff 2012-11-07 08:35:32 CET
(In reply to comment #60)
> univention-printerserver scheint noch ziemlich überarbeitungsbedürftig:
>  * Das Grid für Quota scheint sich nicht automatisch am Anfang zu
> aktualisieren. Deshalb wird ein Dialog nicht initialisiert.
>  * Der QuotaDialog sendet einen GET-Request, d.h. die Seite wird neu geladen
>  * Es gibt einen Button, auf dem "nothing" steht, und der sendet einen
> GET-Request
> 
> Es gibt noch so ein paar Ecken und Kanten. Vieles kann damit zusammenhängen,
> dass ich die ganze Geschichte nicht richtig konfiguriert habe. So beschwert
> sich ein Programm über fehlende Rechte bei postgres.

(In reply to comment #60)
> univention-printerserver scheint noch ziemlich überarbeitungsbedürftig:
>  * Das Grid für Quota scheint sich nicht automatisch am Anfang zu
> aktualisieren. Deshalb wird ein Dialog nicht initialisiert.
>  * Der QuotaDialog sendet einen GET-Request, d.h. die Seite wird neu geladen
>  * Es gibt einen Button, auf dem "nothing" steht, und der sendet einen
> GET-Request
> 
> Es gibt noch so ein paar Ecken und Kanten. Vieles kann damit zusammenhängen,
> dass ich die ganze Geschichte nicht richtig konfiguriert habe. So beschwert
> sich ein Programm über fehlende Rechte bei postgres.

Ausgelagert nach Bug #29087
Comment 62 Dirk Wiesenthal univentionstaff 2012-11-13 19:48:53 CET
(In reply to comment #42)
> Comment #15 und Comment #18 können ignoriert werden.

Comment #18 ignorieren? Ich finde, das stört gewaltig, weil sich der title fast über die gesamte Seite legt und dann immer zu sehen ist. Besonders unangenehm bei UDM-Detailseiten mit Validierungsfehlern (weil dann title "<span style="color: red">..." ist).

Einfach meinen Kommentar ignorieren wollen...
Comment 63 Dirk Wiesenthal univentionstaff 2012-11-16 12:39:31 CET
(In reply to comment #62)
> (In reply to comment #42)
> > Comment #15 und Comment #18 können ignoriert werden.
> 
> Comment #18 ignorieren? Ich finde, das stört gewaltig, weil sich der title fast
> über die gesamte Seite legt und dann immer zu sehen ist. Besonders unangenehm
> bei UDM-Detailseiten mit Validierungsfehlern (weil dann title "<span
> style="color: red">..." ist).
> 
> Einfach meinen Kommentar ignorieren wollen...

Fixed in
  univention-management-console-frontend (2.0.163-1)

Damit müssten alle hier aufgezählten Schwierigkeiten behoben sein.
Comment 64 Florian Best univentionstaff 2012-11-19 08:39:59 CET
(In reply to comment #63)
> (In reply to comment #62)
> > (In reply to comment #42)
> > > Comment #15 und Comment #18 können ignoriert werden.
> > 
> > Comment #18 ignorieren? Ich finde, das stört gewaltig, weil sich der title fast
> > über die gesamte Seite legt und dann immer zu sehen ist. Besonders unangenehm
> > bei UDM-Detailseiten mit Validierungsfehlern (weil dann title "<span
> > style="color: red">..." ist).
> > 
> > Einfach meinen Kommentar ignorieren wollen...
> 
> Fixed in
>   univention-management-console-frontend (2.0.163-1)
> 
> Damit müssten alle hier aufgezählten Schwierigkeiten behoben sein.
OK, die Titel erscheinen nicht mehr.
Comment 65 Stefan Gohmann univentionstaff 2012-12-12 21:11:00 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".