From c38b9536ea1aeb1d69c56d7cf62438f83d3fbacc Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Wed, 11 Feb 2026 06:59:05 +0000
Subject: [PATCH 1/3] Initial plan


From 229fe0d5659afc1e73602a3b2bc2908aa8958448 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Wed, 11 Feb 2026 07:03:25 +0000
Subject: [PATCH 2/3] Apply OpenRewrite recipes to upgrade Spring Boot to
 3.3.13 and Java to 21, migrate javax to jakarta

---
 pom.xml                                                |  8 ++++----
 .../com/photoalbum/controller/DetailController.java    |  2 +-
 .../java/com/photoalbum/controller/HomeController.java |  2 +-
 .../com/photoalbum/controller/PhotoFileController.java |  2 +-
 src/main/java/com/photoalbum/model/Photo.java          | 10 +++++-----
 .../com/photoalbum/service/impl/PhotoServiceImpl.java  |  8 ++++----
 6 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/pom.xml b/pom.xml
index 216b79d5..966e621d 100644
--- a/pom.xml
+++ b/pom.xml
@@ -8,7 +8,7 @@
     <parent>
         <groupId>org.springframework.boot</groupId>
         <artifactId>spring-boot-starter-parent</artifactId>
-        <version>2.7.18</version>
+        <version>3.3.13</version>
         <relativePath/>
     </parent>
 
@@ -21,9 +21,9 @@
     <description>A simple photo storage and gallery application built with Spring Boot and Oracle DB</description>
 
     <properties>
-        <java.version>1.8</java.version>
-        <maven.compiler.source>8</maven.compiler.source>
-        <maven.compiler.target>8</maven.compiler.target>
+        <java.version>21</java.version>
+        <maven.compiler.source>21</maven.compiler.source>
+        <maven.compiler.target>21</maven.compiler.target>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     </properties>
 
diff --git a/src/main/java/com/photoalbum/controller/DetailController.java b/src/main/java/com/photoalbum/controller/DetailController.java
index d138ef95..1c091add 100644
--- a/src/main/java/com/photoalbum/controller/DetailController.java
+++ b/src/main/java/com/photoalbum/controller/DetailController.java
@@ -37,7 +37,7 @@ public String detail(@PathVariable String id, Model model) {
 
         try {
             Optional<Photo> photoOpt = photoService.getPhotoById(id);
-            if (!photoOpt.isPresent()) {
+            if (photoOpt.isEmpty()) {
                 return "redirect:/";
             }
 
diff --git a/src/main/java/com/photoalbum/controller/HomeController.java b/src/main/java/com/photoalbum/controller/HomeController.java
index d9905d97..22d52509 100644
--- a/src/main/java/com/photoalbum/controller/HomeController.java
+++ b/src/main/java/com/photoalbum/controller/HomeController.java
@@ -54,7 +54,7 @@ public String index(Model model) {
      */
     @PostMapping("/upload")
     @ResponseBody
-    public ResponseEntity<Map<String, Object>> uploadPhotos(@RequestParam("files") List<MultipartFile> files) {
+    public ResponseEntity<Map<String, Object>> uploadPhotos(@RequestParam List<MultipartFile> files) {
         Map<String, Object> response = new HashMap<String, Object>();
         List<Map<String, Object>> uploadedPhotos = new ArrayList<Map<String, Object>>();
         List<Map<String, Object>> failedUploads = new ArrayList<Map<String, Object>>();
diff --git a/src/main/java/com/photoalbum/controller/PhotoFileController.java b/src/main/java/com/photoalbum/controller/PhotoFileController.java
index 2f314081..6013ab6e 100644
--- a/src/main/java/com/photoalbum/controller/PhotoFileController.java
+++ b/src/main/java/com/photoalbum/controller/PhotoFileController.java
@@ -45,7 +45,7 @@ public ResponseEntity<Resource> servePhoto(@PathVariable String id) {
             logger.info("=== DEBUGGING: Serving photo request for ID {} ===", id);
             Optional<Photo> photoOpt = photoService.getPhotoById(id);
 
-            if (!photoOpt.isPresent()) {
+            if (photoOpt.isEmpty()) {
                 logger.warn("Photo with ID {} not found", id);
                 return ResponseEntity.notFound().build();
             }
diff --git a/src/main/java/com/photoalbum/model/Photo.java b/src/main/java/com/photoalbum/model/Photo.java
index d1cea3ec..1852ad7a 100644
--- a/src/main/java/com/photoalbum/model/Photo.java
+++ b/src/main/java/com/photoalbum/model/Photo.java
@@ -1,10 +1,10 @@
 package com.photoalbum.model;
 
-import javax.persistence.*;
-import javax.validation.constraints.NotBlank;
-import javax.validation.constraints.NotNull;
-import javax.validation.constraints.Positive;
-import javax.validation.constraints.Size;
+import jakarta.persistence.*;
+import jakarta.validation.constraints.NotBlank;
+import jakarta.validation.constraints.NotNull;
+import jakarta.validation.constraints.Positive;
+import jakarta.validation.constraints.Size;
 
 import java.time.LocalDateTime;
 import java.util.UUID;
diff --git a/src/main/java/com/photoalbum/service/impl/PhotoServiceImpl.java b/src/main/java/com/photoalbum/service/impl/PhotoServiceImpl.java
index fa379a51..9211352c 100644
--- a/src/main/java/com/photoalbum/service/impl/PhotoServiceImpl.java
+++ b/src/main/java/com/photoalbum/service/impl/PhotoServiceImpl.java
@@ -91,7 +91,7 @@ public UploadResult uploadPhoto(MultipartFile file) {
             // Validate file size
             if (file.getSize() > maxFileSizeBytes) {
                 result.setSuccess(false);
-                result.setErrorMessage(String.format("File size exceeds %dMB limit.", maxFileSizeBytes / 1024 / 1024));
+                result.setErrorMessage("File size exceeds %dMB limit.".formatted(maxFileSizeBytes / 1024 / 1024));
                 logger.warn("Upload rejected: File size {} exceeds limit for {}", 
                     file.getSize(), file.getOriginalFilename());
                 return result;
@@ -178,7 +178,7 @@ public UploadResult uploadPhoto(MultipartFile file) {
     public boolean deletePhoto(String id) {
         try {
             Optional<Photo> photoOpt = photoRepository.findById(id);
-            if (!photoOpt.isPresent()) {
+            if (photoOpt.isEmpty()) {
                 logger.warn("Photo with ID {} not found for deletion", id);
                 return false;
             }
@@ -203,7 +203,7 @@ public boolean deletePhoto(String id) {
     @Transactional(readOnly = true)
     public Optional<Photo> getPreviousPhoto(Photo currentPhoto) {
         List<Photo> olderPhotos = photoRepository.findPhotosUploadedBefore(currentPhoto.getUploadedAt());
-        return olderPhotos.isEmpty() ? Optional.<Photo>empty() : Optional.of(olderPhotos.get(0));
+        return olderPhotos.isEmpty() ? Optional.<Photo>empty() : Optional.of(olderPhotos.getFirst());
     }
 
     /**
@@ -213,7 +213,7 @@ public Optional<Photo> getPreviousPhoto(Photo currentPhoto) {
     @Transactional(readOnly = true)
     public Optional<Photo> getNextPhoto(Photo currentPhoto) {
         List<Photo> newerPhotos = photoRepository.findPhotosUploadedAfter(currentPhoto.getUploadedAt());
-        return newerPhotos.isEmpty() ? Optional.<Photo>empty() : Optional.of(newerPhotos.get(0));
+        return newerPhotos.isEmpty() ? Optional.<Photo>empty() : Optional.of(newerPhotos.getFirst());
     }
 
     /**

From 970711bf8d13ec9234c5d64f1532f91ad72eb014 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Wed, 11 Feb 2026 07:03:54 +0000
Subject: [PATCH 3/3] Upgrade Spring Boot from 3.3.13 to 3.4.6

---
 pom.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 966e621d..375ac9fc 100644
--- a/pom.xml
+++ b/pom.xml
@@ -8,7 +8,7 @@
     <parent>
         <groupId>org.springframework.boot</groupId>
         <artifactId>spring-boot-starter-parent</artifactId>
-        <version>3.3.13</version>
+        <version>3.4.6</version>
         <relativePath/>
     </parent>