Univention Bugzilla – Bug 22848
UMC2: Neustarten-Modul
Last modified: 2011-12-13 15:50:35 CET
Das UMC-Modul "Neustarten" sollte in die neue UMC-Architektur überführt werden.
univention-management-console-module-reboot (1.0.0-1) unstable; urgency=low * Initial release (Bug #22848) Das Modul wurde übernommen. @ QA: Es gibt derzeit noch Probleme mit der Übersetzung. Die in dem Python-Modul stehenden Texte werden nicht ins deutsche übersetzt. Das Problem tritt allerdings auch in allen anderen Modulen auf.
Das sieht soweit gut aus, Starten und Stoppen funktioniert. Drei Dinge sind nur aufgefallen: * Der helptext des Moduls wird nicht angezeigt (die richtige Property ist 'helpText') und sollte auch etwas ausführlicher sein (dem Benutzer beschreiben was er in diesem Modul tun kann) * Wenn der Reboot aus irgendwelchen Gründen nicht möglich ist, dann ist dies für den Benutzer nicht ersichtlich. IMHO sollte im JS-Teil der Rückgabewert des Python-Moduls ausgewertet ('success') und eine entsprechende Meldung angezeigt werden. (* Die title-Property beim Erzeugen der umc.widgets.Page ist an dieser Stelle nicht notwendig.)
Die von Alex angemerkten Punkte wurden behoben. Fixed!
QA: Sieht gut aus. Die Fehlermeldung im JavaScript-Code wird allerdings nicht angezeigt, da das Modul den Status auf einen Fehler-Code setzt und somit der normale Fehlerbehandlungsmechanismus greift.
Die entsprechenden Änderungen wurden wieder rückgängig gemacht. Fixed!
QA: ok → verified
Im syslog eines 3.0 System sehe ich folgendes: Aug 4 17:27:49 master python2.6: Reloading configuration ... Aug 4 17:27:49 master python2.6: Failed to parse syntax definition reboot.xml: mismatched tag: line 7, column 4
univention-management-console-module-reboot (1.0.10-1) unstable; urgency=low * fixed syntax error in syntax definition; Bug #22848 Fixed!
(In reply to comment #8) > univention-management-console-module-reboot (1.0.10-1) unstable; urgency=low > > * fixed syntax error in syntax definition; Bug #22848 > > Fixed! → OK, die Syntax der XML-Dateien ist korrekt.
In der Einrückung des Python-Codes für die reboot-Methode werden Tabs und Spaces gemixt. Beim Lesen der Codes ist nicht klar, was eigentlich ausgeführt wird.
(In reply to comment #10) > In der Einrückung des Python-Codes für die reboot-Methode werden Tabs und > Spaces gemixt. Beim Lesen der Codes ist nicht klar, was eigentlich ausgeführt > wird. Das ist angepasst worden. Des Weiteren habe ich die nicht funktionierende Übersetzung im Backend funktionsfähig gemacht, sowie relative Pfade durch absolute ersetzt (bzgl. Programmaufrufe)
Noch zwei Kleinigkeiten: Es sollte eine Rückfrage geben, ob der Reboot wirklich durchgeführt werden soll. Derzeit gibt es wohl einen Python-Exception, wenn Umlaute in der Systemnachricht vorkommen. Am einfachsten sollte die Syntax beschränkt werden, so dass Umlaute nicht erlaubt werden.
Ich bin für eine Unterstützung von Umlauten, auch wenn es etwas aufwendiger ist. Es kommt bei den Kunden nicht gut an, wenn sie einen deutschen Satz eingeben und aus diesem anschließend die Umlaute entfernen müssen.
* Bestätigungsdialog wurde hinzugefügt * Umlaute sind nun möglich Fixed!
(In reply to comment #10) > In der Einrückung des Python-Codes für die reboot-Methode werden Tabs und > Spaces gemixt. Beim Lesen der Codes ist nicht klar, was eigentlich ausgeführt > wird. → OK, wurde angepasst, Übersetzung im Backend funktioniert jetzt. (In reply to comment #11) > ... > Des Weiteren habe ich die nicht funktionierende Übersetzung im Backend > funktionsfähig gemacht, sowie relative Pfade durch absolute ersetzt (bzgl. > Programmaufrufe) → OK, die Pfade sind absolut (In reply to comment #14) > * Bestätigungsdialog wurde hinzugefügt > * Umlaute sind nun möglich > > Fixed! → OK, Dialog ist da, Meldungen mit Umlauten ist auch möglich; aufgefallen ist, dass der Bestätigungsdialog keinen Default-Button besitzt (→ Bug 25100) → VERIFIED
UCS 3.0-0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer neueren Version von UCS erneut auftreten, so sollte dieser Bug dupliziert werden: "Clone This Bug"