mambax7 / planet

language/english/blocks.php

Spacing +20 added lines, -20 removed lines

@@ -11,28 +11,28 @@
     $current_path = str_replace(strpos($current_path, "\\\\", 2) ? "\\\\" : DIRECTORY_SEPARATOR, '/', $current_path);
 }
 $url_arr = explode('/', strstr($current_path, '/modules/'));
-include XOOPS_ROOT_PATH . '/modules/' . $url_arr[2] . '/include/vars.php';
+include XOOPS_ROOT_PATH.'/modules/'.$url_arr[2].'/include/vars.php';
 
-if (defined($GLOBALS['VAR_PREFIXU'] . '_LANG_EN_BLOCKS')) {
+if (defined($GLOBALS['VAR_PREFIXU'].'_LANG_EN_BLOCKS')) {
     return;
 }
-define($GLOBALS['VAR_PREFIXU'] . '_LANG_EN_BLOCKS', 1);
+define($GLOBALS['VAR_PREFIXU'].'_LANG_EN_BLOCKS', 1);
 
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE', 'Type');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_TIME', 'Publish time');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_VIEWS', 'Views');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_RATES', 'Rate times');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_RATING', 'Rating');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_RANDOM', 'Random');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_BOOKMARKS', 'Bookmarks');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TYPE_FEATURED', 'Featured');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_ITEMS', 'Item count');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TITLE_LENGTH', 'Title length');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TIMEFORMAT', 'Time format');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TIMEFORMAT_CUSTOM', 'Custom');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_SUMMARY_LENGTH', 'Show summary length');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_SHOWDESC', 'Show description');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_CATEGORYLIST', 'Allowed categories');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE', 'Type');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_TIME', 'Publish time');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_VIEWS', 'Views');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_RATES', 'Rate times');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_RATING', 'Rating');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_RANDOM', 'Random');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_BOOKMARKS', 'Bookmarks');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TYPE_FEATURED', 'Featured');
+define($GLOBALS['VAR_PREFIXU'].'_MB_ITEMS', 'Item count');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TITLE_LENGTH', 'Title length');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TIMEFORMAT', 'Time format');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TIMEFORMAT_CUSTOM', 'Custom');
+define($GLOBALS['VAR_PREFIXU'].'_MB_SUMMARY_LENGTH', 'Show summary length');
+define($GLOBALS['VAR_PREFIXU'].'_MB_SHOWDESC', 'Show description');
+define($GLOBALS['VAR_PREFIXU'].'_MB_CATEGORYLIST', 'Allowed categories');
 
-define($GLOBALS['VAR_PREFIXU'] . '_MB_AUTHOR', 'Author');
-define($GLOBALS['VAR_PREFIXU'] . '_MB_TIME', 'Time');
+define($GLOBALS['VAR_PREFIXU'].'_MB_AUTHOR', 'Author');
+define($GLOBALS['VAR_PREFIXU'].'_MB_TIME', 'Time');

class/class-IXR.php

Doc Comments +6 added lines, -6 removed lines

@@ -170,7 +170,7 @@ 
 
     /**
      * IXR_Message constructor.
-     * @param $message
+     * @param string|boolean $message
      */
     public function __construct($message)
     {
@@ -897,7 +897,7 @@ 
 
     /**
      * IXR_Base64 constructor.
-     * @param $data
+     * @param string $data
      */
     public function __construct($data)
     {
@@ -939,10 +939,10 @@ 
     }
 
     /**
-     * @param $method
-     * @param $callback
-     * @param $args
-     * @param $help
+     * @param string $method
+     * @param string $callback
+     * @param string[] $args
+     * @param string $help
      */
     public function addCallback($method, $callback, $args, $help)
     {

Spacing +48 added lines, -48 removed lines

@@ -36,7 +36,7 @@ 
             }
         }
         if ($type === 'array') {
-            for ($i = 0, $j = count($this->data); $i < $j; ++$i) {
+            for ($i = 0, $j = count($this->data); $i<$j; ++$i) {
                 $this->data[$i] = new IXR_Value($this->data[$i]);
             }
         }
@@ -75,7 +75,7 @@ 
         /* We have an array - is it an array or a struct ? */
         if ($this->isStruct($this->data)) {
             return 'struct';
-        } else {
+        }else {
             return 'array';
         }
     }
