First pass at standardizing pages.

git-svn-id: http://code.elgg.org/elgg/trunk@6716 36083f99-b078-4883-b0ff-0f9b5a30f544

1 files changed, 124 insertions, 128 deletions

diff --git a/mod/pages/actions/pages/edit.php b/mod/pages/actions/pages/edit.php
index ba6d0acdc..c4075a7bb 100644
--- a/mod/pages/actions/pages/edit.php
+++ b/mod/pages/actions/pages/edit.php
@@ -1,141 +1,137 @@
 <?php
-	/**
-	 * Elgg Pages
-	 * 
-	 * @package ElggPages
-	 * @license http://www.gnu.org/licenses/old-licenses/gpl-2.0.html GNU Public License version 2
-	 * @author Curverider Ltd
-	 * @copyright Curverider Ltd 2008-2010
-	 * @link http://elgg.com/
-	 */
-
-	// Load configuration
-	global $CONFIG;
-	
-	gatekeeper();
-	set_context('pages');
-	
-	//boolean to select correct add to river. It will be new or edit
-	$which_river = 'new';
-
-	// Get group fields
-	$input = array();
-	foreach($CONFIG->pages as $shortname => $valuetype) {
-		$input[$shortname] = get_input($shortname);
-		if ($shortname == 'title') {
-			$input[$shortname] = strip_tags($input[$shortname]);
-		}
-		if ($valuetype == 'tags')
-			$input[$shortname] = string_to_tag_array($input[$shortname]);
+/**
+ * Edit a page
+ *
+ * @package ElggPages
+ */
+
+// Load configuration
+global $CONFIG;
+
+gatekeeper();
+set_context('pages');
+
+//boolean to select correct add to river. It will be new or edit
+$which_river = 'new';
+
+// Get group fields
+$input = array();
+foreach($CONFIG->pages as $shortname => $valuetype) {
+	$input[$shortname] = get_input($shortname);
+	if ($shortname == 'title') {
+		$input[$shortname] = strip_tags($input[$shortname]);
 	}
-	
-	// Get parent
-	$parent_guid = (int)get_input('parent_guid', 0);
-	
-	// New or old?
-	$page = NULL;
-	$pages_guid = (int)get_input('pages_guid');
-	if ($pages_guid)
+	if ($valuetype == 'tags')
+		$input[$shortname] = string_to_tag_array($input[$shortname]);
+}
+
+// Get parent
+$parent_guid = (int)get_input('parent_guid', 0);
+
+// New or old?
+$page = NULL;
+$pages_guid = (int)get_input('pages_guid');
+if ($pages_guid)
+{
+	$page = get_entity($pages_guid);
+	if (!$page->canEdit())
+		$page = NULL; // if we can't edit it, go no further.
+
+	//select river boolean to edit
+	$which_river = 'edit';
+}
+else
+{
+	$page = new ElggObject();
+	if (!$parent_guid)
+		$page->subtype = 'page_top';
+	else
+		$page->subtype = 'page';
+
+	// New instance, so set container_guid
+	$container_guid = get_input('container_guid', $_SESSION['user']->getGUID());
+	$page->container_guid = $container_guid;
+
+	// cache data in session in case data from form does not validate
+	$_SESSION['page_description'] = $input['description'];
+	$_SESSION['page_tags'] = get_input('tags');
+	$_SESSION['page_read_access'] = (int)get_input('access_id');
+	$_SESSION['page_write_access'] = (int)get_input('write_access_id');
+}
+
+// Have we got it? Can we edit it?
+if ($page instanceof ElggObject)
+{
+	// Yes we have, and yes we can.
+
+	// Save fields - note we always save latest description as both description and annotation
+	if (sizeof($input) > 0)
 	{
-		$page = get_entity($pages_guid);
-		if (!$page->canEdit())
-			$page = NULL; // if we can't edit it, go no further.
-			
-		//select river boolean to edit
-		$which_river = 'edit';
+		foreach($input as $shortname => $value) {
+			if ((!$pages_guid) || (($pages_guid) && ($shortname != 'title')))
+				$page->$shortname = $value;
+		}
 	}
-	else
+
+
+	// Validate create
+	if (!$page->title)
 	{
-		$page = new ElggObject();
-		if (!$parent_guid)
-			$page->subtype = 'page_top';
-		else
-			$page->subtype = 'page';
-			
-		// New instance, so set container_guid
-		$container_guid = get_input('container_guid', $_SESSION['user']->getGUID());
-		$page->container_guid = $container_guid;
-
-		// cache data in session in case data from form does not validate
-		$_SESSION['page_description'] = $input['description'];
-		$_SESSION['page_tags'] = get_input('tags');
-		$_SESSION['page_read_access'] = (int)get_input('access_id');
-		$_SESSION['page_write_access'] = (int)get_input('write_access_id');
+		register_error(elgg_echo("pages:notitle"));
+
+		forward($_SERVER['HTTP_REFERER']);
+		exit;
 	}
-	
-	// Have we got it? Can we edit it?
-	if ($page instanceof ElggObject)
+
+	// Access ids
+	$page->access_id = (int)get_input('access_id', ACCESS_PRIVATE);
+
+	// Write access id
+	$page->write_access_id = (int)get_input('write_access_id', ACCESS_PRIVATE);
+
+	// Set parent
+	$page->parent_guid = $parent_guid;
+
+	// Ensure ultimate owner
+	$page->owner_guid = ($page->owner_guid ? $page->owner_guid : $_SESSION['user']->guid);
+
+	// finally save
+	if ($page->save())
 	{
-		// Yes we have, and yes we can.
-		
-		// Save fields - note we always save latest description as both description and annotation
-		if (sizeof($input) > 0)
-		{
-			foreach($input as $shortname => $value) {
-				if ((!$pages_guid) || (($pages_guid) && ($shortname != 'title')))
-					$page->$shortname = $value;
-			}
-		}
-		
-	
-		// Validate create
-		if (!$page->title)
-		{
-			register_error(elgg_echo("pages:notitle"));
-			
-			forward($_SERVER['HTTP_REFERER']);
-			exit;
-		}
-		
-		// Access ids
-		$page->access_id = (int)get_input('access_id', ACCESS_PRIVATE);
-		
-		// Write access id
-		$page->write_access_id = (int)get_input('write_access_id', ACCESS_PRIVATE);
-		
-		// Set parent
-		$page->parent_guid = $parent_guid;
-		
-		// Ensure ultimate owner
-		$page->owner_guid = ($page->owner_guid ? $page->owner_guid : $_SESSION['user']->guid); 
-		
-		// finally save
-		if ($page->save())
-		{
-			
-			// Now save description as an annotation
-			$page->annotate('page', $page->description, $page->access_id);
-
-			// clear cache
-			unset($_SESSION['page_description']);
-			unset($_SESSION['page_tags']);
-			unset($_SESSION['page_read_access']);
-			unset($_SESSION['page_write_access']);
-				
-		
-			system_message(elgg_echo("pages:saved"));
-			
-			//add to river
-			if($which_river == 'new')
-				add_to_river('river/object/page/create','create',$_SESSION['user']->guid,$page->guid);
-			else
-				add_to_river('river/object/page/update','update',$_SESSION['user']->guid,$page->guid);
-		
-			// Forward to the user's profile
-			forward($page->getUrl());
-			exit;
-		}
+
+		// Now save description as an annotation
+		$page->annotate('page', $page->description, $page->access_id);
+
+		// clear cache
+		unset($_SESSION['page_description']);
+		unset($_SESSION['page_tags']);
+		unset($_SESSION['page_read_access']);
+		unset($_SESSION['page_write_access']);
+
+
+		system_message(elgg_echo("pages:saved"));
+
+		//add to river
+		if($which_river == 'new')
+			add_to_river('river/object/page/create','create',$_SESSION['user']->guid,$page->guid);
 		else
-			register_error(elgg_echo('pages:notsaved'));
+			add_to_river('river/object/page/update','update',$_SESSION['user']->guid,$page->guid);
 
+		// Forward to the user's profile
+		forward($page->getUrl());
+		exit;
 	}
 	else
-	{
-		register_error(elgg_echo("pages:noaccess"));
-	}
-	
+		register_error(elgg_echo('pages:notsaved'));
+
+}
+else
+{
+	register_error(elgg_echo("pages:noaccess"));
+}
+
 
-	// Forward to the user's profile
-	forward($page->getUrl());
-	exit;
+// Forward to the user's profile
+forward($page->getUrl());
+exit;
 ?>