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 +5 added lines, -5 removed lines

@@ -481,11 +481,11 @@ 
 
         $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 : "") .
@@ -583,9 +583,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();

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 +3 added lines, -3 removed lines

@@ -101,8 +101,8 @@ 
      */
     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);
@@ -183,7 +183,7 @@ 
      */
     public function selectDBObjects($debug = false) {
         $query = "SELECT" . ($this->unique ? " DISTINCT" : "")
-               . " * FROM " . $this->dbObject->getTableName();
+                . " * FROM " . $this->dbObject->getTableName();
 
         if ($this->conditions != "") {
             $query .= " WHERE " . $this->conditions;

Spacing +6 added lines, -6 removed lines

@@ -137,8 +137,8 @@ 
             $query .= " AND " . $this->conditions;
         }
 
-        $query.= $this->getQueryOrderSQL();
-        $query.= " LIMIT 1";
+        $query .= $this->getQueryOrderSQL();
+        $query .= " LIMIT 1";
 
         $fieldType = DBField::getType($fieldValue);
         if (!$debug) {
@@ -189,8 +189,8 @@ 
             $query .= " WHERE " . $this->conditions;
         }
 
-        $query.= $this->getQueryOrderSQL();
-        $query.= $this->getQueryLimitSQL();
+        $query .= $this->getQueryOrderSQL();
+        $query .= $this->getQueryLimitSQL();
 
         if (!$debug) {
             $stmt = DBCore::doSelectQuery($query);
@@ -226,8 +226,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/DBObject.php

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;
         }
 

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

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;
                             }

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 +8 added lines, -8 removed lines

@@ -219,7 +219,7 @@ 
         $chunks = [];
         foreach ($fieldsList as $fieldName => $fieldValue) {
             if ($fieldName != $idFieldName) {
-                $chunks[]= "`" . $fieldName . "` = '" . $fieldValue . "'";
+                $chunks[] = "`" . $fieldName . "` = '" . $fieldValue . "'";
             }
         }
 
@@ -243,11 +243,11 @@ 
         foreach ($fieldsList as $fieldName => $fieldValue) {
             if ($fieldName != $idFieldName) {
                 if (Tools::isDouble($fieldValue)) {
-                    $typesString.= "d";
+                    $typesString .= "d";
                 } elseif (Tools::isInteger($fieldValue)) {
-                    $typesString.= "i";
+                    $typesString .= "i";
                 } else {
-                    $typesString.= "s";
+                    $typesString .= "s";
                 }
             }
         }
@@ -268,7 +268,7 @@ 
         $type = DBField::castType($type);
         $typesString = "";
         while ($length > 0) {
-            $typesString.= $type;
+            $typesString .= $type;
             $length--;
         }
 
@@ -288,7 +288,7 @@ 
             if (!is_array($fieldValue)) {
                 if (!is_null($fieldValue)) {
                     $chunks[] = $fieldName . " = ?";
-                    $this->types.= DBField::getType($fieldValue);
+                    $this->types .= DBField::getType($fieldValue);
                     $this->params[] = $fieldValue;
                 } else {
                     $chunks[] = $fieldName;
@@ -299,12 +299,12 @@ 
 
                 $chunks[] = $condition;
                 foreach ($localParams as $param) {
-                    $this->types.= DBField::getType($param);
+                    $this->types .= DBField::getType($param);
                     $this->params[] = $param;
                 }
             }
         }
-        $this->query.= implode($separator, $chunks);
+        $this->query .= implode($separator, $chunks);
     }
 
 }