Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-57929

profile_has_required_custom_fields_set fails to detect if a custom profile field is empty

    XMLWordPrintable

    Details

      Description

      The profile_has_required_custom_fields_set introduced by MDL-46946 relies on a sql query to determine if there is some empty custom profile fields or not. Its base assumption is that a field is empty if and only if there is no value for it in {user_info_data}.

      This assumption is not always true. Profile field plugins can override a function named is_empty() to instruct Moodle if a field is empty or not.

      profile_has_required_custom_fields_set() should call is_empty() for each plugin.
      It is possible that there is no value in {user_info_data} and yet the field is not empty. An example is the profilefield_associated plugin.

        Attachments

          Issue Links

          has a non-specific relationship to

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-66720 User is NOT forced to complete required custom user profile fields

          • Minor - Minor loss of function where workaround is possible
          • Open
          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-46946 user_not_fully_set_up() should force users to complete required custom user profile fields

          • Major - Major loss of function, incorrect output
          • Closed
          is duplicated by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-61417 missing redirection when an existing profile field is set to required

          • Minor - Minor loss of function where workaround is possible
          • Closed

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              rezaie9 Shamim Rezaie
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
              Votes:
              12 Vote for this issue
              Watchers:
              20 Start watching this issue

                Dates

                Created:
                Updated:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 3 minutes
                  3m