Add parameter $attributes to ListTag::strings()

vjik · web-flow · commit 88a2f1125828 · 2021-02-23T19:49:52.000+03:00
diff --git a/src/Tag/Base/ListTag.php b/src/Tag/Base/ListTag.php
@@ -27,15 +27,17 @@ public function items(Li ...$items): self
 
     /**
      * @param string[] $strings Array of list items as strings.
+     * @param array $attributes The tag attributes in terms of name-value pairs.
      * @param bool $encode Whether to encode strings passed.
      *
      * @return static
      */
-    public function strings(array $strings, bool $encode = true): self
+    public function strings(array $strings, array $attributes = [], bool $encode = true): self
     {
-        $items = array_map(static function (string $string) use ($encode) {
+        $items = array_map(static function (string $string) use ($attributes, $encode) {
             return Li::tag()
                 ->content($string)
+                ->attributes($attributes)
                 ->encode($encode);
         }, $strings);
         return $this->items(...$items);
diff --git a/tests/Tag/Base/ListTagTest.php b/tests/Tag/Base/ListTagTest.php
@@ -27,6 +27,16 @@ public function testStrings(): void
         $this->assertSame("<test>\n<li>A</li>\n<li>B</li>\n</test>", (string)$tag);
     }
 
+    public function testStringsAttributes(): void
+    {
+        $tag = TestListTag::tag()->strings(['A', 'B'], ['class' => 'red']);
+
+        self::assertSame(
+            "<test>\n<li class=\"red\">A</li>\n<li class=\"red\">B</li>\n</test>",
+            (string)$tag
+        );
+    }
+
     public function testStringsEncode(): void
     {
         $tag = TestListTag::tag()->strings(['<b>A</b>', '<b>B</b>']);
@@ -39,9 +49,9 @@ public function testStringsEncode(): void
 
     public function testStringsWithoutEncode(): void
     {
-        $tag = TestListTag::tag()->strings(['<b>A</b>', '<b>B</b>'], false);
+        $tag = TestListTag::tag()->strings(['<b>A</b>', '<b>B</b>'], [], false);
 
-        $this->assertSame("<test>\n<li><b>A</b></li>\n<li><b>B</b></li>\n</test>", (string)$tag);
+        self::assertSame("<test>\n<li><b>A</b></li>\n<li><b>B</b></li>\n</test>", (string)$tag);
     }
 
     public function testImmutability(): void

Original file line number	Diff line number	Diff line change
`@@ -27,6 +27,16 @@ public function testStrings(): void`
`27`	`27`	`$this->assertSame("<test>\n<li>A</li>\n<li>B</li>\n</test>", (string)$tag);`
`28`	`28`	`}`
`29`	`29`
	`30`	`+ public function testStringsAttributes(): void`
	`31`	`+ {`
	`32`	`+ $tag = TestListTag::tag()->strings(['A', 'B'], ['class' => 'red']);`
	`33`	`+`
	`34`	`+ self::assertSame(`
	`35`	`+ "<test>\n<li class=\"red\">A</li>\n<li class=\"red\">B</li>\n</test>",`
	`36`	`+ (string)$tag`
	`37`	`+ );`
	`38`	`+ }`
	`39`	`+`
`30`	`40`	`public function testStringsEncode(): void`
`31`	`41`	`{`
`32`	`42`	`$tag = TestListTag::tag()->strings(['<b>A</b>', '<b>B</b>']);`
`@@ -39,9 +49,9 @@ public function testStringsEncode(): void`
`39`	`49`
`40`	`50`	`public function testStringsWithoutEncode(): void`
`41`	`51`	`{`
`42`		`- $tag = TestListTag::tag()->strings(['<b>A</b>', '<b>B</b>'], false);`
	`52`	`+ $tag = TestListTag::tag()->strings(['<b>A</b>', '<b>B</b>'], [], false);`
`43`	`53`
`44`		`- $this->assertSame("<test>\n<li><b>A</b></li>\n<li><b>B</b></li>\n</test>", (string)$tag);`
	`54`	`+ self::assertSame("<test>\n<li><b>A</b></li>\n<li><b>B</b></li>\n</test>", (string)$tag);`
`45`	`55`	`}`
`46`	`56`
`47`	`57`	`public function testImmutability(): void`