Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-37359

Typo of variable name in group/externallib.php

    Details

    • Type: Bug
    • Status: Closed
    • Priority: 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

      Description

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

        Gliffy Diagrams

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

          Issue Links

            Activity

            Hide
            markn 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
            markn 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
            samhemelryk Sam Hemelryk added a comment -

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

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

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

            Show
            abgreeve Adrian Greeve added a comment - Tested on the 2.3, 2.4 and master integration branches. No regressions found. Test passed.
            Hide
            stronk7 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
            stronk7 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:
                  Fix Release Date:
                  14/Jan/13