From ceaca577869da601cdc5a986b8d1860fa075dde2 Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Thu, 24 Jul 2014 09:07:57 -0700 Subject: [PATCH] revert: perf($parse): don't use reflective calls in generated functions This reverts commit cbdf0c2afb9836ae4cca6d70cf555ff28f55a1d1. The commit causes tests failures on IE8. I'm not quite sure why and I can't investigate now. Failed build: https://travis-ci.org/angular/angular.js/jobs/30756773 --- src/ng/parse.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ng/parse.js b/src/ng/parse.js index be5abc95e9a5..425d8f6144fb 100644 --- a/src/ng/parse.js +++ b/src/ng/parse.js @@ -1020,7 +1020,7 @@ function getterFn(path, options, fullExp) { // we simply dereference 's' on any .dot notation ? 's' // but if we are first then we check locals first, and if so read it first - : '((k&&k.hasOwnProperty("' + key + '"))?k:s)') + '.' + key + ';\n' + + : '((k&&k.hasOwnProperty("' + key + '"))?k:s)') + '["' + key + '"]' + ';\n' + (options.unwrapPromises ? 'if (s && s.then) {\n' + ' pw("' + fullExp.replace(/(["\r\n])/g, '\\$1') + '");\n' +