Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-2697

priority not always used correctly for CoreCouseOptionsDelegate

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 3.5.2
    • Fix Version/s: 3.6.0
    • Component/s: Courses
    • Labels:
    • Testing Instructions:
      Hide

      This issue only needs to be tested in 1 OS.

      1. In web as admin, configure a course to have competencies and course completion (e.g. Mobile 34 > MM Dev Testing).
      2. In the app, open that course as a student. Check that the order of the tabs at top is: Contents, Participants, Grades, Competencies, Completion.
      Show
      This issue only needs to be tested in 1 OS. In web as admin, configure a course to have competencies and course completion (e.g. Mobile 34 > MM Dev Testing). In the app, open that course as a student. Check that the order of the tabs at top is: Contents, Participants, Grades, Competencies, Completion.
    • Affected Branches:
      MOODLE_35_STABLE
    • Fixed Branches:
      MOODLE_36_STABLE
    • Sprint:
      Moodle Mobile 3.6.0

      Description

      I have a course format which defines the following handlers:

      'handlers' => [ // Different places where the plugin will display content.
                  'oustudyplan' => [ 
                      'delegate' => 'CoreCourseFormatDelegate', 
                      'method' => 'mobile_course_view',
                      'styles' => [
                          'url' => $CFG->wwwroot . '/course/format/oustudyplan/mobile_css.php',
                          'version' => 2018101900
                      ],
                      'displaysectionselector' => false,
                      'displayenabledownload' => false,
                      'init' => 'oustudyplan_init'
                  ],
                  'assessmentpage' => [
                      'delegate' => 'CoreCourseOptionsDelegate',
                      'method' => 'assessment_page',
                      'displaydata' => [
                          'title' => 'special_assessment',
                          'class' => 'oustudyplan-area'
                      ],
                      'priority' => 5,
                      'init' => 'assessment_init'
                  ],
                  'tutorialspage' => [
                      'delegate' => 'CoreCourseOptionsDelegate',
                      'method' => 'tutorials_page',
                      'displaydata' => [
                          'title' => 'special_tutorials',
                          'class' => 'oustudyplan-area'
                      ],
                      'priority' => 4,
                      'init' => 'tutorials_init'
                  ],
                  'forumpage' => [
                      'delegate' => 'CoreCourseOptionsDelegate',
                      'method' => 'forums_page',
                      'displaydata' => [
                          'title' => 'special_forums',
                          'class' => 'oustudyplan-area'
                      ],
                      'priority' => 3,
                      'init' => 'forums_init'
                  ],
                  'resourcespage' => [
                      'delegate' => 'CoreCourseOptionsDelegate',
                      'method' => 'resources_page',
                      'displaydata' => [
                          'title' => 'special_resources',
                          'class' => 'oustudyplan-area'
                      ],
                      'priority' => 2,
                      'init' => 'resources_init'
                  ]
              ],
      

      I have disabled all the other tabs on the course page, except for Contents. As I understand it, this should mean that the tabs are always displayed like this:

      Contents Assessment Tutorials Forums Resources

      However, this order isn't always being respected, as show in the attached screenshot:

       

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            marxjohnson Mark Johnson
            Peer reviewer:
            Pau Ferrer
            Integrator:
            Juan Leyva
            Tester:
            Berta Maroto
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              11/Jan/19