Release for PHP 8 (#20)

* Require PHP >= 8.0
* Add not in token type
* Separate booleans into two different tokens to fix token overriding bug
* Cleanup
* Fix phpstan issues

Author: nicoSWD

.scrutinizer.yml

@@ -35,6 +35,8 @@ tools:
     enabled: true
     excluded_dirs: [tests]
 build:
+  environment:
+    php: 8.0.1
   nodes:
     analysis:
       tests:

.styleci.yml

@@ -1,6 +1,7 @@
 ---
 preset: psr2
 risky: true
+version: 8
 finder:
   name:
     - "*.php"

.travis.yml

@@ -5,6 +5,7 @@ php:
   - 7.2
   - 7.3
   - 7.4
+  - 8.0
 
 script:
   - composer install --dev

README.md

@@ -1,4 +1,4 @@
-## PHP Rule Engine for PHP 7
+## PHP Rule Engine
 
 
 [![Latest Stable Version](https://travis-ci.org/nicoSWD/php-rule-parser.svg?branch=master)](https://travis-ci.org/nicoSWD/php-rule-parser)

composer.json

@@ -1,7 +1,7 @@
 {
   "name": "nicoswd/php-rule-parser",
   "type": "library",
-  "description": "PHP 7 Rule Engine - Rule Parser & Evaluator",
+  "description": "Rule Engine - Rule Parser & Evaluator",
   "keywords": [
     "rule",
     "parser",
@@ -33,7 +33,7 @@
     }
   },
   "require": {
-    "php": ">=7.1"
+    "php": ">=8.0"
   },
   "require-dev": {
     "phpunit/phpunit": "^7.0|^9.0",

phpunit.xml.dist

@@ -1,16 +1,15 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/8.0/phpunit.xsd"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd"
          bootstrap="src/autoload.php">
-    <testsuites>
-        <testsuite name="Unit tests">
-            <directory>tests/</directory>
-        </testsuite>
-    </testsuites>
-
-    <filter>
-        <whitelist processUncoveredFilesFromWhitelist="true">
-            <directory suffix=".php">src</directory>
-        </whitelist>
-    </filter>
+  <coverage processUncoveredFiles="true">
+    <include>
+      <directory suffix=".php">src</directory>
+    </include>
+  </coverage>
+  <testsuites>
+    <testsuite name="Unit tests">
+      <directory>tests/</directory>
+    </testsuite>
+  </testsuites>
 </phpunit>

src/Compiler/StandardCompiler.php

@@ -15,21 +15,18 @@
 
 class StandardCompiler implements CompilerInterface
 {
-    const BOOL_TRUE = '1';
-    const BOOL_FALSE = '0';
+    private const BOOL_TRUE = '1';
+    private const BOOL_FALSE = '0';
 
-    const LOGICAL_AND = '&';
-    const LOGICAL_OR = '|';
+    private const LOGICAL_AND = '&';
+    private const LOGICAL_OR = '|';
 
-    const OPENING_PARENTHESIS = '(';
-    const CLOSING_PARENTHESIS = ')';
+    private const OPENING_PARENTHESIS = '(';
+    private const CLOSING_PARENTHESIS = ')';
 
-    /** @var string */
-    private $output = '';
-    /** @var int */
-    private $openParenthesis = 0;
-    /** @var int */
-    private $closedParenthesis = 0;
+    private string $output = '';
+    private int $openParenthesis = 0;
+    private int $closedParenthesis = 0;
 
     public function getCompiledRule(): string
     {
@@ -48,6 +45,7 @@ private function openParenthesis(): void
         $this->output .= self::OPENING_PARENTHESIS;
     }
 
+    /** @throws ParserException */
     private function closeParenthesis(): void
     {
         if ($this->openParenthesis < 1) {
@@ -58,6 +56,7 @@ private function closeParenthesis(): void
         $this->output .= self::CLOSING_PARENTHESIS;
     }
 
+    /** @throws ParserException */
     public function addParentheses(BaseToken $token): void
     {
         if ($token instanceof TokenOpeningParenthesis) {
@@ -70,6 +69,7 @@ public function addParentheses(BaseToken $token): void
         }
     }
 
+    /** @throws ParserException */
     public function addLogical(BaseToken $token): void
     {
         $lastChar = $this->getLastChar();
@@ -85,6 +85,7 @@ public function addLogical(BaseToken $token): void
         }
     }
 
+    /** @throws MissingOperatorException */
     public function addBoolean(bool $bool): void
     {
         $lastChar = $this->getLastChar();
@@ -105,26 +106,24 @@ private function isIncompleteCondition(): bool
     {
         $lastChar = $this->getLastChar();
 
-        return (
+        return
             $lastChar === self::LOGICAL_AND ||
-            $lastChar === self::LOGICAL_OR
-        );
+            $lastChar === self::LOGICAL_OR;
     }
 
     private function expectOpeningParenthesis(): bool
     {
         $lastChar = $this->getLastChar();
 
-        return (
+        return
             $lastChar === '' ||
             $lastChar === self::LOGICAL_AND ||
             $lastChar === self::LOGICAL_OR ||
-            $lastChar === self::OPENING_PARENTHESIS
-        );
+            $lastChar === self::OPENING_PARENTHESIS;
     }
 
     private function getLastChar(): string
     {
-        return substr($this->output, -1);
+        return substr($this->output, offset: -1);
     }
 }

src/Evaluator/Evaluator.php

@@ -9,11 +9,11 @@
 
 final class Evaluator implements EvaluatorInterface
 {
-    const LOGICAL_AND = '&';
-    const LOGICAL_OR = '|';
+    private const LOGICAL_AND = '&';
+    private const LOGICAL_OR = '|';
 
-    const BOOL_TRUE = '1';
-    const BOOL_FALSE = '0';
+    private const BOOL_TRUE = '1';
+    private const BOOL_FALSE = '0';
 
     public function evaluate(string $group): bool
     {
@@ -25,8 +25,8 @@ public function evaluate(string $group): bool
                 '~\(([^()]+)\)~',
                 $evalGroup,
                 $group,
-                -1,
-                $count
+                limit: -1,
+                count: $count
             );
         } while ($count > 0);
 

src/Expression/BaseExpression.php

@@ -7,12 +7,10 @@
  */
 namespace nicoSWD\Rule\Expression;
 
+use nicoSWD\Rule\Parser\Exception\ParserException;
+
 abstract class BaseExpression
 {
-    /**
-     * @param mixed $leftValue
-     * @param mixed $rightValue
-     * @return bool
-     */
-    abstract public function evaluate($leftValue, $rightValue): bool;
+    /** @throws ParserException */
+    abstract public function evaluate(mixed $leftValue, mixed $rightValue): bool;
 }

src/Expression/EqualExpression.php

@@ -9,7 +9,7 @@
 
 final class EqualExpression extends BaseExpression
 {
-    public function evaluate($leftValue, $rightValue): bool
+    public function evaluate(mixed $leftValue, mixed $rightValue): bool
     {
         return $leftValue == $rightValue;
     }