fix: Handle the null exception

Author: enverern

apps/jobboard-backend/src/main/java/org/bounswe/jobboardbackend/exception/ErrorCode.java

@@ -31,6 +31,7 @@ public enum ErrorCode {
 
     WORKPLACE_NOT_FOUND(HttpStatus.NOT_FOUND),
     WORKPLACE_ALREADY_EXISTS(HttpStatus.CONFLICT),
+    REVIEW_ALREADY_EXISTS(HttpStatus.CONFLICT),
 
     VALIDATION_ERROR(HttpStatus.BAD_REQUEST),
 

apps/jobboard-backend/src/main/java/org/bounswe/jobboardbackend/workplace/service/ReviewService.java

@@ -17,6 +17,8 @@
 import org.springframework.security.access.AccessDeniedException;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.bounswe.jobboardbackend.exception.HandleException;
+import org.bounswe.jobboardbackend.exception.ErrorCode;
 
 import java.util.*;
 import java.util.stream.Collectors;
@@ -50,7 +52,7 @@ public ReviewResponse createReview(Long workplaceId, ReviewCreateRequest req, Us
 
         boolean alreadyReviewed = reviewRepository.existsByWorkplace_IdAndUser_Id(workplaceId, currentUser.getId());
         if (alreadyReviewed) {
-            throw new IllegalStateException("You have already submitted a review for this workplace.");
+            throw new HandleException(ErrorCode.REVIEW_ALREADY_EXISTS, "You have already submitted a review for this workplace.");
         }
 
         currentUser = userRepository.findById(currentUser.getId())

apps/jobboard-backend/src/main/java/org/bounswe/jobboardbackend/workplace/service/WorkplaceService.java

@@ -280,14 +280,17 @@ public void softDelete(Long id, User currentUser) {
 
     private Pageable buildSort(int page, int size, String sortBy) {
         if (sortBy == null || sortBy.isBlank()) {
-            return PageRequest.of(page, size, Sort.by(Sort.Order.asc("companyName").with(Sort.NullHandling.NULLS_LAST)));
+            return PageRequest.of(page, size, Sort.by("companyName"));
         }
 
         return switch (sortBy) {
-            case "nameDesc" -> PageRequest.of(page, size, Sort.by(Sort.Order.desc("companyName").with(Sort.NullHandling.NULLS_LAST)));
-            case "reviewCount" -> PageRequest.of(page, size, Sort.by(Sort.Order.desc("reviewCount").with(Sort.NullHandling.NULLS_LAST), Sort.Order.asc("companyName").with(Sort.NullHandling.NULLS_LAST)));
+            case "nameDesc" -> PageRequest.of(page, size, Sort.by(Sort.Direction.DESC, "companyName"));
+
+            case "reviewCount" -> PageRequest.of(page, size, Sort.by(Sort.Direction.DESC, "reviewCount").and(Sort.by("companyName")));
+
             case "rating" -> PageRequest.of(page, size);
-            default -> PageRequest.of(page, size, Sort.by(Sort.Order.asc("companyName").with(Sort.NullHandling.NULLS_LAST)));
+
+            default -> PageRequest.of(page, size, Sort.by("companyName"));
         };
     }