Immutability en unused variable code quality opmerkingen opgelost

Author: ericwout-overheid

src/main/java/nl/rijksoverheid/moz/controller/DienstverlenerController.java

@@ -77,10 +77,8 @@ public Response addAfdelingToDienstverlener(
         if (request == null) {
             return Response.status(Response.Status.BAD_REQUEST).entity("Request body mag niet leeg zijn").build();
         }
-        var afdeling = dienstverlenerService.addAfdelingToDienstverlener(dienstverlenerNaam, request);
+        dienstverlenerService.addAfdelingToDienstverlener(dienstverlenerNaam, request);
         URI uri = URI.create(String.format("/dienstverlener/%s", dienstverlenerNaam));
         return Response.created(uri).build();
     }
-
-
 }

src/main/java/nl/rijksoverheid/moz/controller/EmailVerificatieController.java

@@ -15,7 +15,7 @@
 @Path("/api/profielservice/v1")
 @Produces(MediaType.APPLICATION_JSON)
 @Consumes(MediaType.APPLICATION_JSON)
-@Tag(name = "EmailVerificatie", description = "Endpoints voor het verifieren van emails")
+@Tag(name = "EmailVerificatie", description = "Endpoints voor het verifiëren van emails")
 public class EmailVerificatieController {
 
     @Inject

src/main/java/nl/rijksoverheid/moz/entity/Dienstverlener.java

@@ -3,15 +3,13 @@
 import io.quarkus.hibernate.orm.panache.PanacheEntity;
 import jakarta.persistence.CascadeType;
 import jakarta.persistence.Entity;
-import jakarta.persistence.EnumType;
-import jakarta.persistence.Enumerated;
 import jakarta.persistence.FetchType;
-import jakarta.persistence.ManyToOne;
 import jakarta.persistence.OneToMany;
 import jakarta.validation.constraints.NotNull;
 import org.hibernate.envers.Audited;
 
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 @Entity
@@ -46,10 +44,15 @@ public void setOin(String oin) {
     }
 
     public List<Afdeling> getAfdelingen() {
-        return afdelingen;
+        return Collections.unmodifiableList(afdelingen);
+    }
+
+    public void addAfdeling(Afdeling afdeling) {
+        afdelingen.add(afdeling);
     }
 
     public void setAfdelingen(List<Afdeling> afdelingen) {
-        this.afdelingen = afdelingen;
+        this.afdelingen.clear();
+        this.afdelingen.addAll(afdelingen);
     }
 }

src/main/java/nl/rijksoverheid/moz/entity/Partij.java

@@ -1,5 +1,6 @@
 package nl.rijksoverheid.moz.entity;
 import java.util.ArrayList;
+import java.util.Collections;
 import java.util.List;
 
 import jakarta.persistence.*;
@@ -21,11 +22,12 @@ public class Partij extends PanacheEntity {
     private List<Voorkeur> voorkeuren = new ArrayList<>();
 
     public List<Voorkeur> getVoorkeuren() {
-        return voorkeuren;
+        return Collections.unmodifiableList(voorkeuren);
     }
 
     public void setVoorkeuren(List<Voorkeur> voorkeuren) {
-        this.voorkeuren = voorkeuren;
+        this.voorkeuren.clear();
+        this.voorkeuren.addAll(voorkeuren);
     }
 
     public static Partij findByIdentificatie(IdentificatieType type, String nummer) {
@@ -37,8 +39,6 @@ public static Partij findByIdentificatie(IdentificatieType type, String nummer)
     """, type, nummer).firstResult();
     }
 
-
-
     public void addIdentificatie(Identificatie identificatie) {
         identificaties.add(identificatie);
         identificatie.setPartij(this);
@@ -49,20 +49,29 @@ public void addVoorkeur(Voorkeur voorkeur) {
         voorkeur.setPartij(this);
     }
 
+    public void removeVoorkeur(Voorkeur voorkeur) {
+        voorkeuren.remove(voorkeur);
+    }
+
     public List<Identificatie> getIdentificaties() {
-        return identificaties;
+        return Collections.unmodifiableList(identificaties);
     }
 
     public void setIdentificaties(List<Identificatie> identificaties) {
-        this.identificaties = identificaties;
+        this.identificaties.clear();
+        this.identificaties.addAll(identificaties);
     }
 
     public List<Contactgegeven> getContactgegevens() {
-        return contactgegevens;
+        return Collections.unmodifiableList(contactgegevens);
     }
 
     public void setContactgegevens(List<Contactgegeven> contactgegevens) {
-        this.contactgegevens = contactgegevens;
+        this.contactgegevens.clear();
+        this.contactgegevens.addAll(contactgegevens);
     }
-}
 
+    public void removeContactgegeven(Contactgegeven contact) {
+        contactgegevens.remove(contact);
+    }
+}

src/main/java/nl/rijksoverheid/moz/services/DienstverlenerService.java

@@ -30,7 +30,7 @@ public Afdeling addAfdelingToDienstverlener(String naam, AfdelingRequest request
         afdeling.setBeschrijving(request.beschrijving);
         afdeling.setDienstverlener(dienstverlener);
 
-        dienstverlener.getAfdelingen().add(afdeling);
+        dienstverlener.addAfdeling(afdeling);
 
         afdeling.persist();
         dienstverlener.persist();
@@ -58,7 +58,7 @@ public Dienstverlener findOrCreateDienstverlener(String naam, String oin) {
         Afdeling defaultAfdeling = new Afdeling();
         defaultAfdeling.setBeschrijving("Alles");
         defaultAfdeling.setDienstverlener(dienstverlener);
-        dienstverlener.getAfdelingen().add(defaultAfdeling);
+        dienstverlener.addAfdeling(defaultAfdeling);
 
         // Persist de dienstverlener (cascadet ook de aangemaakte afdeling)
         dienstverlener.persist();

src/main/java/nl/rijksoverheid/moz/services/PartijService.java

@@ -153,7 +153,7 @@ public boolean deleteContactgegeven(IdentificatieType identificatieType, String
             return false;
         }
 
-        partij.getContactgegevens().remove(contact);
+        partij.removeContactgegeven(contact);
 
         contact.delete();
         return true;
@@ -173,7 +173,7 @@ public boolean deleteVoorkeur(IdentificatieType identificatieType, String identi
             return false;
         }
 
-        partij.getVoorkeuren().remove(voorkeur);
+        partij.removeVoorkeur(voorkeur);
 
         voorkeur.delete();
         return true;