From 83a17e8c2d3419feecd11019c425ca65344cbc45 Mon Sep 17 00:00:00 2001
From: Jordan Wright <jmwright798@gmail.com>
Date: Sat, 2 Apr 2016 18:33:25 -0500
Subject: [PATCH] Added early catching of invalid template values. Fixes #193

---
 models/template.go         | 38 +++++++++++++++++++++++++++++++++++++-
 static/js/app/templates.js |  3 +++
 2 files changed, 40 insertions(+), 1 deletion(-)

diff --git a/models/template.go b/models/template.go
index aa28867b..38527340 100644
--- a/models/template.go
+++ b/models/template.go
@@ -1,7 +1,9 @@
 package models
 
 import (
+	"bytes"
 	"errors"
+	"html/template"
 	"time"
 
 	"github.com/jinzhu/gorm"
@@ -33,7 +35,41 @@ func (t *Template) Validate() error {
 	case t.Text == "" && t.HTML == "":
 		return ErrTemplateMissingParameter
 	}
-	return nil
+	var buff bytes.Buffer
+	// Test that the variables used in the template
+	// validate with no issues
+	td := struct {
+		Result
+		URL         string
+		TrackingURL string
+		Tracker     string
+		From        string
+	}{
+		Result{
+			Email:     "foo@bar.com",
+			FirstName: "Foo",
+			LastName:  "Bar",
+			Position:  "Test",
+		},
+		"http://foo.bar",
+		"http://foo.bar/track",
+		"<img src='http://foo.bar/track",
+		"John Doe <foo@bar.com>",
+	}
+	tmpl, err := template.New("html_template").Parse(t.HTML)
+	if err != nil {
+		return err
+	}
+	err = tmpl.Execute(&buff, td)
+	if err != nil {
+		return err
+	}
+	tmpl, err = template.New("text_template").Parse(t.Text)
+	if err != nil {
+		return err
+	}
+	err = tmpl.Execute(&buff, td)
+	return err
 }
 
 // GetTemplates returns the templates owned by the given user.
diff --git a/static/js/app/templates.js b/static/js/app/templates.js
index 738253e6..250873c8 100644
--- a/static/js/app/templates.js
+++ b/static/js/app/templates.js
@@ -49,6 +49,9 @@ function save(idx) {
                 load()
                 dismiss()
             })
+            .error(function(data) {
+                modalError(data.responseJSON.message)
+            })
     } else {
         // Submit the template
         api.templates.post(template)