webrouse / n-asset-macro

src/Webrouse/AssetMacro/AssetMacro.php

Indentation +183 added lines, -183 removed lines

@@ -55,175 +55,175 @@ 
 			'%node.word, %node.array, $basePath, $template->global->' . self::CONFIG_PROVIDER . '))');
 	}
 
-    /**
-     * @param string $asset     Asset relative path
-     * @param array $args       Other macro arguments
-     * @param string $basePath  Base path
-     * @param array $config     Macro configuration
-     * @return string
-     */
+	/**
+	 * @param string $asset     Asset relative path
+	 * @param array $args       Other macro arguments
+	 * @param string $basePath  Base path
+	 * @param array $config     Macro configuration
+	 * @return string
+	 */
 	public static function resolveAssetPath($asset, array $args, $basePath, array $config) {
-        list($path, $format, $need) = self::processArguments($asset, $args);
-	    $wwwDir = self::normalizePath($config['wwwDir']);
-        $manifest = self::resolveManifest($path, $need, $wwwDir, $config);
-        $revision = $manifest === NULL ? NULL : self::resolveRevision($manifest, $path, $need, $config);
-
-        // Is revision only version (query parameter) or full path to asset?
-        $isVersion = $revision === NULL || strspn($revision,"./") === 0;
-
-        // Check if asset exists
-        if ($config['existCheck'] === TRUE) {
-            $ds = DIRECTORY_SEPARATOR;
-            $filePath = $isVersion ? ($wwwDir . $ds . $path) : ($wwwDir . $ds . self::normalizePath($revision));
-            if (! file_exists($filePath)) {
-                $msg = sprintf("Asset '%s' not found.", $filePath);
-                self::throwError(new AssetNotFoundException($msg), $config['missingAsset'], $need);
-                return '';
-            }
-        }
-
-        // Format output
-        return self::formatOutput($format, $basePath, $path, $revision, $isVersion);
-    }
-
-    /**
-     * @param string $format         Output format
-     * @param string $basePath       Base path
-     * @param string $path           Asset relative path
-     * @param string|null $revision  Asset revision (version or path to file)
-     * @param bool   $isVersion      Is revision only version or full path?
-     * @return string
-     */
-    private static function formatOutput($format, $basePath, $path, $revision, $isVersion)
-    {
-        $revision = $revision ?: 'unknown';
-        $path = $isVersion ?
-            sprintf("%s?v=%s", $path, $revision) :
-            $revision;
-
-        return Strings::replace($format,
-            '/%([^%]+)%/',
-            function ($matches) use ($basePath, $format, $path, $revision) {
-                switch ($matches[1]) {
-                    case 'raw':
-                        return $revision;
-                    case 'basePath':
-                        return $basePath;
-                    case 'path':
-                        return $path;
-                    case 'url':
-                        return sprintf("%s/%s", $basePath, $path);
-                    default:
-                        $msg = sprintf(
-                            "Asset macro: Invalid variable '%s' in format '%s'. " .
-                            "Use one of allowed variables: %%raw%%, %%basePath%%, %%path%%, %%url%%.",
-                            $matches[1],
-                            $format
-                        );
-                        throw new InvalidVariableException($msg);
-                }
-            });
-    }
-
-    /**
-     * @param string $asset  Asset path specified in macro
-     * @param array $args    Macro arguments
-     * @return array
-     */
-    private static function processArguments($asset, array $args)
-    {
-        $format = isset($args['format']) ? $args['format'] : (isset($args[0]) ? $args[0] : '%url%');
-        $need = isset($args['need']) ? $args['need'] : (isset($args[1]) ? $args[1] : TRUE);
-
-        Validators::assert($asset, 'string', 'path');
-        Validators::assert($format, 'string', 'format');
-        Validators::assert($need, 'bool', 'need');
-
-        $path = self::normalizePath($asset);
-
-        return [$path, $format, $need];
-    }
-
-    /**
-     * @param string $asset   Asset path specified in macro
-     * @param bool $need      Fail if manifest doesn't exist?
-     * @param string $wwwDir  Public www dir
-     * @param array $config   Macro configuration
-     * @return null|array
-     */
-    private static function resolveManifest($asset, $need, $wwwDir, array $config) {
-        $manifest = $config['manifest'];
-
-	    // Asset revisions specified directly in configuration
-        if (is_array($manifest)) {
-            return $manifest;
-        }
-
-        // Path to JSON manifest
-        if (is_string($manifest)) {
-            if ( ! file_exists($manifest)) {
-                $msg = sprintf("Manifest file not found: '%s'.", $manifest);
-                self::throwError(new ManifestNotFoundException($msg), $config['missingManifest'], $need);
-                return NULL;
-            }
-            return Json::decode(file_get_contents($manifest), Json::FORCE_ARRAY);
-        }
-
-        // Autodetect manifest path
-        return self::autodetectManifest($asset, $wwwDir, $need, $config);
-    }
-
-    /**
-     * @param string $asset   Asset path specified in macro
-     * @param string $wwwDir  Public www dir
-     * @param bool $need      Fail if asset/manifest doesn't exist?
-     * @param array $config   Macro configuration
-     * @return null|array
-     */
-    private static function autodetectManifest($asset, $wwwDir, $need, array $config) {
-        // Get asset dir, there begins search manifest
-        $dir = realpath($wwwDir . DIRECTORY_SEPARATOR . dirname($asset));
-        if ($dir === FALSE) {
-            $msg = sprintf("Parent dir of asset '%s' not found.", $asset);
-            self::throwError(new DirNotFoundException($msg), $config['missingAsset'], $need);
-            return NULL;
-        }
-
-        // Autodetect manifest
-        $autodetectPaths = $config['autodetect'];
-        while (Strings::startsWith($dir, $wwwDir)) {
-            foreach ($autodetectPaths as $path) {
-                $path = $dir . DIRECTORY_SEPARATOR . $path;
-                if (file_exists($path)) {
-                    return Json::decode(file_get_contents($path), Json::FORCE_ARRAY);
-                }
-            }
-
-            $dir = dirname($dir); // go up
-        }
-
-        $msg = sprintf("Manifest not found in: %s.", implode(', ', $autodetectPaths));
-        self::throwError(new ManifestNotFoundException($msg), $config['missingManifest'], $need);
-        return NULL;
-    }
-
-    /**
-     * @param null|array $manifest  Array of revisions
-     * @param string $path          Asset path
-     * @param bool $need            Fail if revision doesn't exist?
-     * @param array $config         Macro configuration
-     * @return null|string
-     */
-    private static function resolveRevision($manifest, $path, $need, array $config) {
-        $revision = isset($manifest[$path]) ? $manifest[$path] : NULL;
+		list($path, $format, $need) = self::processArguments($asset, $args);
+		$wwwDir = self::normalizePath($config['wwwDir']);
+		$manifest = self::resolveManifest($path, $need, $wwwDir, $config);
+		$revision = $manifest === NULL ? NULL : self::resolveRevision($manifest, $path, $need, $config);
+
+		// Is revision only version (query parameter) or full path to asset?
+		$isVersion = $revision === NULL || strspn($revision,"./") === 0;
+
+		// Check if asset exists
+		if ($config['existCheck'] === TRUE) {
+			$ds = DIRECTORY_SEPARATOR;
+			$filePath = $isVersion ? ($wwwDir . $ds . $path) : ($wwwDir . $ds . self::normalizePath($revision));
+			if (! file_exists($filePath)) {
+				$msg = sprintf("Asset '%s' not found.", $filePath);
+				self::throwError(new AssetNotFoundException($msg), $config['missingAsset'], $need);
+				return '';
+			}
+		}
+
+		// Format output
+		return self::formatOutput($format, $basePath, $path, $revision, $isVersion);
+	}
+
+	/**
+	 * @param string $format         Output format
+	 * @param string $basePath       Base path
+	 * @param string $path           Asset relative path
+	 * @param string|null $revision  Asset revision (version or path to file)
+	 * @param bool   $isVersion      Is revision only version or full path?
+	 * @return string
+	 */
+	private static function formatOutput($format, $basePath, $path, $revision, $isVersion)
+	{
+		$revision = $revision ?: 'unknown';
+		$path = $isVersion ?
+			sprintf("%s?v=%s", $path, $revision) :
+			$revision;
+
+		return Strings::replace($format,
+			'/%([^%]+)%/',
+			function ($matches) use ($basePath, $format, $path, $revision) {
+				switch ($matches[1]) {
+					case 'raw':
+						return $revision;
+					case 'basePath':
+						return $basePath;
+					case 'path':
+						return $path;
+					case 'url':
+						return sprintf("%s/%s", $basePath, $path);
+					default:
+						$msg = sprintf(
+							"Asset macro: Invalid variable '%s' in format '%s'. " .
+							"Use one of allowed variables: %%raw%%, %%basePath%%, %%path%%, %%url%%.",
+							$matches[1],
+							$format
+						);
+						throw new InvalidVariableException($msg);
+				}
+			});
+	}
+
+	/**
+	 * @param string $asset  Asset path specified in macro
+	 * @param array $args    Macro arguments
+	 * @return array
+	 */
+	private static function processArguments($asset, array $args)
+	{
+		$format = isset($args['format']) ? $args['format'] : (isset($args[0]) ? $args[0] : '%url%');
+		$need = isset($args['need']) ? $args['need'] : (isset($args[1]) ? $args[1] : TRUE);
+
+		Validators::assert($asset, 'string', 'path');
+		Validators::assert($format, 'string', 'format');
+		Validators::assert($need, 'bool', 'need');
+
+		$path = self::normalizePath($asset);
+
+		return [$path, $format, $need];
+	}
+
+	/**
+	 * @param string $asset   Asset path specified in macro
+	 * @param bool $need      Fail if manifest doesn't exist?
+	 * @param string $wwwDir  Public www dir
+	 * @param array $config   Macro configuration
+	 * @return null|array
+	 */
+	private static function resolveManifest($asset, $need, $wwwDir, array $config) {
+		$manifest = $config['manifest'];
+
+		// Asset revisions specified directly in configuration
+		if (is_array($manifest)) {
+			return $manifest;
+		}
+
+		// Path to JSON manifest
+		if (is_string($manifest)) {
+			if ( ! file_exists($manifest)) {
+				$msg = sprintf("Manifest file not found: '%s'.", $manifest);
+				self::throwError(new ManifestNotFoundException($msg), $config['missingManifest'], $need);
+				return NULL;
+			}
+			return Json::decode(file_get_contents($manifest), Json::FORCE_ARRAY);
+		}
+
+		// Autodetect manifest path
+		return self::autodetectManifest($asset, $wwwDir, $need, $config);
+	}
+
+	/**
+	 * @param string $asset   Asset path specified in macro
+	 * @param string $wwwDir  Public www dir
+	 * @param bool $need      Fail if asset/manifest doesn't exist?
+	 * @param array $config   Macro configuration
+	 * @return null|array
+	 */
+	private static function autodetectManifest($asset, $wwwDir, $need, array $config) {
+		// Get asset dir, there begins search manifest
+		$dir = realpath($wwwDir . DIRECTORY_SEPARATOR . dirname($asset));
+		if ($dir === FALSE) {
+			$msg = sprintf("Parent dir of asset '%s' not found.", $asset);
+			self::throwError(new DirNotFoundException($msg), $config['missingAsset'], $need);
+			return NULL;
+		}
+
+		// Autodetect manifest
+		$autodetectPaths = $config['autodetect'];
+		while (Strings::startsWith($dir, $wwwDir)) {
+			foreach ($autodetectPaths as $path) {
+				$path = $dir . DIRECTORY_SEPARATOR . $path;
+				if (file_exists($path)) {
+					return Json::decode(file_get_contents($path), Json::FORCE_ARRAY);
+				}
+			}
+
+			$dir = dirname($dir); // go up
+		}
+
+		$msg = sprintf("Manifest not found in: %s.", implode(', ', $autodetectPaths));
+		self::throwError(new ManifestNotFoundException($msg), $config['missingManifest'], $need);
+		return NULL;
+	}
+
+	/**
+	 * @param null|array $manifest  Array of revisions
+	 * @param string $path          Asset path
+	 * @param bool $need            Fail if revision doesn't exist?
+	 * @param array $config         Macro configuration
+	 * @return null|string
+	 */
+	private static function resolveRevision($manifest, $path, $need, array $config) {
+		$revision = isset($manifest[$path]) ? $manifest[$path] : NULL;
         
-        if ($revision === NULL) {
-            $msg = sprintf("Revision for asset '%s' not found in manifest.", $path);
-            self::throwError(new AssetVersionNotFound($msg), $config['missingRevision'], $need);
-        }
+		if ($revision === NULL) {
+			$msg = sprintf("Revision for asset '%s' not found in manifest.", $path);
+			self::throwError(new AssetVersionNotFound($msg), $config['missingRevision'], $need);
+		}
         
-        return $revision;
-    }
+		return $revision;
+	}
 
 
 	/**
@@ -245,27 +245,27 @@ 
 	}
 
 	private static function normalizePath($path, $separator = '\\/')
-    {
-        // Remove any kind of unicode whitespace
-        $normalized = preg_replace('#\p{C}+|^\./#u', '', $path);
+	{
+		// Remove any kind of unicode whitespace
+		$normalized = preg_replace('#\p{C}+|^\./#u', '', $path);
 
-        // Path remove self referring paths ("/./").
-        $normalized = preg_replace('#/\.(?=/)|^\./|\./$#', '', $normalized);
+		// Path remove self referring paths ("/./").
+		$normalized = preg_replace('#/\.(?=/)|^\./|\./$#', '', $normalized);
 
-        // Regex for resolving relative paths
-        $regex = '#\/*[^/\.]+/\.\.#Uu';
+		// Regex for resolving relative paths
+		$regex = '#\/*[^/\.]+/\.\.#Uu';
 
-        while (preg_match($regex, $normalized)) {
-            $normalized = preg_replace($regex, '', $normalized);
-        }
+		while (preg_match($regex, $normalized)) {
+			$normalized = preg_replace($regex, '', $normalized);
+		}
 
-        if (preg_match('#/\.{2}|\.{2}/#', $normalized)) {
-            throw new InvalidPathException(
-                sprintf("Path is outside of the defined root, path: '%s', resolved: '%s'.", $path, $normalized)
-            );
-        }
+		if (preg_match('#/\.{2}|\.{2}/#', $normalized)) {
+			throw new InvalidPathException(
+				sprintf("Path is outside of the defined root, path: '%s', resolved: '%s'.", $path, $normalized)
+			);
+		}
 
-        return rtrim($normalized, $separator);
-    }
+		return rtrim($normalized, $separator);
+	}
 
 }

Spacing +5 added lines, -6 removed lines

@@ -69,13 +69,13 @@ 
         $revision = $manifest === NULL ? NULL : self::resolveRevision($manifest, $path, $need, $config);
 
         // Is revision only version (query parameter) or full path to asset?
-        $isVersion = $revision === NULL || strspn($revision,"./") === 0;
+        $isVersion = $revision === NULL || strspn($revision, "./") === 0;
 
         // Check if asset exists
         if ($config['existCheck'] === TRUE) {
             $ds = DIRECTORY_SEPARATOR;
             $filePath = $isVersion ? ($wwwDir . $ds . $path) : ($wwwDir . $ds . self::normalizePath($revision));
-            if (! file_exists($filePath)) {
+            if (!file_exists($filePath)) {
                 $msg = sprintf("Asset '%s' not found.", $filePath);
                 self::throwError(new AssetNotFoundException($msg), $config['missingAsset'], $need);
                 return '';
@@ -98,12 +98,11 @@ 
     {
         $revision = $revision ?: 'unknown';
         $path = $isVersion ?
-            sprintf("%s?v=%s", $path, $revision) :
-            $revision;
+            sprintf("%s?v=%s", $path, $revision) : $revision;
 
         return Strings::replace($format,
             '/%([^%]+)%/',
-            function ($matches) use ($basePath, $format, $path, $revision) {
+            function($matches) use ($basePath, $format, $path, $revision) {
                 switch ($matches[1]) {
                     case 'raw':
                         return $revision;
@@ -161,7 +160,7 @@ 
 
         // Path to JSON manifest
         if (is_string($manifest)) {
-            if ( ! file_exists($manifest)) {
+            if (!file_exists($manifest)) {
                 $msg = sprintf("Manifest file not found: '%s'.", $manifest);
                 self::throwError(new ManifestNotFoundException($msg), $config['missingManifest'], $need);
                 return NULL;

src/Webrouse/AssetMacro/DI/Extension.php

Indentation +14 added lines, -14 removed lines

@@ -16,21 +16,21 @@ 
 	 * @var array
 	 */
 	public $defaults = [
-	    // Public www dir
+		// Public www dir
 		'wwwDir' => '%wwwDir%',
-        // Assets revision manifest
+		// Assets revision manifest
 		'manifest' => NULL,
-        // Paths for manifest autodetection
+		// Paths for manifest autodetection
 		'autodetect' => [
-            'assets.json',
-		    'busters.json',
+			'assets.json',
+			'busters.json',
 			'versions.json',
-            'manifest.json',
+			'manifest.json',
 			'rev-manifest.json',
 		],
-        // Check the existence of asset file
-        'existCheck' => TRUE,
-        // Error handling
+		// Check the existence of asset file
+		'existCheck' => TRUE,
+		// Error handling
 		'missingAsset' => 'notice',    // exception, notice, or ignore
 		'missingManifest' => 'notice', // exception, notice, or ignore
 		'missingRevision' => 'ignore',  // exception, notice, or ignore
@@ -42,16 +42,16 @@ 
 		$config = $this->getContainerBuilder()->expand($this->validateConfig($this->defaults));
 
 		// Validate configuration
-        Validators::assertField($config, 'wwwDir', 'string');
-        Validators::assertField($config, 'manifest', 'null|string|array');
-        Validators::assertField($config, 'autodetect', 'array');
-        Validators::assertField($config, 'existCheck', 'bool');
+		Validators::assertField($config, 'wwwDir', 'string');
+		Validators::assertField($config, 'manifest', 'null|string|array');
+		Validators::assertField($config, 'autodetect', 'array');
+		Validators::assertField($config, 'existCheck', 'bool');
 		$choices = ['exception', 'notice', 'ignore'];
 		$this->validateChoices('missingAsset', $choices);
 		$this->validateChoices('missingManifest', $choices);
 		$this->validateChoices('missingRevision', $choices);
 
-        // Setup macro
+		// Setup macro
 		$builder = $this->getContainerBuilder();
 		$builder->getDefinition('latte.latteFactory')
 			->addSetup("?->addProvider(?, ?)", ['@self', AssetMacro::CONFIG_PROVIDER, $config])

Spacing +3 added lines, -3 removed lines

@@ -31,9 +31,9 @@ 
         // Check the existence of asset file
         'existCheck' => TRUE,
         // Error handling
-		'missingAsset' => 'notice',    // exception, notice, or ignore
+		'missingAsset' => 'notice', // exception, notice, or ignore
 		'missingManifest' => 'notice', // exception, notice, or ignore
-		'missingRevision' => 'ignore',  // exception, notice, or ignore
+		'missingRevision' => 'ignore', // exception, notice, or ignore
 	];
 
 
@@ -68,7 +68,7 @@ 
 	 */
 	private function validateChoices($key, array $choices)
 	{
-		if ( ! in_array($this->config[$key], $choices)) {
+		if (!in_array($this->config[$key], $choices)) {
 			throw new UnexpectedValueException(sprintf(
 				"Unexpected value '%s' of '%s' configuration key. Allowed values: %s.",
 				$this->config[$key],