Elgg / Elgg

mod/pages/views/default/pages/sidebar/navigation.php

<?php
/**
 * Navigation menu for a user's or a group's pages
 *
 * @uses $vars['page'] Page object if manually setting selected item
 */

// add the jquery treeview files for navigation
elgg_load_css('jquery.treeview');
elgg_require_js('pages/sidebar/navigation');

$selected_page = elgg_extract('page', $vars, false);

$title = elgg_echo('pages:navigation');

pages_register_navigation_tree(elgg_get_page_owner_entity(), $selected_page);

It seems like elgg_get_page_owner_entity() can also be of type false; however, parameter $container of pages_register_navigation_tree() does only seem to accept ElggEntity, maybe add an additional type check?

It seems like $selected_page can also be of type false; however, parameter $selected of pages_register_navigation_tree() does only seem to accept ElggPage, maybe add an additional type check?

$content = elgg_view_menu('pages_nav', [
	'class' => 'pages-nav',
]);

if (!$content) {
	$content = elgg_format_element('p', [], elgg_echo('pages:none'));
}

echo elgg_view_module('aside', $title, $content);