Allow pass null argument to methods Tag::class(), Tag::replaceClass(), BooleanInputTag::label() and BooleanInputTag::sideLabel()

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 - New #74: Add classes for tags `Em`, `Strong`, `B` and `I` (vjik)
 - New #75: Add methods `as()` and `preload()` to the `Link` tag (vjik)
+- New #78: Allow pass `null` argument to methods `Tag::class()` and `Tag::replaceClass()` (vjik)
 
 ## 1.2.0 May 04, 2021
 

composer.json

@@ -20,11 +20,11 @@
         "yiisoft/json": "^1.0"
     },
     "require-dev": {
-        "maglnet/composer-require-checker": "^3.2",
+        "maglnet/composer-require-checker": "^3.3",
         "phpunit/phpunit": "^9.5",
-        "roave/infection-static-analysis-plugin": "^1.7",
+        "roave/infection-static-analysis-plugin": "^1.8",
         "spatie/phpunit-watcher": "^1.23",
-        "vimeo/psalm": "^4.7"
+        "vimeo/psalm": "^4.8"
     },
     "autoload": {
         "psr-4": {

src/Html.php

@@ -1156,7 +1156,10 @@ public static function renderTagAttributes(array $attributes): string
                     if (empty($value)) {
                         continue;
                     }
-                    /** @psalm-var array<string, string> $value */
+                    /**
+                     * @psalm-suppress UnnecessaryVarAnnotation
+                     * @psalm-var array<string, string> $value
+                     */
                     $html .= " $name=\"" . self::encodeAttribute(self::cssStyleFromArray($value)) . '"';
                 } else {
                     $html .= " $name='" . Json::htmlEncode($value) . "'";

src/Tag/Base/Tag.php

@@ -75,29 +75,32 @@ final public function id(?string $id): self
     /**
      * Add one or more CSS classes to the tag.
      *
-     * @param string ...$class One or many CSS classes.
+     * @param string|null ...$class One or many CSS classes.
      *
      * @return static
      */
-    final public function class(string ...$class): self
+    final public function class(?string ...$class): self
     {
         $new = clone $this;
         /** @psalm-suppress MixedArgumentTypeCoercion */
-        Html::addCssClass($new->attributes, $class);
+        Html::addCssClass(
+            $new->attributes,
+            array_filter($class, static fn ($c) => $c !== null),
+        );
         return $new;
     }
 
     /**
      * Replace current tag CSS classes with a new set of classes.
      *
-     * @param string ...$class One or many CSS classes.
+     * @param string|null ...$class One or many CSS classes.
      *
      * @return static
      */
-    final public function replaceClass(string ...$class): self
+    final public function replaceClass(?string ...$class): self
     {
         $new = clone $this;
-        $new->attributes['class'] = $class;
+        $new->attributes['class'] = array_filter($class, static fn ($c) => $c !== null);
         return $new;
     }
 

tests/common/Tag/Base/TagTest.php

@@ -20,6 +20,8 @@ public function dataAttributes(): array
                 '<test checked disabled required="yes">',
                 ['checked' => true, 'disabled' => true, 'hidden' => false, 'required' => 'yes'],
             ],
+            ['<test class="">', ['class' => '']],
+            ['<test class="red">', ['class' => 'red']],
             ['<test class="first second">', ['class' => ['first', 'second']]],
             ['<test>', ['class' => []]],
             ['<test style="width: 100px; height: 200px;">', ['style' => ['width' => '100px', 'height' => '200px']]],
@@ -141,10 +143,29 @@ public function testClass(string $expected, array $class): void
         $this->assertSame($expected, (string)TestTag::tag()->class('main')->class(...$class));
     }
 
+    public function dataNewClass(): array
+    {
+        return [
+            ['<test>', null],
+            ['<test class="">', ''],
+            ['<test class="red">', 'red'],
+        ];
+    }
+
+    /**
+     * @dataProvider dataNewClass
+     */
+    public function testNewClass(string $expected, ?string $class): void
+    {
+        $this->assertSame($expected, (string)TestTag::tag()->class($class));
+    }
+
     public function dataReplaceClass(): array
     {
         return [
             ['<test>', []],
+            ['<test>', [null]],
+            ['<test class="">', ['']],
             ['<test class="main">', ['main']],
             ['<test class="main bold">', ['main bold']],
             ['<test class="main bold">', ['main', 'bold']],
@@ -158,7 +179,7 @@ public function dataReplaceClass(): array
      */
     public function testReplaceClass(string $expected, array $class): void
     {
-        $this->assertSame($expected, (string)TestTag::tag()->replaceClass(...$class));
+        $this->assertSame($expected, (string)TestTag::tag()->class('red')->replaceClass(...$class));
     }
 
     public function testImmutability(): void