From: garvinhicking <garvinhicking>
Date: Thu, 17 Nov 2005 19:54:05 +0000 (+0000)
Subject: Function cleanup by Omid Mottaghi
X-Git-Tag: 1.0~284
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=877cbf3969a1d721d3811f59aee01c4d0336f15c;p=s9y.git

Function cleanup by Omid Mottaghi
---

diff --git a/include/functions_calendars.inc.php b/include/functions_calendars.inc.php
index a512019..edf3ade 100644
--- a/include/functions_calendars.inc.php
+++ b/include/functions_calendars.inc.php
@@ -214,7 +214,7 @@ function jalali_strftime_utf($format, $timestamp) {
                     $output_str.=strftime('%z', $timestamp);
                     break;
                 case '%':
-                    $outpus_str.='%';
+                    $output_str.='%';
                     break;
             }
         }else{
diff --git a/include/functions_config.inc.php b/include/functions_config.inc.php
index 7d763aa..f4dffb3 100644
--- a/include/functions_config.inc.php
+++ b/include/functions_config.inc.php
@@ -799,7 +799,7 @@ function serendipity_deleteGroup($groupid) {
     
     if (!serendipity_checkPermission('adminUsersMaintainOthers')) {
         // Only groups should be accessible where a user has access rights.
-        $my_groups = serendipity_getGroups($serendipity['authorid'], true);
+        $my_groups =& serendipity_getGroups($serendipity['authorid'], true);
         if (!in_array($groupid, $my_groups)) {
             return false;
         }
@@ -850,8 +850,8 @@ function serendipity_intersectGroup($checkuser = null, $myself = null) {
         $myself = $serendipity['authorid'];
     }
     
-    $my_groups  = serendipity_getGroups($myself, true);
-    $his_groups = serendipity_getGroups($checkuser, true);
+    $my_groups  =& serendipity_getGroups($myself, true);
+    $his_groups =& serendipity_getGroups($checkuser, true);
 
     foreach($his_groups AS $his_group) {
         if (in_array($his_group, $my_groups)) {
@@ -871,13 +871,13 @@ function serendipity_updateGroupConfig($groupid, &$perms, &$values) {
 
     if (!serendipity_checkPermission('adminUsersMaintainOthers')) {
         // Only groups should be accessible where a user has access rights.
-        $my_groups = serendipity_getGroups($serendipity['authorid'], true);
+        $my_groups =& serendipity_getGroups($serendipity['authorid'], true);
         if (!in_array($groupid, $my_groups)) {
             return false;
         }
     } 
 
-    $storage = serendipity_fetchGroup($groupid);    
+    $storage =& serendipity_fetchGroup($groupid);    
 
     serendipity_db_query("DELETE FROM {$serendipity['dbPrefix']}groupconfig WHERE id = " . (int)$groupid); 
     foreach ($perms AS $perm => $userlevels) {
diff --git a/include/functions_images.inc.php b/include/functions_images.inc.php
index ab4da54..36e36c9 100644
--- a/include/functions_images.inc.php
+++ b/include/functions_images.inc.php
@@ -98,7 +98,6 @@ function serendipity_deleteImage($id) {
     $dFile  = $file['path'] . $file['name'] . '.' . $file['extension'];
     $dThumb = $file['path'] . $file['name'] . (!empty($file['thumbnail_name']) ? '.' . $file['thumbnail_name'] : '') . '.' . $file['extension'];
 
-    $admin = '';
     if (!serendipity_checkPermission('adminImagesDelete')) {
         return;
     }
@@ -696,7 +695,7 @@ function serendipity_syncThumbs() {
 
     $fcount = count($files);
     for ($x = 0; $x < $fcount; $x++) {
-        $update = $q = array();
+        $update = array();
         $f      = serendipity_parseFileName($files[$x]);
         if (empty($f[1]) || $f[1] == $files[$x]) {
             // No extension means bad file most probably. Skip it.
diff --git a/include/functions_rss.inc.php b/include/functions_rss.inc.php
index 45a4e3f..a8c74bf 100644
--- a/include/functions_rss.inc.php
+++ b/include/functions_rss.inc.php
@@ -54,7 +54,7 @@ function serendipity_printEntries_rss(&$entries, $version, $comments = false, $f
                 $entry['categories'] = array(0 => array(
                     'category_name'      => $entry['category_name'],
                     'feed_category_name' => serendipity_utf8_encode(htmlspecialchars($entry['category_name'])),
-                    'categoryURL'        => serendipity_categoryURL($cat, 'baseURL')
+                    'categoryURL'        => serendipity_categoryURL($entry, 'baseURL')
                 ));
             } else {
                 foreach($entry['categories'] AS $cid => $_cat) {
diff --git a/include/functions_smarty.inc.php b/include/functions_smarty.inc.php
index 4ba9b44..c2610de 100644
--- a/include/functions_smarty.inc.php
+++ b/include/functions_smarty.inc.php
@@ -169,7 +169,7 @@ function serendipity_smarty_printTrackbacks($params, &$smarty) {
     return serendipity_printTrackbacks(serendipity_fetchTrackbacks($params['entry']));
 }
 
-function &serendipity_replaceSmartyVars(&$tpl_source, $smarty) {
+function &serendipity_replaceSmartyVars(&$tpl_source, &$smarty) {
     $tpl_source = str_replace('$CONST.', '$smarty.const.', $tpl_source);
     return $tpl_source;
 }
@@ -304,7 +304,7 @@ function serendipity_smarty_purge() {
 function serendipity_smarty_shutdown($serendipity_directory = '') {
 global $serendipity;
 
-    $cwd = getcwd();
+    #$cwd = getcwd();
     chdir($serendipity_directory);
     $raw_data = ob_get_contents();
     ob_end_clean();
diff --git a/include/functions_trackbacks.inc.php b/include/functions_trackbacks.inc.php
index e17f20e..af5e51c 100644
--- a/include/functions_trackbacks.inc.php
+++ b/include/functions_trackbacks.inc.php
@@ -22,6 +22,7 @@ function serendipity_trackback_is_success($resp) {
 }
 
 function serendipity_pingback_autodiscover($loc, $body) {
+global $serendipity;
     if (!empty($_SERVER['X-PINGBACK'])) {
         $pingback = $_SERVER['X-PINGBACK'];
     } elseif (preg_match('@<link rel="pingback" href="([^"]+)" ?/?>@i', $body, $matches)) {