AgelxNash / DocLister

assets/lib/SimpleTab/controller.abstract.php

Braces +6 added lines, -2 removed lines

@@ -101,7 +101,9 @@ 
         $this->dlParams['table'] = $this->data->tableName();
         $this->dlParams['idField'] = $this->data->fieldPKName();
         $this->dlParams['addWhereList'] = "`{$this->rfName}`={$this->rid}";
-        if (isset($_REQUEST['rows'])) $this->dlParams['display'] = (int)$_REQUEST['rows'];
+        if (isset($_REQUEST['rows'])) {

+            $this->dlParams['display'] = (int)$_REQUEST['rows'];

+        }

         $offset = isset($_REQUEST['page']) ? (int)$_REQUEST['page'] : 1;
         $offset = $offset ? $offset : 1;
         $offset = $this->dlParams['display']*abs($offset-1);
@@ -113,7 +115,9 @@ 
             $this->dlParams['sortDir'] = $_REQUEST['order'];
         }
         foreach ($this->dlParams as &$param) {
-            if (empty($param)) unset($param);
+            if (empty($param)) {

+                unset($param);

+            }

         }
     }
 

Spacing +3 added lines, -3 removed lines

@@ -1,6 +1,6 @@ 
 <?php namespace SimpleTab;
 
-require_once(MODX_BASE_PATH . 'assets/lib/Helpers/FS.php');
+require_once(MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
 
 /**
  * Class AbstractController
@@ -186,8 +186,8 @@ 
     public function getLanguageCode()
     {
         $manager_language = $this->modx->config['manager_language'];
-        if (file_exists(MODX_MANAGER_PATH . "includes/lang/" . $manager_language . ".inc.php")) {
-            include_once MODX_MANAGER_PATH . "includes/lang/" . $manager_language . ".inc.php";
+        if (file_exists(MODX_MANAGER_PATH."includes/lang/".$manager_language.".inc.php")) {
+            include_once MODX_MANAGER_PATH."includes/lang/".$manager_language.".inc.php";
         }
         return isset($modx_lang_attribute) ? $modx_lang_attribute : null;
     }

assets/lib/class.summary.php

Braces +18 added lines, -6 removed lines

@@ -108,7 +108,9 @@ 
         } else if ($this->_useCut !== true && (mb_strlen($resource, 'UTF-8') > $truncLen)) {
 
             $summary = $this->html_substr($resource, $truncLen, $truncOffset, $truncChars);
-            if ($resource != $summary) $this->_useSubstr = true;
+            if ($resource != $summary) {

+                $this->_useSubstr = true;

+            }

         } else {
             $summary = $resource;
         }
@@ -149,15 +151,23 @@ 
                     }
                     // Slash signifies an ending (like </a> or ... />)
                     // substract 2
-                    if ($current_char == '/' && $tag_counter <> 0) $tag_counter -= 2;
+                    if ($current_char == '/' && $tag_counter <> 0) {

+                        $tag_counter -= 2;

+                    }

                     // On a ">" substract 1
-                    if ($current_char == '>') $tag_counter -= 1;
+                    if ($current_char == '>') {

+                        $tag_counter -= 1;

+                    }

                     // If quotes are encountered, start ignoring the tags
                     // (for directory slashes)
-                    if ($current_char == '"') $quotes_on = TRUE;
+                    if ($current_char == '"') {

+                        $quotes_on = TRUE;

+                    }

                 } else {
                     // IF quotes are encountered again, turn it back off
-                    if ($current_char == '"') $quotes_on = FALSE;
+                    if ($current_char == '"') {

+                        $quotes_on = FALSE;

+                    }

                 }
 
                 // Count only the chars outside html tags
@@ -185,7 +195,9 @@ 
         // Original PHP code from The Art of Web: www.the-art-of-web.com
 
         // return with no change if string is shorter than $limit
-        if (mb_strlen($string, 'UTF-8') < $limit) return $string;
+        if (mb_strlen($string, 'UTF-8') < $limit) {

+            return $string;

+        }

 
         $string = mb_substr($string, 0, $limit, 'UTF-8');
         if (false !== ($breakpoint = mb_strrpos($string, $break, 'UTF-8'))) {

Spacing +4 added lines, -4 removed lines

@@ -12,7 +12,7 @@ 
  * @date 31.07.2013
  * @version 1.0.3
  */
-include_once(MODX_BASE_PATH . 'assets/lib/APIHelpers.class.php');
+include_once(MODX_BASE_PATH.'assets/lib/APIHelpers.class.php');
 class SummaryText
 {
     private $_cfg = array('content' => '', 'summary' => '', 'original' => '', 'break' => '');
@@ -91,7 +91,7 @@ 
     protected  function beforeCut($resource, $splitter = '')
     {
         if ($splitter !== '') {
-            $summary = str_replace('<p>' . $splitter . '</p>', $splitter, $resource); // For TinyMCE or if it isn't wrapped inside paragraph tags
+            $summary = str_replace('<p>'.$splitter.'</p>', $splitter, $resource); // For TinyMCE or if it isn't wrapped inside paragraph tags
             $summary = explode($splitter, $summary, 2);
             $this->_useCut = isset($summary[1]);
             $summary = $summary['0'];
@@ -252,10 +252,10 @@ 
                     $tag = mb_substr($tag, 0, strpos($tag, ' '), 'UTF-8');
                 }
                 if (!mb_stristr($tag, 'br', 'UTF-8') && !mb_stristr($tag, 'img', 'UTF-8') && !empty ($tag)) {
-                    $endTags .= '</' . $tag . '>';
+                    $endTags .= '</'.$tag.'>';
                 }
             }
         }
-        return $text . $endTags;
+        return $text.$endTags;
     }
 }
