From: Rafael Xavier de Souza <rxaviers@gmail.com>
Date: Wed, 25 Jun 2014 00:38:43 +0000 (-0700)
Subject: Spinner: Use Globalize 1.x
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=e1b0a2777a0c1a12344e7cd1dd0d8be19f49b040;p=jquery-ui.git

Spinner: Use Globalize 1.x
---

diff --git a/ui/spinner.js b/ui/spinner.js
index 263b29d3f..40b758d41 100644
--- a/ui/spinner.js
+++ b/ui/spinner.js
@@ -14,6 +14,7 @@
 		// AMD. Register as an anonymous module.
 		define([
 			"jquery",
+			"globalize",
 			"./core",
 			"./widget",
 			"./button"
@@ -23,7 +24,7 @@
 		// Browser globals
 		factory( jQuery );
 	}
-}(function( $ ) {
+}(function( $, Globalize ) {
 
 function spinner_modifier( fn ) {
 	return function() {
@@ -407,9 +408,10 @@ return $.widget( "ui.spinner", {
 
 	_parse: function( val ) {
 		if ( typeof val === "string" && val !== "" ) {
-			val = window.Globalize && this.options.numberFormat ?
-				Globalize.parseFloat( val, 10, this.options.culture ) : +val;
+			// FIXME handle optional different locales
+			val = Globalize ? Globalize.parseNumber( val ) : +val;
 		}
+		// TODO: Why not to return NaN?
 		return val === "" || isNaN( val ) ? null : val;
 	},
 
@@ -417,9 +419,8 @@ return $.widget( "ui.spinner", {
 		if ( value === "" ) {
 			return "";
 		}
-		return window.Globalize && this.options.numberFormat ?
-			Globalize.format( value, this.options.numberFormat, this.options.culture ) :
-			value;
+		// FIXME handle optional different locales
+		return Globalize ? Globalize.formatNumber( value, this.options.numberFormat ) : value;
 	},
 
 	_refresh: function() {