Merge pull request #3 from glebLaModa/php8

PHP 8 Support

Author: denpli

.travis.yml

@@ -4,9 +4,8 @@ php:
     - 7.1
     - 7.2
     - 7.3
-    - 7.4snapshot
-    # php8 is not supported yet
-    #- nightly
+    - 7.4
+    - 8.0
 
 env:
     matrix:
@@ -17,8 +16,6 @@ env:
 
 matrix:
     fast_finish: true
-    allow_failures:
-        - php: 7.4snapshot
 
 install:
     - if [[ "$DEPENDENCIES" = 'high' ]]; then travis_retry composer update $DEFAULT_COMPOSER_FLAGS; fi

composer.json

@@ -11,7 +11,7 @@
     ],
     "minimum-stability": "stable",
     "require": {
-        "php": "^7.1"
+        "php": "^7.1 | ^8.0"
     },
     "autoload": {
         "psr-4": {
@@ -28,6 +28,6 @@
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^2.15",
-        "phpunit/phpunit": "^7.5 | ^8.2"
+        "phpunit/phpunit": "^7.5.20 | ^8.2 | ^9.0"
     }
 }

src/Parser/Parser.php

@@ -203,12 +203,14 @@ public function parse(string $data): Barcode
                 $position += $length + strlen($this->config->getGroupSeparator());
             }
 
-            $this->assertValueIsValid($value);
+            if ($length > 0) {
+                $this->assertValueIsValid($value);
 
-            if ($ai) {
-                $foundAIs[$ai] = $value;
-            } else {
-                $buffer[] = $value;
+                if ($ai) {
+                    $foundAIs[$ai] = $value;
+                } else {
+                    $buffer[] = $value;
+                }
             }
         }