MDL-46946 user: Make missing required custom fields trigger profile edit
authorDavid Mudrák <david@moodle.com>
Thu, 8 Sep 2016 20:45:33 +0000 (22:45 +0200)
committerDavid Mudrák <david@moodle.com>
Wed, 21 Sep 2016 15:46:30 +0000 (17:46 +0200)
commit8df850ad6f68f08a9d61f893bc32864b80c64e94
tree6d904b03edaa928bb0c9cca6fc00335b1383ffa8
parent1f2744851fb50cae6602510d3651ecf9797d24a1
MDL-46946 user: Make missing required custom fields trigger profile edit

If there is a required custom field that the user can fill by editing
their profile, and that field is missing, the user should be considered
as not fully set up. Instead, we want to redirect them to edit their
profile first.

There are some exceptions when we want to fall back to the previous
behaviour and check just the name and email fields. These exceptional
cases include checking remote user data in incoming MNet request (no
user id, no custom fields supported) and calls to require_login() with
redirecting disabled (typically ajax filepicker requests on profile
editing page itself).

Additional plugins that call the function user_not_fully_set_up()
themselves, should perform the strict check in most/typical cases. So
the strict mode is enabled by default even if it changes the behaviour
slightly. In improbable case of additional plugins relying on the
previous behaviour of the function, they can use the $strict parameter
and keep performing the lax check. However, I am sure the correct fix in
that case will likely be to stop abusing this function.

Note that custom fields are not currently transferred during the MNet
roaming. So having custom fields configured as required on MNet service
provider site (where users can't edit their profiles) is expected to
display an error (as the site is considered as misconfigured).
admin/tool/monitor/classes/task/check_subscriptions.php
auth/ldap/auth.php
auth/mnet/auth.php
auth/shibboleth/index.php
lib/moodlelib.php
login/lib.php
user/edit.php
user/editlib.php
user/profile/lib.php
user/tests/profilelib_test.php