Bug 29770 - Refresh wird beim Öffnen des Setup-Moduls nicht ausgeführt
Refresh wird beim Öffnen des Setup-Moduls nicht ausgeführt
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC - Basic settings
UCS 3.0
Other Linux
: P2 normal (vote)
: UCS 3.1-0-errata
Assigned To: Dirk Wiesenthal
Alexander Kläser
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-12-12 11:33 CET by Alexander Kläser
Modified: 2012-12-18 13:50 CET (History)
3 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): Troubleshooting
Max CVSS v3 score:
klaeser: Patch_Available+


Attachments
Screenshot System-Setup (23.79 KB, image/png)
2012-12-12 11:33 CET, Alexander Kläser
Details
Patch für System-Setup .js-File (1.30 KB, patch)
2012-12-12 12:06 CET, 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-12-12 11:33:42 CET
Created attachment 4910 [details]
Screenshot System-Setup

Zum einen gibt es einen Syntax-Fehler in SystemRolePage.js (ein Komma zu viel),
aber das scheint das eigentliche Problem noch nicht zu beheben.
Comment 1 Alexander Kläser univentionstaff 2012-12-12 11:34:06 CET
Nach Klick auf "Zurücksetzen" werden die Werte neu geladen.
Comment 2 Alexander Kläser univentionstaff 2012-12-12 12:06:55 CET
Created attachment 4911 [details]
Patch für System-Setup .js-File

Anbei eine mögliche Lösung. Scheinbar kommt JS mit den Events durcheinander, da der Handler regisriert wird und dann die Werte initial gesetzt werden. Werden die Werte zuerst gesetzt und dann der Handler registriert, scheint alles gut zu laufen.
Comment 3 Alexander Kläser univentionstaff 2012-12-12 12:08:10 CET
Ups, der Patch muss reversed werden (mit patch -R ...).
Comment 4 Jascha Geerds univentionstaff 2012-12-13 09:59:24 CET
Ich denke dieser Fix sollte möglichst schnell als Errata released werden:

1) Wenn man das Basis-Einstellungen Modul öffnet, werden die o.g. Werte nicht geladen.
2) Klickt der Benutzer nun auf "Änderungen übernehmen", dann werden die "kaputten" Werte auf dem System gesetzt.
3) Im Anschluss ist das Modul unbenutzbar, weil aufgrund der "kaputten" Werte die Ladeanimation in der Endlosschleife läuft.
Comment 5 Jascha Geerds univentionstaff 2012-12-13 10:17:40 CET
(In reply to comment #1)
> Nach Klick auf "Zurücksetzen" werden die Werte neu geladen.

Bei mir werden nicht alle Werte nachgeladen. Die ComboBox mit der standard Systemsprache bleibt weiterhin leer, sodass dort keine Auswahl vorgenommen werden kann. Die leere ComboBox wird beim Speichern jedoch als gewollte Änderung angezeigt, d.h. es ist unmöglich das Modul zu benutzen ohne die kaputte Änderung zu übertragen. Danach lässt sich das Modul nicht mehr öffnen
Comment 6 Dirk Wiesenthal univentionstaff 2012-12-18 12:17:40 CET
Patch wurde eingespielt. Paket baut.
Comment 7 Janis Meybohm univentionstaff 2012-12-18 12:26:12 CET
Workaround um die UMC schnell wieder einsatzfähig zu bekommen:

# ucr set locale="$(sed -ne "s/locales='\(.*\)'/\1/p" </etc/univention/installation_profile)"
# ucr set locale/default="$(sed -ne "s/locale_default='\(.*\)'/\1/p" </etc/univention/installation_profile)"
# /etc/init.d/univention-management-console-web-server restart
# /etc/init.d/univention-management-console-server restart
Comment 8 Alexander Kläser univentionstaff 2012-12-18 13:13:16 CET
Funktions-QA → OK
 
YAML-Datei: 2012-12-18-univention-system-setup.yaml → OK

Das Errata kann veröffentlicht werden.
Comment 9 Janek Walkenhorst univentionstaff 2012-12-18 13:50:29 CET
http://errata.univention.de/3.1-errata2.html