Skip to content

Commit bdbc293

Browse files
weaverryanfabpot
weaverryan
authored and
fabpot
committed
[Asset] [AssetMapper] New AssetMapper component: Map assets to publicly available, versioned paths
1 parent d06bef8 commit bdbc293

File tree

11 files changed

+422
-2
lines changed

11 files changed

+422
-2
lines changed

‎Controller/AbstractController.php‎

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@
1414
use Psr\Container\ContainerInterface;
1515
use Psr\Link\EvolvableLinkInterface;
1616
use Psr\Link\LinkInterface;
17+
use Symfony\Component\AssetMapper\ImportMap\ImportMapManager;
1718
use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;
1819
use Symfony\Component\DependencyInjection\ParameterBag\ContainerBagInterface;
1920
use Symfony\Component\Form\Extension\Core\Type\FormType;
@@ -44,6 +45,7 @@
4445
use Symfony\Component\WebLink\EventListener\AddLinkHeaderListener;
4546
use Symfony\Component\WebLink\GenericLinkProvider;
4647
use Symfony\Component\WebLink\HttpHeaderSerializer;
48+
use Symfony\Component\WebLink\Link;
4749
use Symfony\Contracts\Service\Attribute\Required;
4850
use Symfony\Contracts\Service\ServiceSubscriberInterface;
4951
use Twig\Environment;
@@ -95,6 +97,7 @@ public static function getSubscribedServices(): array
9597
'security.csrf.token_manager' => '?'.CsrfTokenManagerInterface::class,
9698
'parameter_bag' => '?'.ContainerBagInterface::class,
9799
'web_link.http_header_serializer' => '?'.HttpHeaderSerializer::class,
100+
'asset_mapper.importmap.manager' => '?'.ImportMapManager::class,
98101
];
99102
}
100103

@@ -409,7 +412,7 @@ protected function addLink(Request $request, LinkInterface $link): void
409412
/**
410413
* @param LinkInterface[] $links
411414
*/
412-
protected function sendEarlyHints(iterable $links, Response $response = null): Response
415+
protected function sendEarlyHints(iterable $links = [], Response $response = null, bool $preloadJavaScriptModules = false): Response
413416
{
414417
if (!$this->container->has('web_link.http_header_serializer')) {
415418
throw new \LogicException('You cannot use the "sendEarlyHints" method if the WebLink component is not available. Try running "composer require symfony/web-link".');
@@ -418,6 +421,17 @@ protected function sendEarlyHints(iterable $links, Response $response = null): R
418421
$response ??= new Response();
419422

420423
$populatedLinks = [];
424+
425+
if ($preloadJavaScriptModules) {
426+
if (!$this->container->has('asset_mapper.importmap.manager')) {
427+
throw new \LogicException('You cannot use the JavaScript modules method if the AssetMapper component is not available. Try running "composer require symfony/asset-mapper".');
428+
}
429+
430+
foreach ($this->container->get('asset_mapper.importmap.manager')->getModulesToPreload() as $url) {
431+
$populatedLinks[] = new Link('modulepreload', $url);
432+
}
433+
}
434+
421435
foreach ($links as $link) {
422436
if ($link instanceof EvolvableLinkInterface && !$link->getRels()) {
423437
$link = $link->withRel('preload');

‎DependencyInjection/Compiler/UnusedTagsPass.php‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ class UnusedTagsPass implements CompilerPassInterface
2424
private const KNOWN_TAGS = [
2525
'annotations.cached_reader',
2626
'assets.package',
27+
'asset_mapper.compiler',
2728
'auto_alias',
2829
'cache.pool',
2930
'cache.pool.clearer',

‎DependencyInjection/Configuration.php‎

Lines changed: 94 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,8 @@
1616
use Psr\Log\LogLevel;
1717
use Symfony\Bundle\FullStack;
1818
use Symfony\Component\Asset\Package;
19+
use Symfony\Component\AssetMapper\AssetMapper;
20+
use Symfony\Component\AssetMapper\ImportMap\ImportMapManager;
1921
use Symfony\Component\Cache\Adapter\DoctrineAdapter;
2022
use Symfony\Component\Config\Definition\Builder\ArrayNodeDefinition;
2123
use Symfony\Component\Config\Definition\Builder\NodeBuilder;
@@ -161,6 +163,7 @@ public function getConfigTreeBuilder(): TreeBuilder
161163
$this->addSessionSection($rootNode);
162164
$this->addRequestSection($rootNode);
163165
$this->addAssetsSection($rootNode, $enableIfStandalone);
166+
$this->addAssetMapperSection($rootNode, $enableIfStandalone);
164167
$this->addTranslatorSection($rootNode, $enableIfStandalone);
165168
$this->addValidationSection($rootNode, $enableIfStandalone);
166169
$this->addAnnotationsSection($rootNode, $willBeAvailable);
@@ -810,6 +813,97 @@ private function addAssetsSection(ArrayNodeDefinition $rootNode, callable $enabl
810813
;
811814
}
812815

816+
private function addAssetMapperSection(ArrayNodeDefinition $rootNode, callable $enableIfStandalone): void
817+
{
818+
$rootNode
819+
->children()
820+
->arrayNode('asset_mapper')
821+
->info('Asset Mapper configuration')
822+
->{$enableIfStandalone('symfony/asset-mapper', AssetMapper::class)}()
823+
->fixXmlConfig('path')
824+
->fixXmlConfig('extension')
825+
->fixXmlConfig('importmap_script_attribute')
826+
->children()
827+
// add array node called "paths" that will be an array of strings
828+
->arrayNode('paths')
829+
->info('Directories that hold assets that should be in the mapper. Can be a simple array of an array of ["path/to/assets": "namespace"]')
830+
->example(['assets/'])
831+
->normalizeKeys(false)
832+
->useAttributeAsKey('namespace')
833+
->beforeNormalization()
834+
->always()
835+
->then(function ($v) {
836+
$result = [];
837+
foreach ($v as $key => $item) {
838+
// "dir" => "namespace"
839+
if (\is_string($key)) {
840+
$result[$key] = $item;
841+
842+
continue;
843+
}
844+
845+
if (\is_array($item)) {
846+
// $item = ["namespace" => "the/namespace", "value" => "the/dir"]
847+
$result[$item['value']] = $item['namespace'] ?? '';
848+
} else {
849+
// $item = "the/dir"
850+
$result[$item] = '';
851+
}
852+
}
853+
854+
return $result;
855+
})
856+
->end()
857+
->prototype('scalar')->end()
858+
->end()
859+
->booleanNode('server')
860+
->info('If true, a "dev server" will return the assets from the public directory (true in "debug" mode only by default)')
861+
->defaultValue($this->debug)
862+
->end()
863+
->scalarNode('public_prefix')
864+
->info('The public path where the assets will be written to (and served from when "server" is true)')
865+
->defaultValue('/assets/')
866+
->end()
867+
->booleanNode('strict_mode')
868+
->info('If true, an exception will be thrown if an asset cannot be found when imported from JavaScript or CSS files - e.g. "import \'./non-existent.js\'"')
869+
->defaultValue(true)
870+
->end()
871+
->arrayNode('extensions')
872+
->info('Key-value pair of file extensions set to their mime type.')
873+
->normalizeKeys(false)
874+
->useAttributeAsKey('extension')
875+
->example(['.zip' => 'application/zip'])
876+
->prototype('scalar')->end()
877+
->end()
878+
->scalarNode('importmap_path')
879+
->info('The path of the importmap.php file.')
880+
->defaultValue('%kernel.project_dir%/importmap.php')
881+
->end()
882+
->scalarNode('importmap_polyfill')
883+
->info('URL of the ES Module Polyfill to use, false to disable. Defaults to using a CDN URL.')
884+
->defaultValue(null)
885+
->end()
886+
->arrayNode('importmap_script_attributes')
887+
->info('Key-value pair of attributes to add to script tags output for the importmap.')
888+
->normalizeKeys(false)
889+
->useAttributeAsKey('key')
890+
->example(['data-turbo-track' => 'reload'])
891+
->prototype('scalar')->end()
892+
->end()
893+
->scalarNode('vendor_dir')
894+
->info('The directory to store JavaScript vendors.')
895+
->defaultValue('%kernel.project_dir%/assets/vendor')
896+
->end()
897+
->scalarNode('provider')
898+
->info('The provider (CDN) to use', class_exists(ImportMapManager::class) ? sprintf(' (e.g.: "%s").', implode('", "', ImportMapManager::PROVIDERS)) : '.')
899+
->defaultValue('jspm')
900+
->end()
901+
->end()
902+
->end()
903+
->end()
904+
;
905+
}
906+
813907
private function addTranslatorSection(ArrayNodeDefinition $rootNode, callable $enableIfStandalone): void
814908
{
815909
$rootNode

‎DependencyInjection/FrameworkExtension.php‎

Lines changed: 78 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,8 @@
3131
use Symfony\Bundle\FullStack;
3232
use Symfony\Bundle\MercureBundle\MercureBundle;
3333
use Symfony\Component\Asset\PackageInterface;
34+
use Symfony\Component\AssetMapper\AssetMapper;
35+
use Symfony\Component\AssetMapper\ImportMap\ImportMapManager;
3436
use Symfony\Component\BrowserKit\AbstractBrowser;
3537
use Symfony\Component\Cache\Adapter\AdapterInterface;
3638
use Symfony\Component\Cache\Adapter\ArrayAdapter;
@@ -45,6 +47,7 @@
4547
use Symfony\Component\Config\FileLocator;
4648
use Symfony\Component\Config\Loader\LoaderInterface;
4749
use Symfony\Component\Config\Resource\DirectoryResource;
50+
use Symfony\Component\Config\Resource\FileResource;
4851
use Symfony\Component\Config\ResourceCheckerInterface;
4952
use Symfony\Component\Console\Application;
5053
use Symfony\Component\Console\Command\Command;
@@ -330,6 +333,14 @@ public function load(array $configs, ContainerBuilder $container)
330333
$this->registerAssetsConfiguration($config['assets'], $container, $loader);
331334
}
332335

336+
if ($this->readConfigEnabled('asset_mapper', $container, $config['asset_mapper'])) {
337+
if (!class_exists(AssetMapper::class)) {
338+
throw new LogicException('AssetMapper support cannot be enabled as the AssetMapper component is not installed. Try running "composer require symfony/asset-mapper".');
339+
}
340+
341+
$this->registerAssetMapperConfiguration($config['asset_mapper'], $container, $loader, $this->readConfigEnabled('assets', $container, $config['assets']));
342+
}
343+
333344
if ($this->readConfigEnabled('http_client', $container, $config['http_client'])) {
334345
$this->registerHttpClientConfiguration($config['http_client'], $container, $loader);
335346
}
@@ -1231,6 +1242,57 @@ private function registerAssetsConfiguration(array $config, ContainerBuilder $co
12311242
}
12321243
}
12331244

1245+
private function registerAssetMapperConfiguration(array $config, ContainerBuilder $container, PhpFileLoader $loader, bool $assetEnabled): void
1246+
{
1247+
$loader->load('asset_mapper.php');
1248+
1249+
if (!$assetEnabled) {
1250+
$container->removeDefinition('asset_mapper.asset_package');
1251+
}
1252+
1253+
$publicDirName = $this->getPublicDirectoryName($container);
1254+
$container->getDefinition('asset_mapper')
1255+
->setArgument(3, $config['public_prefix'])
1256+
->setArgument(4, $publicDirName)
1257+
->setArgument(5, $config['extensions'])
1258+
;
1259+
1260+
$paths = $config['paths'];
1261+
foreach ($container->getParameter('kernel.bundles_metadata') as $name => $bundle) {
1262+
if ($container->fileExists($dir = $bundle['path'].'/Resources/public') || $container->fileExists($dir = $bundle['path'].'/public')) {
1263+
$paths[$dir] = sprintf('bundles/%s', preg_replace('/bundle$/', '', strtolower($name)));
1264+
}
1265+
}
1266+
$container->getDefinition('asset_mapper.repository')
1267+
->setArgument(0, $paths);
1268+
1269+
$container->getDefinition('asset_mapper.command.compile')
1270+
->setArgument(4, $publicDirName);
1271+
1272+
if (!$config['server']) {
1273+
$container->removeDefinition('asset_mapper.dev_server_subscriber');
1274+
}
1275+
1276+
$container->getDefinition('asset_mapper.compiler.css_asset_url_compiler')
1277+
->setArgument(0, $config['strict_mode']);
1278+
1279+
$container->getDefinition('asset_mapper.compiler.javascript_import_path_compiler')
1280+
->setArgument(0, $config['strict_mode']);
1281+
1282+
$container
1283+
->getDefinition('asset_mapper.importmap.manager')
1284+
->replaceArgument(1, $config['importmap_path'])
1285+
->replaceArgument(2, $config['vendor_dir'])
1286+
->replaceArgument(3, $config['provider'])
1287+
;
1288+
1289+
$container
1290+
->getDefinition('asset_mapper.importmap.renderer')
1291+
->replaceArgument(2, $config['importmap_polyfill'] ?? ImportMapManager::POLYFILL_URL)
1292+
->replaceArgument(3, $config['importmap_script_attributes'])
1293+
;
1294+
}
1295+
12341296
/**
12351297
* Returns a definition for an asset package.
12361298
*/
@@ -2994,4 +3056,20 @@ private function writeConfigEnabled(string $path, bool $value, array &$config):
29943056
$this->configsEnabled[$path] = $value;
29953057
$config['enabled'] = $value;
29963058
}
3059+
3060+
private function getPublicDirectoryName(ContainerBuilder $container): string
3061+
{
3062+
$defaultPublicDir = 'public';
3063+
3064+
$composerFilePath = $container->getParameter('kernel.project_dir').'/composer.json';
3065+
3066+
if (!file_exists($composerFilePath)) {
3067+
return $defaultPublicDir;
3068+
}
3069+
3070+
$container->addResource(new FileResource($composerFilePath));
3071+
$composerConfig = json_decode(file_get_contents($composerFilePath), true);
3072+
3073+
return $composerConfig['extra']['public-dir'] ?? $defaultPublicDir;
3074+
}
29973075
}

0 commit comments

Comments
 (0)