From 2a575d415f56e64f71dd6c68d0996b465810d7bb Mon Sep 17 00:00:00 2001
From: Sebastian Wilzbach <seb@wilzba.ch>
Date: Tue, 5 Dec 2017 00:55:25 +0100
Subject: [PATCH] Fix #40 - Add a convenience case for string arrays in HTML
 attributes

---
 source/diet/html.d | 21 +++++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/source/diet/html.d b/source/diet/html.d
index 3ed3514..721f388 100644
--- a/source/diet/html.d
+++ b/source/diet/html.d
@@ -220,6 +220,14 @@ unittest {
 	);
 	test!".foo"("<div class=\"foo\"></div>");
 	test!"#foo"("<div id=\"foo\"></div>");
+
+	// Issue 40: convenience case for string arrays in HTML attributes
+	test!`- auto a = ["public"];
+foo(class="#{a}")`
+	(`<foo class="public"></foo>`);
+	test!`- auto a = ["a", "b"];
+foo(class="#{a}")`
+	(`<foo class="a b"></foo>`);
 }
 
 
@@ -335,6 +343,7 @@ private string getElementMixin(ref CTX ctx, in Node node, bool in_pre)
 			}
 
 			ret ~= ctx.statement(node.loc, q{
+				// special case: bool
 				static if (is(typeof(() { return %s; }()) == bool) )
 			}~'{', expr);
 				if (ctx.isHTML5)
@@ -342,6 +351,7 @@ private string getElementMixin(ref CTX ctx, in Node node, bool in_pre)
 				else
 					ret ~= ctx.statement(node.loc, q{if (%s) %s.put(" %s=\"%s\"");}, expr, ctx.rangeName, att.name, att.name);
 
+				// special case: string
 				ret ~= ctx.statement(node.loc, "} else "~q{static if (is(typeof(%s) : const(char)[])) }~"{{", expr);
 				ret ~= ctx.statement(node.loc, q{  auto _diet_val = %s;}, expr);
 				ret ~= ctx.statement(node.loc, q{  if (_diet_val !is null) }~'{');
@@ -349,6 +359,17 @@ private string getElementMixin(ref CTX ctx, in Node node, bool in_pre)
 					ret ~= ctx.statement(node.loc, q{    %s.filterHTMLAttribEscape(_diet_val);}, ctx.rangeName);
 					ret ~= ctx.rawText(node.loc, "\"");
 				ret ~= ctx.statement(node.loc, "  }");
+
+			// special case: string[]
+			ret ~= ctx.statement(node.loc, "}} else static if (is(typeof(%s) : string[])) {{", expr);
+			ret ~= ctx.statement(node.loc, "auto val = %s;", expr);
+			ret ~= ctx.statement(node.loc, "import std.array : join;");
+			ret ~= ctx.statement(node.loc, "if (val !is null) { ");
+			ret ~= ctx.rawText(node.loc, " " ~ att.name~`="`);
+			ret ~= ctx.statement(node.loc, `%s.filterHTMLAttribEscape(val.join(" "));`, ctx.rangeName);
+			ret ~= ctx.rawText(node.loc, `"`);
+			ret ~= ctx.statement(node.loc, "  }");
+
 			ret ~= ctx.statement(node.loc, "}} else {");
 		}