diff options
Diffstat (limited to 'mod')
| -rw-r--r-- | mod/blog/lib/blog.php | 5 | ||||
| -rw-r--r-- | mod/bookmarks/pages/bookmarks/view.php | 1 | ||||
| -rw-r--r-- | mod/file/pages/file/view.php | 1 | ||||
| -rw-r--r-- | mod/groups/lib/discussion.php | 5 | ||||
| -rw-r--r-- | mod/groups/start.php | 2 | ||||
| -rw-r--r-- | mod/likes/actions/likes/delete.php | 17 | ||||
| -rw-r--r-- | mod/likes/languages/en.php | 1 | ||||
| -rw-r--r-- | mod/likes/views/default/annotation/likes.php | 4 | ||||
| -rw-r--r-- | mod/likes/views/default/likes/button.php | 8 | ||||
| -rw-r--r-- | mod/messages/start.php | 10 | ||||
| -rw-r--r-- | mod/pages/pages/pages/view.php | 3 | ||||
| -rw-r--r-- | mod/pages/start.php | 4 | ||||
| -rw-r--r-- | mod/pages/views/default/pages/sidebar/navigation.php | 6 | ||||
| -rw-r--r-- | mod/thewire/pages/thewire/view.php | 1 | ||||
| -rw-r--r-- | mod/thewire/start.php | 3 | 
15 files changed, 37 insertions, 34 deletions
diff --git a/mod/blog/lib/blog.php b/mod/blog/lib/blog.php index 43de7a646..9a02a8cc3 100644 --- a/mod/blog/lib/blog.php +++ b/mod/blog/lib/blog.php @@ -22,8 +22,9 @@ function blog_get_page_content_read($guid = NULL) {  	$return['filter'] = '';  	if (!elgg_instanceof($blog, 'object', 'blog')) { -		$return['content'] = elgg_echo('noaccess'); -		return $return; +		register_error(elgg_echo('noaccess')); +		$_SESSION['last_forward_from'] = current_page_url(); +		forward('');  	}  	$return['title'] = $blog->title; diff --git a/mod/bookmarks/pages/bookmarks/view.php b/mod/bookmarks/pages/bookmarks/view.php index c819b8b41..70a6a5bfe 100644 --- a/mod/bookmarks/pages/bookmarks/view.php +++ b/mod/bookmarks/pages/bookmarks/view.php @@ -8,6 +8,7 @@  $bookmark = get_entity(get_input('guid'));  if (!$bookmark) {  	register_error(elgg_echo('noaccess')); +	$_SESSION['last_forward_from'] = current_page_url();  	forward('');  } diff --git a/mod/file/pages/file/view.php b/mod/file/pages/file/view.php index ec51b30e6..6c9566a89 100644 --- a/mod/file/pages/file/view.php +++ b/mod/file/pages/file/view.php @@ -8,6 +8,7 @@  $file = get_entity(get_input('guid'));  if (!$file) {  	register_error(elgg_echo('noaccess')); +	$_SESSION['last_forward_from'] = current_page_url();  	forward('');  } diff --git a/mod/groups/lib/discussion.php b/mod/groups/lib/discussion.php index 55642644d..02ab27fdc 100644 --- a/mod/groups/lib/discussion.php +++ b/mod/groups/lib/discussion.php @@ -149,8 +149,9 @@ function discussion_handle_view_page($guid) {  	$topic = get_entity($guid);  	if (!$topic) { -		register_error(elgg_echo('discussion:topic:notfound')); -		forward(); +		register_error(elgg_echo('noaccess')); +		$_SESSION['last_forward_from'] = current_page_url(); +		forward('');  	}  	$group = $topic->getContainerEntity(); diff --git a/mod/groups/start.php b/mod/groups/start.php index aeab0649a..e39c37000 100644 --- a/mod/groups/start.php +++ b/mod/groups/start.php @@ -792,7 +792,7 @@ function discussion_page_handler($page) {   * @return string   */  function discussion_override_topic_url($entity) { -	return 'discussion/view/' . $entity->guid; +	return 'discussion/view/' . $entity->guid . '/' . elgg_get_friendly_title($entity->title);  }  /** diff --git a/mod/likes/actions/likes/delete.php b/mod/likes/actions/likes/delete.php index db3036bb4..88cb84467 100644 --- a/mod/likes/actions/likes/delete.php +++ b/mod/likes/actions/likes/delete.php @@ -4,17 +4,12 @@   *   */ -$likes = elgg_get_annotations(array( -	'guid' => (int) get_input('guid'), -	'annotation_owner_guid' => elgg_get_logged_in_user_guid(), -	'annotation_name' => 'likes', -)); -if ($likes) { -	if ($likes[0]->canEdit()) { -		$likes[0]->delete(); -		system_message(elgg_echo("likes:deleted")); -		forward(REFERER); -	} +$id = (int) get_input('id'); +$like = elgg_get_annotation_from_id($id); +if ($like && $like->canEdit()) { +	$like->delete(); +	system_message(elgg_echo("likes:deleted")); +	forward(REFERER);  }  register_error(elgg_echo("likes:notdeleted")); diff --git a/mod/likes/languages/en.php b/mod/likes/languages/en.php index 4a98d7cfe..b9460aa10 100644 --- a/mod/likes/languages/en.php +++ b/mod/likes/languages/en.php @@ -17,6 +17,7 @@ $english = array(  	'likes:userlikedthis' => '%s like',  	'likes:userslikedthis' => '%s likes',  	'likes:river:annotate' => 'likes', +	'likes:delete:confirm' => 'Are you sure you want to unlike this?',  	'river:likes' => 'likes %s %s', diff --git a/mod/likes/views/default/annotation/likes.php b/mod/likes/views/default/annotation/likes.php index d41522fc4..abd4df823 100644 --- a/mod/likes/views/default/annotation/likes.php +++ b/mod/likes/views/default/annotation/likes.php @@ -29,9 +29,9 @@ $friendlytime = elgg_view_friendly_time($like->time_created);  if ($like->canEdit()) {  	$delete_button = elgg_view("output/confirmlink",array( -						'href' => "action/likes/delete?annotation_id={$like->id}", +						'href' => "action/likes/delete?id={$like->id}",  						'text' => "<span class=\"elgg-icon elgg-icon-delete float-alt\"></span>", -						'confirm' => elgg_echo('deleteconfirm'), +						'confirm' => elgg_echo('likes:delete:confirm'),  						'encode_text' => false,  					));  } diff --git a/mod/likes/views/default/likes/button.php b/mod/likes/views/default/likes/button.php index bc7c8fd8a..956bbcb19 100644 --- a/mod/likes/views/default/likes/button.php +++ b/mod/likes/views/default/likes/button.php @@ -24,7 +24,13 @@ if (elgg_is_logged_in() && $vars['entity']->canAnnotate(0, 'likes')) {  		);  		$likes_button = elgg_view('output/url', $params);  	} else { -		$url = elgg_get_site_url() . "action/likes/delete?guid={$guid}"; +		$like = elgg_get_annotations(array( +			'guid' => $guid, +			'annotation_owner_guid' => elgg_get_logged_in_user_guid(), +			'annotation_name' => 'likes', +		)); +		$like = $like[0]; +		$url = elgg_get_site_url() . "action/likes/delete?id={$like->id}";  		$params = array(  			'href' => $url,  			'text' => elgg_view_icon('thumbs-up-alt'), diff --git a/mod/messages/start.php b/mod/messages/start.php index 1a2709324..2e61d6e21 100644 --- a/mod/messages/start.php +++ b/mod/messages/start.php @@ -356,16 +356,10 @@ function messages_count_unread() {  			"msg_msg.name_id='{$map['msg']}' AND msg_msg.value_id='{$map[1]}'",  		),  		'owner_guid' => $user_guid, -		'limit' => 0 +		'count' => true,  	); -	$num_messages = elgg_get_entities_from_metadata($options); - -	if (is_array($num_messages)) { -		return sizeof($num_messages); -	} - -	return 0; +	return elgg_get_entities_from_metadata($options);  }  /** diff --git a/mod/pages/pages/pages/view.php b/mod/pages/pages/pages/view.php index 6b9d03f49..e1c3fdbfd 100644 --- a/mod/pages/pages/pages/view.php +++ b/mod/pages/pages/pages/view.php @@ -9,7 +9,8 @@ $page_guid = get_input('guid');  $page = get_entity($page_guid);  if (!$page) {  	register_error(elgg_echo('noaccess')); -	forward(); +	$_SESSION['last_forward_from'] = current_page_url(); +	forward('');  }  elgg_set_page_owner_guid($page->getContainerGUID()); diff --git a/mod/pages/start.php b/mod/pages/start.php index 834e98870..c385131bd 100644 --- a/mod/pages/start.php +++ b/mod/pages/start.php @@ -106,10 +106,6 @@ function pages_page_handler($page) {  	elgg_load_library('elgg:pages'); -	// add the jquery treeview files for navigation -	elgg_load_js('jquery-treeview'); -	elgg_load_css('jquery-treeview'); -  	if (!isset($page[0])) {  		$page[0] = 'all';  	} diff --git a/mod/pages/views/default/pages/sidebar/navigation.php b/mod/pages/views/default/pages/sidebar/navigation.php index fe017b1a7..65eb500c8 100644 --- a/mod/pages/views/default/pages/sidebar/navigation.php +++ b/mod/pages/views/default/pages/sidebar/navigation.php @@ -5,6 +5,11 @@   * @uses $vars['page'] Page object if manually setting selected item   */ +// add the jquery treeview files for navigation +elgg_load_js('jquery-treeview'); +elgg_load_css('jquery-treeview'); + +  $selected_page = elgg_extract('page', $vars, false);  if ($selected_page) {  	$url = $selected_page->getURL(); @@ -51,4 +56,3 @@ if ($selected_page) {  });  </script> - diff --git a/mod/thewire/pages/thewire/view.php b/mod/thewire/pages/thewire/view.php index 1818e725a..1709e5e9a 100644 --- a/mod/thewire/pages/thewire/view.php +++ b/mod/thewire/pages/thewire/view.php @@ -6,6 +6,7 @@  $post = get_entity(get_input('guid'));  if (!$post) {  	register_error(elgg_echo('noaccess')); +	$_SESSION['last_forward_from'] = current_page_url();  	forward('');  }  $owner = $post->getOwnerEntity(); diff --git a/mod/thewire/start.php b/mod/thewire/start.php index 8e3b5224a..1ba48263a 100644 --- a/mod/thewire/start.php +++ b/mod/thewire/start.php @@ -37,7 +37,7 @@ function thewire_init() {  	elgg_register_plugin_hook_handler('register', 'menu:entity', 'thewire_setup_entity_menu_items');  	// Extend system CSS with our own styles, which are defined in the thewire/css view -	elgg_extend_view('css', 'thewire/css'); +	elgg_extend_view('css/elgg', 'thewire/css');  	//extend views  	elgg_extend_view('activity/thewire', 'thewire/activity_view'); @@ -110,6 +110,7 @@ function thewire_page_handler($page) {  				set_input('guid', $page[1]);  			}  			include "$base_dir/view.php"; +			break;  		case "thread":  			if (isset($page[1])) {  | 
