Bug 48429 - School class get_all does not work with filter_str
School class get_all does not work with filter_str
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: Ucsschool-lib
UCS@school 4.3
Other Linux
: P5 normal (vote)
: UCS@school 4.3 v6-errata
Assigned To: Daniel Tröder
Ole Schwiegert
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2019-01-07 10:45 CET by Ole Schwiegert
Modified: 2019-01-16 15:52 CET (History)
2 users (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?: 3: Will affect average number of 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.103
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 Ole Schwiegert univentionstaff 2019-01-07 10:45:53 CET
If you use School.get_all(lo, filter_str='something') instead of correctly filtering the available schools, all organisational units in the LDAP are returned regardless of the content of the filter string.

If an empty filter string is provided, it works correctly like not providing any at all
Comment 1 Daniel Tröder univentionstaff 2019-01-09 17:36:19 CET
[4.3] 3b6cf34ab Bug #48429: fix get_all() for School objects

Package: ucs-school-lib
Version: 11.0.2-2A~4.3.0.201901091714
Branch: ucs_4.3-0
Scope: ucs-school-4.3

[4.4] 151dab230 Bug #48429: fix get_all() for School objects

Package: ucs-school-lib
Version: 12.0.1-2A~4.4.0.201901091734
Branch: ucs_4.4-0
Scope: ucs-school-4.4
Comment 2 Daniel Tröder univentionstaff 2019-01-09 17:43:51 CET
[4.3 d69c90072] Bug #48429: advisory update
Comment 3 Ole Schwiegert univentionstaff 2019-01-10 08:36:16 CET
Changelog: OK
Advisory: MISSING -> fixed
Package Installs: OK
Bug fixed: OK

I was able to provide a filter string in the get_all() method which worked as expected.

Both for the 4.3 and 4.4 package
Comment 4 Sönke Schwardt-Krummrich univentionstaff 2019-01-16 15:52:01 CET
UCS@school 4.3 v6 has been released.

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

If this error occurs again, please clone this bug.