Moodle
  1. Moodle
  2. MDL-37359

Typo of variable name in group/externallib.php

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.3, 2.4
    • Fix Version/s: 2.3.4, 2.4.1
    • Component/s: Groups
    • Labels:
      None
    • Testing Instructions:
      Hide

      1) Create a web service that will create a new grouping.

      Note, please see the attached files for a demonstration on how to create a web service client to test. They will obviously need to be changed to call this function.

      Show
      1) Create a web service that will create a new grouping. Note, please see the attached files for a demonstration on how to create a web service client to test. They will obviously need to be changed to call this function.
    • Affected Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-37359_master
    • Rank:
      46980

      Description

      Line 582 reads 'descripiton', should be 'description'.

      1. ws_client.php
        1 kB
        Mark Nelson
      2. ws_curl.php
        18 kB
        Mark Nelson

        Issue Links

          Activity

          Hide
          Mark Nelson added a comment -

          Integrator can you please cherry-pick this to the other Moodle branches (2.2 onwards). There should be no issues, very basic fix. Thanks.

          Show
          Mark Nelson added a comment - Integrator can you please cherry-pick this to the other Moodle branches (2.2 onwards). There should be no issues, very basic fix. Thanks.
          Hide
          Sam Hemelryk added a comment -

          This has been integrated now, didn't backport to 22 as that was not affected.

          Show
          Sam Hemelryk added a comment - This has been integrated now, didn't backport to 22 as that was not affected.
          Hide
          Adrian Greeve added a comment -

          Tested on the 2.3, 2.4 and master integration branches.
          No regressions found.
          Test passed.

          Show
          Adrian Greeve added a comment - Tested on the 2.3, 2.4 and master integration branches. No regressions found. Test passed.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And your fantastic code has met core, hope they become good friends for a long period.

          Closing, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - And your fantastic code has met core, hope they become good friends for a long period. Closing, thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: