From 3617ca3614fd77b1f440f67d765c9f07b7e570a0 Mon Sep 17 00:00:00 2001
From: tsantalis <tsantalis@gmail.com>
Date: Sun, 19 Jan 2025 21:04:14 -0500
Subject: [PATCH] Enable detection of Extract Method in commit

https://github.com/spring-projects/spring-boot/commit/793aca60d2d6ad4ff1618bc19672038cd4755120
spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/main/java/org/springframework/boot/gradle/tasks/bundling/BootJar.java
spring-boot-project/spring-boot-tools/spring-boot-gradle-plugin/src/main/java/org/springframework/boot/gradle/tasks/bundling/BootWar.java
---
 .../TernaryOperatorExpression.java            | 20 +++++++++++++++----
 .../replacement/Replacement.java              |  1 +
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/src/main/java/gr/uom/java/xmi/decomposition/TernaryOperatorExpression.java b/src/main/java/gr/uom/java/xmi/decomposition/TernaryOperatorExpression.java
index 8d9e0cf446..c198f6fa11 100644
--- a/src/main/java/gr/uom/java/xmi/decomposition/TernaryOperatorExpression.java
+++ b/src/main/java/gr/uom/java/xmi/decomposition/TernaryOperatorExpression.java
@@ -50,12 +50,15 @@ public String getExpression() {
 	}
 
 	public Replacement makeReplacementWithTernaryOnTheRight(String statement, Map<String, String> parameterToArgumentMap) {
-		if(getElseExpression().getString().equals(statement)) {
+		if(getElseExpression().getString().equals(statement) || getElseExpression().getString().equals(statement + ".get()")) {
 			return new Replacement(statement, getElseExpression().getString(), ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_ELSE);
 		}
-		if(getThenExpression().getString().equals(statement)) {
+		if(getThenExpression().getString().equals(statement) || getThenExpression().getString().equals(statement + ".get()")) {
 			return new Replacement(statement, getThenExpression().getString(), ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_THEN);
 		}
+		if(getCondition().getString().equals(statement) || getCondition().getString().equals(statement + ".get()")) {
+			return new Replacement(statement, getCondition().getString(), ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_CONDITION);
+		}
 		String temp = new String(statement);
 		for(String key : parameterToArgumentMap.keySet()) {
 			if(!key.equals(parameterToArgumentMap.get(key))) {
@@ -69,17 +72,23 @@ public Replacement makeReplacementWithTernaryOnTheRight(String statement, Map<St
 			if(getThenExpression().getString().equals(temp)) {
 				return new Replacement(statement, getThenExpression().getString(), ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_THEN);
 			}
+			if(getCondition().getString().equals(temp)) {
+				return new Replacement(statement, getCondition().getString(), ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_CONDITION);
+			}
 		}
 		return null;
 	}
 
 	public Replacement makeReplacementWithTernaryOnTheLeft(String statement, Map<String, String> parameterToArgumentMap) {
-		if(getElseExpression().getString().equals(statement)) {
+		if(getElseExpression().getString().equals(statement) || getElseExpression().getString().equals(statement + ".get()")) {
 			return new Replacement(getElseExpression().getString(), statement, ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_ELSE);
 		}
-		if(getThenExpression().getString().equals(statement)) {
+		if(getThenExpression().getString().equals(statement) || getThenExpression().getString().equals(statement + ".get()")) {
 			return new Replacement(getThenExpression().getString(), statement, ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_THEN);
 		}
+		if(getCondition().getString().equals(statement) || getCondition().getString().equals(statement + ".get()")) {
+			return new Replacement(getCondition().getString(), statement, ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_CONDITION);
+		}
 		String temp = new String(statement);
 		for(String key : parameterToArgumentMap.keySet()) {
 			if(!key.equals(parameterToArgumentMap.get(key))) {
@@ -93,6 +102,9 @@ public Replacement makeReplacementWithTernaryOnTheLeft(String statement, Map<Str
 			if(getThenExpression().getString().equals(temp)) {
 				return new Replacement(getThenExpression().getString(), statement, ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_THEN);
 			}
+			if(getCondition().getString().equals(temp)) {
+				return new Replacement(getCondition().getString(), statement, ReplacementType.EXPRESSION_REPLACED_WITH_TERNARY_CONDITION);
+			}
 		}
 		return null;
 	}
diff --git a/src/main/java/gr/uom/java/xmi/decomposition/replacement/Replacement.java b/src/main/java/gr/uom/java/xmi/decomposition/replacement/Replacement.java
index 42c4ebd068..c1814f560e 100644
--- a/src/main/java/gr/uom/java/xmi/decomposition/replacement/Replacement.java
+++ b/src/main/java/gr/uom/java/xmi/decomposition/replacement/Replacement.java
@@ -141,6 +141,7 @@ public enum ReplacementType {
 		ARRAY_INITIALIZER_REPLACED_WITH_METHOD_INVOCATION_ARGUMENTS,
 		EXPRESSION_REPLACED_WITH_TERNARY_ELSE,
 		EXPRESSION_REPLACED_WITH_TERNARY_THEN,
+		EXPRESSION_REPLACED_WITH_TERNARY_CONDITION,
 		INFIX_EXPRESSION,
 		COMPOSITE,
 		CONCATENATION, CONDITIONAL,