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

Don't overwrite modified user profile values with empty LDAP values when 'unlocked if empty' is set

    Details

    • Type: New Feature
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.8.6, 1.9.2
    • Component/s: Authentication
    • Labels:
      None
    • Environment:
      All
    • Affected Branches:
      MOODLE_16_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE

      Description

      I understand that if user profile values are locked to LDAP, then LDAP should provide those values.

      But in my mind the purpose of the setting unlocked if empty is to allow the user to fill in a value for the selected field if LDAP is giving nothing for this field. Thus it would make sense to let this value stand in until LDAP is giving a value for this field.

        Gliffy Diagrams

          Attachments

            Activity

            Hide
            dougiamas Martin Dougiamas added a comment -

            From Samuli Karevaara (samuli.karevaara at lamk.fi) Thursday, 5 January 2006, 09:41 PM:

            Added a patch I'm using to fill this feature request.

            Show
            dougiamas Martin Dougiamas added a comment - From Samuli Karevaara (samuli.karevaara at lamk.fi) Thursday, 5 January 2006, 09:41 PM: Added a patch I'm using to fill this feature request.
            Hide
            samulik Samuli Karevaara added a comment -

            The patch was lost in the old tracker, so I'm attaching it again. This is old, for 1.6...

            Show
            samulik Samuli Karevaara added a comment - The patch was lost in the old tracker, so I'm attaching it again. This is old, for 1.6...
            Hide
            iarenaza Iñaki Arenaza added a comment -

            Samuli,

            I fully agree with you, so I'm goint to assign the bug to myself and fix it in 1.8, 1.9 and HEAD using your patch.

            Saludos. Iñaki.

            Show
            iarenaza Iñaki Arenaza added a comment - Samuli, I fully agree with you, so I'm goint to assign the bug to myself and fix it in 1.8, 1.9 and HEAD using your patch. Saludos. Iñaki.
            Hide
            iarenaza Iñaki Arenaza added a comment -

            Patch applied in 1.8, 1.9 and HEAD.

            Samuli, thanks a lot for your patch!

            Saludos. Iñaki.

            Show
            iarenaza Iñaki Arenaza added a comment - Patch applied in 1.8, 1.9 and HEAD. Samuli, thanks a lot for your patch! Saludos. Iñaki.
            Hide
            samulik Samuli Karevaara added a comment -

            Thanks, Iñaki!

            Show
            samulik Samuli Karevaara added a comment - Thanks, Iñaki!
            Hide
            skodak Petr Skoda added a comment -

            thanks

            Show
            skodak Petr Skoda added a comment - thanks

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  11/Jul/08