Added latte functions asset and asset_version instead of filters

README.md

@@ -45,16 +45,14 @@ Configuration options are described in official [Symfony documentation](https://
 <p>Version: {asset_version 'my/awesome/image.png', 'foo'}</p>
 ```
 
-You can also use Filter/Helper equivalent. For example if you want to store result in variable:
+You can also use a function equivalent. For example if you want to store result in variable:
 
 ```latte
-{var $filename = 'my/awesome/image.png'}
+{var $asset = asset('my/awesome/image.png')}
+{var $asset = asset('my/awesome/image.png', 'foo')}
 
-{$filename|asset}
-{$filename|asset: foo}
-
-{$filename|asset_version}
-{$filename|asset_version: foo}
+{var $version = asset_version('my/awesome/image.png')}
+{var $version = asset_version('my/awesome/image.png', 'foo')}
 ```
 
 ## Contributing

src/DI/AssetExtension.php

@@ -113,13 +113,13 @@ public function beforeCompile(): void
 		$resultDefinition = $latteFactory->getResultDefinition();
 
 		if (version_compare(Engine::VERSION, '3', '<')) {
-			# asset filters
-			$resultDefinition->addSetup('addFilter', [
+			# asset functions
+			$resultDefinition->addSetup('addFunction', [
 				'name' => 'asset',
 				'callback' => [$this->prefix('@packages'), 'getUrl'],
 			]);
 
-			$resultDefinition->addSetup('addFilter', [
+			$resultDefinition->addSetup('addFunction', [
 				'name' => 'asset_version',
 				'callback' => [$this->prefix('@packages'), 'getVersion'],
 			]);

src/Latte/AssetExtension.php

@@ -34,7 +34,7 @@ public function getTags(): array
 	/**
 	 * @return array{asset: callable, asset_version: callable}
 	 */
-	public function getFilters(): array
+	public function getFunctions(): array
 	{
 		return [
 			'asset' => [$this->packages, 'getUrl'],

tests/Latte/AssetMacrosAndFiltersTest.phpt → tests/Latte/AssetMacrosAndFunctionsTest.phpt

@@ -14,7 +14,7 @@ use function assert;
 
 require __DIR__ . '/../bootstrap.php';
 
-final class AssetMacrosAndFiltersTest extends TestCase
+final class AssetMacrosAndFunctionsTest extends TestCase
 {
 	private Engine $engine;
 
@@ -46,39 +46,19 @@ final class AssetMacrosAndFiltersTest extends TestCase
 		]);
 	}
 
-	public function testAssetFilter(): void
+	public function testAssetFunction(): void
 	{
-		$defaultPackage = <<< LATTE
-{var \$file = "my/file.json"}
-{\$file|asset}
-LATTE;
-
-		$nestedPackage = <<< LATTE
-{var \$file = "my/nested/file.json"}
-{\$file|asset: json_manifest_strategy}
-LATTE;
-
 		$this->assertLatte([
-			[$defaultPackage, 'https://cdn.example.com/my/file.json?version=2.1'],
-			[$nestedPackage, '/my/nested/file.abc123.json'],
+			['{asset("my/file.json")}', 'https://cdn.example.com/my/file.json?version=2.1'],
+			['{asset("my/nested/file.json", "json_manifest_strategy")}', '/my/nested/file.abc123.json'],
 		]);
 	}
 
-	public function testAssetVersionFilter(): void
+	public function testAssetVersionFunction(): void
 	{
-		$defaultPackage = <<< LATTE
-{var \$file = "my/file.json"}
-{\$file|asset_version}
-LATTE;
-
-		$nestedPackage = <<< LATTE
-{var \$file = "my/nested/file.json"}
-{\$file|asset_version: json_manifest_strategy}
-LATTE;
-
 		$this->assertLatte([
-			[$defaultPackage, '2.1'],
-			[$nestedPackage, '/my/nested/file.abc123.json'],
+			['{asset_version("my/file.json")}', '2.1'],
+			['{asset_version("my/nested/file.json", "json_manifest_strategy")}', '/my/nested/file.abc123.json'],
 		]);
 	}
 
@@ -92,4 +72,4 @@ LATTE;
 	}
 }
 
-(new AssetMacrosAndFiltersTest())->run();
+(new AssetMacrosAndFunctionsTest())->run();