Add ESLint rules prefer-template and template-curly-spacing

.eslintrc

@@ -76,6 +76,8 @@
 		"use-isnan": 2,
 		"valid-typeof": 2,
 		"linebreak-style": [2, "unix"],
+		"prefer-template": 2,
+		"template-curly-spacing": [2, "always"],
 		"quotes": [2, "single"],
 		"semi": [2, "always"],
 		"prefer-const": 2,

client/routes/router.js

@@ -84,7 +84,7 @@ FlowRouter.route('/account/:group?', {
 			params.group = 'Preferences';
 		}
 		params.group = _.capitalize(params.group, true);
-		BlazeLayout.render('main', { center: `account${params.group}` });
+		BlazeLayout.render('main', { center: `account${ params.group }` });
 	}
 });
 

client/startup/roomObserve.js

@@ -1,13 +1,13 @@
 Meteor.startup(function() {
 	ChatRoom.find().observe({
 		added(data) {
-			Session.set('roomData' + data._id, data);
+			Session.set(`roomData${ data._id }`, data);
 		},
 		changed(data) {
-			Session.set('roomData' + data._id, data);
+			Session.set(`roomData${ data._id }`, data);
 		},
 		removed(data) {
-			Session.set('roomData' + data._id, undefined);
+			Session.set(`roomData${ data._id }`, undefined);
 		}
 	});
 });

client/startup/unread.js

@@ -78,6 +78,6 @@ Meteor.startup(function() {
 			});
 		}
 
-		document.title = unread === '' ? siteName : `(${unread}) ${siteName}`;
+		document.title = unread === '' ? siteName : `(${ unread }) ${ siteName }`;
 	});
 });

client/startup/usersObserve.js

