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, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón, Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Dates

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