feat(doctrine)!: rename MappingContext::__invoke() to load()

Author: kbond

src/Filesystem/Doctrine/MappingContext.php

@@ -37,12 +37,12 @@ public function __construct(private ManagerRegistry $registry, private NodeLifec
      *
      * @return T
      */
-    public function __invoke(object|iterable $object): object|iterable
+    public function load(object|iterable $object): object|iterable
     {
         if (\is_iterable($object)) {
             foreach ($object as $item) {
                 if (\is_object($item)) {
-                    $this($item);
+                    $this->load($item);
                 }
             }
 

src/Filesystem/Doctrine/Twig/MappingContextExtension.php

@@ -24,14 +24,14 @@ final class MappingContextExtension extends AbstractExtension
     public function getFilters(): array
     {
         return [
-            new TwigFilter('load_files', [MappingContext::class, '__invoke']),
+            new TwigFilter('load_files', [MappingContext::class, 'load']),
         ];
     }
 
     public function getFunctions(): array
     {
         return [
-            new TwigFunction('load_files', [MappingContext::class, '__invoke']),
+            new TwigFunction('load_files', [MappingContext::class, 'load']),
         ];
     }
 }

tests/Filesystem/Doctrine/DoctrineTestCase.php

@@ -53,6 +53,6 @@ protected function em(): EntityManagerInterface
 
     protected function loadMappingFor(object $object): object
     {
-        return (self::getContainer()->get(Service::class)->objectFileLoader)($object);
+        return self::getContainer()->get(Service::class)->objectFileLoader->load($object);
     }
 }

tests/Filesystem/Doctrine/MappingContextTest.php

@@ -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(MappingContext::class)($fromDb);
-        $fromDb = self::getContainer()->get(MappingContext::class)($fromDb); // ensure multiple calls work
-        self::getContainer()->get(MappingContext::class)([$fromDb]);
+        $fromDb = self::getContainer()->get(MappingContext::class)->load($fromDb);
+        $fromDb = self::getContainer()->get(MappingContext::class)->load($fromDb); // ensure multiple calls work
+        self::getContainer()->get(MappingContext::class)->load([$fromDb]);
 
         $this->assertInstanceOf(LazyFile::class, $fromDb->getFile1());
         $this->assertSame('content1', $fromDb->getFile1()->contents());