AgelxNash / DocLister

assets/modules/redirectMap2/init.php

Indentation +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
 }
 
 if(!is_array($modx->event->params)){
-	$modx->event->params = array();
+    $modx->event->params = array();
 }
 
 $ajax = (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest');
@@ -52,11 +52,11 @@ 
 
 $tpl = \RedirectMap\Action::$TPL;
 if(!is_null($tpl)){
-	$out = $TPL->showHeader();
-	$out .= $TPL->showBody($tpl, $data);
-	$out .= $TPL->showFooter();
+    $out = $TPL->showHeader();
+    $out .= $TPL->showBody($tpl, $data);
+    $out .= $TPL->showFooter();
 }else{
-	header('Content-type: application/json');
-	$out = json_encode($data);
+    header('Content-type: application/json');
+    $out = json_encode($data);
 }
 echo $out;
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 	$out = $TPL->showHeader();
 	$out .= $TPL->showBody($tpl, $data);
 	$out .= $TPL->showFooter();
-}else{
+} else{
 	header('Content-type: application/json');
 	$out = json_encode($data);
 }

Spacing +8 added lines, -8 removed lines

@@ -16,29 +16,29 @@ 
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the MODX Content Manager instead of accessing this file directly.");
 }
 if (!$modx->hasPermission('exec_module')) {
-    header("location: " . $modx->getManagerPath() . "?a=106");
+    header("location: ".$modx->getManagerPath()."?a=106");
 }
 
-if(!is_array($modx->event->params)){
+if (!is_array($modx->event->params)) {
 	$modx->event->params = array();
 }
 
 $ajax = (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest');
 $action = isset($_REQUEST['action']) ? $_REQUEST['action'] : '';
 
-include_once(dirname(__FILE__) . "/src/modRedirectMap.class.php");
+include_once(dirname(__FILE__)."/src/modRedirectMap.class.php");
 include_once(MODX_BASE_PATH.'assets/lib/Module/Action.php');
 include_once(MODX_BASE_PATH.'assets/lib/Module/Helper.php');
 include_once(MODX_BASE_PATH.'assets/lib/Module/Template.php');
 include_once(MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
 
-include_once(dirname(__FILE__) . "/src/Helper.class.php");
+include_once(dirname(__FILE__)."/src/Helper.class.php");
 \RedirectMap\Helper::init($modx);
 
-include_once(dirname(__FILE__) . "/src/Template.class.php");
+include_once(dirname(__FILE__)."/src/Template.class.php");
 $TPL = new \RedirectMap\Template($modx, $ajax, dirname(__FILE__));
 
-include_once(dirname(__FILE__) . "/src/Action.class.php");
+include_once(dirname(__FILE__)."/src/Action.class.php");
 \RedirectMap\Action::init($modx, $TPL, new \RedirectMap\modRedirectMap($modx));
 
 if (!empty($action) && method_exists('\RedirectMap\Action', $action)) {
@@ -51,11 +51,11 @@ 
 }
 
 $tpl = \RedirectMap\Action::$TPL;
-if(!is_null($tpl)){
+if (!is_null($tpl)) {
 	$out = $TPL->showHeader();
 	$out .= $TPL->showBody($tpl, $data);
 	$out .= $TPL->showFooter();
-}else{
+} else {
 	header('Content-type: application/json');
 	$out = json_encode($data);
 }

assets/modules/RelativeTVList/init.php

Indentation +10 added lines, -10 removed lines

@@ -25,21 +25,21 @@
 \DLCity\Action::init($modx, $TPL, new modResource($modx));
 
 if (!empty($action) && method_exists('\DLCity\Action', $action)) {
-	$data = call_user_func_array(array('\DLCity\Action', $action), array());
-	if (!is_array($data)) {
-		$data = array();
-	}
+    $data = call_user_func_array(array('\DLCity\Action', $action), array());
+    if (!is_array($data)) {
+        $data = array();
+    }
 } else {
-	$data = array();
+    $data = array();
 }
 
 $tpl = \DLCity\Action::$TPL;
 if(!is_null($tpl)){
-	$out = $TPL->showHeader();
-	$out .= $TPL->showBody($tpl, $data);
-	$out .= $TPL->showFooter();
+    $out = $TPL->showHeader();
+    $out .= $TPL->showBody($tpl, $data);
+    $out .= $TPL->showFooter();
 }else{
-	header('Content-type: application/json');
-	$out = json_encode($data);
+    header('Content-type: application/json');
+    $out = json_encode($data);
 }
 echo $out;
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
 	$out = $TPL->showHeader();
 	$out .= $TPL->showBody($tpl, $data);
 	$out .= $TPL->showFooter();
-}else{
+} else{
 	header('Content-type: application/json');
 	$out = json_encode($data);
 }

Spacing +6 added lines, -6 removed lines

@@ -3,7 +3,7 @@ 
     die("<b>INCLUDE_ORDERING_ERROR</b><br /><br />Please use the MODX Content Manager instead of accessing this file directly.");
 }
 if (!$modx->hasPermission('exec_module')) {
-    header("location: " . $modx->getManagerPath() . "?a=106");
+    header("location: ".$modx->getManagerPath()."?a=106");
 }
 
 $ajax = (!empty($_SERVER['HTTP_X_REQUESTED_WITH']) && strtolower($_SERVER['HTTP_X_REQUESTED_WITH']) == 'xmlhttprequest');
@@ -15,13 +15,13 @@ 
 include_once(MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
 include_once(MODX_BASE_PATH.'assets/lib/MODxAPI/modResource.php');
 
-include_once(dirname(__FILE__) . "/Helper.class.php");
+include_once(dirname(__FILE__)."/Helper.class.php");
 \DLCity\Helper::init($modx, isset($_REQUEST['mode']) ? $_REQUEST['mode'] : 'city');
 
-include_once(dirname(__FILE__) . "/Template.class.php");
+include_once(dirname(__FILE__)."/Template.class.php");
 $TPL = new \DLCity\Template($modx, $ajax, dirname(__FILE__));
 
-include_once(dirname(__FILE__) . "/Action.class.php");
+include_once(dirname(__FILE__)."/Action.class.php");
 \DLCity\Action::init($modx, $TPL, new modResource($modx));
 
 if (!empty($action) && method_exists('\DLCity\Action', $action)) {
@@ -34,11 +34,11 @@ 
 }
 
 $tpl = \DLCity\Action::$TPL;
-if(!is_null($tpl)){
+if (!is_null($tpl)) {
 	$out = $TPL->showHeader();
 	$out .= $TPL->showBody($tpl, $data);
 	$out .= $TPL->showFooter();
-}else{
+} else {
 	header('Content-type: application/json');
 	$out = json_encode($data);
 }

assets/lib/APIHelpers.class.php

Indentation +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
         for ($i = $len; $i > 0; $i--) {
             switch ($data[rand(0, ($opt - 1))]) {
                 case 'A':
-                	$tmp = rand(65, 90);
+                    $tmp = rand(65, 90);
                     break;
                 case 'a':
                     $tmp = rand(97, 122);

Braces +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
         $text = mb_substr($text, 0, $len+1, $encoding);
         if(mb_substr($text, -1, null, $encoding) == ' '){
             $out = trim($text);
-        }else{
+        } else{
             $out = mb_substr($text, 0, mb_strripos($text, ' ', null, $encoding), $encoding);
         }
         return preg_replace("/(([\.,\-:!?;\s])|(&\w+;))+$/ui", "", $out);

Spacing +16 added lines, -16 removed lines

@@ -8,8 +8,8 @@ 
      * @param string $encoding - кодировка, по-умолчанию UTF-8
      * @return string
      */
-    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), $encoding);
+    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), $encoding);
     }
 
     /**
@@ -18,7 +18,7 @@ 
      * @param string $encoding - кодировка, по-умолчанию UTF-8
      * @return string
      */
-    public static function mb_ucfirst($str, $encoding='UTF-8')
+    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), $encoding);
@@ -36,12 +36,12 @@ 
      * @param string $encoding кодировка
      * @return string
      */
