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

Enable sub-plugins in Local plugins

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.2, 2.4, 2.5
    • Fix Version/s: 2.6
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide

      1/ unzip attached testsub.zip file to your local dir
      2/ make sure one new plugin with two subplugins gets installed
      3/ try to uninstall 1 subplugin from plugins overview page
      4/ install it again
      5/ try to uinstall the local plugin - it should delete own subplugins
      6/ run phpunit tests

      Show
      1/ unzip attached testsub.zip file to your local dir 2/ make sure one new plugin with two subplugins gets installed 3/ try to uninstall 1 subplugin from plugins overview page 4/ install it again 5/ try to uinstall the local plugin - it should delete own subplugins 6/ run phpunit tests
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w27_MDL-26943_m26_localsubplugins

      Description

      Currently, sub-plugins are only available for Activity Modules.
      This makes sense since it's reasonably expensive to check each plug-in of a given type for sub-plugins. However I'd like them to be available for Local plugins too.

      Rationale:
      There wont be many local plugins, so there won't be many to check.
      Local plugins are designed for features that wont fit into the general structure of plugins. Therefore, they should be as flexible as possible - enabling sub-plugins would enhance this.
      In terms of code, it's only a small change.

        Attachments

          Issue Links

            Activity

              People

              • Votes:
                6 Vote for this issue
                Watchers:
                10 Start watching this issue

                Dates

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