Bug 47028 - [4.2] ucs-school-import: Attributes of ImportUser.old_user updated to new user
[4.2] ucs-school-import: Attributes of ImportUser.old_user updated to new user
Product: UCS@school
Classification: Unclassified
Component: Import scripts
UCS@school 4.2
Other Linux
: P5 normal (vote)
: UCS@school 4.2 v10
Assigned To: Daniel Tröder
Sönke Schwardt-Krummrich
Depends on: 47027
  Show dependency treegraph
Reported: 2018-05-17 14:58 CEST by Daniel Tröder
Modified: 2018-07-04 18:07 CEST (History)
1 user (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
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.069
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:


Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Tröder univentionstaff 2018-05-17 14:58:50 CEST
+++ This bug was initially created as a clone of Bug #47027 +++

When modifying an existing user during an import, the attribute "old_user" of ImportUser objects should be the unmodified ImportUser object loaded from LDAP. It is however updated with input data (from a CSV file). Hooks and other code can thus not compare the old user object and the new one.
Comment 2 Daniel Tröder univentionstaff 2018-05-17 16:45:40 CEST
A copy of the user object loaded from LDAP is now created for "old_user", before updating it with data from input.

[4.2] 5b9a0277 Bug #47028: fix old_user attributes being overwritten
[4.2] be9aada2 Bug #47028: changelog
[4.2] 0f812507 Bug #47028: advisory
Comment 3 Sönke Schwardt-Krummrich univentionstaff 2018-05-31 18:12:37 CEST
OK: code change
??: functional test
??: ucs-test
OK: advisory

[4.2 10ea24169] Bug #47028: add 234_modify_user_data_in_pyhooks
Comment 4 Sönke Schwardt-Krummrich univentionstaff 2018-06-01 22:46:14 CEST
OK: functional test
OK: ucs-test
Comment 5 Sönke Schwardt-Krummrich univentionstaff 2018-07-04 18:07:46 CEST
UCS@school 4.2 v10 has been released.


If this error occurs again, please clone this bug.