injitools / cms-Inji

system/Inji/ComposerCmd.php

Doc Comments +4 added lines

@@ -58,6 +58,10 @@
         self::command('install', false, $path);
     }
 
+    /**
+     * @param string $command
+     * @param string $path
+     */
     static function command($command, $needOutput = true, $path = null)
     {
         include_once 'composer/vendor/autoload.php';

system/Inji/Config.php

Doc Comments -2 removed lines

@@ -53,7 +53,6 @@ 
     /**
      * Load app config
      * 
-     * @param string $site_name
      * @return array
      */
     public static function app($app = false)
@@ -74,7 +73,6 @@ 
     /**
      * Load share config
      * 
-     * @param string $site_name
      * @return array
      */
     public static function share($module = '')

Braces +24 added lines, -16 removed lines

@@ -24,11 +24,13 @@ 
      */
     public static function system()
     {
-        if (isset(self::$_configs['system']))
-            return self::$_configs['system'];
+        if (isset(self::$_configs['system'])) {
+                    return self::$_configs['system'];
+        }
 
-        if (!file_exists(INJI_SYSTEM_DIR . '/config/config.php'))
-            return [];
+        if (!file_exists(INJI_SYSTEM_DIR . '/config/config.php')) {
+                    return [];
+        }
 
         return self::$_configs['system'] = include INJI_SYSTEM_DIR . '/config/config.php';
     }
@@ -41,11 +43,13 @@ 
      */
     public static function custom($path)
     {
-        if (isset(self::$_configs['custom'][$path]))
-            return self::$_configs['custom'][$path];
+        if (isset(self::$_configs['custom'][$path])) {
+                    return self::$_configs['custom'][$path];
+        }
 
-        if (!file_exists($path))
-            return [];
+        if (!file_exists($path)) {
+                    return [];
+        }
 
         return self::$_configs['custom'][$path] = include $path;
     }
@@ -61,12 +65,14 @@ 
         if (!$app) {
             $app = App::$primary;
         }
-        if (isset(self::$_configs['app'][$app->name]))
-            return self::$_configs['app'][$app->name];
+        if (isset(self::$_configs['app'][$app->name])) {
+                    return self::$_configs['app'][$app->name];
+        }
 
         $path = $app->path . "/config/config.php";
-        if (!file_exists($path))
-            return array();
+        if (!file_exists($path)) {
+                    return array();
+        }
 
         return self::$_configs['app'][$app->name] = include $path;
     }
@@ -80,13 +86,15 @@ 
     public static function share($module = '')
     {
         if ($module) {
-            if (isset($_configs['shareModules'][$module]))
-                return self::$_configs['shareModules'][$module];
+            if (isset($_configs['shareModules'][$module])) {
+                            return self::$_configs['shareModules'][$module];
+            }
 
             $path = INJI_PROGRAM_DIR . "/config/modules/{$module}.php";
         } else {
-            if (isset($_configs['share']))
-                return self::$_configs['share'];
+            if (isset($_configs['share'])) {
+                            return self::$_configs['share'];
+            }
 
             $path = INJI_PROGRAM_DIR . "/config/config.php";
         }

system/Inji/Inji.php

Doc Comments +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
      * 
      * @param string $eventName
      * @param string $listenCode
-     * @param array|function $callback
+     * @param Closure $callback
      * @param boolean $save
      */
     function listen($eventName, $listenCode, $callback, $save = false)

system/Inji/Log.php

Doc Comments +3 added lines

@@ -90,6 +90,9 @@
         echo '<tr><th>Memory</th><th>' . $this->convertSize(memory_get_peak_usage()) . ' of ' . ini_get('memory_limit') . '</th></tr></table></div>';
     }
 
+    /**
+     * @param integer $size
+     */
     function convertSize($size)
     {
 

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
                 echo "<tr><td>{$log['name']}</td><td" . (round(($log['end'] - $log['start']), 5) > 0.1 ? ' class ="danger"' : '') . ">" . round(($log['end'] - $log['start']), 5) . "</td></tr>";
             }
         }
