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

Remove legacy tool_generator mess

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Use the course generator
      2. VERIFY: no errors are experienced and course generator works
      3. Use the site generator cli tool
      4. VERIFY: no errors are experienced and site is generated
      5. Go to admin/tool/generator/
      6. VERIFY: you are redirected to the make course tool
      Show
      Use the course generator VERIFY: no errors are experienced and course generator works Use the site generator cli tool VERIFY: no errors are experienced and site is generated Go to admin/tool/generator/ VERIFY: you are redirected to the make course tool
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull Master Branch:
      MDL-41311-master

      Description

      There is some completely unmaintained code in admin/tool/generator to create test courses. Its not worked for some time and should be removed.

      In MDL-38197, Sam added a partial replacement. I don't believe we need to worry about backwards compatibility here as it was a developer tool, marked experimental and people keep trying to fix bugs in it/use it when its really not stable at all for use.

      The existing functionality should be replaced by switching to generators (as used by behat and phpunit) but I don't believe we need to wait for that because people are wasting their time trying to use it or monkeypatch it.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              poltawski Dan Poltawski
              Reporter:
              poltawski Dan Poltawski
              Peer reviewer:
              David Monllaó
              Integrator:
              Marina Glancy
              Tester:
              Petr Skoda
              Participants:
              Component watchers:
              Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13