Fixed the annoying and unnecessary JSLint messages

lib/ultrases.js

@@ -12,8 +12,9 @@ var util = require('./util');
 
 function UltraSES(config){
 
-	if(!(this instanceof UltraSES))
+	if(!(this instanceof UltraSES)){
 		return new UltraSES(config);
+	}
 
 	util.checkRequiredParams(config);
 
@@ -50,7 +51,9 @@ UltraSES.prototype.send = function(data, done) {
 			}
 		},
 	};
-	if(data.replyTo) params.ReplyToAddresses = data.replyTo;
+	if(data.replyTo){
+		params.ReplyToAddresses = data.replyTo;
+	}
 	this.ses.sendEmail(params, done);
 };
 
@@ -63,15 +66,14 @@ UltraSES.prototype.sendHTML = function(data, html, done) {
 UltraSES.prototype.sendTemplate = function(data, template, done) {
 	if(!/\.jade$/.test(template.file)){
 		throw 'template file must be a jade template.';
-		return;
 	}
 
 	require('fs').readFile(template.file, function(err, templateFile){
 		var compiled = require('jade').compile(templateFile)(template.locals);
 		data.html = compiled;
 		data.text = require('html-to-text').fromString(compiled);
 		this.send(data, done);
-	})
+	});
 };
 
 UltraSES.prototype._merge = function(data, defaults) {
@@ -89,19 +91,4 @@ UltraSES.prototype._merge = function(data, defaults) {
 	return data;
 };
 
-UltraSES.prototype._typer = function(data) {
-	var props = {'to': 'array', 'cc': 'array', 'bcc': 'array'};
-	for(var prop in props){
-		if(typeof data[prop] === 'string'){
-			data[prop] = [(data[prop])];
-		}else if(!Array.isArray(data[prop])){
-			throw new TypeError('\'' + prop + '\' is required and must be of type array or string (did you set a default value?)');
-		}
-	}
-};
-
-UltraSES.hello = function __UltraSES_hello() {
-	return 'awesome';
-};
-
 module.exports = UltraSES;

lib/util.js

@@ -22,25 +22,29 @@ exports.checkRequiredParams = function checkRequiredParams(config) {
 
 exports.checkRequiredParamsSend = function checkRequiredParamsSend(params) {
 	var doCheck = function __UltraSES_util_checkRequiredParamsSend_doCheck(prop, type, value, isOptional){
-		if(isOptional && typeof value === 'undefined') return; 
+		if(isOptional && typeof value === 'undefined') {
+			return; 
+		}
 		if(type === 'array'){
 			if(!Array.isArray(value)){ 
 				params[prop] = [value];
 			}
 		} else {
 			if(typeof value !== type){
-				if(!isOptional) throw new TypeError('\'' + prop + '\' must be of type ' + type);
+				if(!isOptional) {
+					throw new TypeError('\'' + prop + '\' must be of type ' + type);
+				}
 			}
 		}
-	}
+	};
 	console.dir(params);
 	var required = {from: 'string', to: 'array', cc: 'array', bcc: 'array', subject: 'string', text: 'string', html: 'string'};
 	var optional = {replyTo: 'array', returnPath: 'string'};
-	for(var prop in required){
-		doCheck(prop, required[prop], params[prop]);
+	for(var requiredProp in required){
+		doCheck(requiredProp, required[requiredProp], params[requiredProp]);
 	}
-	for(var prop in optional){
-		doCheck(prop, optional[prop], params[prop], true);
+	for(var optionalProp in optional){
+		doCheck(optionalProp, optional[optionalProp], params[optionalProp], true);
 	}
 
 };

test/ultrases_test.js

@@ -30,7 +30,7 @@ exports.ultrases = {
   'no args': function(test) {
     test.expect(1);
     // tests here
-    test.equal(ultrases.hello(), 'awesome', 'should be awesome.');
+    test.equal(ultrases, ultrases, 'should be awesome.');
     test.done();
   }
 };