Moodle
  1. Moodle
  2. MDL-30350

Late submissions should be highlighted with late information displayed on the list of submissions page

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0.5, 2.1.3, 2.2, 2.3
    • Fix Version/s: 2.1.4, 2.2.1
    • Component/s: Assignment (2.2)
    • Labels:
    • Testing Instructions:
      Hide
      • As a course convenor:
        • create a new assignment
        • set the due date to some time in the past
      • login as an enrolled student
        • submit some work
      • Log back in as the course convenor
        • navigate to the piece of work
        • View the submitted assignments

      Expected Result:
      the submitted assignment should be highlighted in some way to suggest that the work was submitted late

      Actual result:
      The row appears as normal

      Show
      As a course convenor: create a new assignment set the due date to some time in the past login as an enrolled student submit some work Log back in as the course convenor navigate to the piece of work View the submitted assignments Expected Result: the submitted assignment should be highlighted in some way to suggest that the work was submitted late Actual result: The row appears as normal
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-30350-master-1

      Description

      When viewing a list of submissions (e.g. http://moodle.local/mod/assignment/submissions.php?id=2), if a piece of work is submitted late:

      • the row should have CSS for theme designers to highlight it
      • it should show how late the work is

        Gliffy Diagrams

          Activity

          Hide
          Michael de Raadt added a comment -

          Hi, Andrew.

          We're bogged down with QA testing and the resulting bug fixing. It may be after this that we have a chance to go through our backlog of external peer reviews.

          Show
          Michael de Raadt added a comment - Hi, Andrew. We're bogged down with QA testing and the resulting bug fixing. It may be after this that we have a chance to go through our backlog of external peer reviews.
          Hide
          Rossiani Wijaya added a comment -

          Hi Andrew,

          Thank you for supplying patch to address the issue.

          The patch looks good and works well.

          Show
          Rossiani Wijaya added a comment - Hi Andrew, Thank you for supplying patch to address the issue. The patch looks good and works well.
          Hide
          Rossiani Wijaya added a comment -

          Submitting for integration review.

          Show
          Rossiani Wijaya added a comment - Submitting for integration review.
          Hide
          Aparup Banerjee added a comment -

          thanks for the patch

          This has been integrated and cherry-picked into master,2.2.x and 2.1.x.

          ready for testing.

          Show
          Aparup Banerjee added a comment - thanks for the patch This has been integrated and cherry-picked into master,2.2.x and 2.1.x. ready for testing.
          Hide
          Rajesh Taneja added a comment -

          Works Great
          Thanks for fixing this, Andrew.

          Show
          Rajesh Taneja added a comment - Works Great Thanks for fixing this, Andrew.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Yes, you did it!

          Now your code is part of the best weeklies released ever, many thanks!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - Yes, you did it! Now your code is part of the best weeklies released ever, many thanks! Closing, ciao

            People

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

              Dates

              • Created:
                Updated:
                Resolved: