From 1180a7dbfd89b4569b368487168b509e9d7dcfbc Mon Sep 17 00:00:00 2001
From: Andreas Svanberg <andreass@dsv.su.se>
Date: Mon, 2 Dec 2024 10:58:00 +0100
Subject: [PATCH] Fix crashes in the text matching job (#23)
With the migration to Spring it is much stricter in requiring @Transactional for all methods that write to the database.
Fixes #22
Co-authored-by: Tom Zhao <tom.zhao@dsv.su.se>
Reviewed-on: https://gitea.dsv.su.se/DMC/scipro/pulls/23
Reviewed-by: Tom Zhao <tom.zhao@dsv.su.se>
Co-authored-by: Andreas Svanberg <andreass@dsv.su.se>
Co-committed-by: Andreas Svanberg <andreass@dsv.su.se>
---
.../plagiarism/urkund/UrkundSubmissionRepositoryImpl.java | 3 +++
1 file changed, 3 insertions(+)
diff --git a/core/src/main/java/se/su/dsv/scipro/plagiarism/urkund/UrkundSubmissionRepositoryImpl.java b/core/src/main/java/se/su/dsv/scipro/plagiarism/urkund/UrkundSubmissionRepositoryImpl.java
index f05cfca647..d88963780e 100644
--- a/core/src/main/java/se/su/dsv/scipro/plagiarism/urkund/UrkundSubmissionRepositoryImpl.java
+++ b/core/src/main/java/se/su/dsv/scipro/plagiarism/urkund/UrkundSubmissionRepositoryImpl.java
@@ -1,6 +1,7 @@
package se.su.dsv.scipro.plagiarism.urkund;
import com.querydsl.jpa.impl.JPAQuery;
+import jakarta.transaction.Transactional;
import se.su.dsv.scipro.file.FileDescription;
import jakarta.inject.Inject;
@@ -27,6 +28,7 @@ public class UrkundSubmissionRepositoryImpl implements UrkundSubmissionRepositor
}
@Override
+ @Transactional
public UrkundSubmission save(final UrkundSubmission submission) {
final EntityManager em = entityManagerProvider.get();
if (em.contains(submission)) {
@@ -49,6 +51,7 @@ public class UrkundSubmissionRepositoryImpl implements UrkundSubmissionRepositor
}
@Override
+ @Transactional
public void delete(final UrkundSubmission urkundSubmission) {
entityManagerProvider.get().remove(urkundSubmission);
}