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

          Attachments

            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