feat(doctrine)!: rename FileMappingLoader to MappingContext

Author: kbond

src/Filesystem/Doctrine/FileMappingLoader.php renamed to src/Filesystem/Doctrine/MappingContext.php

@@ -17,7 +17,7 @@
 /**
  * @author Kevin Bond <[email protected]>
  */
-final class FileMappingLoader
+final class MappingContext
 {
     /** @var \WeakMap<object,true> */
     private \WeakMap $loaded;

src/Filesystem/Doctrine/Twig/FileMappingLoaderExtension.php renamed to src/Filesystem/Doctrine/Twig/MappingContextExtension.php

@@ -14,24 +14,24 @@
 use Twig\Extension\AbstractExtension;
 use Twig\TwigFilter;
 use Twig\TwigFunction;
-use Zenstruck\Filesystem\Doctrine\FileMappingLoader;
+use Zenstruck\Filesystem\Doctrine\MappingContext;
 
 /**
  * @author Kevin Bond <[email protected]>
  */
-final class FileMappingLoaderExtension extends AbstractExtension
+final class MappingContextExtension extends AbstractExtension
 {
     public function getFilters(): array
     {
         return [
-            new TwigFilter('load_files', [FileMappingLoader::class, '__invoke']),
+            new TwigFilter('load_files', [MappingContext::class, '__invoke']),
         ];
     }
 
     public function getFunctions(): array
     {
         return [
-            new TwigFunction('load_files', [FileMappingLoader::class, '__invoke']),
+            new TwigFunction('load_files', [MappingContext::class, '__invoke']),
         ];
     }
 }

src/Filesystem/Symfony/DependencyInjection/ZenstruckFilesystemExtension.php

@@ -29,8 +29,8 @@
 use Zenstruck\Filesystem;
 use Zenstruck\Filesystem\Doctrine\EventListener\NodeLifecycleListener;
 use Zenstruck\Filesystem\Doctrine\EventListener\NodeMappingListener;
-use Zenstruck\Filesystem\Doctrine\FileMappingLoader;
-use Zenstruck\Filesystem\Doctrine\Twig\FileMappingLoaderExtension;
+use Zenstruck\Filesystem\Doctrine\MappingContext;
+use Zenstruck\Filesystem\Doctrine\Twig\MappingContextExtension;
 use Zenstruck\Filesystem\Event\EventDispatcherFilesystem;
 use Zenstruck\Filesystem\FilesystemRegistry;
 use Zenstruck\Filesystem\Flysystem\AdapterFactory;
@@ -128,7 +128,7 @@ private function registerDoctrine(ContainerBuilder $container, array $config): v
             ->addTag('doctrine.event_listener', ['event' => 'loadClassMetadata'])
         ;
 
-        $container->register(FileMappingLoader::class)
+        $container->register(MappingContext::class)
             ->setArguments([
                 new Reference('doctrine'),
                 new Reference('.zenstruck_filesystem.doctrine.lifecycle_listener'),
@@ -156,7 +156,7 @@ private function registerDoctrine(ContainerBuilder $container, array $config): v
         }
 
         if (isset($container->getParameter('kernel.bundles')['TwigBundle'])) {
-            $container->register('.zenstruck_filesystem.doctrine.twig_extension', FileMappingLoaderExtension::class)
+            $container->register('.zenstruck_filesystem.doctrine.twig_extension', MappingContextExtension::class)
                 ->addTag('twig.extension')
             ;
         }

tests/Filesystem/Doctrine/FileMappingLoaderTest.php renamed to tests/Filesystem/Doctrine/MappingContextTest.php

@@ -11,7 +11,7 @@
 
 namespace Zenstruck\Tests\Filesystem\Doctrine;
 
-use Zenstruck\Filesystem\Doctrine\FileMappingLoader;
+use Zenstruck\Filesystem\Doctrine\MappingContext;
 use Zenstruck\Filesystem\Node\Directory\LazyDirectory;
 use Zenstruck\Filesystem\Node\File;
 use Zenstruck\Filesystem\Node\File\Image\LazyImage;
@@ -23,7 +23,7 @@
 /**
  * @author Kevin Bond <[email protected]>
  */
-final class FileMappingLoaderTest extends DoctrineTestCase
+final class MappingContextTest extends DoctrineTestCase
 {
     /**
      * @test
@@ -50,9 +50,9 @@ public function can_load_files_for_object(): void
         $this->em()->clear();
 
         $fromDb = repository(Entity2::class)->first()->object();
-        $fromDb = self::getContainer()->get(FileMappingLoader::class)($fromDb);
-        $fromDb = self::getContainer()->get(FileMappingLoader::class)($fromDb); // ensure multiple calls work
-        self::getContainer()->get(FileMappingLoader::class)([$fromDb]);
+        $fromDb = self::getContainer()->get(MappingContext::class)($fromDb);
+        $fromDb = self::getContainer()->get(MappingContext::class)($fromDb); // ensure multiple calls work
+        self::getContainer()->get(MappingContext::class)([$fromDb]);
 
         $this->assertInstanceOf(LazyFile::class, $fromDb->getFile1());
         $this->assertSame('content1', $fromDb->getFile1()->contents());

tests/Fixtures/Service.php

@@ -12,7 +12,7 @@
 namespace Zenstruck\Tests\Fixtures;
 
 use Zenstruck\Filesystem;
-use Zenstruck\Filesystem\Doctrine\FileMappingLoader;
+use Zenstruck\Filesystem\Doctrine\MappingContext;
 use Zenstruck\Filesystem\MultiFilesystem;
 use Zenstruck\Filesystem\Node\PathGenerator;
 
@@ -30,7 +30,7 @@ public function __construct(
         public Filesystem $staticFilesystem,
         public MultiFilesystem $multiFilesystem,
         public PathGenerator $pathGenerator,
-        public FileMappingLoader $objectFileLoader,
+        public MappingContext $objectFileLoader,
     ) {
     }
 }