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

Navigation block doesn't handle ampersands in course name correctly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4.1
    • Fix Version/s: 2.3.7, 2.4.4
    • Component/s: Navigation
    • Labels:
    • Testing Instructions:
      Hide
      1. Add a new course with the title "Test & ampersands"
      2. Check in the navigation that the title for the course node is properly formatted.
      3. Enrol a student into the course and log in as that student.
      4. Visit the My home page and check that the title appears correctly.
      Show
      Add a new course with the title "Test & ampersands" Check in the navigation that the title for the course node is properly formatted. Enrol a student into the course and log in as that student. Visit the My home page and check that the title appears correctly.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE
    • Pull Master Branch:
      wip-MDL-38055-m25

      Description

      I just saw that block_navigation doesn't handle ampersands in course names correctly when creating the "My courses" list.

      We have a course with a ampersand in the course name. In the title of the course link in the mycourses list, the ampersand gets displayed as HTML Entity instead of being displayed as ampersand (see screenshot).

      As the content of the mycourses list comes from a AJAX call, I don't know where the issue is handled best.

        Attachments

          Activity

            People

            Assignee:
            samhemelryk Sam Hemelryk
            Reporter:
            abias Alexander Bias
            Peer reviewer:
            Adrian Greeve
            Integrator:
            Damyon Wiese
            Tester:
            Frédéric Massart
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              13/May/13