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

Unavailable course displays incorrect breadcrumbs

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.4, 2.2.1, 2.3
    • Fix Version/s: 2.1.5, 2.2.2
    • Component/s: Course, Navigation
    • Labels:
    • Testing Instructions:
      Hide

      1. Create a new course. Set it to 'unavailable to students'.
      2. Enrol a test student user on the course.
      3. Log in as that user and visit the course.
      The error message 'This course is currently unavailable to students' appears.

      Expected behaviour:
      Breadcrumb trail does not include the unavailable course, nor any other course.

      Actual behaviour:
      Breadcrumb trail includes another course.

      Show
      1. Create a new course. Set it to 'unavailable to students'. 2. Enrol a test student user on the course. 3. Log in as that user and visit the course. The error message 'This course is currently unavailable to students' appears. Expected behaviour: Breadcrumb trail does not include the unavailable course, nor any other course. Actual behaviour: Breadcrumb trail includes another course.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Pull Master Branch:
      wip-MDL-31197-m23

      Description

      When a student looks at an unavailable course and gets the 'This course is currently unavailable to students' error page, that page shows breadcrumbs to another (apparently random) course.

      I assume that the current course name is intentionally not displayed, in order to avoid revealing the name of the course when it is unavailable. If this is the case, then correct behaviour would be for the breadcrumbs on this page to be based on the site course and not include a course.

        Attachments

          Activity

            People

            • Assignee:
              samhemelryk Sam Hemelryk
              Reporter:
              quen Sam Marshall
              Peer reviewer:
              Rajesh Taneja
              Integrator:
              Eloy Lafuente (stronk7)
              Tester:
              Ankit Agarwal
              Participants:
              Component watchers:
              Amaia Anabitarte, Bas Brands, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón, Jake Dallimore, Jun Pataleta, Ryan Wyllie
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                12/Mar/12