phpmyadmin / sql-parser

src/Components/PartitionDefinition.php

Doc Comments +1 added lines, -1 removed lines

@@ -194,7 +194,7 @@
     }
 
     /**
-     * @param PartitionDefinition|PartitionDefinition[] $component the component to be built
+     * @param PartitionDefinition[] $component the component to be built
      * @param array                                     $options   parameters for building
      *
      * @return string

Spacing +1 added lines, -1 removed lines

@@ -247,7 +247,7 @@
         return trim(
             'PARTITION ' . $component->name
             . (empty($component->type) ? '' : ' VALUES ' . $component->type . ' ' . $component->expr . ' ')
-            . ((! empty($component->options) && ! empty($component->type)) ? '' : ' ') . $component->options . $subpartitions
+            . ((!empty($component->options) && !empty($component->type)) ? '' : ' ') . $component->options . $subpartitions
         );
     }
 }

src/Utils/Query.php

Doc Comments +1 added lines, -1 removed lines

@@ -499,7 +499,7 @@
      * @param Statement  $statement the parsed query that has to be modified
      * @param TokensList $list      the list of tokens
      * @param string     $clause    the clause to be returned
-     * @param int|string $type      The type of the search.
+     * @param integer $type      The type of the search.
      *                              If int,
      *                              -1 for everything that was before
      *                              0 only for the clause

Spacing +19 added lines, -19 removed lines

@@ -227,7 +227,7 @@ 
         $flags['querytype'] = 'SELECT';
         $flags['is_select'] = true;
 
-        if (! empty($statement->from)) {
+        if (!empty($statement->from)) {
             $flags['select_from'] = true;
         }
 
@@ -235,55 +235,55 @@ 
             $flags['distinct'] = true;
         }
 
-        if (! empty($statement->group) || ! empty($statement->having)) {
+        if (!empty($statement->group) || !empty($statement->having)) {
             $flags['is_group'] = true;
         }
 
-        if (! empty($statement->into)
+        if (!empty($statement->into)
             && ($statement->into->type === 'OUTFILE')
         ) {
             $flags['is_export'] = true;
         }
 
         $expressions = $statement->expr;
-        if (! empty($statement->join)) {
+        if (!empty($statement->join)) {
             foreach ($statement->join as $join) {
                 $expressions[] = $join->expr;
             }
         }
 
         foreach ($expressions as $expr) {
-            if (! empty($expr->function)) {
+            if (!empty($expr->function)) {
                 if ($expr->function === 'COUNT') {
                     $flags['is_count'] = true;
                 } elseif (in_array($expr->function, static::$FUNCTIONS)) {
                     $flags['is_func'] = true;
                 }
             }
-            if (! empty($expr->subquery)) {
+            if (!empty($expr->subquery)) {
                 $flags['is_subquery'] = true;
             }
         }
 
-        if (! empty($statement->procedure)
+        if (!empty($statement->procedure)
             && ($statement->procedure->name === 'ANALYSE')
         ) {
             $flags['is_analyse'] = true;
         }
 
-        if (! empty($statement->group)) {
+        if (!empty($statement->group)) {
             $flags['group'] = true;
         }
 
-        if (! empty($statement->having)) {
+        if (!empty($statement->having)) {
             $flags['having'] = true;
         }
 
-        if (! empty($statement->union)) {
+        if (!empty($statement->union)) {
             $flags['union'] = true;
         }
 
-        if (! empty($statement->join)) {
+        if (!empty($statement->join)) {
             $flags['join'] = true;
         }
 
@@ -374,10 +374,10 @@ 
             || ($statement instanceof UpdateStatement)
             || ($statement instanceof DeleteStatement)
         ) {
-            if (! empty($statement->limit)) {
+            if (!empty($statement->limit)) {
                 $flags['limit'] = true;
             }
-            if (! empty($statement->order)) {
+            if (!empty($statement->order)) {
                 $flags['order'] = true;
             }
         }
@@ -445,7 +445,7 @@ 
                                 $expr->database : null
                         );
                     }
-                    if (! in_array($arr, $ret['select_tables'])) {
+                    if (!in_array($arr, $ret['select_tables'])) {
                         $ret['select_tables'][] = $arr;
                     }
                 } else {
@@ -464,7 +464,7 @@ 
                             (isset($expr->database) && ($expr->database !== '')) ?
                                 $expr->database : null
                         );
-                        if (! in_array($arr, $ret['select_tables'])) {
+                        if (!in_array($arr, $ret['select_tables'])) {
                             $ret['select_tables'][] = $arr;
                         }
                     }
@@ -501,7 +501,7 @@ 
         ) {
             $expressions = array($statement->table);
         } elseif ($statement instanceof DropStatement) {
-            if (! $statement->options->has('TABLE')) {
+            if (!$statement->options->has('TABLE')) {
                 // No tables are dropped.
                 return array();
             }
@@ -514,7 +514,7 @@ 
 
         $ret = array();
         foreach ($expressions as $expr) {
-            if (! empty($expr->table)) {
+            if (!empty($expr->table)) {
                 $expr->expr = null; // Force rebuild.
                 $expr->alias = null; // Aliases are not required.
                 $ret[] = Expression::build($expr);
@@ -786,7 +786,7 @@ 
 
             $statement .= $token->token;
 
-            if (($token->type === Token::TYPE_DELIMITER) && ! empty($token->token)) {
+            if (($token->type === Token::TYPE_DELIMITER) && !empty($token->token)) {
                 $delimiter = $token->token;
                 $fullStatement = true;
                 break;
@@ -795,7 +795,7 @@ 
 
         // No statement was found so we return the entire query as being the
         // remaining part.
-        if (! $fullStatement) {
+        if (!$fullStatement) {
             return array(
                 null,
                 $query,

src/Statements/LoadStatement.php

Doc Comments +5 added lines

@@ -339,6 +339,11 @@
         }
     }
 
+    /**
+     * @param Parser $parser
+     * @param TokensList $list
+     * @param integer $state
+     */
     public function parseKeywordsAccordingToState($parser, $list, $state)
     {
         $token = $list->tokens[$list->idx];

src/UtfString.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
         $this->byteIdx = 0;
         $this->charIdx = 0;
         $this->byteLen = mb_strlen($str, '8bit');
-        if (! mb_check_encoding($str, 'UTF-8')) {
+        if (!mb_check_encoding($str, 'UTF-8')) {
             $this->charLen = 0;
         } else {
             $this->charLen = mb_strlen($str, 'UTF-8');

src/Components/LockExpression.php

Doc Comments +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
      * @param TokensList $list    the list of tokens that are being parsed
      * @param array      $options parameters for parsing
      *
-     * @return CaseExpression
+     * @return LockExpression
      */
     public static function parse(Parser $parser, TokensList $list, array $options = array())
     {
@@ -98,7 +98,7 @@ 
     }
 
     /**
-     * @param LockExpression|LockExpression[] $component the component to be built
+     * @param LockExpression[] $component the component to be built
      * @param array                           $options   parameters for building
      *
      * @return string

src/Context.php

Doc Comments +1 added lines, -1 removed lines

@@ -326,7 +326,7 @@
      * Checks if the given string is the beginning of a whitespace.
      *
      * @param string $str string to be checked
-     * @param mixed  $end
+     * @param boolean  $end
      *
      * @return int|null the appropriate flag for the comment type
      */

Spacing +5 added lines, -5 removed lines

@@ -275,7 +275,7 @@ 
         $str = strtoupper($str);
 
         if (isset(static::$KEYWORDS[$str])) {
-            if ($isReserved && ! (static::$KEYWORDS[$str] & Token::FLAG_KEYWORD_RESERVED)) {
+            if ($isReserved && !(static::$KEYWORDS[$str] & Token::FLAG_KEYWORD_RESERVED)) {
                 return null;
             }
 
@@ -297,7 +297,7 @@ 
      */
     public static function isOperator($str)
     {
-        if (! isset(static::$OPERATORS[$str])) {
+        if (!isset(static::$OPERATORS[$str])) {
             return null;
         }
 
@@ -489,7 +489,7 @@ 
             // Short context name (must be formatted into class name).
             $context = self::$contextPrefix . $context;
         }
-        if (! class_exists($context)) {
+        if (!class_exists($context)) {
             throw @new LoaderException(
                 'Specified context ("' . $context . '") does not exist.',
                 $context
@@ -526,7 +526,7 @@ 
                     $i -= 2;
                     $part = substr($context, $i, 2);
                     /* No more numeric parts to strip */
-                    if (! is_numeric($part)) {
+                    if (!is_numeric($part)) {
                         break 2;
                     }
                 } while (intval($part) === 0 && $i > 0);
@@ -578,7 +578,7 @@ 
         }
 
         if ((static::$MODE & self::SQL_MODE_NO_ENCLOSING_QUOTES)
-            && (! static::isKeyword($str, true))
+            && (!static::isKeyword($str, true))
         ) {
             return $str;
         }

src/Statements/PurgeStatement.php

Doc Comments +1 added lines, -1 removed lines

@@ -125,7 +125,7 @@
      *
      * @param Parser $parser            the instance that requests parsing
      * @param Token  $token             token to be parsed
-     * @param Array  $expected_keywords array of possibly expected keywords at this point
+     * @param string[]  $expected_keywords array of possibly expected keywords at this point
      */
     private static function parseExpectedKeyword($parser, $token, $expected_keywords)
     {

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 namespace PhpMyAdmin\SqlParser\Statements;
 
 use PhpMyAdmin\SqlParser\Components\Expression;
-use PhpMyAdmin\SqlParser\Components\OptionsArray;
 use PhpMyAdmin\SqlParser\Parser;
 use PhpMyAdmin\SqlParser\Statement;
 use PhpMyAdmin\SqlParser\Token;

src/Statements/LockStatement.php

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
                     break;
                 }
             } elseif ($state === 1) {
-                if (! $this->isLock) {
+                if (!$this->isLock) {
                     // UNLOCK statement should not have any more tokens
                     $parser->error('Unexpected token.', $token);
                     break;

src/Statements/DeleteStatement.php

Spacing +7 added lines, -7 removed lines

@@ -166,25 +166,25 @@
     {
         $ret = 'DELETE ' . OptionsArray::build($this->options);
 
-        if (! is_null($this->columns) && count($this->columns) > 0) {
+        if (!is_null($this->columns) && count($this->columns) > 0) {
             $ret .= ' ' . ExpressionArray::build($this->columns);
         }
-        if (! is_null($this->from) && count($this->from) > 0) {
+        if (!is_null($this->from) && count($this->from) > 0) {
             $ret .= ' FROM ' . ExpressionArray::build($this->from);
         }
-        if (! is_null($this->join) && count($this->join) > 0) {
+        if (!is_null($this->join) && count($this->join) > 0) {
             $ret .= ' ' . JoinKeyword::build($this->join);
         }
-        if (! is_null($this->using) && count($this->using) > 0) {
+        if (!is_null($this->using) && count($this->using) > 0) {
             $ret .= ' USING ' . ExpressionArray::build($this->using);
         }
-        if (! is_null($this->where) && count($this->where) > 0) {
+        if (!is_null($this->where) && count($this->where) > 0) {
             $ret .= ' WHERE ' . Condition::build($this->where);
         }
-        if (! is_null($this->order) && count($this->order) > 0) {
+        if (!is_null($this->order) && count($this->order) > 0) {
             $ret .= ' ORDER BY ' . ExpressionArray::build($this->order);
         }
-        if (! is_null($this->limit) && strlen($this->limit) > 0) {
+        if (!is_null($this->limit) && strlen($this->limit) > 0) {
             $ret .= ' LIMIT ' . Limit::build($this->limit);
         }