Bug 26406 - [umc-create-module] Anpassungen der UMC-Modul-Templates
[umc-create-module] Anpassungen der UMC-Modul-Templates
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UCS 3.0
Other Linux
: P4 enhancement (vote)
: UCS 3.0-2
Assigned To: Florian Best
Andreas Büsching
: interim-1
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-08 10:48 CET by Florian Best
Modified: 2012-07-20 15:25 CEST (History)
4 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): Cleanup
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Best univentionstaff 2012-03-08 10:48:53 CET
Im simple_form Template ist die Einrückung ein paar mal nicht mit Tabs sondern mit Leerzeichen gemacht. Das stört, wenn im Editor eine Einrückung für Tabs != 4 eingestellt ist.
$ grep '    ' management/univention-management-console/umc-module-templates/simple_form/umc/js/MODULEID.js

Lösung:
sed -i 's/    /\t/g' management/univention-management-console/umc-module-templates/simple_form/umc/js/MODULEID.js
Comment 1 Florian Best univentionstaff 2012-03-09 12:01:56 CET
noch eine Sache ist mir aufgefallen:
in *grid*/umc/js/_MODULEID/DetailPage.js gibt es eine methode mit UDM-spezifischen argumenten:
162 »   onClose: function(dn, objectType) {
Comment 2 Jascha Geerds univentionstaff 2012-03-21 10:12:06 CET
An 1-2 Stellen waren auch noch Leerzeichen am Zeilenende bzw. leere Zeilen am Dateiende. Nicht dramatisch, aber unnötig
Comment 3 Florian Best univentionstaff 2012-03-21 10:37:35 CET
in debian/control steht XS-Python-Version: 2.6
ucslint sagt, dass wäre obsolete, könnte demnach auch rausgelöscht werden.
Comment 4 Florian Best univentionstaff 2012-03-21 16:52:03 CET
kleiner typo
Index: DetailPage.js
===================================================================
--- DetailPage.js       (Revision 31315)
+++ DetailPage.js       (Arbeitskopie)
@@ -59,7 +59,7 @@
 
                // Set the opacity for the standby animation to 100% in order to mask
                // GUI changes when the module is opened. Call this.standby(true|false)
-               // to enabled/disable the animation.
+               // to enable/disable the animation.
                this.standbyOpacity = 1;
 
                // set the page header
Comment 5 Andreas Büsching univentionstaff 2012-03-22 08:47:52 CET
(In reply to comment #3)
> in debian/control steht XS-Python-Version: 2.6
> ucslint sagt, dass wäre obsolete, könnte demnach auch rausgelöscht werden.

Das ist für uns die Möglichkeit zu bestimmen, dass die Pakete nur für Python 2.6 gebaut werden (UMC-Module setzen Python 2.6 voraus). Da das momentan alle Pakete so machen würde ich das lassen
Comment 6 Alexander Kläser univentionstaff 2012-03-29 16:06:15 CEST
Derzeit wird in den Templates das Join-Skript im postinst-File auch nicht automatisch auf Master und Backup aufgerufen zu werden. Das kann auch noch angepasst werden.
Comment 7 Florian Best univentionstaff 2012-04-02 13:09:36 CEST
Änderungen wurden in Revision 31874 gemacht:
univention-management-console (4.0.208-1) unstable; urgency=low
  * fixed code indention in umc-create-module templates (Bug #26406)

(In reply to comment #6)
> Derzeit wird in den Templates das Join-Skript im postinst-File auch nicht
> automatisch auf Master und Backup aufgerufen zu werden. Das kann auch noch
> angepasst werden.
Das ist nicht nötig, wird automatisch gemacht.

(In reply to comment #5)
> Das ist für uns die Möglichkeit zu bestimmen, dass die Pakete nur für Python
> 2.6 gebaut werden (UMC-Module setzen Python 2.6 voraus). Da das momentan alle
> Pakete so machen würde ich das lassen
Ok, ist so belassen.

(In reply to comment #2)
> An 1-2 Stellen waren auch noch Leerzeichen am Zeilenende bzw. leere Zeilen am
> Dateiende. Nicht dramatisch, aber unnötig
das wurde auch beachtet
Comment 8 Andreas Büsching univentionstaff 2012-06-18 13:41:27 CEST
Die Korrekturen wurden eingebaut. Changelog-Eintrag existiert
Comment 9 Stefan Gohmann univentionstaff 2012-07-20 15:25:20 CEST
UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".