diff options
| author | Stephen Vance <steve@vance.com> | 2013-07-14 17:37:30 -0400 |
|---|---|---|
| committer | Stephen Vance <steve@vance.com> | 2013-07-14 17:37:30 -0400 |
| commit | 11a2545cfe0b490490b6917c7968504bdefe1926 (patch) | |
| tree | 6c15ea36632ea23d64011c1cb4767c038625968e | |
| parent | 510908218f2fe780460039d2792af06b5db9ce89 (diff) | |
| download | jQuery-Timepicker-Addon-11a2545cfe0b490490b6917c7968504bdefe1926.zip jQuery-Timepicker-Addon-11a2545cfe0b490490b6917c7968504bdefe1926.tar.gz jQuery-Timepicker-Addon-11a2545cfe0b490490b6917c7968504bdefe1926.tar.bz2 | |
Refactor the effective setting computations out of splitDateTime()
| -rw-r--r-- | jquery-ui-timepicker-addon.js | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/jquery-ui-timepicker-addon.js b/jquery-ui-timepicker-addon.js index 37fb2b5..99724d5 100644 --- a/jquery-ui-timepicker-addon.js +++ b/jquery-ui-timepicker-addon.js @@ -1823,6 +1823,10 @@ return String(hour); }; + var computeEffectiveSetting = function (settings, property) { + return settings && settings[property] ? settings[property] : $.timepicker._defaults[property]; + }; + /* * Splits datetime string into date and time substrings. * Throws exception when date can't be parsed @@ -1832,8 +1836,8 @@ try { // The idea is to get the number separator occurrences in datetime and the time format requested (since time has // fewer unknowns, mostly numbers and am/pm). We will use the time pattern to split. - var separator = timeSettings && timeSettings.separator ? timeSettings.separator : $.timepicker._defaults.separator, - format = timeSettings && timeSettings.timeFormat ? timeSettings.timeFormat : $.timepicker._defaults.timeFormat, + var separator = computeEffectiveSetting(timeSettings, 'separator'), + format = computeEffectiveSetting(timeSettings, 'timeFormat'), timeParts = format.split(separator), // how many occurrences of separator may be in our format? timePartsLen = timeParts.length, allParts = dateTimeString.split(separator), |