Moodle
  1. Moodle
  2. MDL-39322

Upload users form misreports the authentication method for site administrators

    Details

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

      For this patch you need a Moodle instance with a site administrator whose authentication method isn't manual.

      1. Create a user upload CSV with at least one row. Include said site administrator.
      2. Attempt the upload. Verify that the tool doesn't update the site administrator. Verify that the status page reports the site administrator's actual authentication method.
      Show
      For this patch you need a Moodle instance with a site administrator whose authentication method isn't manual. Create a user upload CSV with at least one row. Include said site administrator. Attempt the upload. Verify that the tool doesn't update the site administrator. Verify that the status page reports the site administrator's actual authentication method.
    • Affected Branches:
      MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-39322-master
    • Rank:
      49945

      Description

      admin/tool/uploaduser always reports a site administrator's authentication method as 'manual' (the default) because it tests for site admin before loading the authentication method. This is misleading.

        Activity

        Hide
        Rajesh Taneja added a comment -

        Thanks for fixing this Charles,

        Patch looks spot-on, pushing for integration.

        Show
        Rajesh Taneja added a comment - Thanks for fixing this Charles, Patch looks spot-on, pushing for integration.
        Hide
        Dan Poltawski added a comment -

        The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

        TIA and ciao

        Show
        Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
        Hide
        Charles Fulton added a comment -

        Rebased.

        Show
        Charles Fulton added a comment - Rebased.
        Hide
        Damyon Wiese added a comment -

        Thanks Charles,

        The patch looks good.

        This has been integrated to master.

        Show
        Damyon Wiese added a comment - Thanks Charles, The patch looks good. This has been integrated to master.
        Hide
        Andrew Davis added a comment -

        Seems to be working as described. Passing.

        Show
        Andrew Davis added a comment - Seems to be working as described. Passing.
        Hide
        Dan Poltawski added a comment -

        Thanks! You're changes are now spread to the world through this git and our source control repositories.

        No time to rest though, we've got days to make 2.5 the best yet!

        ciao

        Show
        Dan Poltawski added a comment - Thanks! You're changes are now spread to the world through this git and our source control repositories. No time to rest though, we've got days to make 2.5 the best yet! ciao

          People

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

            Dates

            • Created:
              Updated:
              Resolved: