fix(renderer): render predefined attributes as HTML entities

includes `{nbps}`, etc.
also, simplify rendering (no template needed)

Fixes #953

Signed-off-by: Xavier Coulon <[email protected]>

pkg/renderer/sgml/elements.go

@@ -48,8 +48,6 @@ func (r *sgmlRenderer) renderListElements(ctx *renderer.Context, elements []inte
 func (r *sgmlRenderer) renderElement(ctx *renderer.Context, element interface{}) (string, error) {
 	// log.Debugf("rendering element of type `%T`", element)
 	switch e := element.(type) {
-	case []interface{}:
-		return r.renderElements(ctx, e)
 	case *types.TableOfContents:
 		return r.renderTableOfContents(ctx, e)
 	case *types.Section:
@@ -115,8 +113,6 @@ func (r *sgmlRenderer) renderElement(ctx *renderer.Context, element interface{})
 	case *types.FrontMatter:
 		ctx.Attributes.AddAll(e.Attributes)
 		return "", nil
-	case *types.AttributeReference:
-		return r.renderAttributeSubstitution(ctx, e)
 	default:
 		return "", errors.Errorf("unsupported type of element: %T", element)
 	}

pkg/renderer/sgml/html5/paragraph_test.go

@@ -94,9 +94,9 @@ some content`
 		})
 
 		It("paragraph with predefined attribute", func() {
-			source := "hello {plus} world"
+			source := "hello{nbsp}{plus}{nbsp}world"
 			expected := `<div class="paragraph">
-<p>hello &#43; world</p>
+<p>hello&#160;&#43;&#160;world</p>
 </div>
 `
 			Expect(RenderHTML(source)).To(MatchHTML(expected))

pkg/renderer/sgml/html5/templates.go

@@ -58,7 +58,6 @@ var templates = sgml.Templates{
 	PassthroughBlock:          passthroughBlock,
 	Paragraph:                 paragraphTmpl,
 	Preamble:                  preambleTmpl,
-	PredefinedAttribute:       predefinedAttributeTmpl,
 	QAndAList:                 qAndAListTmpl,
 	QAndAListItem:             qAndAListItemTmpl,
 	QuoteBlock:                quoteBlockTmpl,

pkg/renderer/sgml/predefined_attribute.go

@@ -1,21 +1,9 @@
 package sgml
 
 import (
-	"strings"
-
 	"github.com/bytesparadise/libasciidoc/pkg/types"
-	"github.com/pkg/errors"
 )
 
 func (r *sgmlRenderer) renderPredefinedAttribute(a *types.PredefinedAttribute) (string, error) {
-	result := &strings.Builder{}
-	if err := r.predefinedAttribute.Execute(result, struct {
-		Name string
-	}{
-		Name: a.Name,
-	}); err != nil {
-		return "", errors.Wrap(err, "error while rendering predefined attribute")
-	}
-	// log.Debugf("rendered predefined attribute for '%s': '%s'", a.Name, result.String())
-	return result.String(), nil
+	return predefinedAttribute(a.Name), nil
 }

pkg/renderer/sgml/predefined_attribute_test.go

@@ -12,18 +12,18 @@ var _ = DescribeTable("predefined attributes",
 	Entry("sp", "sp", " "),
 	Entry("blank", "blank", ""),
 	Entry("empty", "empty", ""),
-	Entry("nbsp", "nbsp", "\u00a0"),
-	Entry("zwsp", "zwsp", "\u200b"),
-	Entry("wj", "wj", "\u2060"),
+	Entry("nbsp", "nbsp", " "),
+	Entry("zwsp", "zwsp", "​"),
+	Entry("wj", "wj", "⁠"),
 	Entry("apos", "apos", "'"),
 	Entry("quot", "quot", """),
-	Entry("lsquo", "lsquo", "\u2018"),
-	Entry("rsquo", "rsquo", "\u2019"),
-	Entry("ldquo", "ldquo", "\u201c"),
-	Entry("rdquo", "rdquo", "\u201d"),
-	Entry("deg", "deg", "\u00b0"),
-	Entry("plus", "plus", "+"),
-	Entry("brvbar", "brvbar", "\u00a6"),
+	Entry("lsquo", "lsquo", "‘"),
+	Entry("rsquo", "rsquo", "’"),
+	Entry("ldquo", "ldquo", "“"),
+	Entry("rdquo", "rdquo", "”"),
+	Entry("deg", "deg", "°"),
+	Entry("plus", "plus", "+"),
+	Entry("brvbar", "brvbar", "¦"),
 	Entry("vbar", "vbar", "|"),
 	Entry("amp", "amp", "&"),
 	Entry("lt", "lt", "<"),

pkg/renderer/sgml/renderer.go

@@ -36,14 +36,13 @@ func NewRenderer(t Templates) Renderer {
 		templates: t,
 		// Establish some default function handlers.
 		functions: funcMap{
-			"escape":              EscapeString,
-			"trimRight":           trimRight,
-			"trimLeft":            trimLeft,
-			"trim":                trimBoth,
-			"predefinedAttribute": predefinedAttribute,
-			"halign":              halign,
-			"valign":              valign,
-			"lastInStrings":       lastInStrings,
+			"escape":        EscapeString,
+			"trimRight":     trimRight,
+			"trimLeft":      trimLeft,
+			"trim":          trimBoth,
+			"halign":        halign,
+			"valign":        valign,
+			"lastInStrings": lastInStrings,
 		},
 	}
 	return r
@@ -65,20 +64,20 @@ var predefinedAttributes = map[string]string{
 	"sp":             " ",
 	"blank":          "",
 	"empty":          "",
-	"nbsp":           "\u00a0",
-	"zwsp":           "\u200b",
-	"wj":             "\u2060",
+	"nbsp":           " ",
+	"zwsp":           "​",
+	"wj":             "⁠",
 	"apos":           "'",
 	"quot":           """,
-	"lsquo":          "\u2018",
-	"rsquo":          "\u2019",
-	"ldquo":          "\u201c",
-	"rdquo":          "\u201d",
-	"deg":            "\u00b0",
+	"lsquo":          "‘",
+	"rsquo":          "’",
+	"ldquo":          "“",
+	"rdquo":          "”",
+	"deg":            "°",
 	"plus":           "+",
-	"brvbar":         "\u00a6",
+	"brvbar":         "¦",
 	"vbar":           "|", // TODO: maybe convert this because of tables?
-	"amp":            "&",
+	"amp":            "&",
 	"lt":             "<",
 	"gt":             ">",
 	"startsb":        "[",

pkg/renderer/sgml/sgml_renderer.go

@@ -58,7 +58,6 @@ type sgmlRenderer struct {
 	paragraph                 *textTemplate
 	passthroughBlock          *textTemplate
 	preamble                  *textTemplate
-	predefinedAttribute       *textTemplate
 	qAndAList                 *textTemplate
 	qAndAListItem             *textTemplate
 	quoteBlock                *textTemplate
@@ -141,7 +140,6 @@ func (r *sgmlRenderer) prepareTemplates() error {
 		r.paragraph, err = r.newTemplate("paragraph", tmpls.Paragraph, err)
 		r.passthroughBlock, err = r.newTemplate("passthrough", tmpls.PassthroughBlock, err)
 		r.preamble, err = r.newTemplate("preamble", tmpls.Preamble, err)
-		r.predefinedAttribute, err = r.newTemplate("predefined attribute", tmpls.PredefinedAttribute, err)
 		r.qAndAList, err = r.newTemplate("qanda-list", tmpls.QAndAList, err)
 		r.qAndAListItem, err = r.newTemplate("qanda-list-item", tmpls.QAndAListItem, err)
 		r.quoteBlock, err = r.newTemplate("quote-block", tmpls.QuoteBlock, err)

pkg/renderer/sgml/sgml_suite_test.go

@@ -0,0 +1,13 @@
+package sgml_test
+
+import (
+	"testing"
+
+	. "github.com/onsi/ginkgo"
+	. "github.com/onsi/gomega"
+)
+
+func TestSgml(t *testing.T) {
+	RegisterFailHandler(Fail)
+	RunSpecs(t, "Sgml Suite")
+}

pkg/renderer/sgml/templates.go

@@ -51,7 +51,6 @@ type Templates struct {
 	Paragraph                 string
 	PassthroughBlock          string
 	Preamble                  string
-	PredefinedAttribute       string
 	QAndAList                 string
 	QAndAListItem             string
 	QuoteBlock                string