Bug 34173 - Unlimited OU names
Unlimited OU names
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Wizards
UCS@school 3.2
Other Linux
: P5 normal (vote)
: UCS@school 3.2 R2
Assigned To: Sönke Schwardt-Krummrich
Florian Best
:
Depends on:
Blocks: 34264 34417
  Show dependency treegraph
 
Reported: 2014-02-21 12:18 CET by Sönke Schwardt-Krummrich
Modified: 2014-06-12 09:20 CEST (History)
1 user (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 Sönke Schwardt-Krummrich univentionstaff 2014-02-21 12:18:28 CET
Currently there are certain limitations for OU names. To avoid these, it should be added a "displayName"-like attribute to the school OUs. This attribute is shown within the UMC instead of the LDAP OU name.

The UCS@school wizard for adding a new school OU has to be modified:
It should display two text widgets: the first one asks for the "displayName" of the OU. Upon entering any value the javascript uses a to be defined algorithm and creates a UCS@school conform LDAP OU name which is displayed in the second text widget. The user is able to modify the second widget afterwards.
OU name and displayName are passed to the import script "create_ou". The wizard has to continue to check for OU name conflicts before calling create_ou.

A new argument option has to be added to the create_ou script which sets the displayName for a new/existing OU.

The displayName has to be shown at the OU object via extended attribute.
Comment 1 Sönke Schwardt-Krummrich univentionstaff 2014-03-04 17:54:16 CET
1) The schema package ucs-school-import-schema (part of ucs-school-import) has 
been modified to support attribute "displayName" at school OU objects.

2) The CLI tool create_ou now supports the new argument "--displayName" for 
defining the OU's display name during creation/modification of OU objects.

3) A new extended attribute has been added on "UCS@school" tab at OU objects. It shows the OU's display name and is only available if the option "UCSschool-School-OU" has been activated for the corresponding OU object. For better usability, the existing "fileserver" group has been moved "down" by increasing the group position from 1 to 2. Existing installations will be updated via join script.

4) The UCS@school new OU wizard has been updated accordingly. The dialog now shows the display name widget at the top of the form. The user is able to enter any UTF-8 string as display name. When the user switches to the second widget (the original OU name widget for the LDAP name), javascript code fetches the display name, removes whitespaces, all non-characters and non-digits and fills the suggestion into the original ou name textbox. The user is able to modify the suggestion afterwards. The display name and the OU name are required values.

5) Two tests have been added to ucs-test-ucsschool for testing the displayName attribute with create_ou and udm.

6) The path to the udm-test wrapper has been defined as constant in ucs-test.

7) Changelog entries have been inserted to changelog-ucsschool-3.2r2.xml

Still missing: 
- the UCS@school installer has be updated to ask for a display name
- the UCS@school modules have to display the display name instead of the OU 
  name itself
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2014-03-28 15:30:31 CET
> Still missing: 
> - the UCS@school installer has be updated to ask for a display name

→ FIXED

The package ucs-school-umc-installer has been updated:
ucs-school-umc-installer (2.0.2-1)

The UCS@school installer has been updated in the same way as the create OU wizard.
Changelog entry has been added to changelog-ucsschool-3.2r2.xml

> - the UCS@school modules have to display the display name instead of the OU 
>   name itself

This has been seperated into Bug #34264
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2014-04-11 09:39:50 CEST
The widget names/description has to be changed to "School name" and "Internal school id".
Comment 4 Sönke Schwardt-Krummrich univentionstaff 2014-04-24 17:04:46 CEST
(In reply to Sönke Schwardt-Krummrich from comment #3)
> The widget names/description has to be changed to "School name" and
> "Internal school id".

"Schulname" / "Name of the school"
"Schulkürzel" / "School abbreviation"

The tooltip for the latter one has also been updated.

ucs-school-umc-installer (2.0.4-1)
ucs-school-umc-wizards (4.0.5-1)
Comment 5 Florian Best univentionstaff 2014-05-08 13:42:48 CEST
(In reply to Sönke Schwardt-Krummrich from comment #1)
> 1) The schema package ucs-school-import-schema (part of ucs-school-import)
> has 
> been modified to support attribute "displayName" at school OU objects.
OK

> 2) The CLI tool create_ou now supports the new argument "--displayName" for 
> defining the OU's display name during creation/modification of OU objects.
OK

> 3) A new extended attribute has been added on "UCS@school" tab at OU
> objects. It shows the OU's display name and is only available if the option
> "UCSschool-School-OU" has been activated for the corresponding OU object.
OK
> For better usability, the existing "fileserver" group has been moved "down"
> by increasing the group position from 1 to 2.
OK
> Existing installations will be updated via join script.
OK

> 4) The UCS@school new OU wizard has been updated accordingly. The dialog now
> shows the display name widget at the top of the form. The user is able to
> enter any UTF-8 string as display name. When the user switches to the second
> widget (the original OU name widget for the LDAP name), javascript code
> fetches the display name, removes whitespaces, all non-characters and
> non-digits and fills the suggestion into the original ou name textbox. The
> user is able to modify the suggestion afterwards. The display name and the
> OU name are required values.
OK

> 5) Two tests have been added to ucs-test-ucsschool for testing the
> displayName attribute with create_ou and udm.
OK

> 6) The path to the udm-test wrapper has been defined as constant in ucs-test.
OK

> 7) Changelog entries have been inserted to changelog-ucsschool-3.2r2.xml
OK

(In reply to Sönke Schwardt-Krummrich from comment #4)
> "Schulname" / "Name of the school"
> "Schulkürzel" / "School abbreviation"
OK

> The tooltip for the latter one has also been updated.
OK

> > - the UCS@school installer has be updated to ask for a display name
> The UCS@school installer has been updated in the same way as the create OU
> wizard.
→ REOPEN
This fails if the displayName or the DNname contains non-ascii charachters.
schoolinstaller/__init__.py
        if type(ou) != str or type(displayName) != str:
Comment 6 Sönke Schwardt-Krummrich univentionstaff 2014-05-08 14:10:04 CEST
(In reply to Florian Best from comment #5)
> > > - the UCS@school installer has be updated to ask for a display name
> > The UCS@school installer has been updated in the same way as the create OU
> > wizard.
> → REOPEN
> This fails if the displayName or the DNname contains non-ascii charachters.
> schoolinstaller/__init__.py
>         if type(ou) != str or type(displayName) != str:

The bogus check has been removed completely.
ucs-school-umc-installer (2.0.8-1) unstable; urgency=low

Package has been rebuilt and announced.
Comment 7 Florian Best univentionstaff 2014-05-08 14:47:52 CEST
(In reply to Sönke Schwardt-Krummrich from comment #6)
> The bogus check has been removed completely.
> ucs-school-umc-installer (2.0.8-1) unstable; urgency=low

OK
Comment 8 Sönke Schwardt-Krummrich univentionstaff 2014-06-12 09:19:28 CEST
UCS@school 3.2 R2 has been released:
http://docs.univention.de/release-notes-ucsschool-3.2R2-de.html

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