dedalozzo / eoc-cli

src/EoC/CLI/Command/QueryCommand.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-
 use EoC\Opt\ViewQueryOpts;
 
 

Indentation +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     %i - the argument is treated as an integer.
     %f - the argument is treated as a float.
 DESC
-   );
+    );
 
     // General options.
     $this->addOption("key",

Braces +70 added lines, -52 removed lines

@@ -208,81 +208,98 @@ 
       $args = $input->getArgument('keys');
 
       $keys = [];
-      foreach ($args as $key)
-        $keys[] = $this->castArg($key, FALSE);
+      foreach ($args as $key) {
+              $keys[] = $this->castArg($key, FALSE);
+      }
+    } else {
+          $keys = NULL;
     }
-    else
-      $keys = NULL;
 
     // Sets the options.
     $opts = new ViewQueryOpts();
 
     // Key.
-    if ($key = $input->getOption('key'))
-      $opts->setKey($this->castArg($key));
+    if ($key = $input->getOption('key')) {
+          $opts->setKey($this->castArg($key));
+    }
 
     // Start key.
-    if ($startkey = $input->getOption('startkey'))
-      $opts->setStartKey($this->castArg($startkey));
+    if ($startkey = $input->getOption('startkey')) {
+          $opts->setStartKey($this->castArg($startkey));
+    }
 
     // End key.
-    if ($endkey = $input->getOption('endkey'))
-      $opts->setEndKey($this->castArg($endkey));
+    if ($endkey = $input->getOption('endkey')) {
+          $opts->setEndKey($this->castArg($endkey));
+    }
 
     // Start doc id.
-    if ($docId = $input->getOption('startkey-docid'))
-      $opts->setStartDocId($docId);
+    if ($docId = $input->getOption('startkey-docid')) {
+          $opts->setStartDocId($docId);
+    }
 
     // End doc id.
-    if ($docId = $input->getOption('endkey-docid'))
-      $opts->setEndDocId($docId);
+    if ($docId = $input->getOption('endkey-docid')) {
+          $opts->setEndDocId($docId);
+    }
 
     // Limit.
     $limit = (int)$input->getOption('limit');
-    if ($limit > 0)
-      $opts->setLimit($limit);
+    if ($limit > 0) {
+          $opts->setLimit($limit);
+    }
 
     // Group results.
-    if ($input->getOption('group-results'))
-      $opts->groupResults();
+    if ($input->getOption('group-results')) {
+          $opts->groupResults();
+    }
 
     // Sets group level.
     $level = (int)$input->getOption('group-level');
-    if ($level > 0)
-      $opts->setGroupLevel($level);
+    if ($level > 0) {
+          $opts->setGroupLevel($level);
+    }
 
     // Do not reduce.
-    if ($input->getOption('do-not-reduce'))
-      $opts->doNotReduce();
+    if ($input->getOption('do-not-reduce')) {
+          $opts->doNotReduce();
+    }
 
     // Includes docs.
-    if ($input->getOption('include-docs'))
-      $opts->includeDocs();
+    if ($input->getOption('include-docs')) {
+          $opts->includeDocs();
+    }
 
     // Excludes results.
-    if ($input->getOption('exclude-results'))
-      $opts->excludeResults();
+    if ($input->getOption('exclude-results')) {
+          $opts->excludeResults();
+    }
 
     // Excludes endkey.
-    if ($input->getOption('exclude-endkey'))
-      $opts->excludeEndKey();
+    if ($input->getOption('exclude-endkey')) {
+          $opts->excludeEndKey();
+    }
 
     // Reverses order of results.
-    if ($input->getOption('reverse-order'))
-      $opts->reverseOrderOfResults();
+    if ($input->getOption('reverse-order')) {
+          $opts->reverseOrderOfResults();
+    }
 
     // Skips the defined number of documents.
     $skip = (int)$input->getOption('skip');
-    if ($skip > 0)
-      $opts->skipDocs($skip);
+    if ($skip > 0) {
+          $opts->skipDocs($skip);
+    }
 
     // Includes conflicts.
-    if ($input->getOption('include-conflicts'))
-      $opts->includeConflicts();
+    if ($input->getOption('include-conflicts')) {
+          $opts->includeConflicts();
+    }
 
     // Includes missing keys.
