Bug 27991 - Probleme bei Schule-OUs mit einem Bindestrich/Minus im Namen
Probleme bei Schule-OUs mit einem Bindestrich/Minus im Namen
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC
UCS@school 3.0
Other Linux
: P3 normal (vote)
: UCS@school 3.0-errata
Assigned To: Jascha Geerds
Alexander Kläser
:
: 27964 27968 27971 27972 (view as bug list)
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-07-19 12:18 CEST by Alexander Kläser
Modified: 2012-11-09 15:51 CET (History)
2 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

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Kläser univentionstaff 2012-07-19 12:18:45 CEST
Wird derzeit ein Minus (/ein Bindestrich) im Namen der Schul-OU angegeben, so gibt es so treten an einigen Stellen Probleme auf. Die bisher angelegten Bugs werden als Duplicate dieses hier geschlossen.
Comment 1 Alexander Kläser univentionstaff 2012-07-19 12:19:25 CEST
Zumindest könnte das Anlegen von OUs mit "-" verhindert werden in den Wizards und im Import-Skript.
Comment 2 Alexander Kläser univentionstaff 2012-07-19 12:19:58 CEST
*** Bug 27971 has been marked as a duplicate of this bug. ***
Comment 3 Alexander Kläser univentionstaff 2012-07-19 12:23:19 CEST
*** Bug 27972 has been marked as a duplicate of this bug. ***
Comment 4 Alexander Kläser univentionstaff 2012-07-19 12:37:08 CEST
*** Bug 27968 has been marked as a duplicate of this bug. ***
Comment 5 Alexander Kläser univentionstaff 2012-07-19 12:37:17 CEST
*** Bug 27964 has been marked as a duplicate of this bug. ***
Comment 6 Alexander Kläser univentionstaff 2012-07-24 17:17:18 CEST
(In reply to comment #1)
> Zumindest könnte das Anlegen von OUs mit "-" verhindert werden in den Wizards
> und im Import-Skript.

Zunächst wird dies so umgesetzt, später wird auch ein Minus möglich sein (→ siehe Bug 28058).
Comment 7 Jascha Geerds univentionstaff 2012-08-02 11:16:35 CEST
Für diesen Bug wurden zwei Pakete angepasst. Im 'ucs-school-import'-Paket wurde das Import-Skript angepasst, sodass es auch hier nicht mehr möglich ist auf der Kommandozeile Schulen mit einem Bindestrich im Namen anzulegen.

Des Weiteren wurde 'ucs-school-umc-wizards' angepasst, sodass es im Frontend und Backend abgefangen wird.

- Bug wurde im SVN gefixed
- Patches wurden eingescheckt
- Pakete sind im 'ucsschool-errata'-Scope gebaut
- 2x YAML wurden hinzugefügt
- Changelog für UCS@school 3.1 wurde angepasst (im UMC Bereich, ich denke das reicht an dieser Stelle)
Comment 8 Alexander Kläser univentionstaff 2012-08-02 16:18:24 CEST
Kannst du bitte noch die Fehlerausgabe des Import-Skriptes mit ins Log schreiben (als WARN vielleicht?). Das würde bei einer Fehlersuche enorm helfen.
Comment 9 Alexander Kläser univentionstaff 2012-08-02 18:14:26 CEST
"To avoid problems in the UMC it is no longer allowed to use hyphens in the name of a school."

Würde ich nochmal etwas umformulieren, bspw.:

"This update disables the creation of school OUs that contain hyphens in their name in order to avoid errors for some UMC modules."
Comment 10 Alexander Kläser univentionstaff 2012-08-02 18:20:44 CEST
(In reply to comment #9)
> ...
> Würde ich nochmal etwas umformulieren, bspw.:

→ Bitte auch entsprechend im Changelog-Eintrag anpassen.
Comment 11 Jascha Geerds univentionstaff 2012-08-03 09:46:45 CEST
(In reply to comment #10)
> (In reply to comment #9)
> > ...
> > Würde ich nochmal etwas umformulieren, bspw.:
> 
> → Bitte auch entsprechend im Changelog-Eintrag anpassen.

Die Anpassungen an dem Changelog/YAML habe ich schon einmal durchgeführt.
Comment 12 Jascha Geerds univentionstaff 2012-08-03 13:07:33 CEST
Die Anpassungen wurden vorgenommen. Das Paket ist gebaut und die YAML wurde noch einmal angepasst (bzgl. der Paketversion)
Comment 13 Jascha Geerds univentionstaff 2012-08-03 13:18:19 CEST
Ich habe die YAML für ucs-school-import entfernt, da es eine Überschneidung mit einem weiteren errata gab, sodass das Paket bereits draußen ist.
Comment 14 Alexander Kläser univentionstaff 2012-08-03 13:29:36 CEST
Die Fehlermeldung wird noch nicht korrekt ausgegeben (bspw. beim Anlegen eines Benutzers mit falscher E-Mail-Adresse):

process = subprocess.Popen([script, tmpfile.name], subprocess.PIPE)

→ es fehlt ein "stdout="
Comment 15 Alexander Kläser univentionstaff 2012-08-03 13:30:04 CEST
Die Änderungen in Changelog-/YAML-Dateien sind soweit ok.
Comment 16 Alexander Kläser univentionstaff 2012-08-03 14:54:12 CEST
(In reply to comment #13)
> Ich habe die YAML für ucs-school-import entfernt, da es eine Überschneidung mit
> einem weiteren errata gab, sodass das Paket bereits draußen ist.

→ OK, der Patch wurde aus Versehen in das letzte Erratum eingebaut. Die Änderungen funktionieren, insofern kein Problem.


(In reply to comment #7)
> ...
> - Bug wurde im SVN gefixed

→ OK

> - Patches wurden eingescheckt

→ OK

> - Pakete sind im 'ucsschool-errata'-Scope gebaut

→ OK

> - 2x YAML wurden hinzugefügt

→ YAML-Datei OK

> - Changelog für UCS@school 3.1 wurde angepasst (im UMC Bereich, ich denke das
> reicht an dieser Stelle)

→ OK

(In reply to comment #12)
> Die Anpassungen wurden vorgenommen. Das Paket ist gebaut und die YAML wurde
> noch einmal angepasst (bzgl. der Paketversion)

→ OK, Fehlermeldungen werden jetzt geloggt.
Comment 17 Alexander Kläser univentionstaff 2012-08-03 15:52:37 CEST
Erratum wurde veröffentlicht.
Comment 18 Janek Walkenhorst univentionstaff 2012-11-09 15:51:30 CET
http://errata.univention.de/ucsschool-errata11.html