Bug 32337 - "Add User"-Wizard creates a new class instead of adding user to the chosen one
"Add User"-Wizard creates a new class instead of adding user to the chosen one
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Wizards
UCS@school 3.1 R2
Other Linux
: P3 normal (vote)
: UCS@school 3.1 R2 Errata
Assigned To: Florian Best
Sönke Schwardt-Krummrich
:
Depends on:
Blocks: 32430
  Show dependency treegraph
 
Reported: 2013-08-26 08:57 CEST by Michael Grandjean
Modified: 2013-10-07 20:17 CEST (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):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Grandjean univentionstaff 2013-08-26 08:57:58 CEST
I have seen this in two customer environments:

- classes/groups exist without OU-prefix (e.g. klasse10a)
- add a new user with the wizard, assign this user to the said class (klasse10a)
- instead of adding the user to the existing class, a new class is created WITH the OU-prefix (e.g. GYM123-klasse10a) and its only member is the newly created user 

This might be related to Bug 31663
Comment 1 Ingo Steuwer univentionstaff 2013-08-26 16:18:11 CEST
As part of the process, a new group share will also be created.
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2013-08-29 15:10:49 CEST
For class names Univention recommends to add a OU prefix for class names:

"""Es ist zu beachten, dass die Klassennamen domänenweit eindeutig sein müssen. Das heißt eine Klasse 1A kann nicht in mehreren OUs verwendet werden. Daher sollte jedem Klassennamen die OU vorangestellt werden. Bei der Erstellung von Klassen über das UMC-Modul Klasse hinzufügen geschieht dies automatisch."""
Source: 
http://docs.univention.de/ucsschool-handbuch-3.1.html#school:import:general

The UMC wizard should use both groups "Klasse10a" and "GYM123-Klasse10a" but prefer the one *with* OU prefix → check if "$OU-$GROUP" exists, if not use "$GROUP".

Other changes would be too large for an erratum.

In Bug 32430 this problem will be discussed again for UCS@school 3.2
Comment 3 Michael Grandjean univentionstaff 2013-08-29 15:32:04 CEST
Upper/lowercase should be ignored for group prefix
Comment 4 Florian Best univentionstaff 2013-10-01 10:54:10 CEST
Fixed in:

Package: ucs-school-umc-wizards
Version: 3.0.1-1.73.201310011037
Branch: ucs_3.1-0
Scope: ucsschool-3.1-R2

Package: ucs-school-umc-wizards
Version: 4.0.0-1.74.201310011047
Branch: ucs_3.2-0
Scope: ucs-school-3.2

3.2 changelog created
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2013-10-02 16:04:34 CEST
Added new users and selected a class that
- contained no prefix → user has been added; no new group has been created
- contained an ou prefix → user has been added; no new group has been created
- existed with and without ou prefix → user has been added to
  group with ou prefix; no other new group has been created

OK: 3.1 code diff
FIXED: 3.1 changelog
OK: 3.1 package version
OK: 3.1 test
OK: 3.1 package in repo

OK: 3.2 code diff
FIXED: 3.2 changelog
OK: 3.2 package version
OK: 3.2 test

* Changelog entry in 3.2 had a typo → fixed
* Changelog entry in 3.1 was missing. Copied it from 3.2.

→ VERIFIED
Comment 6 Sönke Schwardt-Krummrich univentionstaff 2013-10-07 20:17:57 CEST
UCS@school 3.1 R2-2 has been released:
http://download.univention.de/doc/release-notes-ucsschool-3.1-rev2-2.pdf

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