From 0204aadddbe8033c1b76eb875df62fea9f110860 Mon Sep 17 00:00:00 2001
From: Dominik Picheta <dominikpicheta@gmail.com>
Date: Sat, 30 Jan 2016 10:14:17 +0000
Subject: [PATCH] Fixes @dom96/jester#53.

https://github.com/dom96/jester/issues/53
---
 forum.nim | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/forum.nim b/forum.nim
index b2d65bd..a51419d 100644
--- a/forum.nim
+++ b/forum.nim
@@ -71,6 +71,8 @@ type
     email: string
     ban: string
 
+  ForumError = object of Exception
+
 var
   db: TDbConn
   docConfig: StringTableRef
@@ -110,8 +112,10 @@ proc TextWidget(c: TForumData, name, defaultText: string,
     name, $maxlength, x, if size != -1: "size=\"" & $size & "\"" else: ""]
 
 proc HiddenField(c: TForumData, name, defaultText: string): string =
-  let x = if defaultText != reuseText: defaultText
-          else: xmlEncode(c.req.params.getOrDefault(name))
+  let x = xmlencode(
+            if defaultText != reuseText: defaultText
+            else: c.req.params.getOrDefault(name)
+          )
   return """<input type="hidden" name="$1" value="$2"/>""" % [name, x]
 
 proc TextAreaWidget(c: TForumData, name, defaultText: string): string =
@@ -455,11 +459,15 @@ proc crud(c: TCrud, table: string, data: varargs[string]): TSqlQuery =
     result = sql("delete from " & table & " where id = ?")
 
 template retrSubject(c: expr) =
+  if not c.req.params.hasKey("subject"):
+    raise newException(ForumError, "Subject empty")
   let subject {.inject.} = c.req.params["subject"]
   if subject.strip.len < 3:
     return setError(c, "subject", "Subject not long enough")
 
 template retrContent(c: expr) =
+  if not c.req.params.hasKey("content"):
+    raise newException(ForumError, "Content empty")
   let content {.inject.} = c.req.params["content"]
   if content.strip.len < 2:
     return setError(c, "content", "Content not long enough")