Skip to content

Commit fff06e0

Browse files
fix: centralize Java version checks on ProjectNode creation
1 parent 553bd97 commit fff06e0

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

src/promotionProvider.ts

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,12 +23,10 @@ export class PromotionProvider {
2323

2424
private haveAlreadyPrompted = false;
2525

26-
public checkJavaVersion(data: INodeData[]) {
27-
for (const i of data) {
28-
const javaVersion = i.metaData?.MaxSourceVersion;
29-
if (javaVersion && javaVersion < EARLIEST_JAVA_VERSION_NOT_TO_PROMPT) {
30-
this.triggerJavaVersionUpgrade(javaVersion);
31-
}
26+
public checkJavaVersion(data: INodeData) {
27+
const javaVersion = data.metaData?.MaxSourceVersion;
28+
if (javaVersion && javaVersion < EARLIEST_JAVA_VERSION_NOT_TO_PROMPT) {
29+
this.triggerJavaVersionUpgrade(javaVersion);
3230
}
3331
}
3432

src/views/dependencyDataProvider.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -200,8 +200,8 @@ export class DependencyDataProvider implements TreeDataProvider<ExplorerNode> {
200200
this._rootItems = rootItems;
201201
} else {
202202
const result: INodeData[] = await Jdtls.getProjects(folders[0].uri.toString());
203-
PromotionProvider.getInstance().checkJavaVersion(result);
204203
result.forEach((project) => {
204+
PromotionProvider.getInstance().checkJavaVersion(project);
205205
rootItems.push(new ProjectNode(project, undefined));
206206
});
207207
this._rootItems = rootItems;

src/views/nodeFactory.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ import { PackageRootNode } from "./packageRootNode";
1717
import { ProjectNode } from "./projectNode";
1818
import { WorkspaceNode } from "./workspaceNode";
1919
import { HierarchicalPackageNodeData } from "../java/hierarchicalPackageNodeData";
20+
import { PromotionProvider } from "../promotionProvider";
2021

2122
export class NodeFactory {
2223
/**
@@ -33,6 +34,7 @@ export class NodeFactory {
3334
case NodeKind.Workspace:
3435
return new WorkspaceNode(nodeData, parent);
3536
case NodeKind.Project:
37+
PromotionProvider.getInstance().checkJavaVersion(nodeData);
3638
return new ProjectNode(nodeData, parent);
3739
case NodeKind.Container:
3840
if (!parent || !project) {

0 commit comments

Comments
 (0)