fix: check reflectionIsSupported in decorator runtime (#350)

vitalishapovalov · ktsn · commit 2fc6ab55f0c6 · 2019-11-11T23:38:31.000+08:00
* check reflectionIsSupported in decorator runtime

* fix lint
diff --git a/src/component.ts b/src/component.ts
@@ -83,7 +83,7 @@ export function componentFactory (
 
   forwardStaticMembers(Extended, Component, Super)
 
-  if (reflectionIsSupported) {
+  if (reflectionIsSupported()) {
     copyReflectionMetadata(Extended, Component)
   }
 
diff --git a/src/reflect.ts b/src/reflect.ts
@@ -4,7 +4,9 @@ import { VueClass } from './declarations'
 // The rational behind the verbose Reflect-feature check below is the fact that there are polyfills
 // which add an implementation for Reflect.defineMetadata but not for Reflect.getOwnMetadataKeys.
 // Without this check consumers will encounter hard to track down runtime errors.
-export const reflectionIsSupported = typeof Reflect !== 'undefined' && Reflect.defineMetadata && Reflect.getOwnMetadataKeys
+export function reflectionIsSupported () {
+  return typeof Reflect !== 'undefined' && Reflect.defineMetadata && Reflect.getOwnMetadataKeys
+}
 
 export function copyReflectionMetadata (
   to: VueConstructor,

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ export function componentFactory (`
`83`	`83`
`84`	`84`	`forwardStaticMembers(Extended, Component, Super)`
`85`	`85`
`86`		`- if (reflectionIsSupported) {`
	`86`	`+ if (reflectionIsSupported()) {`
`87`	`87`	`copyReflectionMetadata(Extended, Component)`
`88`	`88`	`}`
`89`	`89`