fix: Send raw error when vm is undefined in Vue plugin (#1118)

Author: kamilogorek

plugins/vue.js

@@ -22,22 +22,22 @@ function vuePlugin(Raven, Vue) {
 
   var _oldOnError = Vue.config.errorHandler;
   Vue.config.errorHandler = function VueErrorHandler(error, vm, info) {
+    var metaData = {};
+
+    // vm and lifecycleHook are not always available
     if (Object.prototype.toString.call(vm) === '[object Object]') {
-      var metaData = {
-        componentName: formatComponentName(vm),
-        propsData: vm.$options.propsData
-      };
-
-      // lifecycleHook is not always available
-      if (typeof info !== 'undefined') {
-        metaData.lifecycleHook = info;
-      }
-
-      Raven.captureException(error, {
-        extra: metaData
-      });
+      metaData.componentName = formatComponentName(vm);
+      metaData.propsData = vm.$options.propsData;
+    }
+
+    if (typeof info !== 'undefined') {
+      metaData.lifecycleHook = info;
     }
 
+    Raven.captureException(error, {
+      extra: metaData
+    });
+
     if (typeof _oldOnError === 'function') {
       _oldOnError.call(this, error, vm, info);
     }

test/plugins/vue.test.js

@@ -84,5 +84,11 @@ describe('Vue plugin', function() {
         }.bind(this)
       );
     });
+
+    it('should still send a raw error when `vm` is not defined', function() {
+      vuePlugin(Raven, this.MockVue);
+      this.MockVue.config.errorHandler(new Error('foo'));
+      assert.isTrue(Raven.captureException.calledOnce);
+    });
   });
 });