Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-29828

Quiz overdue time mis-displayed on the review page

          Details

          • Type: Bug
          • Status: Closed
          • Priority: 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:
          • Pull Master Diff URL:

            Description

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

              Gliffy Diagrams

                Attachments

                  Issue Links

                  has a non-specific relationship to

                  Improvement - An enhancement to an existing Moodle feature. MDL-29627 Quiz access rules should be a type of plugin

                  • Minor - Minor loss of function where workaround is possible
                  • Closed

                    Activity

                    Ascending order - Click to sort in descending order
                    Hide
                    Permalink
                    stronk7 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
                    stronk7 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
                    Permalink
                    nebgor Aparup Banerjee added a comment -

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

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

                    Works!
                    Test passed
                    Thanks

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

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

                    Closing as fixed, ciao

                    Show
                    stronk7 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:
                          Fix Release Date:
                          28/Nov/11