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

Reseting Assign Activities does not delete submissions and feedback files

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.8, 2.6.5, 2.7.2, 2.8
    • Fix Version/s: 2.6.6, 2.7.3
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide

      Create an Assignment with file submissions
      As Student upload a file
      Get the file record on the Database
      As Teacher reset course and mark assignment submissions
      (It may be needed to run cron at this stage)
      Get the file record on the Database and confirm it is no longer there

      Show
      Create an Assignment with file submissions As Student upload a file Get the file record on the Database As Teacher reset course and mark assignment submissions (It may be needed to run cron at this stage) Get the file record on the Database and confirm it is no longer there
    • Workaround:
      Hide

      To solve this change the lines 700 and 715 on mod/assign/locallib.php on function reset_userdata

      Original code (on both lines 700 and 715):

      foreach ($fileareas as $filearea) {
      

      Fixed code:

      foreach ($fileareas as $filearea => $notused) {
      

      Show
      To solve this change the lines 700 and 715 on mod/assign/locallib.php on function reset_userdata Original code (on both lines 700 and 715): foreach ($fileareas as $filearea) { Fixed code: foreach ($fileareas as $filearea => $notused) {
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      When reseting assign submissions, the code tries to erase the files but fails because of the filearea specified.

        Attachments

          Activity

            People

            Assignee:
            pferre22 Pau Ferrer
            Reporter:
            pferre22 Pau Ferrer
            Peer reviewer:
            David Monllaó
            Integrator:
            Damyon Wiese
            Tester:
            Andrew Nicols
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            10 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Nov/14