Merge pull request #1181 from eclipse-tractusx/refactor/codeql-sugges…

…tions refactor: implement CodeQL suggestions
eclipse-tractusx · Jan 28, 2025 · f783260 · f783260
2 parents 14a9a3d + df5fa04
commit f783260
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/...in/kotlin/org/eclipse/tractusx/bpdm/test/testdata/pool/BusinessPartnerNonVerboseValues.kt b/...in/kotlin/org/eclipse/tractusx/bpdm/test/testdata/pool/BusinessPartnerNonVerboseValues.kt
@@ -183,7 +183,7 @@ object BusinessPartnerNonVerboseValues {
         country = BusinessPartnerVerboseValues.address1.country,
         postalCode = BusinessPartnerVerboseValues.address1.postalCode,
         city = BusinessPartnerVerboseValues.address1.city,
-        administrativeAreaLevel1 = BusinessPartnerVerboseValues.address1.administrativeAreaLevel1?.toString(),
+        administrativeAreaLevel1 = BusinessPartnerVerboseValues.address1.administrativeAreaLevel1,
         administrativeAreaLevel2 = BusinessPartnerVerboseValues.address1.administrativeAreaLevel2,
         administrativeAreaLevel3 = BusinessPartnerVerboseValues.address1.administrativeAreaLevel3,
         district = BusinessPartnerVerboseValues.address1.district,
@@ -201,7 +201,7 @@ object BusinessPartnerNonVerboseValues {
         country = BusinessPartnerVerboseValues.address2.country,
         postalCode = BusinessPartnerVerboseValues.address2.postalCode,
         city = BusinessPartnerVerboseValues.address2.city,
-        administrativeAreaLevel1 = BusinessPartnerVerboseValues.address2.administrativeAreaLevel1?.toString(),
+        administrativeAreaLevel1 = BusinessPartnerVerboseValues.address2.administrativeAreaLevel1,
         administrativeAreaLevel2 = BusinessPartnerVerboseValues.address2.administrativeAreaLevel2,
         administrativeAreaLevel3 = BusinessPartnerVerboseValues.address2.administrativeAreaLevel3,
         district = BusinessPartnerVerboseValues.address2.district,
@@ -219,7 +219,7 @@ object BusinessPartnerNonVerboseValues {
         country = BusinessPartnerVerboseValues.address3.country,
         postalCode = BusinessPartnerVerboseValues.address3.postalCode,
         city = BusinessPartnerVerboseValues.address3.city,
-        administrativeAreaLevel1 = BusinessPartnerVerboseValues.address3.administrativeAreaLevel1?.toString(),
+        administrativeAreaLevel1 = BusinessPartnerVerboseValues.address3.administrativeAreaLevel1,
         administrativeAreaLevel2 = BusinessPartnerVerboseValues.address3.administrativeAreaLevel2,
         administrativeAreaLevel3 = BusinessPartnerVerboseValues.address3.administrativeAreaLevel3,
         district = BusinessPartnerVerboseValues.address3.district,

diff --git a/...ol/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/BusinessPartnerSearchService.kt b/...ol/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/service/BusinessPartnerSearchService.kt
@@ -106,7 +106,7 @@ class BusinessPartnerSearchService(
             totalPages = legalEntityPage.totalPages,
             page = paginationRequest.page,
             content = legalEntityPage.content.mapIndexed { index, legalEntity ->
-                val score = legalEntityPage.totalElements - paginationRequest.page * paginationRequest.size - index
+                val score = legalEntityPage.totalElements - paginationRequest.page.toLong() * paginationRequest.size.toLong() - index
                 Pair(score.toFloat(), legalEntity)
             },
             contentSize = legalEntityPage.content.size
@@ -163,7 +163,7 @@ class BusinessPartnerSearchService(
             totalPages = addressPage.totalPages,
             page = paginationRequest.page,
             content = addressPage.content.mapIndexed { index, address ->
-                val score = addressPage.totalElements - paginationRequest.page * paginationRequest.size - index
+                val score = addressPage.totalElements - paginationRequest.page.toLong() * paginationRequest.size.toLong() - index
                 Pair(score.toFloat(), address)
             },
             contentSize = addressPage.content.size

diff --git a/...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/util/CreateSQLScriptToInsertRegions.kt b/...ool/src/main/kotlin/org/eclipse/tractusx/bpdm/pool/util/CreateSQLScriptToInsertRegions.kt
@@ -32,7 +32,7 @@ class CreateSQLScriptToInsertRegions {
     fun readCsv(inputStream: InputStream): List<RegionDb> =
         CSVFormat.Builder.create(CSVFormat.DEFAULT).apply {
             setIgnoreSurroundingSpaces(true)
-        }.build().parse(inputStream.reader())
+        }.get().parse(inputStream.reader())
             .drop(1) // Dropping the header
             .map {
                 RegionDb(