mambax7 / planet

class/article.php

Doc Comments -1 removed lines

@@ -109,7 +109,6 @@
     /**
      * get summary of the article
      * 
-     * @param bool 	$actionOnEmpty flag for truncating content if summary is empty
      * @return string
      */
     function &getSummary($length = 0)

Braces +3 added lines, -1 removed lines

@@ -100,7 +100,9 @@
     function getTime($format = "c")
     {
 	    $time = $this->getVar("art_time");
-	    if(empty($time)) $time = time();
+	    if(empty($time)) {
+	    	$time = time();
+	    }
 	    $time = planet_formatTimestamp($time, $format);
 		return $time;
     }

class/class-IXR.php

Doc Comments +11 added lines

@@ -650,6 +650,10 @@ 
 
 class IXR_Base64 {
     var $data;
+
+    /**
+     * @param string $data
+     */
     function IXR_Base64($data) {
         $this->data = $data;
     }
@@ -694,6 +698,13 @@ 
             'Returns a documentation string for the specified method'
         );
     }
+
+    /**
+     * @param string $method
+     * @param string $callback
+     * @param string[] $args
+     * @param string $help
+     */
     function addCallback($method, $callback, $args, $help) {
         $this->callbacks[$method] = $callback;
         $this->signatures[$method] = $args;

class/feedcreator.class.php

Doc Comments +9 added lines, -2 removed lines

@@ -359,6 +359,9 @@ 
 class UniversalFeedCreator extends FeedCreator {
 	var $_feed;
 
+	/**
+	 * @param string $format
+	 */
 	function _setFormat($format) {
 		switch (strtoupper($format)) {
 
@@ -540,6 +543,7 @@ 
 	 * Adds an FeedItem to the feed.
 	 *
 	 * @param object FeedItem $item The FeedItem to add to the feed.
+	 * @param FeedItem $item
 	 * @access public
 	 */
 	function addItem($item) {
@@ -557,6 +561,7 @@ 
 	 * @static
 	 * @param string    string A string to be truncated.
 	 * @param int        length the maximum length the string should be truncated to
+	 * @param integer $length
 	 * @return string    the truncated string
 	 */
 	function iTrunc($string, $length) {
@@ -655,6 +660,7 @@ 
 	/**
 	 * @since 1.4
 	 * @access private
+	 * @param string $filename
 	 */
 	function _redirect($filename) {
 		// attention, heavily-commented-out-area
@@ -788,7 +794,7 @@ 
 	/**
 	 * Gets the date stored in this FeedDate as an RFC 822 date.
 	 *
-	 * @return a date in RFC 822 format
+	 * @return string date in RFC 822 format
 	 */
 	function rfc822() {
 		//return gmdate("r",$this->unix);
@@ -800,7 +806,7 @@ 
 	/**
 	 * Gets the date stored in this FeedDate as an ISO 8601 date.
 	 *
-	 * @return a date in ISO 8601 format
+	 * @return string date in ISO 8601 format
 	 */
 	function iso8601() {
 		$date = gmdate("Y-m-d\TH:i:sO",$this->unix);
@@ -922,6 +928,7 @@ 
 	/**
 	 * Sets this RSS feed's version number.
 	 * @access private
+	 * @param string $version
 	 */
 	function _setRSSVersion($version) {
 		$this->RSSVersion = $version;

Braces +16 added lines, -6 removed lines

@@ -331,7 +331,7 @@ 
 		// - no truncation (truncating risks producing invalid html)
 		if (!$this->rawFieldContent) {
 			$result = "";
-		}	elseif ($this->syndicateHtml) {
+		} elseif ($this->syndicateHtml) {
 			$result = "<![CDATA[".$this->rawFieldContent."]]>";
 		} else {
 			if ($this->truncSize and is_int($this->truncSize)) {
@@ -616,8 +616,12 @@ 
 
 	function _createStylesheetReferences() {
 		$xml = "";
-		if ($this->cssStyleSheet) $xml .= "<?xml-stylesheet href=\"".$this->cssStyleSheet."\" type=\"text/css\"?>\n";
-		if ($this->xslStyleSheet) $xml .= "<?xml-stylesheet href=\"".$this->xslStyleSheet."\" type=\"text/xsl\"?>\n";
+		if ($this->cssStyleSheet) {
+			$xml .= "<?xml-stylesheet href=\"".$this->cssStyleSheet."\" type=\"text/css\"?>\n";
+		}
+		if ($this->xslStyleSheet) {
+			$xml .= "<?xml-stylesheet href=\"".$this->xslStyleSheet."\" type=\"text/xsl\"?>\n";
+		}
 		return $xml;
 	}
 
@@ -738,7 +742,9 @@ 
 	 */
 	function FeedDate($dateString="") {
 		$tzOffset = 0;
-		if ($dateString=="") $dateString = date("r");
+		if ($dateString=="") {
+			$dateString = date("r");
+		}
 
 		//if (is_integer($dateString)) {
 		if (is_numeric($dateString)) {
@@ -793,7 +799,9 @@ 
 	function rfc822() {
 		//return gmdate("r",$this->unix);
 		$date = gmdate("D, d M Y H:i:s", $this->unix);
-		if (TIME_ZONE!="") $date .= " ".str_replace(":","",TIME_ZONE);
+		if (TIME_ZONE!="") {
+			$date .= " ".str_replace(":","",TIME_ZONE);
+		}
 		return $date;
 	}
 
@@ -805,7 +813,9 @@ 
 	function iso8601() {
 		$date = gmdate("Y-m-d\TH:i:sO",$this->unix);
 		$date = substr($date,0,22) . ':' . substr($date,-2);
-		if (TIME_ZONE!="") $date = str_replace("+00:00",TIME_ZONE,$date);
+		if (TIME_ZONE!="") {
+			$date = str_replace("+00:00",TIME_ZONE,$date);
+		}
 		return $date;
 	}
 

class/magpie.inc.php

Doc Comments +7 added lines

@@ -450,6 +450,10 @@ 
     /**
     * return XML parser, and possibly re-encoded source
     *
+    * @param string $source
+    * @param string $out_enc
+    * @param string|null $in_enc
+    * @param boolean $detect
     */
     function create_parser($source, $out_enc, $in_enc, $detect) {
         if ( substr(phpversion(),0,1) == 5) {
@@ -561,6 +565,9 @@ 
         }
     }
 
+    /**
+     * @param integer $lvl
+     */
     function error ($errormsg, $lvl=E_USER_WARNING) {
         // append PHP's error message if track_errors enabled
         if ( !empty($php_errormsg) ) {

Braces +44 added lines, -73 removed lines

@@ -165,12 +165,10 @@ 
             if ( $el == 'rdf' ) {
                 $this->feed_type = RSS;
                 $this->feed_version = '1.0';
-            }
-            elseif ( $el == 'rss' ) {
+            } elseif ( $el == 'rss' ) {
                 $this->feed_type = RSS;
                 $this->feed_version = $attrs['version'];
-            }
-            elseif ( $el == 'feed' ) {
+            } elseif ( $el == 'feed' ) {
                 $this->feed_type = ATOM;
                 $this->feed_version = $attrs['version'];
                 $this->inchannel = true;
@@ -181,8 +179,7 @@ 
         if ( $el == 'channel' )
         {
             $this->inchannel = true;
-        }
-        elseif ($el == 'item' or $el == 'entry' )
+        } elseif ($el == 'item' or $el == 'entry' )
         {
             $this->initem = true;
             if ( isset($attrs['rdf:about']) ) {
@@ -198,9 +195,7 @@ 
             $el == 'textinput' )
         {
             $this->intextinput = true;
-        }
-
-        elseif (
+        } elseif (
             $this->feed_type == RSS and
             $this->current_namespace == '' and
             $el == 'image' )
@@ -244,8 +239,7 @@ 
             if ( isset($attrs['rel']) and $attrs['rel'] == 'alternate' )
             {
                 $link_el = 'link';
-            }
-            else {
+            } else {
                 $link_el = 'link_' . $attrs['rel'];
             }
 
@@ -264,8 +258,7 @@ 
         if ($this->feed_type == ATOM and $this->incontent)
         {
             $this->append_content( $text );
-        }
-        else {
+        } else {
             $current_el = join('_', array_reverse($this->stack));
             $this->append($current_el, $text);
         }
@@ -279,37 +272,30 @@ 
             $this->items[] = $this->current_item;
             $this->current_item = array();
             $this->initem = false;
-        }
-        elseif ($this->feed_type == RSS and $this->current_namespace == '' and $el == 'textinput' )
+        } elseif ($this->feed_type == RSS and $this->current_namespace == '' and $el == 'textinput' )
         {
             $this->intextinput = false;
-        }
-        elseif ($this->feed_type == RSS and $this->current_namespace == '' and $el == 'image' )
+        } elseif ($this->feed_type == RSS and $this->current_namespace == '' and $el == 'image' )
         {
             $this->inimage = false;
-        }
-        elseif ($this->feed_type == ATOM and in_array($el, $this->_CONTENT_CONSTRUCTS) )
+        } elseif ($this->feed_type == ATOM and in_array($el, $this->_CONTENT_CONSTRUCTS) )
         {
             $this->incontent = false;
-        }
-        elseif ($el == 'channel' or $el == 'feed' )
+        } elseif ($el == 'channel' or $el == 'feed' )
         {
             $this->inchannel = false;
-        }
-        elseif ($this->feed_type == ATOM and $this->incontent  ) {
+        } elseif ($this->feed_type == ATOM and $this->incontent  ) {
             // balance tags properly
             // note:  i don't think this is actually neccessary
             if ( $this->stack[0] == $el )
             {
                 $this->append_content("</$el>");
-            }
-            else {
+            } else {
                 $this->append_content("<$el />");
             }
 
             array_shift( $this->stack );
-        }
-        else {
+        } else {
             array_shift( $this->stack );
         }
 
@@ -328,8 +314,7 @@ 
     function append_content($text) {
         if ( $this->initem ) {
             $this->concat( $this->current_item[ $this->incontent ], $text );
-        }
-        elseif ( $this->inchannel ) {
+        } elseif ( $this->inchannel ) {
             $this->concat( $this->channel[ $this->incontent ], $text );
         }
     }
@@ -344,34 +329,27 @@ 
             if ( $this->initem ) {
                 $this->concat(
                     $this->current_item[ $this->current_namespace ][ $el ], $text);
-            }
-            elseif ($this->inchannel) {
+            } elseif ($this->inchannel) {
                 $this->concat(
                     $this->channel[ $this->current_namespace][ $el ], $text );
-            }
-            elseif ($this->intextinput) {
+            } elseif ($this->intextinput) {
                 $this->concat(
                     $this->textinput[ $this->current_namespace][ $el ], $text );
-            }
-            elseif ($this->inimage) {
+            } elseif ($this->inimage) {
                 $this->concat(
                     $this->image[ $this->current_namespace ][ $el ], $text );
             }
-        }
-        else {
+        } else {
             if ( $this->initem ) {
                 $this->concat(
                     $this->current_item[ $el ], $text);
-            }
-            elseif ($this->intextinput) {
+            } elseif ($this->intextinput) {
                 $this->concat(
                     $this->textinput[ $el ], $text );
-            }
-            elseif ($this->inimage) {
+            } elseif ($this->inimage) {
                 $this->concat(
                     $this->image[ $el ], $text );
-            }
-            elseif ($this->inchannel) {
+            } elseif ($this->inchannel) {
                 $this->concat(
                     $this->channel[ $el ], $text );
             }
@@ -385,10 +363,12 @@ 
             $this->channel['description'] = $this->channel['tagline'];
             for ( $i = 0; $i < count($this->items); $i++) {
                 $item = $this->items[$i];
-                if ( isset($item['summary']) )
-                    $item['description'] = $item['summary'];
-                if ( isset($item['atom_content']))
-                    $item['content']['encoded'] = $item['atom_content'];
+                if ( isset($item['summary']) ) {
+                                    $item['description'] = $item['summary'];
+                }
+                if ( isset($item['atom_content'])) {
+                                    $item['content']['encoded'] = $item['atom_content'];
+                }
 
                 $atom_date = (isset($item['issued']) ) ? $item['issued'] : @$item['modified'];
                 if ( $atom_date ) {
@@ -400,23 +380,23 @@ 
 
                 $this->items[$i] = $item;
             }
-        }
-        elseif ( $this->is_rss() ) {
+        } elseif ( $this->is_rss() ) {
             $this->channel['tagline'] = $this->channel['description'];
             for ( $i = 0; $i < count($this->items); $i++) {
                 $item = $this->items[$i];
-                if ( isset($item['description']))
-                    $item['summary'] = $item['description'];
-                if ( isset($item['content']['encoded'] ) )
-                    $item['atom_content'] = $item['content']['encoded'];
+                if ( isset($item['description'])) {
+                                    $item['summary'] = $item['description'];
+                }
+                if ( isset($item['content']['encoded'] ) ) {
+                                    $item['atom_content'] = $item['content']['encoded'];
+                }
 
                 if ( $this->is_rss() == '1.0' and isset($item['dc']['date']) ) {
                     $epoch = @parse_w3cdtf($item['dc']['date']);
                     if ($epoch and $epoch > 0) {
                         $item['date_timestamp'] = $epoch;
                     }
-                }
-                elseif ( isset($item['pubdate']) ) {
+                } elseif ( isset($item['pubdate']) ) {
                     $epoch = @strtotime($item['pubdate']);
                     if ($epoch > 0) {
                         $item['date_timestamp'] = $epoch;
@@ -432,8 +412,7 @@ 
     function is_rss () {
         if ( $this->feed_type == RSS ) {
             return $this->feed_version;
-        }
-        else {
+        } else {
             return false;
         }
     }
@@ -441,8 +420,7 @@ 
     function is_atom() {
         if ( $this->feed_type == ATOM ) {
             return $this->feed_version;
-        }
-        else {
+        } else {
             return false;
         }
     }
@@ -454,8 +432,7 @@ 
     function create_parser($source, $out_enc, $in_enc, $detect) {
         if ( substr(phpversion(),0,1) == 5) {
             $parser = $this->php5_create_parser($in_enc, $detect);
-        }
-        else {
+        } else {
             list($parser, $source) = $this->php4_create_parser($source, $in_enc, $detect);
         }
         if ($out_enc) {
@@ -479,8 +456,7 @@ 
         // by default php5 does a fine job of detecting input encodings
         if(!$detect && $in_enc) {
             return xml_parser_create($in_enc);
-        }
-        else {
+        } else {
             return xml_parser_create('');
         }
     }
@@ -509,8 +485,7 @@ 
             if (preg_match('/<?xml.*encoding=[\'"](.*?)[\'"].*?>/m', $source, $m)) {
                 $in_enc = strtoupper($m[1]);
                 $this->source_encoding = $in_enc;
-            }
-            else {
+            } else {
                 $in_enc = 'UTF-8';
             }
         }
@@ -555,8 +530,7 @@ 
         $enc = strtoupper($enc);
         if ( in_array($enc, $this->_KNOWN_ENCODINGS) ) {
             return $enc;
-        }
-        else {
+        } else {
             return false;
         }
     }
@@ -568,8 +542,7 @@ 
         }
         if ( MAGPIE_DEBUG ) {
             trigger_error( $errormsg, $lvl);
-        }
-        else {
+        } else {
             error_log( $errormsg, 0);
         }
 
@@ -604,8 +577,7 @@ 
 		$offset = 0;
 		if ( $match[10] == 'Z' ) {
 			# zulu time, aka GMT
-		}
-		else {
+		} else {
 			list( $tz_mod, $tz_hour, $tz_min ) =
 				array( $match[8], $match[9], $match[10]);
 
@@ -625,8 +597,7 @@ 
 		}
 		$epoch = $epoch + $offset;
 		return $epoch;
-	}
-	else {
+	} else {
 		return -1;
 	}
 }

class/xml.php

Doc Comments +4 added lines

@@ -93,6 +93,10 @@
         $this->version = $version;
     }
 
+    /**
+     * @param string $var
+     * @param FeedImage $val
+     */
     function setVar($var, $val, $encoding = false)
     {
 	    if(!empty($encoding)) $val = $this->convert_encoding($val);

Braces +7 added lines, -3 removed lines

@@ -95,7 +95,9 @@ 
 
     function setVar($var, $val, $encoding = false)
     {
-	    if(!empty($encoding)) $val = $this->convert_encoding($val);
+	    if(!empty($encoding)) {
+	    	$val = $this->convert_encoding($val);
+	    }
         $this->$var = $val;
     }
     
@@ -105,7 +107,7 @@ 
 		    foreach(array_keys($val) as $key){
 			    $val[$key] = $this->convert_encoding($val[$key]);
 		    }
-	    }else{
+	    } else{
 		    $val = XoopsLocal::convert_encoding($val, $this->encoding, _CHARSET);
 	    }
 	    return $val;
@@ -136,7 +138,9 @@ 
 
     function addItems(&$items)
     {
-	    if(!is_array($items) || count($items)==0) return;
+	    if(!is_array($items) || count($items)==0) {
+	    	return;
+	    }
 	    foreach($items as $item){
 		    $this->_addItem($item);
 	    }

class/xmlparser.php

Doc Comments -7 removed lines

@@ -53,13 +53,6 @@
      *  @param string $content           string containing the RSS to be parsed
      *
      *
-     *  @param string $output_encoding  output the parsed RSS in this character
-     *                                  set defaults to ISO-8859-1 as this is PHP's
-     *                                  default.
-     *
-     *  @param string $input_encoding   the character set of the incoming RSS source.
-     *                                  Leave blank and Magpie will try to figure it
-     *                                  out.
      *
      */
     function XmlParser ($content, $input_charset, $output_charset = _CHARSET, $tags = array())

Braces +21 added lines, -15 removed lines

@@ -85,8 +85,7 @@ 
         if ( $this->feed_type == ATOM ) {
 	        $this->feed_version = empty($this->feed_version)?"0.3":$this->feed_version;
             return $this->feed_version;
-        }
-        else {
+        } else {
             return false;
         }
     }
@@ -100,7 +99,9 @@ 
     	}
         for ( $i = 0; $i < count($this->items); $i++) {
 	        // ATOM time
-	        if($date = @$this->items[$i]['modified']) continue;
+	        if($date = @$this->items[$i]['modified']) {
+	        	continue;
+	        }
 	        if(empty($date)){
                 $date = @$this->items[$i]['updated'];
         	}
@@ -114,10 +115,11 @@ 
                 $date = @$this->items[$i]['created'];
         	}
 	        $this->items[$i]['modified'] = $date;
-        }
-        elseif($this->is_rss() != '1.0'):
+        } elseif($this->is_rss() != '1.0'):
         for ( $i = 0; $i < count($this->items); $i++) {
-	        if($date = @$this->items[$i]['pubdate']) continue;
+	        if($date = @$this->items[$i]['pubdate']) {
+	        	continue;
+	        }
 	        $this->items[$i]['pubdate'] = @$this->items[$i]['dc']['date'];
         }
         endif;
@@ -139,27 +141,31 @@ 
 	    }
 		
         for ( $i = 0; $i < count($this->items); $i++) {
-	        if($date_timestamp = @$this->items[$i]['date_timestamp']) continue;
+	        if($date_timestamp = @$this->items[$i]['date_timestamp']) {
+	        	continue;
+	        }
 	        if($date_timestamp = @$this->items[$i]['pubdate']){
                 $this->items[$i]['date_timestamp'] = $date_timestamp;
-            }elseif($date_timestamp = @$this->items[$i]['dc']['date']){
+            } elseif($date_timestamp = @$this->items[$i]['dc']['date']){
                 $this->items[$i]['date_timestamp'] = $date_timestamp;
-        	}else{
+        	} else{
 	        	$this->items[$i]['date_timestamp'] = time();
         	}
 	        if(!is_numeric($this->items[$i]['date_timestamp'])){
 		        if($date = parse_w3cdtf($this->items[$i]['date_timestamp'])) {
 		        	$this->items[$i]['date_timestamp'] = $date;
-	        	}elseif($date = strtotime($this->items[$i]['date_timestamp'])) {
+	        	} elseif($date = strtotime($this->items[$i]['date_timestamp'])) {
 		        	$this->items[$i]['date_timestamp'] = $date;
 	        	}
 	        }
 
 			/* remove to avoid redundant encoding conversion */
-            if ( isset($this->items[$i]['summary']) )
-                unset($this->items[$i]['summary']);
-            if ( isset($this->items[$i]['atom_content']))
-                unset($this->items[$i]['atom_content']);
+            if ( isset($this->items[$i]['summary']) ) {
+                            unset($this->items[$i]['summary']);
+            }
+            if ( isset($this->items[$i]['atom_content'])) {
+                            unset($this->items[$i]['atom_content']);
+            }
         }
 	    return;
     }
@@ -178,7 +184,7 @@ 
 		    foreach(array_keys($val) as $key){
 			    $val[$key] = $this->_encoding($val[$key]);
 		    }
-	    }else{
+	    } else{
 		    $val = XoopsLocal::convert_encoding($val, $this->charset_out, $this->charset_in);
 	    }
 	    return $val;

include/functions.ini.php

Doc Comments +6 added lines

@@ -37,11 +37,17 @@
 if(!defined("PLANET_FUNCTIONS_INI")):
 define("PLANET_FUNCTIONS_INI",1);
 
+/**
+ * @param string $name
+ */
 function planet_constant($name)
 {
 	return mod_constant($name);
 }
 
+/**
+ * @param string $name
+ */
 function planet_DB_prefix($name, $isRel = false)
 {
 	return mod_DB_prefix($name, $isRel);

Braces +6 added lines, -4 removed lines

@@ -61,7 +61,7 @@ 
 	
     if(is_object($GLOBALS["xoopsModule"]) && $GLOBALS["xoopsModule"]->getVar("dirname") == $GLOBALS["moddirname"]){
 	    $moduleConfig =& $GLOBALS["xoopsModuleConfig"];
-    }else{
+    } else{
 		$module_handler = &xoops_gethandler('module');
 		$module = $module_handler->getByDirname($GLOBALS["moddirname"]);
 	
@@ -82,12 +82,14 @@ 
 function planet_define_url_delimiter()
 {
 	if(defined("URL_DELIMITER")){
-		if(!in_array(URL_DELIMITER, array("?","/"))) die("Exit on security");
-	}else{
+		if(!in_array(URL_DELIMITER, array("?","/"))) {
+			die("Exit on security");
+		}
+	} else{
 		$moduleConfig = planet_load_config();
 		if(empty($moduleConfig["do_urw"])){
 			define("URL_DELIMITER", "?");
-		}else{
+		} else{
 			define("URL_DELIMITER", "/");
 		}
 	}

include/functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -98,7 +98,7 @@ 
  * @var mixed 	$pattern
  * @var mixed 	$replacement
  *
- * @return bool	true on success
+ * @return null|boolean	true on success
  */
 function planet_parse_class($class_string, $pattern="", $replacement="")
 {
@@ -131,7 +131,7 @@ 
  * @var mixed 	$pattern
  * @var mixed 	$replacement
  *
- * @return bool	true on success
+ * @return null|boolean	true on success
  */
 function planet_parse_function($function_string, $pattern="", $replacement="")
 {

Braces +50 added lines, -20 removed lines

@@ -28,7 +28,9 @@ 
 if (!defined('XOOPS_ROOT_PATH')){ exit(); }
 
 $current_path = __FILE__;
-if ( DIRECTORY_SEPARATOR != '/' ) $current_path = str_replace( strpos( $current_path, '\\\\', 2 ) ? '\\\\' : DIRECTORY_SEPARATOR, '/', $current_path);
+if ( DIRECTORY_SEPARATOR != '/' ) {
+	$current_path = str_replace( strpos( $current_path, '\\\\', 2 ) ? '\\\\' : DIRECTORY_SEPARATOR, '/', $current_path);
+}
 $url_arr = explode('/',strstr($current_path,'/modules/'));
 $GLOBALS["moddirname"] = $url_arr[2];
 
@@ -76,13 +78,13 @@ 
 			foreach($vars as $var){
 				if(is_numeric($var)){
 					$args_numeric[] = $var;
-				}elseif(false === strpos($var, "=")){
+				} elseif(false === strpos($var, "=")){
 					if(is_numeric(substr($var, 1))){
 						$args[$args_abb[strtolower($var{0})]] = intval(substr($var, 1));
-					}else{
+					} else{
 						$args_string[] = urldecode($var);
 					}
-				}else{
+				} else{
 					parse_str($var, $args);
 				}
 			}
@@ -102,7 +104,9 @@ 
  */
 function planet_parse_class($class_string, $pattern="", $replacement="")
 {
-	if(empty($class_string)) return;
+	if(empty($class_string)) {
+		return;
+	}
 	$patterns = array("/\[CLASS_PREFIX\]/");
 	$replacements = array(ucfirst(strtolower($GLOBALS["moddirname"])));
 	if(!empty($pattern) && !is_array($pattern) && !is_array($replacement)){
@@ -135,7 +139,9 @@ 
  */
 function planet_parse_function($function_string, $pattern="", $replacement="")
 {
-	if(empty($function_string)) return;
+	if(empty($function_string)) {
+		return;
+	}
 	$patterns = array("/\[DIRNAME\]/", "/\[VAR_PREFIX\]/");
 	$replacements = array($GLOBALS["moddirname"], $GLOBALS["VAR_PREFIX"]);
 	if(!empty($pattern) && !is_array($pattern) && !is_array($replacement)){
@@ -162,7 +168,9 @@ 
  */
 function planet_formatTimestamp($time, $format = "")
 {
-	if(empty($time)) return "";
+	if(empty($time)) {
+		return "";
+	}
 	return formatTimestamp($time, $format);
 }
 
@@ -172,7 +180,9 @@ 
  */
 function &planet_getUnameFromId( $userid, $usereal = 0, $linked = false )
 {
-	if(!is_array($userid))  $userid = array($userid);
+	if(!is_array($userid)) {
+		$userid = array($userid);
+	}
 	$users = mod_getUnameFromIds($userid, $usereal, $linked);
 
     return $users;
@@ -191,9 +201,13 @@ 
 	$myts =& MyTextSanitizer::getInstance();
 	$link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
     $links = array();
-    if(count($link_array)>0) foreach($link_array as $link){
+    if(count($link_array)>0) {
+    	foreach($link_array as $link){
 	    @list($url, $title) = array_map("trim",preg_split("/ /", $link, 2));
-	    if(empty($url)) continue;
+    }
+	    if(empty($url)) {
+	    	continue;
+	    }
 	    //if(empty($title)) $title = $url;
 	    $links[] = array("url"=>$url, "title"=>$myts->htmlSpecialChars($title));
     }
@@ -220,7 +234,9 @@ 
 {
 	$blog_handler =& xoops_getmodulehandler("blog", $GLOBALS["moddirname"]);
 	$blog_obj =& $blog_handler->get($article->getVar("blog_id"));
-	if(!$pattern = $blog_obj->getVar("blog_trackback")) return false;
+	if(!$pattern = $blog_obj->getVar("blog_trackback")) {
+		return false;
+	}
 	@list($pat, $rep) = array_map("trim",preg_split("#[\s]+#", $pattern));
 	$trackback_url = preg_replace("#".$pat."#", $rep, $article_obj->getVar("art_link"));
 	return planet_trackback($trackback_url, $article);
@@ -271,7 +287,7 @@ 
 		if($fp = fopen($debug_file, "a")){
 			fwrite($fp, $fr);
 			fclose($fp);
-		}else{
+		} else{
 		}
 	}
 	@fclose($fs);
@@ -301,9 +317,11 @@ 
 	$home = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/";
 	$rss2_url = XOOPS_URL."/modules/".$GLOBALS["moddirname"]."/xml.php".URL_DELIMITER."rss2.0/".$id;
 	
-	if ( !$client->query('weblogUpdates.extendedPing', $blogname, $home, $rss2_url ) ) // then try a normal ping
+	if ( !$client->query('weblogUpdates.extendedPing', $blogname, $home, $rss2_url ) ) {
+		// then try a normal ping
 		$client->query('weblogUpdates.ping', $blogname, $home);
-}
+	}
+	}
 
 /**
  * Function to respond to a trackback
@@ -350,8 +368,10 @@ 
 	if($isArray) {
 		$_value = ($value)?explode(",", $value):array();
 		$value = array();
-		if(count($_value)>0) foreach($_value as $string){
+		if(count($_value)>0) {
+			foreach($_value as $string){
 			$key = substr($string, 0, strpos($string,"|"));
+		}
 			$val = substr($string, (strpos($string,"|")+1));
 			$value[$key] = $val;
 		}
@@ -379,9 +399,15 @@ 
 
 function planet_remote_content($url)
 {
-    if($data = planet_fetch_snoopy($url)) return $data;
-    if($data = planet_fetch_CURL($url)) return $data;
-    if($data = planet_fetch_fopen($url)) return $data;
+    if($data = planet_fetch_snoopy($url)) {
+    	return $data;
+    }
+    if($data = planet_fetch_CURL($url)) {
+    	return $data;
+    }
+    if($data = planet_fetch_fopen($url)) {
+    	return $data;
+    }
    	return false;
 }
 
@@ -398,7 +424,9 @@ 
 
 function planet_fetch_CURL($url)
 {
-    if (!function_exists('curl_init') ) return false;
+    if (!function_exists('curl_init') ) {
+    	return false;
+    }
     $ch = curl_init();    // initialize curl handle
     curl_setopt($ch, CURLOPT_URL, $url); // set url to post to
     curl_setopt($ch, CURLOPT_FAILONERROR, 1);
@@ -412,7 +440,9 @@ 
 
 function planet_fetch_fopen($url)
 {
-	if(!$fp = @fopen ($url, 'r')) return false;
+	if(!$fp = @fopen ($url, 'r')) {
+		return false;
+	}
     $data = "";
     while (!feof($fp)) {
         $data .= fgets ($fp, 1024);

index.php

Braces +18 added lines, -8 removed lines

@@ -181,7 +181,7 @@ 
 
 if(!empty($blog_data)){
 	$blogs[$blog_data["id"]] = $blog_data["title"];
-}else{
+} else{
 	$blog_array = array();
 	foreach (array_keys($articles_obj) as $id) {
 		$blog_array[$articles_obj[$id]->getVar("blog_id")] = 1;
@@ -210,7 +210,7 @@ 
 		);
 		if(!empty($xoopsModuleConfig["display_summary"])){
 			$_article["content"] = $articles_obj[$id]->getSummary();
-		}else{
+		} else{
 			$_article["content"] = $articles_obj[$id]->getVar("art_content");
 		}
 	}
@@ -222,10 +222,18 @@ 
 if ( $count_article > $limit) {
 	include(XOOPS_ROOT_PATH."/class/pagenav.php");
 	$start_link = array();
-	if($sort) $start_link[] = "sort=".$sort;
-	if($category_id) $start_link[] = "category=".$category_id;
-	if($blog_id) $start_link[] = "blog=".$blog_id;
-	if($list) $start_link[] = "list=".$list;
+	if($sort) {
+		$start_link[] = "sort=".$sort;
+	}
+	if($category_id) {
+		$start_link[] = "category=".$category_id;
+	}
+	if($blog_id) {
+		$start_link[] = "blog=".$blog_id;
+	}
+	if($list) {
+		$start_link[] = "list=".$list;
+	}
 	$nav = new XoopsPageNav($count_article, $limit, $start, "start", implode("&", $start_link));
 	$pagenav = $nav->renderNav(4);
 } else {
@@ -272,7 +280,7 @@ 
 
 $xoopsTpl -> assign("user_level", !is_object($xoopsUser)?0:($xoopsUser->isAdmin()?2:1));
 if(empty($xoopsModuleConfig["anonymous_rate"]) && !is_object($xoopsUser)){
-}elseif($blog_id>0){
+} elseif($blog_id>0){
 	$xoopsTpl -> assign("canrate", 1);
 }
 
@@ -284,7 +292,9 @@ 
 $valid_sorts = array("views"=>planet_constant("MD_VIEWS"), "rating"=>planet_constant("MD_RATING"), "time"=>planet_constant("MD_TIME"), "default"=>planet_constant("MD_DEFAULT"));
 $sortlinks = array();
 foreach($valid_sorts as $val=>$name){
-	if($val == $sort) continue;
+	if($val == $sort) {
+		continue;
+	}
 	$sortlinks[] = "<a href=\"".$sort_link."/".$val."\">".$name."</a>";
 }
 $xoopsTpl -> assign("link_sort", implode(" | ", $sortlinks));