Added validation to registration form, updated library, added query string to embed

Author: Matt Karl

app/public/js/embed.js

@@ -11,6 +11,21 @@ router.get('/', function(req, res)
 
 router.post('/', function(req, res, next)
 {
+    // Do some basic validation
+    req.checkBody('name', 'Name is required').notEmpty();
+    req.checkBody('username', 'Username must only be alpha characters').isAlpha();
+    req.checkBody('email', 'Email must be a valid email address').isEmail();
+
+    var errors = req.validationErrors();
+
+    if (errors)
+    {
+        return res.render('register',
+        {
+            errors: errors
+        });
+    }
+
 	// Make sure that someone can override
 	// the user default privilege, would be 
 	// ugly to expose edit of this to anonymous

app/public/js/libraries.js

@@ -1,6 +1,6 @@
 {
 	"name": "SpringRollConnect",
-	"version": "1.3.2",
+	"version": "1.3.3",
 	"dependencies": {
 		"jquery": "*",
 		"bootstrap": "*",

app/routes/register.js

@@ -1,5 +1,5 @@
 {
-	"version": "1.3.2",
+	"version": "1.3.3",
 	"private": true,
 	"devDependencies": {
 		"grunt": "^0.4.5",

bower.json

@@ -1,6 +1,6 @@
 {
 	"name": "SpringRollConnect",
-	"version": "1.3.2",
+	"version": "1.3.3",
 	"main": [
 		"src/plugins/jquery-search.js",
 		"src/widgets/*.js",

package.json

@@ -105,7 +105,10 @@
 
 		// Open via api
 		var api = location.pathname.replace('/embed/', '/api/release/');
-		
+		var query;
+		var playOptions = null;
+		var singlePlay = false;
+
 		if (location.search)
 		{
 			var params = parseQuery(location.search.substr(1));
@@ -115,25 +118,72 @@
 			['version', 'status', 'commitId', 'token'].forEach(function(param)
 			{
 				if (params[param])
+				{
 					apiArgs.push(param + "=" + params[param]);
+					delete params[param];
+				}
 			});
 
 			// Check for debug
-			if (params.debug) apiArgs.push("debug=true");
+			if (params.debug)
+			{
+				apiArgs.push("debug=true");
+				delete params.debug;
+			}
 
 			// Show the controls
-			if (params.controls) this.frame.addClass('show-controls');
+			if (params.controls)
+			{
+				this.frame.addClass('show-controls');
+				delete params.controls;
+			}
 
 			// Show the title
-			if (params.title) this.frame.addClass('show-title');
+			if (params.title)
+			{
+				this.frame.addClass('show-title');
+				delete params.title;
+			}
 
 			// Add the arguments
 			if (apiArgs.length)
 			{
 				api += "?" + apiArgs.join("&");
 			}
+
+			// Single play
+			if (params.singlePlay)
+			{
+				singlePlay = params.singlePlay == "true" || params.singlePlay == "1";
+				delete params.singlePlay;
+			}
+
+			// Play options
+			if (params.playOptions)
+			{
+				try
+				{
+					playOptions = JSON.parse(params.playOptions);
+				}
+				catch(e){} // ignore invalid JSON parse
+				delete params.playOptions;
+			}
+
+			// Get any other options and pass them to the query string
+			query = [];
+			for(var param in params)
+			{
+				query.push(param + "=" + params[param]);
+			}
+			query = query.length ? "?" + query.join("&") : '';
 		}
-		this.openRemote(api);
+
+		this.openRemote(api, 
+		{
+			singlePlay: singlePlay,
+			playOptions: playOptions,
+			query: query
+		});
 	};
 
 	function parseQuery(queryString)