Skip to content

Commit

Permalink
Merge pull request #280 from rieschl/qa/mongodb-deprecation
Browse files Browse the repository at this point in the history
Avoid using deprecated methods
  • Loading branch information
driehle authored Oct 15, 2023
2 parents 2f0c146 + aeadd0e commit 82493b8
Show file tree
Hide file tree
Showing 7 changed files with 14 additions and 2 deletions.
1 change: 1 addition & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@
"require": {
"php": "~8.0.0 || ~8.1.0 || ~8.2.0",
"ext-mongodb": "*",
"doctrine/cache": "^1.11 || ^2.0",
"doctrine/doctrine-laminas-hydrator": "^3.3.0",
"doctrine/doctrine-module": "^6.0.4",
"doctrine/event-manager": "^1.2.0 || ^2.0.0",
Expand Down
2 changes: 2 additions & 0 deletions src/Options/Configuration.php
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@

/**
* Configuration options for doctrine mongo
*
* @extends AbstractOptions<int|string|array<string, string>|null>
*/
final class Configuration extends AbstractOptions
{
Expand Down
2 changes: 2 additions & 0 deletions src/Options/Connection.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@

/**
* Connection options for doctrine mongo
*
* @extends AbstractOptions<int|string|array|null>
*/
final class Connection extends AbstractOptions
{
Expand Down
2 changes: 2 additions & 0 deletions src/Options/DocumentManager.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@

/**
* Document manager options for doctrine mongo
*
* @extends AbstractOptions<string>
*/
final class DocumentManager extends AbstractOptions
{
Expand Down
2 changes: 2 additions & 0 deletions src/Options/MongoLoggerCollector.php
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,8 @@

/**
* Configuration options for a collector
*
* @extends AbstractOptions<string>
*/
final class MongoLoggerCollector extends AbstractOptions
{
Expand Down
4 changes: 3 additions & 1 deletion src/Service/ConfigurationFactory.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@

namespace DoctrineMongoODMModule\Service;

use Doctrine\Common\Cache\Psr6\CacheAdapter;
use Doctrine\ODM\MongoDB\Configuration;
use Doctrine\ODM\MongoDB\Types\Type;
use DoctrineMongoODMModule\Options;
Expand Down Expand Up @@ -68,7 +69,8 @@ public function __invoke(ContainerInterface $container, $requestedName, array|nu
}

// caching
$config->setMetadataCacheImpl($container->get($configurationOptions->getMetadataCache()));
$cache = $container->get($configurationOptions->getMetadataCache());
$config->setMetadataCache(CacheAdapter::wrap($cache));

// Register filters
foreach ($configurationOptions->getFilters() as $alias => $class) {
Expand Down
3 changes: 2 additions & 1 deletion tests/Doctrine/ConfigurationFactoryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
namespace DoctrineMongoODMModuleTest\Doctrine;

use Doctrine\Common\Cache\Cache;
use Doctrine\Common\Cache\Psr6\CacheAdapter;
use Doctrine\ODM\MongoDB\APM\CommandLoggerInterface;
use Doctrine\ODM\MongoDB\Configuration as Config;
use Doctrine\ODM\MongoDB\PersistentCollection\PersistentCollectionFactory;
Expand Down Expand Up @@ -99,7 +100,7 @@ public function testCreation(): void

$this->assertInstanceOf(Config::class, $config);

$this->assertSame($metadataCache, $config->getMetadataCacheImpl());
$this->assertEquals(CacheAdapter::wrap($metadataCache), $config->getMetadataCache());
$this->assertSame($mappingDriver, $config->getMetadataDriverImpl());

$this->assertSame(Config::AUTOGENERATE_EVAL, $config->getAutoGenerateProxyClasses());
Expand Down

0 comments on commit 82493b8

Please sign in to comment.