From 2d8b4fcd788efeca517768a8b5740e05eca4e890 Mon Sep 17 00:00:00 2001 From: Iku-turso Date: Thu, 23 Mar 2023 15:03:12 +0200 Subject: [PATCH] Remove more nesting Signed-off-by: Iku-turso Co-authored-by: Gabriel --- .../telemetry-decorator.injectable.ts | 44 ++++++++++--------- 1 file changed, 23 insertions(+), 21 deletions(-) diff --git a/packages/core/src/features/telemetry/renderer/telemetry-decorator.injectable.ts b/packages/core/src/features/telemetry/renderer/telemetry-decorator.injectable.ts index 362188b46c..4b69937367 100644 --- a/packages/core/src/features/telemetry/renderer/telemetry-decorator.injectable.ts +++ b/packages/core/src/features/telemetry/renderer/telemetry-decorator.injectable.ts @@ -50,29 +50,31 @@ const telemetryDecoratorInjectable = getInjectable({ const whiteListed = whiteListMap.get(currentContext.injectable.id); - if (whiteListed) { - let params; - - try { - params = whiteListed.getParams(...args); - } catch (e) { - params = { - error: - "Tried to produce params for telemetry, but getParams() threw an error", - }; - - logError( - `Tried to produce params for telemetry of "${currentContext.injectable.id}", but getParams() threw an error`, - e - ); - } - - emitTelemetry({ - action: currentContext.injectable.id, - params, - }); + if (!whiteListed) { + return instance(...args); } + let params; + + try { + params = whiteListed.getParams(...args); + } catch (e) { + params = { + error: + "Tried to produce params for telemetry, but getParams() threw an error", + }; + + logError( + `Tried to produce params for telemetry of "${currentContext.injectable.id}", but getParams() threw an error`, + e + ); + } + + emitTelemetry({ + action: currentContext.injectable.id, + params, + }); + return instance(...args); }; },