Require PHP 8.2 and modernize codebase (#284)

## Summary
- Require PHP 8.2 as minimum version (PHP 8.0 and 8.1 are EOL)
- Use constructor property promotion throughout
- Add typed properties where missing
- Remove deprecated `{@inheritdoc}` annotations

Author: TheMilek

.github/workflows/build.yml

@@ -17,14 +17,12 @@ jobs:
         strategy:
             fail-fast: false
             matrix:
-                php: ["8.1", "8.2", "8.3"]
+                php: ["8.2", "8.3"]
                 symfony: ["^5.4", "^6.4", "^7.0"]
                 twig: ["^2.12", "^3.3"]
                 exclude:
                     - twig: "^2.12"
                       symfony: "^7.0"
-                    - php: "8.1"
-                      symfony: "^7.0"
 
         steps:
             -

composer.json

@@ -24,7 +24,7 @@
         }
     ],
     "require": {
-        "php": "^8.1",
+        "php": "^8.2",
         "sylius-labs/polyfill-symfony-event-dispatcher": "^1.0",
         "symfony/config": "^5.4 || ^6.4 || ^7.0",
         "symfony/deprecation-contracts": "^2.1 || ^3.0",

src/Bundle/DependencyInjection/SyliusMailerExtension.php

@@ -23,9 +23,6 @@
 
 final class SyliusMailerExtension extends ConfigurableExtension
 {
-    /**
-     * {@inheritdoc}
-     */
     protected function loadInternal(array $mergedConfig, ContainerBuilder $container): void
     {
         $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));

src/Bundle/Renderer/Adapter/EmailDefaultAdapter.php

@@ -20,9 +20,6 @@
 
 final class EmailDefaultAdapter extends AbstractAdapter
 {
-    /** @var EventDispatcherInterface|null */
-    protected $dispatcher;
-
     public function __construct(?EventDispatcherInterface $dispatcher = null)
     {
         $this->dispatcher = $dispatcher;

src/Bundle/Renderer/Adapter/EmailTwigAdapter.php

@@ -18,27 +18,19 @@
 use Sylius\Component\Mailer\Renderer\Adapter\AbstractAdapter;
 use Sylius\Component\Mailer\Renderer\RenderedEmail;
 use Sylius\Component\Mailer\SyliusMailerEvents;
-use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
+use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Twig\Environment;
 use Twig\Loader\ArrayLoader;
 
 class EmailTwigAdapter extends AbstractAdapter
 {
-    /** @var Environment */
-    protected $twig;
-
-    /** @var EventDispatcherInterface|null */
-    protected $dispatcher;
-
-    public function __construct(Environment $twig, ?EventDispatcherInterface $dispatcher = null)
-    {
-        $this->twig = $twig;
+    public function __construct(
+        protected Environment $twig,
+        ?EventDispatcherInterface $dispatcher = null,
+    ) {
         $this->dispatcher = $dispatcher;
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function render(EmailInterface $email, array $data = []): RenderedEmail
     {
         $renderedEmail = $this->getRenderedEmail($email, $data);

src/Bundle/Sender/Adapter/SymfonyMailerAdapter.php

@@ -32,9 +32,6 @@ public function __construct(private MailerInterface $mailer)
     {
     }
 
-    /**
-     * {@inheritdoc}
-     */
     public function send(
         array $recipients,
         string $senderAddress,

src/Bundle/spec/Renderer/Adapter/EmailTwigAdapterSpec.php

@@ -20,7 +20,7 @@
 use Sylius\Component\Mailer\Renderer\Adapter\AbstractAdapter;
 use Sylius\Component\Mailer\Renderer\RenderedEmail;
 use Sylius\Component\Mailer\SyliusMailerEvents;
-use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
+use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Twig\Environment;
 use Twig\Template;
 use Twig\TemplateWrapper;

src/Bundle/spec/Sender/Adapter/SymfonyMailerAdapterSpec.php

@@ -20,11 +20,11 @@
 use Sylius\Component\Mailer\Renderer\RenderedEmail;
 use Sylius\Component\Mailer\Sender\Adapter\AbstractAdapter;
 use Sylius\Component\Mailer\SyliusMailerEvents;
+use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 use Symfony\Component\Mailer\Exception\TransportException;
 use Symfony\Component\Mailer\MailerInterface;
 use Symfony\Component\Mime\Address;
 use Symfony\Component\Mime\Email;
-use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
 
 final class SymfonyMailerAdapterSpec extends ObjectBehavior
 {

src/Component/Event/EmailRenderEvent.php

@@ -18,16 +18,13 @@
 
 class EmailRenderEvent extends Event
 {
-    /** @var RenderedEmail */
-    protected $renderedEmail;
-
-    /** @var string[] */
-    protected $recipients;
-
-    public function __construct(RenderedEmail $renderedEmail, array $recipients = [])
-    {
-        $this->renderedEmail = $renderedEmail;
-        $this->recipients = $recipients;
+    /**
+     * @param string[] $recipients
+     */
+    public function __construct(
+        protected RenderedEmail $renderedEmail,
+        protected array $recipients = [],
+    ) {
     }
 
     public function getRenderedEmail(): RenderedEmail

src/Component/Factory/EmailFactory.php

@@ -18,9 +18,6 @@
 
 class EmailFactory implements EmailFactoryInterface
 {
-    /**
-     * {@inheritdoc}
-     */
     public function createNew(): EmailInterface
     {
         return new Email();