Revert "EntityId: Hard-deprecate Serializable methods"

This reverts commit 1af704088a038e7d522d34e443bf71c52c72e6c4.

Reason for revert: Huge error spike in prod

Change-Id: I26ebf9069a5906b6b81305b170d958757a47cb57

Author: Jaime Nuche

RELEASE-NOTES.md

@@ -5,8 +5,7 @@
 * Removed support for calling `Statement::addNewReference()` and `StatementList` constructor with a
   single array argument, which was deprecated in `Version 9.6.0 (2021-03-31)`. These should now be
   called with a variadic argument list.
-* Added `__serialize()` and `__unserialize()` methods to the `EntityId` interface,
-  and deprecated the `serialize()` and `unserialize()` methods.
+* Added `__serialize()` and `__unserialize()` methods to the `EntityId` interface.
 * Added native type hints to the `Statement` and `StatementList` classes
 * Added `strict_types=1` to `Statement.php`, `StatementList.php`, and related test files
 * Removed support for repository names in entity IDs (e.g. `foo:Q1234`)

src/Entity/ItemId.php

@@ -71,7 +71,6 @@ public function __serialize(): array {
 	 * @return string
 	 */
 	public function serialize() {
-		wfDeprecated( __METHOD__, '1.41' );
 		return $this->serialization;
 	}
 
@@ -85,7 +84,6 @@ public function __unserialize( array $data ): void {
 	 * @param string $serialized
 	 */
 	public function unserialize( $serialized ) {
-		wfDeprecated( __METHOD__, '1.41' );
 		$array = json_decode( $serialized );
 		$this->serialization = is_array( $array ) ? $array[1] : $serialized;
 		$this->serialization = $this->serialization ?? '';

src/Entity/NumericPropertyId.php

@@ -65,7 +65,6 @@ public function __serialize(): array {
 	 * @return string
 	 */
 	public function serialize() {
-		wfDeprecated( __METHOD__, '1.41' );
 		return $this->serialization;
 	}
 
@@ -79,7 +78,6 @@ public function __unserialize( array $data ): void {
 	 * @param string $serialized
 	 */
 	public function unserialize( $serialized ) {
-		wfDeprecated( __METHOD__, '1.41' );
 		$array = json_decode( $serialized );
 		$this->serialization = is_array( $array ) ? $array[1] : $serialized;
 		$this->serialization = $this->serialization ?? '';

tests/unit/Entity/EntityIdTest.php

@@ -3,7 +3,6 @@
 namespace Wikibase\DataModel\Tests\Entity;
 
 use InvalidArgumentException;
-use MediaWikiUnitTestCase;
 use ReflectionClass;
 use Wikibase\DataModel\Entity\EntityId;
 use Wikibase\DataModel\Entity\ItemId;
@@ -22,7 +21,7 @@
  * @author Jeroen De Dauw < jeroendedauw@gmail.com >
  * @author John Erling Blad < jeblad@gmail.com >
  */
-class EntityIdTest extends MediaWikiUnitTestCase {
+class EntityIdTest extends \PHPUnit\Framework\TestCase {
 
 	public static function instanceProvider() {
 		$ids = [];
@@ -74,9 +73,6 @@ public static function deserializationCompatibilityProvider(): array {
 	 * @dataProvider deserializationCompatibilityProvider
 	 */
 	public function testDeserializationCompatibility( $expected, $serialization ) {
-		if ( str_starts_with( $serialization, 'C:' ) ) {
-			$this->expectDeprecationAndContinue( '/::unserialize/' );
-		}
 		$this->assertEquals(
 			$expected,
 			unserialize( $serialization )

tests/unit/Entity/EntityIdValueTest.php

@@ -3,7 +3,6 @@
 namespace Wikibase\DataModel\Tests\Entity;
 
 use DataValues\IllegalValueException;
-use MediaWikiUnitTestCase;
 use Wikibase\DataModel\Entity\EntityId;
 use Wikibase\DataModel\Entity\EntityIdValue;
 use Wikibase\DataModel\Entity\ItemId;
@@ -21,7 +20,7 @@
  * @author Thiemo Kreuz
  * @author Daniel Kinzler
  */
-class EntityIdValueTest extends MediaWikiUnitTestCase {
+class EntityIdValueTest extends \PHPUnit\Framework\TestCase {
 
 	public function testCanConstruct() {
 		$entityId = new ItemId( 'Q123' );
@@ -161,9 +160,6 @@ public static function provideDeserializationCompatibility() {
 	 * @param EntityIdValue $expected
 	 */
 	public function testDeserializationCompatibility( $serialized, EntityIdValue $expected ) {
-		if ( str_contains( $serialized, 'C:32:"Wikibase\DataModel\Entity\ItemId"' ) ) {
-			$this->expectDeprecationAndContinue( '/::unserialize/' );
-		}
 		$id = unserialize( $serialized );
 
 		$this->assertEquals( $expected, $id );

tests/unit/Entity/ItemIdTest.php

@@ -3,7 +3,6 @@
 namespace Wikibase\DataModel\Tests\Entity;
 
 use InvalidArgumentException;
-use MediaWikiUnitTestCase;
 use Wikibase\DataModel\Entity\ItemId;
 
 /**
@@ -15,7 +14,7 @@
  * @license GPL-2.0-or-later
  * @author Jeroen De Dauw < jeroendedauw@gmail.com >
  */
-class ItemIdTest extends MediaWikiUnitTestCase {
+class ItemIdTest extends \PHPUnit\Framework\TestCase {
 
 	/**
 	 * @dataProvider idSerializationProvider
@@ -87,7 +86,6 @@ public function testGetEntityType() {
 
 	public function testSerialize() {
 		$id = new ItemId( 'Q1' );
-		$this->expectDeprecationAndContinue( '/ItemId::serialize/' );
 		$this->assertSame( 'Q1', $id->serialize() );
 	}
 
@@ -96,7 +94,6 @@ public function testSerialize() {
 	 */
 	public function testUnserialize( $json, $expected ) {
 		$id = new ItemId( 'Q1' );
-		$this->expectDeprecationAndContinue( '/ItemId::unserialize/' );
 		$id->unserialize( $json );
 		$this->assertSame( $expected, $id->getSerialization() );
 	}

tests/unit/Entity/NumericPropertyIdTest.php

@@ -3,7 +3,6 @@
 namespace Wikibase\DataModel\Tests\Entity;
 
 use InvalidArgumentException;
-use MediaWikiUnitTestCase;
 use Wikibase\DataModel\Entity\NumericPropertyId;
 
 /**
@@ -14,7 +13,7 @@
  *
  * @license GPL-2.0-or-later
  */
-class NumericPropertyIdTest extends MediaWikiUnitTestCase {
+class NumericPropertyIdTest extends \PHPUnit\Framework\TestCase {
 
 	/**
 	 * @dataProvider idSerializationProvider
@@ -86,7 +85,6 @@ public function testGetEntityType() {
 
 	public function testSerialize() {
 		$id = new NumericPropertyId( 'P1' );
-		$this->expectDeprecationAndContinue( '/NumericPropertyId::serialize/' );
 		$this->assertSame( 'P1', $id->serialize() );
 	}
 
@@ -95,7 +93,6 @@ public function testSerialize() {
 	 */
 	public function testUnserialize( $json, $expected ) {
 		$id = new NumericPropertyId( 'P1' );
-		$this->expectDeprecationAndContinue( '/NumericPropertyId::unserialize/' );
 		$id->unserialize( $json );
 		$this->assertSame( $expected, $id->getSerialization() );
 	}