From 85a4ccec9321502a82ef7e1219123b4d6a667cd4 Mon Sep 17 00:00:00 2001
From: Jente Sondervorst <jentesondervorst@gmail.com>
Date: Wed, 5 Jun 2024 14:44:35 +0200
Subject: [PATCH] Added classifier support to DependencyUseStringNotation
 recipe (#4226)

Co-authored-by: Jente Sondervorst <jente.sondervorst@colruytgroup.com>
---
 .../gradle/DependencyUseStringNotation.java   |  4 +++
 .../DependencyUseStringNotationTest.java      | 28 +++++++++++++++++++
 2 files changed, 32 insertions(+)

diff --git a/rewrite-gradle/src/main/java/org/openrewrite/gradle/DependencyUseStringNotation.java b/rewrite-gradle/src/main/java/org/openrewrite/gradle/DependencyUseStringNotation.java
index 4197fd31081..f7a3565efa6 100644
--- a/rewrite-gradle/src/main/java/org/openrewrite/gradle/DependencyUseStringNotation.java
+++ b/rewrite-gradle/src/main/java/org/openrewrite/gradle/DependencyUseStringNotation.java
@@ -136,6 +136,10 @@ private J.Literal toLiteral(Space prefix, Markers markers, Map<String, Expressio
                     String version = coerceToStringNotation(mapNotation.get("version"));
                     if (version != null) {
                         stringNotation += ":" + version;
+                        String classifier = coerceToStringNotation(mapNotation.get("classifier"));
+                        if (classifier != null) {
+                            stringNotation += ":" + classifier;
+                        }
                     }
 
                     return new J.Literal(randomId(), prefix, markers, stringNotation, "\"" + stringNotation + "\"", Collections.emptyList(), JavaType.Primitive.String);
diff --git a/rewrite-gradle/src/test/java/org/openrewrite/gradle/DependencyUseStringNotationTest.java b/rewrite-gradle/src/test/java/org/openrewrite/gradle/DependencyUseStringNotationTest.java
index cd3bebd7ba2..f82f9630ab7 100644
--- a/rewrite-gradle/src/test/java/org/openrewrite/gradle/DependencyUseStringNotationTest.java
+++ b/rewrite-gradle/src/test/java/org/openrewrite/gradle/DependencyUseStringNotationTest.java
@@ -58,6 +58,34 @@ void basicMap() {
         );
     }
 
+    @Test
+    void withClassifier() {
+        rewriteRun(
+          buildGradle(
+            """
+              plugins {
+                  id 'java-library'
+              }
+              
+              dependencies {
+                  api(group: 'org.openrewrite', name: 'rewrite-core', version: 'latest.release', classifier: 'sources')
+                  implementation group: 'group', name: 'artifact', version: 'version', classifier: 'sources'
+              }
+              """,
+            """
+              plugins {
+                  id 'java-library'
+              }
+              
+              dependencies {
+                  api("org.openrewrite:rewrite-core:latest.release:sources")
+                  implementation "group:artifact:version:sources"
+              }
+              """
+          )
+        );
+    }
+
     @Test
     void basicMapLiteral() {
         rewriteRun(