-    public static function mb_trim_word($html, $len, $encoding = 'UTF-8'){
+    public static function mb_trim_word($html, $len, $encoding = 'UTF-8') {
         $text = trim(preg_replace('|\s+|', ' ', strip_tags($html)));
-        $text = mb_substr($text, 0, $len+1, $encoding);
-        if(mb_substr($text, -1, null, $encoding) == ' '){
+        $text = mb_substr($text, 0, $len + 1, $encoding);
+        if (mb_substr($text, -1, null, $encoding) == ' ') {
             $out = trim($text);
-        }else{
+        } else {
             $out = mb_substr($text, 0, mb_strripos($text, ' ', null, $encoding), $encoding);
         }
         return preg_replace("/(([\.,\-:!?;\s])|(&\w+;))+$/ui", "", $out);
@@ -216,8 +216,8 @@ 
         '[' => '[', '%5B' => '[', ']' => ']', '%5D' => ']',
         '{' => '{', '%7B' => '{', '}' => '}', '%7D' => '}',
         '`' => '`', '%60' => '`'
-    )){
-        switch(true){
+    )) {
+        switch (true) {
             case is_scalar($data):
                 $out = str_replace(
                     array_keys($chars),
@@ -227,7 +227,7 @@ 
                 break;
             case is_array($data):
                 $out = $data;
-                foreach($out as $key => &$val){
+                foreach ($out as $key => &$val) {
                     $val = self::sanitarTag($val, $charset, $chars);
                 }
                 break;
@@ -237,7 +237,7 @@ 
         return $out;
     }
 
-    public static function e($text, $charset = 'UTF-8'){
+    public static function e($text, $charset = 'UTF-8') {
         return is_scalar($text) ? htmlspecialchars($text, ENT_QUOTES, $charset, false) : '';
     }
     /**
@@ -319,12 +319,12 @@ 
         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){
+                switch (true) {
                     case is_scalar($item):
                         $val = $item;
                         break;
@@ -334,7 +334,7 @@ 
                         $val = '';
                         break;
                 }
-                $out[$key . $InsertSuffix] = $val;
+                $out[$key.$InsertSuffix] = $val;
             }
         }
         return $out;

Doc Comments +2 added lines, -2 removed lines

@@ -73,7 +73,7 @@ 
      * @license    GNU General Public License (GPL), http://www.gnu.org/copyleft/gpl.html
      * @param string $email проверяемый email
      * @param boolean $dns проверять ли DNS записи
-     * @return boolean Результат проверки почтового ящика
+     * @return false|string Результат проверки почтового ящика
      * @author Anton Shevchuk
      */
     public static function emailValidate($email, $dns = true)
@@ -146,7 +146,7 @@ 
     }
 
     /**
-     * @param $data
+     * @param string $data
      * @return bool|false|string
      */
     private function _getEnv($data)

assets/lib/Module/Action.php

Braces +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
                     $textMethod = $data['key'].'Text';
                     if(method_exists($modObj, $textMethod)){
                         $out['value'] = $modObj->$textMethod();
-                    }else{
+                    } else{
                         $out['value'] = $modObj->get($data['key']);
                     }
                 }
@@ -84,10 +84,10 @@ 
                 ), self::$modx->getFullTableName(self::TABLE()), "id = ".$dataID);
             if($q){
                 $data['log'] = $oldValue ? 'Запись с ID '.$dataID.' восстановлена' : 'Запись с ID '.$dataID.' удалена';
-            }else{
+            } else{
                 $data['log'] = $oldValue ? 'Не удалось восстановить запись с ID '.$dataID : 'Не удалось удалить запись с ID '.$dataID;
             }
-        }else{
+        } else{
             $data['log'] = '<span class="error">Ошибка</span>. Не удалось определить обновляему запись';
         }
         return $data;

Indentation +5 added lines, -5 removed lines

@@ -9,7 +9,7 @@ 
     public static $TPL = null;
     protected static $TABLE = "site_content";
     protected static $classTable = null;
-	protected static $_tplObj = null;
+    protected static $_tplObj = null;
 
     /**
      * @param \DocumentParser $modx
@@ -26,7 +26,7 @@ 
     public static function init(\DocumentParser $modx, Template $tpl, \MODxAPI $classTable){
         self::setMODX($modx);
         self::$TPL = Template::showLog();
-		self::$_tplObj = $tpl;
+        self::$_tplObj = $tpl;
         self::$classTable = $classTable;
     }
 
@@ -124,7 +124,7 @@ 
      * @return array
      */
     public static function deleted(){
-		$data = array();
+        $data = array();
         $dataID = (int)Template::getParam('docId', $_GET);
         if($dataID>0 && self::_checkObj($dataID)){
             $oldValue = self::_getValue('deleted_at', $dataID);
@@ -149,6 +149,6 @@ 
      * @return null
      */
     public static function getClassTable(){
-		return self::$classTable;
-	}
+        return self::$classTable;
+    }
 }
\ No newline at end of file

Spacing +28 added lines, -28 removed lines

@@ -4,7 +4,7 @@ 
  * Class Action
  * @package Module
  */
