closed #2583

public/src/app.js

@@ -566,8 +566,11 @@ app.uid = null;
 				}
 			});
 
-			require(['taskbar'], function(taskbar) {
+			require(['taskbar', 'helpers'], function(taskbar, helpers) {
 				taskbar.init();
+
+				// templates.js helpers
+				helpers.register();
 			});
 		});
 	};

public/src/helpers.js

@@ -1,35 +0,0 @@
-"use strict";
-/*global templates*/
-
-
-(function(module) {
-	var helpers = {};
-
-	helpers.displayUsersLink = function(config) {
-		return (config.loggedIn || !config.privateUserInfo);
-	};
-
-	helpers.buildMetaTag = function(tag) {
-		var name = tag.name ? 'name="' + tag.name + '" ' : '',
-			property = tag.property ? 'property="' + tag.property + '" ' : '',
-			content = tag.content ? 'content="' + tag.content.replace(/\n/g, ' ') + '" ' : '';
-
-		return '<meta ' + name + property + content + '/>';
-	};
-
-	if ('undefined' !== typeof window) {
-		$(document).ready(module.exports);
-	}
-
-	module.exports = function() {
-		var templates = templates || require('templates.js');
-
-		templates.registerHelper('displayUsersLink', helpers.displayUsersLink);
-		templates.registerHelper('buildMetaTag', helpers.buildMetaTag);
-	};
-
-})('undefined' === typeof module ? {
-	module: {
-		exports: {}
-	}
-} : module);

public/src/modules/helpers.js

@@ -0,0 +1,59 @@
+;(function(exports) {
+	"use strict";
+	/* globals define */
+
+	// export the class if we are in a Node-like system.
+	if (typeof module === 'object' && module.exports === exports) {
+	  exports = module.exports/* = SemVer*/;
+	}
+
+	var helpers = {};
+
+	helpers.displayUsersLink = function(config) {
+		return (config.loggedIn || !config.privateUserInfo);
+	};
+
+	helpers.buildMetaTag = function(tag) {
+		var name = tag.name ? 'name="' + tag.name + '" ' : '',
+			property = tag.property ? 'property="' + tag.property + '" ' : '',
+			content = tag.content ? 'content="' + tag.content.replace(/\n/g, ' ') + '" ' : '';
+
+		return '<meta ' + name + property + content + '/>';
+	};
+
+	// Groups helpers
+	helpers.membershipBtn = function(groupObj) {
+		if (groupObj.isMember) {
+			return '<button class="btn btn-danger" data-action="leave" data-group="' + groupObj.name + '"><i class="fa fa-times"></i> Leave Group</button>';
+		} else {
+			if (groupObj.pending) {
+				return '<button class="btn btn-warning disabled"><i class="fa fa-clock-o"></i> Invitation Pending</button>';
+			} else {
+				return '<button class="btn btn-success" data-action="join" data-group="' + groupObj.name + '"><i class="fa fa-plus"></i> Join Group</button>';
+			}
+		}
+	};
+
+	exports.register = function() {
+		var templates;
+
+		if (typeof module === 'object') {
+			templates = require('templates.js');
+		} else {
+			templates = window.templates;
+		}
+
+		templates.registerHelper('displayUsersLink', helpers.displayUsersLink);
+		templates.registerHelper('buildMetaTag', helpers.buildMetaTag);
+	};
+
+	// Use the define() function if we're in AMD land
+	if (typeof define === 'function' && define.amd) {
+	  define('helpers', exports);
+	}
+
+})(
+	typeof exports === 'object' ? exports :
+	typeof define === 'function' && define.amd ? {} :
+	helpers = {}
+);

src/meta/js.js

@@ -45,7 +45,6 @@ module.exports = function(Meta) {
 				'public/src/variables.js',
 				'public/src/widgets.js',
 				'public/src/translator.js',
-				'public/src/helpers.js',
 				'public/src/overrides.js'
 			],
 			rjs: []

src/webserver.js

@@ -18,7 +18,7 @@ var path = require('path'),
 	routes = require('./routes'),
 	emitter = require('./emitter'),
 
-	helpers = require('./../public/src/helpers')(),
+	helpers = require('./../public/src/modules/helpers'),
 	net;
 
 if(nconf.get('ssl')) {
@@ -48,6 +48,9 @@ if(nconf.get('ssl')) {
 		middleware = middleware(app);
 		routes(app, middleware);
 
+		// Load server-side template helpers
+		helpers.register();
+
 		// Cache static files on production
 		if (global.env !== 'development') {
 			app.enable('cache');