Moodle
  1. Moodle
  2. MDL-29828

Quiz overdue time mis-displayed on the review page

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.2, 2.2
    • Fix Version/s: 2.1.3
    • Component/s: Quiz
    • Labels:
      None
    • Testing Instructions:
      Hide

      1. Create a quiz with a (short) time limit.
      2. Attempt it as a student, and submit it late. The easiest way to do that is by turning off JS.
      3. Review the quiz as a teacher, make sure the amount of time overdue is clearly displayed.

      Show
      1. Create a quiz with a (short) time limit. 2. Attempt it as a student, and submit it late. The easiest way to do that is by turning off JS. 3. Review the quiz as a teacher, make sure the amount of time overdue is clearly displayed.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull from Repository:
    • Pull Master Branch:

      Description

      format_time is called twice. See screen-shot on MDL-29627 for more details.

        Gliffy Diagrams

          Issue Links

            Activity

            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
            Aparup Banerjee added a comment -

            Thanks Tim, this has been integrated and is ready for testing.

            Show
            Aparup Banerjee added a comment - Thanks Tim, this has been integrated and is ready for testing.
            Hide
            Ankit Agarwal added a comment -

            Works!
            Test passed
            Thanks

            Show
            Ankit Agarwal added a comment - Works! Test passed Thanks
            Hide
            Eloy Lafuente (stronk7) added a comment -

            Done, your delicious hacks have been sent upstream, many thanks!

            Closing as fixed, ciao

            Show
            Eloy Lafuente (stronk7) added a comment - Done, your delicious hacks have been sent upstream, many thanks! Closing as fixed, ciao

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved: