AgelxNash / DocLister

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

Spacing +1 added lines, -1 removed lines

@@ -1,5 +1,5 @@
 <?php
-include_once(MODX_BASE_PATH . "assets/lib/Helpers/Collection.php");
+include_once(MODX_BASE_PATH."assets/lib/Helpers/Collection.php");
 
 /**
  * Class DLCollection

assets/lib/Helpers/PHPThumb.php

Upper-Lower-Casing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
      */
     private function setOptions($options)
     {
-        $options = strtr($options, Array("," => "&", "_" => "=", '{' => '[', '}' => ']'));
+        $options = strtr($options, array("," => "&", "_" => "=", '{' => '[', '}' => ']'));
         parse_str($options, $params);
         if(!is_array($params)) $params = array();
 

Spacing +8 added lines, -8 removed lines

@@ -1,7 +1,7 @@ 
 <?php namespace Helpers;
 
-include_once(MODX_BASE_PATH . 'assets/snippets/phpthumb/phpthumb.class.php');
-require_once(MODX_BASE_PATH . 'assets/lib/Helpers/FS.php');
+include_once(MODX_BASE_PATH.'assets/snippets/phpthumb/phpthumb.class.php');
+require_once(MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
 
 /**
  * Class PHPThumb
@@ -34,7 +34,7 @@ 
         $this->thumb->sourceFilename = $inputFile;
         $ext = explode('.', $inputFile);
         $ext = str_replace('jpeg', 'jpg', strtolower(array_pop($ext)));
-        $options = 'f=' . $ext . '&' . $options;
+        $options = 'f='.$ext.'&'.$options;
         $this->setOptions($options);
         if ($this->thumb->GenerateThumbnail() && $this->thumb->RenderToFile($outputFile)) {
             return true;
@@ -55,12 +55,12 @@ 
             case 'jpg':
                 $ext = $this->fs->takeFileExt($file);
                 if ($ext == 'jpeg' || $ext == 'jpg') {
-                    $cmd = '/usr/bin/jpegtran -optimize -progressive -copy none -outfile ' . escapeshellarg($file . '_') . ' ' . escapeshellarg($file);
+                    $cmd = '/usr/bin/jpegtran -optimize -progressive -copy none -outfile '.escapeshellarg($file.'_').' '.escapeshellarg($file);
                     exec($cmd, $result, $return_var);
-                    if ($this->fs->fileSize($file) > $this->fs->fileSize($file . '_')) {
-                        $this->fs->moveFile($file . '_', $file);
+                    if ($this->fs->fileSize($file) > $this->fs->fileSize($file.'_')) {
+                        $this->fs->moveFile($file.'_', $file);
                     } else {
-                        $this->fs->unlink($file . '_');
+                        $this->fs->unlink($file.'_');
                     }
                 }
                 break;
@@ -76,7 +76,7 @@ 
     {
         $options = strtr($options, Array("," => "&", "_" => "=", '{' => '[', '}' => ']'));
         parse_str($options, $params);
-        if(!is_array($params)) $params = array();
+        if (!is_array($params)) $params = array();
 
         foreach ($params as $key => $value) {
             $this->thumb->setParameter($key, $value);

Braces +3 added lines, -1 removed lines

@@ -76,7 +76,9 @@
     {
         $options = strtr($options, Array("," => "&", "_" => "=", '{' => '[', '}' => ']'));
         parse_str($options, $params);
-        if(!is_array($params)) $params = array();
+        if(!is_array($params)) {
+            $params = array();
+        }
 
         foreach ($params as $key => $value) {
             $this->thumb->setParameter($key, $value);

assets/snippets/DocLister/core/controller/site_content_menu.php

Spacing +9 added lines, -9 removed lines

@@ -27,7 +27,7 @@ 
      */
     public function setIDs($IDs)
     {
-        $this->debug->debug('set ID list ' . $this->debug->dumpData($IDs), 'setIDs', 2);
+        $this->debug->debug('set ID list '.$this->debug->dumpData($IDs), 'setIDs', 2);
         $IDs = $this->cleanIDs($IDs);
         $this->debug->debugEnd("setIDs");
 
@@ -280,11 +280,11 @@ 
                 }
 
                 if (isset($data['wrap'])) {
-                    if(is_array($data['wrap'])) $data['wrap'] = $this->parseRow($data['wrap']);
+                    if (is_array($data['wrap'])) $data['wrap'] = $this->parseRow($data['wrap']);
                     $data['wrap'] = $this->parseOuter($data);
                 }
                 $hideSubMenus = $this->getCFGDef('hideSubMenus', 0);
-                $hideSubMenus = !$hideSubMenus || ($hideSubMenus && in_array((int)$data['parent'],
+                $hideSubMenus = !$hideSubMenus || ($hideSubMenus && in_array((int) $data['parent'],
                             $this->activeBranch));
                 if ($hideSubMenus) {
                     $docs[$currentLevel - 1][$data['parent']]['wrap'][] = $data;
@@ -433,8 +433,8 @@ 
      */
     public function getHereId()
     {
-        if (!$hereId = (int)$this->getCFGDef('hereId')) {
-            $hereId = isset($this->modx->documentIdentifier) ? (int)$this->modx->documentIdentifier : 0;
+        if (!$hereId = (int) $this->getCFGDef('hereId')) {
+            $hereId = isset($this->modx->documentIdentifier) ? (int) $this->modx->documentIdentifier : 0;
         }
 
         return $hereId;
@@ -460,7 +460,7 @@ 
             'evenClass'    => ''
         );
         if (isset($data['state'])) {
-            $classes['stateClass'] = $this->getCFGDef($data['state'] . 'Class', $data['state']);
+            $classes['stateClass'] = $this->getCFGDef($data['state'].'Class', $data['state']);
         }
         if (isset($data['here'])) {
             $classes['hereClass'] = $this->getCFGDef('hereClass', 'current');
@@ -481,7 +481,7 @@ 
             $classes['lastClass'] = $this->getCFGDef('lastClass', 'last');
         }
         if ($levelClass = $this->getCFGDef('levelClass', 'level')) {
-            $classes['levelClass'] = $levelClass . $data['level'];
+            $classes['levelClass'] = $levelClass.$data['level'];
         }
         if (isset($data['type']) && $data['type'] == 'reference') {
             $classes['webLinkClass'] = $this->getCFGDef('webLinkClass');
@@ -570,7 +570,7 @@ 
                 }
 
                 $hideSubMenus = $this->getCFGDef('hideSubMenus', 0);
-                $hideSubMenus = !$hideSubMenus || ($hideSubMenus && in_array((int)$data['parent'],
+                $hideSubMenus = !$hideSubMenus || ($hideSubMenus && in_array((int) $data['parent'],
                             $this->activeBranch));
                 if ($hideSubMenus) {
                     $docs[$currentLevel - 1][$data['parent']]['children'][] = $data;
@@ -605,6 +605,6 @@ 
      */
     public function getCFGDef($name, $def = null)
     {
-        return parent::getCFGDef($name . $this->currentLevel, parent::getCFGDef($name, $def));
+        return parent::getCFGDef($name.$this->currentLevel, parent::getCFGDef($name, $def));
     }
 }

Braces +9 added lines, -3 removed lines

@@ -280,7 +280,9 @@ 
                 }
 
                 if (isset($data['wrap'])) {
-                    if(is_array($data['wrap'])) $data['wrap'] = $this->parseRow($data['wrap']);
+                    if(is_array($data['wrap'])) {
+                        $data['wrap'] = $this->parseRow($data['wrap']);
+                    }
                     $data['wrap'] = $this->parseOuter($data);
                 }
                 $hideSubMenus = $this->getCFGDef('hideSubMenus', 0);
@@ -306,7 +308,9 @@ 
                 }
             }
         }
-        if ($joinMenus) $out = $this->parseOuter(array('wrap' => $out));
+        if ($joinMenus) {
+            $out = $this->parseOuter(array('wrap' => $out));
+        }
 
         return $out;
     }
@@ -542,7 +546,9 @@ 
         $currentLevel = count($this->levels);
         $docs = $this->levels;
 
-        if (empty($docs)) return '[]';
+        if (empty($docs)) {
+            return '[]';
+        }
 
         /** @var prepare_DL_Extender_ $extPrepare */
         $extPrepare = $this->getExtender('prepare');

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

Spacing +39 added lines, -39 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
@@ -51,7 +51,7 @@ 
         $this->user["usrid"] = isset($_SESSION['webInternalKey']) ? intval($_SESSION['webInternalKey']) : 0;
         $this->user["id"] = ($this->user["usrid"] > 0) ? (-$this->user["usrid"]) : $this->user["mgrid"];
 
-        $this->debug = (bool)$debug;
+        $this->debug = (bool) $debug;
 
         $this->maxPasses = ($maxpass != '') ? $maxpass : 50;
 
@@ -95,7 +95,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);
@@ -107,7 +107,7 @@ 
         }
         // 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;
         }
 
@@ -138,8 +138,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);
@@ -155,10 +155,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
@@ -187,19 +187,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)
@@ -241,7 +241,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
@@ -249,9 +249,9 @@ 
             $condition = array();
             for ($i = 0; $i < $count; $i++) {
                 $output = trim($output);
-                $this->Log("  |--- Modifier = '" . $modifier_cmd[$i] . "'");
+                $this->Log("  |--- Modifier = '".$modifier_cmd[$i]."'");
                 if ($modifier_value[$i] != '') {
-                    $this->Log("  |--- Options = '" . $modifier_value[$i] . "'");
+                    $this->Log("  |--- Options = '".$modifier_value[$i]."'");
                 }
                 switch ($modifier_cmd[$i]) {
                     #####  Conditional Modifiers
@@ -392,7 +392,7 @@ 
                         break;
                     case "wordwrap": // default: 70
                         $wrapat = intval($modifier_value[$i]) ? intval($modifier_value[$i]) : 70;
-                        $output = preg_replace_callback("@(\b\w+\b)@",function($m) use($wrapat) {return wordwrap($m[1],$wrapat,' ',1);},$output);
+                        $output = preg_replace_callback("@(\b\w+\b)@", function($m) use($wrapat) {return wordwrap($m[1], $wrapat, ' ', 1); },$output);
                         break;
                     case "limit": // default: 100
                         $limit = intval($modifier_value[$i]) ? intval($modifier_value[$i]) : 100;
@@ -412,7 +412,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)) {
@@ -439,7 +439,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;
@@ -464,11 +464,11 @@ 
                     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);
@@ -476,14 +476,14 @@ 
                                 $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");
@@ -499,7 +499,7 @@ 
                             $options = $modifier_value[$i];
                             $custom = eval($cm);
                             $msg = ob_get_contents();
-                            $output = $msg . $custom;
+                            $output = $msg.$custom;
                             ob_end_clean();
                         } else {
                             $output = '';
@@ -507,9 +507,9 @@ 
                         break;
                 }
                 if (count($condition)) {
-                    $this->Log("  |--- Condition = '" . $condition[count($condition) - 1] . "'");
+                    $this->Log("  |--- Condition = '".$condition[count($condition) - 1]."'");
                 }
-                $this->Log("  |--- Output = '" . $output . "'");
+                $this->Log("  |--- Output = '".$output."'");
             }
         }
 
@@ -517,7 +517,7 @@ 
     }
 
     private function runCode($code){
-        return eval("return (" . $code . ");");
+        return eval("return (".$code.");");
     }
     // Event logging (debug)
     /**
@@ -529,7 +529,7 @@ 
             $console = implode("\n", $this->console);
             $this->console = array();
 
-            return '<pre style="overflow: auto;">' . $console . '</pre>';
+            return '<pre style="overflow: auto;">'.$console.'</pre>';
         }
     }
 
@@ -554,8 +554,8 @@ 
     {
         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);
         }
     }
 
@@ -567,15 +567,15 @@ 
     {
         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
@@ -584,7 +584,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);
     }
 
 
@@ -636,7 +636,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];
@@ -744,7 +744,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);
@@ -757,7 +757,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);

Braces +6 added lines, -3 removed lines

@@ -467,7 +467,8 @@ 
                         $snippetName = 'phx:' . $modifier_cmd[$i];
                         if (isset($modx->snippetCache[$snippetName])) {
                             $snippet = $modx->snippetCache[$snippetName];
-                        } else { // not in cache so let's check the db
+                        } 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) . "';";
                             $result = $modx->dbQuery($sql);
                             if ($modx->recordCount($result) == 1) {
@@ -475,7 +476,8 @@ 
                                 $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
+                                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';
                                     if (@file_exists($filename)) {
                                         $file_contents = @file_get_contents($filename);
@@ -516,7 +518,8 @@ 
         return $output;
     }
 
-    private function runCode($code){
+    private function runCode($code)
+    {
         return eval("return (" . $code . ");");
     }
     // Event logging (debug)

assets/lib/APIHelpers.class.php

Spacing +9 added lines, -9 removed lines

@@ -13,7 +13,7 @@ 
      */
     public static function mb_lcfirst($str, $encoding = 'UTF-8')
     {
-        return mb_strtolower(mb_substr($str, 0, 1, $encoding), $encoding) . mb_substr($str, 1, mb_strlen($str),
+        return mb_strtolower(mb_substr($str, 0, 1, $encoding), $encoding).mb_substr($str, 1, mb_strlen($str),
             $encoding);
     }
 
@@ -26,7 +26,7 @@ 
     public static function mb_ucfirst($str, $encoding = 'UTF-8')
     {
         $str = mb_ereg_replace('^[\ ]+', '', $str);
-        $str = mb_strtoupper(mb_substr($str, 0, 1, $encoding), $encoding) . mb_substr($str, 1, mb_strlen($str),
+        $str = mb_strtoupper(mb_substr($str, 0, 1, $encoding), $encoding).mb_substr($str, 1, mb_strlen($str),
                 $encoding);
 
         return $str;
@@ -255,7 +255,7 @@ 
             '`'   => '`',
             '%60' => '`'
         )
-    ) {
+    ){
         switch (true) {
             case is_scalar($data):
                 $out = str_replace(
@@ -349,7 +349,7 @@ 
                     case 'num':
                         $tmp = range('0', '9');
                         foreach ($tmp as $t) {
-                            $data[] = (string)$t;
+                            $data[] = (string) $t;
                         }
                         break;
                     case 'eng':
@@ -388,22 +388,22 @@ 
         if ($prefix == '' && $suffix == '') {
             $out = $data;
         } else {
-            $InsertPrefix = ($prefix != '') ? ($prefix . $addPS) : '';
-            $InsertSuffix = ($suffix != '') ? ($addPS . $suffix) : '';
+            $InsertPrefix = ($prefix != '') ? ($prefix.$addPS) : '';
+            $InsertSuffix = ($suffix != '') ? ($addPS.$suffix) : '';
             foreach ($data as $key => $item) {
-                $key = $InsertPrefix . $key;
+                $key = $InsertPrefix.$key;
                 $val = null;
                 switch (true) {
                     case is_scalar($item):
                         $val = $item;
                         break;
                     case is_array($item):
-                        $val = self::renameKeyArr($item, $key . $sep, $InsertSuffix, '', $sep);
+                        $val = self::renameKeyArr($item, $key.$sep, $InsertSuffix, '', $sep);
                         $out = array_merge($out, $val);
                         $val = '';
                         break;
                 }
-                $out[$key . $InsertSuffix] = $val;
+                $out[$key.$InsertSuffix] = $val;
             }
         }
 

assets/lib/Formatter/SqlFormatter.php

Spacing +48 added lines, -48 removed lines

@@ -757,15 +757,15 @@ 
         self::$reserved = array_keys($reservedMap);
 
         // Set up regular expressions
-        self::$regex_boundaries = '(' . implode('|',
-                array_map(array(__CLASS__, 'quote_regex'), self::$boundaries)) . ')';
-        self::$regex_reserved = '(' . implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$reserved)) . ')';
+        self::$regex_boundaries = '('.implode('|',
+                array_map(array(__CLASS__, 'quote_regex'), self::$boundaries)).')';
+        self::$regex_reserved = '('.implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$reserved)).')';
         self::$regex_reserved_toplevel = str_replace(' ', '\\s+',
-            '(' . implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$reserved_toplevel)) . ')');
+            '('.implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$reserved_toplevel)).')');
         self::$regex_reserved_newline = str_replace(' ', '\\s+',
-            '(' . implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$reserved_newline)) . ')');
+            '('.implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$reserved_newline)).')');
 
-        self::$regex_function = '(' . implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$functions)) . ')';
+        self::$regex_function = '('.implode('|', array_map(array(__CLASS__, 'quote_regex'), self::$functions)).')';
 
         self::$init = true;
     }
@@ -829,10 +829,10 @@ 
 
             // If the variable name is quoted
             if ($string[1] === '"' || $string[1] === '\'' || $string[1] === '`') {
-                $ret[self::TOKEN_VALUE] = $string[0] . self::getQuotedString(substr($string, 1));
+                $ret[self::TOKEN_VALUE] = $string[0].self::getQuotedString(substr($string, 1));
             } // Non-quoted variable name
             else {
-                preg_match('/^(' . $string[0] . '[a-zA-Z0-9\._\$]+)/', $string, $matches);
+                preg_match('/^('.$string[0].'[a-zA-Z0-9\._\$]+)/', $string, $matches);
                 if (!empty($matches)) {
                     $ret[self::TOKEN_VALUE] = $matches[1];
                 }
@@ -844,7 +844,7 @@ 
         }
 
         // Number (decimal, binary, or hex)