-        echo '<tr><th>Summary</th><th>' . round(( microtime(true) - $this->startTime), 5) . '</th></tr>';
+        echo '<tr><th>Summary</th><th>' . round((microtime(true) - $this->startTime), 5) . '</th></tr>';
         echo '<tr><th>Memory</th><th>' . $this->convertSize(memory_get_peak_usage()) . ' of ' . ini_get('memory_limit') . '</th></tr></table></div>';
     }
 

Braces +13 added lines, -10 removed lines

@@ -79,10 +79,12 @@ 
             if (!empty($log['status'])) {
                 echo "<tr class = '{$log['status']}'><td>{$log['name']}</td><td>{$log['status']}</td></tr>";
             } else {
-                if (empty($log['end']))
-                    $log['end'] = microtime(true);
-                if (empty($log['start']))
-                    $log['start'] = microtime(true);
+                if (empty($log['end'])) {
+                                    $log['end'] = microtime(true);
+                }
+                if (empty($log['start'])) {
+                                    $log['start'] = microtime(true);
+                }
                 echo "<tr><td>{$log['name']}</td><td" . (round(($log['end'] - $log['start']), 5) > 0.1 ? ' class ="danger"' : '') . ">" . round(($log['end'] - $log['start']), 5) . "</td></tr>";
             }
         }
@@ -93,12 +95,13 @@ 
     function convertSize($size)
     {
 
-        if ($size < 1024)
-            return $size . "B";
-        elseif ($size < 1048576)
-            return round($size / 1024, 2) . "KB";
-        else
-            return round($size / 1048576, 2) . "MB";
+        if ($size < 1024) {
+                    return $size . "B";
+        } elseif ($size < 1048576) {
+                    return round($size / 1024, 2) . "KB";
+        } else {
+                    return round($size / 1048576, 2) . "MB";
+        }
     }
 
     function __destruct()

system/Inji/Model.php

Doc Comments +8 added lines, -5 removed lines

@@ -312,7 +312,7 @@ 
      * Check model relations path and load need relations
      * 
      * @param string $col
-     * @param strig $rootModel
+     * @param string $rootModel
      */
     static function checkForJoin(&$col, $rootModel)
     {
@@ -356,7 +356,7 @@ 
     /**
      * Information extractor for col relations path
      * 
-     * @param string|array $info
+     * @param string $info
      * @return array
      */
     private static function parseColRecursion($info)
@@ -428,6 +428,9 @@ 
         return Model::$cols[static::table()];
     }
 
