Moodle
  1. Moodle
  2. MDL-34543

New assignment module - Feedback file exists for an assignment but not shown in the Feedback files picker

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 2.3.1
    • Fix Version/s: 2.3.2
    • Component/s: Assignment
    • Labels:
    • Testing Instructions:
      Hide
      1. Create a new assignment (mod_assign)
      2. Open assignment
      3. Click on "view/grade all submissions"
      4. For one of the students click on "Grade"
      5. Add a comment in the feedback section.
      6. Add a file in the feedback section.
      7. Go back to view/grade all submissions and file is displayed.
      8. Click on Grade for the same student, go to the feedback area
      9. VERIFY: that the feedback comment and file appears in the filepicker
      Show
      Create a new assignment (mod_assign) Open assignment Click on "view/grade all submissions" For one of the students click on "Grade" Add a comment in the feedback section. Add a file in the feedback section. Go back to view/grade all submissions and file is displayed. Click on Grade for the same student, go to the feedback area VERIFY: that the feedback comment and file appears in the filepicker
    • Affected Branches:
      MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE
    • Pull Master Branch:
    • Rank:
      42972

      Description

      Upgraded to 2.3.1. Upgraded all assignments to the new type. Overview of assignments show a feedback file for a particular assignment. If you click on 'grade' for this assignment the file is not displayed in the feedback file picker.

      Replication steps:

      1. Create a new assignment
      2. Open assignment
      3. click on "view/grade all submissions"
      4. For one of the students click on "Grade"
      5. Add a comment in the feedback section.
      6. Add a file in the feedback section.
      7. Go back to view/grade all submissions and file is displayed.
      8. Click on Grade for the same student, go to the feedback area

      Expected result: the feedback comment and file appears in the filepicker

      Actual result: no file or comment is shown in the file picker

      So if you save now (or add another file and save) the original file and comment will no longer be visible in view/grade all submissions.

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          Hi, Paul.

          Can you please clarify that this is only for upgraded assignments that previously had feedback files attached? Have you tried this with newly created assignments?

          If you could create a set of replication steps that would allow us to see the same result, that would be helpful.

          Show
          Michael de Raadt added a comment - Hi, Paul. Can you please clarify that this is only for upgraded assignments that previously had feedback files attached? Have you tried this with newly created assignments? If you could create a set of replication steps that would allow us to see the same result, that would be helpful.
          Hide
          Damyon Wiese added a comment -

          Hi Paul,

          Can you also please list which submission and feedback types are enabled for one of these assignments (in the assignment settings page)?

          Thanks, Damyon

          Show
          Damyon Wiese added a comment - Hi Paul, Can you also please list which submission and feedback types are enabled for one of these assignments (in the assignment settings page)? Thanks, Damyon
          Hide
          David Perry added a comment -

          We have this problem too - both with newly created v2.2 and new type (v2.3) assignments. It's a new use case on the course page we want to do this on so no previous assignments to check.

          Show
          David Perry added a comment - We have this problem too - both with newly created v2.2 and new type (v2.3) assignments. It's a new use case on the course page we want to do this on so no previous assignments to check.
          Hide
          Paul Hague added a comment - - edited

          Hi Michael,
          I've just created a new assignment and just the same ...

          Create a new assignment
          Open assignment
          click on "view/grade all submissions"
          For one of the students click on "Grade"
          Add a file in the feedback section.
          Go back to view/grade all submissions and file is displayed.
          Click on Grade for the same student, go to the feedback files - nothing shown
          So if you save now (or add another file and save) the original file will no longer be visible in view/grade all submissions.

          Show
          Paul Hague added a comment - - edited Hi Michael, I've just created a new assignment and just the same ... Create a new assignment Open assignment click on "view/grade all submissions" For one of the students click on "Grade" Add a file in the feedback section. Go back to view/grade all submissions and file is displayed. Click on Grade for the same student, go to the feedback files - nothing shown So if you save now (or add another file and save) the original file will no longer be visible in view/grade all submissions.
          Hide
          Paul Hague added a comment -

          Hi Damyon,

          I enabled all submission types for the test assignment.

          Show
          Paul Hague added a comment - Hi Damyon, I enabled all submission types for the test assignment.
          Hide
          Michael de Raadt added a comment -

          Thanks for the additional information.

          I was able to reproduce the problem.

          Show
          Michael de Raadt added a comment - Thanks for the additional information. I was able to reproduce the problem.
          Hide
          Christiana Nolte added a comment -

          I described the same problem: http://moodle.org/mod/forum/discuss.php?d=204996&parent=896404 (31. July 2012, 13:21) Thanks for working on this!

          Show
          Christiana Nolte added a comment - I described the same problem: http://moodle.org/mod/forum/discuss.php?d=204996&parent=896404 (31. July 2012, 13:21) Thanks for working on this!
          Hide
          Eric Merrill added a comment -

          Note this problem is true for feedback comments too, not just the files, as indicated in MDL-34653

          Show
          Eric Merrill added a comment - Note this problem is true for feedback comments too, not just the files, as indicated in MDL-34653
          Hide
          Eric Merrill added a comment -

          In mod/assign/locallib.php add_grade_form_elements(), the $grade var starts out as a grade object, but then gets overwritten into the grade value as a string, but below it is passed to add_plugin_grade_elements(), which expects the object.

          I changed the code so the grade string is stored in $gradestring, instead of overwriting $grade.

          Show
          Eric Merrill added a comment - In mod/assign/locallib.php add_grade_form_elements(), the $grade var starts out as a grade object, but then gets overwritten into the grade value as a string, but below it is passed to add_plugin_grade_elements(), which expects the object. I changed the code so the grade string is stored in $gradestring, instead of overwriting $grade.
          Hide
          Eric Merrill added a comment -

          Damyon - I added you back to this, do you think you can review my patch, and set for integration if it's up to snuff.

          Thanks

          Show
          Eric Merrill added a comment - Damyon - I added you back to this, do you think you can review my patch, and set for integration if it's up to snuff. Thanks
          Hide
          Damyon Wiese added a comment -

          This fix looks perfect to me - thanks Eric.

          Show
          Damyon Wiese added a comment - This fix looks perfect to me - thanks Eric.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Dan Poltawski added a comment -

          Thanks Eric - i've integrated this to 23 and master.

          Show
          Dan Poltawski added a comment - Thanks Eric - i've integrated this to 23 and master.
          Hide
          Tim Barker added a comment -

          Tested a per testing instructions, tested a few more combinations of this scenario and every time the file appeared.

          Show
          Tim Barker added a comment - Tested a per testing instructions, tested a few more combinations of this scenario and every time the file appeared.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          For the good and the bad... this is now part of Moodle and people around the world will start using it immediately, what a responsibility!

          Many thanks for your collaboration, yay!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - For the good and the bad... this is now part of Moodle and people around the world will start using it immediately, what a responsibility! Many thanks for your collaboration, yay! Closing, ciao
          Hide
          Paul Hague added a comment -

          That's great. Thank you.

          Show
          Paul Hague added a comment - That's great. Thank you.

            People

            • Votes:
              7 Vote for this issue
              Watchers:
              7 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: