Bug 48323 – Don't map the LDAP group description to the course_name

Bug 48323 - Don't map the LDAP group description to the course_name


Summary:	Don't map the LDAP group description to the course_name

Status:	VERIFIED FIXED

Product:	UCS@school
Classification:	Unclassified
Component:	Apple School Manager
Version:	unspecified
Hardware:	Other Mac OS X 10.1

Importance:	P5 normal (vote)
Target Milestone:	---
Assigned To:	Ole Schwiegert
QA Contact:	Tobias Wenzel

URL:
Keywords:

Depends on:
Blocks:	52343
	Show dependency tree / graph

Reported:	2018-12-13 11:22 CET by Michel Smidt
Modified:	2020-11-10 16:02 CET (History)
CC List:	2 users (show)

See Also:
What kind of report is it?:	Feature Request
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?:	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 Michel Smidt 2018-12-13 11:22:46 CET

In courses.csv the description of a group (LDAP field description) is mapped to the CSV field "course_name". This causes e.g. Zuludesk to display the description as a group name.

Comment 1 Ole Schwiegert

2020-09-01 09:30:55 CEST

Fixed in main branch (4.4). Tests were also adapted.

Build will be done with Bug #48346.

Please QA

Comment 2 Ole Schwiegert

2020-09-01 09:31:48 CEST

Further information: course_name will now be mapped to the groups name without the school specific part. For example: SCHOOL1-1a will end up as course name 1a

Comment 3 Tobias Wenzel

2020-09-01 13:43:20 CEST

QA -> All OK -> Reopen -> wait for build with Bug #48346

- changelog -> OK

Code:
 - course.py -> OK
 - test modifications -> OK

Test 02_model_course passes -> OK


In [14]: AsmCourse.header
Out[14]: (u'course_id', u'course_number', u'course_name', u'location_id')

In [5]: got_course = course.as_csv_line()
In [6]: got_course
Out[6]: 
['DEMOSCHOOL-demo_class2', 'DEMOSCHOOL-demo_class2', u'demo_class2', 'DEMOSCHOOL']

In [12]: got_workgroup = workgroup.as_csv_line()
In [13]: got_workgroup
Out[13]: ['DEMOSCHOOL-test-group', 'DEMOSCHOOL-test-group', u'test-group', 'DEMOSCHOOL']

Comment 4 Ole Schwiegert

2020-09-03 09:05:33 CEST

Build and uploaded to TestAppCenter in version 2.2

Comment 5 Tobias Wenzel

2020-09-03 09:18:12 CEST

tested with version 2.2 from TestAppCenter -> All OK -> VERIFY