+    /**
+     * @param string $colName
+     */
     static function genColParams($colName)
     {
         if (empty(static::$cols[$colName]) || static::$storage['type'] == 'moduleConfig') {
@@ -481,7 +484,7 @@ 
      * Create new col in data base
      * 
      * @param string $colName
-     * @return boolean
+     * @return false|null
      */
     static function createCol($colName)
     {
@@ -566,7 +569,7 @@ 
     /**
      * return relations list
      * 
-     * @return array
+     * @return string
      */
     static function relations()
     {
@@ -1062,7 +1065,7 @@ 
      * 
      * @param array $params
      * @param array $where
-     * @return boolean
+     * @return false|null
      */
     static function update($params, $where = [])
     {

Spacing +1 added lines, -1 removed lines

@@ -1035,7 +1035,7 @@
         } else {
             $cols .= '*';
         }
-        $cols .=') as `count`' . (!empty($options['cols']) ? ',' . $options['cols'] : '');
+        $cols .= ') as `count`' . (!empty($options['cols']) ? ',' . $options['cols'] : '');
         App::$cur->db->cols = $cols;
         if (!empty($options['group'])) {
             App::$cur->db->group($options['group']);

Braces +48 added lines, -36 removed lines

@@ -680,17 +680,21 @@ 
     {
 
         $return = array();
-        if (!empty($options['where']))
-            App::$cur->db->where($options['where']);
+        if (!empty($options['where'])) {
+                    App::$cur->db->where($options['where']);
+        }
         if (!empty($options['group'])) {
             App::$cur->db->group($options['group']);
         }
-        if (!empty($options['order']))
-            App::$cur->db->order($options['order']);
-        if (!empty($options['join']))
-            App::$cur->db->join($options['join']);
-        if (!empty($options['distinct']))
-            App::$cur->db->distinct = $options['distinct'];
+        if (!empty($options['order'])) {
+                    App::$cur->db->order($options['order']);
+        }
+        if (!empty($options['join'])) {
+                    App::$cur->db->join($options['join']);
+        }
+        if (!empty($options['distinct'])) {
+                    App::$cur->db->distinct = $options['distinct'];
+        }
 
         foreach (static::$relJoins as $join) {
             App::$cur->db->join($join[0], $join[1]);
@@ -716,14 +720,14 @@ 
         }
         static::$needJoin = [];
 
-        if (!empty($options['limit']))
-            $limit = (int) $options['limit'];
-        else {
+        if (!empty($options['limit'])) {
+                    $limit = (int) $options['limit'];
+        } else {
             $limit = 0;
         }
-        if (!empty($options['start']))
-            $start = (int) $options['start'];
-        else {
+        if (!empty($options['start'])) {
+                    $start = (int) $options['start'];
+        } else {
             $start = 0;
         }
         if ($limit || $start) {
@@ -980,21 +984,23 @@ 
             static::fixPrefix($options['where'], 'first');
         }
         $return = array();
-        if (!empty($options['where']))
-            App::$cur->db->where($options['where']);
-        if (!empty($options['join']))
-            App::$cur->db->join($options['join']);
+        if (!empty($options['where'])) {
+                    App::$cur->db->where($options['where']);
+        }
+        if (!empty($options['join'])) {
+                    App::$cur->db->join($options['join']);
+        }
         if (!empty($options['order'])) {
             App::$cur->db->order($options['order']);
         }
-        if (!empty($options['limit']))
-            $limit = (int) $options['limit'];
-        else {
+        if (!empty($options['limit'])) {
+                    $limit = (int) $options['limit'];
+        } else {
             $limit = 0;
         }
-        if (!empty($options['start']))
-            $start = (int) $options['start'];
-        else {
+        if (!empty($options['start'])) {
+                    $start = (int) $options['start'];
+        } else {
             $start = 0;
         }
         if ($limit || $start) {
@@ -1072,11 +1078,13 @@ 
 
         $values = array();
         foreach ($cols as $col => $param) {
-            if (isset($params[$col]))
-                $values[$col] = $params[$col];
+            if (isset($params[$col])) {
+                            $values[$col] = $params[$col];
+            }
+        }
+        if (!$values) {
+                    return false;
         }
-        if (!$values)
-            return false;
 
 
         if ($where) {
@@ -1257,11 +1265,13 @@ 
         $values = array();
 
         foreach ($this->cols() as $col => $param) {
-            if (isset($this->_params[$col]))
-                $values[$col] = $this->_params[$col];
+            if (isset($this->_params[$col])) {
+                            $values[$col] = $this->_params[$col];
+            }
+        }
+        if (!$values && empty($options['empty'])) {
+                    return false;
         }
-        if (!$values && empty($options['empty']))
-            return false;
 
         if ($this->pk()) {
             $new = false;
@@ -1417,8 +1427,9 @@ 
     {
 
         foreach (static::relations() as $relName => $rel) {
-            if ($rel['col'] == $col)
-                return $relName;
+            if ($rel['col'] == $col) {
+                            return $relName;
+            }
         }
         return NULL;
     }
@@ -1574,8 +1585,9 @@ 
             App::$cur->db->where($relation['relTablePrefix'] . $this->index(), $this->pk());
             App::$cur->db->where($relation['relTablePrefix'] . $relation['model']::index(), $objectId);
             $isset = App::$cur->db->select($relation['relTable'])->fetch_assoc();
-            if ($isset)
-                return true;
+            if ($isset) {
+                            return true;
+            }
 
             App::$cur->db->insert($relation['relTable'], [
                 $relation['relTablePrefix'] . $this->index() => $this->{$this->index()},

system/Inji/Module.php

Doc Comments +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
      * Get all posible directorys for module files
      * 
      * @param string $moduleName
-     * @return array
+     * @return string
      */
     static function getModulePaths($moduleName)
     {

system/Inji/Router.php

Doc Comments +3 added lines, -3 removed lines

@@ -82,9 +82,9 @@
     /**
      * Return posible paths for class path
      * 
-     * @param type $code
-     * @param type $folder
-     * @param type $classPath
+     * @param string $code
+     * @param string $folder
+     * @param string $classPath
      * @return string
      */
     static function genFolderPaths($code, $folder, $classPath)

system/Inji/Tools.php

Doc Comments +1 added lines

@@ -216,6 +216,7 @@
      * @param  $number Integer Число на основе которого нужно сформировать окончание
      * @param  $endingsArray  Array Массив слов или окончаний для чисел (1, 4, 5),
      *         например array('яблоко', 'яблока', 'яблок')
+     * @param string[] $endingArray
      * @return String
      */
     static function getNumEnding($number, $endingArray)

Spacing +5 added lines, -5 removed lines

@@ -38,7 +38,7 @@ 
     static function uriParse($uri)
     {
         $answerPos = strpos($uri, '?');
-        $params = array_slice(explode('/', substr($uri, 0, $answerPos ? $answerPos : strlen($uri) )), 1);
+        $params = array_slice(explode('/', substr($uri, 0, $answerPos ? $answerPos : strlen($uri))), 1);
 
         foreach ($params as $key => $param) {
             if ($param != '') {
@@ -85,7 +85,7 @@ 
     static function resizeImage($img_path, $max_width = 1000, $max_height = 1000, $crop = false, $pos = 'center')
     {
         ini_set("gd.jpeg_ignore_warning", 1);
-        list( $img_width, $img_height, $img_type, $img_tag ) = getimagesize($img_path);
+        list($img_width, $img_height, $img_type, $img_tag) = getimagesize($img_path);
         switch ($img_type) {
             case 1:
                 $img_type = 'gif';
@@ -108,14 +108,14 @@ 
 
         if ($crop === true || $crop == 'q') {
             if ($img_width > $img_height) {
-                $imgX = floor(( $img_width - $img_height ) / 2);
+                $imgX = floor(($img_width - $img_height) / 2);
                 $imgY = 0;
                 $img_width = $img_height;
                 $new_width = $max_width;
                 $new_height = $max_height;
             } else {
                 $imgX = 0;
-                $imgY = floor(( $img_height - $img_width ) / 2);
+                $imgY = floor(($img_height - $img_width) / 2);
                 $img_height = $img_width;
                 $new_width = $max_width;
                 $new_height = $max_height;
@@ -136,7 +136,7 @@ 
             $new_width = $max_width;
             $new_height = $max_height;
 //Находим начальные координаты (центрируем новое изображение)
-            $imgX = (int) (($ow / 2) - ($img_width / 2) );
+            $imgX = (int) (($ow / 2) - ($img_width / 2));
             if ($pos == 'center') {
                 $imgY = (int) (($oh / 2) - ($img_height / 2));
             } else {

Braces +8 added lines, -6 removed lines

@@ -58,8 +58,9 @@ 
      */
     static function createDir($path)
     {
-        if (file_exists($path))
-            return true;
+        if (file_exists($path)) {
+                    return true;
+        }
 
         $path = explode('/', $path);
         $cur = '';
@@ -101,10 +102,11 @@ 
         $imagecreatefromX = "imagecreatefrom{$img_type}";
         $src_res = $imagecreatefromX($img_path);
 
-        if ($img_width / $max_width > $img_height / $max_height)
-            $separator = $img_width / $max_width;
-        else
-            $separator = $img_height / $max_height;
+        if ($img_width / $max_width > $img_height / $max_height) {
+                    $separator = $img_width / $max_width;
+        } else {
+                    $separator = $img_height / $max_height;
+        }
 
         if ($crop === true || $crop == 'q') {
             if ($img_width > $img_height) {

system/Inji/Value.php

Doc Comments +3 added lines

@@ -14,6 +14,9 @@
     public $model = null;
     public $type = 'string';
 
+    /**
+     * @param Model $model
+     */
     function __construct($model, $key)
     {
         $this->model = $model;

Indentation +2 added lines, -2 removed lines

@@ -64,8 +64,8 @@
                 ?>
                 <div class="filter_form_field filter_select">
                   <?php
-                  $form->input('select', "datamanagerFilters[{$col}][value]", $colInfo['label'], ['value' => $value, 'values' => $values, 'noContainer' => true])
-                  ?>
+                    $form->input('select', "datamanagerFilters[{$col}][value]", $colInfo['label'], ['value' => $value, 'values' => $values, 'noContainer' => true])
+                    ?>
                 </div>
                 <?php
                 break;