@@ -1,15 +1,15 @@
 Meteor.startup(function() {
 	Meteor.users.find({}, { fields: { name: 1, username: 1, pictures: 1, status: 1, emails: 1, phone: 1, services: 1, utcOffset: 1 } }).observe({
 		added(user) {
-			Session.set('user_' + user.username + '_status', user.status);
+			Session.set(`user_${ user.username }_status`, user.status);
 			RoomManager.updateUserStatus(user, user.status, user.utcOffset);
 		},
 		changed(user) {
-			Session.set('user_' + user.username + '_status', user.status);
+			Session.set(`user_${ user.username }_status`, user.status);
 			RoomManager.updateUserStatus(user, user.status, user.utcOffset);
 		},
 		removed(user) {
-			Session.set('user_' + user.username + '_status', null);
+			Session.set(`user_${ user.username }_status`, null);
 			RoomManager.updateUserStatus(user, 'offline', null);
 		}
 	});

lib/fileUpload.js

@@ -5,8 +5,8 @@ if (UploadFS) {
 	const initFileStore = function() {
 		const cookie = new Cookies();
 		if (Meteor.isClient) {
-			document.cookie = 'rc_uid=' + escape(Meteor.userId()) + '; path=/';
-			document.cookie = 'rc_token=' + escape(Accounts._storedLoginToken()) + '; path=/';
+			document.cookie = `rc_uid=${ escape(Meteor.userId()) }; path=/`;
+			document.cookie = `rc_token=${ escape(Accounts._storedLoginToken()) }; path=/`;
 		}
 
 		Meteor.fileStore = new UploadFS.store.GridFS({

packages/meteor-accounts-saml/saml_client.js

@@ -60,8 +60,8 @@ const openCenteredPopup = function(url, width, height) {
 		// positioning the popup centered relative to the current window
 		const left = screenX + (outerWidth - width) / 2;
 		const top = screenY + (outerHeight - height) / 2;
-		const features = ('width=' + width + ',height=' + height +
-			',left=' + left + ',top=' + top + ',scrollbars=yes');
+		const features = (`width=${ width },height=${ height
+			},left=${ left },top=${ top },scrollbars=yes`);
 
 		newwindow = window.open(url, 'Login', features);
 		if (newwindow.focus) {
@@ -74,7 +74,7 @@ const openCenteredPopup = function(url, width, height) {
 Accounts.saml.initiateLogin = function(options, callback, dimensions) {
 	// default dimensions that worked well for facebook and google
 	const popup = openCenteredPopup(
-		Meteor.absoluteUrl('_saml/authorize/' + options.provider + '/' + options.credentialToken), (dimensions && dimensions.width) || 650, (dimensions && dimensions.height) || 500);
+		Meteor.absoluteUrl(`_saml/authorize/${ options.provider }/${ options.credentialToken }`), (dimensions && dimensions.width) || 650, (dimensions && dimensions.height) || 500);
 
 	const checkPopupOpen = setInterval(function() {
 		let popupClosed;
@@ -124,6 +124,6 @@ Meteor.logoutWithSaml = function(options/*, callback*/) {
 		}
 		// A nasty bounce: 'result' has the SAML LogoutRequest but we need a proper 302 to redirected from the server.
 		//window.location.replace(Meteor.absoluteUrl('_saml/sloRedirect/' + options.provider + '/?redirect='+result));
-		window.location.replace(Meteor.absoluteUrl('_saml/sloRedirect/' + options.provider + '/?redirect=' + encodeURIComponent(result)));
+		window.location.replace(Meteor.absoluteUrl(`_saml/sloRedirect/${ options.provider }/?redirect=${ encodeURIComponent(result) }`));
 	});
 };

packages/meteor-accounts-saml/saml_rocketchat.js

@@ -10,76 +10,76 @@ RocketChat.settings.addGroup('SAML');
 
 Meteor.methods({
 	addSamlService(name) {
-		RocketChat.settings.add(`SAML_Custom_${name}`, false, {
+		RocketChat.settings.add(`SAML_Custom_${ name }`, false, {
 			type: 'boolean',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'Accounts_OAuth_Custom_Enable'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_provider`, 'provider-name', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_provider`, 'provider-name', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'SAML_Custom_Provider'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_entry_point`, 'https://example.com/simplesaml/saml2/idp/SSOService.php', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_entry_point`, 'https://example.com/simplesaml/saml2/idp/SSOService.php', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'SAML_Custom_Entry_point'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_idp_slo_redirect_url`, 'https://example.com/simplesaml/saml2/idp/SingleLogoutService.php', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_idp_slo_redirect_url`, 'https://example.com/simplesaml/saml2/idp/SingleLogoutService.php', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'SAML_Custom_IDP_SLO_Redirect_URL'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_issuer`, 'https://your-rocket-chat/_saml/metadata/provider-name', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_issuer`, 'https://your-rocket-chat/_saml/metadata/provider-name', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'SAML_Custom_Issuer'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_cert`, '', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_cert`, '', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'SAML_Custom_Cert',
 			multiline: true
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_public_cert`, '', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_public_cert`, '', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			multiline: true,
 			i18nLabel: 'SAML_Custom_Public_Cert'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_private_key`, '', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_private_key`, '', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			multiline: true,
 			i18nLabel: 'SAML_Custom_Private_Key'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_button_label_text`, '', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_button_label_text`, '', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'Accounts_OAuth_Custom_Button_Label_Text'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_button_label_color`, '#FFFFFF', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_button_label_color`, '#FFFFFF', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'Accounts_OAuth_Custom_Button_Label_Color'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_button_color`, '#13679A', {
+		RocketChat.settings.add(`SAML_Custom_${ name }_button_color`, '#13679A', {
 			type: 'string',
 			group: 'SAML',
 			section: name,
 			i18nLabel: 'Accounts_OAuth_Custom_Button_Color'
 		});
-		RocketChat.settings.add(`SAML_Custom_${name}_generate_username`, false, {
+		RocketChat.settings.add(`SAML_Custom_${ name }_generate_username`, false, {
 			type: 'boolean',
 			group: 'SAML',
 			section: name,
@@ -90,20 +90,20 @@ Meteor.methods({
 
 const getSamlConfigs = function(service) {
 	return {
-		buttonLabelText: RocketChat.settings.get(service.key + '_button_label_text'),
-		buttonLabelColor: RocketChat.settings.get(service.key + '_button_label_color'),
-		buttonColor: RocketChat.settings.get(service.key + '_button_color'),
+		buttonLabelText: RocketChat.settings.get(`${ service.key }_button_label_text`),
+		buttonLabelColor: RocketChat.settings.get(`${ service.key }_button_label_color`),
+		buttonColor: RocketChat.settings.get(`${ service.key }_button_color`),
 		clientConfig: {
-			provider: RocketChat.settings.get(service.key + '_provider')
+			provider: RocketChat.settings.get(`${ service.key }_provider`)
 		},
-		entryPoint: RocketChat.settings.get(service.key + '_entry_point'),
-		idpSLORedirectURL: RocketChat.settings.get(service.key + '_idp_slo_redirect_url'),
-		generateUsername: RocketChat.settings.get(service.key + '_generate_username'),
-		issuer: RocketChat.settings.get(service.key + '_issuer'),
+		entryPoint: RocketChat.settings.get(`${ service.key }_entry_point`),
+		idpSLORedirectURL: RocketChat.settings.get(`${ service.key }_idp_slo_redirect_url`),
+		generateUsername: RocketChat.settings.get(`${ service.key }_generate_username`),
+		issuer: RocketChat.settings.get(`${ service.key }_issuer`),
 		secret: {
-			privateKey: RocketChat.settings.get(service.key + '_private_key'),
-			publicCert: RocketChat.settings.get(service.key + '_public_cert'),
-			cert: RocketChat.settings.get(service.key + '_cert')
+			privateKey: RocketChat.settings.get(`${ service.key }_private_key`),
+			publicCert: RocketChat.settings.get(`${ service.key }_public_cert`),
+			cert: RocketChat.settings.get(`${ service.key }_cert`)
 		}
 	};
 };

packages/meteor-accounts-saml/saml_server.js

@@ -39,7 +39,7 @@ Meteor.methods({
 		const providerConfig = getSamlProviderConfig(provider);
 
 		if (Accounts.saml.settings.debug) {
-			console.log('Logout request from ' + JSON.stringify(providerConfig));
+			console.log(`Logout request from ${ JSON.stringify(providerConfig) }`);
 		}
 		// This query should respect upcoming array of SAML logins
 		const user = Meteor.users.findOne({
@@ -52,7 +52,7 @@ Meteor.methods({
 		const sessionIndex = user.services.saml.idpSession;
 		nameID = sessionIndex;
 		if (Accounts.saml.settings.debug) {
-			console.log('NameID for user ' + Meteor.userId() + ' found: ' + JSON.stringify(nameID));
+			console.log(`NameID for user ${ Meteor.userId() } found: ${ JSON.stringify(nameID) }`);
 		}
 
 		const _saml = new SAML(providerConfig);
@@ -76,7 +76,7 @@ Meteor.methods({
 		const _syncRequestToUrl = Meteor.wrapAsync(_saml.requestToUrl, _saml);
 		const result = _syncRequestToUrl(request.request, 'logout');
 		if (Accounts.saml.settings.debug) {
-			console.log('SAML Logout Request ' + result);
+			console.log(`SAML Logout Request ${ result }`);
 		}
 
 
@@ -91,7 +91,7 @@ Accounts.registerLoginHandler(function(loginRequest) {
 
 	const loginResult = Accounts.saml.retrieveCredential(loginRequest.credentialToken);
 	if (Accounts.saml.settings.debug) {
-		console.log('RESULT :' + JSON.stringify(loginResult));
+		console.log(`RESULT :${ JSON.stringify(loginResult) }`);
 	}
 
 	if (loginResult === undefined) {
@@ -186,7 +186,7 @@ const closePopup = function(res, err) {
 	});
 	let content = '<html><head><script>window.close()</script></head><body><H1>Verified</H1></body></html>';
 	if (err) {
-		content = '<html><body><h2>Sorry, an annoying error occured</h2><div>' + err + '</div><a onclick="window.close();">Close Window</a></body></html>';
+		content = `<html><body><h2>Sorry, an annoying error occured</h2><div>${ err }</div><a onclick="window.close();">Close Window</a></body></html>`;
 	}
 	res.end(content, 'utf-8');
 };
@@ -239,13 +239,13 @@ const middleware = function(req, res, next) {
 
 		// Skip everything if there's no service set by the saml middleware
 		if (!service) {
-			throw new Error('Unexpected SAML service ' + samlObject.serviceName);
+			throw new Error(`Unexpected SAML service ${ samlObject.serviceName }`);
 		}
 		let _saml;
 		switch (samlObject.actionName) {
 			case 'metadata':
 				_saml = new SAML(service);
-				service.callbackUrl = Meteor.absoluteUrl('_saml/validate/' + service.provider);
+				service.callbackUrl = Meteor.absoluteUrl(`_saml/validate/${ service.provider }`);
 				res.writeHead(200);
 				res.write(_saml.generateServiceProviderMetadata(service.callbackUrl));
 				res.end();
@@ -258,14 +258,14 @@ const middleware = function(req, res, next) {
 					if (!err) {
 						const logOutUser = function(inResponseTo) {
 							if (Accounts.saml.settings.debug) {
-								console.log('Logging Out user via inResponseTo ' + inResponseTo);
+								console.log(`Logging Out user via inResponseTo ${ inResponseTo }`);
 							}
 							const loggedOutUser = Meteor.users.find({
 								'services.saml.inResponseTo': inResponseTo
 							}).fetch();
 							if (loggedOutUser.length === 1) {
 								if (Accounts.saml.settings.debug) {
-									console.log('Found user ' + loggedOutUser[0]._id);
+									console.log(`Found user ${ loggedOutUser[0]._id }`);
 								}
 								Meteor.users.update({
 									_id: loggedOutUser[0]._id
@@ -309,7 +309,7 @@ const middleware = function(req, res, next) {
 				res.end();
 				break;
 			case 'authorize':
-				service.callbackUrl = Meteor.absoluteUrl('_saml/validate/' + service.provider);
+				service.callbackUrl = Meteor.absoluteUrl(`_saml/validate/${ service.provider }`);
 				service.id = samlObject.credentialToken;
 				_saml = new SAML(service);
 				_saml.getAuthorizeUrl(req, function(err, url) {
@@ -327,7 +327,7 @@ const middleware = function(req, res, next) {
 				Accounts.saml.RelayState = req.body.RelayState;
 				_saml.validateResponse(req.body.SAMLResponse, req.body.RelayState, function(err, profile/*, loggedOut*/) {
 					if (err) {
-						throw new Error('Unable to validate response url: ' + err);
+						throw new Error(`Unable to validate response url: ${ err }`);
 					}
 
 					const credentialToken = profile.inResponseToId || profile.InResponseTo || samlObject.credentialToken;
@@ -337,7 +337,7 @@ const middleware = function(req, res, next) {
 						Accounts.saml._loginResultForCredentialToken[saml_idp_credentialToken] = {
 							profile
 						};
-						const url = Meteor.absoluteUrl('home') + '?saml_idp_credentialToken='+saml_idp_credentialToken;
+						const url = `${ Meteor.absoluteUrl('home') }?saml_idp_credentialToken=${ saml_idp_credentialToken }`;
 						res.writeHead(302, {
 							'Location': url
 						});
@@ -351,7 +351,7 @@ const middleware = function(req, res, next) {
 				});
 				break;
 			default:
-				throw new Error('Unexpected SAML action ' + samlObject.actionName);
+				throw new Error(`Unexpected SAML action ${ samlObject.actionName }`);
 
 		}
 	} catch (err) {