Elgg / Elgg

views/default/admin/configure_utilities/default_widgets.php

<?php
/**
 * Default widgets landing page.
 *
 * @package Elgg.Core
 * @subpackage Administration.DefaultWidgets
 */

elgg_push_context('default_widgets');
$widget_context = get_input('widget_context');
$list = elgg_trigger_plugin_hook('get_list', 'default_widgets', null, []);

// default to something if we can
if (!$widget_context && $list) {

The expression $list of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

	$widget_context = $list[0]['widget_context'];
}

$current_info = null;
$tabs = [];
foreach ($list as $info) {
	$url = "admin/configure_utilities/default_widgets?widget_context={$info['widget_context']}";
	$selected = false;
	if ($widget_context == $info['widget_context']) {
		$selected = true;
		$current_info = $info;
	}

	$tabs[] = [
		'title' => $info['name'],
		'url' => $url,
		'selected' => $selected
	];
}

$tabs_vars = [
	'tabs' => $tabs
];

echo elgg_view('navigation/tabs', $tabs_vars);

echo elgg_view('output/longtext', ['value' => elgg_echo('admin:default_widgets:instructions')]);

if (!$current_info) {
	$content = elgg_echo('admin:default_widgets:unknown_type');
} else {
	// default widgets are owned and saved to the site.
	elgg_set_page_owner_guid(1);
	elgg_push_context($current_info['widget_context']);

	$content = elgg_view_layout('widgets', [
		'num_columns' => $current_info['widget_columns'],
	]);
	elgg_pop_context();
}
elgg_pop_context();

echo $content;