aboutsummaryrefslogtreecommitdiff
path: root/mod/pages/actions
diff options
context:
space:
mode:
authorbrettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>2010-07-16 15:25:25 +0000
committerbrettp <brettp@36083f99-b078-4883-b0ff-0f9b5a30f544>2010-07-16 15:25:25 +0000
commit01cbf6c0c700a97a2eba489638f917f08070095c (patch)
treedd7b4160d2e8d1619e6f0a040f1736635a3c36f1 /mod/pages/actions
parent29fba4847177d6e8be91f9d8899dc28b36b2af00 (diff)
downloadelgg-01cbf6c0c700a97a2eba489638f917f08070095c.tar.gz
elgg-01cbf6c0c700a97a2eba489638f917f08070095c.tar.bz2
More code cleanup to pages.
git-svn-id: http://code.elgg.org/elgg/trunk@6730 36083f99-b078-4883-b0ff-0f9b5a30f544
Diffstat (limited to 'mod/pages/actions')
-rw-r--r--mod/pages/actions/pages/edit.php66
1 files changed, 22 insertions, 44 deletions
diff --git a/mod/pages/actions/pages/edit.php b/mod/pages/actions/pages/edit.php
index c4075a7bb..004b81a25 100644
--- a/mod/pages/actions/pages/edit.php
+++ b/mod/pages/actions/pages/edit.php
@@ -31,22 +31,22 @@ $parent_guid = (int)get_input('parent_guid', 0);
// New or old?
$page = NULL;
$pages_guid = (int)get_input('pages_guid');
-if ($pages_guid)
-{
+
+if ($pages_guid) {
$page = get_entity($pages_guid);
- if (!$page->canEdit())
+ if (!$page->canEdit()) {
$page = NULL; // if we can't edit it, go no further.
+ }
//select river boolean to edit
$which_river = 'edit';
-}
-else
-{
+} else {
$page = new ElggObject();
- if (!$parent_guid)
+ if (!$parent_guid) {
$page->subtype = 'page_top';
- else
+ } else {
$page->subtype = 'page';
+ }
// New instance, so set container_guid
$container_guid = get_input('container_guid', $_SESSION['user']->getGUID());
@@ -60,44 +60,28 @@ else
}
// Have we got it? Can we edit it?
-if ($page instanceof ElggObject)
-{
- // Yes we have, and yes we can.
-
+if ($page instanceof ElggObject) {
// Save fields - note we always save latest description as both description and annotation
- if (sizeof($input) > 0)
- {
+ if (sizeof($input) > 0) {
foreach($input as $shortname => $value) {
- if ((!$pages_guid) || (($pages_guid) && ($shortname != 'title')))
+ if ((!$pages_guid) || (($pages_guid) && ($shortname != 'title'))) {
$page->$shortname = $value;
+ }
}
}
-
- // Validate create
- if (!$page->title)
- {
+ 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->parent_guid = $parent_guid;\
$page->owner_guid = ($page->owner_guid ? $page->owner_guid : $_SESSION['user']->guid);
- // finally save
- if ($page->save())
- {
+ if ($page->save()) {
// Now save description as an annotation
$page->annotate('page', $page->description, $page->access_id);
@@ -108,30 +92,24 @@ if ($page instanceof ElggObject)
unset($_SESSION['page_read_access']);
unset($_SESSION['page_write_access']);
-
system_message(elgg_echo("pages:saved"));
//add to river
- if($which_river == 'new')
+ if ($which_river == 'new') {
add_to_river('river/object/page/create','create',$_SESSION['user']->guid,$page->guid);
- else
+ } else {
add_to_river('river/object/page/update','update',$_SESSION['user']->guid,$page->guid);
+ }
// Forward to the user's profile
forward($page->getUrl());
- exit;
- }
- else
+ } else {
register_error(elgg_echo('pages:notsaved'));
+ }
-}
-else
-{
+} else {
register_error(elgg_echo("pages:noaccess"));
}
-
// Forward to the user's profile
-forward($page->getUrl());
-exit;
-?>
+forward($page->getUrl()); \ No newline at end of file