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

Enable sub-plugins in Local plugins

    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.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

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

                  Dates

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