Updated jshintrc to allow 'supernew' (aka 'weird' constructions). Ref…

…actored res.locals in interpreter.
Mararesliu · Sep 26, 2013 · 1a37d7d · 1a37d7d
1 parent 4bb1966
commit 1a37d7d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 5 deletions.
diff --git a/.jshintrc b/.jshintrc
@@ -22,7 +22,12 @@
   "loopfunc": true,
 
   // Suppress warnings about using assignments where conditionals are expected
-  "boss": true
+  "boss": true,
+
+  // Suppress warnings about "weird constructions"
+  // i.e. allow code like:
+  //  (new (function OneTimeUsePrototype () { } ))
+  "supernew": true
 
   // "bitwise": true,
   // "camelcase": true,

diff --git a/lib/hooks/sockets/interpreter/interpret.js b/lib/hooks/sockets/interpreter/interpret.js
@@ -314,12 +314,11 @@ module.exports = function (sails) {
 		 * Scoped local variables accesible from views
 		 * see also: http://expressjs.com/api.html#res.locals
 		 */
-		function Locals (){
+		res.locals = (new function Locals (){
 			this.partial = function renderPartial () {
 				return sendError('View partials not implemented over socket.io.');
 			};
-		}
-		res.locals = new Locals();
+		}());
 
 		/**
 		 * Set the value of a local variable in the view
@@ -334,7 +333,6 @@ module.exports = function (sails) {
 		};
 
 
-
 		/**
 		 * http://expressjs.com/api.html#res.format
 		 *