[HtmlSanitizer] Some minor changes in the config API

javiereguiluz · fabpot · commit ae23fc8d06bd · 2022-01-07T19:31:55.000+01:00
diff --git a/HtmlSanitizerConfig.php b/HtmlSanitizerConfig.php
@@ -105,7 +105,7 @@ public function __construct()
      * All scripts will be removed but the output may still contain other dangerous
      * behaviors like CSS injection (click-jacking), CSS expressions, ...
      */
-    public function allowAllStaticElements(): static
+    public function allowStaticElements(): static
     {
         $elements = array_merge(
             array_keys(W3CReference::HEAD_ELEMENTS),
diff --git a/README.md b/README.md
@@ -22,7 +22,7 @@ $config = (new HtmlSanitizerConfig())
     // standard. All scripts will be removed but the output may still contain
     // other dangerous behaviors like CSS injection (click-jacking), CSS
     // expressions, ...
-    ->allowAllStaticElements()
+    ->allowStaticElements()
 
     // Allow the "div" element and no attribute can be on it
     ->allowElement('div')
diff --git a/Tests/HtmlSanitizerAllTest.php b/Tests/HtmlSanitizerAllTest.php
@@ -21,7 +21,7 @@ private function createSanitizer(): HtmlSanitizer
     {
         return new HtmlSanitizer(
             (new HtmlSanitizerConfig())
-                ->allowAllStaticElements()
+                ->allowStaticElements()
                 ->allowLinkHosts(['trusted.com', 'external.com'])
                 ->allowMediaHosts(['trusted.com', 'external.com'])
                 ->allowRelativeLinks()
diff --git a/TextSanitizer/StringSanitizer.php b/TextSanitizer/StringSanitizer.php
@@ -26,7 +26,7 @@ final class StringSanitizer
             // "&#34;" is shorter than "&quot;"
             '&quot;',
 
-            // Fix several potential issues in how browsers intepret attributes values
+            // Fix several potential issues in how browsers interpret attributes values
             '+',
             '=',
             '@',
diff --git a/Visitor/DomVisitor.php b/Visitor/DomVisitor.php
@@ -47,7 +47,7 @@ final class DomVisitor
     private array $elementsConfig;
 
     /**
-     * Registry of attributes to forcefuly set on nodes, index by element and attribute.
+     * Registry of attributes to forcefully set on nodes, index by element and attribute.
      *
      * @var array<string, array<string, string>>
      */

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ public function __construct()`
`105`	`105`	`* All scripts will be removed but the output may still contain other dangerous`
`106`	`106`	`* behaviors like CSS injection (click-jacking), CSS expressions, ...`
`107`	`107`	`*/`
`108`		`- public function allowAllStaticElements(): static`
	`108`	`+ public function allowStaticElements(): static`
`109`	`109`	`{`
`110`	`110`	`$elements = array_merge(`
`111`	`111`	`array_keys(W3CReference::HEAD_ELEMENTS),`
Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ private function createSanitizer(): HtmlSanitizer`
`21`	`21`	`{`
`22`	`22`	`return new HtmlSanitizer(`
`23`	`23`	`(new HtmlSanitizerConfig())`
`24`		`- ->allowAllStaticElements()`
	`24`	`+ ->allowStaticElements()`
`25`	`25`	`->allowLinkHosts(['trusted.com', 'external.com'])`
`26`	`26`	`->allowMediaHosts(['trusted.com', 'external.com'])`
`27`	`27`	`->allowRelativeLinks()`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ final class DomVisitor`
`47`	`47`	`private array $elementsConfig;`
`48`	`48`
`49`	`49`	`/**`
`50`		`- * Registry of attributes to forcefuly set on nodes, index by element and attribute.`
	`50`	`+ * Registry of attributes to forcefully set on nodes, index by element and attribute.`
`51`	`51`	`*`
`52`	`52`	`* @var array<string, array<string, string>>`
`53`	`53`	`*/`