From b65f39be2b224cc6a469eecefb353ad31978101a Mon Sep 17 00:00:00 2001 From: Thevakumar-Luheerathan Date: Wed, 14 Feb 2024 16:25:48 +0530 Subject: [PATCH] Fix manifest warning --- .../projects/internal/ManifestBuilder.java | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/compiler/ballerina-lang/src/main/java/io/ballerina/projects/internal/ManifestBuilder.java b/compiler/ballerina-lang/src/main/java/io/ballerina/projects/internal/ManifestBuilder.java index e5f56405621d..2c8388c8052c 100644 --- a/compiler/ballerina-lang/src/main/java/io/ballerina/projects/internal/ManifestBuilder.java +++ b/compiler/ballerina-lang/src/main/java/io/ballerina/projects/internal/ManifestBuilder.java @@ -88,6 +88,8 @@ public class ManifestBuilder { private static final String PACKAGE = "package"; private static final String VERSION = "version"; + public static final String ORG = "org"; + public static final String NAME = "name"; private static final String LICENSE = "license"; private static final String AUTHORS = "authors"; private static final String REPOSITORY = "repository"; @@ -331,24 +333,24 @@ private PackageDescriptor getPackageDescriptor(TomlTableNode tomlTableNode) { TomlTableNode pkgNode = (TomlTableNode) topLevelPkgNode; - org = getStringValueFromTomlTableNode(pkgNode, "org"); - if (org == null) { + org = getStringValueFromTomlTableNode(pkgNode, ORG, ""); + if (pkgNode.entries().get(ORG) == null) { org = defaultOrg().value(); reportDiagnostic(pkgNode, "missing key 'org' in table '[package]' in 'Ballerina.toml'. " + "Defaulting to 'org = \"" + org + "\"'", ProjectDiagnosticErrorCode.MISSING_PKG_INFO_IN_BALLERINA_TOML.diagnosticId(), DiagnosticSeverity.WARNING); } - name = getStringValueFromTomlTableNode(pkgNode, "name"); - if (name == null) { + name = getStringValueFromTomlTableNode(pkgNode, NAME, ""); + if (pkgNode.entries().get(NAME) == null) { name = defaultName(this.projectPath).value(); reportDiagnostic(pkgNode, "missing key 'name' in table '[package]' in 'Ballerina.toml'. " + "Defaulting to 'name = \"" + name + "\"'", ProjectDiagnosticErrorCode.MISSING_PKG_INFO_IN_BALLERINA_TOML.diagnosticId(), DiagnosticSeverity.WARNING); } - version = getStringValueFromTomlTableNode(pkgNode, VERSION); - if (version == null) { + version = getStringValueFromTomlTableNode(pkgNode, VERSION, ""); + if (pkgNode.entries().get(VERSION) == null) { version = defaultVersion().value().toString(); reportDiagnostic(pkgNode, "missing key 'version' in table '[package]' in 'Ballerina.toml'. " + "Defaulting to 'version = \"" + version + "\"'", @@ -570,8 +572,8 @@ private List getLocalRepoDependencies() { TomlTableArrayNode dependencyTableArray = (TomlTableArrayNode) dependencyEntries; for (TomlTableNode dependencyNode : dependencyTableArray.children()) { - String name = getStringValueFromDependencyNode(dependencyNode, "name"); - String org = getStringValueFromDependencyNode(dependencyNode, "org"); + String name = getStringValueFromDependencyNode(dependencyNode, NAME); + String org = getStringValueFromDependencyNode(dependencyNode, ORG); String version = getStringValueFromDependencyNode(dependencyNode, VERSION); String repository = getStringValueFromDependencyNode(dependencyNode, REPOSITORY);