-abstract class Action{
+abstract class Action {
     protected static $modx = null;
     public static $TPL = null;
     protected static $TABLE = "site_content";
@@ -14,7 +14,7 @@ 
     /**
      * @param \DocumentParser $modx
      */
-    public static function setMODX(\DocumentParser $modx){
+    public static function setMODX(\DocumentParser $modx) {
         self::$modx = $modx;
     }
 
@@ -23,7 +23,7 @@ 
      * @param Template $tpl
      * @param \MODxAPI $classTable
      */
-    public static function init(\DocumentParser $modx, Template $tpl, \MODxAPI $classTable){
+    public static function init(\DocumentParser $modx, Template $tpl, \MODxAPI $classTable) {
         self::setMODX($modx);
         self::$TPL = Template::showLog();
 		self::$_tplObj = $tpl;
@@ -33,7 +33,7 @@ 
     /**
      * @return string
      */
-    public static function TABLE(){
+    public static function TABLE() {
         return static::$TABLE;
     }
 
@@ -41,9 +41,9 @@ 
      * @param $id
      * @return bool
      */
-    protected static function _checkObj($id){
+    protected static function _checkObj($id) {
         $q = self::$modx->db->select('id', self::$modx->getFullTableName(self::TABLE()), "id = ".$id);
-        return (self::$modx->db->getRecordCount($q)==1);
+        return (self::$modx->db->getRecordCount($q) == 1);
     }
 
     /**
@@ -51,7 +51,7 @@ 
      * @param $id
      * @return mixed
      */
-    protected static function _getValue($field, $id){
+    protected static function _getValue($field, $id) {
         $q = self::$modx->db->select($field, self::$modx->getFullTableName(self::TABLE()), "id = ".$id);
         return self::$modx->db->getValue($q);
     }
@@ -59,11 +59,11 @@ 
     /**
      * @return array|mixed
      */
-    public static function listValue(){
-        $out = self::_workValue(function($data, $modObj){
+    public static function listValue() {
+        $out = self::_workValue(function($data, $modObj) {
             $listFunction = $data['key'].'Lists';
             $out = method_exists($modObj, $listFunction) ? $modObj->$listFunction() : array();
-            $out['selected'] =  $modObj->get($data['key']);
+            $out['selected'] = $modObj->get($data['key']);
             return $out;
         });
         self::$TPL = null;
@@ -75,14 +75,14 @@ 
      * @param $callback
      * @return array|mixed
      */
-    protected static function _workValue($callback){
+    protected static function _workValue($callback) {
         self::$TPL = 'ajax/getValue';
         $data = Helper::jeditable('data');
         $out = array();
-        if(!empty($data)){
+        if (!empty($data)) {
             $modObj = self::$classTable;
             $modObj->edit($data['id']);
-            if($modObj->getID() && ((is_object($callback) && ($callback instanceof \Closure)) || is_callable($callback))){
+            if ($modObj->getID() && ((is_object($callback) && ($callback instanceof \Closure)) || is_callable($callback))) {
                 $out = call_user_func($callback, $data, $modObj);
             }
         }
@@ -92,15 +92,15 @@ 
     /**
      * @return array|mixed
      */
-    public static function saveValue(){
-        return self::_workValue(function($data, $modObj){
+    public static function saveValue() {
+        return self::_workValue(function($data, $modObj) {
             $out = array();
-            if(isset($_POST['value']) && is_scalar($_POST['value'])){
-                if($modObj->set($data['key'], $_POST['value'])->save()){
+            if (isset($_POST['value']) && is_scalar($_POST['value'])) {
+                if ($modObj->set($data['key'], $_POST['value'])->save()) {
                     $textMethod = $data['key'].'Text';
-                    if(method_exists($modObj, $textMethod)){
+                    if (method_exists($modObj, $textMethod)) {
                         $out['value'] = $modObj->$textMethod();
-                    }else{
+                    } else {
                         $out['value'] = $modObj->get($data['key']);
                     }
                 }
@@ -112,8 +112,8 @@ 
     /**
      * @return array|mixed
      */
-    public static function getValue(){
-        return self::_workValue(function($data, $modObj){
+    public static function getValue() {
+        return self::_workValue(function($data, $modObj) {
             return array(
                 'value' => $modObj->get($data['key'])
             );
@@ -123,32 +123,32 @@ 
     /**
      * @return array
      */
-    public static function deleted(){
+    public static function deleted() {
 		$data = array();
         $dataID = (int)Template::getParam('docId', $_GET);
-        if($dataID>0 && self::_checkObj($dataID)){
+        if ($dataID > 0 && self::_checkObj($dataID)) {
             $oldValue = self::_getValue('deleted_at', $dataID);
             $q = self::$modx->db->update(array(
                     'deleted_at' => empty($oldValue) ? date('Y-m-d H:i:s') : null
                 ), self::$modx->getFullTableName(self::TABLE()), "id = ".$dataID);
-            if($q){
+            if ($q) {
                 $data['log'] = $oldValue ? 'Запись с ID '.$dataID.' восстановлена' : 'Запись с ID '.$dataID.' удалена';
-            }else{
+            } else {
                 $data['log'] = $oldValue ? 'Не удалось восстановить запись с ID '.$dataID : 'Не удалось удалить запись с ID '.$dataID;
             }
-        }else{
+        } else {
             $data['log'] = '<span class="error">Ошибка</span>. Не удалось определить обновляему запись';
         }
         return $data;
     }
-    public static function lists(){
+    public static function lists() {
         self::$TPL = 'ajax/lists';
     }
 
     /**
      * @return null
      */
-    public static function getClassTable(){
+    public static function getClassTable() {
 		return self::$classTable;
 	}
 }
\ No newline at end of file

assets/lib/Formatter/HtmlFormatter.php

Braces +3 added lines, -2 removed lines

@@ -22,8 +22,9 @@
             '<span style="color:'.$tag.';">$1</span>',
             '<span style="font-style:italic;">$1</span>',
         );
-        if($decode)
-            $string = htmlentities($string);
+        if($decode) {

+                    $string = htmlentities($string);

+        }

         return '<pre>'.preg_replace($find, $replace, $string).'</pre>';
     }
 }
\ No newline at end of file

Spacing +9 added lines, -9 removed lines

@@ -4,24 +4,24 @@ 
  * Class HtmlFormatter
  * @package Formatter
  */
-class HtmlFormatter{
+class HtmlFormatter {
     /**
      * @param $string
      * @param bool $decode
      * @return string
      */
-    public static function format($string, $decode = TRUE){
+    public static function format($string, $decode = TRUE) {
         $tag = '#0000ff';
         $att = '#ff0000';
         $val = '#8000ff';
         $com = '#34803a';
         $find = array(
-            '~(\s[a-z].*?=)~',                    // Highlight the attributes
-            '~(<\!--.*?-->)~s',            // Hightlight comments
-            '~("[a-zA-Z0-9\/].*?")~',    // Highlight the values
-            '~(<[a-z].*?>)~',                // Highlight the beginning of the opening tag
-            '~(</[a-z].*?>)~',            // Highlight the closing tag
-            '~(&.*?;)~',                    // Stylize HTML entities
+            '~(\s[a-z].*?=)~', // Highlight the attributes
+            '~(<\!--.*?-->)~s', // Hightlight comments
+            '~("[a-zA-Z0-9\/].*?")~', // Highlight the values
+            '~(<[a-z].*?>)~', // Highlight the beginning of the opening tag
+            '~(</[a-z].*?>)~', // Highlight the closing tag
+            '~(&.*?;)~', // Stylize HTML entities
         );
         $replace = array(
             '<span style="color:'.$att.';">$1</span>',
@@ -31,7 +31,7 @@ 
             '<span style="color:'.$tag.';">$1</span>',
             '<span style="font-style:italic;">$1</span>',
         );
-        if($decode)
+        if ($decode)
             $string = htmlentities($string);
         return '<pre>'.preg_replace($find, $replace, $string).'</pre>';
     }

assets/lib/SimpleTab/table.abstract.php

Braces +21 added lines, -7 removed lines

@@ -42,7 +42,9 @@ 
     public function place($ids, $dir, $rid) {
         $table = $this->makeTable($this->table);
         $ids = $this->cleanIDs($ids, ',', array(0));
-        if(empty($ids) || is_scalar($ids)) return false;
+        if(empty($ids) || is_scalar($ids)) {

+            return false;

+        }

         $rows = $this->query("SELECT count(`{$this->pkName}`) FROM {$table} WHERE `{$this->rfName}`={$rid}");
         $index = $this->modx->db->getValue($rows);
         $cnt = count($ids);
@@ -67,15 +69,25 @@ 
      */
     public function deleteThumb($url, $cache = false) {
         $url = $this->fs->relativePath($url);
-        if (empty($url)) return;
-        if ($this->fs->checkFile($url)) unlink(MODX_BASE_PATH . $url);
+        if (empty($url)) {

+            return;

+        }

+        if ($this->fs->checkFile($url)) {

+            unlink(MODX_BASE_PATH . $url);

+        }

         $dir = $this->fs->takeFileDir($url);
         $iterator = new \FilesystemIterator($dir);
-        if (!$iterator->valid()) rmdir($dir);
-        if ($cache) return;
+        if (!$iterator->valid()) {

+            rmdir($dir);

+        }

+        if ($cache) {

+            return;

+        }

         $thumbsCache = isset($this->params['thumbsCache']) ? $this->params['thumbsCache'] : $this->thumbsCache;
         $thumb = $thumbsCache.$url;
-        if ($this->fs->checkFile($thumb)) $this->deleteThumb($thumb, true);
+        if ($this->fs->checkFile($thumb)) {

+            $this->deleteThumb($thumb, true);

+        }

     }
 
     public function delete($ids, $fire_events = NULL) {
@@ -141,7 +153,9 @@ 
      * @return bool
      */
     public function makeThumb($folder,$url,$options) {
-        if (empty($url)) return false;
+        if (empty($url)) {

+            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);

Indentation +3 added lines, -3 removed lines

@@ -6,9 +6,9 @@
 class dataTable extends \autoTable {
     protected $params = array();
     protected $fs = null;
-	protected $indexName = null;
-	protected $rfName = null;
-	protected $thumbsCache = null;
+    protected $indexName = null;
+    protected $rfName = null;
+    protected $thumbsCache = null;
 
     public function __construct($modx, $debug = false) {
         parent::__construct($modx, $debug);

Spacing +24 added lines, -24 removed lines

@@ -1,7 +1,7 @@ 
 <?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 extends \autoTable {
     protected $params = array();
@@ -30,7 +30,7 @@ 
         return $out;
     }
 
-    public function touch($field){
+    public function touch($field) {
         $this->set($field, date('Y-m-d H:i:s', time() + $this->modx->config['server_offset_time']));
         return $this;
     }
@@ -43,19 +43,19 @@ 
     public function place($ids, $dir, $rid) {
         $table = $this->makeTable($this->table);
         $ids = $this->cleanIDs($ids, ',', array(0));
-        if(empty($ids) || is_scalar($ids)) return false;
+        if (empty($ids) || is_scalar($ids)) return false;
         $rows = $this->query("SELECT count(`{$this->pkName}`) FROM {$table} WHERE `{$this->rfName}`={$rid}");
         $index = $this->modx->db->getValue($rows);
         $cnt = count($ids);
-        $ids = implode(',',$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();
@@ -69,7 +69,7 @@ 
     public function deleteThumb($url, $cache = false) {
         $url = $this->fs->relativePath($url);
         if (empty($url)) return;
-        if ($this->fs->checkFile($url)) unlink(MODX_BASE_PATH . $url);
+        if ($this->fs->checkFile($url)) unlink(MODX_BASE_PATH.$url);
         $dir = $this->fs->takeFileDir($url);
         $iterator = new \FilesystemIterator($dir);
         if (!$iterator->valid()) rmdir($dir);
@@ -85,12 +85,12 @@ 
         return $out;
     }
 
-    public function deleteAll ($ids, $rid, $fire_events = NULL) {
+    public function deleteAll($ids, $rid, $fire_events = NULL) {
         $this->clearIndexes($ids, $rid);
         return $this->delete($ids, $fire_events);
     }
 
-    public function fieldNames(){
+    public function fieldNames() {
         $fields = array_keys($this->getDefaultFields());
         $fields[] = $this->fieldPKName();
         return $fields;
@@ -118,19 +118,19 @@ 
         /* more refactoring  needed */
         if ($targetIndex < $sourceIndex) {
             if (($point == 'top' && $orderDir == 'asc') || ($point == 'bottom' && $orderDir == 'desc')) {
-                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`+1",$table,"`{$this->indexName}`>={$targetIndex} AND `{$this->indexName}`<{$sourceIndex} AND `{$this->rfName}`={$rid}");
-                $rows = $this->modx->db->update("`{$this->indexName}`={$targetIndex}",$table,"`{$this->pkName}`={$sourceId}");             
+                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`+1", $table, "`{$this->indexName}`>={$targetIndex} AND `{$this->indexName}`<{$sourceIndex} AND `{$this->rfName}`={$rid}");
+                $rows = $this->modx->db->update("`{$this->indexName}`={$targetIndex}", $table, "`{$this->pkName}`={$sourceId}");             
             } elseif (($point == 'bottom' && $orderDir == 'asc') || ($point == 'top' && $orderDir == 'desc')) {
-                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`+1",$table,"`{$this->indexName}`>{$targetIndex} AND `{$this->indexName}`<{$sourceIndex} AND `{$this->rfName}`={$rid}");
-                $rows = $this->modx->db->update("`{$this->indexName}`=1+{$targetIndex}",$table,"`{$this->pkName}`={$sourceId}");             
+                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`+1", $table, "`{$this->indexName}`>{$targetIndex} AND `{$this->indexName}`<{$sourceIndex} AND `{$this->rfName}`={$rid}");
+                $rows = $this->modx->db->update("`{$this->indexName}`=1+{$targetIndex}", $table, "`{$this->pkName}`={$sourceId}");             
             }
         } else {
             if (($point == 'bottom' && $orderDir == 'asc') || ($point == 'top' && $orderDir == 'desc')) {
-                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`-1",$table,"`{$this->indexName}`<={$targetIndex} AND `{$this->indexName}`>={$sourceIndex} AND `{$this->rfName}`={$rid}");
-                $rows = $this->modx->db->update("`{$this->indexName}`={$targetIndex}",$table,"`{$this->pkName}`={$sourceId}");             
+                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`-1", $table, "`{$this->indexName}`<={$targetIndex} AND `{$this->indexName}`>={$sourceIndex} AND `{$this->rfName}`={$rid}");
+                $rows = $this->modx->db->update("`{$this->indexName}`={$targetIndex}", $table, "`{$this->pkName}`={$sourceId}");             
             } elseif (($point == 'top' && $orderDir == 'asc') || ($point == 'bottom' && $orderDir == 'desc')) {
-                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`-1",$table,"`{$this->indexName}`<{$targetIndex} AND `{$this->indexName}`>={$sourceIndex} AND `{$this->rfName}`={$rid}");
-                $rows = $this->modx->db->update("`{$this->indexName}`=-1+{$targetIndex}",$table,"`{$this->pkName}`={$sourceId}");                
+                $this->modx->db->update("`{$this->indexName}`=`{$this->indexName}`-1", $table, "`{$this->indexName}`<{$targetIndex} AND `{$this->indexName}`>={$sourceIndex} AND `{$this->rfName}`={$rid}");
+                $rows = $this->modx->db->update("`{$this->indexName}`=-1+{$targetIndex}", $table, "`{$this->pkName}`={$sourceId}");                
             }
         }
         
@@ -143,17 +143,17 @@ 
      * @param $options
      * @return bool
      */
-    public function makeThumb($folder,$url,$options) {
+    public function makeThumb($folder, $url, $options) {
         if (empty($url)) 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)) {
+        if ($thumb->create($inputFile, $outputFile, $options)) {
             return true;
         } else {
-            $this->modx->logEvent(0, 3, $thumb->debugMessages,  __NAMESPACE__);
+            $this->modx->logEvent(0, 3, $thumb->debugMessages, __NAMESPACE__);
             return false;
         }
     }

assets/lib/SimpleTab/plugin.class.php

Braces +13 added lines, -6 removed lines

@@ -45,9 +45,10 @@ 
         }
         //overload plugin and class properties
         $_params = $modx->parseProperties('&template=;;'.$this->params['template'].' &id=;;'.$this->params['id'],$modx->event->activePlugin,'plugin');
-        foreach ($_params as $key=>$value) 
-            if (property_exists ($this, $key)) 
-                $this->$key = $value;
+        foreach ($_params as $key=>$value) {

+                    if (property_exists ($this, $key)) 
+                $this->$key = $value;

+        }

 
         $this->params = array_merge($this->params,$_params);
         $modx->event->_output = "";
@@ -57,7 +58,9 @@ 
     }
 
 	public function clearFolders($ids = array(), $folder) {
-        foreach ($ids as $id) $this->fs->rmDir($folder.$id.'/');
+        foreach ($ids as $id) {

+            $this->fs->rmDir($folder.$id.'/');

+        }

     }
 
     public function checkPermissions() {
@@ -124,7 +127,9 @@ 
 			$scripts = isset($scripts['scripts']) ? $scripts['scripts'] : $scripts['styles'];
 			foreach ($scripts as $name => $params) {
 				$script = $this->assets->registerScript($name,$params);
-                if ($script !== false) $js .= $script;
+                if ($script !== false) {

+                    $js .= $script;

+                }

 			}
 		} else {
 			if ($list == $this->jsListDefault) {
@@ -197,7 +202,9 @@ 
 			$result = $this->modx->db->select('`id`',$eventsTable,"`name` = '{$event}'");
 			if (!$this->modx->db->getRecordCount($result)) {
 				$sql = "INSERT INTO {$eventsTable} VALUES (NULL, '{$event}', '{$eventsType}', '{$this->pluginName} Events')";
-				if (!$this->modx->db->query($sql)) $this->modx->logEvent(0, 3, "Cannot register {$event} event.", $this->pluginName);
+				if (!$this->modx->db->query($sql)) {

+				    $this->modx->logEvent(0, 3, "Cannot register {$event} event.", $this->pluginName);

+				}

 			}
 		}
 	}

Spacing +32 added lines, -32 removed lines

@@ -1,9 +1,9 @@ 
 <?php
 namespace SimpleTab;
-include_once (MODX_BASE_PATH . 'assets/snippets/DocLister/lib/DLTemplate.class.php');
-include_once (MODX_BASE_PATH . 'assets/lib/APIHelpers.class.php');
-require_once (MODX_BASE_PATH . 'assets/lib/Helpers/FS.php');
-require_once (MODX_BASE_PATH . 'assets/lib/Helpers/Assets.php');
+include_once (MODX_BASE_PATH.'assets/snippets/DocLister/lib/DLTemplate.class.php');
+include_once (MODX_BASE_PATH.'assets/lib/APIHelpers.class.php');
+require_once (MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
+require_once (MODX_BASE_PATH.'assets/lib/Helpers/Assets.php');
 
 abstract class Plugin {
 	public $modx = null;
@@ -40,16 +40,16 @@ 
         $this->lang_attribute = $lang_attribute;
         $this->params = $modx->event->params;
         if ($this->checkTemplate && !isset($this->params['template']) && $modx->event->name != 'OnEmptyTrash') {
-			$doc = $modx->getDocument($this->params['id'],'template','all','all');
+			$doc = $modx->getDocument($this->params['id'], 'template', 'all', 'all');
             $this->params['template'] = is_array($doc) ? end($doc) : null;
         }
         //overload plugin and class properties
-        $_params = $modx->parseProperties('&template=;;'.$this->params['template'].' &id=;;'.$this->params['id'],$modx->event->activePlugin,'plugin');
+        $_params = $modx->parseProperties('&template=;;'.$this->params['template'].' &id=;;'.$this->params['id'], $modx->event->activePlugin, 'plugin');
         foreach ($_params as $key=>$value) 
-            if (property_exists ($this, $key)) 
+            if (property_exists($this, $key)) 
                 $this->$key = $value;
 
-        $this->params = array_merge($this->params,$_params);
+        $this->params = array_merge($this->params, $_params);
         $modx->event->_output = "";
         $this->DLTemplate = \DLTemplate::getInstance($this->modx);
         $this->fs = \Helpers\FS::getInstance();
@@ -61,18 +61,18 @@ 
     }
 
     public function checkPermissions() {
-        $templates = isset($this->params['templates']) ? explode(',',$this->params['templates']) : false;
-        $roles = isset($this->params['roles']) ? explode(',',$this->params['roles']) : false;
+        $templates = isset($this->params['templates']) ? explode(',', $this->params['templates']) : false;
+        $roles = isset($this->params['roles']) ? explode(',', $this->params['roles']) : false;
 
-        $tplFlag = ($this->checkTemplate && !$templates || ($templates && !in_array($this->params['template'],$templates)));
+        $tplFlag = ($this->checkTemplate && !$templates || ($templates && !in_array($this->params['template'], $templates)));
 
-        $documents = isset($this->params['documents']) ? explode(',',$this->params['documents']) : false;
+        $documents = isset($this->params['documents']) ? explode(',', $this->params['documents']) : false;
         $docFlag = ($this->checkId && $tplFlag) ? !($documents && in_array($this->params['id'], $documents)) : $tplFlag;
 
-        $ignoreDocs = isset($this->params['ignoreDoc']) ? explode(',',$this->params['ignoreDoc']) : false;
+        $ignoreDocs = isset($this->params['ignoreDoc']) ? explode(',', $this->params['ignoreDoc']) : false;
         $ignoreFlag = ($this->checkId && $ignoreDocs && in_array($this->params['id'], $ignoreDocs));
 
-        return ($docFlag || $ignoreFlag || ($roles && !in_array($_SESSION['mgrRole'],$roles)));
+        return ($docFlag || $ignoreFlag || ($roles && !in_array($_SESSION['mgrRole'], $roles)));
     }
 
     /**
@@ -89,18 +89,18 @@ 
         }
         $output = '';
 		$plugins = $this->modx->pluginEvent;
-		if(($this->renderEvent!=='OnDocFormRender' || (array_search('ManagerManager', $plugins['OnDocFormRender']) === false))) {
-			$jquery = $this->assets->registerScript('jQuery',array(
+		if (($this->renderEvent !== 'OnDocFormRender' || (array_search('ManagerManager', $plugins['OnDocFormRender']) === false))) {
+			$jquery = $this->assets->registerScript('jQuery', array(
                 'version' => '1.9.1',
                 'src'     => 'assets/js/jquery/jquery-1.9.1.min.js'
             ));
             if ($jquery !== false) {
                 $output .= $jquery;
-                $output .='<script type="text/javascript">var jQuery = jQuery.noConflict(true);</script>';    
+                $output .= '<script type="text/javascript">var jQuery = jQuery.noConflict(true);</script>';    
             }
 		}
 		$tpl = MODX_BASE_PATH.$this->tpl;
-		if($this->fs->checkFile($tpl)) {
+		if ($this->fs->checkFile($tpl)) {
 			$output .= '[+js+][+styles+]'.file_get_contents($tpl);
 		} else {
 			$this->modx->logEvent(0, 3, "Cannot load {$this->tpl} .", $this->pluginName);
@@ -114,16 +114,16 @@ 
      * @param array $ph
      * @return string
      */
-    public function renderJS($list,$ph = array()) {
+    public function renderJS($list, $ph = array()) {
     	$js = '';
     	$scripts = MODX_BASE_PATH.$list;
-		if($this->fs->checkFile($scripts)) {
+		if ($this->fs->checkFile($scripts)) {
 			$scripts = @file_get_contents($scripts);
-			$scripts = $this->DLTemplate->parseChunk('@CODE:'.$scripts,$ph);
-			$scripts = json_decode($scripts,true);
+			$scripts = $this->DLTemplate->parseChunk('@CODE:'.$scripts, $ph);
+			$scripts = json_decode($scripts, true);
 			$scripts = isset($scripts['scripts']) ? $scripts['scripts'] : $scripts['styles'];
 			foreach ($scripts as $name => $params) {
-				$script = $this->assets->registerScript($name,$params);
+				$script = $this->assets->registerScript($name, $params);
                 if ($script !== false) $js .= $script;
 			}
 		} else {
@@ -140,7 +140,7 @@ 
 	 * @return array
 	 */
 	public function getTplPlaceholders() {
-		$ph = array ();
+		$ph = array();
 		return $ph;
 	}
 
@@ -152,9 +152,9 @@ 
             $output = $this->prerender();
     		if ($output !== false) {
     			$ph = $this->getTplPlaceholders();
-    			$ph['js'] = $this->renderJS($this->jsListDefault,$ph) . $this->renderJS($this->jsListCustom,$ph);
-    			$ph['styles'] = $this->renderJS($this->cssListDefault,$ph) . $this->renderJS($this->cssListCustom,$ph);
-    			$output = $this->DLTemplate->parseChunk('@CODE:'.$output,$ph);
+    			$ph['js'] = $this->renderJS($this->jsListDefault, $ph).$this->renderJS($this->jsListCustom, $ph);
+    			$ph['styles'] = $this->renderJS($this->cssListDefault, $ph).$this->renderJS($this->cssListCustom, $ph);
+    			$output = $this->DLTemplate->parseChunk('@CODE:'.$output, $ph);
     		}
     		return $output;
         }
@@ -166,11 +166,11 @@ 
     public function renderEmpty() {
         if (!$this->checkPermissions()) {
             $tpl = MODX_BASE_PATH.$this->emptyTpl;
-            if($this->fs->checkFile($tpl)) {
+            if ($this->fs->checkFile($tpl)) {
                 $output = '[+js+]'.file_get_contents($tpl);
                 $ph = $this->getTplPlaceholders();
-                $ph['js'] = $this->renderJS($this->jsListEmpty,$ph);
-                $output = $this->DLTemplate->parseChunk('@CODE:'.$output,$ph);
+                $ph['js'] = $this->renderJS($this->jsListEmpty, $ph);
+                $output = $this->DLTemplate->parseChunk('@CODE:'.$output, $ph);
                 return $output;
             } else {
                 $this->modx->logEvent(0, 3, "Cannot load {$this->emptyTpl} .", $this->pluginName);
@@ -183,7 +183,7 @@ 
      */
     public function checkTable() {
         $sql = "SHOW TABLES LIKE '{$this->_table}'";
-        return $this->modx->db->getRecordCount( $this->modx->db->query($sql));
+        return $this->modx->db->getRecordCount($this->modx->db->query($sql));
     }
 
     public function createTable() {
@@ -194,7 +194,7 @@ 
 	public function registerEvents($events = array(), $eventsType = '6') {
 		$eventsTable = $this->modx->getFullTableName('system_eventnames');
 		foreach ($events as $event) {
-			$result = $this->modx->db->select('`id`',$eventsTable,"`name` = '{$event}'");
+			$result = $this->modx->db->select('`id`', $eventsTable, "`name` = '{$event}'");
 			if (!$this->modx->db->getRecordCount($result)) {
 				$sql = "INSERT INTO {$eventsTable} VALUES (NULL, '{$event}', '{$eventsType}', '{$this->pluginName} Events')";
 				if (!$this->modx->db->query($sql)) $this->modx->logEvent(0, 3, "Cannot register {$event} event.", $this->pluginName);

Indentation +74 added lines, -74 removed lines

@@ -10,31 +10,31 @@ 
  * @package SimpleTab
  */
 abstract class Plugin {
-	public $modx = null;
-	public $pluginName = '';
-	public $params = array();
+    public $modx = null;
+    public $pluginName = '';
+    public $params = array();
     public $table = '';
-	public $tpl = '';
-	public $jsListDefault = '';
-	public $jsListCustom = '';
-	public $cssListDefault = '';
-	public $cssListCustom = '';
-	public $pluginEvents = array();
+    public $tpl = '';
+    public $jsListDefault = '';
+    public $jsListCustom = '';
+    public $cssListDefault = '';
+    public $cssListCustom = '';
+    public $pluginEvents = array();
     public $_table = '';
-	protected $fs = null;
+    protected $fs = null;
     protected $assets = null;
-	protected $emptyTpl = null;
-	protected $jsListEmpty = '';
+    protected $emptyTpl = null;
+    protected $jsListEmpty = '';
 
-	public $DLTemplate = null;
-	public $lang_attribute = '';
+    public $DLTemplate = null;
+    public $lang_attribute = '';
 
-	protected $checkTemplate = true;
-	protected $renderEvent = 'OnDocFormRender';
+    protected $checkTemplate = true;
+    protected $renderEvent = 'OnDocFormRender';
 
-	protected $checkId = true;
+    protected $checkId = true;
 
-	/**
+    /**
      * @param $modx
      * @param string $lang_attribute
      */
@@ -44,7 +44,7 @@ 
         $this->lang_attribute = $lang_attribute;
         $this->params = $modx->event->params;
         if ($this->checkTemplate && !isset($this->params['template']) && $modx->event->name != 'OnEmptyTrash') {
-			$doc = $modx->getDocument($this->params['id'],'template','all','all');
+            $doc = $modx->getDocument($this->params['id'],'template','all','all');
             $this->params['template'] = is_array($doc) ? end($doc) : null;
         }
         //overload plugin and class properties
@@ -96,12 +96,12 @@ 
                 $this->modx->logEvent(0, 3, "Cannot create {$this->table} table.", $this->pluginName);
                 return;
             }
-			$this->registerEvents($this->pluginEvents);
+            $this->registerEvents($this->pluginEvents);
         }
         $output = '';
-		$plugins = $this->modx->pluginEvent;
-		if(($this->renderEvent!=='OnDocFormRender' || (array_search('ManagerManager', $plugins['OnDocFormRender']) === false))) {
-			$jquery = $this->assets->registerScript('jQuery',array(
+        $plugins = $this->modx->pluginEvent;
+        if(($this->renderEvent!=='OnDocFormRender' || (array_search('ManagerManager', $plugins['OnDocFormRender']) === false))) {
+            $jquery = $this->assets->registerScript('jQuery',array(
                 'version' => '1.9.1',
                 'src'     => 'assets/js/jquery/jquery-1.9.1.min.js'
             ));
@@ -109,14 +109,14 @@ 
                 $output .= $jquery;
                 $output .='<script type="text/javascript">var jQuery = jQuery.noConflict(true);</script>';    
             }
-		}
-		$tpl = MODX_BASE_PATH.$this->tpl;
-		if($this->fs->checkFile($tpl)) {
-			$output .= '[+js+][+styles+]'.file_get_contents($tpl);
-		} else {
-			$this->modx->logEvent(0, 3, "Cannot load {$this->tpl} .", $this->pluginName);
+        }
+        $tpl = MODX_BASE_PATH.$this->tpl;
+        if($this->fs->checkFile($tpl)) {
+            $output .= '[+js+][+styles+]'.file_get_contents($tpl);
+        } else {
+            $this->modx->logEvent(0, 3, "Cannot load {$this->tpl} .", $this->pluginName);
             return false;
-		}
+        }
         return $output;
     }
 
@@ -126,48 +126,48 @@ 
      * @return string
      */
     public function renderJS($list,$ph = array()) {
-    	$js = '';
-    	$scripts = MODX_BASE_PATH.$list;
-		if($this->fs->checkFile($scripts)) {
-			$scripts = @file_get_contents($scripts);
-			$scripts = $this->DLTemplate->parseChunk('@CODE:'.$scripts,$ph);
-			$scripts = json_decode($scripts,true);
-			$scripts = isset($scripts['scripts']) ? $scripts['scripts'] : $scripts['styles'];
-			foreach ($scripts as $name => $params) {
-				$script = $this->assets->registerScript($name,$params);
+        $js = '';
+        $scripts = MODX_BASE_PATH.$list;
+        if($this->fs->checkFile($scripts)) {
+            $scripts = @file_get_contents($scripts);
+            $scripts = $this->DLTemplate->parseChunk('@CODE:'.$scripts,$ph);
+            $scripts = json_decode($scripts,true);
+            $scripts = isset($scripts['scripts']) ? $scripts['scripts'] : $scripts['styles'];
+            foreach ($scripts as $name => $params) {
+                $script = $this->assets->registerScript($name,$params);
                 if ($script !== false) $js .= $script;
-			}
-		} else {
-			if ($list == $this->jsListDefault) {
-				$this->modx->logEvent(0, 3, "Cannot load {$this->jsListDefault} .", $this->pluginName);
-			} elseif ($list == $this->cssListDefault) {
-				$this->modx->logEvent(0, 3, "Cannot load {$this->cssListDefault} .", $this->pluginName);
-			}
-		}
-		return $js;
+            }
+        } else {
+            if ($list == $this->jsListDefault) {
+                $this->modx->logEvent(0, 3, "Cannot load {$this->jsListDefault} .", $this->pluginName);
+            } elseif ($list == $this->cssListDefault) {
+                $this->modx->logEvent(0, 3, "Cannot load {$this->cssListDefault} .", $this->pluginName);
+            }
+        }
+        return $js;
     }
 
-	/**
-	 * @return array
-	 */
-	public function getTplPlaceholders() {
-		$ph = array ();
-		return $ph;
-	}
+    /**
+     * @return array
+     */
+    public function getTplPlaceholders() {
+        $ph = array ();
+        return $ph;
+    }
 
     /**
      * @return string
      */
     public function render() {
-		if (!$this->checkPermissions()) {
+        if (!$this->checkPermissions()) {
             $output = $this->prerender();
-    		if ($output !== false) {
-    			$ph = $this->getTplPlaceholders();
-    			$ph['js'] = $this->renderJS($this->jsListDefault,$ph) . $this->renderJS($this->jsListCustom,$ph);
-    			$ph['styles'] = $this->renderJS($this->cssListDefault,$ph) . $this->renderJS($this->cssListCustom,$ph);
-    			$output = $this->DLTemplate->parseChunk('@CODE:'.$output,$ph);
-    		}
-    		return $output;
+            if ($output !== false) {
+                $ph = $this->getTplPlaceholders();
+                $ph['js'] = $this->renderJS($this->jsListDefault,$ph) . $this->renderJS($this->jsListCustom,$ph);
+                $ph['styles'] = $this->renderJS($this->cssListDefault,$ph) . $this->renderJS($this->cssListCustom,$ph);
+                $output = $this->DLTemplate->parseChunk('@CODE:'.$output,$ph);
+            }
+            return $output;
         }
     }
 
@@ -201,8 +201,8 @@ 
      * @return mixed
      */
     public function createTable() {
-    	$sql = '';
-    	return $this->modx->db->query($sql);
+        $sql = '';
+        return $this->modx->db->query($sql);
     }
 
     /**
@@ -210,13 +210,13 @@ 
      * @param string $eventsType
      */
     public function registerEvents($events = array(), $eventsType = '6') {
-		$eventsTable = $this->modx->getFullTableName('system_eventnames');
-		foreach ($events as $event) {
-			$result = $this->modx->db->select('`id`',$eventsTable,"`name` = '{$event}'");
-			if (!$this->modx->db->getRecordCount($result)) {
-				$sql = "INSERT INTO {$eventsTable} VALUES (NULL, '{$event}', '{$eventsType}', '{$this->pluginName} Events')";
-				if (!$this->modx->db->query($sql)) $this->modx->logEvent(0, 3, "Cannot register {$event} event.", $this->pluginName);
-			}
-		}
-	}
+        $eventsTable = $this->modx->getFullTableName('system_eventnames');
+        foreach ($events as $event) {
+            $result = $this->modx->db->select('`id`',$eventsTable,"`name` = '{$event}'");
+            if (!$this->modx->db->getRecordCount($result)) {
+                $sql = "INSERT INTO {$eventsTable} VALUES (NULL, '{$event}', '{$eventsType}', '{$this->pluginName} Events')";
+                if (!$this->modx->db->query($sql)) $this->modx->logEvent(0, 3, "Cannot register {$event} event.", $this->pluginName);
+            }
+        }
+    }
 }
\ No newline at end of file

assets/lib/SimpleTab/controller.abstract.php

Indentation +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     public $isExit = false;
     public $output = null;
     public $params = null;
-	public $fireEvents = true;
+    public $fireEvents = true;
 	
     public $dlParams = array(
         "controller"    =>  "onetable",

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 +8 added lines, -8 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');
 
 abstract class AbstractController {
     public $rfName = '';
@@ -30,15 +30,15 @@ 
 
     protected $modx = null;
 
-    public function __construct(\DocumentParser $modx){
+    public function __construct(\DocumentParser $modx) {
         $this->FS = \Helpers\FS::getInstance();
         $this->modx = $modx;
         $this->params = $modx->event->params;
         $this->rid = isset($_REQUEST[$this->rfName]) ? (int)$_REQUEST[$this->rfName] : 0;
     }
 
-    public function callExit(){
-        if($this->isExit){
+    public function callExit() {
+        if ($this->isExit) {
             echo $this->output;
             exit;
         }
@@ -104,12 +104,12 @@ 
         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);
+        $offset = $this->dlParams['display'] * abs($offset - 1);
         $this->dlParams['offset'] = $offset;
-        if(isset($_REQUEST['sort'])){
+        if (isset($_REQUEST['sort'])) {
             $this->dlParams['sortBy'] = preg_replace('/[^A-Za-z0-9_\-]/', '', $_REQUEST['sort']);
         }
-        if(isset($_REQUEST['order']) && in_array(strtoupper($_REQUEST['order']), array("ASC","DESC"))){
+        if (isset($_REQUEST['order']) && in_array(strtoupper($_REQUEST['order']), array("ASC", "DESC"))) {
             $this->dlParams['sortDir'] = $_REQUEST['order'];
         }
         foreach ($this->dlParams as &$param) {
@@ -119,7 +119,7 @@ 
 
     public function getLanguageCode() {
         $manager_language = $this->modx->config['manager_language'];
-        if(file_exists(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 $modx_lang_attribute;

assets/lib/Helpers/PHPThumb.php

Indentation +5 added lines, -5 removed lines

@@ -7,17 +7,17 @@ 
 
     private $thumb = null;
     protected $fs = null;
-	public $debugMessages = '';
+    public $debugMessages = '';
 
     public function __construct()
     {
-       	$this->thumb = new \phpthumb();
-		$this->fs = FS::getInstance();
+            $this->thumb = new \phpthumb();
+        $this->fs = FS::getInstance();
     }
 
     public function create($inputFile, $outputFile, $options) {
         $this->thumb->sourceFilename = $inputFile;
-		$ext = explode('.',$inputFile);
+        $ext = explode('.',$inputFile);
         $ext = str_replace('jpeg','jpg',strtolower(array_pop($ext)));
         $options = 'f='.$ext.'&'.$options;
         $this->setOptions($options);
@@ -32,7 +32,7 @@ 
     public function optimize($file, $type = 'jpg') {
         switch ($type) {
             case 'jpg':
-				$ext = $this->fs->takeFileExt($file);
+                $ext = $this->fs->takeFileExt($file);
                 if ($ext == 'jpeg' || $ext == 'jpg') {
                     $cmd = '/usr/bin/jpegtran -optimize -progressive -copy none -outfile '.escapeshellarg($file.'_').' '.escapeshellarg($file);
                     exec($cmd, $result, $return_var);

Spacing +4 added lines, -4 removed lines

@@ -1,9 +1,9 @@ 
 <?php namespace Helpers;
 
 include_once(MODX_BASE_PATH.'assets/snippets/phpthumb/phpthumb.class.php');
-require_once(MODX_BASE_PATH . 'assets/lib/Helpers/FS.php');
+require_once(MODX_BASE_PATH.'assets/lib/Helpers/FS.php');
 
-class PHPThumb{
+class PHPThumb {
 
     private $thumb = null;
     protected $fs = null;
@@ -17,8 +17,8 @@ 
 
     public function create($inputFile, $outputFile, $options) {
         $this->thumb->sourceFilename = $inputFile;
-		$ext = explode('.',$inputFile);
-        $ext = str_replace('jpeg','jpg',strtolower(array_pop($ext)));
+		$ext = explode('.', $inputFile);
+        $ext = str_replace('jpeg', 'jpg', strtolower(array_pop($ext)));
         $options = 'f='.$ext.'&'.$options;
         $this->setOptions($options);
         if ($this->thumb->GenerateThumbnail() && $this->thumb->RenderToFile($outputFile)) {

Doc Comments +3 added lines, -3 removed lines

@@ -23,8 +23,8 @@ 
     }
 
     /**
-     * @param $inputFile
-     * @param $outputFile
+     * @param string $inputFile
+     * @param string $outputFile
      * @param $options
      * @return bool
      */
@@ -66,7 +66,7 @@ 
     }
 
     /**
-     * @param $options
+     * @param string $options
      */
     private function setOptions($options) {
         $options = strtr($options, Array("," => "&", "_" => "=", '{' => '[', '}' => ']'));