Moodle
  1. Moodle
  2. MDL-37924

Quiz Admin settings defaults has sets 'fix'=true instead of 'adv'=true.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2.7, 2.3.4, 2.4.1
    • Fix Version/s: 2.3.5, 2.4.2
    • Component/s: Quiz
    • Labels:
      None
    • Rank:
      47681

      Description

      The format for advanced settings defaults was changed quite a while ago, but not all the quiz settings were updated correctly. The fix is to change 'fix' to 'adv' in mod/quiz/settings.php.

        Issue Links

          Activity

          Hide
          Tim Hunt added a comment -

          Submitting for integration.

          Show
          Tim Hunt added a comment - Submitting for integration.
          Hide
          Damyon Wiese 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.

          Cheers!

          Show
          Damyon Wiese 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. Cheers!
          Hide
          Dan Poltawski added a comment -

          Should I fix this in master? (Or you prefer for me to just wait for MDL-37786?)

          Show
          Dan Poltawski added a comment - Should I fix this in master? (Or you prefer for me to just wait for MDL-37786 ?)
          Hide
          Damyon Wiese added a comment -

          I think we should fix this in master just for the history and I'll fix any conflicts in MDL-37786.

          Show
          Damyon Wiese added a comment - I think we should fix this in master just for the history and I'll fix any conflicts in MDL-37786 .
          Hide
          Dan Poltawski added a comment -

          That was my thought too. Thanks.

          Integrated to master, 24 and 23.

          Show
          Dan Poltawski added a comment - That was my thought too. Thanks. Integrated to master, 24 and 23.
          Hide
          Tim Hunt added a comment -

          Well, since Damyon was happy to fix the merge conflicts in MDL-37786, master too is great. Why isn't MDL-37786 in integration yet?

          Show
          Tim Hunt added a comment - Well, since Damyon was happy to fix the merge conflicts in MDL-37786 , master too is great. Why isn't MDL-37786 in integration yet?
          Hide
          Frédéric Massart added a comment -

          Passed, thanks!

          Show
          Frédéric Massart added a comment - Passed, thanks!
          Hide
          Damyon Wiese added a comment -

          Congratulations this fix has been added to Moodle!

          You may want to dedicate this issue to someone special on this Valentines day.

          Thanks!

          Show
          Damyon Wiese added a comment - Congratulations this fix has been added to Moodle! You may want to dedicate this issue to someone special on this Valentines day. Thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved: