From 41d47ca917124c50bf6810f734301b778482975c Mon Sep 17 00:00:00 2001
From: Mike McNeil <mike@balderdash.co>
Date: Thu, 31 Mar 2016 18:13:27 -0500
Subject: [PATCH] Expand on the plan for unescaping a bit.

---
 lib/hooks/views/html-scriptify.js | 16 ++++++++++++----
 1 file changed, 12 insertions(+), 4 deletions(-)

diff --git a/lib/hooks/views/html-scriptify.js b/lib/hooks/views/html-scriptify.js
index ab9d2ef60..cc616f0d2 100644
--- a/lib/hooks/views/html-scriptify.js
+++ b/lib/hooks/views/html-scriptify.js
@@ -81,7 +81,7 @@ module.exports = function htmlScriptify(options){
   }
 
   // console.log('options.data',options.data);
-  console.log('_.keys(options.data)',_.keys(options.data));
+  // console.log('_.keys(options.data)',_.keys(options.data));
 
   // Build and return HTML to inject.
   var html = '<script type="text/javascript">';
@@ -96,9 +96,17 @@ module.exports = function htmlScriptify(options){
     // (see https://github.com/lodash/lodash/blob/4.7.0/dist/lodash.js#L14038)
     // Except it also has to be recursive.  But luckily, we don't have to worry
     // about circular references, since we know this was just serialized.
-    //
-    // TODO: implement
-    html += ' var unescape = ' + (function (u){ /* TODO */ return u; }).toString() + '; ';
+    html += ' var unescape = ' + (function clientSideUnescapeFn(escapedValue){
+      var unescapedValue = escapedValue;
+      // TODO: actually unescape it
+      //
+      // This may kind of suck because it needs to be vanilla JS.
+      // Not planning on any kind of special backwards-compatible support for older browsers,
+      // but if that IS needed, or any other customizations to the client-side escaping code
+      // are necessary, then the built-in client-side escaping can be disabled using
+      // `dontUnescapeOnClient: true`.
+      return unescapedValue;
+    }).toString() + '; ';
   }
   html += ' window.'+options.namespace+' = { ';