Bug 47450 - don't move user when importing into additional school with alphanum lower name
don't move user when importing into additional school with alphanum lower name
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: Import scripts
UCS@school 4.3
Other Linux
: P5 normal (vote)
: UCS@school 4.3 v5
Assigned To: Daniel Tröder
Sönke Schwardt-Krummrich
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2018-08-03 11:26 CEST by Daniel Tröder
Modified: 2018-09-11 11:34 CEST (History)
0 users

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 2: Improvement: Would be a product improvement
Who will be affected by this bug?: 2: Will only affect a few installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.046
Enterprise Customer affected?:
School Customer affected?: Yes
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 Daniel Tröder univentionstaff 2018-08-03 11:26:36 CEST
When the import data (CSV) or configuration (JSON) does not explicitly set the 'school' attribute, the alphanumerical first from the 'schools' attribute is used.

When a user exists in OU 'B' and is imported *additionally* into OU 'A', then a unnecessary move to 'A' is done.

The desired behavior is to not move a user into another OU, unless the import *removes* it from the current one.
Comment 1 Daniel Tröder univentionstaff 2018-08-03 12:41:23 CEST
The "school" attribute is now adapted, when the user would unnecessarily be moved.

Added a ucs-test 240_import_user_dont_move_to_additional.

[4.3] b950f9f81 Bug #47450: don't move user when importing into additional school with alphanumerical lower name
[4.3] 4facea627 Bug #47450: add test to not move when importing OU with alphanum lower name
[4.3] 4358f3726 Bug #47450: advisory
[4.3] edc0ab3c5 Bug #47450: deduplicate schools attribute
[4.3] 104ff1f0f Bug #47450: advisory
Comment 2 Sönke Schwardt-Krummrich univentionstaff 2018-09-04 14:10:55 CEST
OK: code change
OK: functional change
OK: tests
OK: changelog entry
OK: advisory
OK: package built and installable
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2018-09-11 11:34:18 CEST
UCS@school 4.3 v5 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.3v5-de.html

If this error occurs again, please clone this bug.