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

Incompatible signature in sync_users() method between LDAP and DB auth plugins

    XMLWordPrintable

Details

    • Improvement
    • Status: Reopened
    • Major
    • Resolution: Unresolved
    • 3.3.1, 3.4
    • None
    • Authentication

    Description

      In LDAP auth plugin:

      auth_plugin_ldap::sync_users($do_updates=true)

       

      In DB auth plugin:

      auth_plugin_db::sync_users(progress_trace $trace, $do_updates=false)

       

      (the problem is inherited by other plugins, e.g. CAS, which extends one of that)

       

      This situation limit refactoring possibilities, such as:

      • it requires a specific scheduled task for each plugin instead of a generic one
      • it prevents the  pull up of sync_users() as abstract method or parent method in the auth_plugin_base class
      • it forces if/then clauses in external plugins which use the codebase of LDAP o DB one

       

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              aulaweb AulaWeb Università di Genova
              Mark Nelson Mark Nelson
              David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

                Created:
                Updated: