Fixing compat with symfony/web-link 4.4 and own impl of Fig link

Author: weaverryan

composer.json

@@ -32,8 +32,7 @@
         "symfony/framework-bundle": "^3.4 || ^4.0 || ^5.0",
         "symfony/phpunit-bridge": "^4.3.5 || ^5.0",
         "symfony/twig-bundle": "^3.4 || ^4.0 || ^5.0",
-        "symfony/web-link": "^3.4 || ^4.0 || ^5.0",
-        "fig/link-util": "^1.0"
+        "symfony/web-link": "^3.4 || ^4.0 || ^5.0"
     },
     "extra": {
         "thanks": {

src/EventListener/PreLoadAssetsEventListener.php

@@ -9,8 +9,10 @@
 
 namespace Symfony\WebpackEncoreBundle\EventListener;
 
-use Fig\Link\GenericLinkProvider;
-use Fig\Link\Link;
+use Fig\Link\GenericLinkProvider as FigGenericLinkProvider;
+use Fig\Link\Link as FigLink;
+use Symfony\Component\WebLink\GenericLinkProvider;
+use Symfony\Component\WebLink\Link;
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
 use Symfony\Component\HttpKernel\Event\ResponseEvent;
 use Symfony\WebpackEncoreBundle\Asset\TagRenderer;
@@ -39,16 +41,20 @@ public function onKernelResponse($event)
         $request = $event->getRequest();
 
         if (null === $linkProvider = $request->attributes->get('_links')) {
-            $request->attributes->set('_links', new GenericLinkProvider());
+            $request->attributes->set(
+                '_links',
+                // For backwards-compat with symfony/web-link 4.3 and lower
+                class_exists(GenericLinkProvider::class) ? new GenericLinkProvider() : new FigGenericLinkProvider()
+            );
         }
 
-        /** @var GenericLinkProvider $linkProvider */
+        /** @var GenericLinkProvider|FigGenericLinkProvider $linkProvider */
         $linkProvider = $request->attributes->get('_links');
         $defaultAttributes = $this->tagRenderer->getDefaultAttributes();
         $crossOrigin = $defaultAttributes['crossorigin'] ?? false;
 
         foreach ($this->tagRenderer->getRenderedScripts() as $href) {
-            $link = (new Link('preload', $href))->withAttribute('as', 'script');
+            $link = ($this->createLink('preload', $href))->withAttribute('as', 'script');
 
             if (false !== $crossOrigin) {
                 $link = $link->withAttribute('crossorigin', $crossOrigin);
@@ -58,7 +64,7 @@ public function onKernelResponse($event)
         }
 
         foreach ($this->tagRenderer->getRenderedStyles() as $href) {
-            $link = (new Link('preload', $href))->withAttribute('as', 'style');
+            $link = ($this->createLink('preload', $href))->withAttribute('as', 'style');
 
             if (false !== $crossOrigin) {
                 $link = $link->withAttribute('crossorigin', $crossOrigin);
@@ -77,4 +83,16 @@ public static function getSubscribedEvents()
             'kernel.response' => ['onKernelResponse', 50],
         ];
     }
+
+    /**
+     * For backwards-compat with symfony/web-link 4.3 and lower
+     *
+     * @return Link|FigLink
+     */
+    private function createLink(string $rel, string $href)
+    {
+        $class = class_exists(Link::class) ? Link::class : FigLink::class;
+
+        return new $class($rel, $href);
+    }
 }

tests/EventListener/PreLoadAssetsEventListenerTest.php

@@ -9,8 +9,10 @@
 
 namespace Symfony\WebpackEncoreBundle\Tests\Asset;
 
-use Fig\Link\GenericLinkProvider;
-use Fig\Link\Link;
+use Fig\Link\GenericLinkProvider as FigGenericLinkProvider;
+use Fig\Link\Link as FigLink;
+use Symfony\Component\WebLink\GenericLinkProvider;
+use Symfony\Component\WebLink\Link;
 use PHPUnit\Framework\TestCase;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;
@@ -35,10 +37,12 @@ public function testItPreloadsAssets()
         $listener = new PreLoadAssetsEventListener($tagRenderer);
         $listener->onKernelResponse($event);
         $this->assertTrue($request->attributes->has('_links'));
-        /** @var GenericLinkProvider $linkProvider */
+        /** @var GenericLinkProvider|FigGenericLinkProvider $linkProvider */
         $linkProvider = $request->attributes->get('_links');
-        $this->assertInstanceOf(GenericLinkProvider::class, $linkProvider);
-        /** @var Link[] $links */
+
+        $expectedProviderClass = class_exists(GenericLinkProvider::class) ? GenericLinkProvider::class : FigGenericLinkProvider::class;
+        $this->assertInstanceOf($expectedProviderClass, $linkProvider);
+        /** @var Link[]|FigLink[] $links */
         $links = array_values($linkProvider->getLinks());
         $this->assertCount(2, $links);
         $this->assertSame('/file1.js', $links[0]->getHref());
@@ -58,14 +62,16 @@ public function testItReusesExistingLinkProvider()
         $tagRenderer->expects($this->once())->method('getRenderedStyles')->willReturn([]);
 
         $request = new Request();
-        $linkProvider = new GenericLinkProvider([new Link('preload', 'bar.js')]);
+        $linkProviderClass = class_exists(GenericLinkProvider::class) ? GenericLinkProvider::class : FigGenericLinkProvider::class;
+        $linkClass = class_exists(Link::class) ? Link::class : FigLink::class;
+        $linkProvider = new $linkProviderClass([new $linkClass('preload', 'bar.js')]);
         $request->attributes->set('_links', $linkProvider);
 
         $response = new Response();
         $event = $this->createResponseEvent($request, HttpKernelInterface::MASTER_REQUEST, $response);
         $listener = new PreLoadAssetsEventListener($tagRenderer);
         $listener->onKernelResponse($event);
-        /** @var GenericLinkProvider $linkProvider */
+        /** @var GenericLinkProvider|FigGenericLinkProvider $linkProvider */
         $linkProvider = $request->attributes->get('_links');
         $this->assertCount(2, $linkProvider->getLinks());
     }