Merge pull request #18 from ExoUNX/dev

Manifest builder only accepts CSS and JS files now
gaigelama · Sep 18, 2019 · d617d19 · d617d19
2 parents c4e6de9 + cd642d1
commit d617d19
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/src/Builder.php b/src/Builder.php
@@ -139,7 +139,7 @@ private static function selectAlgorithm(): string
      *
      * @return string
      */
-    private static function parseExtension(string $path): string
+    public static function parseExtension(string $path): string
     {
         return preg_replace("#\?.*#", '', pathinfo($path, PATHINFO_EXTENSION));
     }

diff --git a/src/ManifestBuilder.php b/src/ManifestBuilder.php
@@ -132,11 +132,13 @@ private function buildManifest(): array
                 $this->vasriConfig['assets']
             ) as $asset => $alt
         ) {
-            $manifest[$asset] = [
-                'sri'     => $this->builder->sri($alt),
-                'version' => $this->builder->versioning($alt),
-                'alt'     => $alt
-            ];
+            if ($this->builder->parseExtension($asset) === 'js' || $this->builder->parseExtension($asset) === 'css') {
+                $manifest[$asset] = [
+                    'sri'     => $this->builder->sri($alt),
+                    'version' => $this->builder->versioning($alt),
+                    'alt'     => $alt
+                ];
+            }
         }
 
         return $manifest;