Fixes issue #29 and allows optional fields to be omitted during object creation

lib/batch/control.js

@@ -1,3 +1,5 @@
+const utils = require('../utils')
+
 module.exports = {
 	recordTypeCode: {
 		name: 'Record Type Code',
@@ -70,7 +72,7 @@ module.exports = {
 		required: false,
 		type: 'alphanumeric',
 		blank: true,
-		value: ''
+		value: utils.SPACE.repeat(19)
 	},
 
 	reserved: {
@@ -80,7 +82,7 @@ module.exports = {
 		required: false,
 		type: 'alphanumeric',
 		blank: true,
-		value: ''
+		value: utils.SPACE.repeat(6)
 	},
 
 	originatingDFI: {

lib/batch/header.js

@@ -1,3 +1,5 @@
+const utils = require('../utils')
+
 module.exports = {
 	recordTypeCode: {
 		name: 'Record Type Code',
@@ -33,7 +35,7 @@ module.exports = {
 		required: false,
 		type: 'alphanumeric',
 		blank: true,
-		value: ''
+		value: utils.SPACE.repeat(20)
 	},
 
 	companyIdentification: {
@@ -69,7 +71,7 @@ module.exports = {
 		position: 8,
 		required: false,
 		type: 'alphanumeric',
-		value: ''
+		value: utils.SPACE.repeat(6)
 	},
 
 	effectiveEntryDate: {
@@ -88,7 +90,7 @@ module.exports = {
 		required: false,
 		type: 'numeric',
 		blank: true,
-		value: ''
+		value: utils.SPACE.repeat(3)
 	},
 
 	originatorStatusCode: {

lib/entry-addenda/fields.js

@@ -1,3 +1,5 @@
+const utils = require('../utils');
+
 module.exports = {
 	recordTypeCode: {
 		name: 'Record Type Code',
@@ -14,7 +16,7 @@ module.exports = {
 		position: 2,
 		required: true,
 		type: 'numeric',
-    value: '05'
+    	value: '05'
 	},
 
 	paymentRelatedInformation: {
@@ -23,7 +25,7 @@ module.exports = {
 		position: 3,
 		required: false,
 		type: 'alphanumeric',
-		value: ''
+		value: utils.SPACE.repeat(80)
 	},
 
 	addendaSequenceNumber: {

lib/entry/fields.js

@@ -1,3 +1,5 @@
+const utils = require('../utils');
+
 module.exports = {
 	recordTypeCode: {
 		name: 'Record Type Code',
@@ -59,7 +61,7 @@ module.exports = {
 		position: 7,
 		required: false,
 		type: 'alphanumeric',
-		value: ''
+		value: utils.SPACE.repeat(15)
 	},
 
 	individualName: {
@@ -77,7 +79,7 @@ module.exports = {
 		position: 9,
 		required: false,
 		type: 'alphanumeric',
-		value: '',
+		value: utils.SPACE.repeat(2)
 	},
 
 	addendaId: {

lib/error.js

@@ -7,4 +7,4 @@ var nACHError = function(errorObj) {
 nACHError.prototype = Object.create(Error.prototype);
 nACHError.prototype.constructor = nACHError;
 
-module.exports = nACHError;
+module.exports = nACHError;

lib/file/control.js

@@ -65,4 +65,4 @@ module.exports = {
 		blank: true,
 		value: ''
 	}
-};
+};

lib/file/header.js

@@ -1,125 +1,122 @@
 var utils = require('../utils');
 
-// The date/time defaults are dynamic, so always create a new object
-module.exports = function () {
-	return {
-		recordTypeCode: {
-			name: 'Record Type Code',
-			width: 1,
-			position: 1,
-			required: true,
-			type: 'numeric',
-			value: '1'
-		},
+module.exports = {
+	recordTypeCode: {
+		name: 'Record Type Code',
+		width: 1,
+		position: 1,
+		required: true,
+		type: 'numeric',
+		value: '1'
+	},
 
-		priorityCode: {
-			name: 'Priority Code',
-			width: 2,
-			position: 2,
-			required: true,
-			type: 'numeric',
-			value: '01'
-		},
+	priorityCode: {
+		name: 'Priority Code',
+		width: 2,
+		position: 2,
+		required: true,
+		type: 'numeric',
+		value: '01'
+	},
 
-		immediateDestination: {
-			name: 'Immediate Destination',
-			width: 10,
-			position: 3,
-			required: true,
-			type: 'ABA',
-			paddingChar: ' ',
-			value: ''
-		},
+	immediateDestination: {
+		name: 'Immediate Destination',
+		width: 10,
+		position: 3,
+		required: true,
+		type: 'ABA',
+		paddingChar: ' ',
+		value: ''
+	},
 
-		immediateOrigin: {
-			name: 'Immediate Origin',
-			width: 10,
-			position: 4,
-			required: true,
-			type: 'numeric',
-			paddingChar: ' ',
-			value: ''
-		},
+	immediateOrigin: {
+		name: 'Immediate Origin',
+		width: 10,
+		position: 4,
+		required: true,
+		type: 'numeric',
+		paddingChar: ' ',
+		value: ''
+	},
 
-		fileCreationDate: {
-			name: 'File Creation Date',
-			width: 6,
-			position: 5,
-			required: true,
-			type: 'numeric',
-			value: utils.formatDate(new Date())
-		},
+	fileCreationDate: {
+		name: 'File Creation Date',
+		width: 6,
+		position: 5,
+		required: true,
+		type: 'numeric',
+		value: utils.formatDate(new Date())
+	},
 
-		fileCreationTime: {
-			name: 'File Creation Time',
-			width: 4,
-			position: 6,
-			required: true,
-			type: 'numeric',
-			value: utils.formatTime(new Date())
-		},
+	fileCreationTime: {
+		name: 'File Creation Time',
+		width: 4,
+		position: 6,
+		required: true,
+		type: 'numeric',
+		value: utils.formatTime(new Date())
+	},
 
-		fileIdModifier: {
-			name: 'File Modifier',
-			width: 1,
-			position: 7,
-			required: true,
-			type: 'alphanumeric',
-			value: 'A'
-		},
+	fileIdModifier: {
+		name: 'File Modifier',
+		width: 1,
+		position: 7,
+		required: true,
+		type: 'alphanumeric',
+		value: 'A'
+	},
 
-		recordSize: {
-			name: 'Record Size',
-			width: 3,
-			position: 8,
-			type: 'numeric',
-			required: true,
-			value: '094'
-		},
+	recordSize: {
+		name: 'Record Size',
+		width: 3,
+		position: 8,
+		type: 'numeric',
+		required: true,
+		value: '094'
+	},
 
-		blockingFactor: {
-			name: 'Blocking Factor',
-			width: 2,
-			position: 9,
-			type: 'numeric',
-			required: true,
-			value: '10'
-		},
+	blockingFactor: {
+		name: 'Blocking Factor',
+		width: 2,
+		position: 9,
+		type: 'numeric',
+		required: true,
+		value: '10'
+	},
 
-		formatCode: {
-			name: 'Format Code',
-			width: 1,
-			position: 10,
-			required: true,
-			type: 'numeric',
-			value: '1'
-		},
+	formatCode: {
+		name: 'Format Code',
+		width: 1,
+		position: 10,
+		required: true,
+		type: 'numeric',
+		value: '1'
+	},
 
-		immediateDestinationName: {
-			name: 'Immediate Destination Name',
-			width: 23,
-			position: 11,
-			required: true,
-			type: 'alphanumeric',
-			value: ''
-		},
+	immediateDestinationName: {
+		name: 'Immediate Destination Name',
+		width: 23,
+		position: 11,
+		required: true,
+		type: 'alphanumeric',
+		value: ''
+	},
 
-		immediateOriginName: {
-			name: 'Immediate Origin Name',
-			width: 23,
-			position: 12,
-			required: true,
-			type: 'alphanumeric',
-			value: ''
-		},
+	immediateOriginName: {
+		name: 'Immediate Origin Name',
+		width: 23,
+		position: 12,
+		required: true,
+		type: 'alphanumeric',
+		value: ''
+	},
 
-		referenceCode: {
-			name: 'Reference Code',
-			width: 8,
-			position: 13,
-			required: true,
-			type: 'alphanumeric',
-			value: ''
-		}
-	};
+	referenceCode: {
+		name: 'Reference Code',
+		width: 8,
+		position: 13,
+		required: false,
+		type: 'alphanumeric',
+		value: utils.SPACE.repeat(8)
+	}
 };

lib/file/index.js

@@ -10,7 +10,7 @@ function File(options) {
 	this._batches = [];
 
 	// Allow the batch header/control defaults to be overriden if provided
-	this.header = options.header ? _.merge(options.header, require('./header')(), _.defaults) : require('./header')();
+	this.header = options.header ? _.merge(options.header, require('./header'), _.defaults) : _.cloneDeep(require('./header'));
 	this.control = options.control ? _.merge(options.header, require('./control'), _.defaults) : _.cloneDeep(require('./control'));
 
 	// Configure high-level overrides (these override the low-level settings if provided)
@@ -183,4 +183,4 @@ File.prototype.generateFile = function(cb) {
 	});
 };
 
-module.exports = File;
+module.exports = File;

lib/utils.js

@@ -4,6 +4,7 @@ var _ = require('lodash');
 var moment = require('moment');
 var nACHError = require('./error');
 var counter = 0;
+const SPACE = " ";
 
 // Pad a given string to a fixed width using any character or number (defaults to one blank space)
 // Both a string and width are required params for this function, but it also takes two optional
@@ -157,4 +158,5 @@ module.exports.getNextMultiple     = getNextMultiple;
 module.exports.overrideLowLevel    = overrideLowLevel;
 module.exports.computeCheckDigit   = computeCheckDigit;
 module.exports.computeBusinessDay  = computeBusinessDay;
-module.exports.getNextMultipleDiff = getNextMultipleDiff;
+module.exports.getNextMultipleDiff = getNextMultipleDiff;
+module.exports.SPACE               = SPACE;