Moodle
  1. Moodle
  2. MDL-34321

Assignment only partially disappears when Teacher using role switch to Student submits assignment

    Details

    • Story Points:
      8
    • Rank:
      53541
    • Sprint:
      FRONTEND Sprint 7

      Description

      When using Assignment: Advanced Uploading of File, I switched my role to Student (I have, normally, Course Creator and Teacher roles). I submitted an assignment as a test. I usually use a dedicated fake student account for this task, but did not at this point. I successfully uploaded an assignment in this state. Because of the assignment settings, I (as a student) am not able to modify the assignment after it was 'sent for marking.' However, now all teachers see that there's "1 submitted assignment" yet my submission appears no where else. When clicking "View 1 submitted assignment," for instance, the list is empty.

      This is a minor inconvenience, and nothing more. I checked and a regular student assignment submission appears as normal.

      To test:

      1. Create an Assignment: Advanced Uploading of Files
      2. Switch role to Student.
      3. Submit some files and 'send for marking'
      4. Return to normal role
      5. Click "View 1 submitted assignment." Verify assignment does not appear.

        Activity

        Hide
        Michael de Raadt added a comment -

        Thanks for reporting that.

        The function that counts submissions is counting correctly based on the submissions in the database, but the table is not including users who are not enrolled as students (with the permission to submit that teachers don't have) in the course. It could be possible to have the same permission restriction performed on the count.

        Show
        Michael de Raadt added a comment - Thanks for reporting that. The function that counts submissions is counting correctly based on the submissions in the database, but the table is not including users who are not enrolled as students (with the permission to submit that teachers don't have) in the course. It could be possible to have the same permission restriction performed on the count.
        Hide
        Rossiani Wijaya added a comment -

        This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue.

        Show
        Rossiani Wijaya added a comment - This issue was assigned to me automatically, however I will not be able to work on this issue in the immediate future. In order to create a truer sense of the state of this issue and to allow other developers to have chance to become involved, I am removing myself as the assignee of this issue.
        Hide
        Damyon Wiese added a comment -

        Looks good to me.

        [Y] Syntax
        [Y] Whitespace
        [-] Output
        [-] Language
        [-] Databases
        [Y] Testing (instructions and automated tests)
        [-] Security
        [-] Documentation
        [Y] Git
        [-] Third party code
        [Y] Sanity check

        Submitting for integration.

        Show
        Damyon Wiese added a comment - Looks good to me. [Y] Syntax [Y] Whitespace [-] Output [-] Language [-] Databases [Y] Testing (instructions and automated tests) [-] Security [-] Documentation [Y] Git [-] Third party code [Y] Sanity check Submitting for integration.
        Hide
        Sam Hemelryk added a comment -

        Thanks guys - this has been integrated now.

        Show
        Sam Hemelryk added a comment - Thanks guys - this has been integrated now.
        Hide
        Frédéric Massart added a comment -

        Passing. Thanks!

        Show
        Frédéric Massart added a comment - Passing. Thanks!
        Hide
        Damyon Wiese added a comment -

        Twas the week before Christmas,
        And all though HQ
        Devs were scrambling to finish peer review.
        They sent all their issues,
        and rushed out the door -
        "To the beach!" someone heard them roar!

        This issue has been released upstream. Thanks!

        Show
        Damyon Wiese added a comment - Twas the week before Christmas, And all though HQ Devs were scrambling to finish peer review. They sent all their issues, and rushed out the door - "To the beach!" someone heard them roar! This issue has been released upstream. Thanks!

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Agile