Bug 47028 – [4.2] ucs-school-import: Attributes of ImportUser.old_user updated to new user

Bug 47028 - [4.2] ucs-school-import: Attributes of ImportUser.old_user updated to new user


Summary:	[4.2] ucs-school-import: Attributes of ImportUser.old_user updated to new user

Status:	CLOSED FIXED

Product:	UCS@school
Classification:	Unclassified
Component:	Import scripts
Version:	UCS@school 4.2
Hardware:	Other Linux

Importance:	P5 normal (vote)
Target Milestone:	UCS@school 4.2 v10
Assigned To:	Daniel Tröder
QA Contact:	Sönke Schwardt-Krummrich

URL:
Keywords:

Depends on:	47027
Blocks:
	Show dependency tree / graph

Reported:	2018-05-17 14:58 CEST by Daniel Tröder
Modified:	2018-07-04 18:07 CEST (History)
CC List:	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:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Daniel Tröder

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

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

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

2018-06-01 22:46:14 CEST

OK: functional test
OK: ucs-test
→ VERIFIED

Comment 5 Sönke Schwardt-Krummrich

2018-07-04 18:07:46 CEST

UCS@school 4.2 v10 has been released.

https://docs.software-univention.de/changelog-ucsschool-4.2v10-de.html

If this error occurs again, please clone this bug.