[AMOS commit] MDL-67748 admin: Do not show missing capabilities for mobile app service Committed into Git: 2020-04-24 17:20 UTC

[AMOS commit] MDL-67748 admin: Do not show missing capabilities for mobile app service Committed into Git: 2020-04-24 17:20 UTC

by AMOS bot -
Number of replies: 0
Author: David Mudrák
MDL-67748 admin: Do not show missing capabilities for mobile app service
We used to display capabilities like "Manage any calendar entries",
"Delete evidence", "Manage competency frameworks", "View hidden courses"
and others as "Missing capabilities" for the mobile app service tokens.

This gave dangerous impression that the app will not work for students
without these capabilities granted. There are known cases of admins who
started to grant all these caps to the Authenticated user role because
they were afraid the app would not work for them.

The problem here is that the official mobile app service includes some
functions that have these capabilities declared as required. But they
are not really required to use the app. Either the app makes its own
clever checks of capabilities before calling the functions, or sometimes
the capabilities are not even correctly declared.

It is safer for everybody to display this information for custom
services only where the risk of the falsely missing caps is lower and
the information is more accurate.

Also, the help text has been improved so it does not suggest that these
capabilities must be always added. We do not know why the service has
them declared. In some cases, a service has capabilities declared just
because it makes use of them in the if-then fashion.

Additionally, the patch also displays the service short name because it
is actually needed to know.


http://git.moodle.org/gw?p=moodle.git;a=commit;h=71e7806a18756a7d45226480359623e45dbbe769
http://github.com/moodle/moodle/commit/71e7806a18756a7d45226480359623e45dbbe769

+ 3.11 en [missingcaps_help,core_webservice]