bug #34419 [Cache] Disable igbinary on PHP >= 7.4 (nicolas-grekas)

nicolas-grekas · nicolas-grekas · commit 479efc54f30f · 2019-11-17T12:01:01.000+01:00
This PR was merged into the 4.3 branch. Discussion ---------- [Cache] Disable igbinary on PHP >= 7.4 | Q | A | ------------- | --- | Branch? | 4.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - Until igbinary/igbinary#244 is fixed. Commits ------- a2c924d [Cache] Disable igbinary on PHP >= 7.4
diff --git a/src/Symfony/Component/Cache/Marshaller/DefaultMarshaller.php b/src/Symfony/Component/Cache/Marshaller/DefaultMarshaller.php
@@ -25,9 +25,9 @@ class DefaultMarshaller implements MarshallerInterface
     public function __construct(bool $useIgbinarySerialize = null)
     {
         if (null === $useIgbinarySerialize) {
-            $useIgbinarySerialize = \extension_loaded('igbinary') && \PHP_VERSION_ID !== 70400;
-        } elseif ($useIgbinarySerialize && (!\extension_loaded('igbinary') || \PHP_VERSION_ID === 70400)) {
-            throw new CacheException('The "igbinary" PHP extension is not '.(\PHP_VERSION_ID === 70400 ? 'compatible with PHP 7.4.0.' : 'loaded.'));
+            $useIgbinarySerialize = \extension_loaded('igbinary') && \PHP_VERSION_ID < 70400;
+        } elseif ($useIgbinarySerialize && (!\extension_loaded('igbinary') || \PHP_VERSION_ID >= 70400)) {
+            throw new CacheException('The "igbinary" PHP extension is not '.(\PHP_VERSION_ID >= 70400 ? 'compatible with PHP 7.4.' : 'loaded.'));
         }
         $this->useIgbinarySerialize = $useIgbinarySerialize;
     }
@@ -66,7 +66,7 @@ public function unmarshall(string $value)
             return null;
         }
         static $igbinaryNull;
-        if ($value === ($igbinaryNull ?? $igbinaryNull = \extension_loaded('igbinary') && \PHP_VERSION_ID !== 70400 ? igbinary_serialize(null) : false)) {
+        if ($value === ($igbinaryNull ?? $igbinaryNull = \extension_loaded('igbinary') && \PHP_VERSION_ID < 70400 ? igbinary_serialize(null) : false)) {
             return null;
         }
         $unserializeCallbackHandler = ini_set('unserialize_callback_func', __CLASS__.'::handleUnserializeCallback');
diff --git a/src/Symfony/Component/Cache/Tests/Marshaller/DefaultMarshallerTest.php b/src/Symfony/Component/Cache/Tests/Marshaller/DefaultMarshallerTest.php
@@ -24,7 +24,7 @@ public function testSerialize()
             'b' => function () {},
         ];
 
-        $expected = ['a' => \extension_loaded('igbinary') && \PHP_VERSION_ID !== 70400 ? igbinary_serialize(123) : serialize(123)];
+        $expected = ['a' => \extension_loaded('igbinary') && \PHP_VERSION_ID < 70400 ? igbinary_serialize(123) : serialize(123)];
         $this->assertSame($expected, $marshaller->marshall($values, $failed));
         $this->assertSame(['b'], $failed);
     }
