From e46efa26ee0ba4b5191e4c503ea141f694646d01 Mon Sep 17 00:00:00 2001 From: Dannon Baker Date: Wed, 17 Sep 2014 21:11:12 -0400 Subject: [PATCH] Scratch the TODO, just removing those text replacements. Not necessary anymore. --- client/galaxy/scripts/utils/LazyDataLoader.js | 12 +----------- 1 file changed, 1 insertion(+), 11 deletions(-) diff --git a/client/galaxy/scripts/utils/LazyDataLoader.js b/client/galaxy/scripts/utils/LazyDataLoader.js index 90e4387fcf7..2c515a35123 100644 --- a/client/galaxy/scripts/utils/LazyDataLoader.js +++ b/client/galaxy/scripts/utils/LazyDataLoader.js @@ -123,17 +123,7 @@ function LazyDataLoader( config ){ '* text' : window.String, 'text html' : true, 'text xml' : jQuery.parseXML, - - // add NaN, inf, -inf handling to jquery json parser (by default) - // TODO: This functionality shouldn't be required as of - // 2006c39aea16, and we should probably assume correct JSON instead - // of stripping out these strings. - 'text json' : function( json ){ - json = json.replace( /NaN/g, 'null' ); - json = json.replace( /-Infinity/g, 'null' ); - json = json.replace( /Infinity/g, 'null' ); - return jQuery.parseJSON( json ); - } + 'text json' : jQuery.parseJSON }, // interface to begin load (and first recursive call)