squid:UselessImportCheck does not consider static imports used in class annotations

[renoth (Johannes Renoth)]

This code gives a warning that the rule unnecessary import is violated, the import DEFAULT_ALIAS is necessary for compilation however.

package com.sonarqube;

import static Entity.DEFAULT_ALIAS;

[…]

@Entity
@DefaultAlias(DEFAULT_ALIAS)
public class Entity {

public static final String DEFAULT_ALIAS = "alias";

[…]
}

[Quentin (Quentin Jaquier)]

Hello @renoth,

Thanks for the feedback!
This issue is similar to the one reported here:

squid:UselessImportCheck false positive for constants used in @NamedQuery annotation

The following code: import static SomeEntity.FIND_BY_NAME; import javax.persistence.Entity; import javax.persistence.NamedQuery; @NamedQuery( name = FIND_BY_NAME, query = "select se from SomeEntity where name = :name" ) @Entity public class SomeEntity { public static final String FIND_BY_NAME = "SomeEntity.findByName"; private String name; } triggers squid:UselessImportCheck rule for import static SomeEntity.FIND_BY_NAME although the import is used in @NamedQuery annotation. SonarQu…

There is now a ticket to track it!

Best,
Quentin