-        if (preg_match('/^([0-9]+(\.[0-9]+)?|0x[0-9a-fA-F]+|0b[01]+)($|\s|"\'`|' . self::$regex_boundaries . ')/',
+        if (preg_match('/^([0-9]+(\.[0-9]+)?|0x[0-9a-fA-F]+|0b[01]+)($|\s|"\'`|'.self::$regex_boundaries.')/',
             $string, $matches)) {
             return array(
                 self::TOKEN_VALUE => $matches[1],
@@ -853,7 +853,7 @@ 
         }
 
         // Boundary Character (punctuation and symbols)
-        if (preg_match('/^(' . self::$regex_boundaries . ')/', $string, $matches)) {
+        if (preg_match('/^('.self::$regex_boundaries.')/', $string, $matches)) {
             return array(
                 self::TOKEN_VALUE => $matches[1],
                 self::TOKEN_TYPE  => self::TOKEN_TYPE_BOUNDARY
@@ -865,7 +865,7 @@ 
         if (!$previous || !isset($previous[self::TOKEN_VALUE]) || $previous[self::TOKEN_VALUE] !== '.') {
             $upper = strtoupper($string);
             // Top Level Reserved Word
-            if (preg_match('/^(' . self::$regex_reserved_toplevel . ')($|\s|' . self::$regex_boundaries . ')/', $upper,
+            if (preg_match('/^('.self::$regex_reserved_toplevel.')($|\s|'.self::$regex_boundaries.')/', $upper,
                 $matches)) {
                 return array(
                     self::TOKEN_TYPE  => self::TOKEN_TYPE_RESERVED_TOPLEVEL,
@@ -873,7 +873,7 @@ 
                 );
             }
             // Newline Reserved Word
-            if (preg_match('/^(' . self::$regex_reserved_newline . ')($|\s|' . self::$regex_boundaries . ')/', $upper,
+            if (preg_match('/^('.self::$regex_reserved_newline.')($|\s|'.self::$regex_boundaries.')/', $upper,
                 $matches)) {
                 return array(
                     self::TOKEN_TYPE  => self::TOKEN_TYPE_RESERVED_NEWLINE,
@@ -881,7 +881,7 @@ 
                 );
             }
             // Other Reserved Word
-            if (preg_match('/^(' . self::$regex_reserved . ')($|\s|' . self::$regex_boundaries . ')/', $upper,
+            if (preg_match('/^('.self::$regex_reserved.')($|\s|'.self::$regex_boundaries.')/', $upper,
                 $matches)) {
                 return array(
                     self::TOKEN_TYPE  => self::TOKEN_TYPE_RESERVED,
@@ -894,7 +894,7 @@ 
         // this makes it so "count(" is considered a function, but "count" alone is not
         $upper = strtoupper($string);
         // function
-        if (preg_match('/^(' . self::$regex_function . '[(]|\s|[)])/', $upper, $matches)) {
+        if (preg_match('/^('.self::$regex_function.'[(]|\s|[)])/', $upper, $matches)) {
             return array(
                 self::TOKEN_TYPE  => self::TOKEN_TYPE_RESERVED,
                 self::TOKEN_VALUE => substr($string, 0, strlen($matches[1]) - 1)
@@ -902,7 +902,7 @@ 
         }
 
         // Non reserved word
-        preg_match('/^(.*?)($|\s|["\'`]|' . self::$regex_boundaries . ')/', $string, $matches);
+        preg_match('/^(.*?)($|\s|["\'`]|'.self::$regex_boundaries.')/', $string, $matches);
 
         return array(
             self::TOKEN_VALUE => $matches[1],
@@ -1063,7 +1063,7 @@ 
 
             // If we need a new line before the token
             if ($newline) {
-                $return .= "\n" . str_repeat($tab, $indent_level);
+                $return .= "\n".str_repeat($tab, $indent_level);
                 $newline = false;
                 $added_newline = true;
             } else {
@@ -1074,8 +1074,8 @@ 
             if ($token[self::TOKEN_TYPE] === self::TOKEN_TYPE_COMMENT || $token[self::TOKEN_TYPE] === self::TOKEN_TYPE_BLOCK_COMMENT) {
                 if ($token[self::TOKEN_TYPE] === self::TOKEN_TYPE_BLOCK_COMMENT) {
                     $indent = str_repeat($tab, $indent_level);
-                    $return .= "\n" . $indent;
-                    $highlighted = str_replace("\n", "\n" . $indent, $highlighted);
+                    $return .= "\n".$indent;
+                    $highlighted = str_replace("\n", "\n".$indent, $highlighted);
                 }
 
                 $return .= $highlighted;
@@ -1091,12 +1091,12 @@ 
                     if ($inline_indented) {
                         array_shift($indent_types);
                         $indent_level--;
-                        $return .= "\n" . str_repeat($tab, $indent_level);
+                        $return .= "\n".str_repeat($tab, $indent_level);
                     }
 
                     $inline_parentheses = false;
 
-                    $return .= $highlighted . ' ';
+                    $return .= $highlighted.' ';
                     continue;
                 }
 
@@ -1183,14 +1183,14 @@ 
                     $indent_level = 0;
 
                     if ($highlight) {
-                        $return .= "\n" . self::highlightError($token[self::TOKEN_VALUE]);
+                        $return .= "\n".self::highlightError($token[self::TOKEN_VALUE]);
                         continue;
                     }
                 }
 
                 // Add a newline before the closing parentheses (if not already added)
                 if (!$added_newline) {
-                    $return .= "\n" . str_repeat($tab, $indent_level);
+                    $return .= "\n".str_repeat($tab, $indent_level);
                 }
             } // Top level reserved words start a new line and increase the special indent level
             elseif ($token[self::TOKEN_TYPE] === self::TOKEN_TYPE_RESERVED_TOPLEVEL) {
@@ -1207,10 +1207,10 @@ 
                 $newline = true;
                 // Add a newline before the top level reserved word (if not already added)
                 if (!$added_newline) {
-                    $return .= "\n" . str_repeat($tab, $indent_level);
+                    $return .= "\n".str_repeat($tab, $indent_level);
                 } // If we already added a newline, redo the indentation since it may be different now
                 else {
-                    $return = rtrim($return, $tab) . str_repeat($tab, $indent_level);
+                    $return = rtrim($return, $tab).str_repeat($tab, $indent_level);
                 }
 
                 // If the token may have extra whitespace
@@ -1240,7 +1240,7 @@ 
             elseif ($token[self::TOKEN_TYPE] === self::TOKEN_TYPE_RESERVED_NEWLINE) {
                 // Add a newline before the reserved word (if not already added)
                 if (!$added_newline) {
-                    $return .= "\n" . str_repeat($tab, $indent_level);
+                    $return .= "\n".str_repeat($tab, $indent_level);
                 }
 
                 // If the token may have extra whitespace
@@ -1263,7 +1263,7 @@ 
                 $return = rtrim($return, ' ');
             }
 
-            $return .= $highlighted . ' ';
+            $return .= $highlighted.' ';
 
             // If the token shouldn't have a space after it
             if ($token[self::TOKEN_VALUE] === '(' || $token[self::TOKEN_VALUE] === '.') {
@@ -1281,7 +1281,7 @@ 
 
         // If there are unmatched parentheses
         if ($highlight && array_search('block', $indent_types) !== false) {
-            $return .= "\n" . self::highlightError("WARNING: unclosed parentheses or section");
+            $return .= "\n".self::highlightError("WARNING: unclosed parentheses or section");
         }
 
         // Replace tab characters with the configuration tab character
@@ -1334,7 +1334,7 @@ 
             // If this is a query separator
             if ($token[self::TOKEN_VALUE] === ';') {
                 if (!$empty) {
-                    $queries[] = $current_query . ';';
+                    $queries[] = $current_query.';';
                 }
                 $current_query = '';
                 $empty = true;
@@ -1480,9 +1480,9 @@ 
     protected static function highlightQuote($value)
     {
         if (self::is_cli()) {
-            return self::$cli_quote . $value . "\x1b[0m";
+            return self::$cli_quote.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$quote_attributes . '>' . $value . '</span>';
+            return '<span '.self::$quote_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1496,9 +1496,9 @@ 
     protected static function highlightBacktickQuote($value)
     {
         if (self::is_cli()) {
-            return self::$cli_backtick_quote . $value . "\x1b[0m";
+            return self::$cli_backtick_quote.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$backtick_quote_attributes . '>' . $value . '</span>';
+            return '<span '.self::$backtick_quote_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1512,9 +1512,9 @@ 
     protected static function highlightReservedWord($value)
     {
         if (self::is_cli()) {
-            return self::$cli_reserved . $value . "\x1b[0m";
+            return self::$cli_reserved.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$reserved_attributes . '>' . $value . '</span>';
+            return '<span '.self::$reserved_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1532,9 +1532,9 @@ 
         }
 
         if (self::is_cli()) {
-            return self::$cli_boundary . $value . "\x1b[0m";
+            return self::$cli_boundary.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$boundary_attributes . '>' . $value . '</span>';
+            return '<span '.self::$boundary_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1548,9 +1548,9 @@ 
     protected static function highlightNumber($value)
     {
         if (self::is_cli()) {
-            return self::$cli_number . $value . "\x1b[0m";
+            return self::$cli_number.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$number_attributes . '>' . $value . '</span>';
+            return '<span '.self::$number_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1564,9 +1564,9 @@ 
     protected static function highlightError($value)
     {
         if (self::is_cli()) {
-            return self::$cli_error . $value . "\x1b[0m";
+            return self::$cli_error.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$error_attributes . '>' . $value . '</span>';
+            return '<span '.self::$error_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1580,9 +1580,9 @@ 
     protected static function highlightComment($value)
     {
         if (self::is_cli()) {
-            return self::$cli_comment . $value . "\x1b[0m";
+            return self::$cli_comment.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$comment_attributes . '>' . $value . '</span>';
+            return '<span '.self::$comment_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1596,9 +1596,9 @@ 
     protected static function highlightWord($value)
     {
         if (self::is_cli()) {
-            return self::$cli_word . $value . "\x1b[0m";
+            return self::$cli_word.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$word_attributes . '>' . $value . '</span>';
+            return '<span '.self::$word_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1612,9 +1612,9 @@ 
     protected static function highlightVariable($value)
     {
         if (self::is_cli()) {
-            return self::$cli_variable . $value . "\x1b[0m";
+            return self::$cli_variable.$value."\x1b[0m";
         } else {
-            return '<span ' . self::$variable_attributes . '>' . $value . '</span>';
+            return '<span '.self::$variable_attributes.'>'.$value.'</span>';
         }
     }
 
@@ -1640,14 +1640,14 @@ 
     private static function output($string)
     {
         if (self::is_cli()) {
-            return $string . "\n";
+            return $string."\n";
         } else {
             $string = trim($string);
             if (!self::$use_pre) {
                 return $string;
             }
 
-            return '<pre ' . self::$pre_attributes . '>' . $string . '</pre>';
+            return '<pre '.self::$pre_attributes.'>'.$string.'</pre>';
         }
     }
 

Braces +4 added lines, -2 removed lines

@@ -795,7 +795,8 @@ 
             if ($string[0] === '-' || $string[0] === '#') {
                 $last = strpos($string, "\n");
                 $type = self::TOKEN_TYPE_COMMENT;
-            } else { // Comment until closing comment tag
+            } else {
+// Comment until closing comment tag
                 $last = strpos($string, "*/", 2) + 2;
                 $type = self::TOKEN_TYPE_BLOCK_COMMENT;
             }
@@ -1044,7 +1045,8 @@ 
             // Get highlighted token if doing syntax highlighting
             if ($highlight) {
                 $highlighted = self::highlightToken($token);
-            } else { // If returning raw text
+            } else {
+// If returning raw text
                 $highlighted = $token[self::TOKEN_VALUE];
             }
 

assets/lib/SimpleTab/table.abstract.php

Spacing +16 added lines, -16 removed lines

@@ -1,8 +1,8 @@ 
 <?php namespace SimpleTab;
 
-require_once(MODX_BASE_PATH . 'assets/lib/MODxAPI/autoTable.abstract.php');
-require_once(MODX_BASE_PATH . 'assets/lib/Helpers/FS.php');
-require_once(MODX_BASE_PATH . 'assets/lib/Helpers/PHPThumb.php');
+require_once(MODX_BASE_PATH.'assets/lib/MODxAPI/autoTable.abstract.php');
+require_once(MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
+require_once(MODX_BASE_PATH.'assets/lib/Helpers/PHPThumb.php');
 
 /**
  * Class dataTable
@@ -59,7 +59,7 @@ 
         $rows = $this->query("SELECT MIN(`{$this->indexName}`) FROM {$table} WHERE `{$this->pkName}` IN ({$ids})");
         $index = $this->modx->db->getValue($rows);
         $index = $index - 1;
-        $this->query("SET @index := " . $index);
+        $this->query("SET @index := ".$index);
         $this->query("UPDATE {$table} SET `{$this->indexName}` = (@index := @index + 1) WHERE (`{$this->indexName}`>{$index} AND `{$this->rfName}`={$rid} AND `{$this->pkName}` NOT IN ({$ids})) ORDER BY `{$this->indexName}` ASC");
         $out = $this->modx->db->getAffectedRows();
 
@@ -94,13 +94,13 @@ 
         $cnt = count($ids);
         $ids = implode(',', $ids);
         if ($dir == 'top') {
-            $this->query("SET @index := " . ($index - $cnt - 1));
+            $this->query("SET @index := ".($index - $cnt - 1));
             $this->query("UPDATE {$table} SET `{$this->indexName}` = (@index := @index + 1) WHERE (`{$this->pkName}` IN ({$ids})) ORDER BY `{$this->indexName}` ASC");
             $this->query("SET @index := -1");
         } else {
             $this->query("SET @index := -1");
             $this->query("UPDATE {$table} SET `{$this->indexName}` = (@index := @index + 1) WHERE (`{$this->pkName}` IN ({$ids})) ORDER BY `{$this->indexName}` ASC");
-            $this->query("SET @index := " . ($cnt - 1));
+            $this->query("SET @index := ".($cnt - 1));
         }
         $this->query("UPDATE {$table} SET `{$this->indexName}` = (@index := @index + 1) WHERE (`{$this->pkName}` NOT IN ({$ids})) AND `{$this->rfName}` = {$rid} ORDER BY `{$this->indexName}` ASC");
         $out = $this->modx->db->getAffectedRows();
@@ -119,7 +119,7 @@ 
             return;
         }
         if ($this->fs->checkFile($url)) {
-            unlink(MODX_BASE_PATH . $url);
+            unlink(MODX_BASE_PATH.$url);
         }
         $dir = $this->fs->takeFileDir($url);
         $iterator = new \FilesystemIterator($dir);
@@ -129,8 +129,8 @@ 
         if ($cache) {
             return;
         }
-        $thumbsCache = \APIhelpers::getkey($this->params,'thumbsCache',$this->thumbsCache);
-        $thumb = $thumbsCache . $url;
+        $thumbsCache = \APIhelpers::getkey($this->params, 'thumbsCache', $this->thumbsCache);
+        $thumb = $thumbsCache.$url;
         if ($this->fs->checkFile($thumb)) {
             $this->deleteThumb($thumb, true);
         }
@@ -183,7 +183,7 @@ 
         $filename = $this->fs->takeFileName($name);
         $ext = $this->fs->takeFileExt($name);
 
-        return $this->modx->stripAlias($filename) . '.' . $ext;
+        return $this->modx->stripAlias($filename).'.'.$ext;
     }
 
     /**
@@ -196,12 +196,12 @@ 
      */
     public function reorder($source, $target, $point, $rid, $orderDir)
     {
-        $rid = (int)$rid;
+        $rid = (int) $rid;
         $point = strtolower($point);
         $orderDir = strtolower($orderDir);
-        $sourceIndex = (int)$source[$this->indexName];
-        $targetIndex = (int)$target[$this->indexName];
-        $sourceId = (int)$source[$this->pkName];
+        $sourceIndex = (int) $source[$this->indexName];
+        $targetIndex = (int) $target[$this->indexName];
+        $sourceId = (int) $source[$this->pkName];
         $table = $this->makeTable($this->table);
         $rows = 0;
         /* more refactoring  needed */
@@ -246,8 +246,8 @@ 
             return false;
         }
         $thumb = new \Helpers\PHPThumb();
-        $inputFile = MODX_BASE_PATH . $this->fs->relativePath($url);
-        $outputFile = MODX_BASE_PATH . $this->fs->relativePath($folder) . '/' . $this->fs->relativePath($url);
+        $inputFile = MODX_BASE_PATH.$this->fs->relativePath($url);
+        $outputFile = MODX_BASE_PATH.$this->fs->relativePath($folder).'/'.$this->fs->relativePath($url);
         $dir = $this->fs->takeFileDir($outputFile);
         $this->fs->makeDir($dir, $this->modx->config['new_folder_permissions']);
         if ($thumb->create($inputFile, $outputFile, $options)) {

Braces +3 added lines, -1 removed lines

@@ -272,6 +272,8 @@
      */
     public function setParams($params = array())
     {
-        if (is_array($params)) $this->params = $params;
+        if (is_array($params)) {
+            $this->params = $params;
+        }
     }
 }

assets/lib/MODxAPI/modResource.php

Spacing +39 added lines, -39 removed lines

@@ -163,7 +163,7 @@ 
         $this->get_TV();
         $uTable = $this->makeTable("manager_users");
         $aTable = $this->makeTable("user_attributes");
-        $query = "SELECT `u`.`id`, `a`.`email`, `u`.`username`  FROM " . $aTable . " as `a` LEFT JOIN " . $uTable . " as `u` ON `u`.`id`=`a`.`internalKey`";
+        $query = "SELECT `u`.`id`, `a`.`email`, `u`.`username`  FROM ".$aTable." as `a` LEFT JOIN ".$uTable." as `u` ON `u`.`id`=`a`.`internalKey`";
         $query = $this->query($query);
         $this->managerUsers = new DLCollection($modx, empty($query) ? array() : $query);
     }
@@ -225,7 +225,7 @@ 
     public function getUrl()
     {
         $out = null;
-        $id = (int)$this->getID();
+        $id = (int) $this->getID();
         if (!empty($id)) {
             $out = $this->modx->makeUrl($id);
         }
@@ -278,8 +278,8 @@ 
     {
         $out = null;
         if ($this->getID() > 0) {
-            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";
             $tvval = $this->get($tvname);
             if ($this->isTVarrayField($tvname) && is_array($tvval)) {
                 $tvval = implode('||', $tvval);
@@ -323,14 +323,14 @@ 
         if ((is_scalar($value) || $this->isTVarrayField($key) || $this->isJsonField($key)) && is_scalar($key) && !empty($key)) {
             switch ($key) {
                 case 'parent':
-                    $value = (int)$value;
+                    $value = (int) $value;
                     break;
                 case 'template':
                     $value = trim($value);
                     $value = $this->setTemplate($value);
                     break;
                 case 'published':
-                    $value = (int)((bool)$value);
+                    $value = (int) ((bool) $value);
                     if ($value) {
                         $this->field['publishedon'] = time() + $this->modxConfig('server_offset_time');
                     }
@@ -350,7 +350,7 @@ 
                     }
                     break;
                 case 'deleted':
-                    $value = (int)((bool)$value);
+                    $value = (int) ((bool) $value);
                     if ($value) {
                         $this->field['deletedon'] = time() + $this->modxConfig('server_offset_time');
                     } else {
@@ -392,11 +392,11 @@ 
     protected function getUser($value, $default = 0)
     {
         $currentAdmin = APIHelpers::getkey($_SESSION, 'mgrInternalKey', 0);
-        $value = (int)$value;
+        $value = (int) $value;
         if (!empty($value)) {
             $by = $this->findUserBy($value);
-            $exists = $this->managerUsers->exists(function ($key, Helpers\Collection $val) use ($by, $value) {
-                return ($val->containsKey($by) && $val->get($by) === (string)$value);
+            $exists = $this->managerUsers->exists(function($key, Helpers\Collection $val) use ($by, $value) {
+                return ($val->containsKey($by) && $val->get($by) === (string) $value);
             });
             if (!$exists) {
                 $value = 0;
@@ -416,7 +416,7 @@ 
     protected function findUserBy($data)
     {
         switch (true) {
-            case (is_int($data) || ((int)$data > 0 && (string)intval($data) === $data)):
+            case (is_int($data) || ((int) $data > 0 && (string) intval($data) === $data)):
                 $find = 'id';
                 break;
             case filter_var($data, FILTER_VALIDATE_EMAIL):
@@ -466,9 +466,9 @@ 
             $this->markAllEncode();
             $this->newDoc = false;
 
-            $result = $this->query("SELECT * from {$this->makeTable('site_content')} where `id`=" . (int)$id);
+            $result = $this->query("SELECT * from {$this->makeTable('site_content')} where `id`=".(int) $id);
             $this->fromArray($this->modx->db->getRow($result));
-            $result = $this->query("SELECT * from {$this->makeTable('site_tmplvar_contentvalues')} where `contentid`=" . (int)$id);
+            $result = $this->query("SELECT * from {$this->makeTable('site_tmplvar_contentvalues')} where `contentid`=".(int) $id);
             while ($row = $this->modx->db->getRow($result)) {
                 $this->field[$this->tvid[$row['tmplvarid']]] = $row['value'];
             }
@@ -495,7 +495,7 @@ 
     {
         $parent = null;
         if ($this->field['pagetitle'] == '') {
-            $this->log['emptyPagetitle'] = 'Pagetitle is empty in <pre>' . print_r($this->field, true) . '</pre>';
+            $this->log['emptyPagetitle'] = 'Pagetitle is empty in <pre>'.print_r($this->field, true).'</pre>';
 
             return false;
         }
@@ -548,7 +548,7 @@ 
             }
             switch (true) {
                 case $key == 'parent':
-                    $parent = (int)$this->get($key);
+                    $parent = (int) $this->get($key);
                     $q = $this->query("SELECT count(`id`) FROM {$this->makeTable('site_content')} WHERE `id`='{$parent}'");
                     if ($this->modx->db->getValue($q) != 1) {
                         $parent = 0;
@@ -567,10 +567,10 @@ 
 
         if (!empty($this->set)) {
             if ($this->newDoc) {
-                $SQL = "INSERT into {$this->makeTable('site_content')} SET " . implode(', ', $this->set);
+                $SQL = "INSERT into {$this->makeTable('site_content')} SET ".implode(', ', $this->set);
             } else {
-                $SQL = "UPDATE {$this->makeTable('site_content')} SET " . implode(', ',
-                        $this->set) . " WHERE `id` = " . $this->id;
+                $SQL = "UPDATE {$this->makeTable('site_content')} SET ".implode(', ',
+                        $this->set)." WHERE `id` = ".$this->id;
             }
             $this->query($SQL);
 
@@ -634,7 +634,7 @@ 
      * @param $tvId
      * @return bool
      */
-    protected function belongsToTemplate($tvId) {
+    protected function belongsToTemplate($tvId){
         $template = $this->get('template');
 
         return isset($this->tvTpl[$template]) && in_array($tvId, $this->tvTpl[$template]);
@@ -651,12 +651,12 @@ 
         $_ids = $this->cleanIDs($ids, ',', $ignore);
         if (is_array($_ids) && $_ids != array()) {
             $id = $this->sanitarIn($_ids);
-            $uid = (int)$this->modx->getLoginUserId();
+            $uid = (int) $this->modx->getLoginUserId();
             $deletedon = time() + $this->modxConfig('server_offset_time');
             $this->query("UPDATE {$this->makeTable('site_content')} SET `deleted`=1, `deletedby`={$uid}, `deletedon`={$deletedon} WHERE `id` IN ({$id})");
         } else {
-            throw new Exception('Invalid IDs list for mark trash: <pre>' . print_r($ids,
-                    1) . '</pre> please, check ignore list: <pre>' . print_r($ignore, 1) . '</pre>');
+            throw new Exception('Invalid IDs list for mark trash: <pre>'.print_r($ids,
+                    1).'</pre> please, check ignore list: <pre>'.print_r($ignore, 1).'</pre>');
         }
 
         return $this;
@@ -738,15 +738,15 @@ 
     {
         $ignore = array(
             0, //empty document
-            (int)$this->modxConfig('site_start'),
-            (int)$this->modxConfig('error_page'),
-            (int)$this->modxConfig('unauthorized_page'),
-            (int)$this->modxConfig('site_unavailable_page')
+            (int) $this->modxConfig('site_start'),
+            (int) $this->modxConfig('error_page'),
+            (int) $this->modxConfig('unauthorized_page'),
+            (int) $this->modxConfig('site_unavailable_page')
         );
         $data = $this->query("SELECT DISTINCT setting_value FROM {$this->makeTable('web_user_settings')} WHERE `setting_name`='login_home' AND `setting_value`!=''");
         $data = $this->modx->db->makeArray($data);
         foreach ($data as $item) {
-            $ignore[] = (int)$item['setting_value'];
+            $ignore[] = (int) $item['setting_value'];
         }
 
         return array_unique($ignore);
@@ -769,9 +769,9 @@ 
             }
             if (($flag && $this->newDoc) || (!$this->newDoc && $flag && $this->id != $flag)) {
                 $suffix = substr($alias, -2);
-                if (preg_match('/-(\d+)/', $suffix, $tmp) && isset($tmp[1]) && (int)$tmp[1] > 1) {
-                    $suffix = (int)$tmp[1] + 1;
-                    $alias = substr($alias, 0, -2) . '-' . $suffix;
+                if (preg_match('/-(\d+)/', $suffix, $tmp) && isset($tmp[1]) && (int) $tmp[1] > 1) {
+                    $suffix = (int) $tmp[1] + 1;
+                    $alias = substr($alias, 0, -2).'-'.$suffix;
                 } else {
                     $alias .= '-2';
                 }
@@ -799,7 +799,7 @@ 
     {
         $this->modx->_TVnames = array();
         if (empty($this->modx->_TVnames) || $reload) {
-            $result = $this->query('SELECT `id`,`name`,`type` FROM ' . $this->makeTable('site_tmplvars'));
+            $result = $this->query('SELECT `id`,`name`,`type` FROM '.$this->makeTable('site_tmplvars'));
             while ($row = $this->modx->db->GetRow($result)) {
                 $this->modx->_TVnames[$row['name']] = array(
                     "id"   => $row['id'],
@@ -827,7 +827,7 @@ 
      */
     protected function loadTVTemplate()
     {
-        $q = $this->query("SELECT `tmplvarid`, `templateid` FROM " . $this->makeTable('site_tmplvar_templates'));
+        $q = $this->query("SELECT `tmplvarid`, `templateid` FROM ".$this->makeTable('site_tmplvar_templates'));
         $q = $this->modx->db->makeArray($q);
         $this->tvTpl = array();
         foreach ($q as $item) {
@@ -865,20 +865,20 @@ 
      */
     public function setTemplate($tpl)
     {
-        if (!is_numeric($tpl) || $tpl != (int)$tpl) {
+        if (!is_numeric($tpl) || $tpl != (int) $tpl) {
             if (is_scalar($tpl)) {
-                $sql = "SELECT `id` FROM {$this->makeTable('site_templates')} WHERE `templatename` = '" . $this->escape($tpl) . "'";
+                $sql = "SELECT `id` FROM {$this->makeTable('site_templates')} WHERE `templatename` = '".$this->escape($tpl)."'";
                 $rs = $this->query($sql);
                 if (!$rs || $this->modx->db->getRecordCount($rs) <= 0) {
                     throw new Exception("Template {$tpl} is not exists");
                 }
                 $tpl = $this->modx->db->getValue($rs);
             } else {
-                throw new Exception("Invalid template name: " . print_r($tpl, 1));
+                throw new Exception("Invalid template name: ".print_r($tpl, 1));
             }
         }
 
-        return (int)$tpl;
+        return (int) $tpl;
     }
 
     /**
@@ -1002,7 +1002,7 @@ 
         if ($this->isEncodableField($field)) {
             $data = $this->get($field);
             if ($this->isTVarrayField($field)) {
-                $out = is_array($data) ? implode('||', $data) : (string)$data;
+                $out = is_array($data) ? implode('||', $data) : (string) $data;
             } else {
                 $out = json_encode($data);
             }
@@ -1056,7 +1056,7 @@ 
     /**
      * @param int $docId
      */
-    public function getDocumentGroups($docId = 0) {
+    public function getDocumentGroups($docId = 0){
         $out = array();
         $doc = $this->switchObject($docId);
         if (null !== $doc->getID()) {
@@ -1064,7 +1064,7 @@ 
             $docgroup_names = $this->makeTable('documentgroup_names');
 
             $rs = $this->query("SELECT `dg`.`document_group`, `dgn`.`name` FROM {$doc_groups} as `dg` INNER JOIN {$docgroup_names} as `dgn` ON `dgn`.`id`=`dg`.`document_group`
-                WHERE `dg`.`document` = " . $doc->getID());
+                WHERE `dg`.`document` = ".$doc->getID());
             while ($row = $this->modx->db->getRow($rs)) {
                 $out[$row['document_group']] = $row['name'];
             }

Braces +14 added lines, -6 removed lines

@@ -395,7 +395,7 @@ 
         $value = (int)$value;
         if (!empty($value)) {
             $by = $this->findUserBy($value);
-            $exists = $this->managerUsers->exists(function ($key, Helpers\Collection $val) use ($by, $value) {
+            $exists = $this->managerUsers->exists(function ($key, Helpers\Collection $val) use ($by, $value){
                 return ($val->containsKey($by) && $val->get($by) === (string)$value);
             });
             if (!$exists) {
@@ -614,7 +614,9 @@ 
             $this->newDoc = false;
         }
 
-        if (!empty($this->groupIds)) $this->setDocumentGroups($this->id, $this->groupIds);
+        if (!empty($this->groupIds)) {
+            $this->setDocumentGroups($this->id, $this->groupIds);
+        }
         $this->invokeEvent('OnDocFormSave', array(
             'mode'   => $this->mode,
             'id'     => isset($this->id) ? $this->id : '',
@@ -634,7 +636,8 @@ 
      * @param $tvId
      * @return bool
      */
-    protected function belongsToTemplate($tvId) {
+    protected function belongsToTemplate($tvId)
+    {
         $template = $this->get('template');
 
         return isset($this->tvTpl[$template]) && in_array($tvId, $this->tvTpl[$template]);
@@ -816,7 +819,9 @@ 
                 $arrayTVs[] = $name;
             }
         }
-        if (empty($this->tvaFields)) $this->tvaFields = $arrayTVs;
+        if (empty($this->tvaFields)) {
+            $this->tvaFields = $arrayTVs;
+        }
         $this->loadTVTemplate()->loadTVDefault(array_values($this->tv));
 
         return $this;
@@ -1056,7 +1061,8 @@ 
     /**
      * @param int $docId
      */
-    public function getDocumentGroups($docId = 0) {
+    public function getDocumentGroups($docId = 0)
+    {
         $out = array();
         $doc = $this->switchObject($docId);
         if (null !== $doc->getID()) {
@@ -1082,7 +1088,9 @@ 
      */
     public function setDocumentGroups($docId = 0, $groupIds = array())
     {
-        if (!is_array($groupIds)) return $this;
+        if (!is_array($groupIds)) {
+            return $this;
+        }
         if ($this->newDoc && $docId == 0) {
             $this->groupIds = $groupIds;
         } else {