php-opencloud / openstack

src/Common/Api/Operator.php

Doc Comments +3 added lines

@@ -64,6 +64,9 @@
         return new Operation($definition);
     }
 
+    /**
+     * @return ResponseInterface
+     */
     protected function sendRequest(Operation $operation, array $userValues = [], $async = false)
     {
         $uri     = uri_template($operation->getPath(), $userValues);

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/AbstractResource.php

Doc Comments +9 added lines, -1 removed lines

@@ -105,6 +105,9 @@ 
         }
     }
 
+    /**
+     * @param string $type
+     */
     private function isNotNativeType($type)
     {
         return !in_array($type, [
@@ -112,6 +115,11 @@ 
         ]);
     }
 
+    /**
+     * @param string $class
+     *
+     * @return string
+     */
     private function normalizeModelClass($class)
     {
         if (strpos($class, '\\') === false) {
@@ -158,7 +166,7 @@ 
     /**
      * @param array $definition
      *
-     * @return mixed
+     * @return ResponseInterface
      */
     public function executeWithState(array $definition)
     {

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

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/ObjectStore/v1/Models/Object.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
     /**
      * @param array $data {@see \OpenStack\ObjectStore\v1\Api::putObject}
      *
-     * @return $this|\OpenStack\Common\Resource\ResourceInterface|void
+     * @return Object|null
      */
     public function create(array $data)
     {

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