[UTIL][HTML] Move to a unified array

This commit is contained in:
Hugo Sales 2021-09-20 13:13:14 +01:00
parent e8f4563633
commit 8fe8687c5b
No known key found for this signature in database
GPG Key ID: 7D0C7EAFC9D835A0
2 changed files with 15 additions and 25 deletions

View File

@ -40,37 +40,27 @@ abstract class HTML
/** /**
* Creates an HTML tag without attributes * Creates an HTML tag without attributes
*
* @param string $tag
* @param null|mixed $attrs
* @param array|string $content
* @param bool $empty_tag
*
* @return array
*/ */
public static function tag(string $tag, $attrs = null, $content = null, bool $empty_tag = false): array public static function tag(string $tag, mixed $attrs = null, mixed $content = null, array $options = []): array
{ {
return self::attr_tag($tag, $attrs ?? '', $content ?? '', $empty_tag); return self::attr_tag($tag, $attrs ?? '', $content ?? '', $options);
} }
/** /**
* Create tag, possibly with attributes and indentation * Create tag, possibly with attributes and indentation
*
* @param string $tag
* @param array|string $attrs - element attributes
* @param array|string $content - what goes inside the tag
* @param bool $empty_tag
*
* @return array
*/ */
private static function attr_tag(string $tag, $attrs, $content = '', bool $empty_tag = false): array private static function attr_tag(string $tag, mixed $attrs, mixed $content = '', array $options = []): array
{ {
$html = '<' . $tag . (is_string($attrs) ? ($attrs ? ' ' : '') . $attrs : self::attr($attrs)); $html = '<' . $tag . (is_string($attrs) ? ($attrs ? ' ' : '') . $attrs : self::attr($attrs, $options));
if ($empty_tag) { if ($options['empty'] ?? false) {
$html .= '/>'; $html .= '/>';
} else { } else {
$inner = Formatting::indent($content); if ($options['indent'] ?? true) {
$html .= ">\n" . ($inner == '' ? '' : $inner . "\n") . "</{$tag}>"; $inner = Formatting::indent($content);
$html .= ">\n" . ($inner == '' ? '' : $inner . "\n") . "</{$tag}>";
} else {
$html .= ">{$content}</{$tag}>";
}
} }
return explode("\n", $html); return explode("\n", $html);
} }
@ -104,7 +94,7 @@ abstract class HTML
/** /**
* @param array|string $html The input to convert to HTML * @param array|string $html The input to convert to HTML
* @param array $options = [] ['allowed_tags' => string[], 'forbidden_attributes' => string[], 'raw' => bool] * @param array $options = [] ['allowed_tags' => string[], 'forbidden_attributes' => string[], 'raw' => bool, 'indent' => bool]
* *
* @return string * @return string
*/ */
@ -130,7 +120,7 @@ abstract class HTML
throw new \InvalidArgumentException("HTML::html: Tag {$tag} is not allowed"); throw new \InvalidArgumentException("HTML::html: Tag {$tag} is not allowed");
} }
if (!empty($inner)) { if (!empty($inner)) {
$inner = "\n" . Formatting::indent($inner, $indent) . "\n"; $inner = ($options['indent'] ?? true) ? ("\n" . Formatting::indent($inner, $indent) . "\n") : $inner;
} }
$out .= "<{$tag}{$attrs}>{$inner}</{$tag}>"; $out .= "<{$tag}{$attrs}>{$inner}</{$tag}>";
} else { } else {

View File

@ -35,7 +35,7 @@ class HTMLTest extends WebTestCase
static::assertSame("<a href=\"test\">\n <p>\n </p>\n</a>\n", HTML::html(['a' => ['attrs' => ['href' => 'test'], 'p' => '']])); static::assertSame("<a href=\"test\">\n <p>\n </p>\n</a>\n", HTML::html(['a' => ['attrs' => ['href' => 'test'], 'p' => '']]));
static::assertSame("<a>\n <p>\n foo\n </p>\n <br/>\n</a>\n", HTML::html(['a' => ['p' => 'foo', 'br' => 'empty']])); static::assertSame("<a>\n <p>\n foo\n </p>\n <br/>\n</a>\n", HTML::html(['a' => ['p' => 'foo', 'br' => 'empty']]));
static::assertThrows(\InvalidArgumentException::class, fn () => HTML::html(1)); static::assertThrows(\InvalidArgumentException::class, fn () => HTML::html(1));
static::assertSame("<a href=\"test\">\n foo\n</a>", implode("\n", HTML::tag('a', ['href' => 'test'], content: 'foo', empty_tag: false))); static::assertSame("<a href=\"test\">\n foo\n</a>", implode("\n", HTML::tag('a', ['href' => 'test'], content: 'foo', options: ['empty' => false])));
static::assertSame('<br/>', implode("\n", HTML::tag('br', attrs: null, content: null, empty_tag: true))); static::assertSame('<br/>', implode("\n", HTML::tag('br', attrs: null, content: null, options: ['empty' => true])));
} }
} }