\ No newline at end of file

Doc Comments +5 added lines, -5 removed lines

@@ -69,7 +69,7 @@ 
     }
 
     /**
-     * @return mixed
+     * @return string
      */
     public function getCut()
     {
@@ -149,8 +149,8 @@ 
     /**
      * @param $resource
      * @param $truncLen
-     * @param $truncOffset
-     * @param $truncChars
+     * @param integer $truncOffset
+     * @param boolean $truncChars
      * @param string $splitter
      * @return array|mixed|string
      */
@@ -242,7 +242,7 @@ 
      */
     /**
      * @param $string
-     * @param $limit
+     * @param integer $limit
      * @param string $break
      * @return string
      */
@@ -265,7 +265,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return mixed
      */
     protected function rTriming($str)

assets/snippets/DocLister/lib/DLphx.class.php

Braces +69 added lines, -27 removed lines

@@ -40,7 +40,9 @@ 
         $this->maxPasses = ($maxpass != '') ? $maxpass : 50;
         $this->swapSnippetCache = array();
         $modx->setPlaceholder("phx", "&_PHX_INTERNAL_&");
-        if (function_exists('mb_internal_encoding')) mb_internal_encoding($modx->config['modx_charset']);
+        if (function_exists('mb_internal_encoding')) {

+            mb_internal_encoding($modx->config['modx_charset']);

+        }

     }
 
     // Plugin event hook for MODx
