Merge pull request #1096 from vil02/4901_make_java_classes_proper_uti…

…lities Make some Java classes proper utilities
ballerina-platform · Oct 23, 2023 · 88fb66b · 88fb66b
2 parents a1915c5 + 0081b53
commit 88fb66b
Show file tree

Hide file tree

Showing 8 changed files with 24 additions and 11 deletions.
diff --git a/ballerina/Ballerina.toml b/ballerina/Ballerina.toml
@@ -1,7 +1,7 @@
 [package]
 org = "ballerina"
 name = "io"
-version = "1.6.0"
+version = "1.6.1"
 authors = ["Ballerina"]
 keywords = ["io", "json", "xml", "csv", "file"]
 repository = "https://github.com/ballerina-platform/module-ballerina-io"
@@ -15,11 +15,11 @@ graalvmCompatible = true
 [[platform.java17.dependency]]
 groupId = "io.ballerina.stdlib"
 artifactId = "io-native"
-path = "../native/build/libs/io-native-1.6.0.jar"
-version = "1.6.0"
+path = "../native/build/libs/io-native-1.6.1-SNAPSHOT.jar"
+version = "1.6.1"
 
 
 [[platform.java17.dependency]]
-path = "../test-utils/build/libs/io-test-utils-1.6.0.jar"
+path = "../test-utils/build/libs/io-test-utils-1.6.1-SNAPSHOT.jar"
 scope = "testOnly"
-version = "1.6.0"
+version = "1.6.1-SNAPSHOT"
diff --git a/ballerina/Dependencies.toml b/ballerina/Dependencies.toml
@@ -10,7 +10,7 @@ distribution-version = "2201.8.0"
 [[package]]
 org = "ballerina"
 name = "io"
-version = "1.6.0"
+version = "1.6.1"
 dependencies = [
 	{org = "ballerina", name = "jballerina.java"},
 	{org = "ballerina", name = "lang.float"},

diff --git a/changelog.md b/changelog.md
@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - [Fixed the issue related to maintaining order in writing CSV records](https://github.com/ballerina-platform/ballerina-standard-library/issues/3399)
 - [Fixed the fileReadCsv and fileReadCsvAsStream APIs to maintain the order while reading CSV records](https://github.com/ballerina-platform/ballerina-standard-library/issues/3780)
 - [Fixed the issue related to expected type in CSV data mapping](https://github.com/ballerina-platform/ballerina-standard-library/issues/3669)
+- [Make some of the Java classes proper utility classes](https://github.com/ballerina-platform/ballerina-standard-library/issues/4901)
 
 ## [1.3.1] - 2022-11-29
 ### Changed

diff --git a/native/src/main/java/io/ballerina/stdlib/io/nativeimpl/CsvChannelUtils.java b/native/src/main/java/io/ballerina/stdlib/io/nativeimpl/CsvChannelUtils.java
@@ -52,7 +52,9 @@
  *
  * * @since 1.3.0
  */
-public class CsvChannelUtils {
+public final class CsvChannelUtils {
+
+    private CsvChannelUtils() {}
 
     private static final BString FIELD_SEPERATOR = StringUtils.fromString(",");
     private static final BString ROW_SEPERATOR = StringUtils.fromString("");

diff --git a/native/src/main/java/io/ballerina/stdlib/io/nativeimpl/WriteXml.java b/native/src/main/java/io/ballerina/stdlib/io/nativeimpl/WriteXml.java
@@ -30,7 +30,9 @@
  *
  * @since ballerina-0.970.0-alpha3
  */
-public class WriteXml {
+public final class WriteXml {
+
+    private WriteXml() {}
 
     public static Object writeXml(BObject characterChannelObj, BXml content) {
         try {

diff --git a/native/src/main/java/io/ballerina/stdlib/io/utils/IOConstants.java b/native/src/main/java/io/ballerina/stdlib/io/utils/IOConstants.java
@@ -21,7 +21,10 @@
 /**
  * I/O related constants.
  */
-public class IOConstants {
+public final class IOConstants {
+
+    private IOConstants() {}
+
     /**
      * Name of the ByteChannel which will be represented through the native struct.
      *

diff --git a/native/src/main/java/io/ballerina/stdlib/io/utils/PropertyUtils.java b/native/src/main/java/io/ballerina/stdlib/io/utils/PropertyUtils.java
@@ -42,7 +42,10 @@
 /**
  * The utility class used to execute read/write operations to a property file.
  */
-public class PropertyUtils {
+public final class PropertyUtils {
+
+    private PropertyUtils() {}
+
     private static final Logger log = LoggerFactory.getLogger(PropertyUtils.class);
     private static final MapType mapType = TypeCreator.createMapType(PredefinedTypes.TYPE_STRING);
     private static Map<String, Properties> propertiesMap = new HashMap<>();

diff --git a/native/src/test/java/io/ballerina/stdlib/io/util/TestUtil.java b/native/src/test/java/io/ballerina/stdlib/io/util/TestUtil.java
@@ -34,7 +34,9 @@
 /**
  * Contains the util functions related to I/O test cases.
  */
-public class TestUtil {
+public final class TestUtil {
+
+    private TestUtil() {}
 
     /**
      * Opens a channel from a specified file.