Bug 53194 - Setting the ucsschoolAdministrator should automatically adjust the school_admin role
Setting the ucsschoolAdministrator should automatically adjust the school_adm...
Status: NEW
Product: UCS@school
Classification: Unclassified
Component: UMC
UCS@school 4.4
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS@school maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2021-04-29 12:06 CEST by Christina Scheinig
Modified: 2022-06-08 13:55 CEST (History)
4 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?: Yes
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number: 2021041521000332, 2021043021000483, 2021051421000387, 2022060321000331
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 Christina Scheinig univentionstaff 2021-04-29 12:06:24 CEST
Since the consistency check in the diagnostic module shows the missing role e.g. for ucsschool administrators, we should make sure the role is set properly, when setting the ucsschoolAdministrator flag.

At the moment the role is not set automatically, and can only be set via udm, not via UMC, because the extended attribute ucsschoolRoleUsers is invisible by default.
Comment 1 Dirk Schnick univentionstaff 2021-04-30 16:34:47 CEST
From my point of view all the marks (Admin, teacher, student, staff) in UMC should also trigger also the attachment of the role. This is what every customer expects by setting the mark.
Comment 2 Daniel Tröder univentionstaff 2021-05-03 08:27:36 CEST
(In reply to Dirk Schnick from comment #1)
> From my point of view all the marks (Admin, teacher, student, staff) in UMC
> should also trigger also the attachment of the role. This is what every
> customer expects by setting the mark.

When the UCS@school UMC modules are used, that happens.
When non-UCS@school UMC modules are used it doesn't. Other things are then also not set correctly. That's why school customers should only use the UCS@school UMC modules.

Also: changing the role of a user is *not* supported. With one exception: making a staff/teacher an admin.
Comment 3 Christina Scheinig univentionstaff 2021-05-03 10:38:23 CEST
(In reply to Daniel Tröder from comment #2)
> (In reply to Dirk Schnick from comment #1)
> > From my point of view all the marks (Admin, teacher, student, staff) in UMC
> > should also trigger also the attachment of the role. This is what every
> > customer expects by setting the mark.
> 
> When the UCS@school UMC modules are used, that happens.
> When non-UCS@school UMC modules are used it doesn't. Other things are then
> also not set correctly. That's why school customers should only use the
> UCS@school UMC modules.
> 
> Also: changing the role of a user is *not* supported. With one exception:
> making a staff/teacher an admin.

I am confused. Using the UCS@school UMC modules means getting redirected to the ldap-directory-module, using the advanced settings button and changing the user there. this is afaik the same module like the "normal" user module. So it is not possible to work?

Adding a teacher via UCS@school UMC modules it is not possible to set a role a role at all, the field is not editable.

Or am I on the complete wrong way?
Comment 4 Dirk Schnick univentionstaff 2021-05-03 13:02:03 CEST
The endusers will use the non school users module. They did it and they will do. As far as it is useable and as far as there is any @school attribute to configure, the will not get the problem.

Also Christinas statement applies to me.
Comment 5 Daniel Tröder univentionstaff 2021-05-03 13:45:23 CEST
What I am saying is that using a non-school-UMC module to change a school-attribute is not supported unless it is to make a user an administrator.

That customers do unsupported things does not change the fact, that it is not supported.
Comment 6 Daniel Tröder univentionstaff 2021-05-03 13:47:05 CEST
BTW: we will address this particular issue: creation of school admins with correct role attribute.
Because customers should not have to use a non-school-UMC module to change a school-attribute.
Comment 7 Dirk Schnick univentionstaff 2021-05-03 16:51:19 CEST
I think our product should be robust and we should prevent such missunderstandings. 

F.e. Removing all school settings from the normal users module and give the possibility to set everything in a school users module.

Also possible in my mind is a banner that this module is not for creating/modifying school users in the normal users module.

Our customers are not aware about the "do not use the users module in school environment" and I can understand them, as there is a users module, so I can use it to do create or modify users.

The best way from my point of view would be to upgrade the users module to use it; also in a school environment.

The marks in the users module (Apps -> UCS@school Administrator, UCS@school Examuser, UCS@school Staff, UCS@school Student, UCS@school Teacher) are missleading; what are they doing at all? There is a consistency check a student can not be an Admin, but if I set a mark I see no change in the user config. Customers are expecting to set the role of the user there. If they are useless, hide them.

We can point to the documentation, but don't we want a self speaking product without such missunderstandings? Also pointing to the docs will not help us in the support team ;-)
Comment 8 Daniel Tröder univentionstaff 2021-05-03 16:59:22 CEST
(In reply to Dirk Schnick from comment #7)
> I think our product should be robust and we should prevent such
> missunderstandings. 
> 
> F.e. Removing all school settings from the normal users module and give the
> possibility to set everything in a school users module.
> 
> Also possible in my mind is a banner that this module is not for
> creating/modifying school users in the normal users module.
I also think that would be good, but it has been discussed before and was rejected. (I think for technical reasons.)
Maybe we can reopen the discussion by collecting all requirements for a compete school admin UI.
Comment 9 Dirk Schnick univentionstaff 2021-05-04 08:06:38 CEST
(In reply to Daniel Tröder from comment #8)
> I also think that would be good, but it has been discussed before and was
> rejected. (I think for technical reasons.)
> Maybe we can reopen the discussion by collecting all requirements for a
> compete school admin UI.
I think we reopened the discussion already. ;) Waiting for PO's statement.
I have written what is happening and what will happen if we do not change anything here. I'm sure there will be some technical Challenges, but a change in any way seems to be necessary.
Comment 10 Christina Scheinig univentionstaff 2022-06-08 13:55:03 CEST
This still occurs in UCS 5.0-1. A customer opened a support ticket for that issue