php-opencloud / openstack

src/Common/Error/Builder.php

Doc Comments +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
     /**
      * Internal method used when outputting headers in the error description.
      *
-     * @param $name
+     * @param string $name
      *
      * @return string
      */
@@ -58,7 +58,7 @@ 
      * Before outputting custom links, it is validated to ensure that the user is not
      * directed off to a broken link. If a 404 is detected, it is hidden.
      *
-     * @param $link The proposed link
+     * @param string $link The proposed link
      *
      * @return bool
      */

src/Common/JsonPath.php

Doc Comments +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
     /**
      * Set a node in the structure
      *
-     * @param $path  The XPath to use
+     * @param string $path  The XPath to use
      * @param $value The new value of the node
      */
     public function set($path, $value)

src/Common/Resource/HasWaiterTrait.php

Doc Comments +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
      *                              encapsulate your own logic to determine whether the resource has
      *                              successfully transitioned. When TRUE is returned by the callback,
      *                              the loop will end.
-     * @param int|bool $timeout     The maximum timeout in seconds. If the total time taken by the waiter has reached
+     * @param integer $timeout     The maximum timeout in seconds. If the total time taken by the waiter has reached
      *                              or exceed this timeout, the blocking operation will immediately cease. If FALSE
      *                              is provided, the timeout will never be considered.
      * @param int      $sleepPeriod The amount of time to pause between each HTTP request.
@@ -69,7 +69,7 @@ 
     /**
      * Internal method used to identify whether a timeout has been exceeded.
      *
-     * @param bool|int $timeout
+     * @param integer $timeout
      * @param int      $startTime
      *
      * @return bool
@@ -87,7 +87,7 @@ 
      * Convenience method providing a blocking operation until the resource transitions to an
      * ``ACTIVE`` status.
      *
-     * @param int|bool $timeout The maximum timeout in seconds. If the total time taken by the waiter has reached
+     * @param integer $timeout The maximum timeout in seconds. If the total time taken by the waiter has reached
      *                          or exceed this timeout, the blocking operation will immediately cease. If FALSE
      *                          is provided, the timeout will never be considered.
      */

src/ObjectStore/v1/Models/Container.php

Doc Comments +1 added lines, -1 removed lines

@@ -138,7 +138,7 @@
      *
      * @param string $name The name of the object
      *
-     * @return Object
+     * @return \OpenStack\Common\Resource\ResourceInterface
      */
     public function getObject($name)
     {

src/Common/Api/Parameter.php

Spacing +3 added lines, -3 removed lines

@@ -123,7 +123,7 @@ 
         }
 
         $this->location = $this->location ?: self::DEFAULT_LOCATION;
-        $this->required = (bool)$this->required;
+        $this->required = (bool) $this->required;
 
         if (isset($data['items'])) {
             $this->itemSchema = new Parameter($data['items']);
@@ -225,8 +225,8 @@ 
     private function hasCorrectType($userValue)
     {
         // Helper fn to see whether an array is associative (i.e. a JSON object)
-        $isAssociative = function ($value) {
-            return is_array($value) && (bool)count(array_filter(array_keys($value), 'is_string'));
+        $isAssociative = function($value) {
+            return is_array($value) && (bool) count(array_filter(array_keys($value), 'is_string'));
         };
 
         // For params defined as objects, we'll let the user get away with

src/Common/Resource/AbstractResource.php

Spacing +1 added lines, -1 removed lines

@@ -177,7 +177,7 @@
         $limit = isset($userVals['limit']) ? $userVals : false;
         $count = 0;
 
-        $totalReached = function ($count) use ($limit) {
+        $totalReached = function($count) use ($limit) {
             return $limit && $count >= $limit;
         };
 

Doc Comments +7 added lines, -1 removed lines

@@ -96,6 +96,9 @@ 
         }
     }
 
+    /**
+     * @param string $type
+     */
     private function parseDocBlockValue($type, $val)
     {
         if (strpos($type, '[]') === 0 && is_array($val)) {
@@ -120,6 +123,9 @@ 
         ]);
     }
 
+    /**
+     * @return string
+     */
     private function normalizeModelClass($class)
     {
         if (strpos($class, '\\') === false) {
@@ -166,7 +172,7 @@ 
     /**
      * @param array $definition
      *
-     * @return mixed
+     * @return ResponseInterface
      */
     public function executeWithState(array $definition)
     {

src/Common/Service/Builder.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
         }
 
         if (!isset($options['authHandler'])) {
-            $options['authHandler'] = function () use ($options) {
+            $options['authHandler'] = function() use ($options) {
                 return $options['identityService']->generateToken($options);
             };
         }

src/Common/Transport/Middleware.php

Spacing +4 added lines, -4 removed lines

@@ -13,10 +13,10 @@ 
 {
     public static function httpErrors()
     {
-        return function (callable $handler) {
-            return function ($request, array $options) use ($handler) {
+        return function(callable $handler) {
+            return function($request, array $options) use ($handler) {
                 return $handler($request, $options)->then(
-                    function (ResponseInterface $response) use ($request, $handler) {
+                    function(ResponseInterface $response) use ($request, $handler) {
                         if ($response->getStatusCode() < 400) {
                             return $response;
                         }
@@ -29,7 +29,7 @@ 
 
     public static function authHandler(callable $tokenGenerator, Token $token = null)
     {
-        return function (callable $handler) use ($tokenGenerator, $token) {
+        return function(callable $handler) use ($tokenGenerator, $token) {
             return new AuthHandler($handler, $tokenGenerator, $token);
         };
     }

src/Compute/v2/Models/Server.php

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use OpenStack\Common\Resource\Deletable;
 use OpenStack\Common\Resource\Listable;
 use OpenStack\Common\Resource\Retrievable;
-use OpenStack\Common\Resource\RetrievableInterface;
 use OpenStack\Common\Resource\Updateable;
 use OpenStack\Common\Resource\AbstractResource;
 use OpenStack\Common\Transport\Utils;