thecodingmachine / magic-query

src/Mouf/Database/MagicQuery/Twig/SqlTwigEnvironmentFactory.php

Unused Use Statements -2 removed lines

@@ -1,8 +1,6 @@
 <?php
 namespace Mouf\Database\MagicQuery\Twig;
 
-use Doctrine\DBAL\Connection;
-
 /**
  * Class in charge of creating the Twig environment
  */

src/Mouf/Database/QueryWriter/Condition/ParamAvailableCondition.php

Doc Comments -1 removed lines

@@ -24,7 +24,6 @@
     }
 
     /**
-     * @param string $caller
      */
     public function isOk($parameters = null)
     {

src/Mouf/Database/QueryWriter/Condition/ParamEqualsCondition.php

Doc Comments -1 removed lines

@@ -29,7 +29,6 @@
     }
 
     /**
-     * @param string $caller
      */
     public function isOk($parameters = null)
     {

src/Mouf/Database/QueryWriter/Condition/ParamNotAvailableCondition.php

Doc Comments -1 removed lines

@@ -24,7 +24,6 @@
     }
 
     /**
-     * @param string $caller
      */
     public function isOk($parameters = null)
     {

src/SQLParser/Node/AbstractManyInstancesOperator.php

Doc Comments +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
      *
      * @param Connection $dbConnection
      * @param array      $parameters
-     * @param number     $indent
+     * @param integer     $indent
      * @param int        $conditionsMode
      *
      * @return string

src/SQLParser/Node/Expression.php

Doc Comments +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
      *
      * @param Connection $dbConnection
      * @param array      $parameters
-     * @param number     $indent
+     * @param integer     $indent
      * @param int        $conditionsMode
      *
      * @return string

src/SQLParser/Node/NodeFactory.php

Doc Comments +4 added lines, -1 removed lines

@@ -635,6 +635,9 @@ 
         });
     }
 
+    /**
+     * @param \Closure $callback
+     */
     private static function array_map_deep($array, $callback)
     {
         $new = array();
@@ -661,7 +664,7 @@ 
      * @param array       $parameters
      * @param string      $delimiter
      * @param bool|string $wrapInBrackets
-     * @param int|number  $indent
+     * @param integer  $indent
      * @param int         $conditionsMode
      *
      * @return null|string

Spacing +4 added lines, -4 removed lines

@@ -335,7 +335,7 @@ 
             if (!empty($subTree) && !isset($subTree[0])) {
                 $subTree = StatementFactory::toObject($subTree);
             } else {
-                $subTree = array_map(function ($item) {
+                $subTree = array_map(function($item) {
                     if (is_array($item)) {
                         return self::toObject($item);
                     } else {
@@ -365,7 +365,7 @@ 
             array('NOT'),
             array('&&', 'AND'),
             array('XOR'),
-            array('||', 'OR'), );
+            array('||', 'OR'),);
 
     private static $OPERATOR_TO_CLASS = array(
             '=' => 'SQLParser\Node\Equal',
@@ -626,7 +626,7 @@ 
     {
         $instanceDescriptor = $moufManager->createInstance(get_called_class());
 
-        return self::array_map_deep($node, function ($item) use ($moufManager) {
+        return self::array_map_deep($node, function($item) use ($moufManager) {
             if ($item instanceof NodeInterface) {
                 return $item->toInstanceDescriptor($moufManager);
             } else {
@@ -670,7 +670,7 @@ 
     {
         if (is_array($nodes)) {
             $elems = array();
-            array_walk_recursive($nodes, function ($item) use (&$elems, $dbConnection, $indent, $delimiter, $parameters, $conditionsMode) {
+            array_walk_recursive($nodes, function($item) use (&$elems, $dbConnection, $indent, $delimiter, $parameters, $conditionsMode) {
                 if ($item instanceof SqlRenderInterface) {
                     $itemSql = $item->toSql($parameters, $dbConnection, $indent, $conditionsMode);
                     if ($itemSql !== null) {

src/SQLParser/Node/SubQuery.php

Doc Comments +1 added lines, -1 removed lines

@@ -181,7 +181,7 @@
      *
      * @param Connection $dbConnection
      * @param array $parameters
-     * @param number $indent
+     * @param integer $indent
      * @param int $conditionsMode
      *
      * @return string

Unused Use Statements -1 removed lines

@@ -39,7 +39,6 @@
 use SQLParser\Query\Select;
 use Mouf\MoufInstanceDescriptor;
 use Mouf\MoufManager;
-use SQLParser\SqlRenderInterface;
 
 /**
  * This class represents a subquery (and optionally a JOIN .. ON expression in an SQL expression.

src/SQLParser/OracleSQLTranslator.php

Doc Comments +13 added lines

@@ -57,6 +57,9 @@ 
         $this->allTables = array();
     }
 
+    /**
+     * @param string $txt
+     */
     public static function dbgprint($txt)
     {
         if (isset($_ENV['DEBUG'])) {
@@ -95,6 +98,9 @@ 
         return 'DELETE';
     }
 
+    /**
+     * @param string $column
+     */
     public static function getColumnNameFor($column)
     {
         if (strtolower($column) === 'uid') {
@@ -198,6 +204,10 @@ 
         return ($res >= 1);
     }
 
+    /**
+     * @param string $table
+     * @param string $column
+     */
     protected function isCLOBColumn($table, $column)
     {
         $tables = end($this->allTables);
@@ -314,6 +324,9 @@ 
         return 'SELECT '.$sql;
     }
 
+    /**
+     * @param string $sql
+     */
     private function correctColRefStatement($sql)
     {
         $alias = '';

Switch Indentation +8 added lines, -8 removed lines

@@ -368,14 +368,14 @@
     {
         $k = key($parsed);
         switch ($k) {
-        case 'USE':
-        # this statement is not an Oracle statement
-            $this->created = '';
-            break;
-
-        default:
-            $this->created = parent::create($parsed);
-            break;
+            case 'USE':
+            # this statement is not an Oracle statement
+                $this->created = '';
+                break;
+
+            default:
+                $this->created = parent::create($parsed);
+                break;
         }
 
         return $this->created;