jiazhizhong/OpenIsle
1
0
Fork 0
mirror of https://github.com/nagisa77/OpenIsle.git synced 2026-05-05 02:07:27 +08:00
Code Issues Packages Projects Releases Wiki Activity

Ensure DraftService uses transactions

Browse Source
This commit is contained in:
Tim
2025-07-06 10:41:55 +08:00
parent 17ae94e801
commit b7c11741b4
1 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/com/openisle/service/DraftService.java
View File

@@ -8,6 +8,7 @@ import com.openisle.repository.CategoryRepository;
import com.openisle.repository.DraftRepository; import com.openisle.repository.DraftRepository;
import com.openisle.repository.TagRepository; import com.openisle.repository.TagRepository;
import com.openisle.repository.UserRepository; import com.openisle.repository.UserRepository;
import org.springframework.transaction.annotation.Transactional;
import lombok.RequiredArgsConstructor; import lombok.RequiredArgsConstructor;
import org.springframework.stereotype.Service; import org.springframework.stereotype.Service;
@@ -24,6 +25,7 @@ public class DraftService {
private final CategoryRepository categoryRepository; private final CategoryRepository categoryRepository;
private final TagRepository tagRepository; private final TagRepository tagRepository;
@Transactional
public Draft saveDraft(String username, Long categoryId, String title, String content, List<Long> tagIds) { public Draft saveDraft(String username, Long categoryId, String title, String content, List<Long> tagIds) {
User user = userRepository.findByUsername(username) User user = userRepository.findByUsername(username)
.orElseThrow(() -> new IllegalArgumentException("User not found")); .orElseThrow(() -> new IllegalArgumentException("User not found"));
@@ -46,11 +48,13 @@ public class DraftService {
return draftRepository.save(draft); return draftRepository.save(draft);
} }
@Transactional(readOnly = true)
public Optional<Draft> getDraft(String username) { public Optional<Draft> getDraft(String username) {
return userRepository.findByUsername(username) return userRepository.findByUsername(username)
.flatMap(draftRepository::findByAuthor); .flatMap(draftRepository::findByAuthor);
} }
@Transactional
public void deleteDraft(String username) { public void deleteDraft(String username) {
userRepository.findByUsername(username) userRepository.findByUsername(username)
.ifPresent(draftRepository::deleteByAuthor); .ifPresent(draftRepository::deleteByAuthor);