Elgg / Elgg

mod/bookmarks/views/default/resources/bookmarks/view.php

<?php
/**
 * View a bookmark
 *
 * @package ElggBookmarks
 */

$guid = elgg_extract('guid', $vars);

elgg_entity_gatekeeper($guid, 'object', 'bookmarks');

$bookmark = get_entity($guid);

$page_owner = elgg_get_page_owner_entity();

elgg_group_gatekeeper();

$crumbs_title = $page_owner->name;

if ($page_owner instanceof ElggGroup) {
	elgg_push_breadcrumb($crumbs_title, "bookmarks/group/$page_owner->guid/all");
} else {
	elgg_push_breadcrumb($crumbs_title, "bookmarks/owner/$page_owner->username");
}

$title = $bookmark->title;

elgg_push_breadcrumb($title);

It seems like $title can also be of type array; however, parameter $text of elgg_push_breadcrumb() does only seem to accept string, maybe add an additional type check?

$content = elgg_view_entity($bookmark, [
	'full_view' => true,
	'show_responses' => true,
]);

$body = elgg_view_layout('content', [
	'content' => $content,
	'title' => $title,
	'filter' => '',
]);

echo elgg_view_page($title, $body);

It seems like $title can also be of type array; however, parameter $title of elgg_view_page() does only seem to accept string, maybe add an additional type check?