Permalink
Browse files

feature(reportedcontent): only load javascript when needed

  • Loading branch information...
jdalsem committed Mar 24, 2016
1 parent c589e8e commit 29c39cd79b13abc6787e805f83747e035f981b56
@@ -36,8 +36,6 @@ function reportedcontent_init() {
elgg_register_plugin_hook_handler('register', 'menu:user_hover', 'reportedcontent_user_hover_menu'); elgg_register_plugin_hook_handler('register', 'menu:user_hover', 'reportedcontent_user_hover_menu');
// Add admin menu item
// @todo Might want to move this to a 'feedback' section. something other than utils
elgg_register_admin_menu_item('administer', 'reportedcontent', 'administer_utilities'); elgg_register_admin_menu_item('administer', 'reportedcontent', 'administer_utilities');
elgg_register_widget_type( elgg_register_widget_type(
@@ -78,22 +76,30 @@ function reportedcontent_page_handler($page) {
* Add report user link to hover menu * Add report user link to hover menu
*/ */
function reportedcontent_user_hover_menu($hook, $type, $return, $params) { function reportedcontent_user_hover_menu($hook, $type, $return, $params) {
$user = $params['entity']; if (!elgg_is_logged_in()) {
return;
}
$user = elgg_extract('entity', $params);
/* @var ElggUser $user */ /* @var ElggUser $user */
$profile_url = urlencode($user->getURL()); if (elgg_get_logged_in_user_guid() == $user->guid) {
$name = urlencode($user->name); return;
$url = "reportedcontent/add?address=$profile_url&title=$name";
if (elgg_is_logged_in() && elgg_get_logged_in_user_guid() != $user->guid) {
$item = new ElggMenuItem(
'reportuser',
elgg_echo('reportedcontent:user'),
$url);
$item->setSection('action');
$item->addLinkClass('elgg-lightbox');
$return[] = $item;
} }
$href = elgg_http_add_url_query_elements('reportedcontent/add', [
'address' => $user->getURL(),
'title' => $user->name,
]);
$return[] = \ElggMenuItem::factory([
'name' => 'reportuser',
'text' => elgg_echo('reportedcontent:user'),
'href' => $href,
'section' => 'action',
'link_class' => 'elgg-lightbox',
'deps' => 'elgg/reportedcontent',
]);
return $return; return $return;
} }
@@ -5,17 +5,15 @@
* @package ElggReportedContent * @package ElggReportedContent
*/ */
$list = elgg_list_entities_from_metadata([ elgg_require_js('elgg/reportedcontent');
echo elgg_list_entities_from_metadata([
'type' => 'object', 'type' => 'object',
'subtype' => 'reported_content', 'subtype' => 'reported_content',
'order_by_metadata' => [ 'order_by_metadata' => [
'name' => 'state', 'name' => 'state',
'direction' => 'ASC', 'direction' => 'ASC',
'as' => 'text', 'as' => 'text',
], ],
]); 'no_results' => elgg_echo('reportedcontent:none'),
if (!$list) { ]);
$list = '<p class="mtm">' . elgg_echo('reportedcontent:none') . '</p>';
}
echo $list;

0 comments on commit 29c39cd

Please sign in to comment.