Update ktfmt version and replace duplicated formatting options logic …

…with new public API for it
diffplug · Jul 17, 2020 · d7ac565 · d7ac565
1 parent 148ccd2
commit d7ac565
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 14 deletions.
diff --git a/lib/src/main/java/com/diffplug/spotless/kotlin/KtfmtStep.java b/lib/src/main/java/com/diffplug/spotless/kotlin/KtfmtStep.java
@@ -26,33 +26,33 @@
 
 import com.diffplug.spotless.*;
 
-/** Wraps up [ktfmt](https://github.com/facebookincubator/ktfmt) as a FormatterStep. */
+/**
+ * Wraps up [ktfmt](https://github.com/facebookincubator/ktfmt) as a FormatterStep.
+ */
 public class KtfmtStep {
 	// prevent direct instantiation
 	private KtfmtStep() {}
 
-	private static final String DEFAULT_VERSION = "0.15";
+	private static final String DEFAULT_VERSION = "0.16";
 	static final String NAME = "ktfmt";
 	static final String PACKAGE = "com.facebook";
 	static final String MAVEN_COORDINATE = PACKAGE + ":ktfmt:";
 
 	/**
 	 * Used to allow dropbox style option through formatting options.
 	 *
-	 * @see <a href="https://github.com/facebookincubator/ktfmt/blob/bfd3f08059eace1562191d542d11c0f9dbd49332/core/src/main/java/com/facebook/ktfmt/Formatter.kt#L47-L73">ktfmt source</a>
+	 * @see <a href="https://github.com/facebookincubator/ktfmt/blob/38486b0fb2edcabeba5540fcb69c6f1fa336c331/core/src/main/java/com/facebook/ktfmt/Formatter.kt#L47-L80">ktfmt source</a>
 	 */
 	public enum Style {
 		DEFAULT, DROPBOX
 	}
 
-	private static final int MAX_WIDTH_LINE = 100;
-	private static final int BLOCK_INDENT = 4;
-	private static final int CONTINUATION_INDENT = 4;
+	static final String DROPBOX_STYLE_METHOD = "dropboxStyle";
 
 	/**
 	 * The <code>format</code> method is available in the link below.
 	 *
-	 * @see <a href="https://github.com/facebookincubator/ktfmt/blob/bfd3f08059eace1562191d542d11c0f9dbd49332/core/src/main/java/com/facebook/ktfmt/Formatter.kt#L75-L92">ktfmt source</a>
+	 * @see <a href="https://github.com/facebookincubator/ktfmt/blob/38486b0fb2edcabeba5540fcb69c6f1fa336c331/core/src/main/java/com/facebook/ktfmt/Formatter.kt#L82-L99">ktfmt source</a>
 	 */
 	static final String FORMATTER_METHOD = "format";
 
@@ -102,15 +102,10 @@ FormatterFunc createFormat() throws Exception {
 			return input -> {
 				try {
 					if (style == DROPBOX) {
-						// we are duplicating the result of this parsing logic from ktfmt 0.15
-						// https://github.com/facebookincubator/ktfmt/blob/59f7ad8d1fde08f3402a013571c9997316083ebf/core/src/main/java/com/facebook/ktfmt/ParsedArgs.kt#L37
-						// if the code above changes in a future version, we will need to change this code
 						Class<?> formattingOptionsClazz = classLoader.loadClass(pkg + ".ktfmt.FormattingOptions");
-						Object formattingOptions = formattingOptionsClazz.getConstructor(
-								int.class, int.class, int.class).newInstance(
-										MAX_WIDTH_LINE, BLOCK_INDENT, CONTINUATION_INDENT);
 						Method formatterMethod = formatterClazz.getMethod(FORMATTER_METHOD, formattingOptionsClazz,
 								String.class);
+						Object formattingOptions = getDropboxStyleFormattingOptions(classLoader);
 						return (String) formatterMethod.invoke(formatterClazz, formattingOptions, input);
 					} else {
 						Method formatterMethod = formatterClazz.getMethod(FORMATTER_METHOD, String.class);
@@ -121,5 +116,12 @@ FormatterFunc createFormat() throws Exception {
 				}
 			};
 		}
+
+		private Object getDropboxStyleFormattingOptions(ClassLoader classLoader) throws Exception {
+			Class<?> formattingOptionsCompanionClazz = classLoader.loadClass(pkg + ".ktfmt.FormattingOptions$Companion");
+			Object companion = formattingOptionsCompanionClazz.getConstructors()[0].newInstance((Object) null);
+			Method formattingOptionsMethod = formattingOptionsCompanionClazz.getDeclaredMethod(DROPBOX_STYLE_METHOD);
+			return formattingOptionsMethod.invoke(companion);
+		}
 	}
 }
diff --git a/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/KotlinGradleExtensionTest.java b/plugin-gradle/src/test/java/com/diffplug/gradle/spotless/KotlinGradleExtensionTest.java
@@ -147,7 +147,7 @@ public void integration_ktfmt_with_dropbox_style() throws IOException {
 				"repositories { mavenCentral() }",
 				"spotless {",
 				"    kotlinGradle {",
-				"        ktfmt('0.15').dropboxStyle()",
+				"        ktfmt().dropboxStyle()",
 				"    }",
 				"}");
 		setFile("configuration.gradle.kts").toResource("kotlin/ktfmt/dropboxstyle.dirty");