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

Broken JS in group/index.php

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.5, 2.2.2, 2.3
    • Fix Version/s: 2.1.6, 2.2.3
    • Component/s: Groups
    • Labels:
    • Testing Instructions:
      Hide

      Whilst in a course as an editing teacher or administrator:

      • Navigate to Course administration -> Users -> Groups
      • Create a new group
      • Select the group
      • Open your browser web developer tools:
        • In Chrome, the following settings to the developer tools help: Console: Preserve log upon navigation, and in the 'Scripts' tab, click the 'Pause on all exceptions' button
      • Click the 'Delete selected group' button
        • Confirm that no errors were displayed in the javascript console
      Show
      Whilst in a course as an editing teacher or administrator: Navigate to Course administration -> Users -> Groups Create a new group Select the group Open your browser web developer tools: In Chrome, the following settings to the developer tools help: Console: Preserve log upon navigation, and in the 'Scripts' tab, click the 'Pause on all exceptions' button Click the 'Delete selected group' button Confirm that no errors were displayed in the javascript console
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-32514-master-1

      Description

      Whilst thoroughly testing MDL-32005 before submitting for IR, I came across a weird bug in group/index.php. There's an onClick for the "Delete selected groups" button which calls a function which hasn't existed in nearly five years.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                dobedobedoh Andrew Nicols
                Reporter:
                dobedobedoh Andrew Nicols
                Peer reviewer:
                Dan Poltawski
                Integrator:
                Eloy Lafuente (stronk7)
                Tester:
                Ankit Agarwal
                Participants:
                Component watchers:
                Andrew Nicols, Mathew May, Michael Hawkins, Shamim Rezaie, Simey Lameze
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  14/May/12