Merge pull request #16 from tonysm/fix/tm-escape-str-for-regex

Wrap regex contact with preg_quote
tonysm · Jun 30, 2022 · 5597237 · 5597237
2 parents 2424b7e + 3fcec8f
commit 5597237
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/Commands/PinCommand.php b/src/Commands/PinCommand.php
@@ -94,7 +94,7 @@ private function pattern(string $package): string
     {
         return sprintf(
             '#^Importmap::pin\("%s".*$#',
-            $package,
+            preg_quote($package),
         );
     }
 }
diff --git a/src/Packager.php b/src/Packager.php
@@ -60,7 +60,7 @@ public function vendoredPinFor(string $package, string $url): string
     public function packaged(string $package): bool
     {
         return (bool) preg_match(
-            sprintf('#Importmap::pin\("%s"#', str_replace('#', '\#', $package)),
+            sprintf('#Importmap::pin\("%s"#', preg_quote($package)),
             File::get(base_path($this->importmapPath)),
         );
     }
@@ -86,7 +86,7 @@ private function removePackageFromImportmap(string $package)
     {
         $contents = collect(File::lines(base_path($this->importmapPath)))
             ->reject(fn (string $line) => (
-                preg_match(sprintf('#Importmap::pin\("%s"#', $package), $line)
+                preg_match(sprintf('#Importmap::pin\("%s"#', preg_quote($package)), $line)
             ))
             ->join(PHP_EOL);