mambax7 / planet

class/class-IXR.php

Doc Comments +6 added lines, -6 removed lines

@@ -166,7 +166,7 @@ 
 
     /**
      * IXR_Message constructor.
-     * @param $message
+     * @param string|boolean $message
      */
     public function __construct($message) {
         $this->message = $message;
@@ -863,7 +863,7 @@ 
 
     /**
      * IXR_Base64 constructor.
-     * @param $data
+     * @param string $data
      */
     public function __construct($data) {
         $this->data = $data;
@@ -906,10 +906,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) {
         $this->callbacks[$method]  = $callback;

Spacing +47 added lines, -47 removed lines

@@ -35,7 +35,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]);
             }
         }
@@ -73,7 +73,7 @@ 
         /* We have an array - is it an array or a struct ? */
         if ($this->isStruct($this->data)) {
             return 'struct';
-        } else {
+        }else {
             return 'array';
         }
     }
@@ -85,31 +85,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>';
 
@@ -132,7 +132,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;
@@ -148,15 +148,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      = array();   // The stack used to keep track of the current array/struct
+    public $_arraystructs      = array(); // The stack used to keep track of the current array/struct
     public $_arraystructstypes = array(); // Stack keeping track of if things are structs or array
-    public $_currentStructName = array();  // A stack as well
+    public $_currentStructName = array(); // A stack as well
     public $_param;
     public $_value;
     public $_currentTag;
@@ -177,7 +177,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;
         }
@@ -247,17 +247,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;
@@ -270,13 +270,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;
@@ -310,17 +310,17 @@ 
                 $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 {
+                                         - 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;
             }
@@ -402,7 +402,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
@@ -415,19 +415,19 @@ 
             // 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.');
+                                         '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);
@@ -452,12 +452,12 @@ 
      * @param $xml
      */
     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;
     }
@@ -524,7 +524,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')) {
@@ -532,7 +532,7 @@ 
                     'faultCode'   => $result->code,
                     'faultString' => $result->message
                 );
-            } else {
+            }else {
                 $return[] = array($result);
             }
         }
@@ -625,7 +625,7 @@ 
             if (!$this->path) {
                 $this->path = '/';
             }
-        } else {
+        }else {
             $this->server  = $server;
             $this->path    = $path;
             $this->port    = $port;
@@ -652,7 +652,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) {
@@ -679,11 +679,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);
@@ -801,7 +801,7 @@ 
         // $time can be a PHP timestamp or an ISO one
         if (is_numeric($time)) {
             $this->parseTimestamp($time);
-        } else {
+        }else {
             $this->parseIso($time);
         }
     }
@@ -835,7 +835,7 @@ 
      * @return string
      */
     public function getIso() {
-        return $this->year . $this->month . $this->day . 'T' . $this->hour . ':' . $this->minute . ':' . $this->second
+        return $this->year.$this->month.$this->day.'T'.$this->hour.':'.$this->minute.':'.$this->second
                . $this->timezone;
     }
 
@@ -843,7 +843,7 @@ 
      * @return string
      */
     public function getXml() {
-        return '<dateTime.iso8601>' . $this->getIso() . '</dateTime.iso8601>';
+        return '<dateTime.iso8601>'.$this->getIso().'</dateTime.iso8601>';
     }
 
     /**
@@ -873,7 +873,7 @@ 
      * @return string
      */
     public function getXml() {
-        return '<base64>' . base64_encode($this->data) . '</base64>';
+        return '<base64>'.base64_encode($this->data).'</base64>';
     }
 }
 
@@ -930,7 +930,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.');
+                                 'server error. requested method "'.$this->message->methodName.'" not specified.');
         }
         $method     = $this->callbacks[$methodname];
         $signature  = $this->signatures[$methodname];
@@ -943,7 +943,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) {
@@ -992,7 +992,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

@@ -352,7 +352,7 @@ 
     public $_feed;
 
     /**
-     * @param $format
+     * @param string $format
      */
     public function _setFormat($format) {
         switch (strtoupper($format)) {
@@ -529,7 +529,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
      */
@@ -547,6 +547,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) {
@@ -652,7 +653,7 @@ 
     /**
      * @since  1.4
      * @access private
-     * @param $filename
+     * @param string $filename
      */
     public function _redirect($filename) {
         // attention, heavily-commented-out-area
@@ -807,7 +808,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() {
         //return gmdate("r",$this->unix);
@@ -822,7 +823,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() {
         $date = gmdate("Y-m-d\TH:i:sO", $this->unix);
@@ -949,7 +950,7 @@ 
     /**
      * Sets this RSS feed's version number.
      * @access private
-     * @param $version
+     * @param string $version
      */
     public function _setRSSVersion($version) {
         $this->RSSVersion = $version;

Spacing +174 added lines, -174 removed lines

@@ -325,11 +325,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);
             }
         }
@@ -550,29 +550,29 @@ 
      * @return string the truncated string
      */
     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).' ...';
     }
 
     /**
@@ -581,7 +581,7 @@ 
      * Syndic8.com.
      */
     public function _createGeneratorComment() {
-        return "<!-- generator=\"" . FEEDCREATOR_VERSION . "\" -->\n";
+        return "<!-- generator=\"".FEEDCREATOR_VERSION."\" -->\n";
     }
 
     /**
@@ -597,7 +597,7 @@ 
         $ae = '';
         if (is_array($elements)) {
             foreach ($elements as $key => $value) {
-                $ae .= $indentString . "<$key>$value</$key>\n";
+                $ae .= $indentString."<$key>$value</$key>\n";
             }
         }
 
@@ -610,10 +610,10 @@ 
     public function _createStylesheetReferences() {
         $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;
@@ -646,7 +646,7 @@ 
     public function _generateFilename() {
         $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';
     }
 
     /**
@@ -668,9 +668,9 @@ 
         // HTTP redirect, some feed readers' simple HTTP implementations don't follow it
         //Header("Location: ".$filename);
 
-        header('Content-Type: ' . $this->contentType . '; charset=' . $this->encoding . '; filename='
+        header('Content-Type: '.$this->contentType.'; charset='.$this->encoding.'; filename='
                . basename($filename));
-        header('Content-Disposition: inline; filename=' . basename($filename));
+        header('Content-Disposition: inline; filename='.basename($filename));
         readfile($filename, 'r');
         die();
     }
@@ -691,7 +691,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);
         }
     }
@@ -716,7 +716,7 @@ 
             if ($displayContents) {
                 $this->_redirect($filename);
             }
-        } else {
+        }else {
             echo '<br><b>Error creating feed file, please check write permissions.</b><br>';
         }
     }
@@ -749,7 +749,7 @@ 
         }
         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     = array(
+            $months = array(
                 'Jan' => 1,
                 'Feb' => 2,
                 'Mar' => 3,
@@ -765,15 +765,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;
                     }
@@ -791,8 +791,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;
                 }
@@ -813,7 +813,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;
@@ -826,7 +826,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);
         }
@@ -859,7 +859,7 @@ 
      * @return string the feed's complete text
      */
     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';
@@ -870,50 +870,50 @@ 
         $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";
+        $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", '  ')))
+            $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 .= '        <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";
         }
@@ -961,84 +961,84 @@ 
      * @return string the feed's complete text
      */
     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)
