Major refactoring: transform user into object, define parameters used into each file, ...

git-svn-id: https://semanticscuttle.svn.sourceforge.net/svnroot/semanticscuttle/trunk@173 b3834d28-1941-0410-a4f8-b48e95affb8f

1 files changed, 49 insertions, 40 deletions

diff --git a/tagedit.php b/tagedit.php
index aca0607..cd248a5 100644
--- a/tagedit.php
+++ b/tagedit.php
@@ -1,61 +1,70 @@
 <?php
 /***************************************************************************
-Copyright (C) 2006 - 2007 Scuttle project
-http://sourceforge.net/projects/scuttle/
-http://scuttle.org/
+ Copyright (C) 2006 - 2007 Scuttle project
+ http://sourceforge.net/projects/scuttle/
+ http://scuttle.org/
 
-This program is free software; you can redistribute it and/or modify
-it under the terms of the GNU General Public License as published by
-the Free Software Foundation; either version 2 of the License, or
-(at your option) any later version.
+ This program is free software; you can redistribute it and/or modify
+ it under the terms of the GNU General Public License as published by
+ the Free Software Foundation; either version 2 of the License, or
+ (at your option) any later version.
 
-This program is distributed in the hope that it will be useful,
-but WITHOUT ANY WARRANTY; without even the implied warranty of
-MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-GNU General Public License for more details.
+ This program is distributed in the hope that it will be useful,
+ but WITHOUT ANY WARRANTY; without even the implied warranty of
+ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ GNU General Public License for more details.
 
-You should have received a copy of the GNU General Public License
-along with this program; if not, write to the Free Software
-Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
-***************************************************************************/
+ You should have received a copy of the GNU General Public License
+ along with this program; if not, write to the Free Software
+ Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
+ ***************************************************************************/
 
 require_once('header.inc.php');
+
+/* Service creation: only useful services are created */
 $tagservice       = & ServiceFactory :: getServiceInstance('TagService');
 $templateservice  = & ServiceFactory :: getServiceInstance('TemplateService');
 $userservice      = & ServiceFactory :: getServiceInstance('UserService');
 
+/* Managing all possible inputs */
+isset($_POST['confirm']) ? define('POST_CONFIRM', $_POST['confirm']): define('POST_CONFIRM', '');
+isset($_POST['cancel']) ? define('POST_CANCEL', $_POST['cancel']): define('POST_CANCEL', '');
+isset($_POST['description']) ? define('POST_DESCRIPTION', $_POST['description']): define('POST_DESCRIPTION', '');
+isset($_POST['referrer']) ? define('POST_REFERRER', $_POST['referrer']): define('POST_REFERRER', '');
+
+/* Managing current logged user */
+$currentUser = $userservice->getCurrentObjectUser();
+
+/* Managing path info */
 list ($url, $tag) = explode('/', $_SERVER['PATH_INFO']);
-$template   = 'tagedit.tpl';
 
-$logged_on_user = $userservice->getCurrentUser();
+$template   = 'tagedit.tpl';
 
 //permissions
-if($logged_on_user == null) {
-    $tplVars['error'] = T_('Permission denied.');
-    $templateservice->loadTemplate('error.500.tpl', $tplVars);
-    exit();
+if(!$userservice->isLoggedOn()) {
+	$tplVars['error'] = T_('Permission denied.');
+	$templateservice->loadTemplate('error.500.tpl', $tplVars);
+	exit();
 }
 
-if ($_POST['confirm']) {
-
-   if ( strlen($tag)>0 &&
-	$tagservice->updateDescription($tag, $logged_on_user['uId'], $_POST['description'])
-   ) {
-      $tplVars['msg'] = T_('Tag description updated');
-      header('Location: '. $_POST['referrer']);
-   } else {
-      $tplVars['error'] = T_('Failed to update the tag description');
-      $template         = 'error.500.tpl';
-   }
-} elseif ($_POST['cancel']) {
-    $logged_on_user = $userservice->getCurrentUser();
-    header('Location: '. $_POST['referrer']);
+if (POST_CONFIRM) {
+	if ( strlen($tag)>0 &&
+	$tagservice->updateDescription($tag, $currentUser->getId(), POST_DESCRIPTION)
+	) {
+		$tplVars['msg'] = T_('Tag description updated');
+		header('Location: '. POST_REFERRER);
+	} else {
+		$tplVars['error'] = T_('Failed to update the tag description');
+		$template         = 'error.500.tpl';
+	}
+} elseif (POST_CANCEL) {
+	header('Location: '. POST_REFERRER);
 } else {
-   $tplVars['subtitle']    = T_('Edit Tag Description') .': '. $tag;
-   $tplVars['formaction']  = $_SERVER['SCRIPT_NAME'] .'/'. $tag;
-   $tplVars['referrer']    = $_SERVER['HTTP_REFERER'];
-   $tplVars['tag']         = $tag;
-   $tplVars['description'] = $tagservice->getDescription($tag, $logged_on_user['uId']);
+	$tplVars['subtitle']    = T_('Edit Tag Description') .': '. $tag;
+	$tplVars['formaction']  = $_SERVER['SCRIPT_NAME'] .'/'. $tag;
+	$tplVars['referrer']    = $_SERVER['HTTP_REFERER'];
+	$tplVars['tag']         = $tag;
+	$tplVars['description'] = $tagservice->getDescription($tag, $currentUser->getId());
 }
 $templateservice->loadTemplate($template, $tplVars);
 ?>