[Time Conductor] Update formatService usages

...to remove checks for unknown formats.
This commit is contained in:
Victor Woeltjen 2015-11-02 12:28:46 -08:00
parent a1d765f271
commit cecc52f0a9
4 changed files with 4 additions and 19 deletions

View File

@ -26,9 +26,6 @@ define(
function () {
'use strict';
var UNRECOGNIZED_FORMAT_ERROR =
"Unrecognized format for date-time field.";
/**
* Controller to support the date-time entry field.
*
@ -69,9 +66,6 @@ define(
function setFormat(format) {
formatter = formatService.getFormat(format || defaultFormat);
if (!formatter) {
throw new Error(UNRECOGNIZED_FORMAT_ERROR);
}
updateFromModel($scope.ngModel[$scope.field]);
}

View File

@ -26,9 +26,7 @@ define(
function (moment) {
"use strict";
var TICK_SPACING_PX = 150,
UNRECOGNIZED_FORMAT_ERROR =
"Unrecognized format for time range control.";
var TICK_SPACING_PX = 150;
/**
@ -221,11 +219,6 @@ define(
function updateFormat(key) {
formatter = formatService.getFormat(key || defaultFormat);
if (!formatter) {
throw new Error(UNRECOGNIZED_FORMAT_ERROR);
}
updateViewForInnerSpanFromModel($scope.ngModel);
updateTicks();
}

View File

@ -38,7 +38,7 @@
{
"key": "telemetryFormatter",
"implementation": "TelemetryFormatter.js",
"depends": [ "formatService", "DEFAULT_TIME_FORMAT", "$log" ]
"depends": [ "formatService", "DEFAULT_TIME_FORMAT" ]
},
{
"key": "telemetrySubscriber",

View File

@ -40,12 +40,10 @@ define(
* domain values
* @param {string} defaultFormatKey the format to request when no
* format has been otherwise specified
* @param $log Angular's `$log`, to log warnings
*/
function TelemetryFormatter(formatService, defaultFormatKey, $log) {
function TelemetryFormatter(formatService, defaultFormatKey) {
this.formatService = formatService;
this.defaultFormat = formatService.getFormat(defaultFormatKey);
this.$log = $log;
}
/**
@ -62,7 +60,7 @@ define(
this.defaultFormat :
this.formatService.getFormat(key);
return isNaN(v) ? "" : formatter ? formatter.format(v) : String(v);
return isNaN(v) ? "" : formatter.format(v);
};
/**