Asymptix / Framework

framework/app/View.php

Doc Comments +3 added lines

@@ -24,6 +24,9 @@
     private $_messages = array();
     private $_errors = array();
 
+    /**
+     * @param null|Route $tpl
+     */
     public function __construct($tpl) {
         if (!empty($tpl)) {
             if (is_string($tpl)) {

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
                 $this->_tpl = $tpl;
             } elseif (Tools::isInstanceOf($tpl, new Route())) {
                 $this->_route = $tpl;
-                $this->_tpl = $this->_route->controller.'/'.$this->_route->controller.'_'.$this->_route->action;
+                $this->_tpl = $this->_route->controller . '/' . $this->_route->controller . '_' . $this->_route->action;
             } else {
                 throw new \Exception('Invalid view template');
             }
@@ -48,7 +48,7 @@ 
             throw new \Exception('View object was not initialized with template');
         }
 
-        require_once $path.$this->_tpl.$suffix;
+        require_once $path . $this->_tpl . $suffix;
     }
 
     public function setTpl($_tpl)

framework/db/DBSelector.php

Doc Comments +3 added lines

@@ -42,6 +42,9 @@
      */
     private $dbObject = null;
 
+    /**
+     * @param string $className
+     */
     public function __construct($className) {
         if (is_object($className)) {
             $className = get_class($className);

Indentation +2 added lines, -2 removed lines

@@ -112,7 +112,7 @@ 
     public function selectDBObject($debug = false)
     {
         $query = 'SELECT * FROM '.$this->dbObject->getTableName().
-                  ($this->conditions != '' ? ' WHERE '.$this->conditions : '').' LIMIT 1';
+                    ($this->conditions != '' ? ' WHERE '.$this->conditions : '').' LIMIT 1';
 
         if (!$debug) {
             $stmt = DBCore::doSelectQuery($query);
@@ -411,7 +411,7 @@ 
             return $this->selectDBObject();
         }
 
-       /*
+        /*
         * Try to call parent method __call() with same params by default
         */
         $method = substr($methodName, 0, 3);

Spacing +24 added lines, -24 removed lines

@@ -66,8 +66,8 @@ 
             $classNamespaceParts = explode('\\', $this->className);
             if ($className != $classNamespaceParts[count($classNamespaceParts) - 1]) {
                 throw new DBSelectorException(
-                    "Invalid DB object classname '".$className."' in method name. ".
-                    "Valid classname is '".$this->className."'"
+                    "Invalid DB object classname '" . $className . "' in method name. " .
+                    "Valid classname is '" . $this->className . "'"
                 );
             }
         }
@@ -111,8 +111,8 @@ 
      */
     public function selectDBObject($debug = false)
     {
-        $query = 'SELECT * FROM '.$this->dbObject->getTableName().
-                  ($this->conditions != '' ? ' WHERE '.$this->conditions : '').' LIMIT 1';
+        $query = 'SELECT * FROM ' . $this->dbObject->getTableName() .
+                  ($this->conditions != '' ? ' WHERE ' . $this->conditions : '') . ' LIMIT 1';
 
         if (!$debug) {
             $stmt = DBCore::doSelectQuery($query);
@@ -140,10 +140,10 @@ 
      */
     public function selectDBObjectByField($fieldName, $fieldValue, $debug = false)
     {
-        $query = 'SELECT * FROM '.$this->dbObject->getTableName().' WHERE '.$fieldName.' = ?';
+        $query = 'SELECT * FROM ' . $this->dbObject->getTableName() . ' WHERE ' . $fieldName . ' = ?';
 
         if ($this->conditions != '') {
-            $query .= ' AND '.$this->conditions;
+            $query .= ' AND ' . $this->conditions;
         }
 
         $query .= $this->getQueryOrderSQL();
@@ -191,10 +191,10 @@ 
      */
     public function selectDBObjects($debug = false)
     {
-        $query = 'SELECT'.($this->unique ? ' DISTINCT' : '').' * FROM '.$this->dbObject->getTableName();
+        $query = 'SELECT' . ($this->unique ? ' DISTINCT' : '') . ' * FROM ' . $this->dbObject->getTableName();
 
         if ($this->conditions != '') {
-            $query .= ' WHERE '.$this->conditions;
+            $query .= ' WHERE ' . $this->conditions;
         }
 
         $query .= $this->getQueryOrderSQL();
@@ -228,11 +228,11 @@ 
      */
     public function selectDBObjectsByField($fieldName, $fieldValue, $debug = false)
     {
-        $query = 'SELECT * FROM '.$this->dbObject->getTableName();
-        $query .= ' WHERE '.$fieldName.' = ?';
+        $query = 'SELECT * FROM ' . $this->dbObject->getTableName();
+        $query .= ' WHERE ' . $fieldName . ' = ?';
 
         if ($this->conditions != '') {
-            $query .= ' AND '.$this->conditions;
+            $query .= ' AND ' . $this->conditions;
         }
 
         $query .= $this->getQueryOrderSQL();
@@ -263,10 +263,10 @@ 
      */
     public function count()
     {
-        $query = 'SELECT count(*) FROM '.$this->dbObject->getTableName();
+        $query = 'SELECT count(*) FROM ' . $this->dbObject->getTableName();
 
         if ($this->conditions != '') {
-            $query .= ' WHERE '.$this->conditions;
+            $query .= ' WHERE ' . $this->conditions;
         }
 
         return DBCore::selectSingleValue($query);
@@ -279,10 +279,10 @@ 
      */
     public function max()
     {
-        $query = 'SELECT max(`'.$this->field.'`) FROM '.$this->dbObject->getTableName();
+        $query = 'SELECT max(`' . $this->field . '`) FROM ' . $this->dbObject->getTableName();
 
         if ($this->conditions != '') {
-            $query .= ' WHERE '.$this->conditions;
+            $query .= ' WHERE ' . $this->conditions;
         }
 
         return DBCore::selectSingleValue($query);
@@ -295,10 +295,10 @@ 
      */
     public function min()
     {
-        $query = 'SELECT min(`'.$this->field.'`) FROM '.$this->dbObject->getTableName();
+        $query = 'SELECT min(`' . $this->field . '`) FROM ' . $this->dbObject->getTableName();
 
         if ($this->conditions != '') {
-            $query .= ' WHERE '.$this->conditions;
+            $query .= ' WHERE ' . $this->conditions;
         }
 
         return DBCore::selectSingleValue($query);
@@ -312,10 +312,10 @@ 
     private function getQueryOrderSQL()
     {
         if ($this->order != '') {
-            return ' ORDER BY '.$this->order;
+            return ' ORDER BY ' . $this->order;
         }
 
-        return ' ORDER BY '.$this->dbObject->getIdFieldName().' DESC';
+        return ' ORDER BY ' . $this->dbObject->getIdFieldName() . ' DESC';
     }
 
     /**
@@ -327,10 +327,10 @@ 
     {
         if ($this->count !== 'all') {
             if ($this->offset > 0) {
-                return ' LIMIT '.$this->offset.','.$this->count;
+                return ' LIMIT ' . $this->offset . ',' . $this->count;
             }
 
-            return ' LIMIT '.$this->count;
+            return ' LIMIT ' . $this->count;
         }
 
         return '';
@@ -380,9 +380,9 @@ 
         elseif (preg_match('#^selectAll([A-Z]{1}[[:alpha:]]+)s#', $methodName, $matches)) {
             $this->validateClassName(preg_replace('#ie$#', 'y', $matches[1]));
 
-            $this->order = '`'.$this->dbObject->getIdFieldName().'` DESC';
+            $this->order = '`' . $this->dbObject->getIdFieldName() . '` DESC';
             if (isset($methodParams[0])) {
-                $this->order = (string) $methodParams[0];
+                $this->order = (string)$methodParams[0];
             }
 
             $dbObjects = $this->selectDBObjects();
@@ -425,7 +425,7 @@ 
             case 'get':
                 return $this->getFieldValue($fieldName);
             default:
-                throw new DBSelectorException("No method with name '".$methodName."'");
+                throw new DBSelectorException("No method with name '" . $methodName . "'");
         }
     }
 }

framework/mail/POP.php

Doc Comments +1 added lines, -1 removed lines

@@ -110,7 +110,7 @@
     /**
      * Count messages in POP server.
      *
-     * @return type
+     * @return integer
      */
     public function countMessages() {
         fputs($this->connection, "STAT\r\n");

Spacing +7 added lines, -7 removed lines

@@ -62,10 +62,10 @@ 
             fgets($sock, 1024);
 
             // Connection accepted
-            fwrite($sock, 'USER '.$username."\r\n");
+            fwrite($sock, 'USER ' . $username . "\r\n");
             $userresponse = fgets($sock, 1024);
             if ($userresponse[0] == '+') { // User accepted
-                fwrite($sock, 'PASS '.$password."\r\n");
+                fwrite($sock, 'PASS ' . $password . "\r\n");
                 $passresponse = fgets($sock, 1024);
                 if ($passresponse[0] != '+') { // Wrong password
                     $passresponse = str_replace("\r", '', str_replace("\n", '', $passresponse));
@@ -76,12 +76,12 @@ 
                 }
             } else { // Invalid username
                 throw new POPServerException(
-                    "Username '".$username."' not accepted.", 1
+                    "Username '" . $username . "' not accepted.", 1
                 );
             }
         } else {
             throw new POPServerException(
-                'Unable to Connect to '.$host.'. Network Problems could be responsible.', 2
+                'Unable to Connect to ' . $host . '. Network Problems could be responsible.', 2
             );
         }
         $this->connection = $sock;
@@ -125,7 +125,7 @@ 
         $statresponse = fgets($this->connection, 1024);
         $avar = explode(' ', $statresponse);
 
-        return (int) $avar[1];
+        return (int)$avar[1];
     }
 
     /**
@@ -373,8 +373,8 @@ 
         $ret = $L2 = $L3 = null;
         for ($i = 0; $i < $len; $i++) {
             if (isset($avar[$i]) && isset($avar[$i][0]) && isset($avar[$i][1]) && isset($avar[$i][2])) {
-                $L2 = $avar[$i][0].$avar[$i][1];
-                $L3 = $avar[$i][0].$avar[$i][1].$avar[$i][2];
+                $L2 = $avar[$i][0] . $avar[$i][1];
+                $L3 = $avar[$i][0] . $avar[$i][1] . $avar[$i][2];
                 if ($L2 != '  ' && $L3 != 'Rec' && $L2 != '') {
                     $avar2 = explode(':', $avar[$i]);
                     $temp = str_replace("$avar2[0]:", '', $avar[$i]);

framework/ui/controls/UIPagination.php

Doc Comments +3 added lines

@@ -62,6 +62,9 @@
         $this->lastPage = $this->currentPage + $this->pagesOffset;
     }
 
+    /**
+     * @param integer $currentPage
+     */
     public function setCurrentPage($currentPage) {
         if (Tools::isInteger($currentPage)) {
             if ($currentPage < 0) {

Spacing +2 added lines, -2 removed lines

@@ -1,6 +1,6 @@ 
 <?php
 
-require_once realpath(dirname(__FILE__)).'/../UIControl.php';
+require_once realpath(dirname(__FILE__)) . '/../UIControl.php';
 
 /**
  * Pagination UI control class.
@@ -82,7 +82,7 @@ 
 
     public static function getCurrentPageNumber($page, $totalPages)
     {
-        $page = (int) $page;
+        $page = (int)$page;
 
         if ($page < 1) {
             return 1;

framework/db/DBPreparedQuery.php

Indentation +2 added lines, -2 removed lines

@@ -139,9 +139,9 @@
             $typeByValue = DBField::getType($value);
             if ($typeByValue != 's') {
                 if ($type != $typeByValue && !(
-                       ($type == 'd' && $typeByValue == 'i') || // We can put integer as double
+                        ($type == 'd' && $typeByValue == 'i') || // We can put integer as double
                        ($type == 's' && $typeByValue == 'i') // We can put integer as string
-                   )
+                    )
                 ) {
                     throw new DBCoreException(
                         "Invalid query parameters types string ('" . $value .

Spacing +10 added lines, -10 removed lines

@@ -140,7 +140,7 @@ 
 
             if (!in_array($type, ['i', 'd', 's', 'b'])) {
                 throw new DBCoreException(
-                    "Invalid query parameters types string (type '".$type.
+                    "Invalid query parameters types string (type '" . $type .
                     "' is undefined, only 'i', 'd', 's' and 'b' types are acceptable)"
                 );
             }
@@ -153,30 +153,30 @@ 
                    )
                 ) {
                     throw new DBCoreException(
-                        "Invalid query parameters types string ('".$value.
-                        "' is not '".$type."' type but '".$typeByValue."' detected)"
+                        "Invalid query parameters types string ('" . $value .
+                        "' is not '" . $type . "' type but '" . $typeByValue . "' detected)"
                     );
                 }
             } else { // in case if we try send non-string parameters as a string value
                 switch ($type) {
                     case 'i':
-                        if (!(Tools::isNumeric($value) && ((string) (int) $value === $value))) {
+                        if (!(Tools::isNumeric($value) && ((string)(int)$value === $value))) {
                             throw new DBCoreException(
-                                "Invalid query parameters types string ('".$value."' is not '".$type.')'
+                                "Invalid query parameters types string ('" . $value . "' is not '" . $type . ')'
                             );
                         }
                         break;
                     case 'd':
                         if (!Tools::isDoubleString($value)) {
                             throw new DBCoreException(
-                                "Invalid query parameters types string ('".$value."' is not '".$type.')'
+                                "Invalid query parameters types string ('" . $value . "' is not '" . $type . ')'
                             );
                         }
                         break;
                     case 'b':
                         if (!in_array(strtolower($value), ['true', 'false'])) {
                             throw new DBCoreException(
-                                "Invalid query parameters types string ('".$value."' is not '".$type.')'
+                                "Invalid query parameters types string ('" . $value . "' is not '" . $type . ')'
                             );
                         }
                         break;
@@ -210,7 +210,7 @@ 
         $chunks = [];
         foreach (array_keys($fieldsList) as $fieldName) {
             if ($fieldName != $idFieldName) {
-                $chunks[] = '`'.$fieldName.'` = ?';
+                $chunks[] = '`' . $fieldName . '` = ?';
             }
         }
 
@@ -230,7 +230,7 @@ 
         $chunks = [];
         foreach ($fieldsList as $fieldName => $fieldValue) {
             if ($fieldName != $idFieldName) {
-                $chunks[] = '`'.$fieldName."` = '".$fieldValue."'";
+                $chunks[] = '`' . $fieldName . "` = '" . $fieldValue . "'";
             }
         }
 
@@ -303,7 +303,7 @@ 
         foreach ($values as $fieldName => $fieldValue) {
             if (!is_array($fieldValue)) {
                 if (!is_null($fieldValue)) {
-                    $chunks[] = $fieldName.' = ?';
+                    $chunks[] = $fieldName . ' = ?';
                     $this->types .= DBField::getType($fieldValue);
                     $this->params[] = $fieldValue;
                 } else {

framework/db/DBCore.php

Doc Comments +2 added lines, -3 removed lines

@@ -292,7 +292,6 @@ 
      * Return parameters from the statment with dynamic number of parameters.
      *
      * @param resource $stmt   Statement.
-     * @param array    $params Parameters.
      */
     public static function bindResults($stmt)
     {
@@ -495,7 +494,7 @@ 
      * Returns printable field description string for table fields list generator.
      *
      * @param string $field      Field name.
-     * @param array  $attributes List of field attributes.
+     * @param string  $attributes List of field attributes.
      *
      * @return string
      */
@@ -818,7 +817,7 @@ 
     /**
      * Executes SQL query with single record and value result and return this value.
      *
-     * @param mixed  $query  SQL query template string or DBPreparedQuery object
+     * @param string  $query  SQL query template string or DBPreparedQuery object
      *                       if single parameter.
      * @param string $types  Types string (ex: "isdb").
      * @param array  $params Parameters in the same order like types string.

Indentation +1 added lines, -1 removed lines

@@ -639,7 +639,7 @@
     {
         if (!empty($dbObject) && is_object($dbObject)) {
             $query = 'DELETE FROM '.$dbObject->getTableName().
-                     ' WHERE '.$dbObject->getIdFieldName().' = ? LIMIT 1';
+                        ' WHERE '.$dbObject->getIdFieldName().' = ? LIMIT 1';
             if (Tools::isInteger($dbObject->getId())) {
                 $typesString = 'i';
             } else {

Spacing +30 added lines, -30 removed lines

@@ -115,7 +115,7 @@ 
     public function openConnection($connResource, $connName = null)
     {
         if ($connName !== null) {
-            $connName = (string) $connName;
+            $connName = (string)$connName;
             if (isset($this->connections[$connName])) {
                 throw new DBCoreException('You trying to save a connection with an existing name');
             }
@@ -139,7 +139,7 @@ 
     public function getConnection($connName)
     {
         if (!isset($this->connections[$connName])) {
-            throw new DBCoreException('Unknown connection: '.$connName);
+            throw new DBCoreException('Unknown connection: ' . $connName);
         }
 
         return $this->connections[$connName];
@@ -304,7 +304,7 @@ 
                 $resultSet[$field->table] = [];
             }
             $resultSet[$field->table][$field->name] = $fieldsCounter++;
-            $parameterName = 'variable'.$fieldsCounter; //$field->name;
+            $parameterName = 'variable' . $fieldsCounter; //$field->name;
             $$parameterName = null;
             $parameters[] = &$$parameterName;
         }
@@ -420,7 +420,7 @@ 
     public static function getTableFieldsList($tableName)
     {
         if (!empty($tableName)) {
-            $query = 'SHOW FULL COLUMNS FROM '.$tableName;
+            $query = 'SHOW FULL COLUMNS FROM ' . $tableName;
             $stmt = self::doSelectQuery($query);
             if ($stmt !== false) {
                 $stmt->bind_result(
@@ -467,7 +467,7 @@ 
          || strpos($type, 'datetime') === 0
          || strpos($type, 'timestamp') === 0
          || strpos($type, 'date') === 0) {
-            return '"'.$value.'"';
+            return '"' . $value . '"';
         } elseif (strpos($type, 'int') === 0
          || strpos($type, 'tinyint') === 0
          || strpos($type, 'smallint') === 0
@@ -504,17 +504,17 @@ 
         $extra = trim($attributes['extra']);
         $comment = trim($attributes['comment']);
 
-        $fieldStr = "'".$field."' => ";
+        $fieldStr = "'" . $field . "' => ";
         if ($attributes['null'] === 'YES' && is_null($attributes['default'])) {
             $fieldStr .= 'null';
         } else {
             $fieldStr .= self::getPrintableSQLValue($attributes['type'], $attributes['default']);
         }
-        $fieldStr .= ', // '.$attributes['type'].
-            ', '.(($attributes['null'] == 'NO') ? 'not null' : 'null')
-            .", default '".$attributes['default']."'".
-            ($extra ? ', '.$extra : '').
-            ($comment ? ' ('.$comment.')' : '')."\n";
+        $fieldStr .= ', // ' . $attributes['type'] .
+            ', ' . (($attributes['null'] == 'NO') ? 'not null' : 'null')
+            .", default '" . $attributes['default'] . "'" .
+            ($extra ? ', ' . $extra : '') .
+            ($comment ? ' (' . $comment . ')' : '') . "\n";
 
         return $fieldStr;
     }
@@ -572,12 +572,12 @@ 
         $idFieldName = $dbObject->getIdFieldName();
 
         if (Tools::isInteger($fieldsList[$idFieldName])) {
-            $query = 'INSERT INTO '.$dbObject->getTableName().'
+            $query = 'INSERT INTO ' . $dbObject->getTableName() . '
                           SET '.DBPreparedQuery::sqlQMValuesString($fieldsList, $idFieldName);
             $typesString = DBPreparedQuery::sqlTypesString($fieldsList, $idFieldName);
             $valuesList = self::createValuesList($fieldsList, $idFieldName);
         } else {
-            $query = 'INSERT INTO '.$dbObject->getTableName().'
+            $query = 'INSERT INTO ' . $dbObject->getTableName() . '
                           SET '.DBPreparedQuery::sqlQMValuesString($fieldsList);
             $typesString = DBPreparedQuery::sqlTypesString($fieldsList);
             $valuesList = self::createValuesList($fieldsList);
@@ -607,9 +607,9 @@ 
         $fieldsList = $dbObject->getFieldsList();
         $idFieldName = $dbObject->getIdFieldName();
 
-        $query = 'UPDATE '.$dbObject->getTableName().'
-                  SET '.DBPreparedQuery::sqlQMValuesString($fieldsList, $idFieldName).'
-                  WHERE '.$idFieldName.' = ?
+        $query = 'UPDATE ' . $dbObject->getTableName() . '
+                  SET '.DBPreparedQuery::sqlQMValuesString($fieldsList, $idFieldName) . '
+                  WHERE '.$idFieldName . ' = ?
                   LIMIT 1';
         $typesString = DBPreparedQuery::sqlTypesString($fieldsList, $idFieldName);
         if (Tools::isInteger($fieldsList[$idFieldName])) {
@@ -638,8 +638,8 @@ 
     public static function deleteDBObject($dbObject)
     {
         if (!empty($dbObject) && is_object($dbObject)) {
-            $query = 'DELETE FROM '.$dbObject->getTableName().
-                     ' WHERE '.$dbObject->getIdFieldName().' = ? LIMIT 1';
+            $query = 'DELETE FROM ' . $dbObject->getTableName() .
+                     ' WHERE ' . $dbObject->getIdFieldName() . ' = ? LIMIT 1';
             if (Tools::isInteger($dbObject->getId())) {
                 $typesString = 'i';
             } else {
@@ -694,7 +694,7 @@ 
                 return;
             }
         } elseif ($stmt->num_rows > 1) {
-            throw new DBCoreException("More than single record of '".$className."' entity selected");
+            throw new DBCoreException("More than single record of '" . $className . "' entity selected");
         }
     }
 
@@ -871,7 +871,7 @@ 
     public static function __callStatic($methodName, $methodParams)
     {
         if (strrpos($methodName, 'ies') == strlen($methodName) - 3) {
-            $methodName = substr($methodName, 0, strlen($methodName) - 3).'ys';
+            $methodName = substr($methodName, 0, strlen($methodName) - 3) . 'ys';
         }
 
         /*
@@ -918,12 +918,12 @@ 
                 $dbObject = new $className();
 
                 if (!isInstanceOf($dbObject, $className)) {
-                    throw new DBCoreException("Class with name '".$className."' is not exists");
+                    throw new DBCoreException("Class with name '" . $className . "' is not exists");
                 }
 
-                $query = 'DELETE FROM '.$dbObject->getTableName().'
-                          WHERE '.$dbObject->getIdFieldName().'
-                             IN ('.DBPreparedQuery::sqlQMString($itemsNumber).')';
+                $query = 'DELETE FROM ' . $dbObject->getTableName() . '
+                          WHERE '.$dbObject->getIdFieldName() . '
+                             IN ('.DBPreparedQuery::sqlQMString($itemsNumber) . ')';
 
                 return self::doUpdateQuery($query, $types, $idsList);
             }
@@ -936,7 +936,7 @@ 
          */
         if (preg_match('#delete([a-zA-Z]+)#', $methodName, $matches)) {
             return call_user_func(
-                [self::getInstance(), $methodName.'s'],
+                [self::getInstance(), $methodName . 's'],
                 [$methodParams[0]]
             );
         }
@@ -947,7 +947,7 @@ 
         if (preg_match('#set([a-zA-Z]+)Activation#', $methodName, $matches)) {
             $className = $matches[1];
             if (strrpos($className, 'ies') == strlen($className) - 3) {
-                $className = substr($className, 0, strlen($className) - 3).'y';
+                $className = substr($className, 0, strlen($className) - 3) . 'y';
             } else {
                 $className = substr($className, 0, strlen($className) - 1);
             }
@@ -967,17 +967,17 @@ 
                 $dbObject = new $className();
 
                 if (!isInstanceOf($dbObject, $className)) {
-                    throw new DBCoreException("Class with name '".$className."' is not exists");
+                    throw new DBCoreException("Class with name '" . $className . "' is not exists");
                 }
 
-                $query = 'UPDATE '.$dbObject->getTableName().' SET `'.$activationFieldName."` = '".$activationValue."'
-                          WHERE ".$dbObject->getIdFieldName().' IN ('.DBPreparedQuery::sqlQMString($itemsNumber).')';
+                $query = 'UPDATE ' . $dbObject->getTableName() . ' SET `' . $activationFieldName . "` = '" . $activationValue . "'
+                          WHERE ".$dbObject->getIdFieldName() . ' IN (' . DBPreparedQuery::sqlQMString($itemsNumber) . ')';
 
                 return self::doUpdateQuery($query, $types, $idsList);
             }
         }
 
-        throw new DBCoreException('No such method "'.$methodName.'"');
+        throw new DBCoreException('No such method "' . $methodName . '"');
     }
 }
 

framework/db/DBObject.php

Doc Comments +3 added lines, -3 removed lines

@@ -58,11 +58,11 @@ 
     /**
      * Sets primary key value.
      *
-     * @param mixed $recordId Key vaue.
+     * @param integer $recordId Key vaue.
      *
      * @throws DBCoreException If object has no field with such name.
      *
-     * @return bool Success flag.
+     * @return DBObject Success flag.
      */
     public function setId($recordId)
     {
@@ -532,7 +532,7 @@ 
     /**
      * Deletes DB record for current DBObject.
      *
-     * @return mixed Number of affected rows (1 if some record was deleted,
+     * @return integer Number of affected rows (1 if some record was deleted,
      *               0 - if no) or FALSE if some error occurred.
      */
     public function delete()

Spacing +11 added lines, -11 removed lines

@@ -98,9 +98,9 @@ 
     public function saveActivationFlag()
     {
         return DBCore::doUpdateQuery(
-            'UPDATE '.static::TABLE_NAME.'
+            'UPDATE ' . static::TABLE_NAME . '
                 SET activation = ?
-             WHERE '.static::ID_FIELD_NAME.' = ?
+             WHERE '.static::ID_FIELD_NAME . ' = ?
              LIMIT 1',
             'ii',
             [$this->activation, $this->id]
@@ -169,9 +169,9 @@ 
     public function saveRemovementFlag()
     {
         return DBCore::doUpdateQuery(
-            'UPDATE '.static::TABLE_NAME.'
+            'UPDATE ' . static::TABLE_NAME . '
                 SET removed = ?
-             WHERE '.static::ID_FIELD_NAME.' = ?
+             WHERE '.static::ID_FIELD_NAME . ' = ?
              LIMIT 1',
             'ii',
             [$this->removed, $this->id]
@@ -434,14 +434,14 @@ 
     {
         switch ($this->dbQuery->getType()) {
             case DBQueryType::SELECT:
-                $this->dbQuery->query = 'SELECT * FROM '.static::TABLE_NAME;
+                $this->dbQuery->query = 'SELECT * FROM ' . static::TABLE_NAME;
                 break;
             case DBQueryType::UPDATE:
-                $this->dbQuery->query = 'UPDATE '.static::TABLE_NAME.' SET ';
+                $this->dbQuery->query = 'UPDATE ' . static::TABLE_NAME . ' SET ';
                 $this->dbQuery->sqlPushValues($this->dbQuery->fields);
                 break;
             case DBQueryType::DELETE:
-                $this->dbQuery->query = 'DELETE FROM '.static::TABLE_NAME;
+                $this->dbQuery->query = 'DELETE FROM ' . static::TABLE_NAME;
                 break;
         }
 
@@ -466,11 +466,11 @@ 
                 $this->dbQuery->query .= ' ORDER BY';
                 if (is_array($this->dbQuery->order)) {
                     foreach ($this->dbQuery->order as $fieldName => $ord) {
-                        $this->dbQuery->query .= ' '.$fieldName.' '.$ord.',';
+                        $this->dbQuery->query .= ' ' . $fieldName . ' ' . $ord . ',';
                     }
                     $this->dbQuery->query = substr($this->dbQuery->query, 0, strlen($this->dbQuery->query) - 1);
                 } elseif (is_string($this->dbQuery->order)) {
-                    $this->dbQuery->query .= ' '.$this->dbQuery->order;
+                    $this->dbQuery->query .= ' ' . $this->dbQuery->order;
                 }
             }
         }
@@ -482,13 +482,13 @@ 
         if ($this->isNewRecord()) {
             if (!is_null($this->dbQuery->limit)) {
                 if (Tools::isInteger($this->dbQuery->limit)) {
-                    $this->dbQuery->query .= ' LIMIT '.$this->dbQuery->limit;
+                    $this->dbQuery->query .= ' LIMIT ' . $this->dbQuery->limit;
                     $count = $this->dbQuery->limit;
                 } elseif (is_array($this->dbQuery->limit) && count($this->dbQuery->limit) == 2) {
                     $offset = $this->dbQuery->limit[0];
                     $count = $this->dbQuery->limit[1];
                     if (Tools::isInteger($offset) && Tools::isInteger($count)) {
-                        $this->dbQuery->query .= ' LIMIT '.$offset.', '.$count;
+                        $this->dbQuery->query .= ' LIMIT ' . $offset . ', ' . $count;
                     } else {
                         throw new DBCoreException('Invalid LIMIT param in select() method.');
                     }

framework/ui/UIComponent.php

Doc Comments +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
      * css($propertyName, $propertyValue) Set the value of a CSS property.
      *
      * @param string $propertyName  Name of a CSS property.
-     * @param mixed  $propertyValue Value of a CSS property.
+     * @param string  $propertyValue Value of a CSS property.
      *
      * @return Value of the CSS property or NULL if property is not exists.
      */

Spacing +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
             if (isset($this->$attributeName)) {
                 $this->$attributeName = $attributeValue;
             } else {
-                throw new \Exception("Wrong attribute '".$attributeName."' for ".get_class($this).' component');
+                throw new \Exception("Wrong attribute '" . $attributeName . "' for " . get_class($this) . ' component');
             }
         }
     }
@@ -170,7 +170,7 @@ 
     {
         $style = '';
         foreach ($this->style as $propertyName => $propertyValue) {
-            $style .= $propertyName.':'.$propertyValue.';';
+            $style .= $propertyName . ':' . $propertyValue . ';';
         }
 
         return $style;

generators/beans_generator.php

Spacing +8 added lines, -8 removed lines

@@ -15,7 +15,7 @@ 
 OutputStream::start();
 
 if (!file_exists(RESULTS_PATH) || is_file(RESULTS_PATH)) {
-    OutputStream::msg(OutputStream::MSG_ERROR, "Destination directory '".RESULTS_PATH."' doesn't exists.");
+    OutputStream::msg(OutputStream::MSG_ERROR, "Destination directory '" . RESULTS_PATH . "' doesn't exists.");
     OutputStream::close();
     exit();
 }
@@ -26,9 +26,9 @@ 
 if ($stmt !== false) {
     $tpl = file_get_contents(CLASS_TPL);
     while ($resultSet = DBCore::bindResults($stmt)) {
-        $tableName = $resultSet['TABLE_NAMES']['Tables_in_'.conf\Config::getDBConfigParam('DBNAME')];
+        $tableName = $resultSet['TABLE_NAMES']['Tables_in_' . conf\Config::getDBConfigParam('DBNAME')];
 
-        OutputStream::msg(OutputStream::MSG_INFO, "Reading structure for table '".$tableName."'...");
+        OutputStream::msg(OutputStream::MSG_INFO, "Reading structure for table '" . $tableName . "'...");
 
         $idFieldName = 'id';
         $fieldsListStr = '';
@@ -38,7 +38,7 @@ 
                 if ($attributes['key'] === 'PRI') {
                     $idFieldName = $field;
                 }
-                $fieldsListStr .= '        '.DBCore::getPrintableFieldString($field, $attributes);
+                $fieldsListStr .= '        ' . DBCore::getPrintableFieldString($field, $attributes);
             }
             $fieldsListStr = substr($fieldsListStr, 0, strlen($fieldsListStr) - 1);
 
@@ -52,11 +52,11 @@ 
                 date('Y'), AUTHOR, EMAIL,
             ], $tpl);
 
-            file_put_contents(RESULTS_PATH.$className.'.php', $content);
+            file_put_contents(RESULTS_PATH . $className . '.php', $content);
 
-            OutputStream::msg(OutputStream::MSG_SUCCESS, "Class '".RESULTS_PATH.$className.".php' generated.");
+            OutputStream::msg(OutputStream::MSG_SUCCESS, "Class '" . RESULTS_PATH . $className . ".php' generated.");
         } else {
-            OutputStream::msg(OutputStream::MSG_ERROR, "Can't read structure for table '".$tableName."'.");
+            OutputStream::msg(OutputStream::MSG_ERROR, "Can't read structure for table '" . $tableName . "'.");
         }
     }
 
@@ -70,7 +70,7 @@ 
 {
     $underlinesReplaced = preg_replace_callback(
         '/_([a-zA-Z]{1})/',
-        function ($matches) {
+        function($matches) {
             return strtoupper($matches[1]);
         },
         $tableName