-    if ($input->getOption('include-missing-keys'))
-      $opts->includeMissingKeys();
+    if ($input->getOption('include-missing-keys')) {
+          $opts->includeMissingKeys();
+    }
 
     if ($view == "_temp_view") {
 
@@ -290,30 +307,31 @@ 
       if ($fileName = $input->getOption('map')) {
         $map = file_get_contents($fileName);
 
-        if ($fileName = $input->getOption('reduce'))
-          $reduce = file_get_contents($fileName);
-        else
-          $reduce = "";
+        if ($fileName = $input->getOption('reduce')) {
+                  $reduce = file_get_contents($fileName);
+        } else {
+                  $reduce = "";
+        }
 
         $language = $input->getOption('language');
-        if (empty($language))
-          $language = "php";
+        if (empty($language)) {
+                  $language = "php";
+        }
+      } else {
+              throw new \InvalidArgumentException("You didn't provided the map function.");
       }
-      else
-        throw new \InvalidArgumentException("You didn't provided the map function.");
 
       print_r($couch->queryTempView($map, $reduce, $keys, $opts, $language));
-    }
-    elseif ($view == "_all_docs") {
+    } elseif ($view == "_all_docs") {
       print_r($couch->queryAllDocs($keys, $opts));
-    }
-    else {
+    } else {
       $names = explode('/', $view, 2);
 
-      if (count($names) == 2)
-        var_dump($couch->queryView($names[0], $names[1], $keys, $opts));
-      else
-        throw new \InvalidArgumentException("You have to specify design-doc/view-name.");
+      if (count($names) == 2) {
+              var_dump($couch->queryView($names[0], $names[1], $keys, $opts));
+      } else {
+              throw new \InvalidArgumentException("You have to specify design-doc/view-name.");
+      }
     }
 
   }

src/EoC/CLI/Command/UseCommand.php

Braces +7 added lines, -6 removed lines

@@ -45,20 +45,21 @@
 
     if ($shmKey = ftok($_SERVER['PHP_SELF'], 'd')) {
 
-      if (@$shmId = shmop_open($shmKey, 'a', 0644, 0))
-        shmop_delete($shmId);
+      if (@$shmId = shmop_open($shmKey, 'a', 0644, 0)) {
+              shmop_delete($shmId);
+      }
 
       $shmId = shmop_open($shmKey, 'n', 0644, mb_strlen($database));
 
       if ($shmId) {
         $shmBytesWritten = shmop_write($shmId, $database, 0);
         shmop_close($shmId);
+      } else {
+              throw new \RuntimeException("Couldn't create shared memory segment.");
       }
-      else
-        throw new \RuntimeException("Couldn't create shared memory segment.");
+    } else {
+          throw new \RuntimeException("Cannot get a System V IPC key.");
     }
-    else
-      throw new \RuntimeException("Cannot get a System V IPC key.");
   }
 
 }
\ No newline at end of file

src/EoC/CLI/Command/AllDbsCommand.php

Braces +3 added lines, -2 removed lines

@@ -41,8 +41,9 @@
     $couch = $this->getConnection();
 
     $dbs = $couch->getAllDbs();
-    foreach ($dbs as $db)
-      echo $db.PHP_EOL;
+    foreach ($dbs as $db) {
+          echo $db.PHP_EOL;
+    }
   }
 
 }
\ No newline at end of file

src/EoC/CLI/Command/ConnectCommand.php

Braces +10 added lines, -8 removed lines

@@ -74,26 +74,28 @@
 
     // We reset the database in use.
     if ($shmKey = ftok($_SERVER['PHP_SELF'], 'd')) {
-      if (@$shmId = shmop_open($shmKey, "a", 0644, 0))
-        shmop_delete($shmId);
+      if (@$shmId = shmop_open($shmKey, "a", 0644, 0)) {
+              shmop_delete($shmId);
+      }
     }
 
     if ($shmKey = ftok($_SERVER['PHP_SELF'], 'c')) {
 
-      if (@$shmId = shmop_open($shmKey, 'a', 0644, 0))
-        shmop_delete($shmId);
+      if (@$shmId = shmop_open($shmKey, 'a', 0644, 0)) {
+              shmop_delete($shmId);
+      }
 
       $shmId = shmop_open($shmKey, 'n', 0644, mb_strlen($serialized));
 
       if ($shmId) {
         $shmBytesWritten = shmop_write($shmId, $serialized, 0);
         shmop_close($shmId);
+      } else {
+              throw new \RuntimeException("Couldn't create shared memory segment.");
       }
-      else
-        throw new \RuntimeException("Couldn't create shared memory segment.");
+    } else {
+          throw new \RuntimeException("Cannot get a System V IPC key.");
     }
-    else
-      throw new \RuntimeException("Cannot get a System V IPC key.");
   }
 
 }
\ No newline at end of file

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-
 use EoC\Opt\ViewQueryOpts;
 
 

