Affects Version/s: 2.1.3, 2.3
ShowEnable the popup display option for the page module. Create a new page activity with the display option set to popup. Click Save and display Check that the navigation bar only shows the activity name once. Browse to the course and click on the page link in the main content. Ensure you get a popup.
- Enable the popup display option for the page module.
- Create a new page activity with the display option set to popup.
- Click Save and display
- Check that the navigation bar only shows the activity name once.
- Browse to the course and click on the page link in the main content.
- Ensure you get a popup.
Affected Branches:MOODLE_21_STABLE, MOODLE_23_STABLE
Fixed Branches:MOODLE_21_STABLE, MOODLE_22_STABLE
Pull from Repository:
Pull Master Branch:wip-
Pull Master Diff URL:
When a Page resource is set to open in a popup window and you either edit that resource or click it from the Recent Activity block, the breadcrumbs on the resulting page list the page name twice. For example, if the page name is "Page" the breadcrumb entry for the page shows up as "PagePage". I have reproduced this in 2.1.3 and the latest 2.3. See attached screenshot from the latest 2.3, where it also includes an icon in between the double names. My 2.1.3 does not have the icon but it is using a custom theme whereas my 2.3 is not.
Steps to reproduce:
- Create a new Page in a course and set the page options to display in a popup. Save.
- Click the edit icon next to the page name or click the page name in the Recent Activity block.
- Notice the double name in the breadcrumbs.
It does not double the page name when the page display is set to "Open" rather than "In a popup."
The line of code that is duplicating the name is in render_navigation_node() within lib/outputrenderers.php, line 2514 in the current 2.3:
$content already has the name and then the name is appended to it, duplicating it. I don't know this part of code well enough to know if the function itself is wrong or if what's being fed into the function is wrong.