Drop support for symfony 3.4

CHANGELOG.md

@@ -8,7 +8,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Added 
 * CI coverage for PHP 8.1 and 8.2
 ### Removed
-* Removed support for PHP < 7.4
+* Support for PHP < 7.4
+* Support for Symfony < 4.4
 * Docker images dev-dependency from https://github.com/ilario-pierbattista/docker-php-mongodb-bundle
 
 ## [1.5.0] (2022-01-06)

composer.json

@@ -22,12 +22,12 @@
         "php": "^7.4 || ^8.0",
         "ext-mongodb": "^1.6",
         "mongodb/mongodb": "^1.5",
-        "symfony/framework-bundle": "^3.4 || ^4.3 || ^5.0 || ^6.0"
+        "symfony/framework-bundle": "^4.4 || ^5.0 || ^6.0"
     },
     "require-dev": {
         "matthiasnoback/symfony-dependency-injection-test": "^4",
-        "symfony/web-profiler-bundle": "^3.4 || ^4.3 || ^5.0 || ^6.0",
-        "symfony/console": "^3.4 || ^4.3 || ^5.0 || ^6.0",
+        "symfony/web-profiler-bundle": "^4.4 || ^5.0 || ^6.0",
+        "symfony/console": "^4.4 || ^5.0 || ^6.0",
         "phpunit/phpunit": "^9.6.13",
         "symfony/phpunit-bridge": "^6.0",
         "facile-it/facile-coding-standard": "^0.4.0",

phpunit.xml.dist

@@ -13,7 +13,7 @@
   </coverage>
   <php>
     <ini name="error_reporting" value="-1"/>
-    <env name="SYMFONY_DEPRECATIONS_HELPER" value="disabled"/>
+    <env name="SYMFONY_DEPRECATIONS_HELPER" value="true"/>
   </php>
   <testsuites>
     <testsuite name="MongoBundle Test Suite">

rector.php

@@ -2,10 +2,11 @@
 
 declare(strict_types=1);
 
-use Rector\CodeQuality\Rector\Class_\InlineConstructorDefaultToPropertyRector;
 use Rector\Config\RectorConfig;
+use Rector\PHPUnit\PHPUnit100\Rector\Class_\AddProphecyTraitRector;
+use Rector\PHPUnit\Set\PHPUnitLevelSetList;
 use Rector\PHPUnit\Set\PHPUnitSetList;
-use Rector\Set\ValueObject\LevelSetList;
+use Rector\Symfony\Set\SymfonyLevelSetList;
 
 return static function (RectorConfig $rectorConfig): void {
     $rectorConfig->paths([
@@ -14,9 +15,11 @@
     ]);
 
     // register a single rule
-    $rectorConfig->rule(\Rector\PHPUnit\PHPUnit100\Rector\Class_\AddProphecyTraitRector::class);
+    $rectorConfig->rule(AddProphecyTraitRector::class);
 
     $rectorConfig->sets([
         PHPUnitSetList::PHPUNIT_90,
+        PHPUnitLevelSetList::UP_TO_PHPUNIT_90,
+        SymfonyLevelSetList::UP_TO_SYMFONY_44
     ]);
 };

src/Capsule/Collection.php

@@ -215,11 +215,7 @@ private function prepareQuery(string $method, $filters, $data, array $options):
         );
 
         $event = new QueryEvent($query);
-        if (Kernel::VERSION_ID >= 40300) {
-            $this->eventDispatcher->dispatch($event, QueryEvent::QUERY_PREPARED);
-        } else {
-            $this->eventDispatcher->dispatch(QueryEvent::QUERY_PREPARED, $event);
-        }
+        $this->eventDispatcher->dispatch($event, QueryEvent::QUERY_PREPARED);
 
         return $query;
     }
@@ -258,7 +254,7 @@ private function notifyQueryExecution(Query $queryLog)
         if (Kernel::VERSION_ID >= 40300) {
             $this->eventDispatcher->dispatch($event, QueryEvent::QUERY_EXECUTED);
         } else {
-            $this->eventDispatcher->dispatch(QueryEvent::QUERY_EXECUTED, $event);
+            $this->eventDispatcher->dispatch($event, QueryEvent::QUERY_EXECUTED);
         }
     }
 

src/Command/DropCollectionCommand.php

@@ -28,7 +28,7 @@ protected function configure()
     /**
      * {@inheritdoc}
      */
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $collection = $input->getArgument('collection');
 

src/Command/DropDatabaseCommand.php

@@ -26,7 +26,7 @@ protected function configure()
     /**
      * {@inheritdoc}
      */
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->io->writeln(sprintf('Dropping database %s', $this->connection->getDatabaseName()));
         $this->connection->drop();

src/Command/LoadFixturesCommand.php

@@ -42,7 +42,7 @@ protected function initialize(InputInterface $input, OutputInterface $output)
     /**
      * {@inheritdoc}
      */
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->io->writeln('Loading mongo fixtures');
         /** @var Application $application */

src/DependencyInjection/Configuration.php

@@ -21,9 +21,7 @@ final class Configuration implements ConfigurationInterface
     public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('mongo_db_bundle');
-        $rootBuilder = \method_exists(TreeBuilder::class, 'getRootNode')
-            ? $treeBuilder->getRootNode()
-            : $treeBuilder->root('mongo_db_bundle');
+        $rootBuilder = $treeBuilder->getRootNode();
 
         self::addDataCollection($rootBuilder->children());
         self::addClients($rootBuilder->children());

src/Services/ClientRegistry.php

@@ -121,11 +121,7 @@ public function getClient(string $name, ?string $databaseName = null): Client
             $this->clients[$clientKey] = $this->buildClient($name, $conf->getUri(), $options, $conf->getDriverOptions());
 
             $event = new ConnectionEvent($clientKey);
-            if (Kernel::VERSION_ID >= 40300) {
-                $this->eventDispatcher->dispatch($event, ConnectionEvent::CLIENT_CREATED);
-            } else {
-                $this->eventDispatcher->dispatch(ConnectionEvent::CLIENT_CREATED, $event);
-            }
+            $this->eventDispatcher->dispatch($event, ConnectionEvent::CLIENT_CREATED);
         }
 
         return $this->clients[$clientKey];

tests/Functional/Command/AbstractCommandTest.php

@@ -70,7 +70,7 @@ protected function configure()
             ->setDescription('fake test command');
     }
 
-    protected function execute(InputInterface $input, OutputInterface $output)
+    protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $this->io->writeln('Executed');