make tracing enablement check work without arguments

Author: lobsterkatie

packages/tracing/src/browser/request.ts

@@ -1,4 +1,3 @@
-import { getCurrentHub } from '@sentry/hub';
 import { addInstrumentationHandler, isInstanceOf, isMatchingPattern } from '@sentry/utils';
 
 import { Span } from '../span';
@@ -145,13 +144,7 @@ export function fetchCallback(
   shouldCreateSpan: (url: string) => boolean,
   spans: Record<string, Span>,
 ): void {
-  const currentClientOptions = getCurrentHub()
-    .getClient()
-    ?.getOptions();
-  if (
-    !(currentClientOptions && hasTracingEnabled(currentClientOptions)) ||
-    !(handlerData.fetchData && shouldCreateSpan(handlerData.fetchData.url))
-  ) {
+  if (!hasTracingEnabled() || !(handlerData.fetchData && shouldCreateSpan(handlerData.fetchData.url))) {
     return;
   }
 
@@ -219,13 +212,10 @@ export function xhrCallback(
   shouldCreateSpan: (url: string) => boolean,
   spans: Record<string, Span>,
 ): void {
-  const currentClientOptions = getCurrentHub()
-    .getClient()
-    ?.getOptions();
   if (
-    !(currentClientOptions && hasTracingEnabled(currentClientOptions)) ||
-    !(handlerData.xhr && handlerData.xhr.__sentry_xhr__ && shouldCreateSpan(handlerData.xhr.__sentry_xhr__.url)) ||
-    handlerData.xhr.__sentry_own_request__
+    !hasTracingEnabled() ||
+    handlerData.xhr?.__sentry_own_request__ ||
+    !(handlerData.xhr?.__sentry_xhr__ && shouldCreateSpan(handlerData.xhr.__sentry_xhr__.url))
   ) {
     return;
   }

packages/tracing/src/hubextensions.ts

@@ -43,7 +43,7 @@ function traceHeaders(this: Hub): { [key: string]: string } {
  */
 function sample<T extends Transaction>(transaction: T, options: Options, samplingContext: SamplingContext): T {
   // nothing to do if tracing is not enabled
-  if (!hasTracingEnabled(options)) {
+  if (!hasTracingEnabled()) {
     transaction.sampled = false;
     return transaction;
   }

packages/tracing/src/utils.ts

@@ -14,7 +14,14 @@ export const TRACEPARENT_REGEXP = new RegExp(
  *
  * Tracing is enabled when at least one of `tracesSampleRate` and `tracesSampler` is defined in the SDK config.
  */
-export function hasTracingEnabled(options: Options): boolean {
+export function hasTracingEnabled(
+  options: Options | undefined = getCurrentHub()
+    .getClient()
+    ?.getOptions(),
+): boolean {
+  if (!options) {
+    return false;
+  }
   return 'tracesSampleRate' in options || 'tracesSampler' in options;
 }