chore: upgrade to spring 6

dhis-2/dhis-api/pom.xml

@@ -49,6 +49,10 @@
       <groupId>com.fasterxml.jackson.core</groupId>
       <artifactId>jackson-databind</artifactId>
     </dependency>
+    <dependency>
+      <groupId>jakarta.xml.bind</groupId>
+      <artifactId>jakarta.xml.bind-api</artifactId>
+    </dependency>
     <dependency>
       <groupId>com.fasterxml.jackson.dataformat</groupId>
       <artifactId>jackson-dataformat-xml</artifactId>
@@ -117,18 +121,14 @@
       <groupId>org.geotools</groupId>
       <artifactId>gt-geojson</artifactId>
     </dependency>
-    <dependency>
-      <groupId>javax.xml.bind</groupId>
-      <artifactId>jaxb-api</artifactId>
-    </dependency>
     <dependency>
       <groupId>org.projectlombok</groupId>
       <artifactId>lombok</artifactId>
       <scope>provided</scope>
     </dependency>
     <dependency>
       <groupId>org.hibernate</groupId>
-      <artifactId>hibernate-core</artifactId>
+      <artifactId>hibernate-core-jakarta</artifactId>
     </dependency>
     <dependency>
       <groupId>com.google.code.findbugs</groupId>
@@ -139,8 +139,8 @@
       <artifactId>gt-opengis</artifactId>
     </dependency>
     <dependency>
-      <groupId>javax.persistence</groupId>
-      <artifactId>javax.persistence-api</artifactId>
+      <groupId>jakarta.persistence</groupId>
+      <artifactId>jakarta.persistence-api</artifactId>
     </dependency>
 
     <!-- SMPP -->
@@ -186,8 +186,8 @@
       <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>javax.servlet</groupId>
-      <artifactId>javax.servlet-api</artifactId>
+      <groupId>jakarta.servlet</groupId>
+      <artifactId>jakarta.servlet-api</artifactId>
     </dependency>
   </dependencies>
 </project>

dhis-2/dhis-api/src/main/java/org/hisp/dhis/common/ObjectDeletionRequestedEvent.java

