]> gitweb.fluxo.info Git - semanticscuttle.git/commitdiff
Minor Refactoring: add getIdFromUser()
authormensonge <mensonge@b3834d28-1941-0410-a4f8-b48e95affb8f>
Thu, 15 Jan 2009 18:05:08 +0000 (18:05 +0000)
committermensonge <mensonge@b3834d28-1941-0410-a4f8-b48e95affb8f>
Thu, 15 Jan 2009 18:05:08 +0000 (18:05 +0000)
git-svn-id: https://semanticscuttle.svn.sourceforge.net/svnroot/semanticscuttle/trunk@238 b3834d28-1941-0410-a4f8-b48e95affb8f

alltags.php
password.php
populartags.php
profile.php
search.php
services/userservice.php
watch.php
watchlist.php

index dc84cf3e5ca9c046f096dfb64f9827716fd705ea..45b13ab95f4219dd6e2a087c5ef8a37f7a44d761 100644 (file)
@@ -1,23 +1,23 @@
 <?php
 /***************************************************************************
-Copyright (C) 2004 - 2006 Scuttle project
-http://sourceforge.net/projects/scuttle/
-http://scuttle.org/
+ Copyright (C) 2004 - 2006 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');
 
@@ -29,51 +29,47 @@ $cacheservice =& ServiceFactory::getServiceInstance('CacheService');
 
 list($url, $user) = explode('/', $_SERVER['PATH_INFO']);
 if (!$user) {
-    header('Location: '. createURL('populartags'));
-    exit;
+       header('Location: '. createURL('populartags'));
+       exit;
 }
 
 if ($usecache) {
-    // Generate hash for caching on
-    $hashtext = $_SERVER['REQUEST_URI'];
-    if ($userservice->isLoggedOn()) {
-        $hashtext .= $userservice->getCurrentUserID();
-    }
-    $hash = md5($hashtext);
-
-    // Cache for an hour
-    $cacheservice->Start($hash, 3600);
+       // Generate hash for caching on
+       $hashtext = $_SERVER['REQUEST_URI'];
+       if ($userservice->isLoggedOn()) {
+               $hashtext .= $userservice->getCurrentUserID();
+       }
+       $hash = md5($hashtext);
+
+       // Cache for an hour
+       $cacheservice->Start($hash, 3600);
 }
 
 // Header variables
 $pagetitle = T_('All Tags');
 
 if (isset($user) && $user != '') {
-    if (is_int($user)) {
-      $userid = intval($user);
-    } else {
-        if ($userinfo = $userservice->getObjectUserByUsername($user)) {
-            $userid = $userinfo->getId();
-        } else {
-            $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
-            $templateservice->loadTemplate('error.404.tpl', $tplVars);
-            //throw a 404 error
-            exit();
-        }
-    }
-    $pagetitle .= ': '. ucfirst($user);
+
+       $userid = $userservice->getIdFromUser($user);
+       if($userid == NULL) {
+               $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
+               $templateservice->loadTemplate('error.404.tpl', $tplVars);
+               exit();
+       }
+       
+       $pagetitle .= ': '. ucfirst($user);
 } else {
-    $userid = NULL;
+       $userid = NULL;
 }
 
 $tags =& $b2tservice->getTags($userid);
-$tplVars['tags'] =& $b2tservice->tagCloud($tags, 5, 90, 225, getSortOrder()); 
+$tplVars['tags'] =& $b2tservice->tagCloud($tags, 5, 90, 225, getSortOrder());
 $tplVars['user'] = $user;
 
 if (isset($userid)) {
-    $tplVars['cat_url'] = createURL('bookmarks', '%s/%s');
+       $tplVars['cat_url'] = createURL('bookmarks', '%s/%s');
 } else {
-    $tplVars['cat_url'] = createURL('tags', '%2$s');
+       $tplVars['cat_url'] = createURL('tags', '%2$s');
 }
 
 $tplVars['sidebar_blocks'] = array('linked');
@@ -84,7 +80,7 @@ $tplVars['subtitle'] = $pagetitle;
 $templateservice->loadTemplate('tags.tpl', $tplVars);
 
 if ($usecache) {
-    // Cache output if existing copy has expired
-    $cacheservice->End($hash);
+       // Cache output if existing copy has expired
+       $cacheservice->End($hash);
 }
 ?>
index 430a355540d07eba93ba3175e23a2a92543c9b13..b1739782feac1d1724b69df908c17b4b76a4891f 100644 (file)
@@ -44,7 +44,7 @@ if (POST_SUBMITTED != '') {
 
                // NO MATCH
                $userinfo = $userservice->getObjectUserByUsername(POST_USERNAME);
-               if ($userinfo == '') {
+               if ($userinfo == NULL) {
                        $tplVars['error'] = T_('No matches found for that username.');
 
                } elseif (POST_EMAIL != $userinfo->getEmail()) {
index c20cabc60d41f8a74900a973303f75fbbb9f80a7..79380276fd7ccdf0fff8808d9033aa906873cf73 100644 (file)
@@ -1,23 +1,23 @@
 <?php
 /***************************************************************************
-Copyright (C) 2004 - 2006 Scuttle project
-http://sourceforge.net/projects/scuttle/
-http://scuttle.org/
+ Copyright (C) 2004 - 2006 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');
 
@@ -29,50 +29,46 @@ $cacheservice =& ServiceFactory::getServiceInstance('CacheService');
 list($url, $user) = explode('/', $_SERVER['PATH_INFO']);
 
 if ($usecache) {
-    // Generate hash for caching on
-    $hashtext = $_SERVER['REQUEST_URI'];
-    if ($userservice->isLoggedOn()) {
-        $hashtext .= $currentUser->getId();
-        if ($currentUser->getUsername() == $user) {
-            $hashtext .= $user;
-        }
-    }
-    $hash = md5($hashtext);
-
-    // Cache for an hour
-    $cacheservice->Start($hash, 3600);
+       // Generate hash for caching on
+       $hashtext = $_SERVER['REQUEST_URI'];
+       if ($userservice->isLoggedOn()) {
+               $hashtext .= $currentUser->getId();
+               if ($currentUser->getUsername() == $user) {
+                       $hashtext .= $user;
+               }
+       }
+       $hash = md5($hashtext);
+
+       // Cache for an hour
+       $cacheservice->Start($hash, 3600);
 }
 
 // Header variables
 $pagetitle = T_('Popular Tags');
 
 if (isset($user) && $user != '') {
-    if (is_int($user)) {
-      $userid = intval($user);
-    } else {
-       $userinfo = $userservice->getObjectUserByUsername($user);
-        if ($userinfo != '') {
-            $userid = $userinfo->getId();
-        } else {
-            $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
-            $templateservice->loadTemplate('error.404.tpl', $tplVars);
-            //throw a 404 error
-            exit();
-        }
-    }
-    $pagetitle .= ': '. ucfirst($user);
+
+       $userid = $userservice->getIdFromUser($user);
+       if($userid == NULL) {
+               $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
+               $templateservice->loadTemplate('error.404.tpl', $tplVars);
+               //throw a 404 error
+               exit();
+       }
+
+       $pagetitle .= ': '. ucfirst($user);
 } else {
-    $userid = NULL;
+       $userid = NULL;
 }
 
 $tags = $b2tservice->getPopularTags($userid, 150);
-$tplVars['tags'] =& $b2tservice->tagCloud($tags, 5, 90, 225, getSortOrder('alphabet_asc')); 
+$tplVars['tags'] =& $b2tservice->tagCloud($tags, 5, 90, 225, getSortOrder('alphabet_asc'));
 $tplVars['user'] = $user;
 
 if (isset($userid)) {
-    $tplVars['cat_url'] = createURL('bookmarks', '%s/%s');
+       $tplVars['cat_url'] = createURL('bookmarks', '%s/%s');
 } else {
-    $tplVars['cat_url'] = createURL('tags', '%2$s');
+       $tplVars['cat_url'] = createURL('tags', '%2$s');
 }
 
 $tplVars['sidebar_blocks'] = array('linked');
@@ -81,8 +77,8 @@ $tplVars['loadjs'] = true;
 
 $templateservice->loadTemplate('tags.tpl', $tplVars);
 
-if ($usecache) {    
-    // Cache output if existing copy has expired
-    $cacheservice->End($hash);
+if ($usecache) {
+       // Cache output if existing copy has expired
+       $cacheservice->End($hash);
 }
 ?>
index 56d651500a3d0bed88703edb0f3de0c80fc66607..2d001014297ba0d50819d8dda790890de6cd9e1b 100644 (file)
@@ -1,23 +1,23 @@
 <?php
 /***************************************************************************
-Copyright (C) 2004 - 2006 Scuttle project
-http://sourceforge.net/projects/scuttle/
-http://scuttle.org/
+ Copyright (C) 2004 - 2006 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');
 
@@ -41,29 +41,30 @@ isset($_SESSION['token_stamp']) ? define('SESSION_TOKENSTAMP', $_SESSION['token_
 @list($url, $user) = isset($_SERVER['PATH_INFO']) ? explode('/', $_SERVER['PATH_INFO']) : NULL;
 
 if ($user) {
-    if (is_int($user)) {
-        $userid = intval($user);
-    } else {
-        $user = urldecode($user);
-        $userinfo = $userservice->getObjectUserByUsername($user);
-        if ($userinfo == '') {
-            $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
-            $templateservice->loadTemplate('error.404.tpl', $tplVars);
-            exit();
-        } else {
-            $userid =& $userinfo->getId();
-        }
-    }
+       
+       if (is_int($user)) {
+               $userid = intval($user);
+       } else {
+               $user = urldecode($user);
+               $userinfo = $userservice->getObjectUserByUsername($user);
+               if ($userinfo == NULL) {
+                       $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
+                       $templateservice->loadTemplate('error.404.tpl', $tplVars);
+                       exit();
+               } else {
+                       $userid =& $userinfo->getId();
+               }
+       }
 } else {
-    $tplVars['error'] = T_('Username was not specified');
-    $templateservice->loadTemplate('error.404.tpl', $tplVars);
-    exit();
+       $tplVars['error'] = T_('Username was not specified');
+       $templateservice->loadTemplate('error.404.tpl', $tplVars);
+       exit();
 }
 
 if ($userservice->isLoggedOn() && $user == $currentUser->getUsername()) {
-    $title = T_('My Profile');
+       $title = T_('My Profile');
 } else {
-    $title = T_('Profile') .': '. $user;
+       $title = T_('Profile') .': '. $user;
 }
 $tplVars['pagetitle'] = $title;
 $tplVars['subtitle'] = $title;
@@ -72,55 +73,55 @@ $tplVars['user'] = $user;
 $tplVars['userid'] = $userid;
 
 if (POST_SUBMITTED!='' && $currentUser->getId() == $userid) {
-    $error = false;
-    $detPass = trim(POST_PASS);
-    $detPassConf = trim(POST_PASSCONF);
-    $detName = trim(POST_NAME);
-    $detMail = trim(POST_MAIL);
-    $detPage = trim(POST_PAGE);
-    $detDesc = filter(POST_DESC);
-    
-    // manage token preventing from CSRF vulnaribilities 
-    if ( SESSION_TOKEN == ''
-        || time() - SESSION_TOKENSTAMP > 600 //limit token lifetime, optionnal
-        || SESSION_TOKEN != POST_TOKEN) {
-        $error = true;
-        $tplVars['error'] = T_('Invalid Token');
-    }
-    
-    if ($detPass != $detPassConf) {
-        $error = true;
-        $tplVars['error'] = T_('Password and confirmation do not match.');
-    }
-    if ($detPass != "" && strlen($detPass) < 6) {
-        $error = true;
-        $tplVars['error'] = T_('Password must be at least 6 characters long.');
-    }
-    if (!$userservice->isValidEmail($detMail)) {
-        $error = true;
-        $tplVars['error'] = T_('E-mail address is not valid.');
-    }
-    if (!$error) {
-        if (!$userservice->updateUser($userid, $detPass, $detName, $detMail, $detPage, $detDesc)) {
-            $tplvars['error'] = T_('An error occurred while saving your changes.');
-        } else {
-            $tplVars['msg'] = T_('Changes saved.');
-        }
-    }
-    $userinfo = $userservice->getObjectUserByUsername($user);
+       $error = false;
+       $detPass = trim(POST_PASS);
+       $detPassConf = trim(POST_PASSCONF);
+       $detName = trim(POST_NAME);
+       $detMail = trim(POST_MAIL);
+       $detPage = trim(POST_PAGE);
+       $detDesc = filter(POST_DESC);
+
+       // manage token preventing from CSRF vulnaribilities
+       if ( SESSION_TOKEN == ''
+       || time() - SESSION_TOKENSTAMP > 600 //limit token lifetime, optionnal
+       || SESSION_TOKEN != POST_TOKEN) {
+               $error = true;
+               $tplVars['error'] = T_('Invalid Token');
+       }
+
+       if ($detPass != $detPassConf) {
+               $error = true;
+               $tplVars['error'] = T_('Password and confirmation do not match.');
+       }
+       if ($detPass != "" && strlen($detPass) < 6) {
+               $error = true;
+               $tplVars['error'] = T_('Password must be at least 6 characters long.');
+       }
+       if (!$userservice->isValidEmail($detMail)) {
+               $error = true;
+               $tplVars['error'] = T_('E-mail address is not valid.');
+       }
+       if (!$error) {
+               if (!$userservice->updateUser($userid, $detPass, $detName, $detMail, $detPage, $detDesc)) {
+                       $tplvars['error'] = T_('An error occurred while saving your changes.');
+               } else {
+                       $tplVars['msg'] = T_('Changes saved.');
+               }
+       }
+       $userinfo = $userservice->getObjectUserByUsername($user);
 }
 
 if (!$userservice->isLoggedOn() || $currentUser->getId() != $userid) {
-    $templatename = 'profile.tpl.php';
+       $templatename = 'profile.tpl.php';
 } else {
        //Token Init
        $_SESSION['token'] = md5(uniqid(rand(), true));
        $_SESSION['token_stamp'] = time();
-       
-    $templatename = 'editprofile.tpl.php';
-    $tplVars['formaction']  = createURL('profile', $user);
-    $tplVars['token'] = $_SESSION['token'];
-    
+
+       $templatename = 'editprofile.tpl.php';
+       $tplVars['formaction']  = createURL('profile', $user);
+       $tplVars['token'] = $_SESSION['token'];
+
 }
 
 $tplVars['objectUser'] = $userinfo;
index 56a910f7c46797214518e297c1d96f09704c663c..122e1e2080585a09d5951a613b9210fb955fe065 100644 (file)
@@ -95,18 +95,14 @@ if (POST_TERMS != '') {
                }
 
                if (isset($s_user)) {
-                       if (is_numeric($s_user)) {
-                               $s_user = intval($s_user);
-                       } else {
-                               $userinfo = $userservice->getObjectUserByUsername($s_user);
-                               if ($userinfo == '' ) {
-                                       $tplVars['error'] = sprintf(T_('User with username %s was not found'), $s_user);
-                                       $templateservice->loadTemplate('error.404.tpl', $tplVars);
-                                       exit();
-                               } else {
-                                       $s_user =& $userinfo->getId();
-                               }
+                       
+                       $s_user = $userservice->getIdFromUser($s_user);
+                       if($s_user == NULL) {
+                               $tplVars['error'] = sprintf(T_('User with username %s was not found'), $s_user);
+                               $templateservice->loadTemplate('error.404.tpl', $tplVars);
+                               exit();
                        }
+                       
                }
        }
        $bookmarks =& $bookmarkservice->getBookmarks($start, $perpage, $s_user, NULL, $terms, getSortOrder(), $s_watchlist, $s_start, $s_end);
index fa0d5c9c3e19117aa7db7d8c86ec84adfa5e51a7..21b91366984b3bce8bd3c031b3bfe0696590f3c5 100644 (file)
@@ -127,7 +127,25 @@ class UserService {
 
        function getObjectUserByUsername($username) {
                $user = $this->_getuser($this->getFieldName('username'), $username);
-               return new User($user[$this->getFieldName('primary')], $username);
+               if($user != false) {
+                       return new User($user[$this->getFieldName('primary')], $username);
+               } else {
+                       return NULL;
+               }
+       }
+
+       /* Takes an numerical "id" or a string "username"
+        and returns the numerical "id" if the user exists else returns NULL */
+       function getIdFromUser($user) {
+               if (is_int($user)) {
+                       return intval($user);
+               } else {
+                       $objectUser = $this->getObjectUserByUsername($user);
+                       if($objectUser != NULL) {
+                               return $objectUser->getId();
+                       }
+               }
+               return NULL;
        }
 
        function getUser($id) {
@@ -465,7 +483,7 @@ class UserService {
 
                        // Check if the email domain has a DNS record
                        //if ($this->_checkdns($emailDomain)) {
-                               return true;
+                       return true;
                        //}
                }
                return false;
@@ -520,7 +538,7 @@ class User {
                }
                return $this->name;
        }
