dedalozzo / eoc-client

src/EoC/Doc/TDoc.php

Braces +20 added lines, -14 removed lines

@@ -49,19 +49,22 @@ 
 
 
   public function setMetadata($name, $value, $override = TRUE, $allowNull = TRUE) {
-    if (is_null($value) && !$allowNull)
-      return;
+    if (is_null($value) && !$allowNull) {
+          return;
+    }
 
-    if ($this->isMetadataPresent($name) && !$override)
-      return;
+    if ($this->isMetadataPresent($name) && !$override) {
+          return;
+    }
 
     $this->meta[$name] = $value;
   }
 
 
   public function unsetMetadata($name) {
-    if (array_key_exists($name, $this->meta))
-      unset($this->meta[$name]);
+    if (array_key_exists($name, $this->meta)) {
+          unset($this->meta[$name]);
+    }
   }
 
 
@@ -95,9 +98,9 @@ 
     if (Helper\ArrayHelper::isAssociative($array)) {
       $this->meta = array_merge($this->meta, $array);
       $this->fixDocId();
+    } else {
+          throw new \InvalidArgumentException("\$array must be an associative array.");
     }
-    else
-      throw new \InvalidArgumentException("\$array must be an associative array.");
   }
 
 
@@ -114,8 +117,9 @@ 
         JSON_PRESERVE_ZERO_FRACTION
     );
 
-    if ($json === FALSE)
-      throw new JSONErrorException(json_last_error_msg());
+    if ($json === FALSE) {
+          throw new JSONErrorException(json_last_error_msg());
+    }
 
     return $json;
   }
@@ -157,8 +161,9 @@ 
 
 
   public function unsetId() {
-    if ($this->isMetadataPresent('_id'))
-      unset($this->meta['_id']);
+    if ($this->isMetadataPresent('_id')) {
+          unset($this->meta['_id']);
+    }
   }
 
 
@@ -178,8 +183,9 @@ 
 
 
   public function unsetRev() {
-    if ($this->isMetadataPresent('_rev'))
-      unset($this->meta['_rev']);
+    if ($this->isMetadataPresent('_rev')) {
+          unset($this->meta['_rev']);
+    }
   }
 
 }
\ No newline at end of file

src/EoC/Message/Response.php

Braces +31 added lines, -24 removed lines

@@ -423,13 +423,15 @@ 
    */
   protected function parseStatusCode($rawMessage) {
     $matches = [];
-    if (preg_match('%HTTP/1\.[0-1] (\d\d\d) %', $rawMessage, $matches))
-      $this->statusCode = $matches[1];
-    else
-      throw new \UnexpectedValueException("HTTP Status Code undefined.");
+    if (preg_match('%HTTP/1\.[0-1] (\d\d\d) %', $rawMessage, $matches)) {
+          $this->statusCode = $matches[1];
+    } else {
+          throw new \UnexpectedValueException("HTTP Status Code undefined.");
+    }
 
-    if (!array_key_exists($this->statusCode, self::$supportedStatusCodes))
-      throw new \UnexpectedValueException("HTTP Status Code unknown.");
+    if (!array_key_exists($this->statusCode, self::$supportedStatusCodes)) {
+          throw new \UnexpectedValueException("HTTP Status Code unknown.");
+    }
   }
 
 
@@ -449,10 +451,11 @@ 
             },
             strtolower(trim($matches[1])));
 
-        if (isset($this->header[$matches[1]]))
-          $this->header[$matches[1]] = array($this->header[$matches[1]], $matches[2]);
-        else
-          $this->header[$matches[1]] = trim($matches[2]);
+        if (isset($this->header[$matches[1]])) {
+                  $this->header[$matches[1]] = array($this->header[$matches[1]], $matches[2]);
+        } else {
+                  $this->header[$matches[1]] = trim($matches[2]);
+        }
       }
     }
   }
@@ -463,11 +466,13 @@ 
    * @param[in] string $rawMessage The raw message.
    */
   protected function parseStatusCodeAndHeader($rawMessage) {
-    if (!is_string($rawMessage))
-      throw new \InvalidArgumentException("\$rawMessage must be a string.");
+    if (!is_string($rawMessage)) {
+          throw new \InvalidArgumentException("\$rawMessage must be a string.");
+    }
 
-    if (empty($rawMessage))
-      throw new \UnexpectedValueException("\$rawMessage is null.");
+    if (empty($rawMessage)) {
+          throw new \UnexpectedValueException("\$rawMessage is null.");
+    }
 
     $this->parseStatusCode($rawMessage);
 
@@ -481,8 +486,9 @@ 
 
     $rawMessage = preg_split('/\r\n\r\n/', $rawMessage, 2);
 
-    if (empty($rawMessage))
-      throw new \RuntimeException("The server didn't return a valid Response for the Request.");
+    if (empty($rawMessage)) {
+          throw new \RuntimeException("The server didn't return a valid Response for the Request.");
+    }
 
     // $rawMessage[0] contains header fields.
     $this->parseHeaderFields($rawMessage[0]);
@@ -508,9 +514,9 @@ 
   public function setStatusCode($value) {
     if (array_key_exists($value, self::$supportedStatusCodes)) {
       $this->statusCode = $value;
+    } else {
+          throw new \UnexpectedValueException("Status Code $value is not supported.");
     }
-    else
-      throw new \UnexpectedValueException("Status Code $value is not supported.");
   }
 
 
@@ -529,12 +535,13 @@ 
    * @param[in] string $description A description for the Status Code.
    */
   public static function addCustomStatusCode($code, $description) {
-    if (in_array($code, self::$supportedStatusCodes))
-      throw new \UnexpectedValueException("Status Code $code is supported and already exists.");
-    elseif (is_int($code) and $code > 0)
-      self::$supportedStatusCodes[$code] = $description;
-    else
-      throw new \InvalidArgumentException("\$code must be a positive integer.");
+    if (in_array($code, self::$supportedStatusCodes)) {
+          throw new \UnexpectedValueException("Status Code $code is supported and already exists.");
+    } elseif (is_int($code) and $code > 0) {
+          self::$supportedStatusCodes[$code] = $description;
+    } else {
+          throw new \InvalidArgumentException("\$code must be a positive integer.");
+    }
   }
 
 }
\ No newline at end of file