Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7, 1.8, 1.9
    • Fix Version/s: 1.8.13, 1.9.9, 2.0
    • Component/s: Authentication
    • Labels:
      None
    • Environment:
      Any system using CAS authentication and syncing.
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_17_STABLE, MOODLE_18_STABLE, MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE, MOODLE_19_STABLE, MOODLE_20_STABLE

      Description

      In reviewing the log output generated from the cas_ldap_sync_users.php I noticed that the revive section had nothing but bracketed text. Looking at auth.php I discovered the following lines of code had a typo in them

      if (update_record('user', $updateuser))

      { echo "\t"; print_string('auth_dbreviveser', 'auth', array($user->username, $user->id)); echo "\n"; }

      else

      { echo "\t"; print_string('auth_dbreviveusererror', 'auth', $user->username); echo "\n"; }

      The auth_dbreviveser and auth_dbreviveusererror should actually be auth_dbreviveduser and auth_dbrevivedusererror

        Gliffy Diagrams

          Activity

          Hide
          jcharaoui Jerome Charaoui added a comment -

          These typos are also present in the auth/ldap module (lines 683 and 685) as well as the auth/db module (line 365), in CVS HEAD.

          Show
          jcharaoui Jerome Charaoui added a comment - These typos are also present in the auth/ldap module (lines 683 and 685) as well as the auth/db module (line 365), in CVS HEAD.
          Hide
          iarenaza Iñaki Arenaza added a comment -

          The same typo was present in auth/cas, auth/db and auth/ldap.

          This should be fixed in CVS for both 1.8.x, 1.9.x and HEAD.

          Thanks a lot to Scott Karren and Jêrome Charaoui for the bug report and comments

          Saludos,
          Iñaki.

          Show
          iarenaza Iñaki Arenaza added a comment - The same typo was present in auth/cas, auth/db and auth/ldap. This should be fixed in CVS for both 1.8.x, 1.9.x and HEAD. Thanks a lot to Scott Karren and Jêrome Charaoui for the bug report and comments Saludos, Iñaki.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                8/Jun/10