Fix broken resource tracking

rollerworks-graveyard · Dec 3, 2019 · 1cf5cb1 · 1cf5cb1
1 parent 8fca897
commit 1cf5cb1
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/src/DependencyInjection/Compiler/RouteResourcePass.php b/src/DependencyInjection/Compiler/RouteResourcePass.php
@@ -35,6 +35,6 @@ function ($id) {
             array_keys($container->findTaggedServiceIds(self::TAG_NAME))
         );
 
-        $container->getDefinition('rollerworks_route_autowiring.route_loader')->replaceArgument(2, $trackedResources);
+        $container->getDefinition('rollerworks_route_autowiring.route_loader')->replaceArgument(1, $trackedResources);
     }
 }
diff --git a/src/Resources/config/services/routing.xml b/src/Resources/config/services/routing.xml
@@ -8,7 +8,6 @@
         <service id="rollerworks_route_autowiring.route_loader" class="Rollerworks\Bundle\RouteAutowiringBundle\RouteSlotLoader" public="false">
             <tag name="routing.loader" />
             <argument type="service" id="service_container" />
-            <argument type="collection" /> <!-- Routing imports. Populated by RouteAutowiringPass -->
             <argument type="collection" /> <!-- Tracked resources. Populated by RouteAutowiringPass -->
         </service>
 

diff --git a/tests/DependencyInjection/Compiler/RouteResourcePassTest.php b/tests/DependencyInjection/Compiler/RouteResourcePassTest.php
@@ -42,7 +42,7 @@ public function it_registers_the_route_resources()
         $this->compile();
 
         $loaderDef = $this->container->findDefinition('rollerworks_route_autowiring.route_loader');
-        $resources = $loaderDef->getArgument(2);
+        $resources = $loaderDef->getArgument(1);
 
         // Resources are provided as service References, but id cannot be predicted.
         // So instead loop trough each and analyze the actual referenced service definition.