@@ -88,31 +88,31 @@ 
         /* Return XML for this value */
         switch ($this->type) {
             case 'boolean':
-                return '<boolean>' . ($this->data ? '1' : '0') . '</boolean>';
+                return '<boolean>'.($this->data ? '1' : '0').'</boolean>';
                 break;
             case 'int':
-                return '<int>' . $this->data . '</int>';
+                return '<int>'.$this->data.'</int>';
                 break;
             case 'double':
-                return '<double>' . $this->data . '</double>';
+                return '<double>'.$this->data.'</double>';
                 break;
             case 'string':
-                return '<string>' . htmlspecialchars($this->data) . '</string>';
+                return '<string>'.htmlspecialchars($this->data).'</string>';
                 break;
             case 'array':
-                $return = '<array><data>' . "\n";
+                $return = '<array><data>'."\n";
                 foreach ($this->data as $item) {
-                    $return .= '  <value>' . $item->getXml() . "</value>\n";
+                    $return .= '  <value>'.$item->getXml()."</value>\n";
                 }
                 $return .= '</data></array>';
 
                 return $return;
                 break;
             case 'struct':
-                $return = '<struct>' . "\n";
+                $return = '<struct>'."\n";
                 foreach ($this->data as $name => $value) {
                     $return .= "  <member><name>$name</name><value>";
-                    $return .= $value->getXml() . "</value></member>\n";
+                    $return .= $value->getXml()."</value></member>\n";
                 }
                 $return .= '</struct>';
 
@@ -136,7 +136,7 @@ 
         /* Nasty function to check if an array is a struct or not */
         $expected = 0;
         foreach ($array as $key => $value) {
-            if ((string)$key != (string)$expected) {
+            if ((string) $key != (string) $expected) {
                 return true;
             }
             ++$expected;
@@ -152,15 +152,15 @@ 
 class IXR_Message
 {
     public $message;
-    public $messageType;  // methodCall / methodResponse / fault
+    public $messageType; // methodCall / methodResponse / fault
     public $faultCode;
     public $faultString;
     public $methodName;
     public $params;
     // Current variable stacks
-    public $_arraystructs      = [];   // The stack used to keep track of the current array/struct
+    public $_arraystructs      = []; // The stack used to keep track of the current array/struct
     public $_arraystructstypes = []; // Stack keeping track of if things are structs or array
-    public $_currentStructName = [];  // A stack as well
+    public $_currentStructName = []; // A stack as well
     public $_param;
     public $_value;
     public $_currentTag;
@@ -183,7 +183,7 @@ 
     public function parse()
     {
         // first remove the XML declaration
-        $this->message = preg_replace('/<\?xml(.*)?\?' . '>/', '', $this->message);
+        $this->message = preg_replace('/<\?xml(.*)?\?'.'>/', '', $this->message);
         if (trim($this->message) == '') {
             return false;
         }
@@ -256,17 +256,17 @@ 
         switch ($tag) {
             case 'int':
             case 'i4':
-                $value                     = (int)trim($this->_currentTagContents);
+                $value                     = (int) trim($this->_currentTagContents);
                 $this->_currentTagContents = '';
                 $valueFlag                 = true;
                 break;
             case 'double':
-                $value                     = (double)trim($this->_currentTagContents);
+                $value                     = (double) trim($this->_currentTagContents);
                 $this->_currentTagContents = '';
                 $valueFlag                 = true;
                 break;
             case 'string':
-                $value                     = (string)trim($this->_currentTagContents);
+                $value                     = (string) trim($this->_currentTagContents);
                 $this->_currentTagContents = '';
                 $valueFlag                 = true;
                 break;
@@ -279,13 +279,13 @@ 
             case 'value':
                 // "If no type is indicated, the type is string."
                 if (trim($this->_currentTagContents) != '') {
-                    $value                     = (string)$this->_currentTagContents;
+                    $value                     = (string) $this->_currentTagContents;
                     $this->_currentTagContents = '';
                     $valueFlag                 = true;
                 }
                 break;
             case 'boolean':
-                $value                     = (boolean)trim($this->_currentTagContents);
+                $value                     = (boolean) trim($this->_currentTagContents);
                 $this->_currentTagContents = '';
                 $valueFlag                 = true;
                 break;
@@ -319,16 +319,16 @@ 
                 $value = trim($value);
             }
             */
-            if (count($this->_arraystructs) > 0) {
+            if (count($this->_arraystructs)>0) {
                 // Add value to struct or array
-                if ($this->_arraystructstypes[count($this->_arraystructstypes) - 1] === 'struct') {
+                if ($this->_arraystructstypes[count($this->_arraystructstypes)-1] === 'struct') {
                     // Add to struct
-                    $this->_arraystructs[count($this->_arraystructs) - 1][$this->_currentStructName[count($this->_currentStructName) - 1]] = $value;
-                } else {
+                    $this->_arraystructs[count($this->_arraystructs)-1][$this->_currentStructName[count($this->_currentStructName)-1]] = $value;
+                }else {
                     // Add to array
-                    $this->_arraystructs[count($this->_arraystructs) - 1][] = $value;
+                    $this->_arraystructs[count($this->_arraystructs)-1][] = $value;
                 }
-            } else {
+            }else {
                 // Just add as a paramater
                 $this->params[] = $value;
             }
@@ -413,7 +413,7 @@ 
     public function call($methodname, $args)
     {
         if (!$this->hasMethod($methodname)) {
-            return new IXR_Error(-32601, 'server error. requested method ' . $methodname . ' does not exist.');
+            return new IXR_Error(-32601, 'server error. requested method '.$methodname.' does not exist.');
         }
         $method = $this->callbacks[$methodname];
         // Perform the callback and send the response
@@ -426,18 +426,18 @@ 
             // It's a class method - check it exists
             $method = substr($method, 5);
             if (!method_exists($this, $method)) {
-                return new IXR_Error(-32601, 'server error. requested class method "' . $method . '" does not exist.');
+                return new IXR_Error(-32601, 'server error. requested class method "'.$method.'" does not exist.');
             }
             // Call the method
             $result = $this->$method($args);
-        } else {
+        }else {
             // It's a function - does it exist?
             if (is_array($method)) {
                 if (!method_exists($method[0], $method[1])) {
-                    return new IXR_Error(-32601, 'server error. requested object method "' . $method[1] . '" does not exist.');
+                    return new IXR_Error(-32601, 'server error. requested object method "'.$method[1].'" does not exist.');
                 }
             } elseif (!function_exists($method)) {
-                return new IXR_Error(-32601, 'server error. requested function "' . $method . '" does not exist.');
+                return new IXR_Error(-32601, 'server error. requested function "'.$method.'" does not exist.');
             }
             // Call the function
             $result = call_user_func($method, $args);
@@ -464,12 +464,12 @@ 
      */
     public function output($xml)
     {
-        $xml    = '<?xml version="1.0"?>' . "\n" . $xml;
+        $xml    = '<?xml version="1.0"?>'."\n".$xml;
         $length = strlen($xml);
         header('Connection: close');
-        header('Content-Length: ' . $length);
+        header('Content-Length: '.$length);
         header('Content-Type: text/xml');
-        header('Date: ' . date('r'));
+        header('Date: '.date('r'));
         echo $xml;
         exit;
     }
@@ -542,7 +542,7 @@ 
             $params = $call['params'];
             if ($method === 'system.multicall') {
                 $result = new IXR_Error(-32600, 'Recursive calls to system.multicall are forbidden');
-            } else {
+            }else {
                 $result = $this->call($method, $params);
             }
             if (is_a($result, 'IXR_Error')) {
@@ -550,7 +550,7 @@ 
                     'faultCode'   => $result->code,
                     'faultString' => $result->message
                 ];
-            } else {
+            }else {
                 $return[] = [$result];
             }
         }
@@ -586,7 +586,7 @@ 
 EOD;
         foreach ($this->args as $arg) {
             $this->xml .= '<param><value>';
-            $v         = new IXR_Value($arg);
+            $v = new IXR_Value($arg);
             $this->xml .= $v->getXml();
             $this->xml .= "</value></param>\n";
         }
@@ -647,7 +647,7 @@ 
             if (!$this->path) {
                 $this->path = '/';
             }
-        } else {
+        }else {
             $this->server  = $server;
             $this->path    = $path;
             $this->port    = $port;
@@ -675,7 +675,7 @@ 
         $request .= $xml;
         // Now send the request
         if ($this->debug) {
-            echo '<pre>' . htmlspecialchars($request) . "\n</pre>\n\n";
+            echo '<pre>'.htmlspecialchars($request)."\n</pre>\n\n";
         }
         $fp = @fsockopen($this->server, $this->port, $errno, $errstr, $this->timeout);
         if (!$fp) {
@@ -702,11 +702,11 @@ 
                 $gettingHeaders = false;
             }
             if (!$gettingHeaders) {
-                $contents .= trim($line) . "\n";
+                $contents .= trim($line)."\n";
             }
         }
         if ($this->debug) {
-            echo '<pre>' . htmlspecialchars($contents) . "\n</pre>\n\n";
+            echo '<pre>'.htmlspecialchars($contents)."\n</pre>\n\n";
         }
         // Now parse what we've got back
         $this->message = new IXR_Message($contents);
@@ -831,7 +831,7 @@ 
         // $time can be a PHP timestamp or an ISO one
         if (is_numeric($time)) {
             $this->parseTimestamp($time);
-        } else {
+        }else {
             $this->parseIso($time);
         }
     }
@@ -868,7 +868,7 @@ 
      */
     public function getIso()
     {
-        return $this->year . $this->month . $this->day . 'T' . $this->hour . ':' . $this->minute . ':' . $this->second . $this->timezone;
+        return $this->year.$this->month.$this->day.'T'.$this->hour.':'.$this->minute.':'.$this->second.$this->timezone;
     }
 
     /**
@@ -876,7 +876,7 @@ 
      */
     public function getXml()
     {
-        return '<dateTime.iso8601>' . $this->getIso() . '</dateTime.iso8601>';
+        return '<dateTime.iso8601>'.$this->getIso().'</dateTime.iso8601>';
     }
 
     /**
@@ -909,7 +909,7 @@ 
      */
     public function getXml()
     {
-        return '<base64>' . base64_encode($this->data) . '</base64>';
+        return '<base64>'.base64_encode($this->data).'</base64>';
     }
 }
 
@@ -964,7 +964,7 @@ 
         }
         // Over-rides default call method, adds signature check
         if (!$this->hasMethod($methodname)) {
-            return new IXR_Error(-32601, 'server error. requested method "' . $this->message->methodName . '" not specified.');
+            return new IXR_Error(-32601, 'server error. requested method "'.$this->message->methodName.'" not specified.');
         }
         $method     = $this->callbacks[$methodname];
         $signature  = $this->signatures[$methodname];
@@ -977,7 +977,7 @@ 
         // Check the argument types
         $ok         = true;
         $argsbackup = $args;
-        for ($i = 0, $j = count($args); $i < $j; ++$i) {
+        for ($i = 0, $j = count($args); $i<$j; ++$i) {
             $arg  = array_shift($args);
             $type = array_shift($signature);
             switch ($type) {
@@ -1027,7 +1027,7 @@ 
     public function methodSignature($method)
     {
         if (!$this->hasMethod($method)) {
-            return new IXR_Error(-32601, 'server error. requested method "' . $method . '" not specified.');
+            return new IXR_Error(-32601, 'server error. requested method "'.$method.'" not specified.');
         }
         // We should be returning an array of types
         $types  = $this->signatures[$method];

class/feedcreator.class.php

Doc Comments +7 added lines, -6 removed lines

@@ -369,7 +369,7 @@ 
     public $_feed;
 
     /**
-     * @param $format
+     * @param string $format
      */
     public function _setFormat($format)
     {
@@ -566,7 +566,7 @@ 
     /**
      * Adds an FeedItem to the feed.
      *
-     * @param $item
+     * @param FeedItem $item
      * @internal param FeedItem $object $item The FeedItem to add to the feed.
      * @access   public
      */
@@ -585,6 +585,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
      */
     public function iTrunc($string, $length)
@@ -696,7 +697,7 @@ 
     /**
      * @since  1.4
      * @access private
-     * @param $filename
+     * @param string $filename
      */
     public function _redirect($filename)
     {
@@ -853,7 +854,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
      */
     public function rfc822()
     {
@@ -869,7 +870,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
      */
     public function iso8601()
     {
@@ -999,7 +1000,7 @@ 
     /**
      * Sets this RSS feed's version number.
      * @access private
-     * @param $version
+     * @param string $version
      */
     public function _setRSSVersion($version)
     {

Spacing +170 added lines, -170 removed lines

@@ -342,11 +342,11 @@ 
         if (!$this->rawFieldContent) {
             $result = '';
         } elseif ($this->syndicateHtml) {
-            $result = '<![CDATA[' . $this->rawFieldContent . ']]>';
-        } else {
+            $result = '<![CDATA['.$this->rawFieldContent.']]>';
+        }else {
             if ($this->truncSize && is_int($this->truncSize)) {
                 $result = FeedCreator::iTrunc(htmlspecialchars($this->rawFieldContent), $this->truncSize);
-            } else {
+            }else {
                 $result = htmlspecialchars($this->rawFieldContent);
             }
         }
@@ -589,29 +589,29 @@ 
      */
     public function iTrunc($string, $length)
     {
-        if (strlen($string) <= $length) {
+        if (strlen($string)<=$length) {
             return $string;
         }
 
         $pos = strrpos($string, '.');
-        if ($pos >= $length - 4) {
-            $string = substr($string, 0, $length - 4);
+        if ($pos>=$length-4) {
+            $string = substr($string, 0, $length-4);
             $pos    = strrpos($string, '.');
         }
-        if ($pos >= $length * 0.4) {
-            return substr($string, 0, $pos + 1) . ' ...';
+        if ($pos>=$length*0.4) {
+            return substr($string, 0, $pos+1).' ...';
         }
 
         $pos = strrpos($string, ' ');
-        if ($pos >= $length - 4) {
-            $string = substr($string, 0, $length - 4);
+        if ($pos>=$length-4) {
+            $string = substr($string, 0, $length-4);
             $pos    = strrpos($string, ' ');
         }
-        if ($pos >= $length * 0.4) {
-            return substr($string, 0, $pos) . ' ...';
+        if ($pos>=$length*0.4) {
+            return substr($string, 0, $pos).' ...';
         }
 
-        return substr($string, 0, $length - 4) . ' ...';
+        return substr($string, 0, $length-4).' ...';
     }
 
     /**
@@ -621,7 +621,7 @@ 
      */
     public function _createGeneratorComment()
     {
-        return "<!-- generator=\"" . FEEDCREATOR_VERSION . "\" -->\n";
+        return "<!-- generator=\"".FEEDCREATOR_VERSION."\" -->\n";
     }
 
     /**
@@ -638,7 +638,7 @@ 
         $ae = '';
         if (is_array($elements)) {
             foreach ($elements as $key => $value) {
-                $ae .= $indentString . "<$key>$value</$key>\n";
+                $ae .= $indentString."<$key>$value</$key>\n";
             }
         }
 
@@ -652,10 +652,10 @@ 
     {
         $xml = '';
         if ($this->cssStyleSheet) {
-            $xml .= "<?xml-stylesheet href=\"" . $this->cssStyleSheet . "\" type=\"text/css\"?>\n";
+            $xml .= "<?xml-stylesheet href=\"".$this->cssStyleSheet."\" type=\"text/css\"?>\n";
         }
         if ($this->xslStyleSheet) {
-            $xml .= "<?xml-stylesheet href=\"" . $this->xslStyleSheet . "\" type=\"text/xsl\"?>\n";
+            $xml .= "<?xml-stylesheet href=\"".$this->xslStyleSheet."\" type=\"text/xsl\"?>\n";
         }
 
         return $xml;
@@ -690,7 +690,7 @@ 
     {
         $fileInfo = pathinfo($_SERVER['PHP_SELF']);
 
-        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension']) + 1)) . '.xml';
+        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension'])+1)).'.xml';
     }
 
     /**
@@ -713,8 +713,8 @@ 
         // HTTP redirect, some feed readers' simple HTTP implementations don't follow it
         //Header("Location: ".$filename);
 
-        header('Content-Type: ' . $this->contentType . '; charset=' . $this->encoding . '; filename=' . basename($filename));
-        header('Content-Disposition: inline; filename=' . basename($filename));
+        header('Content-Type: '.$this->contentType.'; charset='.$this->encoding.'; filename='.basename($filename));
+        header('Content-Disposition: inline; filename='.basename($filename));
         readfile($filename, 'r');
         die();
     }
@@ -736,7 +736,7 @@ 
         if ($filename == '') {
             $filename = $this->_generateFilename();
         }
-        if (file_exists($filename) && (time() - filemtime($filename) < $timeout)) {
+        if (file_exists($filename) && (time()-filemtime($filename)<$timeout)) {
             $this->_redirect($filename);
         }
     }
@@ -762,7 +762,7 @@ 
             if ($displayContents) {
                 $this->_redirect($filename);
             }
-        } else {
+        }else {
             echo '<br><b>Error creating feed file, please check write permissions.</b><br>';
         }
     }
@@ -795,7 +795,7 @@ 
             return;
         }
         if (preg_match("~(?:(?:Mon|Tue|Wed|Thu|Fri|Sat|Sun),\\s+)?(\\d{1,2})\\s+([a-zA-Z]{3})\\s+(\\d{4})\\s+(\\d{2}):(\\d{2}):(\\d{2})\\s+(.*)~", $dateString, $matches)) {
-            $months     = [
+            $months = [
                 'Jan' => 1,
                 'Feb' => 2,
                 'Mar' => 3,
@@ -811,15 +811,15 @@ 
             ];
             $this->unix = mktime($matches[4], $matches[5], $matches[6], $months[$matches[2]], $matches[1], $matches[3]);
             if (substr($matches[7], 0, 1) == '+' || substr($matches[7], 0, 1) == '-') {
-                $tzOffset = (substr($matches[7], 0, 3) * 60 + substr($matches[7], -2)) * 60;
-            } else {
+                $tzOffset = (substr($matches[7], 0, 3)*60+substr($matches[7], -2))*60;
+            }else {
                 if (strlen($matches[7]) == 1) {
                     $oneHour = 3600;
                     $ord     = ord($matches[7]);
-                    if ($ord < ord('M')) {
-                        $tzOffset = (ord('A') - $ord - 1) * $oneHour;
-                    } elseif ($ord >= ord('M') && $matches[7] !== 'Z') {
-                        $tzOffset = ($ord - ord('M')) * $oneHour;
+                    if ($ord<ord('M')) {
+                        $tzOffset = (ord('A')-$ord-1)*$oneHour;
+                    } elseif ($ord>=ord('M') && $matches[7] !== 'Z') {
+                        $tzOffset = ($ord-ord('M'))*$oneHour;
                     } elseif ($matches[7] === 'Z') {
                         $tzOffset = 0;
                     }
@@ -837,8 +837,8 @@ 
         if (preg_match("~(\\d{4})-(\\d{2})-(\\d{2})T(\\d{2}):(\\d{2}):(\\d{2})(.*)~", $dateString, $matches)) {
             $this->unix = mktime($matches[4], $matches[5], $matches[6], $matches[2], $matches[3], $matches[1]);
             if (substr($matches[7], 0, 1) == '+' || substr($matches[7], 0, 1) == '-') {
-                $tzOffset = (substr($matches[7], 0, 3) * 60 + substr($matches[7], -2)) * 60;
-            } else {
+                $tzOffset = (substr($matches[7], 0, 3)*60+substr($matches[7], -2))*60;
+            }else {
                 if ($matches[7] === 'Z') {
                     $tzOffset = 0;
                 }
@@ -860,7 +860,7 @@ 
         //return gmdate("r",$this->unix);
         $date = gmdate('D, d M Y H:i:s', $this->unix);
         if (TIME_ZONE != '') {
-            $date .= ' ' . str_replace(':', '', TIME_ZONE);
+            $date .= ' '.str_replace(':', '', TIME_ZONE);
         }
 
         return $date;
@@ -874,7 +874,7 @@ 
     public function iso8601()
     {
         $date = gmdate("Y-m-d\TH:i:sO", $this->unix);
-        $date = substr($date, 0, 22) . ':' . substr($date, -2);
+        $date = substr($date, 0, 22).':'.substr($date, -2);
         if (TIME_ZONE != '') {
             $date = str_replace('+00:00', TIME_ZONE, $date);
         }
@@ -909,7 +909,7 @@ 
      */
     public function createFeed()
     {
-        $feed = "<?xml version=\"1.0\" encoding=\"" . $this->encoding . "\"?>\n";
+        $feed = "<?xml version=\"1.0\" encoding=\"".$this->encoding."\"?>\n";
         $feed .= $this->_createGeneratorComment();
         if ($this->cssStyleSheet == '') {
             $cssStyleSheet = 'http://www.w3.org/2000/08/w3c-synd/style.css';
@@ -920,49 +920,49 @@ 
         $feed .= "    xmlns:rdf=\"http://www.w3.org/1999/02/22-rdf-syntax-ns#\"\n";
         $feed .= "    xmlns:slash=\"http://purl.org/rss/1.0/modules/slash/\"\n";
         $feed .= "    xmlns:dc=\"http://purl.org/dc/elements/1.1/\">\n";
-        $feed .= "    <channel rdf:about=\"" . $this->syndicationURL . "\">\n";
-        $feed .= '        <title>' . htmlspecialchars($this->title) . "</title>\n";
-        $feed .= '        <description>' . htmlspecialchars($this->description) . "</description>\n";
-        $feed .= '        <link>' . $this->link . "</link>\n";
+        $feed .= "    <channel rdf:about=\"".$this->syndicationURL."\">\n";
+        $feed .= '        <title>'.htmlspecialchars($this->title)."</title>\n";
+        $feed .= '        <description>'.htmlspecialchars($this->description)."</description>\n";
+        $feed .= '        <link>'.$this->link."</link>\n";
         if ($this->image != null) {
-            $feed .= "        <image rdf:resource=\"" . $this->image->url . "\">\n";
+            $feed .= "        <image rdf:resource=\"".$this->image->url."\">\n";
         }
-        $now  = new FeedDate();
-        $feed .= '       <dc:date>' . htmlspecialchars($now->iso8601()) . "</dc:date>\n";
+        $now = new FeedDate();
+        $feed .= '       <dc:date>'.htmlspecialchars($now->iso8601())."</dc:date>\n";
         $feed .= "        <items>\n";
         $feed .= "            <rdf:Seq>\n";
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
-            $feed .= "                <rdf:li rdf:resource=\"" . htmlspecialchars($this->items[$i]->link) . "\">\n";
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
+            $feed .= "                <rdf:li rdf:resource=\"".htmlspecialchars($this->items[$i]->link)."\">\n";
         }
         $feed .= "            </rdf:Seq>\n";
         $feed .= "        </items>\n";
         $feed .= "    </channel>\n";
         if ($this->image != null) {
-            $feed .= "    <image rdf:about=\"" . $this->image->url . "\">\n";
-            $feed .= '        <title>' . $this->image->title . "</title>\n";
-            $feed .= '        <link>' . $this->image->link . "</link>\n";
-            $feed .= '        <url>' . $this->image->url . "</url>\n";
+            $feed .= "    <image rdf:about=\"".$this->image->url."\">\n";
+            $feed .= '        <title>'.$this->image->title."</title>\n";
+            $feed .= '        <link>'.$this->image->link."</link>\n";
+            $feed .= '        <url>'.$this->image->url."</url>\n";
             $feed .= "    </image>\n";
         }
         $feed .= $this->_createAdditionalElements($this->additionalElements, '    ');
 
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
-            $feed .= "    <item rdf:about=\"" . htmlspecialchars($this->items[$i]->link) . "\">\n";
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
+            $feed .= "    <item rdf:about=\"".htmlspecialchars($this->items[$i]->link)."\">\n";
             //$feed.= "        <dc:type>Posting</dc:type>\n";
             $feed .= "        <dc:format>text/html</dc:format>\n";
             if ($this->items[$i]->date != null) {
                 $itemDate = new FeedDate($this->items[$i]->date);
-                $feed     .= '        <dc:date>' . htmlspecialchars($itemDate->iso8601()) . "</dc:date>\n";
+                $feed .= '        <dc:date>'.htmlspecialchars($itemDate->iso8601())."</dc:date>\n";
             }
             if ($this->items[$i]->source != '') {
-                $feed .= '        <dc:source>' . htmlspecialchars($this->items[$i]->source) . "</dc:source>\n";
+                $feed .= '        <dc:source>'.htmlspecialchars($this->items[$i]->source)."</dc:source>\n";
             }
             if ($this->items[$i]->author != '') {
-                $feed .= '        <dc:creator>' . htmlspecialchars($this->items[$i]->author) . "</dc:creator>\n";
+                $feed .= '        <dc:creator>'.htmlspecialchars($this->items[$i]->author)."</dc:creator>\n";
             }
-            $feed .= '        <title>' . htmlspecialchars(strip_tags(strtr($this->items[$i]->title, "\n\r", '  '))) . "</title>\n";
-            $feed .= '        <link>' . htmlspecialchars($this->items[$i]->link) . "</link>\n";
-            $feed .= '        <description>' . htmlspecialchars($this->items[$i]->description) . "</description>\n";
+            $feed .= '        <title>'.htmlspecialchars(strip_tags(strtr($this->items[$i]->title, "\n\r", '  ')))."</title>\n";
+            $feed .= '        <link>'.htmlspecialchars($this->items[$i]->link)."</link>\n";
+            $feed .= '        <description>'.htmlspecialchars($this->items[$i]->description)."</description>\n";
             $feed .= $this->_createAdditionalElements($this->items[$i]->additionalElements, '        ');
             $feed .= "    </item>\n";
         }
@@ -1013,79 +1013,79 @@ 
      */
     public function createFeed()
     {
-        $feed                       = "<?xml version=\"1.0\" encoding=\"" . $this->encoding . "\"?>\n";
+        $feed = "<?xml version=\"1.0\" encoding=\"".$this->encoding."\"?>\n";
         $feed                       .= $this->_createGeneratorComment();
         $feed                       .= $this->_createStylesheetReferences();
-        $feed                       .= "<rss version=\"" . $this->RSSVersion . "\">\n";
+        $feed                       .= "<rss version=\"".$this->RSSVersion."\">\n";
         $feed                       .= "    <channel>\n";
-        $feed                       .= '        <title>' . FeedCreator::iTrunc(htmlspecialchars($this->title), 100) . "</title>\n";
+        $feed                       .= '        <title>'.FeedCreator::iTrunc(htmlspecialchars($this->title), 100)."</title>\n";
         $this->descriptionTruncSize = 500;
-        $feed                       .= '        <description>' . $this->getDescription() . "</description>\n";
-        $feed                       .= '        <link>' . $this->link . "</link>\n";
+        $feed                       .= '        <description>'.$this->getDescription()."</description>\n";
+        $feed                       .= '        <link>'.$this->link."</link>\n";
         $now                        = new FeedDate();
-        $feed                       .= '        <lastBuildDate>' . htmlspecialchars($now->rfc822()) . "</lastBuildDate>\n";
-        $feed                       .= '        <generator>' . FEEDCREATOR_VERSION . "</generator>\n";
+        $feed                       .= '        <lastBuildDate>'.htmlspecialchars($now->rfc822())."</lastBuildDate>\n";
+        $feed                       .= '        <generator>'.FEEDCREATOR_VERSION."</generator>\n";
 
         if ($this->image != null) {
             $feed .= "        <image>\n";
-            $feed .= '            <url>' . $this->image->url . "</url>\n";
-            $feed .= '            <title>' . FeedCreator::iTrunc(htmlspecialchars($this->image->title), 100) . "</title>\n";
-            $feed .= '            <link>' . $this->image->link . "</link>\n";
+            $feed .= '            <url>'.$this->image->url."</url>\n";
+            $feed .= '            <title>'.FeedCreator::iTrunc(htmlspecialchars($this->image->title), 100)."</title>\n";
+            $feed .= '            <link>'.$this->image->link."</link>\n";
             if ($this->image->width != '') {
-                $feed .= '            <width>' . $this->image->width . "</width>\n";
+                $feed .= '            <width>'.$this->image->width."</width>\n";
             }
             if ($this->image->height != '') {
-                $feed .= '            <height>' . $this->image->height . "</height>\n";
+                $feed .= '            <height>'.$this->image->height."</height>\n";
             }
             if ($this->image->description != '') {
-                $feed .= '            <description>' . $this->image->getDescription() . "</description>\n";
+                $feed .= '            <description>'.$this->image->getDescription()."</description>\n";
             }
             $feed .= "        </image>\n";
         }
         if ($this->language != '') {
-            $feed .= '        <language>' . $this->language . "</language>\n";
+            $feed .= '        <language>'.$this->language."</language>\n";
         }
         if ($this->copyright != '') {
-            $feed .= '        <copyright>' . FeedCreator::iTrunc(htmlspecialchars($this->copyright), 100) . "</copyright>\n";
+            $feed .= '        <copyright>'.FeedCreator::iTrunc(htmlspecialchars($this->copyright), 100)."</copyright>\n";
         }
         if ($this->editor != '') {
-            $feed .= '        <managingEditor>' . FeedCreator::iTrunc(htmlspecialchars($this->editor), 100) . "</managingEditor>\n";
+            $feed .= '        <managingEditor>'.FeedCreator::iTrunc(htmlspecialchars($this->editor), 100)."</managingEditor>\n";
         }
         if ($this->webmaster != '') {
-            $feed .= '        <webMaster>' . FeedCreator::iTrunc(htmlspecialchars($this->webmaster), 100) . "</webMaster>\n";
+            $feed .= '        <webMaster>'.FeedCreator::iTrunc(htmlspecialchars($this->webmaster), 100)."</webMaster>\n";
         }
         if ($this->pubDate != '') {
             $pubDate = new FeedDate($this->pubDate);
-            $feed    .= '        <pubDate>' . htmlspecialchars($pubDate->rfc822()) . "</pubDate>\n";
+            $feed .= '        <pubDate>'.htmlspecialchars($pubDate->rfc822())."</pubDate>\n";
         }
         if ($this->category != '') {
-            $feed .= '        <category>' . htmlspecialchars($this->category) . "</category>\n";
+            $feed .= '        <category>'.htmlspecialchars($this->category)."</category>\n";
         }
         if ($this->docs != '') {
-            $feed .= '        <docs>' . FeedCreator::iTrunc(htmlspecialchars($this->docs), 500) . "</docs>\n";
+            $feed .= '        <docs>'.FeedCreator::iTrunc(htmlspecialchars($this->docs), 500)."</docs>\n";
         }
         if ($this->ttl != '') {
-            $feed .= '        <ttl>' . htmlspecialchars($this->ttl) . "</ttl>\n";
+            $feed .= '        <ttl>'.htmlspecialchars($this->ttl)."</ttl>\n";
         }
         if ($this->rating != '') {
-            $feed .= '        <rating>' . FeedCreator::iTrunc(htmlspecialchars($this->rating), 500) . "</rating>\n";
+            $feed .= '        <rating>'.FeedCreator::iTrunc(htmlspecialchars($this->rating), 500)."</rating>\n";
         }
         if ($this->skipHours != '') {
-            $feed .= '        <skipHours>' . htmlspecialchars($this->skipHours) . "</skipHours>\n";
+            $feed .= '        <skipHours>'.htmlspecialchars($this->skipHours)."</skipHours>\n";
         }
         if ($this->skipDays != '') {
-            $feed .= '        <skipDays>' . htmlspecialchars($this->skipDays) . "</skipDays>\n";
+            $feed .= '        <skipDays>'.htmlspecialchars($this->skipDays)."</skipDays>\n";
         }
         $feed .= $this->_createAdditionalElements($this->additionalElements, '    ');
 
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             $feed .= "        <item>\n";
-            $feed .= '            <title>' . FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100) . "</title>\n";
-            $feed .= '            <link>' . htmlspecialchars($this->items[$i]->link) . "</link>\n";
-            $feed .= '            <description>' . $this->items[$i]->getDescription() . "</description>\n";
+            $feed .= '            <title>'.FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100)."</title>\n";
+            $feed .= '            <link>'.htmlspecialchars($this->items[$i]->link)."</link>\n";
+            $feed .= '            <description>'.$this->items[$i]->getDescription()."</description>\n";
 
             if ($this->items[$i]->author != '') {
-                $feed .= '            <author>' . htmlspecialchars($this->items[$i]->author) . "</author>\n";
+                $feed .= '            <author>'.htmlspecialchars($this->items[$i]->author)."</author>\n";
             }
             /*
             // on hold
@@ -1094,17 +1094,17 @@ 
             }
             */
             if ($this->items[$i]->category != '') {
-                $feed .= '            <category>' . htmlspecialchars($this->items[$i]->category) . "</category>\n";
+                $feed .= '            <category>'.htmlspecialchars($this->items[$i]->category)."</category>\n";
             }
             if ($this->items[$i]->comments != '') {
-                $feed .= '            <comments>' . htmlspecialchars($this->items[$i]->comments) . "</comments>\n";
+                $feed .= '            <comments>'.htmlspecialchars($this->items[$i]->comments)."</comments>\n";
             }
             if ($this->items[$i]->date != '') {
                 $itemDate = new FeedDate($this->items[$i]->date);
-                $feed     .= '            <pubDate>' . htmlspecialchars($itemDate->rfc822()) . "</pubDate>\n";
+                $feed .= '            <pubDate>'.htmlspecialchars($itemDate->rfc822())."</pubDate>\n";
             }
             if ($this->items[$i]->guid != '') {
-                $feed .= '            <guid>' . htmlspecialchars($this->items[$i]->guid) . "</guid>\n";
+                $feed .= '            <guid>'.htmlspecialchars($this->items[$i]->guid)."</guid>\n";
             }
             $feed .= $this->_createAdditionalElements($this->items[$i]->additionalElements, '        ');
             $feed .= "        </item>\n";
@@ -1157,32 +1157,32 @@ 
      */
     public function createFeed()
     {
-        $feed            = "<?xml version=\"1.0\" encoding=\"" . $this->encoding . "\"?>\n";
+        $feed = "<?xml version=\"1.0\" encoding=\"".$this->encoding."\"?>\n";
         $feed            .= $this->_createStylesheetReferences();
         $feed            .= "<feed version=\"0.1\" xmlns=\"http://example.com/newformat#\">\n";
-        $feed            .= '    <title>' . FeedCreator::iTrunc(htmlspecialchars($this->title), 100) . "</title>\n";
+        $feed            .= '    <title>'.FeedCreator::iTrunc(htmlspecialchars($this->title), 100)."</title>\n";
         $this->truncSize = 500;
-        $feed            .= '    <subtitle>' . $this->getDescription() . "</subtitle>\n";
-        $feed            .= '    <link>' . $this->link . "</link>\n";
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        $feed            .= '    <subtitle>'.$this->getDescription()."</subtitle>\n";
+        $feed            .= '    <link>'.$this->link."</link>\n";
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             $feed     .= "    <entry>\n";
-            $feed     .= '        <title>' . FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100) . "</title>\n";
-            $feed     .= '        <link>' . htmlspecialchars($this->items[$i]->link) . "</link>\n";
+            $feed     .= '        <title>'.FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100)."</title>\n";
+            $feed     .= '        <link>'.htmlspecialchars($this->items[$i]->link)."</link>\n";
             $itemDate = new FeedDate($this->items[$i]->date);
-            $feed     .= '        <created>' . htmlspecialchars($itemDate->iso8601()) . "</created>\n";
-            $feed     .= '        <issued>' . htmlspecialchars($itemDate->iso8601()) . "</issued>\n";
-            $feed     .= '        <modified>' . htmlspecialchars($itemDate->iso8601()) . "</modified>\n";
-            $feed     .= '        <id>' . htmlspecialchars($this->items[$i]->guid) . "</id>\n";
+            $feed     .= '        <created>'.htmlspecialchars($itemDate->iso8601())."</created>\n";
+            $feed     .= '        <issued>'.htmlspecialchars($itemDate->iso8601())."</issued>\n";
+            $feed     .= '        <modified>'.htmlspecialchars($itemDate->iso8601())."</modified>\n";
+            $feed     .= '        <id>'.htmlspecialchars($this->items[$i]->guid)."</id>\n";
             if ($this->items[$i]->author != '') {
                 $feed .= "        <author>\n";
-                $feed .= '            <name>' . htmlspecialchars($this->items[$i]->author) . "</name>\n";
+                $feed .= '            <name>'.htmlspecialchars($this->items[$i]->author)."</name>\n";
                 if ($this->items[$i]->authorEmail != '') {
-                    $feed .= '            <email>' . $this->items[$i]->authorEmail . "</email>\n";
+                    $feed .= '            <email>'.$this->items[$i]->authorEmail."</email>\n";
                 }
                 $feed .= "        </author>\n";
             }
             $feed .= "        <content type=\"text/html\" xml:lang=\"en-us\">\n";
-            $feed .= "            <div xmlns=\"http://www.w3.org/1999/xhtml\">" . $this->items[$i]->getDescription() . "</div>\n";
+            $feed .= "            <div xmlns=\"http://www.w3.org/1999/xhtml\">".$this->items[$i]->getDescription()."</div>\n";
             $feed .= "        </content>\n";
             $feed .= "    </entry>\n";
         }
@@ -1224,50 +1224,50 @@ 
      */
     public function createFeed()
     {
-        $feed = "<?xml version=\"1.0\" encoding=\"" . $this->encoding . "\"?>\n";
+        $feed = "<?xml version=\"1.0\" encoding=\"".$this->encoding."\"?>\n";
         $feed .= $this->_createGeneratorComment();
         $feed .= $this->_createStylesheetReferences();
         $feed .= "<feed version=\"0.3\" xmlns=\"http://purl.org/atom/ns#\"";
         if ($this->language != '') {
-            $feed .= " xml:lang=\"" . $this->language . "\"";
+            $feed .= " xml:lang=\"".$this->language."\"";
         }
         $feed .= ">\n";
-        $feed .= '    <title>' . htmlspecialchars($this->title) . "</title>\n";
-        $feed .= '    <tagline>' . htmlspecialchars($this->description) . "</tagline>\n";
-        $feed .= "    <link rel=\"alternate\" type=\"text/html\" href=\"" . htmlspecialchars($this->link) . "\">\n";
-        $feed .= '    <id>' . htmlspecialchars($this->link) . "</id>\n";
-        $now  = new FeedDate();
-        $feed .= '    <modified>' . htmlspecialchars($now->iso8601()) . "</modified>\n";
+        $feed .= '    <title>'.htmlspecialchars($this->title)."</title>\n";
+        $feed .= '    <tagline>'.htmlspecialchars($this->description)."</tagline>\n";
+        $feed .= "    <link rel=\"alternate\" type=\"text/html\" href=\"".htmlspecialchars($this->link)."\">\n";
+        $feed .= '    <id>'.htmlspecialchars($this->link)."</id>\n";
+        $now = new FeedDate();
+        $feed .= '    <modified>'.htmlspecialchars($now->iso8601())."</modified>\n";
         if ($this->editor != '') {
             $feed .= "    <author>\n";
-            $feed .= '        <name>' . $this->editor . "</name>\n";
+            $feed .= '        <name>'.$this->editor."</name>\n";
             if ($this->editorEmail != '') {
-                $feed .= '        <email>' . $this->editorEmail . "</email>\n";
+                $feed .= '        <email>'.$this->editorEmail."</email>\n";
             }
             $feed .= "    </author>\n";
         }
-        $feed .= '    <generator>' . FEEDCREATOR_VERSION . "</generator>\n";
+        $feed .= '    <generator>'.FEEDCREATOR_VERSION."</generator>\n";
         $feed .= $this->_createAdditionalElements($this->additionalElements, '    ');
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             $feed .= "    <entry>\n";
-            $feed .= '        <title>' . htmlspecialchars(strip_tags($this->items[$i]->title)) . "</title>\n";
-            $feed .= "        <link rel=\"alternate\" type=\"text/html\" href=\"" . htmlspecialchars($this->items[$i]->link) . "\">\n";
+            $feed .= '        <title>'.htmlspecialchars(strip_tags($this->items[$i]->title))."</title>\n";
+            $feed .= "        <link rel=\"alternate\" type=\"text/html\" href=\"".htmlspecialchars($this->items[$i]->link)."\">\n";
             if ($this->items[$i]->date == '') {
                 $this->items[$i]->date = time();
             }
             $itemDate = new FeedDate($this->items[$i]->date);
-            $feed     .= '        <created>' . htmlspecialchars($itemDate->iso8601()) . "</created>\n";
-            $feed     .= '        <issued>' . htmlspecialchars($itemDate->iso8601()) . "</issued>\n";
-            $feed     .= '        <modified>' . htmlspecialchars($itemDate->iso8601()) . "</modified>\n";
-            $feed     .= '        <id>' . htmlspecialchars($this->items[$i]->link) . "</id>\n";
+            $feed     .= '        <created>'.htmlspecialchars($itemDate->iso8601())."</created>\n";
+            $feed     .= '        <issued>'.htmlspecialchars($itemDate->iso8601())."</issued>\n";
+            $feed     .= '        <modified>'.htmlspecialchars($itemDate->iso8601())."</modified>\n";
+            $feed     .= '        <id>'.htmlspecialchars($this->items[$i]->link)."</id>\n";
             $feed     .= $this->_createAdditionalElements($this->items[$i]->additionalElements, '        ');
             if ($this->items[$i]->author != '') {
                 $feed .= "        <author>\n";
-                $feed .= '            <name>' . htmlspecialchars($this->items[$i]->author) . "</name>\n";
+                $feed .= '            <name>'.htmlspecialchars($this->items[$i]->author)."</name>\n";
                 $feed .= "        </author>\n";
             }
             if ($this->items[$i]->description != '') {
-                $feed .= '        <summary>' . htmlspecialchars($this->items[$i]->description) . "</summary>\n";
+                $feed .= '        <summary>'.htmlspecialchars($this->items[$i]->description)."</summary>\n";
             }
             $feed .= "    </entry>\n";
         }
@@ -1312,23 +1312,23 @@ 
             //$line = rtrim($line); // remove trailing white space -> no =20\r\n necessary
             $linlen  = strlen($line);
             $newline = '';
-            for ($i = 0; $i < $linlen; ++$i) {
+            for ($i = 0; $i<$linlen; ++$i) {
                 $c   = substr($line, $i, 1);
                 $dec = ord($c);
-                if (($dec == 32) && ($i == ($linlen - 1))) { // convert space at eol only
+                if (($dec == 32) && ($i == ($linlen-1))) { // convert space at eol only
                     $c = '=20';
-                } elseif (($dec == 61) || ($dec < 32) || ($dec > 126)) { // always encode "\t", which is *not* required
-                    $h2 = floor($dec / 16);
-                    $h1 = floor($dec % 16);
-                    $c  = $escape . $hex["$h2"] . $hex["$h1"];
+                } elseif (($dec == 61) || ($dec<32) || ($dec>126)) { // always encode "\t", which is *not* required
+                    $h2 = floor($dec/16);
+                    $h1 = floor($dec%16);
+                    $c  = $escape.$hex["$h2"].$hex["$h1"];
                 }
-                if ((strlen($newline) + strlen($c)) >= $line_max) { // CRLF is not counted
-                    $output  .= $newline . $escape . $eol; // soft line break; " =\r\n" is okay
+                if ((strlen($newline)+strlen($c))>=$line_max) { // CRLF is not counted
+                    $output .= $newline.$escape.$eol; // soft line break; " =\r\n" is okay
                     $newline = '';
                 }
                 $newline .= $c;
             } // end of for
-            $output .= $newline . $eol;
+            $output .= $newline.$eol;
         }
 
         return trim($output);
@@ -1340,23 +1340,23 @@ 
      */
     public function createFeed()
     {
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             if ($this->items[$i]->author != '') {
                 $from = $this->items[$i]->author;
-            } else {
+            }else {
                 $from = $this->title;
             }
             $itemDate = new FeedDate($this->items[$i]->date);
-            $feed     .= 'From ' . strtr(MBOXCreator::qp_enc($from), ' ', '_') . ' ' . date('D M d H:i:s Y', $itemDate->unix()) . "\n";
+            $feed     .= 'From '.strtr(MBOXCreator::qp_enc($from), ' ', '_').' '.date('D M d H:i:s Y', $itemDate->unix())."\n";
             $feed     .= "Content-Type: text/plain;\n";
-            $feed     .= "   charset=\"" . $this->encoding . "\"\n";
+            $feed     .= "   charset=\"".$this->encoding."\"\n";
             $feed     .= "Content-Transfer-Encoding: quoted-printable\n";
             $feed     .= "Content-Type: text/plain\n";
-            $feed     .= "From: \"" . MBOXCreator::qp_enc($from) . "\"\n";
-            $feed     .= 'Date: ' . $itemDate->rfc822() . "\n";
-            $feed     .= 'Subject: ' . MBOXCreator::qp_enc(FeedCreator::iTrunc($this->items[$i]->title, 100)) . "\n";
+            $feed     .= "From: \"".MBOXCreator::qp_enc($from)."\"\n";
+            $feed     .= 'Date: '.$itemDate->rfc822()."\n";
+            $feed     .= 'Subject: '.MBOXCreator::qp_enc(FeedCreator::iTrunc($this->items[$i]->title, 100))."\n";
             $feed     .= "\n";
-            $body     = chunk_split(MBOXCreator::qp_enc($this->items[$i]->description));
+            $body = chunk_split(MBOXCreator::qp_enc($this->items[$i]->description));
             $feed     .= preg_replace("~\nFrom ([^\n]*)(\n?)~", "\n>From $1$2\n", $body);
             $feed     .= "\n";
             $feed     .= "\n";
@@ -1375,7 +1375,7 @@ 
     {
         $fileInfo = pathinfo($_SERVER['PHP_SELF']);
 
-        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension']) + 1)) . '.mbox';
+        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension'])+1)).'.mbox';
     }
 }
 
@@ -1401,35 +1401,35 @@ 
      */
     public function createFeed()
     {
-        $feed = "<?xml version=\"1.0\" encoding=\"" . $this->encoding . "\"?>\n";
+        $feed = "<?xml version=\"1.0\" encoding=\"".$this->encoding."\"?>\n";
         $feed .= $this->_createGeneratorComment();
         $feed .= $this->_createStylesheetReferences();
         $feed .= "<opml xmlns:xsd=\"http://www.w3.org/2001/XMLSchema\" xmlns:xsi=\"http://www.w3.org/2001/XMLSchema-instance\">\n";
         $feed .= "    <head>\n";
-        $feed .= '        <title>' . htmlspecialchars($this->title) . "</title>\n";
+        $feed .= '        <title>'.htmlspecialchars($this->title)."</title>\n";
         if ($this->pubDate != '') {
             $date = new FeedDate($this->pubDate);
-            $feed .= '         <dateCreated>' . $date->rfc822() . "</dateCreated>\n";
+            $feed .= '         <dateCreated>'.$date->rfc822()."</dateCreated>\n";
         }
         if ($this->lastBuildDate != '') {
             $date = new FeedDate($this->lastBuildDate);
-            $feed .= '         <dateModified>' . $date->rfc822() . "</dateModified>\n";
+            $feed .= '         <dateModified>'.$date->rfc822()."</dateModified>\n";
         }
         if ($this->editor != '') {
-            $feed .= '         <ownerName>' . $this->editor . "</ownerName>\n";
+            $feed .= '         <ownerName>'.$this->editor."</ownerName>\n";
         }
         if ($this->editorEmail != '') {
-            $feed .= '         <ownerEmail>' . $this->editorEmail . "</ownerEmail>\n";
+            $feed .= '         <ownerEmail>'.$this->editorEmail."</ownerEmail>\n";
         }
         $feed .= "    </head>\n";
         $feed .= "    <body>\n";
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             $feed  .= "    <outline type=\"rss\" ";
             $title = htmlspecialchars(strip_tags(strtr($this->items[$i]->title, "\n\r", '  ')));
-            $feed  .= " title=\"" . $title . "\"";
-            $feed  .= " text=\"" . $title . "\"";
+            $feed  .= " title=\"".$title."\"";
+            $feed  .= " text=\"".$title."\"";
             //$feed.= " description=\"".htmlspecialchars($this->items[$i]->description)."\"";
-            $feed .= " url=\"" . htmlspecialchars($this->items[$i]->link) . "\"";
+            $feed .= " url=\"".htmlspecialchars($this->items[$i]->link)."\"";
             $feed .= ">\n";
         }
         $feed .= "    </body>\n";
@@ -1513,7 +1513,7 @@ 
 
         //if no stylePrefix is set, generate it yourself depending on the script name
         if ($this->stylePrefix == '') {
-            $this->stylePrefix = str_replace('.', '_', $this->_generateFilename()) . '_';
+            $this->stylePrefix = str_replace('.', '_', $this->_generateFilename()).'_';
         }
 
         //set an openInNewWindow_token_to be inserted or not
@@ -1524,49 +1524,49 @@ 
         // use this array to put the lines in and implode later with "document.write" javascript
         $feedArray = [];
         if ($this->image != null) {
-            $imageStr = "<a href='" . $this->image->link . "'" . $targetInsert . '>' . "<img src='" . $this->image->url . "' border='0' alt='" . FeedCreator::iTrunc(htmlspecialchars($this->image->title), 100) . "' align='" . $this->imageAlign . "' ";
+            $imageStr = "<a href='".$this->image->link."'".$targetInsert.'>'."<img src='".$this->image->url."' border='0' alt='".FeedCreator::iTrunc(htmlspecialchars($this->image->title), 100)."' align='".$this->imageAlign."' ";
             if ($this->image->width) {
-                $imageStr .= " width='" . $this->image->width . "' ";
+                $imageStr .= " width='".$this->image->width."' ";
             }
             if ($this->image->height) {
-                $imageStr .= " height='" . $this->image->height . "' ";
+                $imageStr .= " height='".$this->image->height."' ";
             }
-            $imageStr    .= '></a>';
+            $imageStr .= '></a>';
             $feedArray[] = $imageStr;
         }
 
         if ($this->title) {
-            $feedArray[] = "<div class='" . $this->stylePrefix . "title'><a href='" . $this->link . "' " . $targetInsert . " class='" . $this->stylePrefix . "title'>" . FeedCreator::iTrunc(htmlspecialchars($this->title), 100) . '</a></div>';
+            $feedArray[] = "<div class='".$this->stylePrefix."title'><a href='".$this->link."' ".$targetInsert." class='".$this->stylePrefix."title'>".FeedCreator::iTrunc(htmlspecialchars($this->title), 100).'</a></div>';
         }
         if ($this->getDescription()) {
-            $feedArray[] = "<div class='" . $this->stylePrefix . "description'>" . str_replace(']]>', '', str_replace('<![CDATA[', '', $this->getDescription())) . '</div>';
+            $feedArray[] = "<div class='".$this->stylePrefix."description'>".str_replace(']]>', '', str_replace('<![CDATA[', '', $this->getDescription())).'</div>';
         }
 
         if ($this->header) {
-            $feedArray[] = "<div class='" . $this->stylePrefix . "header'>" . $this->header . '</div>';
+            $feedArray[] = "<div class='".$this->stylePrefix."header'>".$this->header.'</div>';
         }
 
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
-            if ($this->separator && $i > 0) {
-                $feedArray[] = "<div class='" . $this->stylePrefix . "separator'>" . $this->separator . '</div>';
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
+            if ($this->separator && $i>0) {
+                $feedArray[] = "<div class='".$this->stylePrefix."separator'>".$this->separator.'</div>';
             }
 
             if ($this->items[$i]->title) {
                 if ($this->items[$i]->link) {
-                    $feedArray[] = "<div class='" . $this->stylePrefix . "item_title'><a href='" . $this->items[$i]->link . "' class='" . $this->stylePrefix . "item_title'" . $targetInsert . '>' . FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100) . '</a></div>';
-                } else {
-                    $feedArray[] = "<div class='" . $this->stylePrefix . "item_title'>" . FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100) . '</div>';
+                    $feedArray[] = "<div class='".$this->stylePrefix."item_title'><a href='".$this->items[$i]->link."' class='".$this->stylePrefix."item_title'".$targetInsert.'>'.FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100).'</a></div>';
+                }else {
+                    $feedArray[] = "<div class='".$this->stylePrefix."item_title'>".FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100).'</div>';
                 }
             }
             if ($this->items[$i]->getDescription()) {
-                $feedArray[] = "<div class='" . $this->stylePrefix . "item_description'>" . str_replace(']]>', '', str_replace('<![CDATA[', '', $this->items[$i]->getDescription())) . '</div>';
+                $feedArray[] = "<div class='".$this->stylePrefix."item_description'>".str_replace(']]>', '', str_replace('<![CDATA[', '', $this->items[$i]->getDescription())).'</div>';
             }
         }
         if ($this->footer) {
-            $feedArray[] = "<div class='" . $this->stylePrefix . "footer'>" . $this->footer . '</div>';
+            $feedArray[] = "<div class='".$this->stylePrefix."footer'>".$this->footer.'</div>';
         }
 
-        $feed = '' . implode($feedArray, "\r\n");
+        $feed = ''.implode($feedArray, "\r\n");
 
         return $feed;
     }
@@ -1582,7 +1582,7 @@ 
     {
         $fileInfo = pathinfo($_SERVER['PHP_SELF']);
 
-        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension']) + 1)) . '.html';
+        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension'])+1)).'.html';
     }
 }
 
@@ -1614,7 +1614,7 @@ 
 
         $jsFeed = '';
         foreach ($feedArray as $value) {
-            $jsFeed .= "document.write('" . trim(addslashes($value)) . "');\n";
+            $jsFeed .= "document.write('".trim(addslashes($value))."');\n";
         }
 
         return $jsFeed;
@@ -1631,7 +1631,7 @@ 
     {
         $fileInfo = pathinfo($_SERVER['PHP_SELF']);
 
-        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension']) + 1)) . '.js';
+        return substr($fileInfo['basename'], 0, -(strlen($fileInfo['extension'])+1)).'.js';
     }
 }
 

class/magpie.inc.php

Doc Comments +8 added lines, -8 removed lines

@@ -306,7 +306,7 @@ 
     // smart append - field and namespace aware
 
     /**
-     * @param $el
+     * @param string $el
      * @param $text
      */
     public function append($el, $text)
@@ -390,7 +390,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return string|false
      */
     public function is_rss()
     {
@@ -402,7 +402,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return string|false
      */
     public function is_atom()
     {
@@ -415,10 +415,10 @@ 
 
     /**
      * return XML parser, and possibly re-encoded source
-     * @param $source
-     * @param $out_enc
-     * @param $in_enc
-     * @param $detect
+     * @param string $source
+     * @param string $out_enc
+     * @param string|null $in_enc
+     * @param boolean $detect
      * @return array
      */
     public function create_parser($source, $out_enc, $in_enc, $detect)
@@ -528,7 +528,7 @@ 
 
     /**
      * @param $enc
-     * @return bool|string
+     * @return string|false
      */
     public function known_encoding($enc)
     {

Spacing +32 added lines, -32 removed lines

@@ -35,16 +35,16 @@ 
 {
     public $parser;
 
-    public $current_item = [];  // item currently being parsed
-    public $items        = [];  // collection of parsed items
-    public $channel      = [];  // hash of channel fields
+    public $current_item = []; // item currently being parsed
+    public $items        = []; // collection of parsed items
+    public $channel      = []; // hash of channel fields
     public $textinput    = [];
     public $image        = [];
     public $feed_type;
     public $feed_version;
-    public $encoding     = '';       // output encoding of parsed rss
+    public $encoding     = ''; // output encoding of parsed rss
 
-    public $_source_encoding = '';     // only set if we have to parse xml prolog
+    public $_source_encoding = ''; // only set if we have to parse xml prolog
 
     public $ERROR   = '';
     public $WARNING = '';
@@ -101,13 +101,13 @@ 
         # if PHP xml isn't compiled in, die
         #
         if (!function_exists('xml_parser_create')) {
-            $this->error("Failed to load PHP's XML Extension. " . 'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
+            $this->error("Failed to load PHP's XML Extension. ".'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
         }
 
         list($parser, $source) = $this->create_parser($source, $output_encoding, $input_encoding, $detect_encoding);
 
         if (!is_resource($parser)) {
-            $this->error("Failed to create an instance of PHP's XML parser. " . 'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
+            $this->error("Failed to create an instance of PHP's XML parser. ".'http://www.php.net/manual/en/ref.xml.php', E_USER_ERROR);
         }
 
         $this->parser = $parser;
@@ -217,8 +217,8 @@ 
         elseif ($this->feed_type == ATOM && $el === 'link') {
             if (isset($attrs['rel']) && $attrs['rel'] === 'alternate') {
                 $link_el = 'link';
-            } else {
-                $link_el = 'link_' . $attrs['rel'];
+            }else {
+                $link_el = 'link_'.$attrs['rel'];
             }
 
             $this->append($link_el, $attrs['href']);
@@ -236,7 +236,7 @@ 
     {
         if ($this->feed_type == ATOM && $this->incontent) {
             $this->append_content($text);
-        } else {
+        }else {
             $current_el = implode('_', array_reverse($this->stack));
             $this->append($current_el, $text);
         }
@@ -267,12 +267,12 @@ 
             // 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);
         }
 
@@ -324,7 +324,7 @@ 
             } 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) {
@@ -342,7 +342,7 @@ 
         // if atom populate rss fields
         if ($this->is_atom()) {
             $this->channel['description'] = $this->channel['tagline'];
-            for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+            for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
                 $item = $this->items[$i];
                 if (isset($item['summary'])) {
                     $item['description'] = $item['summary'];
@@ -354,7 +354,7 @@ 
                 $atom_date = isset($item['issued']) ? $item['issued'] : @$item['modified'];
                 if ($atom_date) {
                     $epoch = @parse_w3cdtf($atom_date);
-                    if ($epoch && $epoch > 0) {
+                    if ($epoch && $epoch>0) {
                         $item['date_timestamp'] = $epoch;
                     }
                 }
@@ -363,7 +363,7 @@ 
             }
         } elseif ($this->is_rss()) {
             $this->channel['tagline'] = $this->channel['description'];
-            for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+            for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
                 $item = $this->items[$i];
                 if (isset($item['description'])) {
                     $item['summary'] = $item['description'];
@@ -374,12 +374,12 @@ 
 
                 if ($this->is_rss() === '1.0' && isset($item['dc']['date'])) {
                     $epoch = @parse_w3cdtf($item['dc']['date']);
-                    if ($epoch && $epoch > 0) {
+                    if ($epoch && $epoch>0) {
                         $item['date_timestamp'] = $epoch;
                     }
                 } elseif (isset($item['pubdate'])) {
                     $epoch = @strtotime($item['pubdate']);
-                    if ($epoch > 0) {
+                    if ($epoch>0) {
                         $item['date_timestamp'] = $epoch;
                     }
                 }
@@ -396,7 +396,7 @@ 
     {
         if ($this->feed_type == RSS) {
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -408,7 +408,7 @@ 
     {
         if ($this->feed_type == ATOM) {
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -425,7 +425,7 @@ 
     {
         if (substr(PHP_VERSION, 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) {
@@ -452,7 +452,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('');
         }
     }
@@ -485,7 +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';
             }
         }
@@ -535,7 +535,7 @@ 
         $enc = strtoupper($enc);
         if (in_array($enc, $this->_KNOWN_ENCODINGS)) {
             return $enc;
-        } else {
+        }else {
             return false;
         }
     }
@@ -552,14 +552,14 @@ 
         }
         if (MAGPIE_DEBUG) {
             trigger_error($errormsg, $lvl);
-        } else {
+        }else {
             error_log($errormsg, 0);
         }
 
         $notices = E_USER_NOTICE | E_NOTICE;
         if ($lvl & $notices) {
             $this->WARNING = $errormsg;
-        } else {
+        }else {
             $this->ERROR = $errormsg;
         }
     }
@@ -595,12 +595,12 @@ 
         ];
 
         # calc epoch for current date assuming GMT
-        $epoch = gmmktime((int)$hours, (int)$minutes, (int)$seconds, (int)$month, (int)$day, (int)$year);
+        $epoch = gmmktime((int) $hours, (int) $minutes, (int) $seconds, (int) $month, (int) $day, (int) $year);
 
         $offset = 0;
         if ($match[10] === 'Z') {
             # zulu time, aka GMT
-        } else {
+        }else {
             list($tz_mod, $tz_hour, $tz_min) = [$match[8], $match[9], $match[10]];
 
             # zero out the variables
@@ -611,20 +611,20 @@ 
                 $tz_min = 0;
             }
 
-            $offset_secs = (($tz_hour * 60) + $tz_min) * 60;
+            $offset_secs = (($tz_hour*60)+$tz_min)*60;
 
             # is timezone ahead of GMT?  then subtract offset
             #
             if ($tz_mod == '+') {
-                $offset_secs = $offset_secs * -1;
+                $offset_secs = $offset_secs*-1;
             }
 
             $offset = $offset_secs;
         }
-        $epoch = $epoch + $offset;
+        $epoch = $epoch+$offset;
 
         return $epoch;
-    } else {
+    }else {
         return -1;
     }
 }

class/utility.php

Doc Comments +3 added lines, -3 removed lines

@@ -117,7 +117,7 @@ 
          * @var mixed  $pattern
          * @var mixed  $replacement
          *
-         * @return bool true on success
+         * @return null|boolean true on success
          */
         public static function planetParseClass($class_string, $pattern = '', $replacement = '')
         {
@@ -153,7 +153,7 @@ 
          * @var mixed  $pattern
          * @var mixed  $replacement
          *
-         * @return bool true on success
+         * @return null|boolean true on success
          */
         public static function planetParseFunction($function_string, $pattern = '', $replacement = '')
         {
@@ -520,7 +520,7 @@ 
 
         /**
          * @param $url
-         * @return bool|string
+         * @return false|string
          */
         public static function planetFetchFopen($url)
         {

Spacing +42 added lines, -42 removed lines

@@ -38,9 +38,9 @@ 
 if (!defined('planet_FUNCTIONS')):
     define('planet_FUNCTIONS', 1);
 
-    require XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/vars.php';
-    require_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
-    require_once XOOPS_ROOT_PATH . '/Frameworks/art/functions.php';
+    require XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/vars.php';
+    require_once XOOPS_ROOT_PATH.'/class/xoopslists.php';
+    require_once XOOPS_ROOT_PATH.'/Frameworks/art/functions.php';
 
     /**
      * Class PlanetUtility
@@ -75,7 +75,7 @@ 
         */
         public static function planetParseArguments(&$args_numeric, &$args, &$args_string)
         {
-            $args_abb     = [
+            $args_abb = [
                 'a' => 'article',
                 'b' => 'blog',
                 'c' => 'category',
@@ -90,24 +90,24 @@ 
             if (preg_match("/[^\?]*\.php[\/|\?]([^\?]*)/i", Request::getUrl('REQUEST_URI', '', 'SERVER'), $matches)) {
                 $vars = preg_split("/[\/|&]/", $matches[1]);
                 $vars = array_map('trim', $vars);
-                if (count($vars) > 0) {
+                if (count($vars)>0) {
                     foreach ($vars as $var) {
                         if (is_numeric($var)) {
                             $args_numeric[] = $var;
                         } elseif (false === strpos($var, '=')) {
                             if (is_numeric(substr($var, 1))) {
-                                $args[$args_abb[strtolower($var{0})]] = (int)substr($var, 1);
-                            } else {
+                                $args[$args_abb[strtolower($var{0})]] = (int) substr($var, 1);
+                            }else {
                                 $args_string[] = urldecode($var);
                             }
-                        } else {
+                        }else {
                             parse_str($var, $args);
                         }
                     }
                 }
             }
 
-            return (count($args) + count($args_numeric) + count($args_string) == 0) ? null : true;
+            return (count($args)+count($args_numeric)+count($args_string) == 0) ? null : true;
         }
 
         /**
@@ -130,7 +130,7 @@ 
                 $pattern     = [$pattern];
                 $replacement = [$replacement];
             }
-            if (is_array($pattern) && count($pattern) > 0) {
+            if (is_array($pattern) && count($pattern)>0) {
                 $ii = 0;
                 foreach ($pattern as $pat) {
                     if (!in_array($pat, $patterns)) {
@@ -166,7 +166,7 @@ 
                 $pattern     = [$pattern];
                 $replacement = [$replacement];
             }
-            if (is_array($pattern) && count($pattern) > 0) {
+            if (is_array($pattern) && count($pattern)>0) {
                 $ii = 0;
                 foreach ($pattern as $pat) {
                     if (!in_array($pat, $patterns)) {
@@ -209,7 +209,7 @@ 
             if (!is_array($userid)) {
                 $userid = [$userid];
             }
-            $users =& mod_getUnameFromIds($userid, $usereal, $linked);
+            $users = & mod_getUnameFromIds($userid, $usereal, $linked);
 
             return $users;
         }
@@ -226,7 +226,7 @@ 
             $myts       = MyTextSanitizer::getInstance();
             $link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
             $links      = [];
-            if (count($link_array) > 0) {
+            if (count($link_array)>0) {
                 foreach ($link_array as $link) {
                     @list($url, $title) = array_map('trim', preg_split('/ /', $link, 2));
                     if (empty($url)) {
@@ -246,7 +246,7 @@ 
          */
         public static function planetGetTemplate($pagename)
         {
-            return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
+            return $GLOBALS['VAR_PREFIX'].'_'.$pagename.'.tpl';
         }
 
         /**
@@ -275,7 +275,7 @@ 
             }
 
             @list($pat, $rep) = array_map('trim', preg_split("#[\s]+#", $pattern));
-            $trackback_url = preg_replace('#' . $pat . '#', $rep, $article_obj->getVar('art_link'));
+            $trackback_url = preg_replace('#'.$pat.'#', $rep, $article_obj->getVar('art_link'));
 
             return static::planetTrackback($trackback_url, $article);
         }
@@ -291,7 +291,7 @@ 
 
             $title         = $article->getVar('art_title');
             $excerpt       = $article->getVar('art_content');
-            $blog_name     = $xoopsConfig['sitename'] . '-' . $xoopsModule->getVar('name');
+            $blog_name     = $xoopsConfig['sitename'].'-'.$xoopsModule->getVar('name');
             $title         = xoops_utf8_encode($title);
             $excerpt       = xoops_utf8_encode($excerpt);
             $blog_name     = xoops_utf8_encode($blog_name);
@@ -299,15 +299,15 @@ 
             $title1        = urlencode($title);
             $excerpt1      = urlencode($excerpt);
             $name1         = urlencode($blog_name);
-            $url           = urlencode(XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.article.php' . URL_DELIMITER . '' . $article->getVar('art_id'));
+            $url           = urlencode(XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.article.php'.URL_DELIMITER.''.$article->getVar('art_id'));
             $query_string  = "title=$title1&url=$url&blog_name=$name1&excerpt=$excerpt1&charset=$charset";
             $trackback_url = parse_url($trackback_url);
 
-            $http_request = 'POST ' . $trackback_url['path'] . ($trackback_url['query'] ? '?' . $trackback_url['query'] : '') . " HTTP/1.0\r\n";
-            $http_request .= 'Host: ' . $trackback_url['host'] . "\r\n";
-            $http_request .= 'Content-Type: application/x-www-form-urlencoded; charset=' . $charset . "\r\n";
-            $http_request .= 'Content-Length: ' . strlen($query_string) . "\r\n";
-            $http_request .= 'User-Agent: XOOPS Blogs/' . XOOPS_VERSION;
+            $http_request = 'POST '.$trackback_url['path'].($trackback_url['query'] ? '?'.$trackback_url['query'] : '')." HTTP/1.0\r\n";
+            $http_request .= 'Host: '.$trackback_url['host']."\r\n";
+            $http_request .= 'Content-Type: application/x-www-form-urlencoded; charset='.$charset."\r\n";
+            $http_request .= 'Content-Length: '.strlen($query_string)."\r\n";
+            $http_request .= 'User-Agent: XOOPS Blogs/'.XOOPS_VERSION;
             $http_request .= "\r\n\r\n";
             $http_request .= $query_string;
             if ('' == $trackback_url['port']) {
@@ -316,11 +316,11 @@ 
             $fs = @fsockopen($trackback_url['host'], $trackback_url['port'], $errno, $errstr, 4);
             @fwrite($fs, $http_request);
             if ($xoopsModuleConfig['do_debug']) {
-                $debug_file = XOOPS_CACHE_PATH . '/' . $GLOBALS['moddirname'] . '_trackback.log';
+                $debug_file = XOOPS_CACHE_PATH.'/'.$GLOBALS['moddirname'].'_trackback.log';
                 $fr         = "\n*****\nRequest:\n\n$http_request\n\nResponse:\n\n";
                 $fr         .= "CHARSET:$charset\n";
                 $fr         .= "NAME:$blog_name\n";
-                $fr         .= 'TITLE:' . $title . "\n";
+                $fr         .= 'TITLE:'.$title."\n";
                 $fr         .= "EXCERPT:$excerpt\n\n";
                 while (!@feof($fs)) {
                     $fr .= @fgets($fs, 4096);
@@ -330,7 +330,7 @@ 
                 if ($fp = fopen($debug_file, 'a')) {
                     fwrite($fp, $fr);
                     fclose($fp);
-                } else {
+                }else {
                 }
             }
             @fclose($fs);
@@ -350,19 +350,19 @@ 
                     PlanetUtility::planetGetPing($serv, $id);
                 }
             }
-            require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/class-IXR.php';
+            require_once XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/class-IXR.php';
 
             // using a timeout of 3 seconds should be enough to cover slow servers
             $client            = new IXR_Client($server, false);
             $client->timeout   = 3;
-            $client->useragent .= ' -- XOOPS Article/' . XOOPS_VERSION;
+            $client->useragent .= ' -- XOOPS Article/'.XOOPS_VERSION;
 
             // when set to true, this outputs debug messages by itself
             $client->debug = false;
 
             $blogname = xoops_utf8_encode($GLOBALS['xoopsModule']->getVar('name'));
-            $home     = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/';
-            $rss2_url = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/xml.php' . URL_DELIMITER . 'rss2.0/' . $id;
+            $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
                 $client->query('weblogUpdates.ping', $blogname, $home);
@@ -378,16 +378,16 @@ 
         {
             $charset       = 'utf-8';
             $error_message = xoops_utf8_encode($error_message);
-            header('Content-Type: text/xml; charset="' . $charset . '"');
+            header('Content-Type: text/xml; charset="'.$charset.'"');
             if ($error) {
-                echo '<?xml version="1.0" encoding="' . $charset . '"?' . ">\n";
+                echo '<?xml version="1.0" encoding="'.$charset.'"?'.">\n";
                 echo "<response>\n";
                 echo "<error>1</error>\n";
                 echo "<message>$error_message</message>\n";
                 echo '</response>';
                 die();
-            } else {
-                echo '<?xml version="1.0" encoding="' . $charset . '"?' . ">\n";
+            }else {
+                echo '<?xml version="1.0" encoding="'.$charset.'"?'.">\n";
                 echo "<response>\n";
                 echo "<error>0</error>\n";
                 echo '</response>';
@@ -407,11 +407,11 @@ 
             if (is_array($string)) {
                 $value = [];
                 foreach ($string as $key => $val) {
-                    $value[] = $key . '|' . $val;
+                    $value[] = $key.'|'.$val;
                 }
                 $string = implode(',', $value);
             }
-            setcookie($GLOBALS['VAR_PREFIX'] . $name, $string, (int)$expire, '/');
+            setcookie($GLOBALS['VAR_PREFIX'].$name, $string, (int) $expire, '/');
         }
 
         /**
@@ -421,14 +421,14 @@ 
          */
         public static function planetGetCookie($name, $isArray = false)
         {
-            $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'] . $name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'] . $name] : null;
+            $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'].$name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'].$name] : null;
             if ($isArray) {
                 $_value = $value ? explode(',', $value) : [];
                 $value  = [];
-                if (count($_value) > 0) {
+                if (count($_value)>0) {
                     foreach ($_value as $string) {
                         $key         = substr($string, 0, strpos($string, '|'));
-                        $val         = substr($string, strpos($string, '|') + 1);
+                        $val         = substr($string, strpos($string, '|')+1);
                         $value[$key] = $val;
                     }
                 }
@@ -487,7 +487,7 @@ 
          */
         public static function planetFetchSnoopy($url)
         {
-            require_once XOOPS_ROOT_PATH . '/class/snoopy.php';
+            require_once XOOPS_ROOT_PATH.'/class/snoopy.php';
             $snoopy = new Snoopy;
             $data   = '';
             if (@$snoopy->fetch($url)) {
@@ -506,10 +506,10 @@ 
             if (!function_exists('curl_init')) {
                 return false;
             }
-            $ch = curl_init();    // initialize curl handle
+            $ch = curl_init(); // initialize curl handle
             curl_setopt($ch, CURLOPT_URL, $url); // set url to post to
             curl_setopt($ch, CURLOPT_FAILONERROR, 1);
-            curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1);// allow redirects
+            curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1); // allow redirects
             curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1); // return into a variable
             curl_setopt($ch, CURLOPT_TIMEOUT, 30); // times out after 31s
             $data = curl_exec($ch); // run the whole process
@@ -551,7 +551,7 @@ 
             if ($index === false) {
                 return false;
             }
-            $index = strlen($haystack) - strlen($needle) - $index;
+            $index = strlen($haystack)-strlen($needle)-$index;
 
             return $index;
         }

class/xmlparser.php

Doc Comments +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
     }
 
     /**
-     * @return bool|string
+     * @return string|false
      */
     public function is_atom()
     {

Spacing +7 added lines, -7 removed lines

@@ -30,7 +30,7 @@ 
  */
 global $msg;
 
-require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/class/magpie.inc.php';
+require_once XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/class/magpie.inc.php';
 
 /**
  * XmlParser
@@ -92,7 +92,7 @@ 
             $this->feed_version = empty($this->feed_version) ? '0.3' : $this->feed_version;
 
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -105,7 +105,7 @@ 
                 $this->channel['tagline'] = @$this->channel['subtitle'];
                 unset($this->channel['subtitle']);
             }
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             // ATOM time
             if ($date = @$this->items[$i]['modified']) {
                 continue;
@@ -124,7 +124,7 @@ 
             }
             $this->items[$i]['modified'] = $date;
         } elseif ($this->is_rss() !== '1.0'):
-            for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+            for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
                 if ($date = @$this->items[$i]['pubdate']) {
                     continue;
                 }
@@ -150,7 +150,7 @@ 
             unset($this->channel['tagline']);
         }
 
-        for ($i = 0, $iMax = count($this->items); $i < $iMax; ++$i) {
+        for ($i = 0, $iMax = count($this->items); $i<$iMax; ++$i) {
             if ($date_timestamp = @$this->items[$i]['date_timestamp']) {
                 continue;
             }
@@ -158,7 +158,7 @@ 
                 $this->items[$i]['date_timestamp'] = $date_timestamp;
             } 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'])) {
@@ -204,7 +204,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);
         }
 

include/functions.php

Doc Comments +3 added lines, -3 removed lines

@@ -111,7 +111,7 @@ 
      * @var mixed  $pattern
      * @var mixed  $replacement
      *
-     * @return bool true on success
+     * @return null|boolean true on success
      */
     function planetParseClass($class_string, $pattern = '', $replacement = '')
     {
@@ -147,7 +147,7 @@ 
      * @var mixed  $pattern
      * @var mixed  $replacement
      *
-     * @return bool true on success
+     * @return null|boolean true on success
      */
     function planetParseFunction($function_string, $pattern = '', $replacement = '')
     {
@@ -512,7 +512,7 @@ 
 
     /**
      * @param $url
-     * @return bool|string
+     * @return false|string
      */
     function planet_fetch_fopen($url)
     {

Spacing +42 added lines, -42 removed lines

@@ -38,9 +38,9 @@ 
 if (!defined('planet_FUNCTIONS')):
     define('planet_FUNCTIONS', 1);
 
-    require XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/vars.php';
-    require_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
-    require_once XOOPS_ROOT_PATH . '/Frameworks/art/functions.php';
+    require XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/vars.php';
+    require_once XOOPS_ROOT_PATH.'/class/xoopslists.php';
+    require_once XOOPS_ROOT_PATH.'/Frameworks/art/functions.php';
 
     /**
      * Function to display messages
@@ -69,7 +69,7 @@ 
     */
     function planetParseArguments(&$args_numeric, &$args, &$args_string)
     {
-        $args_abb     = [
+        $args_abb = [
             'a' => 'article',
             'b' => 'blog',
             'c' => 'category',
@@ -84,24 +84,24 @@ 
         if (preg_match("/[^\?]*\.php[\/|\?]([^\?]*)/i", Request::getUrl('REQUEST_URI', '', 'SERVER'), $matches)) {
             $vars = preg_split("/[\/|&]/", $matches[1]);
             $vars = array_map('trim', $vars);
-            if (count($vars) > 0) {
+            if (count($vars)>0) {
                 foreach ($vars as $var) {
                     if (is_numeric($var)) {
                         $args_numeric[] = $var;
                     } elseif (false === strpos($var, '=')) {
                         if (is_numeric(substr($var, 1))) {
-                            $args[$args_abb[strtolower($var{0})]] = (int)substr($var, 1);
-                        } else {
+                            $args[$args_abb[strtolower($var{0})]] = (int) substr($var, 1);
+                        }else {
                             $args_string[] = urldecode($var);
                         }
-                    } else {
+                    }else {
                         parse_str($var, $args);
                     }
                 }
             }
         }
 
-        return (count($args) + count($args_numeric) + count($args_string) == 0) ? null : true;
+        return (count($args)+count($args_numeric)+count($args_string) == 0) ? null : true;
     }
 
     /**
@@ -124,7 +124,7 @@ 
             $pattern     = [$pattern];
             $replacement = [$replacement];
         }
-        if (is_array($pattern) && count($pattern) > 0) {
+        if (is_array($pattern) && count($pattern)>0) {
             $ii = 0;
             foreach ($pattern as $pat) {
                 if (!in_array($pat, $patterns)) {
@@ -160,7 +160,7 @@ 
             $pattern     = [$pattern];
             $replacement = [$replacement];
         }
-        if (is_array($pattern) && count($pattern) > 0) {
+        if (is_array($pattern) && count($pattern)>0) {
             $ii = 0;
             foreach ($pattern as $pat) {
                 if (!in_array($pat, $patterns)) {
@@ -203,7 +203,7 @@ 
         if (!is_array($userid)) {
             $userid = [$userid];
         }
-        $users =& mod_getUnameFromIds($userid, $usereal, $linked);
+        $users = & mod_getUnameFromIds($userid, $usereal, $linked);
 
         return $users;
     }
@@ -220,7 +220,7 @@ 
         $myts       = MyTextSanitizer::getInstance();
         $link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
         $links      = [];
-        if (count($link_array) > 0) {
+        if (count($link_array)>0) {
             foreach ($link_array as $link) {
                 @list($url, $title) = array_map('trim', preg_split('/ /', $link, 2));
                 if (empty($url)) {
@@ -240,7 +240,7 @@ 
      */
     function planetGetTemplate($pagename)
     {
-        return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
+        return $GLOBALS['VAR_PREFIX'].'_'.$pagename.'.tpl';
     }
 
     /**
@@ -268,7 +268,7 @@ 
             return false;
         }
         @list($pat, $rep) = array_map('trim', preg_split("#[\s]+#", $pattern));
-        $trackback_url = preg_replace('#' . $pat . '#', $rep, $article_obj->getVar('art_link'));
+        $trackback_url = preg_replace('#'.$pat.'#', $rep, $article_obj->getVar('art_link'));
 
         return planetTrackback($trackback_url, $article);
     }
@@ -284,7 +284,7 @@ 
 
         $title         = $article->getVar('art_title');
         $excerpt       = $article->getVar('art_content');
-        $blog_name     = $xoopsConfig['sitename'] . '-' . $xoopsModule->getVar('name');
+        $blog_name     = $xoopsConfig['sitename'].'-'.$xoopsModule->getVar('name');
         $title         = xoops_utf8_encode($title);
         $excerpt       = xoops_utf8_encode($excerpt);
         $blog_name     = xoops_utf8_encode($blog_name);
@@ -292,15 +292,15 @@ 
         $title1        = urlencode($title);
         $excerpt1      = urlencode($excerpt);
         $name1         = urlencode($blog_name);
-        $url           = urlencode(XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.article.php' . URL_DELIMITER . '' . $article->getVar('art_id'));
+        $url           = urlencode(XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.article.php'.URL_DELIMITER.''.$article->getVar('art_id'));
         $query_string  = "title=$title1&url=$url&blog_name=$name1&excerpt=$excerpt1&charset=$charset";
         $trackback_url = parse_url($trackback_url);
 
-        $http_request = 'POST ' . $trackback_url['path'] . ($trackback_url['query'] ? '?' . $trackback_url['query'] : '') . " HTTP/1.0\r\n";
-        $http_request .= 'Host: ' . $trackback_url['host'] . "\r\n";
-        $http_request .= 'Content-Type: application/x-www-form-urlencoded; charset=' . $charset . "\r\n";
-        $http_request .= 'Content-Length: ' . strlen($query_string) . "\r\n";
-        $http_request .= 'User-Agent: XOOPS Blogs/' . XOOPS_VERSION;
+        $http_request = 'POST '.$trackback_url['path'].($trackback_url['query'] ? '?'.$trackback_url['query'] : '')." HTTP/1.0\r\n";
+        $http_request .= 'Host: '.$trackback_url['host']."\r\n";
+        $http_request .= 'Content-Type: application/x-www-form-urlencoded; charset='.$charset."\r\n";
+        $http_request .= 'Content-Length: '.strlen($query_string)."\r\n";
+        $http_request .= 'User-Agent: XOOPS Blogs/'.XOOPS_VERSION;
         $http_request .= "\r\n\r\n";
         $http_request .= $query_string;
         if ('' == $trackback_url['port']) {
@@ -309,11 +309,11 @@ 
         $fs = @fsockopen($trackback_url['host'], $trackback_url['port'], $errno, $errstr, 4);
         @fwrite($fs, $http_request);
         if ($xoopsModuleConfig['do_debug']) {
-            $debug_file = XOOPS_CACHE_PATH . '/' . $GLOBALS['moddirname'] . '_trackback.log';
+            $debug_file = XOOPS_CACHE_PATH.'/'.$GLOBALS['moddirname'].'_trackback.log';
             $fr         = "\n*****\nRequest:\n\n$http_request\n\nResponse:\n\n";
             $fr         .= "CHARSET:$charset\n";
             $fr         .= "NAME:$blog_name\n";
-            $fr         .= 'TITLE:' . $title . "\n";
+            $fr         .= 'TITLE:'.$title."\n";
             $fr         .= "EXCERPT:$excerpt\n\n";
             while (!@feof($fs)) {
                 $fr .= @fgets($fs, 4096);
@@ -323,7 +323,7 @@ 
             if ($fp = fopen($debug_file, 'a')) {
                 fwrite($fp, $fr);
                 fclose($fp);
-            } else {
+            }else {
             }
         }
         @fclose($fs);
@@ -343,19 +343,19 @@ 
                 planetGetPing($serv, $id);
             }
         }
-        require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/class-IXR.php';
+        require_once XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/class-IXR.php';
 
         // using a timeout of 3 seconds should be enough to cover slow servers
         $client            = new IXR_Client($server, false);
         $client->timeout   = 3;
-        $client->useragent .= ' -- XOOPS Article/' . XOOPS_VERSION;
+        $client->useragent .= ' -- XOOPS Article/'.XOOPS_VERSION;
 
         // when set to true, this outputs debug messages by itself
         $client->debug = false;
 
         $blogname = xoops_utf8_encode($GLOBALS['xoopsModule']->getVar('name'));
-        $home     = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/';
-        $rss2_url = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/xml.php' . URL_DELIMITER . 'rss2.0/' . $id;
+        $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
             $client->query('weblogUpdates.ping', $blogname, $home);
@@ -371,16 +371,16 @@ 
     {
         $charset       = 'utf-8';
         $error_message = xoops_utf8_encode($error_message);
-        header('Content-Type: text/xml; charset="' . $charset . '"');
+        header('Content-Type: text/xml; charset="'.$charset.'"');
         if ($error) {
-            echo '<?xml version="1.0" encoding="' . $charset . '"?' . ">\n";
+            echo '<?xml version="1.0" encoding="'.$charset.'"?'.">\n";
             echo "<response>\n";
             echo "<error>1</error>\n";
             echo "<message>$error_message</message>\n";
             echo '</response>';
             die();
-        } else {
-            echo '<?xml version="1.0" encoding="' . $charset . '"?' . ">\n";
+        }else {
+            echo '<?xml version="1.0" encoding="'.$charset.'"?'.">\n";
             echo "<response>\n";
             echo "<error>0</error>\n";
             echo '</response>';
@@ -400,11 +400,11 @@ 
         if (is_array($string)) {
             $value = [];
             foreach ($string as $key => $val) {
-                $value[] = $key . '|' . $val;
+                $value[] = $key.'|'.$val;
             }
             $string = implode(',', $value);
         }
-        setcookie($GLOBALS['VAR_PREFIX'] . $name, $string, (int)$expire, '/');
+        setcookie($GLOBALS['VAR_PREFIX'].$name, $string, (int) $expire, '/');
     }
 
     /**
@@ -414,14 +414,14 @@ 
      */
     function planetGetCookie($name, $isArray = false)
     {
-        $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'] . $name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'] . $name] : null;
+        $value = isset($_COOKIE[$GLOBALS['VAR_PREFIX'].$name]) ? $_COOKIE[$GLOBALS['VAR_PREFIX'].$name] : null;
         if ($isArray) {
             $_value = $value ? explode(',', $value) : [];
             $value  = [];
-            if (count($_value) > 0) {
+            if (count($_value)>0) {
                 foreach ($_value as $string) {
                     $key         = substr($string, 0, strpos($string, '|'));
-                    $val         = substr($string, strpos($string, '|') + 1);
+                    $val         = substr($string, strpos($string, '|')+1);
                     $value[$key] = $val;
                 }
             }
@@ -479,7 +479,7 @@ 
      */
     function planet_fetch_snoopy($url)
     {
-        require_once XOOPS_ROOT_PATH . '/class/snoopy.php';
+        require_once XOOPS_ROOT_PATH.'/class/snoopy.php';
         $snoopy = new Snoopy;
         $data   = '';
         if (@$snoopy->fetch($url)) {
@@ -498,10 +498,10 @@ 
         if (!function_exists('curl_init')) {
             return false;
         }
-        $ch = curl_init();    // initialize curl handle
+        $ch = curl_init(); // initialize curl handle
         curl_setopt($ch, CURLOPT_URL, $url); // set url to post to
         curl_setopt($ch, CURLOPT_FAILONERROR, 1);
-        curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1);// allow redirects
+        curl_setopt($ch, CURLOPT_FOLLOWLOCATION, 1); // allow redirects
         curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1); // return into a variable
         curl_setopt($ch, CURLOPT_TIMEOUT, 30); // times out after 31s
         $data = curl_exec($ch); // run the whole process
@@ -543,7 +543,7 @@ 
         if ($index === false) {
             return false;
         }
-        $index = strlen($haystack) - strlen($needle) - $index;
+        $index = strlen($haystack)-strlen($needle)-$index;
 
         return $index;
     }

view.archive.php

Doc Comments +1 added lines, -1 removed lines

@@ -287,7 +287,7 @@
 // Used in get_calendar
 /**
  * @param $num
- * @return mixed
+ * @return double
  */
 function planet_calendar_week_mod($num)
 {

Spacing +59 added lines, -59 removed lines

@@ -26,7 +26,7 @@ 
 // ------------------------------------------------------------------------ //
 use Xmf\Request;
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (PlanetUtility::planetParseArguments($args_num, $args, $args_str)) {
     $args['year']  = @$args_num[0];
@@ -45,26 +45,26 @@ 
 $articleHandler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
 $blogHandler    = xoops_getModuleHandler('blog', $GLOBALS['moddirname']);
 
-$xoopsOption['xoops_pagetitle'] = $xoopsModule->getVar('name') . ' - ' . planet_constant('MD_ACHIVE');
+$xoopsOption['xoops_pagetitle'] = $xoopsModule->getVar('name').' - '.planet_constant('MD_ACHIVE');
 $xoopsOption['template_main']   = PlanetUtility::planetGetTemplate('archive');
-require_once XOOPS_ROOT_PATH . '/header.php';
-include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/vars.php';
+require_once XOOPS_ROOT_PATH.'/header.php';
+include XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/vars.php';
 
 $year = empty($year) ? date('Y') : $year;
-if ($month < 1) {
+if ($month<1) {
     $month        = $day = 0;
     $page['time'] = sprintf(planet_constant('MD_TIME_Y'), $year);
-} elseif ($day < 1) {
+} elseif ($day<1) {
     $day          = 0;
     $page['time'] = sprintf(planet_constant('MD_TIME_YM'), $year, $month);
-} else {
+}else {
     $page['time'] = sprintf(planet_constant('MD_TIME_YMD'), $year, $month, $day);
 }
 $time = ['year' => $year, 'month' => $month, 'day' => $day];
 if ($xoopsUser) {
-    $timeoffset = ($xoopsUser->getVar('timezone_offset') - $xoopsConfig['server_TZ']) * 3600;
-} else {
-    $timeoffset = ($xoopsConfig['default_TZ'] - $xoopsConfig['server_TZ']) * 3600;
+    $timeoffset = ($xoopsUser->getVar('timezone_offset')-$xoopsConfig['server_TZ'])*3600;
+}else {
+    $timeoffset = ($xoopsConfig['default_TZ']-$xoopsConfig['server_TZ'])*3600;
 }
 
 $criteria = new CriteriaCompo();
@@ -87,7 +87,7 @@ 
 $articles = [];
 $blogs_id = [];
 foreach ($articles_obj as $id => $article) {
-    $articles[]                            = [
+    $articles[] = [
         'id'      => $id,
         'blog'    => ['id' => $article->getVar('blog_id'), 'title' => ''],
         'title'   => $article->getVar('art_title'),
@@ -98,20 +98,20 @@ 
     $blogs_id[$article->getVar('blog_id')] = 1;
     unset($_article);
 }
-$criteria_blog = new Criteria('blog_id', '(' . implode(',', array_keys($blog_array)) . ')', 'IN');
+$criteria_blog = new Criteria('blog_id', '('.implode(',', array_keys($blog_array)).')', 'IN');
 $blogs         = $blogHandler->getList($criteria_blog);
 foreach (array_keys($articles) as $key) {
     $articles[$key]['blog']['title'] = $blogs[$articles[$key]['blog']['id']];
 }
-if ($blog_id > 0) {
+if ($blog_id>0) {
     $page['blog'] = $blogs[$blog_id];
 }
 
-if ($articles_count > $xoopsModuleConfig['articles_perpage']) {
-    include XOOPS_ROOT_PATH . '/class/pagenav.php';
-    $nav     = new XoopsPageNav($articles_count, $xoopsModuleConfig['articles_perpage'], $start, 'start', 'month=' . $month . '&amp;day=' . $day . '&amp;year=' . $year . '&amp;blog=' . (int)$blog_id);
+if ($articles_count>$xoopsModuleConfig['articles_perpage']) {
+    include XOOPS_ROOT_PATH.'/class/pagenav.php';
+    $nav     = new XoopsPageNav($articles_count, $xoopsModuleConfig['articles_perpage'], $start, 'start', 'month='.$month.'&amp;day='.$day.'&amp;year='.$year.'&amp;blog='.(int) $blog_id);
     $pagenav = $nav->renderNav(4);
-} else {
+}else {
     $pagenav = '';
 }
 
@@ -120,43 +120,43 @@ 
 $months   = null;
 if (empty($start)) {
     if ($blog_id) {
-        $blog_criteria = ' AND blog_id=' . $blog_id;
-    } else {
+        $blog_criteria = ' AND blog_id='.$blog_id;
+    }else {
         $blog_criteria = '';
     }
     // Get monthly list
     if (empty($month)) {
-        $sql    = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count
-            FROM " . planet_DB_prefix('article') . "
+        $sql = "SELECT MONTH(FROM_UNIXTIME(art_time - $timeoffset)) AS mon, COUNT(DISTINCT art_id) AS count
+            FROM ".planet_DB_prefix('article')."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
-            " . $blog_criteria . '
+            ".$blog_criteria.'
             GROUP BY mon
             ';
         $result = $xoopsDB->query($sql);
         $months = [];
         while ($myrow = $xoopsDB->fetchArray($result)) {
             $months[] = [
-                'title' => planet_constant('MD_MONTH_' . (int)$myrow['mon']) . ' (' . (int)$myrow['count'] . ')',
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $year . '/' . $myrow['mon'] . '/b' . $blog_id
+                'title' => planet_constant('MD_MONTH_'.(int) $myrow['mon']).' ('.(int) $myrow['count'].')',
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$year.'/'.$myrow['mon'].'/b'.$blog_id
             ];
         }
         $timenav['prev'] = [
-            'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . ($year - 1) . '/b' . $blog_id,
-            'title' => sprintf(planet_constant('MD_TIME_Y'), $year - 1)
+            'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.($year-1).'/b'.$blog_id,
+            'title' => sprintf(planet_constant('MD_TIME_Y'), $year-1)
         ];
-        if ($year < date('Y')) {
+        if ($year<date('Y')) {
             $timenav['next'] = [
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . ($year + 1) . '/b' . $blog_id,
-                'title' => sprintf(planet_constant('MD_TIME_Y'), $year + 1)
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.($year+1).'/b'.$blog_id,
+                'title' => sprintf(planet_constant('MD_TIME_Y'), $year+1)
             ];
         }
     } // Get daily list
     elseif (empty($day)) {
-        $sql    = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count
-            FROM " . planet_DB_prefix('article') . "
+        $sql = "SELECT DAY(FROM_UNIXTIME(art_time - $timeoffset)) AS day, COUNT(DISTINCT a.art_id) AS count
+            FROM ".planet_DB_prefix('article')."
             WHERE YEAR(FROM_UNIXTIME(art_time - $timeoffset)) = $year
             AND MONTH(FROM_UNIXTIME(art_time - $timeoffset)) = $month
-            " . $blog_criteria . '
+            ".$blog_criteria.'
             GROUP BY day
             ';
         $result = $xoopsDB->query($sql);
@@ -164,35 +164,35 @@ 
         while ($myrow = $xoopsDB->fetchArray($result)) {
             $days[$myrow['day']]['count'] = $myrow['count'];
         }
-        for ($i = 1; $i <= 31; ++$i) {
+        for ($i = 1; $i<=31; ++$i) {
             if (!isset($days[$i])) {
                 continue;
             }
             $days[$i] = [
                 'title' => $days[$i]['count'],
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $year . '/' . $month . '/' . $i . '/b' . $blog_id
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$year.'/'.$month.'/'.$i.'/b'.$blog_id
             ];
         }
         $calendar   = planet_getCalendar($year, $month, $days);
-        $month_next = $month + 1;
-        $month_prev = $month - 1;
+        $month_next = $month+1;
+        $month_prev = $month-1;
         $_year      = $year;
         if ($month == 12) {
             $month_next = 1;
-            $_year      = $year + 1;
+            $_year      = $year+1;
         }
         if ($month == 1) {
             $month_pre = 12;
-            $_year     = $year - 1;
+            $_year     = $year-1;
         }
         $timenav['prev'] = [
-            'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $_year . '/' . $month_prev . '/b' . $blog_id,
-            'title' => planet_constant('MD_MONTH_' . $month_prev)
+            'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$_year.'/'.$month_prev.'/b'.$blog_id,
+            'title' => planet_constant('MD_MONTH_'.$month_prev)
         ];
-        if ($year < date('Y') || $month < date('n')) {
+        if ($year<date('Y') || $month<date('n')) {
             $timenav['next'] = [
-                'url'   => XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.archive.php' . URL_DELIMITER . '' . $_year . '/' . $month_next . '/b' . $blog_id,
-                'title' => planet_constant('MD_MONTH_' . $month_next)
+                'url'   => XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.archive.php'.URL_DELIMITER.''.$_year.'/'.$month_next.'/b'.$blog_id,
+                'title' => planet_constant('MD_MONTH_'.$month_next)
             ];
         }
     }
@@ -212,7 +212,7 @@ 
 $xoopsTpl->assign('timenav', $timenav);
 $xoopsTpl->assign('pagenav', $pagenav);
 
-require_once __DIR__ . '/footer.php';
+require_once __DIR__.'/footer.php';
 
 /**
  * @param null $year
@@ -229,24 +229,24 @@ 
     ob_start();
     echo '<table id="calendar">';
     echo '<caption>';
-    printf(planet_constant('MD_TIME_YM'), $year, planet_constant('MD_MONTH_' . $month));
+    printf(planet_constant('MD_TIME_YM'), $year, planet_constant('MD_MONTH_'.$month));
     echo '</caption>';
 
-    for ($i = 1; $i <= 7; ++$i) {
-        echo "\n\t\t<th abbr=\"" . planet_constant('MD_WEEK_' . $i) . "\" scope=\"col\" title=\"" . planet_constant('MD_WEEK_' . $i) . "\">" . planet_constant('MD_WEEK_' . $i) . '</th>';
+    for ($i = 1; $i<=7; ++$i) {
+        echo "\n\t\t<th abbr=\"".planet_constant('MD_WEEK_'.$i)."\" scope=\"col\" title=\"".planet_constant('MD_WEEK_'.$i)."\">".planet_constant('MD_WEEK_'.$i).'</th>';
     }
 
     echo '<tr>';
 
     // See how much we should pad in the beginning
     $week_begins = 1;
-    $pad         = planet_calendar_week_mod(date('w', $unixmonth) - $week_begins);
+    $pad         = planet_calendar_week_mod(date('w', $unixmonth)-$week_begins);
     if (0 != $pad) {
-        echo "\n\t\t" . '<td colspan="' . $pad . '">&nbsp;</td>';
+        echo "\n\t\t".'<td colspan="'.$pad.'">&nbsp;</td>';
     }
 
-    $daysinmonth = (int)date('t', $unixmonth);
-    for ($day = 1; $day <= $daysinmonth; ++$day) {
+    $daysinmonth = (int) date('t', $unixmonth);
+    for ($day = 1; $day<=$daysinmonth; ++$day) {
         if (isset($newrow) && $newrow) {
             echo "\n\t</tr>\n\t<tr>\n\t\t";
         }
@@ -255,26 +255,26 @@ 
         echo '<td>';
 
         if (!empty($days[$day]['url'])) {
-            echo '<a href="' . $days[$day]['url'] . "\"";
+            echo '<a href="'.$days[$day]['url']."\"";
             if (!empty($days[$day]['title'])) {
-                echo "title=\"" . $days[$day]['title'] . "\"";
+                echo "title=\"".$days[$day]['title']."\"";
             }
             echo ">$day</a>";
         } elseif (!empty($days[$day]['title'])) {
-            echo "<acronym title=\"" . $days[$day]['title'] . "\">$day</acronym>";
-        } else {
+            echo "<acronym title=\"".$days[$day]['title']."\">$day</acronym>";
+        }else {
             echo $day;
         }
         echo '</td>';
 
-        if (6 == planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year)) - $week_begins)) {
+        if (6 == planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year))-$week_begins)) {
             $newrow = true;
         }
     }
 
-    $pad = 7 - planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year)) - $week_begins);
+    $pad = 7-planet_calendar_week_mod(date('w', mktime(0, 0, 0, $month, $day, $year))-$week_begins);
     if ($pad != 0 && $pad != 7) {
-        echo "\n\t\t" . '<td class="pad" colspan="' . $pad . '">&nbsp;</td>';
+        echo "\n\t\t".'<td class="pad" colspan="'.$pad.'">&nbsp;</td>';
     }
 
     echo "\n\t</tr>\n\t</tbody>\n\t</table>";
@@ -293,5 +293,5 @@ 
 {
     $base = 7;
 
-    return ($num - $base * floor($num / $base));
+    return ($num-$base*floor($num/$base));
 }

blocks/block.php

Spacing +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
     $current_path = str_replace(strpos($current_path, "\\\\", 2) ? "\\\\" : DIRECTORY_SEPARATOR, '/', $current_path);
 }
 $url_arr = explode('/', strstr($current_path, '/modules/'));
-include XOOPS_ROOT_PATH . '/modules/' . $url_arr[2] . '/include/vars.php';
-include XOOPS_ROOT_PATH . '/modules/' . $url_arr[2] . '/class/utility.php';
+include XOOPS_ROOT_PATH.'/modules/'.$url_arr[2].'/include/vars.php';
+include XOOPS_ROOT_PATH.'/modules/'.$url_arr[2].'/class/utility.php';
 //require_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/functions.php';
 
 /**