@@ -43,8 +43,8 @@ public function testNativeUnserialize()
      */
     public function testIgbinaryUnserialize()
     {
-        if (\PHP_VERSION_ID === 70400) {
-            $this->markTestSkipped('igbinary is not compatible with PHP 7.4.0.');
+        if (\PHP_VERSION_ID >= 70400) {
+            $this->markTestSkipped('igbinary is not compatible with PHP 7.4.');
         }
 
         $marshaller = new DefaultMarshaller();
@@ -67,8 +67,8 @@ public function testNativeUnserializeNotFoundClass()
      */
     public function testIgbinaryUnserializeNotFoundClass()
     {
-        if (\PHP_VERSION_ID === 70400) {
-            $this->markTestSkipped('igbinary is not compatible with PHP 7.4.0.');
+        if (\PHP_VERSION_ID >= 70400) {
+            $this->markTestSkipped('igbinary is not compatible with PHP 7.4.');
         }
 
         $this->expectException('DomainException');
@@ -95,8 +95,8 @@ public function testNativeUnserializeInvalid()
      */
     public function testIgbinaryUnserializeInvalid()
     {
-        if (\PHP_VERSION_ID === 70400) {
-            $this->markTestSkipped('igbinary is not compatible with PHP 7.4.0');
+        if (\PHP_VERSION_ID >= 70400) {
+            $this->markTestSkipped('igbinary is not compatible with PHP 7.4.');
         }
 
         $this->expectException('DomainException');

Original file line number	Diff line number	Diff line change
`@@ -25,9 +25,9 @@ class DefaultMarshaller implements MarshallerInterface`
`25`	`25`	`public function __construct(bool $useIgbinarySerialize = null)`
`26`	`26`	`{`
`27`	`27`	`if (null === $useIgbinarySerialize) {`
`28`		`- $useIgbinarySerialize = \extension_loaded('igbinary') && \PHP_VERSION_ID !== 70400;`
`29`		`- } elseif ($useIgbinarySerialize && (!\extension_loaded('igbinary') \|\| \PHP_VERSION_ID === 70400)) {`
`30`		`- throw new CacheException('The "igbinary" PHP extension is not '.(\PHP_VERSION_ID === 70400 ? 'compatible with PHP 7.4.0.' : 'loaded.'));`
	`28`	`+ $useIgbinarySerialize = \extension_loaded('igbinary') && \PHP_VERSION_ID < 70400;`
	`29`	`+ } elseif ($useIgbinarySerialize && (!\extension_loaded('igbinary') \|\| \PHP_VERSION_ID >= 70400)) {`
	`30`	`+ throw new CacheException('The "igbinary" PHP extension is not '.(\PHP_VERSION_ID >= 70400 ? 'compatible with PHP 7.4.' : 'loaded.'));`
`31`	`31`	`}`
`32`	`32`	`$this->useIgbinarySerialize = $useIgbinarySerialize;`
`33`	`33`	`}`
`@@ -66,7 +66,7 @@ public function unmarshall(string $value)`
`66`	`66`	`return null;`
`67`	`67`	`}`
`68`	`68`	`static $igbinaryNull;`
`69`		`- if ($value === ($igbinaryNull ?? $igbinaryNull = \extension_loaded('igbinary') && \PHP_VERSION_ID !== 70400 ? igbinary_serialize(null) : false)) {`
	`69`	`+ if ($value === ($igbinaryNull ?? $igbinaryNull = \extension_loaded('igbinary') && \PHP_VERSION_ID < 70400 ? igbinary_serialize(null) : false)) {`
`70`	`70`	`return null;`
`71`	`71`	`}`
`72`	`72`	`$unserializeCallbackHandler = ini_set('unserialize_callback_func', __CLASS__.'::handleUnserializeCallback');`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public function testSerialize()`
`24`	`24`	`'b' => function () {},`
`25`	`25`	`];`
`26`	`26`
`27`		`- $expected = ['a' => \extension_loaded('igbinary') && \PHP_VERSION_ID !== 70400 ? igbinary_serialize(123) : serialize(123)];`
	`27`	`+ $expected = ['a' => \extension_loaded('igbinary') && \PHP_VERSION_ID < 70400 ? igbinary_serialize(123) : serialize(123)];`
`28`	`28`	`$this->assertSame($expected, $marshaller->marshall($values, $failed));`
`29`	`29`	`$this->assertSame(['b'], $failed);`
`30`	`30`	`}`
`@@ -43,8 +43,8 @@ public function testNativeUnserialize()`
`43`	`43`	`*/`
`44`	`44`	`public function testIgbinaryUnserialize()`
`45`	`45`	`{`
`46`		`- if (\PHP_VERSION_ID === 70400) {`
`47`		`- $this->markTestSkipped('igbinary is not compatible with PHP 7.4.0.');`
	`46`	`+ if (\PHP_VERSION_ID >= 70400) {`
	`47`	`+ $this->markTestSkipped('igbinary is not compatible with PHP 7.4.');`
`48`	`48`	`}`
`49`	`49`
`50`	`50`	`$marshaller = new DefaultMarshaller();`
`@@ -67,8 +67,8 @@ public function testNativeUnserializeNotFoundClass()`
`67`	`67`	`*/`
`68`	`68`	`public function testIgbinaryUnserializeNotFoundClass()`
`69`	`69`	`{`
`70`		`- if (\PHP_VERSION_ID === 70400) {`
`71`		`- $this->markTestSkipped('igbinary is not compatible with PHP 7.4.0.');`
	`70`	`+ if (\PHP_VERSION_ID >= 70400) {`
	`71`	`+ $this->markTestSkipped('igbinary is not compatible with PHP 7.4.');`
`72`	`72`	`}`
`73`	`73`
`74`	`74`	`$this->expectException('DomainException');`
`@@ -95,8 +95,8 @@ public function testNativeUnserializeInvalid()`
`95`	`95`	`*/`
`96`	`96`	`public function testIgbinaryUnserializeInvalid()`
`97`	`97`	`{`
`98`		`- if (\PHP_VERSION_ID === 70400) {`
`99`		`- $this->markTestSkipped('igbinary is not compatible with PHP 7.4.0');`
	`98`	`+ if (\PHP_VERSION_ID >= 70400) {`
	`99`	`+ $this->markTestSkipped('igbinary is not compatible with PHP 7.4.');`
`100`	`100`	`}`
`101`	`101`
`102`	`102`	`$this->expectException('DomainException');`