+            $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 .= '            <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)
+            $feed .= '        <copyright>'.FeedCreator::iTrunc(htmlspecialchars($this->copyright), 100)
                      . "</copyright>\n";
         }
         if ($this->editor != '') {
-            $feed .= '        <managingEditor>' . FeedCreator::iTrunc(htmlspecialchars($this->editor), 100)
+            $feed .= '        <managingEditor>'.FeedCreator::iTrunc(htmlspecialchars($this->editor), 100)
                      . "</managingEditor>\n";
         }
         if ($this->webmaster != '') {
-            $feed .= '        <webMaster>' . FeedCreator::iTrunc(htmlspecialchars($this->webmaster), 100)
+            $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
@@ -1047,17 +1047,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";
@@ -1107,33 +1107,33 @@ 
      * @return string
      */
     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)
+            $feed .= '        <title>'.FeedCreator::iTrunc(htmlspecialchars(strip_tags($this->items[$i]->title)), 100)
                      . "</title>\n";
-            $feed .= '        <link>' . htmlspecialchars($this->items[$i]->link) . "</link>\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()
+            $feed .= "            <div xmlns=\"http://www.w3.org/1999/xhtml\">".$this->items[$i]->getDescription()
                      . "</div>\n";
             $feed .= "        </content>\n";
             $feed .= "    </entry>\n";
@@ -1174,51 +1174,51 @@ 
      * @return string
      */
     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";
+        $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 .= '    <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 .= '        <title>'.htmlspecialchars(strip_tags($this->items[$i]->title))."</title>\n";
             $feed .= "        <link rel=\"alternate\" type=\"text/html\" href=\""
-                     . htmlspecialchars($this->items[$i]->link) . "\"/>\n";
+                     . 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";
         }
@@ -1260,23 +1260,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);
@@ -1287,22 +1287,22 @@ 
      * @return string the feed's complete text
      */
     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));
             $feed .= preg_replace("~\nFrom ([^\n]*)(\n?)~", "\n>From $1$2\n", $body);
@@ -1322,7 +1322,7 @@ 
     public function _generateFilename() {
         $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';
     }
 }
 
@@ -1346,35 +1346,35 @@ 
      * @return string
      */
     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";
@@ -1449,7 +1449,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
@@ -1460,26 +1460,26 @@ 
         // use this array to put the lines in and implode later with "document.write" javascript
         $feedArray = array();
         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>';
             $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(']]>', '',
+            $feedArray[] = "<div class='".$this->stylePrefix."description'>".str_replace(']]>', '',
                                                                                                str_replace('<![CDATA[',
                                                                                                            '',
                                                                                                            $this->getDescription()))
@@ -1487,29 +1487,29 @@ 
         }
 
         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 . '>'
+                    $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'>"
+                }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(']]>', '',
+                $feedArray[] = "<div class='".$this->stylePrefix."item_description'>".str_replace(']]>', '',
                                                                                                         str_replace('<![CDATA[',
                                                                                                                     '',
                                                                                                                     $this->items[$i]->getDescription()))
@@ -1517,10 +1517,10 @@ 
             }
         }
         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;
     }
@@ -1535,7 +1535,7 @@ 
     public function _generateFilename() {
         $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';
     }
 }
 
@@ -1559,7 +1559,7 @@ 
 
         $jsFeed = '';
         foreach ($feedArray as $value) {
-            $jsFeed .= "document.write('" . trim(addslashes($value)) . "');\n";
+            $jsFeed .= "document.write('".trim(addslashes($value))."');\n";
         }
 
         return $jsFeed;
@@ -1575,7 +1575,7 @@ 
     public function _generateFilename() {
         $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

@@ -302,7 +302,7 @@ 
 
     // smart append - field and namespace aware
     /**
-     * @param $el
+     * @param string $el
      * @param $text
      */
     public function append($el, $text) {
@@ -384,7 +384,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return string|false
      */
     public function is_rss() {
         if ($this->feed_type == RSS) {
@@ -395,7 +395,7 @@ 
     }
 
     /**
-     * @return bool
+     * @return string|false
      */
     public function is_atom() {
         if ($this->feed_type == ATOM) {
@@ -407,10 +407,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) {
@@ -517,7 +517,7 @@ 
 
     /**
      * @param $enc
-     * @return bool|string
+     * @return string|false
      */
     public function known_encoding($enc) {
         $enc = strtoupper($enc);

Spacing +31 added lines, -31 removed lines

@@ -35,16 +35,16 @@ 
 {
     public $parser;
 
-    public $current_item = array();  // item currently being parsed
-    public $items        = array();  // collection of parsed items
-    public $channel      = array();  // hash of channel fields
+    public $current_item = array(); // item currently being parsed
+    public $items        = array(); // collection of parsed items
+    public $channel      = array(); // hash of channel fields
     public $textinput    = array();
     public $image        = array();
     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,7 +101,7 @@ 
         # 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',
+            $this->error("Failed to load PHP's XML Extension. ".'http://www.php.net/manual/en/ref.xml.php',
                          E_USER_ERROR);
         }
 
@@ -218,8 +218,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 @@ 
     public function feed_cdata($p, $text) {
         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);
         }
@@ -266,12 +266,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);
         }
 
@@ -319,7 +319,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) {
@@ -336,7 +336,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'];
@@ -348,7 +348,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;
                     }
                 }
@@ -357,7 +357,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'];
@@ -368,12 +368,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;
                     }
                 }
