From afc5f1e8b23ccd17383e176106058f7b836faba7 Mon Sep 17 00:00:00 2001
From: overlookmotel <557937+overlookmotel@users.noreply.github.com>
Date: Wed, 11 Dec 2024 14:54:49 +0000
Subject: [PATCH] refactor(transformer/class-properties): de-deduplicate code
 (#7805)

Follow-on after #7798. Pure refactor. Include `transform_chain_element_recursively` call once instead of twice.
---
 .../oxc_transformer/src/es2022/class_properties/private.rs   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/crates/oxc_transformer/src/es2022/class_properties/private.rs b/crates/oxc_transformer/src/es2022/class_properties/private.rs
index 6f9c709d2287c..31872cba60b54 100644
--- a/crates/oxc_transformer/src/es2022/class_properties/private.rs
+++ b/crates/oxc_transformer/src/es2022/class_properties/private.rs
@@ -1159,8 +1159,9 @@ impl<'a, 'ctx> ClassProperties<'a, 'ctx> {
             return result;
         }
 
+        let result = self.transform_chain_element_recursively(callee, ctx);
         if !is_optional {
-            return self.transform_chain_element_recursively(callee, ctx);
+            return result;
         }
 
         // `o?.Foo.#self.getSelf?.()?.self.#m();`
@@ -1169,7 +1170,7 @@ impl<'a, 'ctx> ClassProperties<'a, 'ctx> {
         //                            then use it as a first argument of `getSelf` call.
         //
         // TODO(improve-on-babel): Consider remove this logic, because it seems no runtime behavior change.
-        let result = self.transform_chain_element_recursively(callee, ctx)?;
+        let result = result?;
         let object = callee.to_member_expression_mut().object_mut();
         let (assignment, context) = self.duplicate_object(ctx.ast.move_expression(object), ctx);
         *object = assignment;