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

Assign: Plugin Deleted tells users to delete a non-existant path

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Duplicate
    • Affects Version/s: 2.4
    • Fix Version/s: None
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide

      1. Install an assignment submission (or assignment feedback) plugin - for example: https://moodle.org/plugins/view.php?plugin=assignsubmission_pdf
      2. Go to Settings -> Plugins -> Assignment plugins -> Submission plugins -> Manage submission plugins
      3. Delete the installed plugin
      4. Verify that the path to be deleted is the actual path

      Show
      1. Install an assignment submission (or assignment feedback) plugin - for example: https://moodle.org/plugins/view.php?plugin=assignsubmission_pdf 2. Go to Settings -> Plugins -> Assignment plugins -> Submission plugins -> Manage submission plugins 3. Delete the installed plugin 4. Verify that the path to be deleted is the actual path
    • Affected Branches:
      MOODLE_24_STABLE

      Description

      I was testing the pdf submission plugin and then removed it and received a message that:

      All data associated with the plugin 'PDF submission' has been deleted from the database. To prevent the plugin re-installing itself, you should now delete this directory from your server: /mod/assign/assignsubmission/pdf

      The erroneous code is located in mod/assign/adminlib.php in the view_plugin_deleted function:

      echo $OUTPUT->notification(get_string('plugindeletefiles', 'moodle', array('name'=>get_string('pluginname', $this->subtype . '_' . $plugin), 'directory'=>('/mod/assign/' . $this->subtype . '/'.$plugin))));

      The assumption is that the subtype (assignsubmission) is going to be the directory path; however, the actual path is: mod/assign/submission/pdf.

      Peace - Anthony

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              aborrow Anthony Borrow
              Reporter:
              aborrow Anthony Borrow
              Peer reviewer:
              Damyon Wiese Damyon Wiese
              Participants:
              Component watchers:
              Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: