Skip to content

ANALYZERS-10 Add jdkHome to ModuleMetadata #5360

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
leonardo-pilastri-sonarsource wants to merge 1 commit into from
Closed

ANALYZERS-10 Add jdkHome to ModuleMetadata #5360

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions java-frontend/src/main/java/org/sonar/java/DefaultModuleMetadata.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@
import javax.annotation.CheckForNull;
import org.sonar.api.batch.bootstrap.ProjectDefinition;
import org.sonar.api.config.Configuration;
import org.sonar.java.classpath.ClasspathProperties;
import org.sonar.java.model.JavaVersionImpl;
import org.sonar.plugins.java.api.JavaVersion;
import org.sonar.plugins.java.api.internal.ModuleMetadata;
Expand All @@ -30,18 +31,25 @@ public class DefaultModuleMetadata implements ModuleMetadata {
private final JavaVersion javaVersion;
private final ProjectDefinition projectDefinition;
private final boolean ignoreUnnamedModuleForSplitPackage;
private final String jdkHome;

public DefaultModuleMetadata(ProjectDefinition projectDefinition, Configuration configuration) {
this.javaVersion = JavaVersionImpl.readFromConfiguration(configuration);
this.projectDefinition = projectDefinition;
this.ignoreUnnamedModuleForSplitPackage = configuration.getBoolean(SONAR_IGNORE_UNNAMED_MODULE_FOR_SPLIT_PACKAGE).orElse(false);
this.jdkHome = configuration.get(ClasspathProperties.SONAR_JAVA_JDK_HOME).orElse("");
}

@Override
public JavaVersion javaVersion() {
return javaVersion;
}

@Override
public String jdkHome() {
return jdkHome;
}

@Override
public String moduleKey() {
var root = getRootProject();
Expand Down
5 changes: 5 additions & 0 deletions java-frontend/src/main/java/org/sonar/plugins/java/api/internal/ModuleMetadata.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,11 @@ public interface ModuleMetadata {
*/
JavaVersion javaVersion();

/**
* Returns the JDK home path of the module being analyzed.
*/
String jdkHome();

/**
* Returns the module key of the module being analyzed.
*/
Expand Down