Add missing UPGRADE notes for 7.1

Author: wouterj

UPGRADE-7.1.md

@@ -30,7 +30,9 @@ Components
  * [Form](#Form)
  * [Intl](#Intl)
  * [HttpClient](#HttpClient)
- * [PropertyInfo](#PropertyInfo)
+ * [HttpKernel](#HttpKernel)
+ * [Security](#Security)
+ * [Serializer](#Serializer)
  * [Translation](#Translation)
  * [Workflow](#Workflow)
 
@@ -50,17 +52,66 @@ DependencyInjection
  * [BC BREAK] When used in the `prependExtension()` method, the `ContainerConfigurator::import()` method now prepends the configuration instead of appending it
  * Deprecate `#[TaggedIterator]` and `#[TaggedLocator]` attributes, use `#[AutowireIterator]` and `#[AutowireLocator]` instead
 
+   *Before*
+   ```php
+   use Symfony\Component\DependencyInjection\Attribute\TaggedIterator;
+   use Symfony\Component\DependencyInjection\Attribute\TaggedLocator;
+
+   class HandlerCollection
+   {
+       public function __construct(
+           #[TaggedIterator('app.handler', indexAttribute: 'key')]
+           iterable $handlers,
+
+           #[TaggedLocator('app.handler')]
+           private ContainerInterface $locator,
+       ) {
+       }
+   }
+   ```
+
+   *After*
+   ```php
+   use Symfony\Component\DependencyInjection\Attribute\AutowireIterator;
+   use Symfony\Component\DependencyInjection\Attribute\AutowireLocator;
+
+   class HandlerCollection
+   {
+       public function __construct(
+           #[AutowireIterator('app.handler', indexAttribute: 'key')]
+           iterable $handlers,
+
+           #[AutowireLocator('app.handler')]
+           private ContainerInterface $locator,
+       ) {
+       }
+   }
+   ```
+
 DoctrineBridge
 --------------
 
- * Deprecated `DoctrineExtractor::getTypes()`, use `DoctrineExtractor::getType()` instead
+ * Deprecate `DoctrineExtractor::getTypes()`, use `DoctrineExtractor::getType()` instead
+ * Mark class `ProxyCacheWarmer` as `final`
 
 ExpressionLanguage
 ------------------
 
  * Deprecate passing `null` as the allowed variable names to `ExpressionLanguage::lint()` and `Parser::lint()`,
    pass the `IGNORE_UNKNOWN_VARIABLES` flag instead to ignore unknown variables during linting
 
+   *Before*
+   ```php
+   $expressionLanguage->lint('a + 1', null);
+   ```
+
+   *After*
+   ```php
+   use Symfony\Component\ExpressionLanguage\Parser;
+
+   $expressionLanguage->lint('a + 1', [], Parser::IGNORE_UNKNOWN_VARIABLES);
+   ```
+
 Form
 ----
 
@@ -69,6 +120,7 @@ Form
 FrameworkBundle
 ---------------
 
+ * [BC BREAK] Enabling `framework.rate_limiter` requires `symfony/rate-limiter` 7.1 or higher
  * Mark classes `ConfigBuilderCacheWarmer`, `Router`, `SerializerCacheWarmer`, `TranslationsCacheWarmer`, `Translator` and `ValidatorCacheWarmer` as `final`
  * Deprecate the `router.cache_dir` config option, the Router will always use the `kernel.build_dir` parameter
  * Reset env vars when resetting the container
@@ -78,6 +130,37 @@ HttpClient
 
  * Deprecate the `setLogger()` methods of the `NoPrivateNetworkHttpClient`, `TraceableHttpClient` and `ScopingHttpClient` classes, configure the logger of the wrapped clients directly instead
 
+   *Before*
+   ```php
+   // ...
+   use Symfony\Component\HttpClient\HttpClient;
+   use Symfony\Component\HttpClient\NoPrivateNetworkHttpClient;
+
+   $publicClient = new NoPrivateNetworkHttpClient(HttpClient::create());
+   $publicClient->setLogger(new Logger());
+   ```
+
+   *After*
+   ```php
+   // ...
+   use Symfony\Component\HttpClient\HttpClient;
+   use Symfony\Component\HttpClient\NoPrivateNetworkHttpClient;
+
+   $client = HttpClient::create();
+   $client->setLogger(new Logger());
+
+   $publicClient = new NoPrivateNetworkHttpClient($client);
+   ```
+
+HttpKernel
+----------
+
+ * The `Extension` class is marked as internal, extend the `Extension` class from the DependencyInjection component instead
+ * Deprecate `Extension::addAnnotatedClassesToCompile()`
+ * Deprecate `AddAnnotatedClassesToCachePass`
+ * Deprecate the `setAnnotatedClassCache()` and `getAnnotatedClassesToCompile()` methods of the `Kernel` class
+ * Deprecate the `addAnnotatedClassesToCompile()` and `getAnnotatedClassesToCompile()` methods of the `Extension` class
+
 Intl
 ----
 
@@ -89,18 +172,49 @@ Mailer
 
  * Postmark's "406 - Inactive recipient" API error code now results in a `PostmarkDeliveryEvent` instead of throwing a `HttpTransportException`
 
-HttpKernel
-----------
+Security
+--------
 
- * The `Extension` class is marked as internal, extend the `Extension` class from the DependencyInjection component instead
- * Deprecate `Extension::addAnnotatedClassesToCompile()`
- * Deprecate `AddAnnotatedClassesToCachePass`
- * Deprecate the `setAnnotatedClassCache()` and `getAnnotatedClassesToCompile()` methods of the `Kernel` class
+ * Change the first and second argument of `OidcTokenHandler` to `Jose\Component\Core\AlgorithmManager` and `Jose\Component\Core\JWKSet` respectively
 
 SecurityBundle
 --------------
 
  * Mark class `ExpressionCacheWarmer` as `final`
+ * Deprecate options `algorithm` and `key` of `oidc` token handler, use
+   `algorithms` and `keyset` instead
+
+   *Before*
+   ```yaml
+   security:
+       firewalls:
+           main:
+               access_token:
+                   token_handler:
+                       oidc:
+                           algorithm: 'ES256'
+                           key: '{"kty":"...","k":"..."}'
+                           # ...
+   ```
+
+   *After*
+   ```yaml
+   security:
+       firewalls:
+           main:
+               access_token:
+                   token_handler:
+                       oidc:
+                           algorithms: ['ES256']
+                           keyset: '{"keys":[{"kty":"...","k":"..."}]}'
+                           # ...
+   ```
+ * Deprecate the `security.access_token_handler.oidc.jwk` service, use `security.access_token_handler.oidc.jwkset` instead
+
+Serializer
+----------
+
+ * Deprecate the `withDefaultContructorArguments()` method of `AbstractNormalizerContextBuilder`, use `withDefaultContructorArguments()` instead (note the typo in the old method name)
 
 Translation
 -----------
@@ -111,12 +225,13 @@ TwigBundle
 ----------
 
  * Mark class `TemplateCacheWarmer` as `final`
+ * Deprecate the `base_template_class` config option, this option is no-op when using Twig 3+
 
 Validator
 ---------
 
  * Deprecate not passing a value for the `requireTld` option to the `Url` constraint (the default value will become `true` in 8.0)
- * Deprecate `Bic::INVALID_BANK_CODE_ERROR`
+ * Deprecate `Bic::INVALID_BANK_CODE_ERROR`, as ISO 9362 defines no restrictions on BIC bank code characters
 
 Workflow
 --------

src/Symfony/Component/ExpressionLanguage/ExpressionLanguage.php

@@ -101,7 +101,7 @@ public function parse(Expression|string $expression, array $names, int $flags =
     public function lint(Expression|string $expression, ?array $names, int $flags = 0): void
     {
         if (null === $names) {
-            trigger_deprecation('symfony/expression-language', '7.1', 'Passing "null" as the second argument of "%s()" is deprecated, pass "self::IGNORE_UNKNOWN_VARIABLES" instead as a third argument.', __METHOD__);
+            trigger_deprecation('symfony/expression-language', '7.1', 'Passing "null" as the second argument of "%s()" is deprecated, pass "%s\Parser::IGNORE_UNKNOWN_VARIABLES" instead as a third argument.', __METHOD__, __NAMESPACE__);
 
             $flags |= Parser::IGNORE_UNKNOWN_VARIABLES;
             $names = [];

src/Symfony/Component/ExpressionLanguage/Parser.php

@@ -112,7 +112,7 @@ public function parse(TokenStream $stream, array $names = [], int $flags = 0): N
     public function lint(TokenStream $stream, ?array $names = [], int $flags = 0): void
     {
         if (null === $names) {
-            trigger_deprecation('symfony/expression-language', '7.1', 'Passing "null" as the second argument of "%s()" is deprecated, pass "self::IGNORE_UNKNOWN_VARIABLES" instead as a third argument.', __METHOD__);
+            trigger_deprecation('symfony/expression-language', '7.1', 'Passing "null" as the second argument of "%s()" is deprecated, pass "%s::IGNORE_UNKNOWN_VARIABLES" instead as a third argument.', __METHOD__, __CLASS__);
 
             $flags |= self::IGNORE_UNKNOWN_VARIABLES;
             $names = [];

src/Symfony/Component/PropertyInfo/Extractor/ConstructorArgumentTypeExtractorInterface.php

@@ -28,8 +28,6 @@ interface ConstructorArgumentTypeExtractorInterface
      *
      * @return LegacyType[]|null
      *
-     * @deprecated since Symfony 7.1, use "getTypeFromConstructor" instead
-     *
      * @internal
      */
     public function getTypesFromConstructor(string $class, string $property): ?array;

src/Symfony/Component/PropertyInfo/PropertyInfoCacheExtractor.php

@@ -64,9 +64,6 @@ public function getType(string $class, string $property, array $context = []): ?
         return $this->extract('getType', [$class, $property, $context]);
     }
 
-    /**
-     * @deprecated since Symfony 7.1, use "getType" instead
-     */
     public function getTypes(string $class, string $property, array $context = []): ?array
     {
         return $this->extract('getTypes', [$class, $property, $context]);