@@ -60,7 +62,9 @@ 
     {
         global $modx;
         // If we already reached max passes don't get at it again.
-        if ($this->curPass == $this->maxPasses) return $template;
+        if ($this->curPass == $this->maxPasses) {

+            return $template;

+        }

         // Set template pre-process hash
         $st = md5($template);
         // Replace non-call characters in the template: [, ]
@@ -78,7 +82,9 @@ 
         // Set template post-process hash
         $et = md5($template);
         // If template has changed, parse it once more...
-        if ($st != $et) $template = $this->Parse($template);
+        if ($st != $et) {

+            $template = $this->Parse($template);

+        }

         // Write an event log if debugging is enabled and there is something to log
         if ($this->debug && $this->debugLog) {
             $modx->logEvent($this->curPass, 1, $this->createEventLog(), $this->name . ' ' . $this->version);
@@ -196,9 +202,13 @@ 
         $et = md5($template); // Post-process template hash
 
         // Log an event if this was the maximum pass
-        if ($this->curPass == $this->maxPasses) $this->Log("Max passes reached. infinite loop protection so exiting.\n If you need the extra passes set the max passes to the highest count of nested tags in your template.");
+        if ($this->curPass == $this->maxPasses) {

+            $this->Log("Max passes reached. infinite loop protection so exiting.\n If you need the extra passes set the max passes to the highest count of nested tags in your template.");

+        }

         // If this pass is not at maximum passes and the template hash is not the same, get at it again.
-        if (($this->curPass < $this->maxPasses) && ($st != $et)) $template = $this->ParseValues($template);
+        if (($this->curPass < $this->maxPasses) && ($st != $et)) {

+            $template = $this->ParseValues($template);

+        }

 
         return $template;
     }
@@ -217,7 +227,9 @@ 
             for ($i = 0; $i < $count; $i++) {
                 $output = trim($output);
                 $this->Log("  |--- Modifier = '" . $modifier_cmd[$i] . "'");
-                if ($modifier_value[$i] != '') $this->Log("  |--- Options = '" . $modifier_value[$i] . "'");
+                if ($modifier_value[$i] != '') {

+                    $this->Log("  |--- Options = '" . $modifier_value[$i] . "'");

+                }

                 switch ($modifier_cmd[$i]) {
                     #####  Conditional Modifiers
                     case "input":
@@ -260,7 +272,9 @@ 
                     case "ir":
                     case "memberof":
                     case "mo": // Is Member Of  (same as inrole but this one can be stringed as a conditional)
-                        if ($output == "&_PHX_INTERNAL_&") $output = $this->user["id"];
+                        if ($output == "&_PHX_INTERNAL_&") {

+                            $output = $this->user["id"];

+                        }

                         $grps = ($this->strlen($modifier_value[$i]) > 0) ? explode(",", $modifier_value[$i]) : array();
                         $condition[] = intval($this->isMemberOfWebGroupByUserId($output, $grps));
                         break;
@@ -377,11 +391,15 @@ 
                         $output = eval("return " . $filter . ";");
                         break;
                     case "isnotempty":
-                        if (!empty($output)) $output = $modifier_value[$i];
+                        if (!empty($output)) {

+                            $output = $modifier_value[$i];

+                        }

                         break;
                     case "isempty":
                     case "ifempty":
-                        if (empty($output)) $output = $modifier_value[$i];
+                        if (empty($output)) {

+                            $output = $modifier_value[$i];

+                        }

                         break;
                     case "nl2br":
                         $output = nl2br($output);
@@ -391,7 +409,9 @@ 
                         break;
                     case "set":
                         $c = $i + 1;
-                        if ($count > $c && $modifier_cmd[$c] == "value") $output = preg_replace("~([^a-zA-Z0-9])~", "", $modifier_value[$i]);
+                        if ($count > $c && $modifier_cmd[$c] == "value") {

+                            $output = preg_replace("~([^a-zA-Z0-9])~", "", $modifier_value[$i]);

+                        }

                         break;
                     case "value":
                         if ($i > 0 && $modifier_cmd[$i - 1] == "set") {
@@ -403,11 +423,15 @@ 
                         $output = md5($output);
                         break;
                     case "userinfo":
-                        if ($output == "&_PHX_INTERNAL_&") $output = $this->user["id"];
+                        if ($output == "&_PHX_INTERNAL_&") {

+                            $output = $this->user["id"];

+                        }

                         $output = $this->ModUser($output, $modifier_value[$i]);
                         break;
                     case "inrole": // deprecated
-                        if ($output == "&_PHX_INTERNAL_&") $output = $this->user["id"];
+                        if ($output == "&_PHX_INTERNAL_&") {

+                            $output = $this->user["id"];

+                        }

                         $grps = ($this->strlen($modifier_value[$i]) > 0) ? explode(",", $modifier_value[$i]) : array();
                         $output = intval($this->isMemberOfWebGroupByUserId($output, $grps));
                         break;
@@ -451,7 +475,9 @@ 
                         ob_end_clean();
                         break;
                 }
-                if (count($condition)) $this->Log("  |--- Condition = '" . $condition[count($condition) - 1] . "'");
+                if (count($condition)) {

+                    $this->Log("  |--- Condition = '" . $condition[count($condition) - 1] . "'");

+                }

                 $this->Log("  |--- Output = '" . $output . "'");
             }
         }
@@ -531,7 +557,9 @@ 
         global $modx;
 
         // if $groupNames is not an array return false
-        if (!is_array($groupNames)) return false;
+        if (!is_array($groupNames)) {

+            return false;

+        }

 
         // if the user id is a negative number make it positive
         if (intval($userid) < 0) {
@@ -548,8 +576,9 @@ 
             $grpNames = $this->cache["mo"][$userid];
         }
         // Check if a supplied group matches a webgroup from the array we just created
-        foreach ($groupNames as $k => $v)
-            if (in_array(trim($v), $grpNames)) return true;
+        foreach ($groupNames as $k => $v) {

+                    if (in_array(trim($v), $grpNames)) return true;

+        }

 
         // If we get here the above logic did not find a match, so return false
         return false;
@@ -572,52 +601,65 @@ 
     // Sets a placeholder variable which can only be access by PHx
     function setPHxVariable($name, $value)
     {
-        if ($name != "phx") $this->placeholders[$name] = $value;
+        if ($name != "phx") {

+            $this->placeholders[$name] = $value;

+        }

     }
 
     //mbstring
     function substr($str, $s, $l = null)
     {
-        if (function_exists('mb_substr')) return mb_substr($str, $s, $l);
+        if (function_exists('mb_substr')) {

+            return mb_substr($str, $s, $l);

+        }

         return substr($str, $s, $l);
     }
 
     function strlen($str)
     {
-        if (function_exists('mb_strlen')) return mb_strlen($str);
+        if (function_exists('mb_strlen')) {

+            return mb_strlen($str);

+        }

         return strlen($str);
     }
 
     function strtolower($str)
     {
-        if (function_exists('mb_strtolower')) return mb_strtolower($str);
+        if (function_exists('mb_strtolower')) {

+            return mb_strtolower($str);

+        }

         return strtolower($str);
     }
 
     function strtoupper($str)
     {
-        if (function_exists('mb_strtoupper')) return mb_strtoupper($str);
+        if (function_exists('mb_strtoupper')) {

+            return mb_strtoupper($str);

+        }

         return strtoupper($str);
     }
 
     function ucfirst($str)
     {
-        if (function_exists('mb_strtoupper') && function_exists('mb_substr') && function_exists('mb_strlen'))
-            return mb_strtoupper(mb_substr($str, 0, 1)) . mb_substr($str, 1, mb_strlen($str));
+        if (function_exists('mb_strtoupper') && function_exists('mb_substr') && function_exists('mb_strlen')) {

+                    return mb_strtoupper(mb_substr($str, 0, 1)) . mb_substr($str, 1, mb_strlen($str));

+        }

         return ucfirst($str);
     }
 
     function lcfirst($str)
     {
-        if (function_exists('mb_strtolower') && function_exists('mb_substr') && function_exists('mb_strlen'))
-            return mb_strtolower(mb_substr($str, 0, 1)) . mb_substr($str, 1, mb_strlen($str));
+        if (function_exists('mb_strtolower') && function_exists('mb_substr') && function_exists('mb_strlen')) {

+                    return mb_strtolower(mb_substr($str, 0, 1)) . mb_substr($str, 1, mb_strlen($str));

+        }

         return lcfirst($str);
     }
 
     function ucwords($str)
     {
-        if (function_exists('mb_convert_case'))
-            return mb_convert_case($str, MB_CASE_TITLE);
+        if (function_exists('mb_convert_case')) {

+                    return mb_convert_case($str, MB_CASE_TITLE);

+        }

         return ucwords($str);
     }
 

Doc Comments +17 added lines, -17 removed lines

@@ -120,7 +120,7 @@ 
     // Parser: Preparation, cleaning and checkup
     /**
      * @param string $template
-     * @return mixed|string
+     * @return string
      */
     public function Parse($template = '')
     {
@@ -268,7 +268,7 @@ 
     // Parser: modifier detection and eXtended processing if needed
     /**
      * @param $input
-     * @param $modifiers
+     * @param string $modifiers
      * @return mixed|null|string
      */
     public function Filter($input, $modifiers)
@@ -600,8 +600,8 @@ 
     // positive userid = manager, negative integer = webuser
     /**
      * @param $userid
-     * @param $field
-     * @return mixed
+     * @param string $field
+     * @return string
      */
     public function ModUser($userid, $field)
     {
@@ -656,7 +656,7 @@ 
 
     // Returns the value of a PHx/MODx placeholder.
     /**
-     * @param $name
+     * @param string $name
      * @return mixed
      */
     public function getPHxVariable($name)
@@ -684,9 +684,9 @@ 
 
     //mbstring
     /**
-     * @param $str
-     * @param $s
-     * @param null $l
+     * @param string $str
+     * @param integer $s
+     * @param integer $l
      * @return string
      */
     public function substr($str, $s, $l = null)
@@ -696,7 +696,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return int
      */
     public function strlen($str)
@@ -706,7 +706,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function strtolower($str)
@@ -716,7 +716,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function strtoupper($str)
@@ -726,7 +726,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function ucfirst($str)
@@ -737,7 +737,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function lcfirst($str)
@@ -748,7 +748,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function ucwords($str)
@@ -759,7 +759,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function strrev($str)
@@ -769,7 +769,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return string
      */
     public function str_shuffle($str)
@@ -780,7 +780,7 @@ 
     }
 
     /**
-     * @param $str
+     * @param string $str
      * @return int
      */
     public function str_word_count($str)

Spacing +37 added lines, -37 removed lines

@@ -10,7 +10,7 @@ 
 #	Date: March 22, 2013
 #
 ####*/
-include_once(MODX_BASE_PATH . 'assets/lib/APIHelpers.class.php');
+include_once(MODX_BASE_PATH.'assets/lib/APIHelpers.class.php');
 
 /**
  * Class DLphx
@@ -137,7 +137,7 @@ 
         preg_match_all('~(?:=`[^`@]*?)(\[\+([^:\+\[\]]+)([^\[\]]*?)\+\])~s', $template, $matches);
         if ($matches[0]) {
             $template = str_replace($matches[1], '', $template);
-            $this->Log("Cleaning unsolved tags: \n" . implode("\n", $matches[2]));
+            $this->Log("Cleaning unsolved tags: \n".implode("\n", $matches[2]));
         }
         // Restore non-call characters in the template: [, ]
         $template = str_replace($this->safetags[1], $this->safetags[2], $template);
@@ -147,7 +147,7 @@ 
         if ($st != $et) $template = $this->Parse($template);
         // Write an event log if debugging is enabled and there is something to log
         if ($this->debug && $this->debugLog) {
-            $modx->logEvent($this->curPass, 1, $this->createEventLog(), $this->name . ' ' . $this->version);
+            $modx->logEvent($this->curPass, 1, $this->createEventLog(), $this->name.' '.$this->version);
             $this->debugLog = false;
         }
         // Return the processed template
@@ -178,8 +178,8 @@ 
             for ($i = 0; $i < $count; $i++) {
                 $var_search[] = $matches[0][$i];
                 $input = $matches[1][$i];
-                $this->Log('MODX Chunk: ' . $input);
-                $input = $modx->mergeChunkContent('{{' . $input . '}}');
+                $this->Log('MODX Chunk: '.$input);
+                $input = $modx->mergeChunkContent('{{'.$input.'}}');
                 $var_replace[] = $this->Filter($input, $matches[2][$i]);
             }
             $template = str_replace($var_search, $var_replace, $template);
@@ -195,10 +195,10 @@ 
             // for each detected snippet
             for ($i = 0; $i < $count; $i++) {
                 $snippet = $matches[2][$i]; // snippet call
-                $this->Log("MODx Snippet -> " . $snippet);
+                $this->Log("MODx Snippet -> ".$snippet);
 
                 // Let MODx evaluate snippet
-                $replace = $modx->evalSnippets("[[" . $snippet . "]]");
+                $replace = $modx->evalSnippets("[[".$snippet."]]");
                 $this->LogSnippet($replace);
 
                 // Replace values
@@ -227,19 +227,19 @@ 
                 switch ($matches[1][$i]) {
                     // Document / Template Variable eXtended
                     case "*":
-                        $this->Log("MODx TV/DV: " . $input);
-                        $input = $modx->mergeDocumentContent("[*" . $input . "*]");
+                        $this->Log("MODx TV/DV: ".$input);
+                        $input = $modx->mergeDocumentContent("[*".$input."*]");
                         $replace = $this->Filter($input, $modifiers);
                         break;
                     // MODx Setting eXtended
                     case "(":
-                        $this->Log("MODx Setting variable: " . $input);
-                        $input = $modx->mergeSettingsContent("[(" . $input . ")]");
+                        $this->Log("MODx Setting variable: ".$input);
+                        $input = $modx->mergeSettingsContent("[(".$input.")]");
                         $replace = $this->Filter($input, $modifiers);
                         break;
                     // MODx Placeholder eXtended
                     default:
-                        $this->Log("MODx / PHx placeholder variable: " . $input);
+                        $this->Log("MODx / PHx placeholder variable: ".$input);
                         // Check if placeholder is set
                         if (!array_key_exists($input, $this->placeholders) && !array_key_exists($input, $modx->placeholders)) {
                             // not set so try again later.
@@ -275,7 +275,7 @@ 
     {
         global $modx;
         $output = $input;
-        $this->Log("  |--- Input = '" . $output . "'");
+        $this->Log("  |--- Input = '".$output."'");
         if (preg_match_all('~:([^:=]+)(?:=`(.*?)`(?=:[^:=]+|$))?~s', $modifiers, $matches)) {
             $modifier_cmd = $matches[1]; // modifier command
             $modifier_value = $matches[2]; // modifier value
@@ -283,8 +283,8 @@ 
             $condition = array();
             for ($i = 0; $i < $count; $i++) {
                 $output = trim($output);
-                $this->Log("  |--- Modifier = '" . $modifier_cmd[$i] . "'");
-                if ($modifier_value[$i] != '') $this->Log("  |--- Options = '" . $modifier_value[$i] . "'");
+                $this->Log("  |--- Modifier = '".$modifier_cmd[$i]."'");
+                if ($modifier_value[$i] != '') $this->Log("  |--- Options = '".$modifier_value[$i]."'");
                 switch ($modifier_cmd[$i]) {
                     #####  Conditional Modifiers
                     case "input":
@@ -339,14 +339,14 @@ 
                         break;
                     case "show":
                         $conditional = implode(' ', $condition);
-                        $isvalid = intval(eval("return (" . $conditional . ");"));
+                        $isvalid = intval(eval("return (".$conditional.");"));
                         if (!$isvalid) {
                             $output = NULL;
                         }
                         break;
                     case "then":
                         $conditional = implode(' ', $condition);
-                        $isvalid = intval(eval("return (" . $conditional . ");"));
+                        $isvalid = intval(eval("return (".$conditional.");"));
                         if ($isvalid) {
                             $output = $modifier_value[$i];
                         } else {
@@ -355,7 +355,7 @@ 
                         break;
                     case "else":
                         $conditional = implode(' ', $condition);
-                        $isvalid = intval(eval("return (" . $conditional . ");"));
+                        $isvalid = intval(eval("return (".$conditional.");"));
                         if (!$isvalid) {
                             $output = $modifier_value[$i];
                         }
@@ -442,7 +442,7 @@ 
                     case "math":
                         $filter = preg_replace("~([a-zA-Z\n\r\t\s])~", "", $modifier_value[$i]);
                         $filter = str_replace("?", $output, $filter);
-                        $output = eval("return " . $filter . ";");
+                        $output = eval("return ".$filter.";");
                         break;
                     case "isnotempty":
                         if (!empty($output)) $output = $modifier_value[$i];
@@ -463,7 +463,7 @@ 
                         break;
                     case "value":
                         if ($i > 0 && $modifier_cmd[$i - 1] == "set") {
-                            $modx->SetPlaceholder("phx." . $output, $modifier_value[$i]);
+                            $modx->SetPlaceholder("phx.".$output, $modifier_value[$i]);
                         }
                         $output = NULL;
                         break;
@@ -484,25 +484,25 @@ 
                     default:
                         $snippet = '';
                         // modified by Anton Kuzmin (23.06.2010) //
-                        $snippetName = 'phx:' . $modifier_cmd[$i];
+                        $snippetName = 'phx:'.$modifier_cmd[$i];
                         if (isset($modx->snippetCache[$snippetName])) {
                             $snippet = $modx->snippetCache[$snippetName];
                         } else { // not in cache so let's check the db
-                            $sql = "SELECT snippet FROM " . $modx->getFullTableName("site_snippets") . " WHERE " . $modx->getFullTableName("site_snippets") . ".name='" . $modx->db->escape($snippetName) . "';";
+                            $sql = "SELECT snippet FROM ".$modx->getFullTableName("site_snippets")." WHERE ".$modx->getFullTableName("site_snippets").".name='".$modx->db->escape($snippetName)."';";
                             $result = $modx->dbQuery($sql);
                             if ($modx->recordCount($result) == 1) {
                                 $row = $modx->fetchRow($result);
                                 $snippet = $modx->snippetCache[$row['name']] = $row['snippet'];
                                 $this->Log("  |--- DB -> Custom Modifier");
                             } else if ($modx->recordCount($result) == 0) { // If snippet not found, look in the modifiers folder
-                                $filename = $modx->config['rb_base_dir'] . 'plugins/phx/modifiers/' . $modifier_cmd[$i] . '.phx.php';
+                                $filename = $modx->config['rb_base_dir'].'plugins/phx/modifiers/'.$modifier_cmd[$i].'.phx.php';
                                 if (@file_exists($filename)) {
                                     $file_contents = @file_get_contents($filename);
-                                    $file_contents = str_replace('<' . '?php', '', $file_contents);
-                                    $file_contents = str_replace('?' . '>', '', $file_contents);
+                                    $file_contents = str_replace('<'.'?php', '', $file_contents);
+                                    $file_contents = str_replace('?'.'>', '', $file_contents);
                                     $file_contents = str_replace('<?', '', $file_contents);
                                     $snippet = $modx->snippetCache[$snippetName] = $file_contents;
-                                    $modx->snippetCache[$snippetName . 'Props'] = '';
+                                    $modx->snippetCache[$snippetName.'Props'] = '';
                                     $this->Log("  |--- File ($filename) -> Custom Modifier");
                                 } else {
                                     $this->Log("  |--- PHX Error:  {$modifier_cmd[$i]} could not be found");
@@ -517,15 +517,15 @@ 
                             $options = $modifier_value[$i];
                             $custom = eval($cm);
                             $msg = ob_get_contents();
-                            $output = $msg . $custom;
+                            $output = $msg.$custom;
                             ob_end_clean();
                         } else {
                             $output = '';
                         }
                         break;
                 }
-                if (count($condition)) $this->Log("  |--- Condition = '" . $condition[count($condition) - 1] . "'");
-                $this->Log("  |--- Output = '" . $output . "'");
+                if (count($condition)) $this->Log("  |--- Condition = '".$condition[count($condition) - 1]."'");
+                $this->Log("  |--- Output = '".$output."'");
             }
         }
         return $output;
@@ -540,7 +540,7 @@ 
         if (!empty($this->console)) {
             $console = implode("\n", $this->console);
             $this->console = array();
-            return '<pre style="overflow: auto;">' . $console . '</pre>';
+            return '<pre style="overflow: auto;">'.$console.'</pre>';
         }
     }
 
@@ -564,7 +564,7 @@ 
     {
         if ($this->debug) {
             $this->debugLog = true;
-            $this->console[] = (count($this->console) + 1 - $this->curPass) . " [" . strftime("%H:%M:%S", time()) . "] " . $this->LogClean($string);
+            $this->console[] = (count($this->console) + 1 - $this->curPass)." [".strftime("%H:%M:%S", time())."] ".$this->LogClean($string);
         }
     }
 
@@ -576,14 +576,14 @@ 
     {
         if ($this->debug) {
             $this->debugLog = true;
-            $this->console[] = (count($this->console) + 1 - $this->curPass) . " [" . strftime("%H:%M:%S", time()) . "] " . "  |--- Returns: <div style='margin: 10px;'>" . $this->LogClean($string) . "</div>";
+            $this->console[] = (count($this->console) + 1 - $this->curPass)." [".strftime("%H:%M:%S", time())."] "."  |--- Returns: <div style='margin: 10px;'>".$this->LogClean($string)."</div>";
         }
     }
 
     // Log pass
     public function LogPass()
     {
-        $this->console[] = "<div style='margin: 2px;margin-top: 5px;border-bottom: 1px solid black;'>Pass " . $this->curPass . "</div>";
+        $this->console[] = "<div style='margin: 2px;margin-top: 5px;border-bottom: 1px solid black;'>Pass ".$this->curPass."</div>";
     }
 
     // Log pass
@@ -592,7 +592,7 @@ 
      */
     public function LogSource($string)
     {
-        $this->console[] = "<div style='margin: 2px;margin-top: 5px;border-bottom: 1px solid black;'>Source:</div>" . $this->LogClean($string);
+        $this->console[] = "<div style='margin: 2px;margin-top: 5px;border-bottom: 1px solid black;'>Source:</div>".$this->LogClean($string);
     }
 
 
@@ -641,7 +641,7 @@ 
         if (!array_key_exists($userid, $this->cache["mo"])) {
             $tbl = $modx->getFullTableName("webgroup_names");
             $tbl2 = $modx->getFullTableName("web_groups");
-            $sql = "SELECT wgn.name FROM $tbl wgn INNER JOIN $tbl2 wg ON wg.webgroup=wgn.id AND wg.webuser='" . $userid . "'";
+            $sql = "SELECT wgn.name FROM $tbl wgn INNER JOIN $tbl2 wg ON wg.webgroup=wgn.id AND wg.webuser='".$userid."'";
             $this->cache["mo"][$userid] = $grpNames = $modx->db->getColumn("name", $sql);
         } else {
             $grpNames = $this->cache["mo"][$userid];
@@ -732,7 +732,7 @@ 
     public function ucfirst($str)
     {
         if (function_exists('mb_strtoupper') && function_exists('mb_substr') && function_exists('mb_strlen'))
-            return mb_strtoupper(mb_substr($str, 0, 1)) . mb_substr($str, 1, mb_strlen($str));
+            return mb_strtoupper(mb_substr($str, 0, 1)).mb_substr($str, 1, mb_strlen($str));
         return ucfirst($str);
     }
 
@@ -743,7 +743,7 @@ 
     public function lcfirst($str)
     {
         if (function_exists('mb_strtolower') && function_exists('mb_substr') && function_exists('mb_strlen'))
-            return mb_strtolower(mb_substr($str, 0, 1)) . mb_substr($str, 1, mb_strlen($str));
+            return mb_strtolower(mb_substr($str, 0, 1)).mb_substr($str, 1, mb_strlen($str));
         return lcfirst($str);
     }
 

assets/snippets/DocLister/core/extender/tv.extender.inc

Braces +3 added lines, -1 removed lines

@@ -64,7 +64,9 @@
     {

         $out = array();

         $tv = $this->getTVid($tvlist);

-        if (empty($tv)) return $out;

+        if (empty($tv)) {

+            return $out;

+        }

         $tvId = array_keys($tv);

         $tbl_site_tmplvar_contentvalues = $this->DocLister->getTable($this->tvValuesTable());

         $sanitarInIDs = $this->DocLister->sanitarIn($IDs);

Spacing +11 added lines, -11 removed lines

@@ -109,7 +109,7 @@ 
         $tvPrefix = $this->_TVprefix;

 

         foreach ($rows as $item) {

-            $out[$item['contentid']][$tvPrefix . $tv[$item['tmplvarid']]] = $item['value'];

+            $out[$item['contentid']][$tvPrefix.$tv[$item['tmplvarid']]] = $item['value'];

         }

         $tmp = explode(",", $this->DocLister->sanitarIn($IDs, ',', false));

         foreach ($tmp as $item) {

@@ -159,7 +159,7 @@ 
                 $out = array_unique($tmp);

                 $tmp = array();

                 foreach ($out as $item) {

-                    $tmp[] = $this->_TVprefix . $item;

+                    $tmp[] = $this->_TVprefix.$item;

                 }

             }

         } else {

@@ -177,8 +177,8 @@ 
      */

     public function renderTV($iddoc, $tvname, $tvval, $param)

     {

-        include_once MODX_MANAGER_PATH . "includes/tmplvars.format.inc.php";

-        include_once MODX_MANAGER_PATH . "includes/tmplvars.commands.inc.php";

+        include_once MODX_MANAGER_PATH."includes/tmplvars.format.inc.php";

+        include_once MODX_MANAGER_PATH."includes/tmplvars.commands.inc.php";

         return getTVDisplayFormat($tvname, $tvval, $param['display'], $param['display_params'], $param['type'], $iddoc, '');

     }

 

@@ -214,7 +214,7 @@ 
         $rows = $this->modx->db->makeArray($rs);

         $out = array();

         foreach ($rows as $item) {

-            $out[$this->_TVprefix . $item['name']] = $item;

+            $out[$this->_TVprefix.$item['name']] = $item;

         }

         return $out;

     }

@@ -251,17 +251,17 @@ 
                 $item = explode(" ", trim($item), 2);

                 if (isset($TVnames[$item[0]])) {

                     $exists = $this->DocLister->checkTableAlias($item[0], $this->tvValuesTable());

-                    $prefix = 'dltv_' . $item[0] . '_' . ($i + 1);

+                    $prefix = 'dltv_'.$item[0].'_'.($i + 1);

                     $prefix = $this->storeTable ? $this->DocLister->TableAlias($item[0], $this->tvValuesTable(), $prefix) : $prefix;

                     if (!$exists) {

-                        $table .= " LEFT JOIN " . $this->DocLister->getTable($this->tvValuesTable(), $prefix) . "

-                        on `" . $prefix . "`.`contentid`=`c`.`id` AND `" . $prefix . "`.`tmplvarid`=" . $TVnames[$item[0]];

+                        $table .= " LEFT JOIN ".$this->DocLister->getTable($this->tvValuesTable(), $prefix)."

+                        on `" . $prefix."`.`contentid`=`c`.`id` AND `".$prefix."`.`tmplvarid`=".$TVnames[$item[0]];

                     }

                     if (in_array($item[0], $withDefault)) {

                         $exists = $this->DocLister->checkTableAlias($item[0], "site_tmplvars");

-                        $dPrefix = $this->storeTable ? $this->DocLister->TableAlias($item[0], "site_tmplvars", 'd_' . $prefix) : 'd_' . $prefix;

+                        $dPrefix = $this->storeTable ? $this->DocLister->TableAlias($item[0], "site_tmplvars", 'd_'.$prefix) : 'd_'.$prefix;

                         if (!$exists) {

-                            $table .= " LEFT JOIN " . $this->DocLister->getTable("site_tmplvars", $dPrefix) . " on `" . $dPrefix . "`.`id` = " . $TVnames[$item[0]];

+                            $table .= " LEFT JOIN ".$this->DocLister->getTable("site_tmplvars", $dPrefix)." on `".$dPrefix."`.`id` = ".$TVnames[$item[0]];

                         }

                         $field = "IFNULL(`{$prefix}`.`value`, `{$dPrefix}`.`default_text`)";

                     } else {

@@ -271,7 +271,7 @@ 
                 }

                 $item = implode(" ", $item);

             }

-            $sort = "ORDER BY " . implode(",", $matches);

+            $sort = "ORDER BY ".implode(",", $matches);

         }

         return array($table, $sort);

     }

assets/snippets/DocLister/core/extender/user.extender.inc

Braces +3 added lines, -1 removed lines

@@ -82,7 +82,9 @@
 

     private function getInfo($user = array())

     {

-        if (!isset($this->table['users'], $this->table['attributes'])) $this->table = $this->getTable();

+        if (!isset($this->table['users'], $this->table['attributes'])) {

+            $this->table = $this->getTable();

+        }

 

         $out = array();

         if (count($user) > 0) {

assets/snippets/DocLister/core/extender/jotcount.extender.inc

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
         $comments = array();

         if (count($docs)) {

             $from = $this->DocLister->getTable('jot_content');

-            $rs = $this->DocLister->dbQuery("SELECT uparent, COUNT(*) as total FROM {$from} WHERE uparent IN (" . implode(',', $docs) . ") AND published=1 AND deleted=0 GROUP BY uparent");

+            $rs = $this->DocLister->dbQuery("SELECT uparent, COUNT(*) as total FROM {$from} WHERE uparent IN (".implode(',', $docs).") AND published=1 AND deleted=0 GROUP BY uparent");

             $counts = $this->modx->db->makeArray($rs);

             foreach ($counts as $v) {

                 $comments[$v['uparent']] = $v['total'];

assets/snippets/DocLister/core/extender/request.extender.inc

Spacing +2 added lines, -2 removed lines

@@ -147,14 +147,14 @@
         $link = '';

         if (is_array($item) && count($item) > 0) {

             foreach ($item as $nameItem => $valItem) {

-                $link[] = $name . "[" . (intval($nameItem) ? "" : $nameItem) . "]=" . urlencode($valItem);

+                $link[] = $name."[".(intval($nameItem) ? "" : $nameItem)."]=".urlencode($valItem);

             }

             $link = implode("&", $link);

         } else {

             if ($name == '') {

                 $link = urlencode($item);

             } else {

-                $link = $name . "=" . urlencode($item);

+                $link = $name."=".urlencode($item);

             }

         }

         return $link;

assets/snippets/DocLister/core/extender/summary.extender.inc

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     protected function run()

     {

         if (!empty($this->_cfg['action']) && !empty($this->_cfg['content'])) {

-            include_once(MODX_BASE_PATH . 'assets/lib/class.summary.php');

+            include_once(MODX_BASE_PATH.'assets/lib/class.summary.php');

             $summary = new SummaryText($this->_cfg['content'], $this->_cfg['action'], $this->getCFGDef('breakSummary', null));

             $summary->setCut($this->getCFGDef('cutSummary', '<cut/>'));

             $this->_cfg['content'] = $summary->run($this->getCFGDef('dotSummary', 0));

assets/snippets/DocLister/core/extender/paginate.extender.inc

Spacing +3 added lines, -3 removed lines

@@ -11,7 +11,7 @@ 
  * @author Agel_Nash <Agel_Nash@xaker.ru>

  * @see http://mis-algoritmos.com/2007/05/27/digg-style-pagination-class

  */

-include_once(MODX_BASE_PATH . "assets/snippets/DocLister/lib/DLpaginate.class.php");

+include_once(MODX_BASE_PATH."assets/snippets/DocLister/lib/DLpaginate.class.php");

 

 class paginate_DL_Extender extends extDocLister

 {

@@ -171,7 +171,7 @@ 
             $url = explode("?", $url, 2);

             $url = $url[0];

             if (!empty($q)) {

-                $url .= "?" . $q;

+                $url .= "?".$q;

             }

         }

         return $url;

@@ -186,6 +186,6 @@ 
     public function getRequestName($name)

     {

         $id = $this->DocLister->getCFGDef('id', '');

-        return (($id != '') ? ($id . "_") : "") . $name;

+        return (($id != '') ? ($id."_") : "").$name;

     }

 }

Doc Comments +1 added lines, -1 removed lines

@@ -180,7 +180,7 @@
     /**

      * Имя REQUEST переменной в пагинаторе

      *

-     * @param $name основное имя REQUEST переменной в пагинаторе

+     * @param string $name основное имя REQUEST переменной в пагинаторе

      * @return string

      */

     public function getRequestName($name)