From c1411ffc81908882edaace315c46e500419d2aba Mon Sep 17 00:00:00 2001
From: cash <cash@36083f99-b078-4883-b0ff-0f9b5a30f544>
Date: Sat, 26 Feb 2011 14:23:32 +0000
Subject: Refs #2982 adding and deleting friend collections works - editing
 does not

git-svn-id: http://code.elgg.org/elgg/trunk@8490 36083f99-b078-4883-b0ff-0f9b5a30f544
---
 actions/friends/addcollection.php                | 36 ------------
 actions/friends/collections/add.php              | 36 ++++++++++++
 actions/friends/collections/delete.php           | 36 ++++++++++++
 actions/friends/collections/edit.php             | 13 +++++
 actions/friends/deletecollection.php             | 36 ------------
 actions/friends/editcollection.php               | 13 -----
 engine/lib/users.php                             | 14 +++--
 pages/friends/add.php                            | 23 --------
 pages/friends/collections.php                    | 23 --------
 pages/friends/collections/add.php                | 26 +++++++++
 pages/friends/collections/pickercallback.php     | 71 ++++++++++++++++++++++++
 pages/friends/collections/view.php               | 23 ++++++++
 pages/friends/edit.php                           | 30 ----------
 pages/friends/pickercallback.php                 | 55 ------------------
 views/default/core/friends/collection.php        | 32 +++++++----
 views/default/core/friends/collections.php       | 13 ++---
 views/default/core/friends/collectiontabs.php    | 31 ++++++-----
 views/default/forms/friends/collectionfields.php | 15 -----
 views/default/forms/friends/collections/edit.php | 53 ++++++++++++++++++
 views/default/forms/friends/edit.php             | 53 ------------------
 20 files changed, 311 insertions(+), 321 deletions(-)
 delete mode 100644 actions/friends/addcollection.php
 create mode 100644 actions/friends/collections/add.php
 create mode 100644 actions/friends/collections/delete.php
 create mode 100644 actions/friends/collections/edit.php
 delete mode 100644 actions/friends/deletecollection.php
 delete mode 100644 actions/friends/editcollection.php
 delete mode 100644 pages/friends/add.php
 delete mode 100644 pages/friends/collections.php
 create mode 100644 pages/friends/collections/add.php
 create mode 100644 pages/friends/collections/pickercallback.php
 create mode 100644 pages/friends/collections/view.php
 delete mode 100644 pages/friends/edit.php
 delete mode 100644 pages/friends/pickercallback.php
 delete mode 100644 views/default/forms/friends/collectionfields.php
 create mode 100644 views/default/forms/friends/collections/edit.php
 delete mode 100644 views/default/forms/friends/edit.php