@@ -33,8 +33,9 @@
  * @author Lars Helge Overland
  */
 public class ObjectDeletionRequestedEvent extends ApplicationEvent {
+
   /** Should rollback the transaction if DeleteNotAllowedException is thrown */
-  private boolean shouldRollBack = true;
+  private boolean rollback = true;
 
   // -------------------------------------------------------------------------
   // Constructors
@@ -44,15 +45,16 @@ public ObjectDeletionRequestedEvent(Object source) {
     super(source);
   }
 
+  public ObjectDeletionRequestedEvent(Object source, boolean rollback) {
+    super(source);
+    this.rollback = rollback;
+  }
+
   // -------------------------------------------------------------------------
   // Getter && Setter
   // -------------------------------------------------------------------------
 
-  public boolean isShouldRollBack() {
-    return shouldRollBack;
-  }
-
-  public void setShouldRollBack(boolean shouldRollBack) {
-    this.shouldRollBack = shouldRollBack;
+  public boolean isRollback() {
+    return rollback;
   }
 }

dhis-2/dhis-api/src/main/java/org/hisp/dhis/deletedobject/DeletedObject.java

@@ -32,17 +32,17 @@
 import com.fasterxml.jackson.dataformat.xml.annotation.JacksonXmlRootElement;
 import com.google.common.base.MoreObjects;
 import com.google.common.base.Objects;
+import jakarta.persistence.Column;
+import jakarta.persistence.Entity;
+import jakarta.persistence.GeneratedValue;
+import jakarta.persistence.Id;
+import jakarta.persistence.SequenceGenerator;
+import jakarta.persistence.Table;
+import jakarta.persistence.Temporal;
+import jakarta.persistence.TemporalType;
+import jakarta.persistence.UniqueConstraint;
 import java.io.Serializable;
 import java.util.Date;
-import javax.persistence.Column;
-import javax.persistence.Entity;
-import javax.persistence.GeneratedValue;
-import javax.persistence.Id;
-import javax.persistence.SequenceGenerator;
-import javax.persistence.Table;
-import javax.persistence.Temporal;
-import javax.persistence.TemporalType;
-import javax.persistence.UniqueConstraint;
 import org.apache.commons.lang3.StringUtils;
 import org.hisp.dhis.common.DxfNamespaces;
 import org.hisp.dhis.common.IdentifiableObject;

dhis-2/dhis-api/src/main/java/org/hisp/dhis/fileresource/SimpleImageResource.java

@@ -28,10 +28,10 @@
 package org.hisp.dhis.fileresource;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+import jakarta.xml.bind.annotation.XmlRootElement;
 import java.util.HashMap;
 import java.util.Map;
 import java.util.Objects;
-import javax.xml.bind.annotation.XmlRootElement;
 
 /** This is simple class to represent the list of possible static image resources. */
 @XmlRootElement

dhis-2/dhis-api/src/main/java/org/hisp/dhis/hibernate/HibernateProxyUtils.java

@@ -29,7 +29,6 @@
 
 import java.util.Objects;
 import org.hibernate.Hibernate;
-import org.hibernate.proxy.HibernateProxy;
 import org.hibernate.proxy.HibernateProxyHelper;
 
 /**
@@ -55,15 +54,4 @@ public static Class getRealClass(Object object) {
   public static <T> T unproxy(T proxy) {
     return (T) Hibernate.unproxy(proxy);
   }
-
-  public static <T> void initializeAndUnproxy(T entity) {
-    if (entity == null) {
-      return;
-    }
-
-    Hibernate.initialize(entity);
-    if (entity instanceof HibernateProxy) {
-      ((HibernateProxy) entity).getHibernateLazyInitializer().getImplementation();
-    }
-  }
 }

dhis-2/dhis-api/src/main/java/org/hisp/dhis/message/MessageSender.java

@@ -27,8 +27,10 @@
  */
 package org.hisp.dhis.message;
 
+import java.util.List;
 import java.util.Set;
 import java.util.concurrent.Future;
+import org.hisp.dhis.outboundmessage.OutboundMessage;
 import org.hisp.dhis.outboundmessage.OutboundMessageBatch;
 import org.hisp.dhis.outboundmessage.OutboundMessageResponse;
 import org.hisp.dhis.outboundmessage.OutboundMessageResponseSummary;
@@ -39,6 +41,7 @@
  * @author Lars Helge Overland
  */
 public interface MessageSender {
+
   /**
    * Sends a message. The given message will be sent to the given set of users.
    *
@@ -71,4 +74,14 @@ ListenableFuture<OutboundMessageResponseSummary> sendMessageBatchAsync(
 
   /** To check if given service is configured and ready to use. */
   boolean isConfigured();
+
+  default List<OutboundMessage> getMessagesByEmail(String recipient) {
+    return List.of();
+  }
+
+  default void clearMessages() {}
+
+  default List<OutboundMessage> getAllMessages() {
+    return List.of();
+  }
 }

dhis-2/dhis-api/src/main/java/org/hisp/dhis/scheduling/JobConfigurationService.java

@@ -32,6 +32,7 @@
 import java.util.Map;
 import javax.annotation.Nonnull;
 import org.hisp.dhis.feedback.ConflictException;
+import org.hisp.dhis.feedback.NotFoundException;
 import org.hisp.dhis.schema.Property;
 import org.hisp.dhis.user.UserDetails;
 import org.springframework.util.MimeType;
@@ -61,6 +62,13 @@ String create(JobConfiguration config, MimeType contentType, InputStream content
 
   void createDefaultJob(JobType type, UserDetails actingUser);
 
+  String createInTransaction(
+      JobConfiguration jobConfiguration, MimeType contentType, InputStream content)
+      throws ConflictException, NotFoundException;
+
+  String createInTransaction(JobConfiguration jobConfiguration)
+      throws ConflictException, NotFoundException;
+
   /**
    * Updates all {@link JobConfiguration}s that are not {@link JobConfiguration#isEnabled()} to
    * state {@link JobStatus#DISABLED} in case they are in state {@link JobStatus#SCHEDULED}.

dhis-2/dhis-api/src/main/java/org/hisp/dhis/scheduling/JobConfigurationStore.java

@@ -175,6 +175,8 @@ public interface JobConfigurationStore extends GenericDimensionalObjectStore<Job
    */
   boolean tryRevertNow(@Nonnull String jobId);
 
+  boolean executeNow(@Nonnull String jobId);
+
   /**
    * A successful update means the DB state flipped from {@link JobStatus#SCHEDULED} to {@link
    * JobStatus#RUNNING}.

dhis-2/dhis-api/src/main/java/org/hisp/dhis/scheduling/JobSchedulerService.java

@@ -27,15 +27,18 @@
  */
 package org.hisp.dhis.scheduling;
 
+import java.io.InputStream;
 import java.util.List;
 import java.util.Set;
 import javax.annotation.CheckForNull;
 import javax.annotation.Nonnull;
+import org.hisp.dhis.common.NonTransactional;
 import org.hisp.dhis.common.UID;
 import org.hisp.dhis.feedback.ConflictException;
 import org.hisp.dhis.feedback.ForbiddenException;
 import org.hisp.dhis.feedback.NotFoundException;
 import org.hisp.dhis.scheduling.JobProgress.Progress;
+import org.springframework.util.MimeType;
 
 /**
  * This is the external API (called by users via controller API) for the scheduling.
@@ -60,6 +63,22 @@ public interface JobSchedulerService {
    */
   void executeNow(@Nonnull String jobId) throws ConflictException, NotFoundException;
 
+  /**
+   * Executes a job configuration in a separate transaction.
+   *
+   * @param jobId the job id to execute
+   * @throws NotFoundException
+   * @throws ConflictException
+   */
+  void runInTransaction(String jobId) throws NotFoundException, ConflictException;
+
+  @NonTransactional
+  void createThenExecute(JobConfiguration config, MimeType contentType, InputStream content)
+      throws ConflictException, NotFoundException;
+
+  @NonTransactional
+  void createThenExecute(JobConfiguration config) throws ConflictException, NotFoundException;
+
   /**
    * Reverts the {@link JobStatus} of the job from {@link JobStatus#RUNNING} to the appropriate
    * status after a failed execution. For an ad-hoc job this is {@link JobStatus#DISABLED}, for a

dhis-2/dhis-api/src/main/java/org/hisp/dhis/security/AuthenticationSerializer.java

@@ -27,7 +27,7 @@
  */
 package org.hisp.dhis.security;
 
-import javax.xml.bind.DatatypeConverter;
+import jakarta.xml.bind.DatatypeConverter;
 import org.apache.commons.lang3.SerializationUtils;
 import org.springframework.security.core.Authentication;
 

dhis-2/dhis-api/src/main/java/org/hisp/dhis/sms/config/SmsConfiguration.java

@@ -28,11 +28,11 @@
 package org.hisp.dhis.sms.config;
 
 import com.fasterxml.jackson.annotation.JsonProperty;
+import jakarta.xml.bind.annotation.XmlRootElement;
 import java.io.Serial;
 import java.io.Serializable;
 import java.util.ArrayList;
 import java.util.List;
-import javax.xml.bind.annotation.XmlRootElement;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import lombok.Setter;

dhis-2/dhis-api/src/main/java/org/hisp/dhis/user/UserAccountService.java

@@ -27,8 +27,8 @@
  */
 package org.hisp.dhis.user;
 
+import jakarta.servlet.http.HttpServletRequest;
 import java.io.IOException;
-import javax.servlet.http.HttpServletRequest;
 import org.hisp.dhis.common.auth.RegistrationParams;
 import org.hisp.dhis.common.auth.UserInviteParams;
 import org.hisp.dhis.common.auth.UserRegistrationParams;

dhis-2/dhis-services/dhis-service-administration/pom.xml

@@ -101,8 +101,8 @@
       <scope>test</scope>
     </dependency>
     <dependency>
-      <groupId>javax.persistence</groupId>
-      <artifactId>javax.persistence-api</artifactId>
+      <groupId>jakarta.persistence</groupId>
+      <artifactId>jakarta.persistence-api</artifactId>
     </dependency>
     <dependency>
       <groupId>org.slf4j</groupId>

dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/dataintegrity/hibernate/HibernateDataIntegrityStore.java

@@ -29,10 +29,10 @@
 
 import static java.util.stream.Collectors.toUnmodifiableList;
 
+import jakarta.persistence.EntityManager;
+import jakarta.persistence.PersistenceContext;
 import java.util.Date;
 import java.util.List;
-import javax.persistence.EntityManager;
-import javax.persistence.PersistenceContext;
 import lombok.RequiredArgsConstructor;
 import org.hisp.dhis.dataintegrity.DataIntegrityCheck;
 import org.hisp.dhis.dataintegrity.DataIntegrityDetails;

dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/merge/dataelement/DataElementMergeService.java

@@ -28,12 +28,12 @@
 package org.hisp.dhis.merge.dataelement;
 
 import com.google.common.collect.ImmutableList;
+import jakarta.persistence.EntityManager;
 import java.util.HashSet;
 import java.util.List;
 import java.util.Set;
 import javax.annotation.Nonnull;
 import javax.annotation.PostConstruct;
-import javax.persistence.EntityManager;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.hisp.dhis.common.UID;

dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/merge/dataelement/handler/DataDataElementMergeHandler.java

@@ -27,6 +27,7 @@
  */
 package org.hisp.dhis.merge.dataelement.handler;
 
+import jakarta.persistence.EntityManager;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.Comparator;
@@ -35,7 +36,6 @@
 import java.util.function.BiPredicate;
 import java.util.stream.Collectors;
 import javax.annotation.Nonnull;
-import javax.persistence.EntityManager;
 import lombok.RequiredArgsConstructor;
 import lombok.extern.slf4j.Slf4j;
 import org.hisp.dhis.dataelement.DataElement;

dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/merge/orgunit/handler/DataOrgUnitMergeHandler.java

@@ -29,7 +29,7 @@
 
 import static org.hisp.dhis.common.IdentifiableObjectUtils.getIdentifiers;
 
-import javax.persistence.EntityManager;
+import jakarta.persistence.EntityManager;
 import lombok.RequiredArgsConstructor;
 import org.hisp.dhis.common.IdentifiableObjectUtils;
 import org.hisp.dhis.dataapproval.DataApprovalAuditService;

dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/merge/orgunit/handler/TrackerOrgUnitMergeHandler.java

@@ -27,7 +27,7 @@
  */
 package org.hisp.dhis.merge.orgunit.handler;
 
-import javax.persistence.EntityManager;
+import jakarta.persistence.EntityManager;
 import lombok.RequiredArgsConstructor;
 import org.hisp.dhis.common.IdentifiableObjectUtils;
 import org.hisp.dhis.merge.orgunit.OrgUnitMergeRequest;

dhis-2/dhis-services/dhis-service-administration/src/main/java/org/hisp/dhis/sms/scheduling/SendScheduledMessageJob.java

@@ -41,16 +41,14 @@
 import org.hisp.dhis.sms.outbound.OutboundSms;
 import org.hisp.dhis.sms.outbound.OutboundSmsService;
 import org.hisp.dhis.sms.outbound.OutboundSmsStatus;
-import org.springframework.beans.factory.annotation.Qualifier;
 import org.springframework.stereotype.Component;
 
 @RequiredArgsConstructor
 @Component("sendScheduledMessageJob")
 public class SendScheduledMessageJob implements Job {
   private final OutboundSmsService outboundSmsService;
 
-  @Qualifier("smsMessageSender")
-  private final MessageSender smsSender;
+  private final MessageSender smsMessageSender;
 
   // -------------------------------------------------------------------------
   // Implementation
@@ -65,7 +63,7 @@ public JobType getJobType() {
   public void execute(JobConfiguration config, JobProgress progress) {
     progress.startingProcess("Starting to send outbound messages");
     progress.startingStage("Validating environment setup");
-    if (!smsSender.isConfigured()) {
+    if (!smsMessageSender.isConfigured()) {
       progress.failedStage("SMS gateway configuration does not exist, job aborted");
       return;
     }
@@ -94,7 +92,8 @@ private void sendMessages(JobProgress progress) {
           outboundSms -> {
             outboundSms.setDate(new Date());
             outboundSms.setStatus(OutboundSmsStatus.SENT);
-            smsSender.sendMessage(null, outboundSms.getMessage(), outboundSms.getRecipients());
+            smsMessageSender.sendMessage(
+                null, outboundSms.getMessage(), outboundSms.getRecipients());
           });
     }
   }