Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: 2.0
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Gliffy Diagrams

        Issue Links

          Activity

          Hide
          Jamie Pratt added a comment -

          Would be great if you had time to tackle this Tim.

          Show
          Jamie Pratt added a comment - Would be great if you had time to tackle this Tim.
          Hide
          Jamie Pratt added a comment -

          Attaching a file settingstree.php that uses the new mechanism in the admin tree to let activity modules do more than just add one page to the admin menu. This file includes commented out code to include general settings for all reports and a comment about where to add code for the quiz module general settings page.

          It also searches mod/quiz/report sub folders for any settings.php pages. If any are found or if some general settings for reports have been defined a sub folder will be added quiz/reports.

          Right now you can drop this into mod/quiz. You won't see anything until some actual pages are added to the extra categories on the admin menu. If you drop this into mod/quiz then you will no longer see the settings page defined by config.html The same page can be used in HEAD and 1.9

          I added a TODO comment to an if branch in HEAD that needs deleting as soon as quiz/config.html is migrated to used the admin menu api.

          Show
          Jamie Pratt added a comment - Attaching a file settingstree.php that uses the new mechanism in the admin tree to let activity modules do more than just add one page to the admin menu. This file includes commented out code to include general settings for all reports and a comment about where to add code for the quiz module general settings page. It also searches mod/quiz/report sub folders for any settings.php pages. If any are found or if some general settings for reports have been defined a sub folder will be added quiz/reports. Right now you can drop this into mod/quiz. You won't see anything until some actual pages are added to the extra categories on the admin menu. If you drop this into mod/quiz then you will no longer see the settings page defined by config.html The same page can be used in HEAD and 1.9 I added a TODO comment to an if branch in HEAD that needs deleting as soon as quiz/config.html is migrated to used the admin menu api.
          Hide
          Jamie Pratt added a comment -

          The TODO comment is in admin/settings/plugins.php

          Show
          Jamie Pratt added a comment - The TODO comment is in admin/settings/plugins.php
          Hide
          Tim Hunt added a comment -

          I have nearly finished this, but it is 6:00pm Friday and I have had enough. Just the review options to go. Should be done on Monday.

          Show
          Tim Hunt added a comment - I have nearly finished this, but it is 6:00pm Friday and I have had enough. Just the review options to go. Should be done on Monday.
          Hide
          Tim Hunt added a comment -

          Done.

          Note that if non of the quiz reports have any settings, then the quiz settings appear directly under Plugins -> Activity modules. If some quiz reports do have settings, then Plugins -> Activity modules -> Quiz becomes an expandable menu with General settings and XXX Report settings, etc.

          Show
          Tim Hunt added a comment - Done. Note that if non of the quiz reports have any settings, then the quiz settings appear directly under Plugins -> Activity modules. If some quiz reports do have settings, then Plugins -> Activity modules -> Quiz becomes an expandable menu with General settings and XXX Report settings, etc.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: