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

Issues with the dependency check during a plugin installation

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Install a contributed plugin that declares some other plugin as a dependency (I used assignsubmission_helixassign for testing during the development).
      2. At the Plugins check page, make sure there are now two links available: (1) The Install link takes you to the page at the Plugins directory to install or download the required plugin. (2) The Upload link takes you to the page where you can upload the required plugin manually.
      Show
      Install a contributed plugin that declares some other plugin as a dependency (I used assignsubmission_helixassign for testing during the development). At the Plugins check page, make sure there are now two links available: (1) The Install link takes you to the page at the Plugins directory to install or download the required plugin. (2) The Upload link takes you to the page where you can upload the required plugin manually.
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-47069-master-plugin-dependency
    • Pull Master Diff URL:

      Description

      If a plugin declares dependencies on other plugin, we currently display the other component as a link to the plugins directory that eventually allows to install it. There are two issues with this I am currently facing.

      • If the plugin explicitly declares dependency on a standard plugin (such as mod_forum), we display the link too. But there is no point for that as standard plugins are not available in the directory.
      • There is no easy way to navigate to the admin screen where plugin can be uploaded manually other than directly typing-in that tool URL.

        Attachments

          Activity

            People

            Assignee:
            mudrd8mz David Mudrák (@mudrd8mz)
            Reporter:
            mudrd8mz David Mudrák (@mudrd8mz)
            Peer reviewer:
            Ankit Agarwal
            Integrator:
            Dan Poltawski
            Tester:
            Mark Nelson
            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:
              10/Nov/14