From 8771cfafcb3888b7403653cacaf4934aeb9f3196 Mon Sep 17 00:00:00 2001
From: Tomi Karlstedt <tomi.karlstedt@reaktor.com>
Date: Mon, 16 Dec 2024 13:56:01 +0200
Subject: [PATCH] OPHYK-395 remove apache commons-lang use

---
 .../kayttooikeus/repositories/criteria/AnomusCriteria.java   | 5 ++---
 .../vm/sade/kayttooikeus/service/impl/KutsuServiceImpl.java  | 3 +--
 .../fi/vm/sade/kayttooikeus/service/CryptoServiceTest.java   | 2 +-
 3 files changed, 4 insertions(+), 6 deletions(-)

diff --git a/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/repositories/criteria/AnomusCriteria.java b/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/repositories/criteria/AnomusCriteria.java
index 1880e8b9a..832d2ab40 100644
--- a/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/repositories/criteria/AnomusCriteria.java
+++ b/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/repositories/criteria/AnomusCriteria.java
@@ -13,7 +13,6 @@
 import fi.vm.sade.kayttooikeus.service.external.OrganisaatioClient;
 import fi.vm.sade.kayttooikeus.util.OrganisaatioMyontoPredicate;
 import lombok.*;
-import org.apache.commons.lang.BooleanUtils;
 import org.springframework.util.CollectionUtils;
 import org.springframework.util.StringUtils;
 
@@ -68,7 +67,7 @@ public AnomusCriteriaFunction<QAnomus, QKayttoOikeusRyhma, QHaettuKayttoOikeusRy
 
             builder = this.condition(qAnomus, builder, this.getInSameOrganisationPredicate(organisaatioClient, qAnomus));
 
-            if (BooleanUtils.isTrue(this.adminView)) {
+            if (Boolean.TRUE.equals(this.adminView)) {
                 QKayttoOikeusRyhma qAdminViewKor = new QKayttoOikeusRyhma("adminViewKor");
                 QKayttoOikeus qAdminViewKo = new QKayttoOikeus("adminViewKo");
                 JPQLQuery<KayttoOikeusRyhma> adminViewSubquery = JPAExpressions.select(qAdminViewKor)
@@ -78,7 +77,7 @@ public AnomusCriteriaFunction<QAnomus, QKayttoOikeusRyhma, QHaettuKayttoOikeusRy
                 builder.and(qKayttoOikeusRyhma.in(adminViewSubquery));
             }
 
-            if (BooleanUtils.isTrue(this.onlyActive)) {
+            if (Boolean.TRUE.equals(this.onlyActive)) {
                 builder.and(qHaettuKayttoOikeusRyhma.tyyppi.eq(KayttoOikeudenTila.ANOTTU)
                         .or(qHaettuKayttoOikeusRyhma.tyyppi.isNull()));
             }
diff --git a/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/service/impl/KutsuServiceImpl.java b/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/service/impl/KutsuServiceImpl.java
index a4da9624a..371bfc85f 100644
--- a/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/service/impl/KutsuServiceImpl.java
+++ b/kayttooikeus-service/src/main/java/fi/vm/sade/kayttooikeus/service/impl/KutsuServiceImpl.java
@@ -24,7 +24,6 @@
 import fi.vm.sade.oppijanumerorekisteri.dto.*;
 import lombok.RequiredArgsConstructor;
 
-import org.apache.commons.lang.BooleanUtils;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.data.domain.Sort;
 import org.springframework.stereotype.Service;
@@ -118,7 +117,7 @@ private void setCriteriaByUser(KutsuCriteria kutsuCriteria) {
             } else {
                 organisaatioOidLimit = permissionCheckerService.getCurrentUserOrgnisationsWithPalveluRole(palveluRoolit);
             }
-            if (BooleanUtils.isTrue(kutsuCriteria.getSubOrganisations())) {
+            if (Boolean.TRUE.equals(kutsuCriteria.getSubOrganisations())) {
                 organisaatioOidLimit = organisaatioOidLimit.stream()
                         .flatMap(organisaatioOid -> organisaatioClient.listWithChildOids(organisaatioOid,
                                 new OrganisaatioMyontoPredicate(false)).stream())
diff --git a/kayttooikeus-service/src/test/java/fi/vm/sade/kayttooikeus/service/CryptoServiceTest.java b/kayttooikeus-service/src/test/java/fi/vm/sade/kayttooikeus/service/CryptoServiceTest.java
index c0e0dc558..bbaef352a 100644
--- a/kayttooikeus-service/src/test/java/fi/vm/sade/kayttooikeus/service/CryptoServiceTest.java
+++ b/kayttooikeus-service/src/test/java/fi/vm/sade/kayttooikeus/service/CryptoServiceTest.java
@@ -2,7 +2,7 @@
 
 import fi.vm.sade.kayttooikeus.config.properties.AuthProperties;
 import fi.vm.sade.kayttooikeus.service.impl.CryptoServiceImpl;
-import org.apache.commons.lang.StringUtils;
+import org.apache.commons.lang3.StringUtils;
 import org.junit.Before;
 import org.junit.Ignore;
 import org.junit.Test;