expandWildcardImports: necessary imports in method call expressions removed

Aperta
#2,833 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
52/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
java
Ambito
tooling

Direzione di ricerca

Inizia in lib/src/javaParser/java/com/diffplug/spotless/glue/javaparser/ExpandWildcardsFormatterFunc.java intorno alle righe 181-188 e riproduci l'esempio Java mostrato. Controlla la gestione dell'espansione dei wildcard per gli scope delle chiamate di metodo, quindi verifica che sia JSONObject sia JSON siano ancora importati e che il codice formattato venga compilato.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Issue

The new expandWildcardImports step removes necessary imports in method call expressions, e.g.:

import com.alibaba.fastjson.*;
public class Demo {
  public static void main() {
    final JSONObject jsonObject = new JSONObject();
    final Script inlineScript = JSON.toJSONString(jsonObject);
  }
}

becomes

import com.alibaba.fastjson.JSONObject;
public class Demo {
  public static void main() {
    final JSONObject jsonObject = new JSONObject();
    final Script inlineScript = JSON.toJSONString(jsonObject);
  }
}

The import import com.alibaba.fastjson.JSON is missing and the code fails to compile.

Java version

openjdk 25.0.1 2025-10-21
OpenJDK Runtime Environment (build 25.0.1+8-Ubuntu-125.10)
OpenJDK 64-Bit Server VM (build 25.0.1+8-Ubuntu-125.10, mixed mode, sharing)

Solution

Code like JSON.toJSONString(jsonObject) seems to be a method call expression:

https://github.com/diffplug/spotless/blob/8e776ec835b443b2c7d7e9e662aac268fa270050/lib/src/javaParser/java/com/diffplug/spotless/glue/javaparser/ExpandWildcardsFormatterFunc.java#L181-L188

The current code above only checks the method (toJSONString), but not the scope of the method. Adding this resolved the issue for me:

			n.getScope().ifPresent(s -> {
				ResolvedType type = n.getSymbolResolver().calculateType(n.getScope().get());
				if (type != null && type.isReference()) {
					matchTypeName(importMap, type.asReferenceType().getQualifiedName(), false);
				}
			});
Lingua principale
Java
Stelle
5.7k
Fork
560
Merge medio
1g 13h
PR unite (30g)
43

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di diffplug/spotless

Tutte le issue di diffplug/spotless

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.