rasti 4.2.0 → 4.2.1

package/README.md

@@ -1,7 +1,7 @@
 <p align="center">
     <picture>
-        <source media="(prefers-color-scheme: dark)" srcset="https://cdn.jsdelivr.net/gh/8tentaculos/rasti@v4.2.0/docs/logo-dark.svg">
-        <img alt="Rasti.js" src="https://cdn.jsdelivr.net/gh/8tentaculos/rasti@v4.2.0/docs/logo.svg" height="120">
+        <source media="(prefers-color-scheme: dark)" srcset="https://cdn.jsdelivr.net/gh/8tentaculos/rasti@v4.2.1/docs/logo-dark.svg">
+        <img alt="Rasti.js" src="https://cdn.jsdelivr.net/gh/8tentaculos/rasti@v4.2.1/docs/logo.svg" height="120">
     </picture>
 </p>
 

package/dist/rasti.js

@@ -755,19 +755,28 @@
     Model.attributePrefix = '';
 
     /**
-     * Define `key` on `target` as an own data property. Unlike an assignment, it does not go
-     * through an accessor declared on the prototype, so it shadows a getter with no setter
-     * instead of throwing.
-     * @param {object} target Object to define the property on.
+     * Assign an option to `target`. A getter with no setter on the prototype would make the
+     * assignment throw, so in that case `key` is defined as an own data property, which
+     * shadows the getter. Any other accessor receives the value through its setter.
+     * @param {object} target Object to assign the option to.
      * @param {string} key Property name.
      * @param {any} value Property value.
      * @return {object} The target object.
      * @module
      * @private
      */
-    const defineOwn = (target, key, value) => Object.defineProperty(target, key, {
-        value, writable : true, enumerable : true, configurable : true
-    });
+    const assignOption = (target, key, value) => {
+        try {
+            target[key] = value;
+        } catch (error) {
+            // Assigning to a getter with no setter throws a TypeError; anything else propagates.
+            if (!(error instanceof TypeError)) throw error;
+            Object.defineProperty(target, key, {
+                value, writable : true, enumerable : true, configurable : true
+            });
+        }
+        return target;
+    };
 
     /*
      * These option keys will be extended on the view instance.
@@ -838,11 +847,11 @@
             // Mutable array to store handlers to be called on destroy.
             this.destroyQueue = [];
             this.viewOptions = [];
-            // Extend "this" with options, as own properties so an option overrides
-            // a getter declared by a subclass instead of being assigned through it.
+            // Extend "this" with options. An option overrides a getter declared by a
+            // subclass even when the getter has no setter.
             viewOptions.forEach(key => {
                 if (key in options) {
-                    defineOwn(this, key, options[key]);
+                    assignOption(this, key, options[key]);
                     this.viewOptions.push(key);
                 }
             });
@@ -2424,11 +2433,11 @@
         constructor(options = {}) {
             super(...arguments);
             this.componentOptions = [];
-            // Extend "this" with options, as own properties so an option overrides
-            // a getter declared by a subclass instead of being assigned through it.
+            // Extend "this" with options. An option overrides a getter declared by a
+            // subclass even when the getter has no setter.
             componentOptions.forEach(key => {
                 if (key in options) {
-                    defineOwn(this, key, options[key]);
+                    assignOption(this, key, options[key]);
                     this.componentOptions.push(key);
                 }
             });