refactor(core): Don't bother instantiating breadcrumb when type is in…

…valid
bugsnag · Nov 19, 2019 · b245c79 · b245c79
1 parent fa55906
commit b245c79
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/packages/core/client.js b/packages/core/client.js
@@ -135,10 +135,10 @@ class BugsnagClient {
     // if no message, discard
     if (!message) return
 
-    const crumb = new BugsnagBreadcrumb(message, metadata, type)
-
     // check the breadcrumb is the list of enabled types
-    if (!this.config.enabledBreadcrumbTypes || !includes(this.config.enabledBreadcrumbTypes, crumb.type)) return
+    if (!this.config.enabledBreadcrumbTypes || !includes(this.config.enabledBreadcrumbTypes, type)) return
+
+    const crumb = new BugsnagBreadcrumb(message, metadata, type)
 
     // push the valid crumb onto the queue and maintain the length
     this.breadcrumbs.push(crumb)