Code review

Author: haphan

src/BlockStorage/v2/Models/Volume.php

@@ -165,9 +165,8 @@ public function parseMetadata(ResponseInterface $response): array
      *
      * @param bool $bootable
      */
-    public function setBootable(bool $bootable)
+    public function setBootable(bool $bootable = true)
     {
-        $bootable = boolval($bootable);
         $this->execute($this->api->postVolumeBootable(), ['id' => $this->id, 'bootable' => $bootable]);
     }
 
@@ -186,15 +185,15 @@ public function setImageMetadata(array $metadata)
      *
      * @param array $options
      *
-     * $options['status']             = (string)    The volume status.
-     * $options['migrationStatus']   = (string)    The volume migration status.
-     * $options['attachStatus']      = (string)    The volume attach status.      [OPTIONAL]
+     * $options['status']          = (string) The volume status.
+     * $options['migrationStatus'] = (string) The volume migration status.
+     * $options['attachStatus']    = (string) The volume attach status.    [OPTIONAL]
      *
      * @see https://developer.openstack.org/api-ref/block-storage/v2/index.html#volume-actions-volumes-action
      */
     public function resetStatus(array $options)
     {
-        $options += ['id' => $this->id];
+        $options = array_merge($options, ['id' => $this->id]);
         $this->execute($this->api->postResetStatus(), $options);
     }
 }

src/ObjectStore/v1/Models/StorageObject.php

@@ -35,10 +35,10 @@ class StorageObject extends OperatorResource implements Creatable, Deletable, Ha
     /** @var string */
     public $contentType;
 
-    /** @var \DateTimeImmutable */
+    /** @var string */
     public $contentLength;
 
-    /** @var string */
+    /** @var \DateTimeImmutable */
     public $lastModified;
 
     /** @var array */