src/EoC/CLI/Command/CompactCommand.php

Braces +5 added lines, -4 removed lines

@@ -44,10 +44,11 @@
     $couch = $this->getConnection();
     $couch->selectDb($this->getDatabase());
 
-    if ($designDoc = $input->getOption('design-doc'))
-      $couch->compactView($designDoc);
-    else
-      $couch->compactDb();
+    if ($designDoc = $input->getOption('design-doc')) {
+          $couch->compactView($designDoc);
+    } else {
+          $couch->compactDb();
+    }
   }
 
 }
\ No newline at end of file

src/EoC/CLI/Command/AbstractCommand.php

Unused Use Statements -1 removed lines

@@ -15,7 +15,6 @@
 use Symfony\Component\Console\Input\InputOption;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
-
 use EoC\Opt\ViewQueryOpts;
 
 

Spacing +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
     elseif ($this->isFormatted($arg))
       return $arg;
     else
-      return (string)$arg ;
+      return (string)$arg;
   }
 
 

Braces +29 added lines, -26 removed lines

@@ -39,10 +39,11 @@ 
    * @return bool
    */
   protected function isArray($arg) {
-    if (preg_match('/\A[\[]([^\[\]]+)[\]]\z/i', $arg, $matches))
-      return TRUE;
-    else
-      return FALSE;
+    if (preg_match('/\A[\[]([^\[\]]+)[\]]\z/i', $arg, $matches)) {
+          return TRUE;
+    } else {
+          return FALSE;
+    }
   }
 
 
@@ -94,8 +95,7 @@ 
       }
 
       return TRUE;
-    }
-    else {
+    } else {
       return FALSE;
     }
   }
@@ -117,18 +117,19 @@ 
 
       $argument = [];
       foreach ($values as $value) {
-        if ($this->isFormatted($value))
-          $argument[] = $value;
-        else
-          $argument[] = (string)$value;
+        if ($this->isFormatted($value)) {
+                  $argument[] = $value;
+        } else {
+                  $argument[] = (string)$value;
+        }
       }
 
       return $argument;
+    } elseif ($this->isFormatted($arg)) {
+          return $arg;
+    } else {
+          return (string)$arg ;
     }
-    elseif ($this->isFormatted($arg))
-      return $arg;
-    else
-      return (string)$arg ;
   }
 
 
@@ -144,15 +145,16 @@ 
       $shmSize = shmop_size($shmId);
 
       // Now lets read the memory segment.
-      if ($buffer = shmop_read($shmId, 0, $shmSize))
-        $connection = unserialize($buffer);
-      else
-        throw new \RuntimeException("You are not connected to the server.");
+      if ($buffer = shmop_read($shmId, 0, $shmSize)) {
+              $connection = unserialize($buffer);
+      } else {
+              throw new \RuntimeException("You are not connected to the server.");
+      }
 
       shmop_close($shmId);
+    } else {
+          throw new \RuntimeException("You are not connected to the server.");
     }
-    else
-      throw new \RuntimeException("You are not connected to the server.");
 
     return new Couch(new CurlAdapter($connection['server'], $connection['user'], $connection['password']));
   }
@@ -170,15 +172,16 @@ 
       $shmSize = shmop_size($shmId);
 
       // Now lets read the memory segment.
-      if ($buffer = shmop_read($shmId, 0, $shmSize))
-        $database = $buffer;
-      else
-        throw new \RuntimeException("No database selected.");
+      if ($buffer = shmop_read($shmId, 0, $shmSize)) {
+              $database = $buffer;
+      } else {
+              throw new \RuntimeException("No database selected.");
+      }
 
       shmop_close($shmId);
+    } else {
+          throw new \RuntimeException("No database selected.");
     }
-    else
-      throw new \RuntimeException("No database selected.");
 
     return $database;
   }

src/EoC/CLI/Command/DeleteCommand.php

Braces +4 added lines, -4 removed lines

@@ -48,15 +48,15 @@
     // In case there is a database in use, we select it, because you the selected database can't be deleted.
     try {
       $couch->selectDb($this->getDatabase());
-    }
-    catch (\RuntimeException $e) {}
+    } catch (\RuntimeException $e) {}
 
     $question = new ConfirmationQuestion('Are you sure you want delete the database? [Y/n]', FALSE);
 
     $helper = $this->getHelper('question');
 
-    if ($helper->ask($input, $output, $question))
-      $couch->deleteDb($database);
+    if ($helper->ask($input, $output, $question)) {
+          $couch->deleteDb($database);
+    }
   }
 
 }
\ No newline at end of file