Bug 41300 - Disable autosearch for new wizards in multi-school environments
Disable autosearch for new wizards in multi-school environments
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC - Wizards
UCS@school 4.1 R2
Other Linux
: P5 enhancement (vote)
: UCS@school 4.1 R2 vXXX
Assigned To: Daniel Tröder
Florian Best
: interim-1
Depends on: 36031
Blocks:
  Show dependency treegraph
 
Reported: 2016-05-19 15:54 CEST by Michael Grandjean
Modified: 2016-10-04 13:24 CEST (History)
8 users (show)

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?: 4: Will affect most 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.091
Enterprise Customer affected?:
School Customer affected?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional): External feedback, Large environments, Usability
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Grandjean univentionstaff 2016-05-19 15:54:46 CEST
This should become the default for multi-school environments in the future.

+++ This bug was initially created as a clone of Bug #36031 +++

Reported at #2014093021000693:

The autosearch for the new school wizards (especially the school user wizard) should be disabled by default. Perhaps we can set "directory/manager/web/modules/.*/.*/search/autosearch" in postinst.
Comment 1 Florian Best univentionstaff 2016-07-05 13:27:52 CEST
I don't think we should change this. What is the reason to change this?
We have a documentation for large environments where this could be documented.
Comment 2 Michael Grandjean univentionstaff 2016-07-05 13:57:57 CEST
(In reply to Florian Best from comment #1)
> I don't think we should change this. What is the reason to change this?
> We have a documentation for large environments where this could be
> documented.

It's about usability. Multi-school environments will most likely have _many_ users, usually several thousand to ten thousands. In this case, opening the users schoolwizard _always_ takes a very long time loading users until it can actually be used. This is annoying and does cast a poor light on UCS@school: a customer bluntly asked me if we don't test our product for the size of our target audience. 

Yes, one can change this setting manually, if one finds and reads the "performance guide". But to be honest, that's a big 'if'. And if every customer running a multischool environment has to change this setting anyway, why shouldn't we make this a default?

(Personally, I don't see a point in displaying 1000 users or more in a list, at all. You don't scroll down the whole list to find a certain user, do you?)
Comment 3 Michel Smidt 2016-07-05 14:06:09 CEST
I completely agree with Michael. I had a customers as well who did not know about this option and wasted a lot of time to wait until I informed him about the UCR-Variable.
I guess this has to be reflected by the software.
Comment 4 Daniel Tröder univentionstaff 2016-07-18 14:20:10 CEST
In multi-school environments autosearch is now deactivated by default in all school related UMC modules, using a UCR policy at the LDAP-root.

Code: 71055
Advisory: 71056
Comment 5 Florian Best univentionstaff 2016-09-14 16:05:44 CEST
Please revert all changes. We should add this to the performance documentation.
Comment 6 Sönke Schwardt-Krummrich univentionstaff 2016-09-14 16:17:39 CEST
(In reply to Florian Best from comment #5)
> Please revert all changes. We should add this to the performance
> documentation.

We (Jan Christoph, Florian, Sönke) rediscussed this issue. User want the autosearch feature but do not want to wait. So we found out that the fix is not to disable autosearch but to improve autosearch.

A change of behaviour is also problematic in an errata update.
Comment 7 Daniel Tröder univentionstaff 2016-09-14 16:20:00 CEST
The usual solution to this problem is "paging".
Comment 8 Florian Best univentionstaff 2016-09-14 16:21:53 CEST
(In reply to Daniel Tröder from comment #7)
> The usual solution to this problem is "paging".
It seems we will add paging in UCS 4.2 :)
Comment 9 Sönke Schwardt-Krummrich univentionstaff 2016-09-14 16:22:14 CEST
"performance documentation" == performance-de.xml
→ If I'm not wrong, the UCR variable for UCS@school wizard are already documented. Do we need to document also the UDM UCR variables in performance.xml?
Comment 10 Florian Best univentionstaff 2016-09-14 16:24:32 CEST
(In reply to Sönke Schwardt-Krummrich from comment #9)
> "performance documentation" == performance-de.xml
> → If I'm not wrong, the UCR variable for UCS@school wizard are already
> documented. Do we need to document also the UDM UCR variables in
> performance.xml?
No. the UCS@school performance documentation should refer to the UCS performance guide:
http://docs.software-univention.de/performance-guide-4.1.html#umc:search:auto
Comment 11 Sönke Schwardt-Krummrich univentionstaff 2016-09-15 10:12:22 CEST
(In reply to Florian Best from comment #10)
> (In reply to Sönke Schwardt-Krummrich from comment #9)
> > "performance documentation" == performance-de.xml
> > → If I'm not wrong, the UCR variable for UCS@school wizard are already
> > documented. Do we need to document also the UDM UCR variables in
> > performance.xml?
> No. the UCS@school performance documentation should refer to the UCS
> performance guide:
> http://docs.software-univention.de/performance-guide-4.1.html#umc:search:auto

Yep, this is the better solution. I second that.

→ Please document the UCS@school variables in UCS@school performance manual section (if not already done) and refer to the UCS performance guide for further configuration settings.

(In reply to Florian Best from comment #8)
> It seems we will add paging in UCS 4.2 :)

Great.
Comment 12 Daniel Tröder univentionstaff 2016-09-15 10:38:03 CEST
r72609: reverted UCR setting in join script, added note to manual
r72610: cleanup
r72611: workaround ucslint mistake
r72612: build#->yaml
Comment 13 Florian Best univentionstaff 2016-09-20 15:00:06 CEST
OK: revert
OK: adjustments in ucs-school-metapackage
OK: YAML
~OK: manual/wording. (I think we cannot prevent the hardcoded UCS 4.1 link)
Comment 14 Sönke Schwardt-Krummrich univentionstaff 2016-10-04 13:24:48 CEST
UCS@school 4.1 R2 v5 has been released.

http://docs.software-univention.de/changelog-ucsschool-4.1R2v5-de.html

If this error occurs again, please clone this bug.