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

Remove legacy tool_generator mess

    XMLWordPrintable

Details

    • MOODLE_26_STABLE
    • MOODLE_26_STABLE
    • MDL-41311-master
    • 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

    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

              poltawski Dan Poltawski
              poltawski Dan Poltawski
              David Monllaó David Monllaó
              Marina Glancy Marina Glancy
              Petr Skoda Petr Skoda
              Andrew Lyons, Huong Nguyen, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                18/Nov/13