From: Jukka Kurkela <jukka.kurkela@gmail.com>
Date: Wed, 1 Dec 2021 22:53:55 +0000 (+0200)
Subject: Fix resolver caching issue when setting values (#9938)
X-Git-Tag: v3.6.2~2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=db995af1584f981ef1b75df55db5043ded687087;p=thirdparty%2FChart.js.git

Fix resolver caching issue when setting values (#9938)
---

diff --git a/src/helpers/helpers.config.js b/src/helpers/helpers.config.js
index 4d27f9e5b..30567c912 100644
--- a/src/helpers/helpers.config.js
+++ b/src/helpers/helpers.config.js
@@ -76,8 +76,7 @@ export function _createResolver(scopes, prefixes = [''], rootScopes = scopes, fa
      */
     set(target, prop, value) {
       const storage = target._storage || (target._storage = getTarget());
-      storage[prop] = value; // set to top level scope
-      delete target[prop]; // remove from cache
+      target[prop] = storage[prop] = value; // set to top level scope + cache
       delete target._keys; // remove cached keys
       return true;
     }
diff --git a/test/specs/helpers.config.tests.js b/test/specs/helpers.config.tests.js
index 58e295b58..76d056a33 100644
--- a/test/specs/helpers.config.tests.js
+++ b/test/specs/helpers.config.tests.js
@@ -492,6 +492,7 @@ describe('Chart.helpers.config', function() {
         resolver.value = false;
         expect(options.value).toBeFalse();
         expect(defaults.value).toBeTrue();
+        expect(resolver.value).toBeFalse();
       });
 
       it('should set values of sub-objects to first scope', function() {
@@ -505,6 +506,7 @@ describe('Chart.helpers.config', function() {
         resolver.sub.value = false;
         expect(options.sub.value).toBeFalse();
         expect(defaults.sub.value).toBeTrue();
+        expect(resolver.sub.value).toBeFalse();
       });
 
       it('should throw when setting a value and options is frozen', function() {