Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-34839

samigo: Property to hide Scores screen Delete submission column from instructor view

    XMLWordPrintable

    Details

    • 12 status:
      Resolved
    • Property addition/change required:
      Yes
    • Previous Issue Keys:
      SAM-3210

      Description

      Per my comment in SAM-2901, the Delete Submissions option in Tests & Quizzes is a potential pitfall for instructors to accidentally, permanently remove student work. There are other workarounds for adjusting scores to correct which submission is scored, and you can allow a student a retake (rather than wiping their submission).

      UVA has a solution for this issue that we intend to contribute back: a Sakai property, samigo.removeSubmission.restricted which behaves as follows:

      1. If samigo.removeSubmission.restricted = true is set at the institutional level, the Delete Submission option is not visible to instructors by default.
      2. Admin users are ALWAYS able to Delete Submission on behalf of an instructor, if requested.
      3. If an instructor/project site administrator complains and has a valid reason for deleting published assessment after submissions have been made, admin user can add the property to an individual site, samigo.removeSubmission.restricted = false. (Setting the site-level property true again hides the option from instructors.)

      NOTE: In UVA's instance, the above property was intended to be applied WITH samigo.removePubAssessment.restricted.afterStart (SAM-2147) enabled at the institutional level. As such, testing steps for After Fix below were copied from our local jira and express desired behavior when BOTH properties are enabled.

      Testing Steps

      Before Fix (To see undesirable behavior, and where fix is needed):

      1. In a site with at least 2 students, as instructor, create and publish 2 assessments with NO time limit.
      2. Go to Published Copies and verify there IS a Remove option enabled in the drop-down menu for each test.
      3. Remove one of the assessments and verify it's deleted.
      4. Log in as student and submit the remaining assessment.
      5. As instructor, go to Published Copies and verify the Remove option is disabled in the drop-down menu for the test.
      6. Go to Scores screen for the test and note presence of Delete column to remove submissions.
      7. Click the X for the submission, confirm, and verify it's gone.
      8. Go back to Published Copies and verify the Remove option is enabled in the drop-down menu for the test.
      9. Select Remove and confirm you can delete the assessment.
      10. Publish a THIRD assessment.
      11. Start taking it as a student. Do NOT submit; leave it in progress.
      12. Log in as instructor and verify that there Remove option is disabled in the drop-down menu for the test.
      13. As a DIFFERENT student (not the one whose test is in progress), submit the test.
      14. As instructor, go to Published Copies.
      15. Verify that the Remove option is disabled in the drop-down menu for the test.
      16. Go to Scores screen for test and note presence of Delete column to remove submissions.
      17. Click the X for the completed submission, confirm, and verify it's gone.
      18. Go back to Published Copies and verify there is still a test in progress and Remove option is disabled in the drop-down menu for the test.

      After Fix (Verify behavior with conditionals in place):

      1. In a site with at least 2 students, as instructor, create and publish 2 assessments with NO time limit.
      2. Go to Published Copies and verify there IS a Remove option in the drop-down menu for each test.
      3. Remove one of the assessments and verify it's deleted.
      4. Start taking the remaining assessment as a student. Do NOT submit; leave it in progress
      5. As instructor, go to Published Copies and verify the Remove option in the drop-down menu is disabled.
      6. Log back in as the student and submit the test.
      7. As instructor, go to Published Copies and verify the Remove option in the drop-down is still disabled.
      8. Log in as Student 2 and start the assessment. Save but do NOT submit.
      9. As instructor, go to Scores screen for test and note there is NO place to delete submissions.
      10. Log in as admin.
      11. Go to Published Copies and verify the Remove option IS enabled in the drop-down menu for the test. You can select it and get the warning that you need to delete submissions before removing the assessment.
      12. Use the remove submissions warning button to go to the Scores screen and delete the submission.
      13. Click the X for the submission, confirm, and verify it's gone.
      14. Go back to Published Copies and verify the Remove option IS enabled in the drop-down menu for the test (in spite of there being a test in progress).
      15. Select Remove and confirm you can delete the assessment.
      16. Go to Administration Workspace > Sites.
      17. Locate your site and add the property samigo.removeSubmission.restricted = false at the site level.
      18. Log in as instructor and publish ANOTHER test.
      19. Log in as student and submit it.
      20. Log in as instructor, confirm that you have the Remove option available for the test with the submission.
      21. You can select it and get the warning that you need to delete submissions before removing the assessment.
      22. Use the remove submissions warning button to go to the Scores screen and delete the submission.
      23. Go back to Published Copies and confirm you can delete the test.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  bjones86 Brian Jones
                  Reporter:
                  rainribbon Tiffany Stull
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  11 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Git Source Code