@@ -389,7 +389,7 @@ 
     public function is_rss() {
         if ($this->feed_type == RSS) {
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -400,7 +400,7 @@ 
     public function is_atom() {
         if ($this->feed_type == ATOM) {
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -416,7 +416,7 @@ 
     public function create_parser($source, $out_enc, $in_enc, $detect) {
         if (substr(phpversion(), 0, 1) == 5) {
             $parser = $this->php5_create_parser($in_enc, $detect);
-        } else {
+        }else {
             list($parser, $source) = $this->php4_create_parser($source, $in_enc, $detect);
         }
         if ($out_enc) {
@@ -442,7 +442,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('');
         }
     }
@@ -474,7 +474,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';
             }
         }
@@ -523,7 +523,7 @@ 
         $enc = strtoupper($enc);
         if (in_array($enc, $this->_KNOWN_ENCODINGS)) {
             return $enc;
-        } else {
+        }else {
             return false;
         }
     }
@@ -539,14 +539,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;
         }
     }
@@ -580,12 +580,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) = array($match[8], $match[9], $match[10]);
 
             # zero out the variables
@@ -596,20 +596,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/xmlparser.php

Doc Comments +1 added lines, -1 removed lines

@@ -83,7 +83,7 @@
     }
 
     /**
-     * @return bool|string
+     * @return string|false
      */
     public function is_atom() {
         if ($this->feed_type == 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
@@ -90,7 +90,7 @@ 
             $this->feed_version = empty($this->feed_version) ? '0.3' : $this->feed_version;
 
             return $this->feed_version;
-        } else {
+        }else {
             return false;
         }
     }
@@ -102,7 +102,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;
@@ -122,7 +122,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'])) {
@@ -202,7 +202,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

@@ -108,7 +108,7 @@ 
      * @var mixed  $pattern
      * @var mixed  $replacement
      *
