Honor safe flag when retrieving cache. Closes #278.

telerik · Aug 29, 2014 · 0cc2b86 · 0cc2b86
1 parent 5cf001b
commit 0cc2b86
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/kendo.core.js b/src/kendo.core.js
@@ -2506,7 +2506,8 @@ function pad(number, digits, end) {
         },
 
         getter: function(expression, safe) {
-            return getterCache[expression] = getterCache[expression] || new Function("d", "return " + kendo.expr(expression, safe));
+            var key = expression + safe;
+            return getterCache[key] = getterCache[key] || new Function("d", "return " + kendo.expr(expression, safe));
         },
 
         setter: function(expression) {

diff --git a/tests/core/accessor.js b/tests/core/accessor.js
@@ -100,6 +100,12 @@ test("getter access property with safe options type is correct", function() {
     equal(getter("foo.bar", true)(dataItem), 0);
 });
 
+test("getter cache honors safe flag", function() {
+    var dataItem = {};
+    getter("foo.bar");
+    equal(getter("foo.bar", true)(dataItem), undefined);
+});
+
 test("setter creates function which sets the specified expression", function() {
     var data = { foo: "foo" };