L-electron-Rare/moodle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MDL-87416 glossary: set page title on glossary entry details page.

Browse Source
This commit is contained in:
Stefan Topfstedt
2026-01-05 11:28:51 -08:00
parent 8316779bd9
commit 9ffeee1892
2 changed files with 36 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
public/mod/glossary/showentry.php
+4
View File
@@ -31,6 +31,10 @@ if ($eid) {
$entry->concept = format_string($entry->concept, true, ["escape" => false]);
$entries = array($entry);
$PAGE->set_title(implode(\moodle_page::TITLE_SEPARATOR, [
$entry->concept,
$entry->glossaryname,
]));
} else if ($concept) {
$course = $DB->get_record('course', array('id'=>$courseid), '*', MUST_EXIST);
require_course_login($course);
public/mod/glossary/tests/behat/showentry.feature
+32
View File
@@ -0,0 +1,32 @@
@mod @mod_glossary
Feature: Show glossary entry
In order to see the details of a glossary entry
As a user
I should be able to view the entry details on one page
Background:
Given the following "users" exist:
| username | firstname | lastname | email |
| teacher1 | Teacher | 1 | teacher1@example.com |
And the following "courses" exist:
| fullname | shortname | category |
| Course 1 | C1 | 0 |
And the following "course enrolments" exist:
| user | course | role |
| teacher1 | C1 | editingteacher |
And the following "activities" exist:
| activity | name | intro | displayformat | course | idnumber |
| glossary | Underused words | Say these more often. | fullwithauthor | C1 | g1 |
And the following "mod_glossary > categories" exist:
| glossary | name |
| g1 | nouns |
And the following "mod_glossary > entries" exist:
| glossary | concept | definition | user | categories |
| g1 | clamor | A loud noise. | teacher1 | nouns |
Scenario: View glossary entry details
Given I am on the "Underused words" "glossary activity" page logged in as teacher1
When I click on "Entry link: clamor" "link"
Then the page title should contain "clamor | Underused words"
And "clamor" "heading" should exist
And I should see "A loud noise."