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

Delete testing code pushed upstream without noticing

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Groups
    • Labels:
      None

      Description

      This is a regression of MDL-31973.

      Some code was inadvertently sent upstream with that issue. This followup is simply about take rid of it. Credit goes to Andrew Nicols by detecting and providing a patch.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            This is the commit:

            https://git.luns.net.uk/moodle.git/commit/e299340a9db917e2ea5e3b9a483130b103c66d4b

            No testing instructions needed... applying.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - This is the commit: https://git.luns.net.uk/moodle.git/commit/e299340a9db917e2ea5e3b9a483130b103c66d4b No testing instructions needed... applying.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Integrated (master), thanks!

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Integrated (master), thanks!
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Nobody tested this, passing.

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Nobody tested this, passing.
            Hide
            stronk7 Eloy Lafuente (stronk7) added a comment -

            Many thanks for the hard work.

            These changes have been spread upstream and are already available in the git and cvs repositories.

            Ciao

            Show
            stronk7 Eloy Lafuente (stronk7) added a comment - Many thanks for the hard work. These changes have been spread upstream and are already available in the git and cvs repositories. Ciao

              People

              • Assignee:
                stronk7 Eloy Lafuente (stronk7)
                Reporter:
                stronk7 Eloy Lafuente (stronk7)
                Integrator:
                Eloy Lafuente (stronk7)
                Tester:
                Nobody
                Participants:
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  3/Dec/12