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/db/DBQuery.php

Spacing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
                 } else {
                     OutputStream::message(OutputStream::MSG_ERROR, "Number of types is not equal parameters number.");
                     OutputStream::message(OutputStream::MSG_INFO, "T: " . $types);
-                    OutputStream::message(OutputStream::MSG_INFO, "A: [" . implode(", ", $params)  . "]");
+                    OutputStream::message(OutputStream::MSG_INFO, "A: [" . implode(", ", $params) . "]");
                 }
             }
         } else {

framework/db/DBQueryCondition.php

Spacing +2 added lines, -2 removed lines

@@ -106,7 +106,7 @@ 
                     break;
                 case ("IN"):
                 case ("NOT IN"):
-                    if (is_array($value) &&  !empty($value)) {
+                    if (is_array($value) && !empty($value)) {
                         $dataList = [];
                         foreach ($value as $dataItem) {
                             $dataList[] = DBField::sqlValue($this->field->type, $dataItem);
@@ -116,7 +116,7 @@ 
                         if ($count > 0) {
                             $qmStr = "?";
                             $tStr = $this->field->type;
-                            for ($i = 1; $i < $count; $i ++) {
+                            for ($i = 1; $i < $count; $i++) {
                                 $qmStr .= ", ?";
                                 $tStr .= $this->field->type;
                             }