Annontate deprecated code that will be removed after the next release

Author: johanib

library/EngineBlock/Corto/Model/Consent.php

@@ -95,6 +95,8 @@ public function explicitConsentWasGivenFor(ServiceProvider $serviceProvider): Co
      *
      * The caller must pass the ConsentVersion already retrieved by explicitConsentWasGivenFor or
      * implicitConsentWasGivenFor to avoid a second identical DB query.
+     *
+     * @deprecated Remove after stable consent hash is running in production
      */
     public function upgradeAttributeHashFor(ServiceProvider $serviceProvider, ConsentType $consentType, ConsentVersion $consentVersion): void
     {
@@ -141,6 +143,7 @@ protected function _getConsentUid()
         return $this->_response->getNameIdValue();
     }
 
+    /** @deprecated Remove after stable consent hash is running in production */
     protected function _getAttributesHash($attributes): string
     {
         return $this->_hashService->getUnstableAttributesHash($attributes, $this->_mustStoreValues);
@@ -173,6 +176,7 @@ private function _storeConsent(ServiceProvider $serviceProvider, ConsentType $co
         return $this->_hashService->storeConsentHash($parameters);
     }
 
+    /** @deprecated Remove after stable consent hash is running in production */
     private function _updateConsent(ServiceProvider $serviceProvider, ConsentType $consentType): bool
     {
         $consentUid = $this->_getConsentUid();

src/OpenConext/EngineBlock/Authentication/Value/ConsentHashQuery.php

@@ -23,6 +23,7 @@ final class ConsentHashQuery
     public function __construct(
         public readonly string $hashedUserId,
         public readonly string $serviceId,
+        /** @deprecated Remove after stable consent hash is running in production */
         public readonly string $attributeHash,
         public readonly string $attributeStableHash,
         public readonly string $consentType,

src/OpenConext/EngineBlock/Authentication/Value/ConsentStoreParameters.php

@@ -25,6 +25,7 @@ public function __construct(
         public readonly string $serviceId,
         public readonly string $attributeStableHash,
         public readonly string $consentType,
+        /** @deprecated Remove after stable consent hash is running in production */
         public readonly ?string $attributeHash = null,
     ) {
     }

src/OpenConext/EngineBlock/Authentication/Value/ConsentUpdateParameters.php

@@ -22,10 +22,12 @@ final class ConsentUpdateParameters
 {
     public function __construct(
         public readonly string $attributeStableHash,
+        /** @deprecated Remove after stable consent hash is running in production */
         public readonly string $attributeHash,
         public readonly string $hashedUserId,
         public readonly string $serviceId,
         public readonly string $consentType,
+        /** @deprecated Remove after stable consent hash is running in production */
         public readonly bool $clearLegacyHash = false,
     ) {
     }

src/OpenConext/EngineBlock/Authentication/Value/ConsentVersion.php

@@ -23,6 +23,7 @@
 final class ConsentVersion
 {
     const STABLE = 'stable';
+    /** @deprecated Remove after stable consent hash is running in production */
     const UNSTABLE = 'unstable';
     const NOT_GIVEN = 'not-given';
 
@@ -36,6 +37,7 @@ public static function stable(): ConsentVersion
         return new self(self::STABLE);
     }
 
+    /** @deprecated Remove after stable consent hash is running in production */
     public static function unstable(): ConsentVersion
     {
         return new self(self::UNSTABLE);
@@ -70,6 +72,7 @@ public function __toString()
         return $this->consentVersion;
     }
 
+    /** @deprecated Remove after stable consent hash is running in production */
     public function isUnstable(): bool
     {
         return $this->consentVersion === self::UNSTABLE;

src/OpenConext/EngineBlock/Service/Consent/ConsentHashService.php

@@ -32,6 +32,7 @@
 
 final class ConsentHashService implements ConsentHashServiceInterface
 {
+    /** @deprecated Remove after stable consent hash is running in production */
     private const FEATURE_MIGRATION = 'eb.stable_consent_hash_migration';
 
     /**
@@ -101,6 +102,8 @@ public function getStableConsentHash(ConsentAttributes $attributes): string
     }
 
     /**
+     * @deprecated Remove after stable consent hash is running in production
+     *
      * The old way of calculating the attribute hash, this is not stable as a change of the attribute order,
      * change of case, stray/empty attributes, and renumbered indexes can cause the hash to change. Leaving the
      * user to give consent once again for a service she previously gave consent for.

src/OpenConext/EngineBlock/Service/Consent/ConsentHashServiceInterface.php

@@ -36,6 +36,7 @@ public function updateConsentHash(ConsentUpdateParameters $parameters): bool;
 
     public function countTotalConsent(string $consentUid): int;
 
+    /** @deprecated Remove after stable consent hash is running in production */
     public function getUnstableAttributesHash(array $attributes, bool $mustStoreValues): string;
 
     public function getStableConsentHash(ConsentAttributes $attributes): string;