diff --git a/actions/friends/addcollection.php b/actions/friends/addcollection.php
deleted file mode 100644
index eac8bced8..000000000
--- a/actions/friends/addcollection.php
+++ /dev/null
@@ -1,36 +0,0 @@
-<?php
-/**
- * Elgg collection add page
- *
- * @package Elgg.Core
- * @subpackage Friends.Collections
- */
-
-$collection_name = get_input('collection_name');
-$friends = get_input('friends_collection');
-
-//first check to make sure that a collection name has been set and create the new colection
-if ($collection_name) {
-
-	//create the collection
-	$create_collection = create_access_collection($collection_name, elgg_get_logged_in_user_guid());
-
-	//if the collection was created and the user passed some friends from the form, add them
-	if ($create_collection && (!empty($friends))) {
-		//add friends to the collection
-		foreach ($friends as $friend) {
-			add_user_to_access_collection($friend, $create_collection);
-		}
-	}
-
-	// Success message
-	system_message(elgg_echo("friends:collectionadded"));
-	// Forward to the collections page
-	forward("pg/collections/" . elgg_get_logged_in_user_entity()->username);
-
-} else {
-	register_error(elgg_echo("friends:nocollectionname"));
-
-	// Forward to the add collection page
-	forward("pg/collections/add");
-}
diff --git a/actions/friends/collections/add.php b/actions/friends/collections/add.php
new file mode 100644
index 000000000..eac8bced8
--- /dev/null
+++ b/actions/friends/collections/add.php
@@ -0,0 +1,36 @@
+<?php
+/**
+ * Elgg collection add page
+ *
+ * @package Elgg.Core
+ * @subpackage Friends.Collections
+ */
+
+$collection_name = get_input('collection_name');
+$friends = get_input('friends_collection');
+
+//first check to make sure that a collection name has been set and create the new colection
+if ($collection_name) {
+
+	//create the collection
+	$create_collection = create_access_collection($collection_name, elgg_get_logged_in_user_guid());
+
+	//if the collection was created and the user passed some friends from the form, add them
+	if ($create_collection && (!empty($friends))) {
+		//add friends to the collection
+		foreach ($friends as $friend) {
+			add_user_to_access_collection($friend, $create_collection);
+		}
+	}
+
+	// Success message
+	system_message(elgg_echo("friends:collectionadded"));
+	// Forward to the collections page
+	forward("pg/collections/" . elgg_get_logged_in_user_entity()->username);
+
+} else {
+	register_error(elgg_echo("friends:nocollectionname"));
+
+	// Forward to the add collection page
+	forward("pg/collections/add");
+}
diff --git a/actions/friends/collections/delete.php b/actions/friends/collections/delete.php
new file mode 100644
index 000000000..f0ceaf023
--- /dev/null
+++ b/actions/friends/collections/delete.php
@@ -0,0 +1,36 @@
+<?php
+/**
+ * Elgg friends: delete collection action
+ *
+ * @package Elgg.Core
+ * @subpackage Friends.Collections
+ */
+
+$collection_id = (int) get_input('collection');
+
+// Check to see that the access collection exist and grab its owner
+$get_collection = get_access_collection($collection_id);
+
+if ($get_collection) {
+
+	if ($get_collection->owner_guid == elgg_get_logged_in_user_guid()) {
+
+		$delete_collection = delete_access_collection($collection_id);
+
+		// Success message
+		if ($delete_collection) {
+			system_message(elgg_echo("friends:collectiondeleted"));
+		} else {
+			register_error(elgg_echo("friends:collectiondeletefailed"));
+		}
+	} else {
+		// Failure message
+		register_error(elgg_echo("friends:collectiondeletefailed"));
+	}
+} else {
+	// Failure message
+	register_error(elgg_echo("friends:collectiondeletefailed"));
+}
+
+// Forward to the collections page
+forward("pg/collections/" . elgg_get_logged_in_user_entity()->username);
diff --git a/actions/friends/collections/edit.php b/actions/friends/collections/edit.php
new file mode 100644
index 000000000..8fd1eae18
--- /dev/null
+++ b/actions/friends/collections/edit.php
@@ -0,0 +1,13 @@
+<?php
+/**
+ * Elgg collection add page
+ *
+ * @package Elgg.Core
+ * @subpackage Friends.Collections
+ */
+
+$collection_id = get_input('collection_id');
+$friends = get_input('friend');
+
+//chech the collection exists and the current user owners it
+update_access_collection($collection_id, $friends);
\ No newline at end of file
diff --git a/actions/friends/deletecollection.php b/actions/friends/deletecollection.php
deleted file mode 100644
index f0ceaf023..000000000
--- a/actions/friends/deletecollection.php
+++ /dev/null
@@ -1,36 +0,0 @@
-<?php
-/**
- * Elgg friends: delete collection action
- *
- * @package Elgg.Core
- * @subpackage Friends.Collections
- */
-
-$collection_id = (int) get_input('collection');
-
-// Check to see that the access collection exist and grab its owner
-$get_collection = get_access_collection($collection_id);
-
-if ($get_collection) {
-
-	if ($get_collection->owner_guid == elgg_get_logged_in_user_guid()) {
-
-		$delete_collection = delete_access_collection($collection_id);
-
-		// Success message
-		if ($delete_collection) {
-			system_message(elgg_echo("friends:collectiondeleted"));
-		} else {
-			register_error(elgg_echo("friends:collectiondeletefailed"));
-		}
-	} else {
-		// Failure message
-		register_error(elgg_echo("friends:collectiondeletefailed"));
-	}
-} else {
-	// Failure message
-	register_error(elgg_echo("friends:collectiondeletefailed"));
-}
-
-// Forward to the collections page
-forward("pg/collections/" . elgg_get_logged_in_user_entity()->username);
diff --git a/actions/friends/editcollection.php b/actions/friends/editcollection.php
deleted file mode 100644
index 8fd1eae18..000000000
--- a/actions/friends/editcollection.php
+++ /dev/null
@@ -1,13 +0,0 @@
-<?php
-/**
- * Elgg collection add page
- *
- * @package Elgg.Core
- * @subpackage Friends.Collections
- */
-
-$collection_id = get_input('collection_id');
-$friends = get_input('friend');
-
-//chech the collection exists and the current user owners it
-update_access_collection($collection_id, $friends);
\ No newline at end of file
diff --git a/engine/lib/users.php b/engine/lib/users.php
index fd68ef276..1b9a6128b 100644
--- a/engine/lib/users.php
+++ b/engine/lib/users.php
@@ -1145,18 +1145,20 @@ function friends_of_page_handler($page_elements) {
  */
 function collections_page_handler($page_elements) {
 	elgg_set_context('friends');
+	$base = elgg_get_config('path');
 	if (isset($page_elements[0])) {
 		if ($page_elements[0] == "add") {
 			set_page_owner(elgg_get_logged_in_user_guid());
 			collections_submenu_items();
-			require_once(dirname(dirname(dirname(__FILE__))) . "/pages/friends/add.php");
+			require_once "{$base}pages/friends/collections/add.php";
 		} else {
-			if ($user = get_user_by_username($page_elements[0])) {
+			$user = get_user_by_username($page_elements[0]);
+			if ($user) {
 				set_page_owner($user->getGUID());
 				if (elgg_get_logged_in_user_guid() == elgg_get_page_owner_guid()) {
 					collections_submenu_items();
 				}
-				require_once(dirname(dirname(dirname(__FILE__))) . "/pages/friends/collections.php");
+				require_once "{$base}pages/friends/collections/view.php";
 			}
 		}
 	}
@@ -1566,9 +1568,9 @@ function users_init() {
 	elgg_register_action('avatar/crop');
 	elgg_register_action('profile/edit');
 
-	elgg_register_action('friends/addcollection');
-	elgg_register_action('friends/deletecollection');
-	elgg_register_action('friends/editcollection');
+	elgg_register_action('friends/collections/add');
+	elgg_register_action('friends/collections/delete');
+	elgg_register_action('friends/collections/edit');
 
 	elgg_register_plugin_hook_handler('entity:icon:url', 'user', 'user_avatar_hook');
 
diff --git a/pages/friends/add.php b/pages/friends/add.php
deleted file mode 100644
index 96da26ae1..000000000
--- a/pages/friends/add.php
+++ /dev/null
@@ -1,23 +0,0 @@
-<?php
-/**
- * Elgg add a collection of friends
- *
- * @package Elgg
- * @subpackage Core
- */
-
-// You need to be logged in for this one
-gatekeeper();
-
-$title = elgg_echo('friends:collections:add');
-
-$content = elgg_view_title($title);
-
-$content .= elgg_view_form('friends/edit', array(), array(
-	'friends' => get_user_friends(elgg_get_logged_in_user_guid(), "", 9999)
-	)
-);
-
-$body = elgg_view_layout('one_sidebar', array('content' => $content));
-
-echo elgg_view_page(elgg_echo('friends:collections:add'), $body);
diff --git a/pages/friends/collections.php b/pages/friends/collections.php
deleted file mode 100644
index fa7527f4d..000000000
--- a/pages/friends/collections.php
+++ /dev/null
@@ -1,23 +0,0 @@
-<?php
-/**
- * Elgg collections of friends
- *
- * @package Elgg
- * @subpackage Core
- */
-
-// You need to be logged in for this one
-gatekeeper();
-
-$title = elgg_echo('friends:collections');
-
-$content = elgg_view_access_collections(elgg_get_logged_in_user_guid());
-
-$body = elgg_view_layout('content', array(
-	'filter' => false,
-	'content' => $content,
-	'title' => $title,
-	'context' => 'collections',
-));
-
-echo elgg_view_page($title, $body);
diff --git a/pages/friends/collections/add.php b/pages/friends/collections/add.php
new file mode 100644
index 000000000..51425db37
--- /dev/null
+++ b/pages/friends/collections/add.php
@@ -0,0 +1,26 @@
+<?php
+/**
+ * Elgg add a collection of friends
+ *
+ * @package Elgg
+ * @subpackage Core
+ */
+
+// You need to be logged in for this one
+gatekeeper();
+
+$title = elgg_echo('friends:collections:add');
+
+$content = elgg_view_title($title);
+
+$form_body = elgg_view('forms/friends/collections/edit', array(
+	'friends' => get_user_friends(elgg_get_logged_in_user_guid(), "", 9999)
+));
+$content .= elgg_view('input/form', array(
+	'action' => 'action/friends/collections/add',
+	'body' => $form_body,
+));
+
+$body = elgg_view_layout('one_sidebar', array('content' => $content));
+
+echo elgg_view_page(elgg_echo('friends:collections:add'), $body);
diff --git a/pages/friends/collections/pickercallback.php b/pages/friends/collections/pickercallback.php
new file mode 100644
index 000000000..8da44d8c0
--- /dev/null
+++ b/pages/friends/collections/pickercallback.php
@@ -0,0 +1,71 @@
+<?php
+/**
+ * Elgg friends picker callback
+ *
+ * @package Elgg
+ * @subpackage Core
+ */
+
+// Load Elgg engine
+require_once(dirname(dirname(dirname(dirname(__FILE__)))) . "/engine/start.php");
+
+$site_url = elgg_get_site_url();
+
+// Get callback type (list or picker)
+$type = get_input('type', 'picker');
+
+$collection = (int) get_input('collection', 0);
+$members = get_members_of_access_collection($collection, true);
+if (!$members) {
+	$members = array();
+}
+
+$friendspicker = (int) get_input('friendspicker', 0);
+
+// Get page owner (bomb out if there isn't one)
+$pageowner = elgg_get_page_owner_entity();
+if (!$pageowner) {
+	forward();
+	exit;
+}
+
+// Depending on the view type, launch a different view
+switch($type) {
+	case 'list':
+		$js_segment = elgg_view('core/friends/tablelistcountupdate', array(
+			'friendspicker' => $friendspicker,
+			'count' => sizeof($members),
+		));
+		$content = elgg_view('core/friends/tablelist', array(
+			'entities' => $members,
+			'content' => $js_segment,
+		));
+		break;
+	default:
+		$friends = $pageowner->getFriends('', 9999);
+		$params = array(
+			'collection' => get_access_collection($collection),
+			'friends' => $friends,
+			'friendspicker' => $friendspicker,
+		);
+		$content = elgg_view_form('friends/collections/edit', array(), $params);
+		/*
+		$form_body = elgg_view('input/hidden', array(
+			'name' => 'collection_id',
+			'value' => get_access_collection($collection)->id,
+		));
+		$content = elgg_view('input/friendspicker', array(
+			'entities' => $friends,
+			'value' => $members,
+			'callback' => true,
+			'friendspicker' => $friendspicker,
+			'formcontents' => $form_body,
+			'formtarget' => $site_url . 'action/friends/collections/edit',
+		));
+		 * 
+		 */
+		break;
+}
+
+// Output the content
+echo $content;
\ No newline at end of file
diff --git a/pages/friends/collections/view.php b/pages/friends/collections/view.php
new file mode 100644
index 000000000..fa7527f4d
--- /dev/null
+++ b/pages/friends/collections/view.php
@@ -0,0 +1,23 @@
+<?php
+/**
+ * Elgg collections of friends
+ *
+ * @package Elgg
+ * @subpackage Core
+ */
+
+// You need to be logged in for this one
+gatekeeper();
+
+$title = elgg_echo('friends:collections');
+
+$content = elgg_view_access_collections(elgg_get_logged_in_user_guid());
+
+$body = elgg_view_layout('content', array(
+	'filter' => false,
+	'content' => $content,
+	'title' => $title,
+	'context' => 'collections',
+));
+
+echo elgg_view_page($title, $body);
diff --git a/pages/friends/edit.php b/pages/friends/edit.php
deleted file mode 100644
index 665929319..000000000
--- a/pages/friends/edit.php
+++ /dev/null
@@ -1,30 +0,0 @@
-<?php
-/**
- * Elgg add a collection of friends
- *
- * @package Elgg
- * @subpackage Core
- */
-
-// You need to be logged in for this one
-gatekeeper();
-
-$title = elgg_echo('friends:collectionedit');
-
-$content = elgg_view_title($title);
-
-//grab the collection id passed to the edit form
-$collection_id = get_input('collection');
-
-//get the full collection
-$collection = get_access_collection($collection_id);
-
-//get all members of the collection
-$collection_members = get_members_of_access_collection($collection_id);
-
-$content .= elgg_view_form('friends/edit', array(), array('collection' => $collection,
-	'collection_members' => $collection_members));
-
-$body = elgg_view_layout('one_sidebar', array('content' => $content));
-
-echo elgg_view_page($title, $body);
\ No newline at end of file
diff --git a/pages/friends/pickercallback.php b/pages/friends/pickercallback.php
deleted file mode 100644
index f613f1ae1..000000000
--- a/pages/friends/pickercallback.php
+++ /dev/null
@@ -1,55 +0,0 @@
-<?php
-/**
- * Elgg friends picker callback
- *
- * @package Elgg
- * @subpackage Core
- */
-
-// Load Elgg engine
-require_once(dirname(dirname(dirname(__FILE__))) . "/engine/start.php");
-
-$site_url = elgg_get_site_url();
-
-// Get callback type (list or picker)
-$type = get_input('type', 'picker');
-
-$collection = (int) get_input('collection', 0);
-$members = get_members_of_access_collection($collection, true);
-if (!$members) {
-	$members = array();
-}
-
-$friendspicker = (int) get_input('friendspicker', 0);
-
-// Get page owner (bomb out if there isn't one)
-$pageowner = elgg_get_page_owner_entity();
-if (!$pageowner) {
-	forward();
-	exit;
-}
-
-// Depending on the view type, launch a different view
-switch($type) {
-	case 'list':
-		$js_segment = elgg_view('core/friends/tablelistcountupdate',
-			array('friendspicker' => $friendspicker, 'count' => sizeof($members)));
-		$content = elgg_view('core/friends/tablelist',
-			array('entities' => $members, 'content' => $js_segment));
-		break;
-	default:
-		$friends = $pageowner->getFriends('', 9999);
-		$content = elgg_view('input/friendspicker', array(
-			'entities' => $friends,
-			'value' => $members,
-			'callback' => true,
-			'friendspicker' => $friendspicker,
-			'formcontents' => elgg_view('forms/friends/collectionfields',
-				array('collection' => get_access_collection($collection))),
-			'formtarget' => $site_url . 'action/friends/editcollection',
-		));
-		break;
-}
-
-// Output the content
-echo $content;
\ No newline at end of file
diff --git a/views/default/core/friends/collection.php b/views/default/core/friends/collection.php
index 89c44a2b3..95f5944e7 100644
--- a/views/default/core/friends/collection.php
+++ b/views/default/core/friends/collection.php
@@ -1,13 +1,10 @@
 <?php
 /**
- * Elgg friends collection
- * Lists one of a user's friends collections
+ * View a friends collection
  *
  * @package Elgg
  * @subpackage Core
  *
- * @see collections.php
- *
  * @uses $vars['collection'] The individual friends collection
  */
 
@@ -25,8 +22,10 @@ echo "<li><h2>";
 if ($coll->owner_guid == elgg_get_logged_in_user_guid()) {
 	echo "<div class=\"friends_collections_controls\">";
 	echo elgg_view('output/confirmlink', array(
-			'href' => 'action/friends/deletecollection?collection=' . $coll->id,
-			'class' => 'delete_collection'
+			'href' => 'action/friends/collections/delete?collection=' . $coll->id,
+			'class' => 'delete_collection',
+			'text' => elgg_view_icon('delete'),
+			'text_encode' => false,
 		));
 	echo "</div>";
 }
@@ -34,16 +33,27 @@ echo $coll->name;
 echo " (<span id=\"friends_membership_count{$vars['friendspicker']}\">{$count}</span>) </h2>";
 
 // individual collection panels
-if ($friends = $vars['collection']->entities) {
-	$content = elgg_view('core/friends/collectiontabs', array('owner' => elgg_get_logged_in_user_entity(), 'collection' => $vars['collection'], 'friendspicker' => $vars['friendspicker']));
+$friends = $vars['collection']->entities;
+if ($friends) {
+	$content = elgg_view('core/friends/collectiontabs', array(
+		'owner' => elgg_get_logged_in_user_entity(),
+		'collection' => $vars['collection'],
+		'friendspicker' => $vars['friendspicker'],
+	));
 
-	echo elgg_view('input/friendspicker', array('entities' => $friends, 'value' => $members, 'content' => $content, 'replacement' => '', 'friendspicker' => $vars['friendspicker']));
-	?>
+	echo elgg_view('input/friendspicker', array(
+		'entities' => $friends,
+		'value' => $members,
+		'content' => $content,
+		'replacement' => '',
+		'friendspicker' => $vars['friendspicker'],
+	));
+?>
 <?php //@todo JS 1.8: no ?>
 	<script type="text/javascript">
 	$(document).ready(function () {
 
-			$('#friends-picker_placeholder<?php echo $vars['friendspicker']; ?>').load('<?php echo elgg_get_site_url(); ?>pages/friends/pickercallback.php?username=<?php echo elgg_get_logged_in_user_entity()->username; ?>&type=list&collection=<?php echo $vars['collection']->id; ?>');
+			$('#friends-picker_placeholder<?php echo $vars['friendspicker']; ?>').load('<?php echo elgg_get_site_url(); ?>pages/friends/collections/pickercallback.php?username=<?php echo elgg_get_logged_in_user_entity()->username; ?>&type=list&collection=<?php echo $vars['collection']->id; ?>');
 
 	});
 	</script>
diff --git a/views/default/core/friends/collections.php b/views/default/core/friends/collections.php
index 5a052b121..5c1c824e7 100644
--- a/views/default/core/friends/collections.php
+++ b/views/default/core/friends/collections.php
@@ -9,17 +9,16 @@
  * @uses $vars['collections'] The array of friends collections
  */
 
-if (!isset($friendspicker)) {
-	$friendspicker = 0;
-}
-
-
 if (is_array($vars['collections']) && sizeof($vars['collections'])) {
 	echo "<ul id=\"friends_collections_accordian\">";
 
-	foreach($vars['collections'] as $collection) {
+	$friendspicker = 0;
+	foreach ($vars['collections'] as $collection) {
 		$friendspicker++;
-		echo elgg_view('core/friends/collection', array('collection' => $collection, 'friendspicker' => $friendspicker));
+		echo elgg_view('core/friends/collection', array(
+			'collection' => $collection,
+			'friendspicker' => $friendspicker,
+		));
 	}
 
 	echo "</ul>";
diff --git a/views/default/core/friends/collectiontabs.php b/views/default/core/friends/collectiontabs.php
index 168d2cdfb..2fafe6a8c 100644
--- a/views/default/core/friends/collectiontabs.php
+++ b/views/default/core/friends/collectiontabs.php
@@ -16,38 +16,43 @@ $ownerid = $vars['owner']->getGUID();
 
 ?>
 
-<div id="elgg-tabs">
-<ul>
-<li class="selected"><a href="#" class="collectionmembers<?php echo $friendspicker; ?>"><?php echo elgg_echo('friends:collections:members'); ?></a></li>
-
-<li><a href="#" class="editmembers<?php echo $friendspicker; ?>"><?php echo elgg_echo('friends:collections:edit'); ?></a></li>
-	
+<ul class="elgg-tabs">
+	<li class="elgg-state-selected">
+		<a href="#" class="collectionmembers<?php echo $friendspicker; ?>">
+			<?php echo elgg_echo('friends:collections:members'); ?>
+		</a>
+	</li>
+	<li>
+		<a href="#" class="editmembers<?php echo $friendspicker; ?>">
+			<?php echo elgg_echo('friends:collections:edit'); ?>
+		</a>
+	</li>
 </ul>
-</div>
+
 <?php //@todo JS 1.8: no ?>
 <script type="text/javascript">
 $(document).ready(function () {
 
 	$('a.collectionmembers<?php echo $friendspicker; ?>').click(function () {
 		// load collection members pane
-		$('#friends-picker_placeholder<?php echo $friendspicker; ?>').load('<?php echo elgg_get_site_url(); ?>pages/friends/pickercallback.php?username=<?php echo elgg_get_logged_in_user_entity()->username; ?>&type=list&collection=<?php echo $collectionid; ?>&friendspicker=<?php echo $friendspicker; ?>');
+		$('#friends-picker_placeholder<?php echo $friendspicker; ?>').load('<?php echo elgg_get_site_url(); ?>pages/friends/collections/pickercallback.php?username=<?php echo elgg_get_logged_in_user_entity()->username; ?>&type=list&collection=<?php echo $collectionid; ?>&friendspicker=<?php echo $friendspicker; ?>');
 
 		// remove selected state from previous tab
-		$(this).parent().parent().find("li.selected").removeClass("selected");
+		$(this).parent().parent().find("li.elgg-state-selected").removeClass("elgg-state-selected");
 		// add selected class to current tab
-		$(this).parent().addClass("selected");
+		$(this).parent().addClass("elgg-state-selected");
 
 		return false;
 	});
 
 	$('a.editmembers<?php echo $friendspicker; ?>').click(function () {
 		// load friends picker pane
-		$('#friends-picker_placeholder<?php echo $friendspicker; ?>').load('<?php echo elgg_get_site_url(); ?>pages/friends/pickercallback.php?username=<?php echo elgg_get_logged_in_user_entity()->username; ?>&type=picker&collection=<?php echo $collectionid; ?>&friendspicker=<?php echo $friendspicker; ?>');
+		$('#friends-picker_placeholder<?php echo $friendspicker; ?>').load('<?php echo elgg_get_site_url(); ?>pages/friends/collections/pickercallback.php?username=<?php echo elgg_get_logged_in_user_entity()->username; ?>&type=picker&collection=<?php echo $collectionid; ?>&friendspicker=<?php echo $friendspicker; ?>');
 
 		// remove selected state from previous tab
-		$(this).parent().parent().find("li.selected").removeClass("selected");
+		$(this).parent().parent().find("li.elgg-state-selected").removeClass("elgg-state-selected");
 		// add selected class to current tab
-		$(this).parent().addClass("selected");
+		$(this).parent().addClass("elgg-state-selected");
 
 		return false;
 	});
diff --git a/views/default/forms/friends/collectionfields.php b/views/default/forms/friends/collectionfields.php
deleted file mode 100644
index 35db3a962..000000000
--- a/views/default/forms/friends/collectionfields.php
+++ /dev/null
@@ -1,15 +0,0 @@
-<?php
-/**
- * Elgg friend collections required hidden fields for js friends picker form
- *
- * @package Elgg
- * @subpackage Core
- */
-
-if (isset($vars['collection'])) {
-?>
-
-	<input type="hidden" name="collection_id" value="<?php echo $vars['collection']->id; ?>" />
-
-<?php
-}
\ No newline at end of file
diff --git a/views/default/forms/friends/collections/edit.php b/views/default/forms/friends/collections/edit.php
new file mode 100644
index 000000000..975307085
--- /dev/null
+++ b/views/default/forms/friends/collections/edit.php
@@ -0,0 +1,53 @@
+<?php
+/**
+ * Form body for editing or adding a friend collection
+ *
+ * @package Elgg
+ * @subpackage Core
+ *
+ * @uses $vars['collection'] Optionally, the collection to edit
+ */
+
+// Set title, form destination
+if (isset($vars['collection'])) {
+	$title = $vars['collection']->name;
+	$highlight = 'default';
+} else  {
+	$title = "";
+	$highlight = 'all';
+}
+
+echo "<div><label>" . elgg_echo("friends:collectionname") . "<br/>";
+echo elgg_view("input/text", array(
+		"name" => "collection_name",
+		"value" => $title,
+	));
+echo "</label></div>";
+
+echo "<div>";
+if ($vars['collection_members']) {
+	echo elgg_echo("friends:collectionfriends") . "<br />";
+	foreach ($vars['collection_members'] as $mem) {
+		echo elgg_view_entity_icon($mem, 'tiny');
+		echo $mem->name;
+	}
+}
+echo "</div>";
+
+echo "<div><label>" . elgg_echo("friends:addfriends") . "</label>";
+echo elgg_view('input/friendspicker', array(
+	'entities' => $vars['friends'],
+	'name' => 'friends_collection',
+	'highlight' => $highlight,
+));
+echo "</div>";
+
+echo "<div>";
+if (isset($vars['collection'])) {
+	echo elgg_view('input/hidden', array(
+		'name' => 'collection_id',
+		'value' => $vars['collection']->id,
+	));
+}
+echo elgg_view('input/submit', array('name' => 'submit', 'value' => elgg_echo('save')));
+echo "</div>";
diff --git a/views/default/forms/friends/edit.php b/views/default/forms/friends/edit.php
deleted file mode 100644
index 902113c9c..000000000
--- a/views/default/forms/friends/edit.php
+++ /dev/null
@@ -1,53 +0,0 @@
-<?php
-/**
- * Elgg friend collections add/edit
- *
- * @package Elgg
- * @subpackage Core
- *
- * @uses $vars['object'] Optionally, the collection edit
- */
-
-// var_export($vars['collection'][0]->id);
-
-// Set title, form destination
-if (isset($vars['collection'])) {
-	$action = "friends/editcollection";
-	$title = $vars['collection'][0]->name;
-	$highlight = 'default';
-} else  {
-	$action = "friends/addcollection";
-	$title = "";
-	$highlight = 'all';
-}
-
-
-$form_body = "<div class='contentWrapper'><div><label>" . elgg_echo("friends:collectionname") . "<br />" .
-	elgg_view("input/text", array(
-		"name" => "collection_name",
-		"value" => $title,
-	)) . "</label></div>";
-
-$form_body .= "<div>";
-
-if($vars['collection_members']){
-	$form_body .= elgg_echo("friends:collectionfriends") . "<br />";
-	foreach($vars['collection_members'] as $mem){
-		$form_body .= elgg_view_entity_icon($mem, 'tiny');
-		$form_body .= $mem->name;
-	}
-}
-
-$form_body .= "</div>";
-
-$form_body .= "<div><label>" . elgg_echo("friends:addfriends") . "</label>".
-			elgg_view('input/friendspicker',array('entities' => $vars['friends'], 'name' => 'friends_collection', 'highlight' => $highlight)) . "</div>";
-
-$form_body .= "<div>";
-if (isset($vars['collection'])) {
-	$form_body .= elgg_view('input/hidden', array('name' => 'collection_id', 'value' => "{$vars['collection'][0]->id}"));
-}
-$form_body .= elgg_view('input/submit', array('name' => 'submit', 'value' => elgg_echo('save')));
-$form_body .= "</div></div>";
-
-echo $form_body;
\ No newline at end of file
-- 
cgit v1.2.3