From 9d70d7c27e56e75408980d82471f737911a43371 Mon Sep 17 00:00:00 2001 From: John-David Dalton Date: Sun, 8 Jul 2012 03:39:38 -0400 Subject: [PATCH] Fix `_.sortBy` in the mobile build attempt two. Former-commit-id: 8f54b16a901acc4ce91f071c5b36c632334f7dde --- build/pre-compile.js | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/build/pre-compile.js b/build/pre-compile.js index c8e3ade81..e14575f69 100644 --- a/build/pre-compile.js +++ b/build/pre-compile.js @@ -258,13 +258,14 @@ } snippets.forEach(function(snippet) { var result = snippet, - isSortBy = /var sortBy\b/.test(result); + isSortBy = /var sortBy\b/.test(result), + isInlined = !/\bcreateIterator\b/.test(result); // minify properties properties.forEach(function(property, index) { - // add quotes around properties in the inlined method of the mobile build - // so Closure Compiler won't mung them - if (isSortBy) { + // add quotes around properties in the inlined `sortBy` of the mobile + // build so Closure Compiler won't mung them + if (isSortBy && isInlined) { result = result .replace(RegExp('\\.' + property + '\\b', 'g'), "['" + minNames[index] + "']") .replace(RegExp('\\b' + property + ' *:', 'g'), "'" + minNames[index] + "':");