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)) {

framework/db/DBCore.php

Doc Comments +1 added lines, -2 removed lines

@@ -271,7 +271,6 @@ 
      * Return parameters from the statment with dynamic number of parameters.
      *
      * @param resource $stmt Statement.
-     * @param array $params Parameters.
      */
     public static function bindResults($stmt) {
         $resultSet = array();
@@ -769,7 +768,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

@@ -598,7 +598,7 @@
     public static function deleteDBObject($dbObject) {
         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 +7 added lines, -7 removed lines

@@ -472,13 +472,13 @@ 
         $extra = trim($attributes['extra']);
         $comment = trim($attributes['comment']);
 
-        $fieldStr= "'" . $field . "' => ";
+        $fieldStr = "'" . $field . "' => ";
         if ($attributes['null'] === 'YES' && is_null($attributes['default'])) {
-            $fieldStr.= "null";
+            $fieldStr .= "null";
         } else {
-            $fieldStr.= self::getPrintableSQLValue($attributes['type'], $attributes['default']);
+            $fieldStr .= self::getPrintableSQLValue($attributes['type'], $attributes['default']);
         }
-        $fieldStr.= ", // " . $attributes['type'] .
+        $fieldStr .= ", // " . $attributes['type'] .
             ", " . (($attributes['null'] == "NO") ? "not null" : "null")
             . ", default '" . $attributes['default'] . "'" .
             ($extra ? ", " . $extra : "") .
@@ -573,9 +573,9 @@ 
                   LIMIT 1";
         $typesString = DBPreparedQuery::sqlTypesString($fieldsList, $idFieldName);
         if (Tools::isInteger($fieldsList[$idFieldName])) {
-            $typesString.= "i";
+            $typesString .= "i";
         } else {
-            $typesString.= "s";
+            $typesString .= "s";
         }
         $valuesList = self::createValuesList($fieldsList, $idFieldName);
         $valuesList[] = $dbObject->getId();
@@ -917,7 +917,7 @@ 
                     throw new DBCoreException("Class with name '" . $className . "' is not exists");
                 }
 
-                $query = "UPDATE " . $dbObject->getTableName() . " SET `" . $activationFieldName . "` = '" . $activationValue ."'
+                $query = "UPDATE " . $dbObject->getTableName() . " SET `" . $activationFieldName . "` = '" . $activationValue . "'
                           WHERE " . $dbObject->getIdFieldName() . " IN (" . DBPreparedQuery::sqlQMString($itemsNumber) . ")";
 
                 return self::doUpdateQuery($query, $types, $idsList);

framework/db/DBObject.php

Doc Comments +3 added lines, -3 removed lines

@@ -52,9 +52,9 @@ 
     /**
      * Sets primary key value.
      *
-     * @param mixed $recordId Key vaue.
+     * @param integer $recordId Key vaue.
      *
-     * @return boolean Success flag.
+     * @return DBObject Success flag.
      * @throws DBCoreException If object has no field with such name.
      */
     public function setId($recordId) {
@@ -495,7 +495,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 +8 added lines, -8 removed lines

@@ -414,11 +414,11 @@ 
          */
         if ($this->isNewRecord()) {
             if (!empty($this->dbQuery->conditions)) {
-                $this->dbQuery->query.= " WHERE ";
+                $this->dbQuery->query .= " WHERE ";
                 $this->dbQuery->sqlPushValues($this->dbQuery->conditions, " AND ");
             }
         } else {
-            $this->dbQuery->query.= " WHERE ";
+            $this->dbQuery->query .= " WHERE ";
             $this->dbQuery->sqlPushValues([static::ID_FIELD_NAME => $this->id]);
         }
 
@@ -427,14 +427,14 @@ 
          */
         if ($this->isNewRecord()) {
             if (!empty($this->dbQuery->order)) {
-                $this->dbQuery->query.= " ORDER BY";
+                $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;
                 }
             }
         }
@@ -446,13 +446,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.");
                     }
@@ -461,7 +461,7 @@ 
                 }
             }
         } else {
-            $this->dbQuery->query.= " LIMIT 1";
+            $this->dbQuery->query .= " LIMIT 1";
             $count = 1;
         }
 

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

@@ -103,7 +103,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);
@@ -383,7 +383,7 @@ 
             return $this->selectDBObject();
         }
 
-       /*
+        /*
         * Try to call parent method __call() with same params by default
         */
         $method = substr($methodName, 0, 3);

Spacing +8 added lines, -8 removed lines

@@ -103,7 +103,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);
@@ -136,8 +136,8 @@ 
             $query .= " AND " . $this->conditions;
         }
 
-        $query.= $this->getQueryOrderSQL();
-        $query.= " LIMIT 1";
+        $query .= $this->getQueryOrderSQL();
+        $query .= " LIMIT 1";
 
         $fieldType = DBField::getType($fieldValue);
         if (!$debug) {
@@ -178,14 +178,14 @@ 
      * @return array<DBObject>
      */
     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.= $this->getQueryOrderSQL();
-        $query.= $this->getQueryLimitSQL();
+        $query .= $this->getQueryOrderSQL();
+        $query .= $this->getQueryLimitSQL();
 
         if (!$debug) {
             $stmt = DBCore::doSelectQuery($query);
@@ -219,8 +219,8 @@ 
             $query .= " AND " . $this->conditions;
         }
 
-        $query.= $this->getQueryOrderSQL();
-        $query.= $this->getQueryLimitSQL();
+        $query .= $this->getQueryOrderSQL();
+        $query .= $this->getQueryLimitSQL();
 
         $fieldType = DBField::getType($fieldValue);
         if (!$debug) {

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

@@ -131,10 +131,10 @@ 
         fputs($this->connection, "TOP $messageNumber 0\r\n");
         $buffer = "";
         $headerReceived = 0;
-        while( $headerReceived == 0 ) {
-            $temp = fgets( $this->connection, 1024 );
+        while ($headerReceived == 0) {
+            $temp = fgets($this->connection, 1024);
             $buffer .= $temp;
-            if( $temp == ".\r\n" ) {
+            if ($temp == ".\r\n") {
                 $headerReceived = 1;
             }
         }
@@ -186,7 +186,7 @@ 
     public function getMessages() {
         $messages = [];
 
-        for ($i=1; ; $i++) {
+        for ($i = 1; ; $i++) {
             $message = $this->getMessage($i);
             if ($message === false) {
                 break;
@@ -205,7 +205,7 @@ 
     public function getBouncedEmails($delete = true, $number = null) {
         $emails = [];
 
-        for ($i = 1; (is_null($number) ? true : $i <= $number) ; $i++) {
+        for ($i = 1; (is_null($number) ? true : $i <= $number); $i++) {
             $message = $this->getMessage($i);
             if ($message !== false) {
                 $markers = [
@@ -265,7 +265,7 @@ 
     public function getEmails($delete = true, $number = null) {
         $emails = [];
 
-        for ($i = 1; (is_null($number) ? true : $i <= $number) ; $i++) {
+        for ($i = 1; (is_null($number) ? true : $i <= $number); $i++) {
             $message = $this->getMessage($i);
             if ($message !== false) {
                 $failSignaturePos = 0;
@@ -350,7 +350,7 @@ 
         $len = count($avar);
         $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] ) ){
+            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];
                 if ($L2 != "  " && $L3 != "Rec" && $L2 != "") {

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) {

Braces +6 added lines, -2 removed lines

@@ -77,8 +77,12 @@
     public static function getCurrentPageNumber($page, $totalPages) {
         $page = (integer)$page;
 
-        if ($page < 1) return 1;
-        if ($page > $totalPages) return $totalPages;
+        if ($page < 1) {
+            return 1;
+        }
+        if ($page > $totalPages) {
+            return $totalPages;
+        }
         return $page;
     }
 

framework/ui/UIComponent.php

Doc Comments +1 added lines, -1 removed lines

@@ -136,7 +136,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.
      */

framework/web/Http.php

Spacing +10 added lines, -10 removed lines

@@ -123,7 +123,7 @@ 
             $urlParts['path'] = '/';
         }
 
-        $sock = fsockopen($urlParts['host'], (isset($urlParts['port']) ? (int) $urlParts['port'] : 80), $errno, $errstr, 30);
+        $sock = fsockopen($urlParts['host'], (isset($urlParts['port']) ? (int)$urlParts['port'] : 80), $errno, $errstr, 30);
         if (!$sock) {
             throw new HttpException("$errstr ($errno)");
         }
@@ -134,7 +134,7 @@ 
         fwrite($sock, $request);
         $response = '';
         while (!feof($sock)) {
-            $response.= fread($sock, 8192);
+            $response .= fread($sock, 8192);
         }
         fclose($sock);
 
@@ -209,29 +209,29 @@ 
 
         // Data goes in the path for a GET request
         if ($type == self::GET) {
-            $parts['path'].= '?' . $postString;
+            $parts['path'] .= '?' . $postString;
         }
 
         $request = "$type " . $parts['path'] . " HTTP/1.1\r\n";
-        $request.= "Host: " . $parts['host'] . "\r\n";
+        $request .= "Host: " . $parts['host'] . "\r\n";
 
         if ($type == self::POST) {
-            $request.= "Content-Type: application/x-www-form-urlencoded\r\n";
-            $request.= "Content-Length: " . strlen($postString) . "\r\n";
+            $request .= "Content-Type: application/x-www-form-urlencoded\r\n";
+            $request .= "Content-Length: " . strlen($postString) . "\r\n";
         }
-        $request.= "Connection: Close\r\n";
-        $request.= "\r\n";
+        $request .= "Connection: Close\r\n";
+        $request .= "\r\n";
 
         // Data goes in the request body for a POST request
         if ($type == self::POST && isset($postString)) {
-            $request.= $postString;
+            $request .= $postString;
         }
 
         fwrite($sock, $request);
 
         $response = "";
         while (!feof($sock) && $result = fgets($sock)) {
-            $response.= $result;
+            $response .= $result;
         }
 
         fclose($sock);

framework/web/Request.php

Spacing +1 added lines, -1 removed lines

@@ -243,7 +243,7 @@
                 case ('boolean'):
                 case ('bool'):
                 case ('b'):
-                    $_FIELDS[$fieldName] = (boolean) $_FIELDS[$fieldName];
+                    $_FIELDS[$fieldName] = (boolean)$_FIELDS[$fieldName];
             }
         } else {
             throw new \Exception("No field '" . $fieldName . "' in global fields list.");