From ca3b2a20277a9cfbacacb5a700dc0965a8249480 Mon Sep 17 00:00:00 2001
From: Dale Lotts <dlotts@knightrider.com>
Date: Sun, 31 Jan 2016 14:38:03 -0600
Subject: [PATCH] refactor(directive): Split out parsers into their own service

It may be reasonable to share parsers with other directives
---
 demo/index.html      |  4 +-
 karma.conf.js        |  2 +-
 package.json         |  7 ++--
 src/dateTimeInput.js | 91 +++++++++++++++++++++++---------------------
 4 files changed, 55 insertions(+), 49 deletions(-)

diff --git a/demo/index.html b/demo/index.html
index eedf347..dbb0d04 100644
--- a/demo/index.html
+++ b/demo/index.html
@@ -128,8 +128,8 @@
             </div>
 
         <p class="help-block">
-            Input format is 'M/D/YYYY' ( note the MMM for month ).
-            <br> Display format is 'M/D/YYYY'.
+            Input format is 'M/D/YYYY h:mm A'.
+            <br> Display format is 'M/D/YYYY h:mm A'.
             <br>Stored as: {{ data.dateDropDown }}
         </p>
         <div class="alert alert-info" role="alert">
diff --git a/karma.conf.js b/karma.conf.js
index c765c5c..b69f9aa 100644
--- a/karma.conf.js
+++ b/karma.conf.js
@@ -90,7 +90,7 @@ module.exports = function (config) {
     // - Safari (only Mac)
     // - PhantomJS
     // - IE (only Windows)
-    browsers: ['Chrome'],
+    browsers: ['PhantomJS'],
 
 
     // If browser does not capture in given timeout [ms], kill it
diff --git a/package.json b/package.json
index 3703fe4..ec98be1 100644
--- a/package.json
+++ b/package.json
@@ -24,6 +24,7 @@
     "grunt": "^0.4.4",
     "grunt-bump": "^0.7.0",
     "gulp": "^3.8.11",
+    "gulp-htmlmin": "^1.3.0",
     "gulp-jscs": "^3.0.2",
     "gulp-jshint": "^2.0.0",
     "jasmine-core": "^2.4.1",
@@ -35,11 +36,11 @@
     "karma-coverage": "^0.5.3",
     "karma-firefox-launcher": "^0.1.7",
     "karma-jasmine": "^0.3.2",
-    "karma-phantomjs-launcher": "^0.2.3",
+    "karma-phantomjs-launcher": "^1.0.0",
     "karma-threshold-reporter": "^0.1.12",
-    "lodash": "^4.0.0",
+    "lodash": "^4.1.0",
     "matchdep": "^1.0.0",
-    "phantomjs": "^1.9.19",
+    "phantomjs-prebuilt": "^2.1.3",
     "plato": "^1.5.0",
     "run-browser": "^2.0.2",
     "semantic-release": "^6.2.0",
diff --git a/src/dateTimeInput.js b/src/dateTimeInput.js
index c291d1f..462970e 100644
--- a/src/dateTimeInput.js
+++ b/src/dateTimeInput.js
@@ -24,10 +24,54 @@
 }(function (angular, moment) {
 	'use strict';
 	angular.module('ui.dateTimeInput', [])
-		.value('dateTimeInputConfig')
-		.directive('dateTimeInput', [dateTimeInputDirective]);
+		.service('dateTimeParserFactory', DateTimeParserFactoryService)
+		.directive('dateTimeInput', DateTimeInputDirective);
 
-	function dateTimeInputDirective() {
+	DateTimeParserFactoryService.$inject = [];
+
+	function DateTimeParserFactoryService() {
+		return function ParserFactory(modelType, inputFormats, dateParseStrict) {
+			var result;
+			// Behaviors
+			switch (modelType) {
+				case 'Date':
+					result = dateParser;
+					break;
+				case 'moment':
+					result = momentParser;
+					break;
+				case 'milliseconds':
+					result = millisecondParser;
+					break;
+				default: // It is assumed that the modelType is a formatting string.
+					result = stringParserFactory(modelType);
+			}
+
+			return result;
+
+			function dateParser(viewValue) {
+				return momentParser(viewValue).toDate();
+			}
+
+			function momentParser(viewValue) {
+				return moment(viewValue, inputFormats, moment.locale(), dateParseStrict);
+			}
+
+			function millisecondParser(viewValue) {
+				return moment.utc(viewValue, inputFormats, moment.locale(), dateParseStrict).valueOf();
+			}
+
+			function stringParserFactory(modelFormat) {
+				return function stringParser(viewValue) {
+					return momentParser(viewValue).format(modelFormat);
+				};
+			}
+		};
+	}
+
+	DateTimeInputDirective.$inject = ['dateTimeParserFactory'];
+
+	function DateTimeInputDirective(dateTimeParserFactory) {
 		return {
 			require: 'ngModel',
 			restrict: 'A',
@@ -58,7 +102,7 @@
 			var inputFormats = [attrs.dateTimeInput, modelType].concat(scope.dateFormats).concat([moment.ISO_8601]).filter(unique);
 
 			// Behaviors
-			controller.$parsers.unshift(parserFactory(modelType));
+			controller.$parsers.unshift(dateTimeParserFactory(modelType, inputFormats, dateParseStrict));
 
 			controller.$formatters.push(formatter);
 
@@ -66,7 +110,6 @@
 
 			element.bind('blur', applyFormatters);
 
-
 			// Implementation
 
 			function unique(value, index, self) {
@@ -91,44 +134,6 @@
 				return modelValue;
 			}
 
-			function parserFactory(modelType) {
-				var result;
-				// Behaviors
-				switch (modelType) {
-					case 'Date':
-						result = dateParser;
-						break;
-					case 'moment':
-						result = momentParser;
-						break;
-					case 'milliseconds':
-						result = millisecondParser;
-						break;
-					default: // It is assumed that the modelType is a formatting string.
-						result = stringParserFactory(modelType);
-				}
-
-				return result;
-
-				function dateParser(viewValue) {
-					return momentParser(viewValue).toDate();
-				}
-
-				function momentParser(viewValue) {
-					return moment(viewValue, inputFormats, moment.locale(), dateParseStrict);
-				}
-
-				function millisecondParser(viewValue) {
-					return moment.utc(viewValue, inputFormats, moment.locale(), dateParseStrict).valueOf();
-				}
-
-				function stringParserFactory(modelFormat) {
-					return function stringParser(viewValue) {
-						return momentParser(viewValue).format(modelFormat);
-					};
-				}
-			}
-
 			function applyFormatters() {
 				controller.$viewValue = controller.$formatters.filter(keepAll).reverse().reduce(applyFormatter, controller.$modelValue);
 				controller.$render();