-       
+
        function getEmail() {
                // Look for value only if not already set
                if(!isset($this->email)) {
@@ -540,7 +558,7 @@ class User {
                }
                return $this->homepage;
        }
-       
+
        function getContent() {
                // Look for value only if not already set
                if(!isset($this->content)) {
@@ -549,7 +567,7 @@ class User {
                        $this->content = $user['uContent'];
                }
                return $this->content;
-       }       
+       }
 
        function getDatetime() {
                // Look for value only if not already set
index 9fc480b34ac6a39c1fcc17e80f6fba880bd0a41b..841317db1c2e085d04a890f951baa21f6b57e806 100644 (file)
--- a/watch.php
+++ b/watch.php
@@ -34,17 +34,12 @@ $currentUser = $userservice->getCurrentObjectUser();
 if ($userservice->isLoggedOn() && $user) {
        $pagetitle = '';
 
-       if (is_int($user)) {
-               $userid = intval($user);
-       } else {
-               $userinfo = $userservice->getObjectUserByUsername($user);
-               if ($userinfo == '') {
-                       $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
-                       $templateservice->loadTemplate('error.404.tpl', $tplVars);
-                       exit();
-               } else {
-                       $userid =& $userinfo->getId();
-               }
+       $userid = $userservice->getIdFromUser($user);
+       
+       if($userid == NULL) {
+               $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
+               $templateservice->loadTemplate('error.404.tpl', $tplVars);
+               exit();
        }
 
        $watched = $userservice->getWatchStatus($userid, $currentUser->getId());
index d8249370e32d7630ed62fb44ba6be2f65c1837b0..2cdb5ffcc9fe7ade55e20f4f0469da09a71ed3e7 100644 (file)
@@ -56,7 +56,7 @@ if ($user) {
         $userid = intval($user);
     } else {
        $userinfo = $userservice->getObjectUserByUsername($user);
-        if ($userinfo == '' ) {
+        if ($userinfo == NULL ) {
             // Throw a 404 error
             $tplVars['error'] = sprintf(T_('User with username %s was not found'), $user);
             $templateservice->loadTemplate('error.404.tpl', $tplVars);