-     * @return bool true on success
+     * @return null|boolean true on success
      */
     function planet_parse_class($class_string, $pattern = '', $replacement = '') {
         if (empty($class_string)) {
@@ -143,7 +143,7 @@ 
      * @var mixed  $pattern
      * @var mixed  $replacement
      *
-     * @return bool true on success
+     * @return null|boolean true on success
      */
     function planet_parse_function($function_string, $pattern = '', $replacement = '') {
         if (empty($function_string)) {
@@ -494,7 +494,7 @@ 
 
     /**
      * @param $url
-     * @return bool|string
+     * @return false|string
      */
     function planet_fetch_fopen($url) {
         if (!$fp = @fopen($url, 'r')) {

Spacing +43 added lines, -43 removed lines

@@ -37,9 +37,9 @@ 
 if (!defined('planet_FUNCTIONS')):
     define('planet_FUNCTIONS', 1);
 
-    require XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/vars.php';
-    include_once XOOPS_ROOT_PATH . '/class/xoopslists.php';
-    include_once XOOPS_ROOT_PATH . '/Frameworks/art/functions.php';
+    require XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/vars.php';
+    include_once XOOPS_ROOT_PATH.'/class/xoopslists.php';
+    include_once XOOPS_ROOT_PATH.'/Frameworks/art/functions.php';
 
     /**
      * Function to display messages
@@ -66,7 +66,7 @@ 
      * - "&" in a string
     */
     function planet_parse_args(&$args_numeric, &$args, &$args_string) {
-        $args_abb     = array(
+        $args_abb = array(
             'a' => 'article',
             'b' => 'blog',
             'c' => 'category',
@@ -81,24 +81,24 @@ 
         if (preg_match("/[^\?]*\.php[\/|\?]([^\?]*)/i", $_SERVER['REQUEST_URI'], $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;
     }
 
     /**
@@ -120,7 +120,7 @@ 
             $pattern     = array($pattern);
             $replacement = array($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)) {
@@ -155,7 +155,7 @@ 
             $pattern     = array($pattern);
             $replacement = array($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)) {
@@ -196,7 +196,7 @@ 
         if (!is_array($userid)) {
             $userid = array($userid);
         }
-        $users =& mod_getUnameFromIds($userid, $usereal, $linked);
+        $users = & mod_getUnameFromIds($userid, $usereal, $linked);
 
         return $users;
     }
@@ -212,7 +212,7 @@ 
         $myts       = MyTextSanitizer::getInstance();
         $link_array = preg_split("/(\r\n|\r|\n)( *)/", $text);
         $links      = array();
-        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)) {
@@ -231,7 +231,7 @@ 
      * @return string
      */
     function planet_getTemplate($pagename) {
-        return $GLOBALS['VAR_PREFIX'] . '_' . $pagename . '.tpl';
+        return $GLOBALS['VAR_PREFIX'].'_'.$pagename.'.tpl';
     }
 
     /**
@@ -257,7 +257,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 planet_trackback($trackback_url, $article);
     }
@@ -272,7 +272,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);
@@ -280,18 +280,18 @@ 
         $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'] ? '?'
+        $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 .= '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']) {
@@ -300,11 +300,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);
@@ -314,7 +314,7 @@ 
             if ($fp = fopen($debug_file, 'a')) {
                 fwrite($fp, $fr);
                 fclose($fp);
-            } else {
+            }else {
             }
         }
         @fclose($fs);
@@ -333,19 +333,19 @@ 
                 planet_ping($serv, $id);
             }
         }
-        include_once XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/class-IXR.php';
+        include_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);
@@ -360,16 +360,16 @@ 
     function planet_trackback_response($error = 0, $error_message = '') {
         $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>';
@@ -388,11 +388,11 @@ 
         if (is_array($string)) {
             $value = array();
             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, '/');
     }
 
     /**
@@ -401,14 +401,14 @@ 
      * @return array|null
      */
     function planet_getcookie($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) : array();
             $value  = array();
-            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;
                 }
             }
@@ -462,7 +462,7 @@ 
      * @return string
      */
     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)) {
@@ -480,10 +480,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
@@ -523,7 +523,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

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

Spacing +74 added lines, -74 removed lines

@@ -24,7 +24,7 @@ 
 // URL: http://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (planet_parse_args($args_num, $args, $args_str)) {
     $args['year']  = @$args_num[0];
@@ -32,37 +32,37 @@ 
     $args['day']   = @$args_num[2];
 }
 
-$day     = (int)(empty($_GET['day']) ? @$args['day'] : $_GET['day']);
-$year    = (int)(empty($_GET['year']) ? @$args['year'] : $_GET['year']);
-$month   = (int)(empty($_GET['month']) ? @$args['month'] : $_GET['month']);
-$blog_id = (int)(empty($_GET['blog']) ? @$args['blog'] : $_GET['blog']);
-$start   = (int)(empty($_GET['start']) ? @$args['start'] : $_GET['start']);
+$day     = (int) (empty($_GET['day']) ? @$args['day'] : $_GET['day']);
+$year    = (int) (empty($_GET['year']) ? @$args['year'] : $_GET['year']);
+$month   = (int) (empty($_GET['month']) ? @$args['month'] : $_GET['month']);
+$blog_id = (int) (empty($_GET['blog']) ? @$args['blog'] : $_GET['blog']);
+$start   = (int) (empty($_GET['start']) ? @$args['start'] : $_GET['start']);
 
 $page['title'] = planet_constant('MD_ACHIVE');
 
 $article_handler = xoops_getModuleHandler('article', $GLOBALS['moddirname']);
 $blog_handler    = 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']   = planet_getTemplate('archive');
-include_once XOOPS_ROOT_PATH . '/header.php';
-include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/vars.php';
+include_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 = array('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();
@@ -79,13 +79,13 @@ 
 $criteria->setStart($start);
 $criteria->setLimit($xoopsModuleConfig['articles_perpage']);
 
-$articles_obj   =& $article_handler->getAll($criteria, array('uid', 'art_title', 'art_time', 'blog_id', 'art_content'));
+$articles_obj   = & $article_handler->getAll($criteria, array('uid', 'art_title', 'art_time', 'blog_id', 'art_content'));
 $articles_count = $article_handler->getCount($criteria);
 
 $articles = array();
 $blogs_id = array();
 foreach ($articles_obj as $id => $article) {
-    $articles[]                            = array(
+    $articles[] = array(
         'id'      => $id,
         'blog'    => array('id' => $article->getVar('blog_id'), 'title' => ''),
         'title'   => $article->getVar('art_title'),
@@ -96,22 +96,22 @@ 
     $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         = $blog_handler->getList($criteria_blog);
 foreach (array_keys($articles) as $key) {
     $articles[$key]['blog']['title'] = $blogs[$articles[$key]['blog']['id']];
 }
-if ($blog_id > 0) {
-    $page['blog'] = $blogs[$blog_id];;
+if ($blog_id>0) {
+    $page['blog'] = $blogs[$blog_id]; ;
 }
 
-if ($articles_count > $xoopsModuleConfig['articles_perpage']) {
-    include XOOPS_ROOT_PATH . '/class/pagenav.php';
+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);
+                                'month='.$month.'&amp;day='.$day.'&amp;year='.$year.'&amp;blog='
+                                . (int) $blog_id);
     $pagenav = $nav->renderNav(4);
-} else {
+}else {
     $pagenav = '';
 }
 
@@ -120,46 +120,46 @@ 
 $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 = array();
         while ($myrow = $xoopsDB->fetchArray($result)) {
             $months[] = array(
-                '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'] = array(
-            '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'] = array(
-                '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);
@@ -167,38 +167,38 @@ 
         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] = array(
                 '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'] = array(
-            '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'] = array(
-                '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)
             );
         }
     }
@@ -218,7 +218,7 @@ 
 $xoopsTpl->assign('timenav', $timenav);
 $xoopsTpl->assign('pagenav', $pagenav);
 
-include_once __DIR__ . '/footer.php';
+include_once __DIR__.'/footer.php';
 
 /**
  * @param null $year
@@ -234,25 +234,25 @@ 
     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";
         }
@@ -261,26 +261,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>";
@@ -298,5 +298,5 @@ 
 function planet_calendar_week_mod($num) {
     $base = 7;
 
-    return ($num - $base * floor($num / $base));
+    return ($num-$base*floor($num/$base));
 }

index.php

Spacing +68 added lines, -68 removed lines

@@ -24,10 +24,10 @@ 
 // URL: http://xoops.org                         //
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 if (preg_match("/\/notification_update\.php/i", $_SERVER['REQUEST_URI'], $matches)) {
-    include XOOPS_ROOT_PATH . '/include/notification_update.php';
+    include XOOPS_ROOT_PATH.'/include/notification_update.php';
     exit();
 }
 
@@ -37,17 +37,17 @@ 
 }
 
 /* Start */
-$start = (int)(empty($_GET['start']) ? @$args['start'] : $_GET['start']);
+$start = (int) (empty($_GET['start']) ? @$args['start'] : $_GET['start']);
 /* Specified Category */
-$category_id = (int)(empty($_GET['category']) ? @$args['category'] : $_GET['category']);
+$category_id = (int) (empty($_GET['category']) ? @$args['category'] : $_GET['category']);
 /* Specified Blog */
-$blog_id = (int)(empty($_GET['blog']) ? @$args['blog'] : $_GET['blog']);
+$blog_id = (int) (empty($_GET['blog']) ? @$args['blog'] : $_GET['blog']);
 /* Specified Bookmar(Favorite) UID */
-$uid = (int)(empty($_GET['uid']) ? @$args['uid'] : $_GET['uid']);
+$uid = (int) (empty($_GET['uid']) ? @$args['uid'] : $_GET['uid']);
 /* Sort by term */
 $sort = empty($_GET['sort']) ? @$args['sort'] : $_GET['sort'];
 /* Display as list */
-$list = (int)(empty($_GET['list']) ? @$args['list'] : $_GET['list']);
+$list = (int) (empty($_GET['list']) ? @$args['list'] : $_GET['list']);
 
 // restore $_SERVER['REQUEST_URI']
 if (!empty($REQUEST_URI_parsed)) {
@@ -55,35 +55,35 @@ 
     $_args            = array('start', 'sort', 'uid', 'list');
     foreach ($_args as $arg) {
         if (!empty(${$arg})) {
-            $args_REQUEST_URI[] = $arg . '=' . ${$arg};
+            $args_REQUEST_URI[] = $arg.'='.${$arg};
         }
     }
     if (!empty($blog_id)) {
-        $args_REQUEST_URI[] = 'blog=' . $blog_id;
+        $args_REQUEST_URI[] = 'blog='.$blog_id;
     }
     if (!empty($category_id)) {
-        $args_REQUEST_URI[] = 'category=' . $category_id;
+        $args_REQUEST_URI[] = 'category='.$category_id;
     }
-    $_SERVER['REQUEST_URI'] = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php'
-                              . (empty($args_REQUEST_URI) ? '' : '?' . implode('&', $args_REQUEST_URI));
+    $_SERVER['REQUEST_URI'] = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'
+                              . (empty($args_REQUEST_URI) ? '' : '?'.implode('&', $args_REQUEST_URI));
 }
 
 $xoopsOption['template_main'] = planet_getTemplate('index');
 $xoops_module_header          = '
-    <link rel="alternate" type="application/rss+xml" title="' . $xoopsModule->getVar('name') . ' rss" href="'
-                                . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/xml.php".URL_DELIMITER."rss/c'
-                                . $category_id . '/b' . $blog_id . '/u' . $uid . '" />
-    <link rel="alternate" type="application/rss+xml" title="' . $xoopsModule->getVar('name') . ' rdf" href="'
-                                . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/xml.php".URL_DELIMITER."rdf/c'
-                                . $category_id . '/b' . $blog_id . '/u' . $uid . '" />
-    <link rel="alternate" type="application/atom+xml" title="' . $xoopsModule->getVar('name') . ' atom" href="'
-                                . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/xml.php".URL_DELIMITER."atom/c'
-                                . $category_id . '/b' . $blog_id . '/u' . $uid . '" />
+    <link rel="alternate" type="application/rss+xml" title="' . $xoopsModule->getVar('name').' rss" href="'
+                                . XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/xml.php".URL_DELIMITER."rss/c'
+                                . $category_id.'/b'.$blog_id.'/u'.$uid.'" />
+    <link rel="alternate" type="application/rss+xml" title="' . $xoopsModule->getVar('name').' rdf" href="'
+                                . XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/xml.php".URL_DELIMITER."rdf/c'
+                                . $category_id.'/b'.$blog_id.'/u'.$uid.'" />
+    <link rel="alternate" type="application/atom+xml" title="' . $xoopsModule->getVar('name').' atom" href="'
+                                . XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/xml.php".URL_DELIMITER."atom/c'
+                                . $category_id.'/b'.$blog_id.'/u'.$uid.'" />
     ';
 
 $xoopsOption['xoops_module_header'] = $xoops_module_header;
-include_once XOOPS_ROOT_PATH . '/header.php';
-include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/vars.php';
+include_once XOOPS_ROOT_PATH.'/header.php';
+include XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/vars.php';
 
 // Following part will not be executed after cache
 $category_handler = xoops_getModuleHandler('category', $GLOBALS['moddirname']);
@@ -95,7 +95,7 @@ 
 $criteria       = new CriteriaCompo();
 $article_prefix = '';
 /* Specific category */
-if ($category_id > 0) {
+if ($category_id>0) {
     $category_obj = $category_handler->get($category_id);
     $criteria->add(new Criteria('bc.cat_id', $category_id));
     $uid            = 0;
@@ -105,8 +105,8 @@ 
     $article_prefix = 'a.';
 }
 /* Specific blog */
-if ($blog_id > 0) {
-    $blog_obj =& $blog_handler->get($blog_id);
+if ($blog_id>0) {
+    $blog_obj = & $blog_handler->get($blog_id);
     if ($blog_obj->getVar('blog_status')
         || (is_object($xoopsUser)
             && $xoopsUser->getVar('uid') == $blog_obj->getVar('blog_submitter'))
@@ -133,7 +133,7 @@ 
     $article_prefix = '';
 }
 /* User bookmarks(favorites) */
-if ($uid > 0) {
+if ($uid>0) {
     $criteria->add(new Criteria('bm.bm_uid', $uid));
     $category_id      = 0;
     $blog_id          = 0;
@@ -152,13 +152,13 @@ 
 $sort  = empty($sort) ? 'time' : $sort;
 switch ($sort) {
     case 'views':
-        $sortby = $article_prefix . 'art_views';
+        $sortby = $article_prefix.'art_views';
         break;
     case 'rating':
-        $sortby = $article_prefix . 'art_rating';
+        $sortby = $article_prefix.'art_rating';
         break;
     case 'time':
-        $sortby = $article_prefix . 'art_time';
+        $sortby = $article_prefix.'art_time';
         break;
     case 'default':
     default:
@@ -171,33 +171,33 @@ 
 $criteria->setLimit($limit);
 
 $tags = empty($list) ? '' : array(
-    $article_prefix . 'art_title',
-    $article_prefix . 'blog_id',
-    $article_prefix . 'art_time'
+    $article_prefix.'art_title',
+    $article_prefix.'blog_id',
+    $article_prefix.'art_time'
 );
 switch ($query_type) {
     case 'category':
-        $articles_obj  =& $article_handler->getByCategory($criteria, $tags);
+        $articles_obj  = & $article_handler->getByCategory($criteria, $tags);
         $count_article = $article_handler->getCountByCategory($criteria);
         break;
     case 'bookmark':
-        $articles_obj  =& $article_handler->getByBookmark($criteria, $tags);
+        $articles_obj  = & $article_handler->getByBookmark($criteria, $tags);
         $count_article = $article_handler->getCountByBookmark($criteria);
         break;
     default:
-        $articles_obj  =& $article_handler->getAll($criteria, $tags);
+        $articles_obj  = & $article_handler->getAll($criteria, $tags);
         $count_article = $article_handler->getCount($criteria);
         break;
 }
 
 if (!empty($blog_data)) {
     $blogs[$blog_data['id']] = $blog_data['title'];
-} else {
+}else {
     $blog_array = array();
     foreach (array_keys($articles_obj) as $id) {
         $blog_array[$articles_obj[$id]->getVar('blog_id')] = 1;
     }
-    $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         = $blog_handler->getList($criteria_blog);
 }
 
@@ -223,7 +223,7 @@ 
         ));
         if (!empty($xoopsModuleConfig['display_summary'])) {
             $_article['content'] = $articles_obj[$id]->getSummary();
-        } else {
+        }else {
             $_article['content'] = $articles_obj[$id]->getVar('art_content');
         }
     }
@@ -232,24 +232,24 @@ 
 }
 unset($articles_obj);
 
-if ($count_article > $limit) {
-    include XOOPS_ROOT_PATH . '/class/pagenav.php';
+if ($count_article>$limit) {
+    include XOOPS_ROOT_PATH.'/class/pagenav.php';
     $start_link = array();
     if ($sort) {
-        $start_link[] = 'sort=' . $sort;
+        $start_link[] = 'sort='.$sort;
     }
     if ($category_id) {
-        $start_link[] = 'category=' . $category_id;
+        $start_link[] = 'category='.$category_id;
     }
     if ($blog_id) {
-        $start_link[] = 'blog=' . $blog_id;
+        $start_link[] = 'blog='.$blog_id;
     }
     if ($list) {
-        $start_link[] = 'list=' . $list;
+        $start_link[] = 'list='.$list;
     }
     $nav     = new XoopsPageNav($count_article, $limit, $start, 'start', implode('&amp;', $start_link));
     $pagenav = $nav->renderNav(4);
-} else {
+}else {
     $pagenav = '';
 }
 
@@ -258,36 +258,36 @@ 
 
 if ($category_id || $blog_id || $uid) {
     $xoopsTpl->assign('link_index',
-                      "<a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . "/index.php\" title=\""
-                      . planet_constant('MD_INDEX') . "\" target=\"_self\">" . planet_constant('MD_INDEX') . '</a>');
+                      "<a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname']."/index.php\" title=\""
+                      . planet_constant('MD_INDEX')."\" target=\"_self\">".planet_constant('MD_INDEX').'</a>');
 }
 
-$link_switch = "<a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php'
-               . (empty($category_id) ? '' : '/c' . $category_id) . (empty($uid) ? '' : '/u' . $uid)
-               . (empty($blog_id) ? '' : '/b' . $blog_id) . (empty($list) ? '/l1' : '') . "\" title=\""
-               . (empty($list) ? planet_constant('MD_LISTVIEW') : planet_constant('MD_FULLVIEW')) . "\">"
-               . (empty($list) ? planet_constant('MD_LISTVIEW') : planet_constant('MD_FULLVIEW')) . '</a>';
+$link_switch = "<a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'
+               . (empty($category_id) ? '' : '/c'.$category_id).(empty($uid) ? '' : '/u'.$uid)
+               . (empty($blog_id) ? '' : '/b'.$blog_id).(empty($list) ? '/l1' : '')."\" title=\""
+               . (empty($list) ? planet_constant('MD_LISTVIEW') : planet_constant('MD_FULLVIEW'))."\">"
+               . (empty($list) ? planet_constant('MD_LISTVIEW') : planet_constant('MD_FULLVIEW')).'</a>';
 $xoopsTpl->assign('link_switch', $link_switch);
 
-$link_blogs = "<a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/view.blogs.php'
-              . (empty($category_id) ? '' : '/c' . $category_id) . (empty($uid) ? '' : '/u' . $uid) . "\" title=\""
-              . planet_constant('MD_BLOGS') . "\">" . planet_constant('MD_BLOGS') . '</a>';
+$link_blogs = "<a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/view.blogs.php'
+              . (empty($category_id) ? '' : '/c'.$category_id).(empty($uid) ? '' : '/u'.$uid)."\" title=\""
+              . planet_constant('MD_BLOGS')."\">".planet_constant('MD_BLOGS').'</a>';
 $xoopsTpl->assign('link_blogs', $link_blogs);
 
 if (empty($uid) && is_object($xoopsUser)) {
     $xoopsTpl->assign('link_bookmark',
-                      "<a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php' . URL_DELIMITER
-                      . 'u' . $xoopsUser->getVar('uid') . "\" title=\"" . planet_constant('MD_BOOKMARKS')
-                      . "\" target=\"_self\">" . planet_constant('MD_BOOKMARKS') . '</a>');
+                      "<a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'.URL_DELIMITER
+                      . 'u'.$xoopsUser->getVar('uid')."\" title=\"".planet_constant('MD_BOOKMARKS')
+                      . "\" target=\"_self\">".planet_constant('MD_BOOKMARKS').'</a>');
 }
 
 if ($xoopsModuleConfig['newblog_submit'] == 1 || is_object($xoopsUser)) {
     $xoopsTpl->assign('link_submit',
-                      "<a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . "/action.blog.php\" title=\""
-                      . _SUBMIT . "\" target=\"_blank\">" . _SUBMIT . '</a>');
+                      "<a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname']."/action.blog.php\" title=\""
+                      . _SUBMIT."\" target=\"_blank\">"._SUBMIT.'</a>');
 }
 
-$xoopsTpl->assign('pagetitle', $xoopsModule->getVar('name') . '::' . planet_constant('MD_ARTICLES'));
+$xoopsTpl->assign('pagetitle', $xoopsModule->getVar('name').'::'.planet_constant('MD_ARTICLES'));
 $xoopsTpl->assign('category', @$category_data);
 $xoopsTpl->assign('blog', @$blog_data);
 $xoopsTpl->assign('user', @$user_data);
@@ -297,13 +297,13 @@ 
 
 $xoopsTpl->assign('user_level', !is_object($xoopsUser) ? 0 : ($xoopsUser->isAdmin() ? 2 : 1));
 if (empty($xoopsModuleConfig['anonymous_rate']) && !is_object($xoopsUser)) {
-} elseif ($blog_id > 0) {
+} elseif ($blog_id>0) {
     $xoopsTpl->assign('canrate', 1);
 }
 
-$sort_link   = XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php' . (empty($category_id) ? '' : '/c'
+$sort_link = XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'.(empty($category_id) ? '' : '/c'
                                                                                                              . $category_id)
-               . (empty($uid) ? '' : '/u' . $uid) . (empty($blog_id) ? '' : '/b' . $blog_id)
+               . (empty($uid) ? '' : '/u'.$uid).(empty($blog_id) ? '' : '/b'.$blog_id)
                . (empty($list) ? '' : '/l1');
 $valid_sorts = array(
     'views'   => planet_constant('MD_VIEWS'),
@@ -311,12 +311,12 @@ 
     'time'    => planet_constant('MD_TIME'),
     'default' => planet_constant('MD_DEFAULT')
 );
-$sortlinks   = array();
+$sortlinks = array();
 foreach ($valid_sorts as $val => $name) {
     if ($val == $sort) {
         continue;
     }
-    $sortlinks[] = "<a href=\"" . $sort_link . '/' . $val . "\">" . $name . '</a>';
+    $sortlinks[] = "<a href=\"".$sort_link.'/'.$val."\">".$name.'</a>';
 }
 $xoopsTpl->assign('link_sort', implode(' | ', $sortlinks));
 $xoopsTpl->assign('version', $xoopsModule->getVar('version'));
@@ -329,4 +329,4 @@ 
     $_GET['blog'] = $blog_id;
 }
 
-include_once __DIR__ . '/footer.php';
+include_once __DIR__.'/footer.php';

action.blog.php

Spacing +23 added lines, -23 removed lines

@@ -25,11 +25,11 @@ 
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 $op      = !empty($_POST['op']) ? $_POST['op'] : (!empty($_GET['op']) ? $_GET['op'] : '');
 $blog_id = !empty($_POST['blog']) ? $_POST['blog'] : (!empty($_GET['blog']) ? $_GET['blog'] : 0);
-$blog_id = is_array($blog_id) ? array_map('intval', $blog_id) : (int)$blog_id;
+$blog_id = is_array($blog_id) ? array_map('intval', $blog_id) : (int) $blog_id;
 
 if (empty($xoopsModuleConfig['newblog_submit']) && (!is_object($xoopsUser) || !$xoopsUser->isAdmin())) {
     redirect_header('index.php', 2, _NOPERM);
@@ -40,10 +40,10 @@ 
 }
 
 if ($op === 'save' && !$GLOBALS['xoopsSecurity']->check()) {
-    redirect_header('javascript:history.go(-1);', 1, planet_constant('MD_INVALID') . ': security check failed');
+    redirect_header('javascript:history.go(-1);', 1, planet_constant('MD_INVALID').': security check failed');
 }
-include XOOPS_ROOT_PATH . '/header.php';
-include XOOPS_ROOT_PATH . '/modules/' . $xoopsModule->getVar('dirname') . '/include/vars.php';
+include XOOPS_ROOT_PATH.'/header.php';
+include XOOPS_ROOT_PATH.'/modules/'.$xoopsModule->getVar('dirname').'/include/vars.php';
 
 $blog_handler     = xoops_getModuleHandler('blog', $GLOBALS['moddirname']);
 $category_handler = xoops_getModuleHandler('category', $GLOBALS['moddirname']);
@@ -53,25 +53,25 @@ 
     case 'save':
 
         if ($blog_id) {
-            $blog_obj =& $blog_handler->get($blog_id);
+            $blog_obj = & $blog_handler->get($blog_id);
             if ($xoopsUser->isAdmin()) {
                 $blog_obj->setVar('blog_status', @$_POST['blog_status']);
             }
-        } else {
+        }else {
             if ($blog_exists = $blog_handler->getCount(new Criteria('blog_feed',
                                                                     $myts->addSlashes(trim($_POST['blog_feed']))))
             ) {
                 redirect_header('index.php', 2, planet_constant('MD_BLOGEXISTS'));
             }
 
-            $blog_obj =& $blog_handler->create();
+            $blog_obj = & $blog_handler->create();
             $blog_obj->setVar('blog_submitter', is_object($xoopsUser) ? $xoopsUser->getVar('uid') : planet_getIP(true));
 
             switch ($xoopsModuleConfig['newblog_submit']) {
                 case 2:
                     if (!is_object($xoopsUser)) {
                         $status = 0;
-                    } else {
+                    }else {
                         $status = 1;
                     }
                     break;
@@ -83,7 +83,7 @@ 
                 default:
                     if (!is_object($xoopsUser) || !$xoopsUser->isAdmin()) {
                         $status = 0;
-                    } else {
+                    }else {
                         $status = 1;
                     }
                     break;
@@ -113,22 +113,22 @@ 
             $blog_handler->setCategories($blog_id, $_POST['categories']);
         }
         $message = planet_constant('MD_DBUPDATED');
-        redirect_header('index.php' . URL_DELIMITER . 'b' . $blog_id, 2, $message);
+        redirect_header('index.php'.URL_DELIMITER.'b'.$blog_id, 2, $message);
 
     /* edit a single blog */
     case 'edit':
     default:
         if (!empty($_POST['fetch'])) {
-            $blog_obj =& $blog_handler->fetch($_POST['blog_feed']);
+            $blog_obj = & $blog_handler->fetch($_POST['blog_feed']);
             $blog_obj->setVar('blog_id', $blog_id);
-        } else {
-            $blog_obj =& $blog_handler->get($blog_id);
+        }else {
+            $blog_obj = & $blog_handler->get($blog_id);
         }
         $categories = isset($_POST['categories']) ? $_POST['categories'] : array();
         if (in_array('-1', $categories)) {
             $categories = array();
         }
-        if (empty($categories) && $blog_id > 0) {
+        if (empty($categories) && $blog_id>0) {
             $crit       = new Criteria('bc.blog_id', $blog_id);
             $categories = array_keys($category_handler->getByBlog($crit));
         }
@@ -136,24 +136,24 @@ 
             $categories = array(0 => _NONE);
         }
 
-        echo "<fieldset><legend style='font-weight: bold; color: #900;'>" . _EDIT . '</legend>';
+        echo "<fieldset><legend style='font-weight: bold; color: #900;'>"._EDIT.'</legend>';
         echo '<br>';
         if (empty($blog_id) && $blog_obj->getVar('blog_feed')) {
             $criteria  = new Criteria('blog_feed', $blog_obj->getVar('blog_feed'));
-            $blogs_obj =& $blog_handler->getList($criteria);
-            if (count($blogs_obj) > 0) {
-                echo "<div class=\"errorMsg\">" . planet_constant('MD_BLOGEXISTS');
+            $blogs_obj = & $blog_handler->getList($criteria);
+            if (count($blogs_obj)>0) {
+                echo "<div class=\"errorMsg\">".planet_constant('MD_BLOGEXISTS');
                 foreach (array_keys($blogs_obj) as $bid) {
-                    echo "<br><a href=\"" . XOOPS_URL . '/modules/' . $GLOBALS['moddirname'] . '/index.php'
-                         . URL_DELIMITER . 'b' . $bid . "\" target=\"_blank\">" . $blogs_obj[$bid] . '</a>';
+                    echo "<br><a href=\"".XOOPS_URL.'/modules/'.$GLOBALS['moddirname'].'/index.php'
+                         . URL_DELIMITER.'b'.$bid."\" target=\"_blank\">".$blogs_obj[$bid].'</a>';
                 }
                 echo '</div>';
                 unset($blogs_obj, $criteria);
             }
         }
-        include XOOPS_ROOT_PATH . '/modules/' . $GLOBALS['moddirname'] . '/include/form.blog.php';
+        include XOOPS_ROOT_PATH.'/modules/'.$GLOBALS['moddirname'].'/include/form.blog.php';
         echo '</fieldset>';
         break;
 }
 
-include XOOPS_ROOT_PATH . '/footer.php';
+include XOOPS_ROOT_PATH.'/footer.php';

trackback.php

Spacing +25 added lines, -25 removed lines

@@ -25,11 +25,11 @@ 
 // Project: Article Project                                                 //
 // ------------------------------------------------------------------------ //
 
-include __DIR__ . '/header.php';
+include __DIR__.'/header.php';
 
 // trackback is done by a POST
 $art_id     = explode('/', $_SERVER['REQUEST_URI']);
-$article_id = (int)$art_id[count($art_id) - 1];
+$article_id = (int) $art_id[count($art_id)-1];
 $url        = $_POST['url'];
 $title      = $_POST['title'];
 $excerpt    = $_POST['excerpt'];
@@ -39,7 +39,7 @@ 
 if (empty($xoopsModuleConfig['trackback_option'])) {
     planet_trackback_response(1, 'Trackback is closed');
 }
-if (!strlen($title . $url . $blog_name)) {
+if (!strlen($title.$url.$blog_name)) {
     planet_trackback_response(1, planet_constant('MD_INVALID'));
 }
 
@@ -47,7 +47,7 @@ 
     $trackback_handler = xoops_getModuleHandler('trackback', $GLOBALS['moddirname']);
     $criteria          = new CriteriaCompo(new Criteria('art_id', $article_id));
     $criteria->add(new Criteria('tb_url', $url));
-    if ($trackback_handler->getCount($criteria) > 0) {
+    if ($trackback_handler->getCount($criteria)>0) {
         planet_trackback_response(1, 'We already have a ping from that URI for this article.');
     }
 
@@ -55,17 +55,17 @@ 
     $title     = XoopsLocal::convert_encoding($title, _CHARSET, $charset);
     $excerpt   = XoopsLocal::convert_encoding($excerpt, _CHARSET, $charset);
     $blog_name = XoopsLocal::convert_encoding($blog_name, _CHARSET, $charset);
-    $tb_status = (int)$xoopsModuleConfig['trackback_option'];
+    $tb_status = (int) $xoopsModuleConfig['trackback_option'];
 
     $com_pid    = 0;
     $com_itemid = $article_id;
     $com_rootid = 0;
     $com_title  = $title;
     $com_text   = $excerpt;
-    $com_text .= "\n\n[TRACKBACK]" . _POSTEDBY . ': ';
+    $com_text .= "\n\n[TRACKBACK]"._POSTEDBY.': ';
     if (!empty($url)) {
-        $com_text .= '[url=' . $url . ']' . $blog_name . '[/url]';
-    } else {
+        $com_text .= '[url='.$url.']'.$blog_name.'[/url]';
+    }else {
         $com_text .= $blog_name;
     }
     $com_modid = $xoopsModule->getVar('mid');
@@ -122,14 +122,14 @@ 
                 if (isset($comment_config['callbackFile'])) {
                     $callbackfile = trim($comment_config['callbackFile']);
                     if ($callbackfile != ''
-                        && file_exists(XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile)
+                        && file_exists(XOOPS_ROOT_PATH.'/modules/'.$moddir.'/'.$callbackfile)
                     ) {
-                        include_once XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile;
+                        include_once XOOPS_ROOT_PATH.'/modules/'.$moddir.'/'.$callbackfile;
                     }
                     if (!function_exists($comment_config['callback']['approve'])) {
                         $skip = true;
                     }
-                } else {
+                }else {
                     $skip = true;
                 }
             }
@@ -147,14 +147,14 @@ 
                 if (isset($comment_config['callbackFile'])) {
                     $callbackfile = trim($comment_config['callbackFile']);
                     if ($callbackfile != ''
-                        && file_exists(XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile)
+                        && file_exists(XOOPS_ROOT_PATH.'/modules/'.$moddir.'/'.$callbackfile)
                     ) {
-                        include_once XOOPS_ROOT_PATH . '/modules/' . $moddir . '/' . $callbackfile;
+                        include_once XOOPS_ROOT_PATH.'/modules/'.$moddir.'/'.$callbackfile;
                     }
                     if (!function_exists($comment_config['callback']['update'])) {
                         $skip = true;
                     }
-                } else {
+                }else {
                     $skip = true;
                 }
             }
@@ -172,8 +172,8 @@ 
         // trigger notification event if necessary
         if ($notify_event) {
             $not_modid = $com_modid;
-            include_once XOOPS_ROOT_PATH . '/include/notification_functions.php';
-            $not_catinfo  =& notificationCommentCategoryInfo($not_modid);
+            include_once XOOPS_ROOT_PATH.'/include/notification_functions.php';
+            $not_catinfo  = & notificationCommentCategoryInfo($not_modid);
             $not_category = $not_catinfo['name'];
             $not_itemid   = $com_itemid;
             $not_event    = $notify_event;
@@ -181,23 +181,23 @@ 
             // point to a viewable page (i.e. not the system administration
             // module).
             $comment_tags = array();
-            $not_module   =& $xoopsModule;
+            $not_module   = & $xoopsModule;
             if (!isset($comment_url)) {
-                $com_config  =& $not_module->getInfo('comments');
-                $comment_url = $com_config['pageName'] . '?';
+                $com_config  = & $not_module->getInfo('comments');
+                $comment_url = $com_config['pageName'].'?';
                 $comment_url .= $com_config['itemName'];
             }
-            $comment_tags['X_COMMENT_URL'] = XOOPS_URL . '/modules/' . $not_module->getVar('dirname') . '/'
-                                             . $comment_url . '=' . $com_itemid . '&com_id=' . $newcid
-                                             . '&com_rootid=' . $com_rootid . '&com_mode=' . $com_mode
-                                             . '&com_order=' . $com_order . '#comment' . $newcid;
-            $notification_handler          = xoops_getHandler('notification');
+            $comment_tags['X_COMMENT_URL'] = XOOPS_URL.'/modules/'.$not_module->getVar('dirname').'/'
+                                             . $comment_url.'='.$com_itemid.'&com_id='.$newcid
+                                             . '&com_rootid='.$com_rootid.'&com_mode='.$com_mode
+                                             . '&com_order='.$com_order.'#comment'.$newcid;
+            $notification_handler = xoops_getHandler('notification');
             $notification_handler->triggerEvent($not_category, $not_itemid, $not_event, $comment_tags, false,
                                                 $not_modid);
         }
 
         planet_trackback_response(0);
-    } else {
+    }else {
         planet_trackback_response(1, xoops_error($comment->getHtmlErrors()));
     }
 }