Backport #98: HHVM >3.18.4 & >3.20.2 reports as PHP-7.1 but does not support constant visibility

Author: marc-mabe

src/Enum.php

@@ -378,9 +378,11 @@ private static function detectConstants($class)
 
             do {
                 $scopeConstants = array();
-                if (PHP_VERSION_ID >= 70100) {
+                if (\PHP_VERSION_ID >= 70100 && method_exists('ReflectionClass', 'getReflectionConstants')) {
                     // Since PHP-7.1 visibility modifiers are allowed for class constants
                     // for enumerations we are only interested in public once.
+                    // NOTE: HHVM > 3.26.2 still does not support private/protected constants.
+                    //       It allows the visibility keyword but ignores it.
                     foreach ($reflection->getReflectionConstants() as $reflConstant) {
                         if ($reflConstant->isPublic()) {
                             $scopeConstants[ $reflConstant->getName() ] = $reflConstant->getValue();

tests/MabeEnumTest/EnumTest.php

@@ -297,6 +297,9 @@ public function testConstVisibility()
         if (PHP_VERSION_ID < 70100) {
             $this->markTestSkipped('This test is for PHP-7.1 and upper only');
         }
+        if (defined('HHVM_VERSION')) {
+            $this->markTestSkipped('HHVM does not support constant visibility');
+        }
 
         $constants = ConstVisibilityEnum::getConstants();
         $this->assertSame(array(
@@ -310,6 +313,9 @@ public function testConstVisibilityExtended()
         if (PHP_VERSION_ID < 70100) {
             $this->markTestSkipped('This test is for PHP-7.1 and upper only');
         }
+        if (defined('HHVM_VERSION')) {
+            $this->markTestSkipped('HHVM does not support constant visibility');
+        }
 
         $constants = ConstVisibilityEnumExtended::getConstants();
         $this->assertSame(array(