ifacesoft / ice

Source/Ice/Core/DataSource.php

Doc Comments +1 added lines, -1 removed lines

@@ -276,7 +276,7 @@
      * Execute query
      *
      * @param Query $query
-     * @param $ttl
+     * @param integer|null $ttl
      * @return QueryResult
      * @throws \Exception
      * @author dp <denis.a.shestakov@gmail.com>

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 namespace Ice\Core;
 
 use Ice\Core;
-use Ice\Helper\String;
 
 /**
  * Class DataSource

Source/Ice/Core/Environment.php

Doc Comments +2 added lines, -2 removed lines

@@ -135,7 +135,7 @@ 
      * Retern Data Provider by class name
      *
      * @param  string $class Class (found data provider for this class)
-     * @param $key
+     * @param string|null $key
      * @param  string $index Index of data provider
      * @return DataProvider
      * @author dp <denis.a.shestakov@gmail.com>
@@ -151,7 +151,7 @@ 
     /**
      * Return data provider key by class name
      *
-     * @param $class
+     * @param string $class
      * @param $key
      * @return string
      * @author dp <denis.a.shestakov@gmail.com>

Source/Ice/Core/Logger.php

Doc Comments +11 added lines, -4 removed lines

@@ -215,6 +215,9 @@ 
         ])->save(true);
     }
 
+    /**
+     * @param string $type
+     */
     private function getFbType($type)
     {
         switch ($type) {
@@ -501,8 +504,8 @@ 
      * Warning
      *
      * @param  $message
-     * @param  $file
-     * @param  $line
+     * @param  string $file
+     * @param  integer $line
      * @param  \Exception $e
      * @param  null $errcontext
      * @param  int $errno
@@ -522,8 +525,8 @@ 
      * Fatal - throw ice exception
      *
      * @param  $message
-     * @param  $file
-     * @param  $line
+     * @param  string $file
+     * @param  integer $line
      * @param  \Exception $e
      * @param  null $errcontext
      * @param  int $errno
@@ -548,6 +551,10 @@ 
         throw $this->createException($message, $file, $line, $e, $errcontext, $errno, $exceptionClass);
     }
 
+    /**
+     * @param string $value
+     * @param string $label
+     */
     public static function log($value, $label = null, $type = 'LOG', $options = [])
     {
         $value = str_replace(["\n", "\t"], ' ', $value);

Source/Ice/Core/Model.php

Doc Comments +34 added lines, -6 removed lines

@@ -283,6 +283,10 @@ 
         return $this;
     }
 
+    /**
+     * @param string $model
+     * @param boolean $isAffected
+     */
     private function setModelValue($model, $isAffected)
     {
         if (!($model instanceof Model)) {
@@ -338,7 +342,7 @@ 
     /**
      * Gets full model field name if send short name (for example: '/name' for model User -> user_name)
      *
-     * @param  $fieldName
+     * @param  string|null $fieldName
      * @return string
      *
      * @author dp <denis.a.shestakov@gmail.com>
@@ -361,6 +365,10 @@ 
         return $fieldName;
     }
 
+    /**
+     * @param string $fieldName
+     * @param boolean $isAffected
+     */
     private function setPkValue($fieldName, $fieldValue, $isAffected)
     {
         if (!$this->isPkName($fieldName)) {
@@ -473,6 +481,10 @@ 
         return array_key_exists($fieldName, $this->row);
     }
 
+    /**
+     * @param string $fieldName
+     * @param boolean $isAffected
+     */
     private function setValue($fieldName, $fieldValue, $isAffected)
     {
         if (!$this->isFieldName($fieldName)) {
@@ -487,6 +499,10 @@ 
         return [$fieldName => $fieldValue];
     }
 
+    /**
+     * @param string $fieldName
+     * @param boolean $isAffected
+     */
     private function setJsonValue($fieldName, $fieldValue, $isAffected)
     {
         $jsonFieldName = $fieldName . '__json';
@@ -692,6 +708,10 @@ 
         return new $modelClass($row);
     }
 
+    /**
+     * @param string $fieldName
+     * @param boolean $isAffected
+     */
     private function setSpatialValue($fieldName, $fieldValue, $isAffected)
     {
         $geoFieldName = $fieldName . '__geo';
@@ -717,6 +737,10 @@ 
         return [$geoFieldName => $fieldValue];
     }
 
+    /**
+     * @param string $fieldName
+     * @param boolean $isAffected
+     */
     private function setFkValue($fieldName, $fieldValue, $isAffected)
     {
         $fkFieldName = $fieldName . '__fk';
@@ -1167,7 +1191,7 @@ 
 
     /**
      * @param Model $modelClass
-     * @param $dataSourceKey
+     * @param string|null $dataSourceKey
      */
     private function update($modelClass, $dataSourceKey)
     {
@@ -1207,8 +1231,8 @@ 
 
     /**
      * @param Model $modelClass
-     * @param $isSmart
-     * @param $dataSourceKey
+     * @param boolean $isSmart
+     * @param string|null $dataSourceKey
      */
     private function insert($modelClass, $isSmart, $dataSourceKey)
     {
@@ -1277,7 +1301,7 @@ 
     /**
      * Execute select from data source
      *
-     * @param  $fieldNames
+     * @param  string $fieldNames
      * @param  string|null $dataSourceKey
      * @param  int $ttl
      * @return Model|null
@@ -1318,7 +1342,7 @@ 
      * Execute delete for model
      *
      * @param  string|null $dataSourceKey
-     * @return Model|null
+     * @return Model
      *
      * @author dp <denis.a.shestakov@gmail.com>
      *
@@ -1461,6 +1485,10 @@ 
             : Query::getBuilder($selfModelClass)->pk($this->getPk());
     }
 
+    /**
+     * @param Model $modelClass
+     * @param string $fieldNames
+     */
     public function fetchOne($modelClass, $fieldNames, $lazy = false)
     {
         $modelClass = Model::getClass($modelClass);

Unused Use Statements -4 removed lines

@@ -9,15 +9,11 @@
 
 namespace Ice\Core;
 
-use Assetic\Test\Filter\PackerFilterTest;
 use Ice\Core;
-use Ice\Helper\Arrays;
 use Ice\Helper\Json;
 use Ice\Helper\Model as Helper_Model;
 use Ice\Helper\Object;
 use Ice\Helper\Spatial;
-use Ice\Widget\Model_Form;
-use Ice\Widget\Model_Table;
 
 /**
  * Class Model

Source/Ice/Core/Model/Collection.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     /**
      * Private constructor for model collection
      *
-     * @param $modelClass
+     * @param Model $modelClass
      * @param array $rows
      *
      * @author dp <denis.a.shestakov@gmail.com>

Source/Ice/Core/Module.php

Doc Comments +2 added lines, -2 removed lines

@@ -78,7 +78,7 @@ 
      * Check table belongs to module
      *
      * @param  $tableName
-     * @param  $dataSourceKey
+     * @param  string $dataSourceKey
      * @return bool
      * @author dp <denis.a.shestakov@gmail.com>
      *
@@ -231,7 +231,7 @@ 
     /**
      * Load module configs
      *
-     * @param $vendor
+     * @param string $vendor
      * @param string $context
      * @param array $modules
      *

Unused Use Statements -1 removed lines

@@ -10,7 +10,6 @@
 namespace Ice\Core;
 
 use Ice\Core;
-use Ice\Exception\ModuleNotFound;
 use Ice\Helper\Directory;
 use Ice\Helper\File;
 use Ice\Helper\String;

Source/Ice/Core/Query.php

Doc Comments +4 added lines, -4 removed lines

@@ -78,7 +78,7 @@ 
      * Create new instance of query
      *
      * @param  QueryBuilder $queryBuilder
-     * @param  $dataSourceKey
+     * @param  string|null $dataSourceKey
      * @return Query
      *
      * @author dp <denis.a.shestakov@gmail.com>
@@ -403,7 +403,7 @@ 
      * @desc Результат запроса - единственная запись таблицы.
      *
      * @param  null $pk
-     * @param  null $ttl
+     * @param  integer|null $ttl
      * @return array|null
      * @author dp <denis.a.shestakov@gmail.com>
      *
@@ -443,8 +443,8 @@ 
     /**
      * Return model from data
      *
-     * @param  null $pk
-     * @param  null $ttl
+     * @param  null|string $pk
+     * @param  integer|null $ttl
      * @return Model|null
      * @author dp <denis.a.shestakov@gmail.com>
      *

Source/Ice/Core/QueryBuilder.php

Doc Comments +18 added lines, -14 removed lines

@@ -245,7 +245,7 @@ 
      * @param  array $fieldNameValues
      * @param  $sqlComparison
      * @param  array|string $modelTableData Key -> modelClass, value -> tableAlias
-     * @param $isUse
+     * @param boolean $isUse
      * @return QueryBuilder
      * @author dp <denis.a.shestakov@gmail.com>
      *
@@ -364,8 +364,8 @@ 
      *
      * @param Model $modelClass
      * @param $fieldNames
-     * @param $isValidate
-     * @param $isInvalidate boolean
+     * @param boolean $isValidate
+     * @param boolean $isInvalidate boolean
      *
      * @author dp <denis.a.shestakov@gmail.com>
      *
@@ -391,7 +391,7 @@ 
     /**
      * Set in query part where expression 'IS NULL'
      *
-     * @param  $fieldName
+     * @param  string $fieldName
      * @param  array $modelTableData
      * @param  string $sqlLogical
      * @param bool $isUse
@@ -592,8 +592,8 @@ 
     /**
      * Set in query part where expression '> ?'
      *
-     * @param  $fieldName
-     * @param  $fieldValue
+     * @param  string $fieldName
+     * @param  string $fieldValue
      * @param  array $modelTableData
      * @param  string $sqlLogical
      * @param bool $isUse
@@ -680,7 +680,7 @@ 
      *
      * @param  $fieldName
      * @param  $fieldValue
-     * @param  array $modelTableData
+     * @param  string $modelTableData
      * @param  string $sqlLogical
      * @param bool $isUse
      * @return QueryBuilder
@@ -728,7 +728,7 @@ 
     /**
      * Set in query part where expression '== 1' is boolean true(1)
      *
-     * @param  $fieldName
+     * @param  string $fieldName
      * @param  array $modelTableData
      * @param  string $sqlLogical
      * @return QueryBuilder
@@ -764,8 +764,8 @@ 
     /**
      * Set in query part where expression 'like ?'
      *
-     * @param  $fieldName
-     * @param  $fieldValue
+     * @param  string $fieldName
+     * @param  string $fieldValue
      * @param  array $modelTableData
      * @param  string $sqlLogical
      * @param bool $isUse
@@ -908,7 +908,7 @@ 
     /**
      * Return table alias for model class for query
      *
-     * @return Model
+     * @return null|string
      *
      * @author dp <denis.a.shestakov@gmail.com>
      *
@@ -1287,7 +1287,7 @@ 
      *
      * @param  array $data Key-value array
      * @param  $part
-     * @param  $dataSourceKey
+     * @param  string|null $dataSourceKey
      * @return Query
      *
      * @author dp <denis.a.shestakov@gmail.com>
@@ -1758,6 +1758,10 @@ 
         return $this->addTrigger('afterSelect', $trigger, $params, $modelClass, $isUse);
     }
 
+    /**
+     * @param string $type
+     * @param boolean $isUse
+     */
     private function addTrigger($type, $trigger, $params, $modelClass, $isUse)
     {
         if (!$isUse) {
@@ -1809,7 +1813,7 @@ 
     }
 
     /**
-     * @param Widget[]|mixed $widgets
+     * @param string $widgets
      * @param bool $applyWidgetQueryBuilderParts
      * @return QueryBuilder
      */
@@ -1877,7 +1881,7 @@ 
      * Set query part limit
      *
      * @param  $limit
-     * @param  int|null $offset
+     * @param  integer $offset
      * @return QueryBuilder
      *
      * @author dp <denis.a.shestakov@gmail.com>

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 use Ice\Exception\Error;
 use Ice\Exception\QueryBuilder_Join;
 use Ice\Helper\Object;
-use Ice\QueryTranslator\Defined;
 use Ice\Widget\Form;
 
 /**

Source/Ice/Core/QueryResult.php

Doc Comments +3 added lines

@@ -546,6 +546,9 @@
         return $string;
     }
 
+    /**
+     * @return integer
+     */
     public function getFoundRows()
     {
         return $this->result[QueryResult::FOUND_ROWS];