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
    • Rank:
      37443

      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

        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 Škoda added a comment -

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

          Show
          Petr Škoda 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: