dedalozzo / eoc-client

src/EoC/Info/DbInfo.php

Braces +2 added lines, -2 removed lines

@@ -82,9 +82,9 @@
       $this->purgeSeq = $info['purge_seq'];
       $this->compactRunning = $info['compact_running'];
       $this->committedUpdateSeq = $info['committed_update_seq'];
+    } else {
+          throw new \Exception("\$info must be an associative array.");
     }
-    else
-      throw new \Exception("\$info must be an associative array.");
   }
 
 

src/EoC/Doc/LocalDoc.php

Braces +3 added lines, -2 removed lines

@@ -25,8 +25,9 @@
    * @brief Removes `_local/` from he document identifier.
    */
   protected function fixDocId() {
-    if (isset($this->meta['_id']))
-      $this->meta['_id'] = preg_replace('%\A_local/%m', "", $this->meta['_id']);
+    if (isset($this->meta['_id'])) {
+          $this->meta['_id'] = preg_replace('%\A_local/%m', "", $this->meta['_id']);
+    }
   }
 
 

src/EoC/Doc/DesignDoc.php

Braces +55 added lines, -46 removed lines

@@ -63,8 +63,9 @@ 
    * @brief Removes `_design/` from he document identifier.
    */
   protected function fixDocId() {
-    if (isset($this->meta['_id']))
-      $this->meta['_id'] = preg_replace('%\A_design/%m', "", $this->meta['_id']);
+    if (isset($this->meta['_id'])) {
+          $this->meta['_id'] = preg_replace('%\A_design/%m', "", $this->meta['_id']);
+    }
   }
 
 
@@ -94,8 +95,9 @@ 
    */
   public function resetHandlers() {
     foreach ($this->sections as $name => $value) {
-      if (array_key_exists($name, $this->meta))
-        unset($this->meta[$name]);
+      if (array_key_exists($name, $this->meta)) {
+              unset($this->meta[$name]);
+      }
     }
   }
 
@@ -108,16 +110,17 @@ 
    */
   public function getHandlerAttributes($section, $name = "") {
     if (empty($name)) { // The handler doesn't have a name.
-      if (array_key_exists($section, $this->meta))
-        return $this->meta[$section];
-      else
-        throw new \Exception(sprintf("Can't find '%s' handler in the design document.", $section));
-    }
-    else { // The handler has a name.
-      if (@array_key_exists($name, $this->meta[$section]))
-        return $this->meta[$section][$name];
-      else
-        throw new \Exception(sprintf("Can't find '%s' handler in the design document '%s' section.", $name, $section));
+      if (array_key_exists($section, $this->meta)) {
+              return $this->meta[$section];
+      } else {
+              throw new \Exception(sprintf("Can't find '%s' handler in the design document.", $section));
+      }
+    } else { // The handler has a name.
+      if (@array_key_exists($name, $this->meta[$section])) {
+              return $this->meta[$section][$name];
+      } else {
+              throw new \Exception(sprintf("Can't find '%s' handler in the design document '%s' section.", $name, $section));
+      }
     }
   }
 
@@ -133,22 +136,25 @@ 
     $section = $handler->getSection();
 
     if (property_exists($handler, "name")) {
-      if (!$handler->isConsistent())
-        throw new \Exception(sprintf("The '%s' handler '%s' is not consistent.", $section, $handler->name));
-
-      if (@array_key_exists($handler->name, $this->meta[$section]))
-        throw new \Exception(sprintf("The '%s' handler '%s' already exists.", $section, $handler->name));
-      else
-        $this->meta[$section][$handler->name] = $handler->asArray();
-    }
-    else {
-      if (!$handler->isConsistent())
-        throw new \Exception(sprintf("The '%s' handler is not consistent.", $section));
-
-      if (array_key_exists($section, $this->meta))
-        throw new \Exception(sprintf("The '%s' handler already exists.", $section));
-      else
-        $this->meta[$section] = $handler;
+      if (!$handler->isConsistent()) {
+              throw new \Exception(sprintf("The '%s' handler '%s' is not consistent.", $section, $handler->name));
+      }
+
+      if (@array_key_exists($handler->name, $this->meta[$section])) {
+              throw new \Exception(sprintf("The '%s' handler '%s' already exists.", $section, $handler->name));
+      } else {
+              $this->meta[$section][$handler->name] = $handler->asArray();
+      }
+    } else {
+      if (!$handler->isConsistent()) {
+              throw new \Exception(sprintf("The '%s' handler is not consistent.", $section));
+      }
+
+      if (array_key_exists($section, $this->meta)) {
+              throw new \Exception(sprintf("The '%s' handler already exists.", $section));
+      } else {
+              $this->meta[$section] = $handler;
+      }
     }
   }
 
@@ -163,16 +169,17 @@ 
    */
   public function removeHandler($section, $name = "") {
     if (empty($name)) { // The handler doesn't have a name.
-      if (array_key_exists($section, $this->meta))
-        unset($this->meta[$section]);
-      else
-        throw new \Exception(sprintf("Can't find the '%s' handler.", $section));
-    }
-    else { // The handler has a name.
-      if (@array_key_exists($name, $this->meta[$section]))
-        unset($this->meta[$section][$name]);
-      else
-        throw new \Exception(sprintf("Can't find the '%s' handler '%s'", $section, $name));
+      if (array_key_exists($section, $this->meta)) {
+              unset($this->meta[$section]);
+      } else {
+              throw new \Exception(sprintf("Can't find the '%s' handler.", $section));
+      }
+    } else { // The handler has a name.
+      if (@array_key_exists($name, $this->meta[$section])) {
+              unset($this->meta[$section][$name]);
+      } else {
+              throw new \Exception(sprintf("Can't find the '%s' handler '%s'", $section, $name));
+      }
     }
   }
 
@@ -190,16 +197,18 @@ 
 
 
   public function setLanguage($value) {
-    if (!empty($value))
-      $this->meta['language'] = strtolower((string)$value);
-    else
-      throw new \InvalidArgumentException("\$language must be a non-empty string.");
+    if (!empty($value)) {
+          $this->meta['language'] = strtolower((string)$value);
+    } else {
+          throw new \InvalidArgumentException("\$language must be a non-empty string.");
+    }
   }
 
 
   public function unsetLanguage() {
-    if ($this->isMetadataPresent('language'))
-      unset($this->meta['language']);
+    if ($this->isMetadataPresent('language')) {
+          unset($this->meta['language']);
+    }
   }
 
   //! @endcond

src/EoC/Doc/DocRef.php

Braces +5 added lines, -4 removed lines

@@ -59,10 +59,11 @@
 
 
   public function setId($value) {
-    if (!empty($value))
-      $this->id = (string)$value;
-    else
-      throw new \Exception("\$id must be a non-empty string.");
+    if (!empty($value)) {
+          $this->id = (string)$value;
+    } else {
+          throw new \Exception("\$id must be a non-empty string.");
+    }
   }
 
 

src/EoC/Doc/Attachment/Attachment.php

Braces +15 added lines, -12 removed lines

@@ -41,8 +41,9 @@ 
     $instance = new self();
 
     if (file_exists($fileName)) {
-      if (is_dir($fileName))
-        throw new \Exception("The file $fileName is a directory.");
+      if (is_dir($fileName)) {
+              throw new \Exception("The file $fileName is a directory.");
+      }
 
       $instance->name = basename($fileName);
 
@@ -58,14 +59,15 @@ 
         finfo_close($finfo);
         fclose($fd);
 
-        if ($instance->data === FALSE)
-          throw new \Exception("Error reading the file $fileName.");
+        if ($instance->data === FALSE) {
+                  throw new \Exception("Error reading the file $fileName.");
+        }
+      } else {
+              throw new \Exception("Cannot open the file $fileName.");
       }
-      else
-        throw new \Exception("Cannot open the file $fileName.");
+    } else {
+          throw new \Exception("The file $fileName doesn't exist.");
     }
-    else
-      throw new \Exception("The file $fileName doesn't exist.");
 
     return $instance;
   }
@@ -101,11 +103,12 @@ 
       $bytes = fwrite($fd, $this->data);
       fclose($fd);
 
-      if ($bytes === FALSE)
-        throw new \Exception("Error writing the file `$this->name`.");
+      if ($bytes === FALSE) {
+              throw new \Exception("Error writing the file `$this->name`.");
+      }
+    } else {
+          throw new \Exception("Cannot create the file `$this->name`.");
     }
-    else
-      throw new \Exception("Cannot create the file `$this->name`.");
   }
 
 

src/EoC/Generator/UUID.php

Braces +34 added lines, -25 removed lines

@@ -131,17 +131,18 @@ 
 
   /* Auto-detect UUID format */
   static private function detectFormat($src) {
-    if (is_string($src))
-      return self::FMT_STRING;
-    else if (is_array($src)) {
+    if (is_string($src)) {
+          return self::FMT_STRING;
+    } else if (is_array($src)) {
       $len = count($src);
-      if ($len == 1 || ($len % 2) == 0)
-        return $len;
-      else
-        return (-1);
+      if ($len == 1 || ($len % 2) == 0) {
+              return $len;
+      } else {
+              return (-1);
+      }
+    } else {
+          return self::FMT_BINARY;
     }
-    else
-      return self::FMT_BINARY;
   }
 
   /*
@@ -151,8 +152,9 @@ 
   static public function generate($type, $fmt = self::FMT_BYTE,
                                   $node = "", $ns = "") {
     $func = self::$m_generate[$type];
-    if (!isset($func))
-      return null;
+    if (!isset($func)) {
+          return null;
+    }
     $conv = self::$m_convert[self::FMT_FIELD][$fmt];
 
     $uuid = self::$func($ns, $node);
@@ -164,8 +166,9 @@ 
    */
   static public function convert($uuid, $from, $to) {
     $conv = self::$m_convert[$from][$to];
-    if (!isset($conv))
-      return ($uuid);
+    if (!isset($conv)) {
+          return ($uuid);
+    }
 
     return (self::$conv($uuid));
   }
@@ -181,8 +184,9 @@ 
     $uuid['time_low'] = mt_rand(0, 0xffff) + (mt_rand(0, 0xffff) << 16);
     $uuid['time_mid'] = mt_rand(0, 0xffff);
     $uuid['clock_seq_low'] = mt_rand(0, 255);
-    for ($i = 0; $i < 6; $i++)
-      $uuid['node'][$i] = mt_rand(0, 255);
+    for ($i = 0; $i < 6; $i++) {
+          $uuid['node'][$i] = mt_rand(0, 255);
+    }
     return ($uuid);
   }
 
@@ -205,8 +209,9 @@ 
     /* Hash the namespace and node and convert to a byte array */
     $val = $hash($raw, true);
     $tmp = unpack('C16', $val);
-    foreach (array_keys($tmp) as $key)
-      $byte[$key - 1] = $tmp[$key];
+    foreach (array_keys($tmp) as $key) {
+          $byte[$key - 1] = $tmp[$key];
+    }
 
     /* Convert byte array to a field array */
     $field = self::conv_byte2field($byte);
@@ -264,8 +269,9 @@ 
       * Node should be set to the 48-bit IEEE node identifier, but
       * we leave it for the user to supply the node.
       */
-    for ($i = 0; $i < 6; $i++)
-      $uuid['node'][$i] = ord(substr($node, $i, 1));
+    for ($i = 0; $i < 6; $i++) {
+          $uuid['node'][$i] = ord(substr($node, $i, 1));
+    }
 
     return ($uuid);
   }
@@ -283,8 +289,9 @@ 
     $uuid[8] = $src['clock_seq_hi'];
     $uuid[9] = $src['clock_seq_low'];
 
-    for ($i = 0; $i < 6; $i++)
-      $uuid[10+$i] = $src['node'][$i];
+    for ($i = 0; $i < 6; $i++) {
+          $uuid[10+$i] = $src['node'][$i];
+    }
 
     return ($uuid);
   }
@@ -313,8 +320,9 @@ 
     $field['clock_seq_hi'] = $uuid[8];
     $field['clock_seq_low'] = $uuid[9];
 
-    for ($i = 0; $i < 6; $i++)
-      $field['node'][$i] = $uuid[10+$i];
+    for ($i = 0; $i < 6; $i++) {
+          $field['node'][$i] = $uuid[10+$i];
+    }
     return ($field);
   }
 
@@ -338,8 +346,9 @@ 
     $field['time_hi'] = ($parts[2]);
     $field['clock_seq_hi'] = ($parts[3] & 0xff00) >> 8;
     $field['clock_seq_low'] = $parts[3] & 0x00ff;
-    for ($i = 0; $i < 6; $i++)
-      $field['node'][$i] = $parts[4+$i];
+    for ($i = 0; $i < 6; $i++) {
+          $field['node'][$i] = $parts[4+$i];
+    }
 
     return ($field);
   }

src/EoC/Handler/ViewHandler.php

Braces +18 added lines, -12 removed lines

@@ -144,8 +144,9 @@ 
   public static function checkFn($fnImpl, $fnDef, $fnRegex) {
     Lint::checkSourceCode($fnImpl);
 
-    if (!preg_match($fnRegex, $fnImpl))
-      throw new \Exception("The \$closure must be defined like: $fnDef");
+    if (!preg_match($fnRegex, $fnImpl)) {
+          throw new \Exception("The \$closure must be defined like: $fnDef");
+    }
   }
 
 
@@ -153,14 +154,17 @@ 
     $view = [];
     $view['map'] = $this->mapFn;
 
-    if (!empty($this->language))
-      $view['language'] = $this->language;
+    if (!empty($this->language)) {
+          $view['language'] = $this->language;
+    }
 
-    if (!empty($this->reduceFn))
-      $view['reduce'] = $this->reduceFn;
+    if (!empty($this->reduceFn)) {
+          $view['reduce'] = $this->reduceFn;
+    }
 
-    if (!empty($this->options))
-      $view['options'] = $this->options;
+    if (!empty($this->options)) {
+          $view['options'] = $this->options;
+    }
 
     return $view;
   }
@@ -225,8 +229,9 @@ 
   public function setMapFn($value) {
     $fn = stripslashes((string)$value);
 
-    if ($this->language == "php")
-      self::checkFn($fn, self::MAP_DEFINITION, self::MAP_REGEX);
+    if ($this->language == "php") {
+          self::checkFn($fn, self::MAP_DEFINITION, self::MAP_REGEX);
+    }
 
     $this->mapFn = $fn;
   }
@@ -240,8 +245,9 @@ 
   public function setReduceFn($value) {
     $fn = stripslashes((string)$value);
 
-    if ($this->language == "php")
-      self::checkFn($fn, self::REDUCE_DEFINITION, self::REDUCE_REGEX);
+    if ($this->language == "php") {
+          self::checkFn($fn, self::REDUCE_DEFINITION, self::REDUCE_REGEX);
+    }
 
     $this->reduceFn = $fn;
   }

src/EoC/Opt/ViewQueryOpts.php

Braces +15 added lines, -12 removed lines

@@ -36,8 +36,9 @@ 
    * @attention Chainable.
    */
   public function unsetOpt($name) {
-    if (array_key_exists($name, $this->options))
-      unset($this->options['name']);
+    if (array_key_exists($name, $this->options)) {
+          unset($this->options['name']);
+    }
 
     return $this;
   }
@@ -137,10 +138,11 @@ 
    * @attention Chainable.
    */
   public function setLimit($value) {
-    if (is_int($value) && $value >= 0)
-      $this->options["limit"] = $value;
-    else
-      throw new \Exception("\$value must be a non-negative integer.");
+    if (is_int($value) && $value >= 0) {
+          $this->options["limit"] = $value;
+    } else {
+          throw new \Exception("\$value must be a non-negative integer.");
+    }
 
     return $this;
   }
@@ -173,9 +175,9 @@ 
     if (is_int($value) && $value > 0) {
       $this->options["group"] = "false"; // This parameter is used only if 'group' is 'false'.
       $this->options["group_level"] = $value;
+    } else {
+          throw new \Exception("\$value must be a positive integer.");
     }
-    else
-      throw new \Exception("\$value must be a positive integer.");
 
     return $this;
   }
@@ -311,10 +313,11 @@ 
    * @attention Chainable.
    */
   public function skipDocs($number) {
-    if (is_int($number) && $number > 0)
-      $this->options["skip"] = $number;
-    else
-      throw new \Exception("\$number must be a positive integer.");
+    if (is_int($number) && $number > 0) {
+          $this->options["skip"] = $number;
+    } else {
+          throw new \Exception("\$number must be a positive integer.");
+    }
 
     return $this;
   }

src/EoC/Doc/AbstractDoc.php

Braces +6 added lines, -4 removed lines

@@ -149,8 +149,9 @@ 
 
 
   public function unsetId() {
-    if ($this->isMetadataPresent('_id'))
-      unset($this->meta['_id']);
+    if ($this->isMetadataPresent('_id')) {
+          unset($this->meta['_id']);
+    }
   }
 
 
@@ -170,8 +171,9 @@ 
 
 
   public function unsetRev() {
-    if ($this->isMetadataPresent('_rev'))
-      unset($this->meta['_rev']);
+    if ($this->isMetadataPresent('_rev')) {
+          unset($this->meta['_rev']);
+    }
   }
 
   //! @endcond