Moodle
  1. Moodle
  2. MDL-31035

Missing config parameter showuseridentity after upgrade

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: 2.2.2
    • Component/s: Installation
    • Labels:
    • Testing Instructions:
      Hide

      No testing should be necessary, just use any user selection table two panes and look for regressions.

      Show
      No testing should be necessary, just use any user selection table two panes and look for regressions.
    • Workaround:
      Hide

      The migration process won't create the "showuseridentity" parameter in mdl_config table
      I add this parameter at the end of the table, with following values :
      id => next value
      name => showuseridentity
      value => email

      And it's works ...

      I don't know why the process doesn't create this parameter

      Show
      The migration process won't create the "showuseridentity" parameter in mdl_config table I add this parameter at the end of the table, with following values : id => next value name => showuseridentity value => email And it's works ... I don't know why the process doesn't create this parameter
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w03_MDL-31035_m23_identitynotice

      Description

      I have an error in the function get_extra_user_fields(...) in file moodle/lib/moodlelib.php

      The test for empty string for "$CFG->showuseridentity" doesn't works with 3 equal sign but works with 2

      line 3366 in moodlelib.php, Moodle release 2.2+ (Build 20111223)
      The site is an upgrade from moodle 1.9.8+

      A display of $CFG show that "showuseridentity" doesn't exist , gettype($CFG->showuseridentity) returns NULL

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Michael de Raadt added a comment -

            Thanks for reporting that and providing a work-around.

            Show
            Michael de Raadt added a comment - Thanks for reporting that and providing a work-around.
            Hide
            Petr Skoda added a comment -

            Thanks for the report, fix should be in the next weekly build.

            Show
            Petr Skoda added a comment - Thanks for the report, fix should be in the next weekly build.
            Hide
            Sam Hemelryk added a comment -

            Thanks Petr - this has been integrated now.

            Show
            Sam Hemelryk added a comment - Thanks Petr - this has been integrated now.
            Hide
            Tim Barker added a comment -

            Starting testing

            Show
            Tim Barker added a comment - Starting testing
            Hide
            Tim Barker added a comment -

            This could be tester doing the following:
            1) Ran and upgrade from 1.9.8 to integration/MOODLE_22_STABLE
            2) After the upgrade run SELECT * FROM mdl_config WHERE name - 'showuseridentity';

            Show
            Tim Barker added a comment - This could be tester doing the following: 1) Ran and upgrade from 1.9.8 to integration/MOODLE_22_STABLE 2) After the upgrade run SELECT * FROM mdl_config WHERE name - 'showuseridentity';
            Hide
            Eloy Lafuente (stronk7) added a comment -

            This is now available in the git and cvs repositories.

            Consider the responsibility of your fingerprints engraved there for future generations!

            Thanks for the work, closing, ciao

            Show
            Eloy Lafuente (stronk7) added a comment - This is now available in the git and cvs repositories. Consider the responsibility of your fingerprints engraved there for future generations! Thanks for the work, closing, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: