alphadevx / alpha

Alpha/Controller/ListActiveRecordsController.php

Doc Comments +2 added lines, -2 removed lines

@@ -96,7 +96,7 @@ 
      *
      * @param alpha\Util\Http\Request $request
      *
-     * @return alpha\Util\Http\Response
+     * @return Response
      *
      * @since 1.0
      */
@@ -120,7 +120,7 @@ 
      *
      * @param alpha\Util\Http\Request $request
      *
-     * @return alpha\Util\Http\Response
+     * @return Response
      *
      * @since 1.0
      */

Alpha/Model/ActiveRecord.php

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use Alpha\Model\Type\Integer;
 use Alpha\Model\Type\Timestamp;
-use Alpha\Model\Type\Enum;
 use Alpha\Model\Type\Relation;
 use Alpha\Util\Config\ConfigProvider;
 use Alpha\Util\Logging\Logger;

Doc Comments +6 added lines, -6 removed lines

@@ -1162,7 +1162,7 @@ 
      * to be overridden in case you want to do something different with the ID of your objects outside
      * of the standard 11 digit OID sequence used internally in Alpha.
      *
-     * @return int 11 digit zero-padded OID value.
+     * @return string 11 digit zero-padded OID value.
      *
      * @since 1.0
      */
@@ -1179,7 +1179,7 @@ 
      * Gets the OID for the object in zero-padded format (same as getID()).  This version is final so cannot
      * be overridden.
      *
-     * @return int 11 digit zero-padded OID value.
+     * @return string 11 digit zero-padded OID value.
      *
      * @since 1.0
      */
@@ -2176,7 +2176,7 @@ 
     /**
      * Starts a new database transaction.
      *
-     * @param $BO The ActiveRecord instance to pass to the database provider. Leave empty to have a new Person passed.
+     * @param ActiveRecord $BO The ActiveRecord instance to pass to the database provider. Leave empty to have a new Person passed.
      *
      * @since 1.0
      *
@@ -2209,7 +2209,7 @@ 
     /**
      * Commits the current database transaction.
      *
-     * @param $BO The ActiveRecord instance to pass to the database provider. Leave empty to have a new Person passed.
+     * @param ActiveRecord $BO The ActiveRecord instance to pass to the database provider. Leave empty to have a new Person passed.
      *
      * @since 1.0
      *
@@ -2242,7 +2242,7 @@ 
     /**
      * Aborts the current database transaction.
      *
-     * @param $BO The ActiveRecord instance to pass to the database provider. Leave empty to have a new Person passed.
+     * @param ActiveRecord $BO The ActiveRecord instance to pass to the database provider. Leave empty to have a new Person passed.
      *
      * @since 1.0
      *
@@ -2413,7 +2413,7 @@ 
     /**
      * Check to see if an attribute exists on the BO.
      *
-     * @param $attribute The attribute name.
+     * @param string $attribute The attribute name.
      *
      * @return bool
      *

Alpha/Model/ActiveRecordProviderMySQL.php

Doc Comments +1 added lines, -1 removed lines

@@ -2383,7 +2383,7 @@
      * parameters.  If arrays of attribute names and values are provided, only those will be bound to
      * the supplied statement.
      *
-     * @param mysqli_stmt $stmt The SQL statement to bind to.
+     * @param \mysqli_stmt $stmt The SQL statement to bind to.
      * @param array Optional array of BO attributes.
      * @param array Optional array of BO values.
      *

Spacing +4 added lines, -4 removed lines

@@ -1033,7 +1033,7 @@ 
         } else {
             // there has been an error, so decrement the version number back
             $temp = $this->BO->getVersionNumber()->getValue();
-            $this->BO->set('version_num', $temp - 1);
+            $this->BO->set('version_num', $temp-1);
 
             // check for unique violations
             if (self::getConnection()->errno == '1062') {
@@ -1065,7 +1065,7 @@ 
         $this->BO->setLastQuery($sqlQuery);
         $stmt = self::getConnection()->stmt_init();
 
-        $newVersionNumber = $this->BO->getVersionNumber()->getValue() + 1;
+        $newVersionNumber = $this->BO->getVersionNumber()->getValue()+1;
 
         if ($stmt->prepare($sqlQuery)) {
             if ($this->BO->getPropObject($attribute) instanceof Integer) {
@@ -2441,7 +2441,7 @@ 
 
                     if ($propName == 'version_num') {
                         $temp = $this->BO->getVersionNumber()->getValue();
-                        $this->BO->set('version_num', $temp + 1);
+                        $this->BO->set('version_num', $temp+1);
                         $bindingsTypes .= 'i';
                         array_push($params, $this->BO->getVersionNumber()->getValue());
                     }
@@ -2542,7 +2542,7 @@ 
         $singleQuote1 = mb_strpos($error, "'");
         $singleQuote2 = mb_strrpos($error, "'");
 
-        $value = mb_substr($error, $singleQuote1, ($singleQuote2 - $singleQuote1) + 1);
+        $value = mb_substr($error, $singleQuote1, ($singleQuote2-$singleQuote1)+1);
         self::$logger->debug('<<findOffendingValue ['.$value.'])');
 
         return $value;

Alpha/Model/Article.php

Doc Comments +1 added lines, -1 removed lines

@@ -435,7 +435,7 @@
     /**
      * Method for returning the calculated score for this article.
      *
-     * @return float
+     * @return string
      *
      * @since 1.0
      */

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 use Alpha\Model\Type\Boolean;
 use Alpha\Model\Type\Relation;
 use Alpha\Util\Logging\Logger;
-use Alpha\Util\Config\Configprovider;
 use Alpha\Util\Http\Session\SessionProviderFactory;
 use Alpha\Exception\ValidationException;
 use Alpha\Exception\FileNotFoundException;

Spacing +1 added lines, -1 removed lines

@@ -452,7 +452,7 @@
         }
 
         if ($vote_count > 0) {
-            $score = $total_score / $vote_count;
+            $score = $total_score/$vote_count;
         }
 
         return sprintf('%01.2f', $score);

Alpha/Util/Config/ConfigProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
     /**
      * Get config value.
      *
-     * @param $key string
+     * @param string $key string
      *
      * @return string
      *

Alpha/Util/File/FileUtils.php

Doc Comments +2 added lines, -2 removed lines

@@ -541,7 +541,7 @@ 
      * @param string $sourceDir    The path to the source directory.
      * @param string $fileList     The HTML list of files generated (pass by reference).
      * @param int    $fileCount    The current file count (used in recursive calls).
-     * @param array  $excludeFiles An array of file names to exclude from the list rendered.
+     * @param string[]  $excludeFiles An array of file names to exclude from the list rendered.
      *
      * @return int The current filecount for the directory.
      *
@@ -578,7 +578,7 @@ 
      * Recursively deletes the contents of the directory indicated (the directory itself is not deleted).
      *
      * @param string $sourceDir    The path to the source directory.
-     * @param array  $excludeFiles An array of file names to exclude from the deletion.
+     * @param string[]  $excludeFiles An array of file names to exclude from the deletion.
      *
      * @throws Alpha\Exception\AlphaException
      *

Alpha/Util/Graph/GraphNode.php

Doc Comments +2 added lines, -2 removed lines

@@ -284,7 +284,7 @@ 
     /**
      * Get the node offset.
      *
-     * @return string
+     * @return integer
      *
      * @since 1.0
      */
@@ -460,7 +460,7 @@ 
     /**
      * Calculates and returns the midpoint X coordinate of the children of this node.
      *
-     * @return int
+     * @return double
      *
      * @since 1.0
      */

Spacing +6 added lines, -6 removed lines

@@ -467,9 +467,9 @@ 
     public function getChildrenCenter()
     {
         $node = $this->getChildAt(0);
-        $node1 = $this->getChildAt(count($this->children) - 1);
+        $node1 = $this->getChildAt(count($this->children)-1);
 
-        return $node->getOffset() + (($node1->getOffset() - $node->getOffset()) + $node1->getWidth()) / 2;
+        return $node->getOffset()+(($node1->getOffset()-$node->getOffset())+$node1->getWidth())/2;
     }
 
     /**
@@ -529,14 +529,14 @@ 
         $yc = 0;
         $xd = 0;
         $yd = 0;
-        $xa = $this->x + ($this->width / 2);
-        $ya = $this->y + $this->height;
+        $xa = $this->x+($this->width/2);
+        $ya = $this->y+$this->height;
 
         foreach ($this->children as $child) {
-            $xd = $xc = $child->getX() + ($child->getWidth() / 2);
+            $xd = $xc = $child->getX()+($child->getWidth()/2);
             $yd = $child->getY();
             $xb = $xa;
-            $yb = $yc = $ya + ($yd - $ya) / 2;
+            $yb = $yc = $ya+($yd-$ya)/2;
             $this->links[$child->id]['xa'] = $xa;
             $this->links[$child->id]['ya'] = $ya;
             $this->links[$child->id]['xb'] = $xb;

Alpha/Util/Http/Session/SessionProviderInterface.php

Doc Comments +5 added lines, -2 removed lines

@@ -50,6 +50,7 @@ 
      * Starts a new session, or resumes an existing session if there is already a session ID available.
      *
      * @since 2.0
+     * @return void
      */
     public function init();
 
@@ -57,13 +58,14 @@ 
      * Destroys the current session.
      *
      * @since 2.0
+     * @return void
      */
     public function destroy();
 
     /**
      * Get the key value from the session.  Returns false if nothing is found.
      *
-     * @param $key
+     * @param string $key
      *
      * @since 2.0
      *
@@ -78,6 +80,7 @@ 
      * @param mixed  $value
      *
      * @since 2.0
+     * @return void
      */
     public function set($key, $value);
 
@@ -85,9 +88,9 @@ 
      * Deletes the value provided at that key in the session.
      *
      * @param string $key
-     * @param mixed  $value
      *
      * @since 2.0
+     * @return void
      */
     public function delete($key);
 

Alpha/Util/Image/ImageUtils.php

Doc Comments +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
     /**
      * Saves the GD image resource to the file path indicated.
      *
-     * @param image  $imageResource The GD image resource to save
+     * @param resource  $imageResource The GD image resource to save
      * @param string $type          The image type (jpg, png, or gif)
      * @param string $destination   The desination file path of the image file to create
      *

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
         list($originalWidth, $originalHeight) = $imageInfo;
 
         if ($originalWidth > $originalHeight) {
-            $originalX = floor(($originalWidth - $originalHeight) / 2);
+            $originalX = floor(($originalWidth-$originalHeight)/2);
             $sourceWidth = $sourceHeight = $originalHeight;
         } else {
             $sourceWidth = $sourceHeight = $originalWidth;