Merge pull request #198 from dr0p/bugfix/issue-191-cannot-read-enumer…

…able-from-undefined Bugfix (issue-191): Add a guard for missing property descriptor when inheriting enumerable
mobxjs · Jun 4, 2019 · e002bde · e002bde
2 parents d90d045 + 1535f52
commit e002bde
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/create-view-model.ts b/src/create-view-model.ts
@@ -58,10 +58,11 @@ export class ViewModel<T> implements IViewModel<T> {
                 this.localComputedValues.set(key, computed(derivation.bind(this)))
             }
 
-            const { enumerable } = Object.getOwnPropertyDescriptor(model, key)
+            const descriptor = Object.getOwnPropertyDescriptor(model, key);
+            const additionalDescriptor = descriptor ? { enumerable: descriptor.enumerable } : {};
 
             Object.defineProperty(this, key, {
-                enumerable,
+                ...additionalDescriptor,
                 configurable: true,
                 get: () => {
                     if (isComputedProp(model, key)) return this.localComputedValues.get(key).get()