Bug fixes: correct minor bugs appearing with 'notice' level in PHP

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

diff --git a/api/httpauth.inc.php b/api/httpauth.inc.php
index f693d35488237eacfd726f28b9f08598da4c4e62..d0198fddda2425618a0b04bfff0bc6062bb80347 100644 (file)
--- a/api/httpauth.inc.php
+++ b/api/httpauth.inc.php
@@ -13,7 +13,6 @@ function authenticate() {
 }
 
 if(!$userservice->isLoggedOn()) {
-
     /* Maybe we have caught authentication data in $_SERVER['REMOTE_USER']
     ( Inspired by http://www.yetanothercommunitysystem.com/article-321-regle-comment-utiliser-l-authentification-http-en-php-chez-ovh ) */
     if((!isset($_SERVER['PHP_AUTH_USER']) || !isset($_SERVER['PHP_AUTH_PW']))

diff --git a/bookmarks.php b/bookmarks.php
index a97fb05272da3183781a6c672a280cbe75d58ca9..eca291d71a91743c12a4a09918c6055118d597b6 100644 (file)
--- a/bookmarks.php
+++ b/bookmarks.php
@@ -209,12 +209,12 @@ if ($templatename == 'editbookmark.tpl') {
     $tplVars['start'] = $start;
     $tplVars['bookmarkCount'] = $start + 1;
 
-    $bookmarks =& $bookmarkservice->getBookmarks($start, $perpage, $userid, $cat, $terms, getSortOrder());
+    $bookmarks =& $bookmarkservice->getBookmarks($start, $perpage, $userid, $cat, null, getSortOrder());
     $tplVars['total'] = $bookmarks['total'];
     $tplVars['bookmarks'] =& $bookmarks['bookmarks'];
     $tplVars['cat_url'] = createURL('bookmarks', '%s/%s');
     $tplVars['nav_url'] = createURL('bookmarks', '%s/%s%s');
-    if ($user == $currentUsername) {
+    if ($userservice->isLoggedOn() && $user == $currentUsername) {
         $title = T_('My Bookmarks') . filter($catTitle);
     } else {
         $title = filter($pagetitle);

diff --git a/header.inc.php b/header.inc.php
index 9d147b043d5f0d1bb84a17d2533c3f814eedbbcc..820b5f9c4244a4666131e33a4c3d6ef0c9cb2c7e 100644 (file)
--- a/header.inc.php
+++ b/header.inc.php
@@ -3,6 +3,7 @@ ini_set('display_errors', '1');
 ini_set('mysql.trace_mode', '0');
 
 error_reporting(E_ALL ^ E_NOTICE);
+//error_reporting(E_ALL);
 
 define('DEBUG', true);
 session_start();
@@ -30,4 +31,5 @@ if (!isset($root)) {
     }
     $root = 'http://'. $_SERVER['HTTP_HOST'] . $root;
 }
+
 ?>

diff --git a/includes/php-gettext/streams.php b/includes/php-gettext/streams.php
index d57aac6496b2b1da2419cff07f2d04de9455bd56..4237de141b8b424a59890ee89b74b8972e161a6d 100644 (file)
--- a/includes/php-gettext/streams.php
+++ b/includes/php-gettext/streams.php
@@ -108,6 +108,7 @@ class FileReader {
 
       // PHP 5.1.1 does not read more than 8192 bytes in one fread()
       // the discussions at PHP Bugs suggest it's the intended behaviour
+      $data = '';
       while ($bytes > 0) {
         $chunk  = fread($this->_fd, $bytes);
         $data  .= $chunk;
@@ -163,4 +164,4 @@ class CachedFileReader extends StringReader {
 }
 
 
-?>
\ No newline at end of file
+?>

diff --git a/login.php b/login.php
index 4d212a94def47199bce52884d4b2a74a706e82cf..f09996e7265c7b4a3c12f9746eb2ecca7636d0a3 100644 (file)
--- a/login.php
+++ b/login.php
@@ -25,10 +25,12 @@ $templateservice =& ServiceFactory::getServiceInstance('TemplateService');
 
 $tplVars = array();
 
+$keeppass = isset($_POST['keeppass'])&&($_POST['keeppass']=='yes')?true:false;
+
 $login = false;
 if (isset($_POST['submitted']) && isset($_POST['username']) && isset($_POST['password'])) {
     $posteduser = trim(utf8_strtolower($_POST['username']));
-    $login = $userservice->login($posteduser, $_POST['password'], ($_POST['keeppass'] == "yes")); 
+    $login = $userservice->login($posteduser, $_POST['password'], $keeppass); 
     if ($login) {
         if ($_POST['query'])
             header('Location: '. createURL('bookmarks', $posteduser .'?'. $_POST['query']));

diff --git a/password.php b/password.php
index b7daa70ea2eea47b8a5e67c6fcdf3bee3f5bb0c0..1763fd239d1e393e51d16edb0cb72d44ae047f6f 100644 (file)
--- a/password.php
+++ b/password.php
@@ -24,7 +24,7 @@ $templateservice =& ServiceFactory::getServiceInstance('TemplateService');
 $tplVars = array();
 
 // IF SUBMITTED
-if ($_POST['submitted']) {
+if (isset($_POST['submitted'])) {
 
     // NO USERNAME
     if (!$_POST['username']) {

diff --git a/populartags.php b/populartags.php
index a51ff68a1da76a8e2d6d1865990d64defb948c28..1b99ca25332c905fb23efb5d7aa177cfe5a8f0cf 100644 (file)
--- a/populartags.php
+++ b/populartags.php
@@ -66,7 +66,7 @@ if (isset($user) && $user != '') {
     $userid = NULL;
 }
 
-$tags = $b2tservice->getPopularTags($userid, 150, $logged_on_userid);
+$tags = $b2tservice->getPopularTags($userid, 150);
 $tplVars['tags'] =& $b2tservice->tagCloud($tags, 5, 90, 225, getSortOrder('alphabet_asc')); 
 $tplVars['user'] = $user;
 

diff --git a/register.php b/register.php
index 9cd6f39b2de3884bed28041e837b6ac923ae1ec5..e60a686eadd31b97bcd172f2317aefc2a2d9aa50 100644 (file)
--- a/register.php
+++ b/register.php
@@ -25,7 +25,7 @@ $templateservice =& ServiceFactory::getServiceInstance('TemplateService');
 
 $tplVars = array();
 
-if ($_POST['submitted']) {
+if (isset($_POST['submitted'])) {
     $posteduser = trim(utf8_strtolower($_POST['username']));
 
     // Check if form is incomplete

diff --git a/search.inc.php b/search.inc.php
index 3900535be5ab16045b9dd10258fdd4720f44f78d..832a70c2cd2415abe9ae289500de3936e1fb0f3b 100644 (file)
--- a/search.inc.php
+++ b/search.inc.php
@@ -1,3 +1,10 @@
+<?php
+/* Manage input */
+$select_watchlist = isset($select_watchlist)?$select_watchlist:'';
+$select_all = isset($select_all)?$select_all:'';
+?>
+
+
 <form id="search" action="<?php echo createURL('search'); ?>" method="post">
     <table>
     <tr>
@@ -16,12 +23,12 @@
                 <?php
                 if (!in_array($range, array($currentUsername, 'all', 'watchlist'))) {
                 ?>
-                <option value="<?php echo $user ?>"<?php echo $selectUser; ?>><?php echo T_("this user's bookmarks"); ?></option>
+                <option value="<?php echo $user ?>"<?php //echo $selectUser; ?>><?php echo T_("this user's bookmarks"); ?></option>
                 <?php
                 }
                 if ($logged_on) {
                 ?>
-                <option value="<?php echo $currentUsername; ?>"<?php echo $selectMy; ?>><?php echo T_('my bookmarks'); ?></option>
+                <option value="<?php echo $currentUsername; ?>"<?php //echo $selectMy; ?>><?php echo T_('my bookmarks'); ?></option>
                 <option value="watchlist"<?php echo $select_watchlist; ?>><?php echo T_('my watchlist'); ?></option>
                 <?php
                 }
@@ -37,7 +44,7 @@
         <?php
         }
         ?>
-        <td><input type="text" name="terms" size="30" value="<?php echo filter($terms); ?>" /></td>
+        <td><input type="text" name="terms" size="30" value="<?php $terms=!isset($terms)?'':$terms; echo filter($terms); ?>" /></td>
         <td><input type="submit" value="<?php echo T_('Search' /* Submit button */); ?>" /></td>
     </tr>
     </table>

diff --git a/services/bookmark2tagservice.php b/services/bookmark2tagservice.php
index 148ad3a615172349828bcd1791c3ac453c853839..121ba8d569a94752898e82ffb3109fe3a705d0f9 100644 (file)
--- a/services/bookmark2tagservice.php
+++ b/services/bookmark2tagservice.php
@@ -266,7 +266,8 @@ class Bookmark2TagService {
             message_die(GENERAL_ERROR, 'Could not get related tags', '', __LINE__, __FILE__, $query, $this->db);
             return false;
         }
-        return $this->db->sql_fetchrowset($dbresult);
+       $output = $this->db->sql_fetchrowset($dbresult);
+        return $output;
     }
 
     // Returns the most popular tags used for a particular bookmark hash
@@ -322,7 +323,8 @@ class Bookmark2TagService {
             return false;
         }
 
-        return $this->db->sql_fetchrowset($dbresult);
+       $output = $this->db->sql_fetchrowset($dbresult);
+        return $output;
     }
 
     function hasTag($bookmarkid, $tag) {
@@ -366,7 +368,8 @@ class Bookmark2TagService {
     function &tagCloud($tags = NULL, $steps = 5, $sizemin = 90, $sizemax = 225, $sortOrder = NULL) {
 
         if (is_null($tags) || count($tags) < 1) {
-            return false;
+           $output = false;
+            return $output;
         }
 
         $min = $tags[count($tags) - 1]['bCount'];

diff --git a/services/bookmarkservice.php b/services/bookmarkservice.php
index 5339004665cad122d51c4005db9fbd7715077f8c..2cd42e218c2822bf5f0a8a1c67bf247d9063236c 100644 (file)
--- a/services/bookmarkservice.php
+++ b/services/bookmarkservice.php
@@ -280,6 +280,7 @@ class BookmarkService {
             $query_3 .= ' AND ('. $query_3_1 .') AND B.bStatus IN (0, 1)';
         }
 
+       $query_5 = '';
        if($hash == null) {
            $query_5.= ' GROUP BY B.bHash';
        }
@@ -389,7 +390,9 @@ class BookmarkService {
             $row['tags'] = $b2tservice->getTagsForBookmark(intval($row['bId']));
             $bookmarks[] = $row;
         }
-        return array ('bookmarks' => $bookmarks, 'total' => $total);
+
+       $output = array ('bookmarks' => $bookmarks, 'total' => $total);
+        return $output;
     }
 
     function deleteBookmark($bookmarkid) {

diff --git a/services/tag2tagservice.php b/services/tag2tagservice.php
index 8b2b731dd633b2827c5de1c3cda26d61178a281e..bf4f866aed4cbf214b3f4048b7202576f8e98f96 100644 (file)
--- a/services/tag2tagservice.php
+++ b/services/tag2tagservice.php
@@ -62,7 +62,7 @@ class Tag2TagService {
            $query.= " AND uId = '".$uId."'";
        }
 //die($query);
-        if (! ($dbresult =& $this->db->sql_query_limit($query, $limit)) ){
+        if (! ($dbresult =& $this->db->sql_query($query)) ){
             message_die(GENERAL_ERROR, 'Could not get related tags', '', __LINE__, __FILE__, $query, $this->db);
             return false;
         }

diff --git a/services/userservice.php b/services/userservice.php
index 9e3adca0b8e2258410eb2f73b9f2eae8b605b0e0..3d2058fdd0ca13e14efd7396fe44db6cce988208 100644 (file)
--- a/services/userservice.php
+++ b/services/userservice.php
@@ -127,7 +127,7 @@ class UserService {
             if ($id = $this->getCurrentUserId())
                 $currentuser = $this->getUser($id);
             else
-                return;
+                return null;
         }
         return $currentuser;
     }

diff --git a/tag2tagadd.php b/tag2tagadd.php
index bc13b16deaba054075729b53c6c7b20ea5ad6f70..935cf889fe7368ae9097308dd8a8b58b3d1539be 100644 (file)
--- a/tag2tagadd.php
+++ b/tag2tagadd.php
@@ -24,10 +24,12 @@ $tag2tagservice = & ServiceFactory :: getServiceInstance('Tag2TagService');
 $templateservice = & ServiceFactory :: getServiceInstance('TemplateService');
 $userservice = & ServiceFactory :: getServiceInstance('UserService');
 
+
+
 $logged_on_user = $userservice->getCurrentUser();
 
 //permissions
-if($logged_on_user == null) {
+if($logged_on_user  == null) {
     $tplVars['error'] = T_('Permission denied.');
     $templateservice->loadTemplate('error.500.tpl', $tplVars);
     exit();

diff --git a/templates/bookmarks.tpl.php b/templates/bookmarks.tpl.php
index 6a778bbb22dfb11e7e4290bb1ae33450758f760b..f7cc45bc96c919613600ede81a19eb19bca73e1f 100644 (file)
--- a/templates/bookmarks.tpl.php
+++ b/templates/bookmarks.tpl.php
@@ -43,17 +43,22 @@ if($logged_on_userid>0) {
 
 
 <?php
-$userObject = $userservice->getUserByUsername($user);
 /* Private tag description */
-if(isset($currenttag) && strlen($user)>0 && $tagservice->getDescription($currenttag, $userObject['uId'])):?>
+if(isset($currenttag) && isset($user)) {
+    $userObject = $userservice->getUserByUsername($user);
+    if($tagservice->getDescription($currenttag, $userObject['uId'])) { ?>
+
 <p class="commondescription">
 <?php
-
     $description = $tagservice->getDescription($currenttag, $userObject['uId']);
     echo nl2br(filter($description['tDescription']));
 ?>
 </p>
-<?php endif ?>
+
+<?php 
+    }
+}
+?>
 
 <?php if (count($bookmarks) > 0) { ?>
 <script type="text/javascript">

diff --git a/templates/sidebar.block.linked.php b/templates/sidebar.block.linked.php
index a39f590cc9b1dc3b732e1ba9a5db9dfcdf194f6e..efe75a8e55ae14419ab86d62a92223f870fa8ef1 100644 (file)
--- a/templates/sidebar.block.linked.php
+++ b/templates/sidebar.block.linked.php
@@ -4,13 +4,20 @@ $userservice =& ServiceFactory::getServiceInstance('UserService');
 
 require_once('sidebar.linkedtags.inc.php');
 
+/* Manage input */
+$user = isset($user)?$user:'';
+$userid = isset($userid)?$userid:0;
+$currenttag = isset($currenttag)?$currenttag:'';
+$summarizeLinkedTags = isset($summarizeLinkedTags)?$summarizeLinkedTags:false;
+
+
 $logged_on_userid = $userservice->getCurrentUserId();
 if ($logged_on_userid === false) {
     $logged_on_userid = NULL;
 }
 
 $explodedTags = array();
-if ($currenttag) {
+if (strlen($currenttag)>0) {
     $explodedTags = explode('+', $currenttag);
 } else {
     if($summarizeLinkedTags == true) {

diff --git a/templates/sidebar.block.menu.php b/templates/sidebar.block.menu.php
index 0c7087a8ff7fd8d99c264edc9240367e92f0b25e..578db599811be2a9781be5343c5c63a5b3001b62 100644 (file)
--- a/templates/sidebar.block.menu.php
+++ b/templates/sidebar.block.menu.php
@@ -4,6 +4,11 @@ $userservice =& ServiceFactory::getServiceInstance('UserService');
 
 require_once('sidebar.linkedtags.inc.php');
 
+/* Manage input */
+$userid = isset($userid)?$userid:0;
+$user = isset($user)?$user:null;
+
+
 $logged_on_userid = $userservice->getCurrentUserId();
 if ($logged_on_userid === false) {
     $logged_on_userid = NULL;
@@ -16,7 +21,7 @@ if(($logged_on_userid != null) && ($userid === $logged_on_userid)) {
     $editingMode = false;
 }
 
-if(strlen($user)==0) {
+if(!isset($user)) {
     $cat_url = createURL('tags', '%2$s');
 }
 

diff --git a/templates/sidebar.block.recent.php b/templates/sidebar.block.recent.php
index e720533014974fb055376a23f6a8dcc90da6b6df..cc04b970691fb276620bdd3a2209a858d0c1f085 100644 (file)
--- a/templates/sidebar.block.recent.php
+++ b/templates/sidebar.block.recent.php
@@ -2,6 +2,9 @@
 $b2tservice =& ServiceFactory::getServiceInstance('Bookmark2TagService');
 $userservice =& ServiceFactory::getServiceInstance('UserService');
 
+/* Manage input */
+$userid = isset($userid)?$userid:0;
+
 $logged_on_userid = $userservice->getCurrentUserId();
 if ($logged_on_userid === false) {
     $logged_on_userid = NULL;

diff --git a/templates/sidebar.linkedtags.inc.php b/templates/sidebar.linkedtags.inc.php
index 95b420a036a52e52c88d8698f8397844f9aa26c0..555eafc003d673b7cbedd6b5ee743b51caf465ce 100644 (file)
--- a/templates/sidebar.linkedtags.inc.php
+++ b/templates/sidebar.linkedtags.inc.php
@@ -19,6 +19,8 @@ function displayLinkedTags($tag, $linkType, $uId, $cat_url, $user, $editingMode
        } else {
            $link = '> ';       
        }
+    } else  {
+       $link = '';
     }
 
     $output = '';
@@ -69,7 +71,7 @@ function displayLinkedTags($tag, $linkType, $uId, $cat_url, $user, $editingMode
                    $displayLinkedTags = displayLinkedTags($linkedTag, $linkType, $uId, $cat_url, $user, $editingMode, $precedentTag, $level + 1, $stopList);
                    $output.= $displayLinkedTags['output'];
                }
-               if(is_array($displayLinkedTags['stopList'])) {
+               if(isset($displayLinkedTags) && is_array($displayLinkedTags['stopList'])) {
                    $stopList = array_merge($stopList, $displayLinkedTags['stopList']);
                    $stopList = array_unique($stopList);
                }

diff --git a/templates/toolbar.inc.php b/templates/toolbar.inc.php
index ab17bc5092343bb2dc9f6cf04569a2b8e020fdb0..593ee0f3e1fe3797f65cf76c9f099257ebc3f27a 100644 (file)
--- a/templates/toolbar.inc.php
+++ b/templates/toolbar.inc.php
@@ -21,7 +21,7 @@ if ($userservice->isLoggedOn()) {
 ?>
 
     <ul id="navigation">
-       <li><a href="<?php echo createURL('populartags', $cUsername); ?>"><?php echo T_('Popular Tags'); ?></a></li>
+       <li><a href="<?php echo createURL('populartags'); ?>"><?php echo T_('Popular Tags'); ?></a></li>
         <li><a href="<?php echo createURL('about'); ?>"><?php echo T_('About'); ?></a></li>
         <li class="access"><a href="<?php echo createURL('login'); ?>"><?php echo T_('Log In'); ?></a></li>
         <li class="access"><a href="<?php echo createURL('register'); ?>"><?php echo T_('Register'); ?></a></li>

diff --git a/templates/top.inc.php b/templates/top.inc.php
index 58eacb5dce19d0796dd4c803d0636e8240d86f4d..707af8b444ecf436f556aa61f119a96f764ae27b 100644 (file)
--- a/templates/top.inc.php
+++ b/templates/top.inc.php
@@ -7,11 +7,13 @@
     <link rel="icon" type="image/png" href="<?php echo $GLOBALS['root']; ?>icon.png" />
     <link rel="stylesheet" type="text/css" href="<?php echo $GLOBALS['root']; ?>scuttle.css" />
     <?php
-    $size = count($rsschannels);
-    for ($i = 0; $i < $size; $i++) {
-        echo '<link rel="alternate" type="application/rss+xml" title="'. $rsschannels[$i][0] .'" href="'. $rsschannels[$i][1] .'" />';
+    if(isset($rsschannels)) {
+       $size = count($rsschannels);
+       for ($i = 0; $i < $size; $i++) {
+           echo '<link rel="alternate" type="application/rss+xml" title="'. $rsschannels[$i][0] .'" href="'. $rsschannels[$i][1] .'" />';
+       }
     }
-    if ($loadjs) {
+    if (isset($loadjs)) {
         echo '<script type="text/javascript" src="'. $GLOBALS['root'] .'jsScuttle.php"></script>';
     }
     ?>