using note specification for advanced paging queries

Author: tamu-sad-iii

src/main/java/edu/tamu/app/controller/NoteController.java

@@ -9,13 +9,11 @@
 import org.springframework.web.bind.annotation.RestController;
 
 import edu.tamu.app.model.Note;
-import edu.tamu.app.model.Service;
 import edu.tamu.app.model.repo.NoteRepo;
 import edu.tamu.app.model.request.FilteredPageRequest;
 import edu.tamu.framework.aspect.annotation.ApiCredentials;
 import edu.tamu.framework.aspect.annotation.ApiData;
 import edu.tamu.framework.aspect.annotation.ApiMapping;
-import edu.tamu.framework.aspect.annotation.ApiModel;
 import edu.tamu.framework.aspect.annotation.ApiValidatedModel;
 import edu.tamu.framework.aspect.annotation.ApiValidation;
 import edu.tamu.framework.aspect.annotation.ApiVariable;
@@ -30,14 +28,10 @@ public class NoteController {
     @Autowired
     private NoteRepo noteRepo;
 
-    @ApiMapping("/by-service/{pinned}")
+    @ApiMapping("/query")
     @Auth(role = "ROLE_ANONYMOUS")
-    public ApiResponse getAllNotesByService(@ApiModel Service service, @ApiVariable String pinned) {
-        if (Boolean.valueOf(pinned)) {
-            return new ApiResponse(SUCCESS, noteRepo.findAllByServiceAndPinnedTrue(service));
-        } else {
-            return new ApiResponse(SUCCESS, noteRepo.findAllByService(service));
-        }
+    public ApiResponse getAllNotesByService(@ApiData FilteredPageRequest filteredPageRequest) {
+        return new ApiResponse(SUCCESS, noteRepo.findAll(filteredPageRequest.getSpecification(), filteredPageRequest.getPageRequest()));
     }
 
     @ApiMapping("/{id}")
@@ -71,7 +65,7 @@ public ApiResponse remove(@ApiValidatedModel Note note) {
     @ApiMapping("/page")
     @Auth(role = "ROLE_ANONYMOUS")
     public ApiResponse page(@ApiData FilteredPageRequest filteredPageRequest) {
-        return new ApiResponse(SUCCESS, noteRepo.findAll(filteredPageRequest.toPageRequest()));
+        return new ApiResponse(SUCCESS, noteRepo.findAll(filteredPageRequest.getPageRequest()));
     }
 
 }

src/main/java/edu/tamu/app/model/repo/NoteRepo.java

@@ -9,18 +9,19 @@
 import org.springframework.data.jpa.repository.JpaSpecificationExecutor;
 
 import edu.tamu.app.model.Note;
-import edu.tamu.app.model.Service;
 import edu.tamu.app.model.repo.custom.NoteRepoCustom;
 
 public interface NoteRepo extends JpaRepository<Note, Long>, NoteRepoCustom, JpaSpecificationExecutor<Note> {
 
     public Page<Note> findAll(Specification<Note> specification, Pageable pageable);
 
+    public List<Note> findAll(Specification<Note> specification);
+
     public Page<Note> findAll(Pageable pageable);
 
-    public List<Note> findAllByService(Service service);
+    public List<Note> findAllByServiceId(Long id);
 
-    public List<Note> findAllByServiceAndPinnedTrue(Service service);
+    public List<Note> findAllByServiceIdAndPinnedTrue(Long id);
 
     public void delete(Note note);
 

src/main/java/edu/tamu/app/model/repo/impl/ServiceRepoImpl.java

@@ -45,7 +45,7 @@ public Service update(Service service) {
 
     @Override
     public void delete(Service service) {
-        noteRepo.findAllByService(service).parallelStream().forEach(note -> {
+        noteRepo.findAllByServiceId(service.getId()).parallelStream().forEach(note -> {
             noteRepo.delete(note);
         });
         serviceRepo.delete(service.getId());

src/main/java/edu/tamu/app/model/repo/specification/NoteSpecification.java

@@ -13,7 +13,6 @@
 
 public class NoteSpecification<E> implements Specification<E> {
 
-    @SuppressWarnings("unused")
     private Map<String, String[]> filters;
 
     public NoteSpecification(Map<String, String[]> filters) {
@@ -22,8 +21,32 @@ public NoteSpecification(Map<String, String[]> filters) {
 
     @Override
     public Predicate toPredicate(Root<E> root, CriteriaQuery<?> query, CriteriaBuilder cb) {
-        List<Predicate> datePredicates = new ArrayList<Predicate>();
-        return cb.and(datePredicates.toArray(new Predicate[datePredicates.size()]));
+
+        List<Predicate> pinnedPredicates = new ArrayList<Predicate>();
+        List<Predicate> servicePredicates = new ArrayList<Predicate>();
+
+        for (Map.Entry<String, String[]> entry : filters.entrySet()) {
+            String key = entry.getKey();
+            String[] values = entry.getValue();
+
+            switch (key) {
+            case "pinned":
+                for (String value : values) {
+                    pinnedPredicates.add(cb.like(cb.lower(root.get(key).as(String.class)), "%" + value.toLowerCase() + "%"));
+                }
+                break;
+            case "service":
+                for (String value : values) {
+                    servicePredicates.add(cb.like(cb.lower(root.get(key).get("id").as(String.class)), "%" + value.toLowerCase() + "%"));
+                }
+                break;
+            default:
+                break;
+            }
+
+        }
+
+        return cb.and(cb.and(pinnedPredicates.toArray(new Predicate[pinnedPredicates.size()])), cb.and(servicePredicates.toArray(new Predicate[servicePredicates.size()])));
     }
 
 }

src/main/java/edu/tamu/app/model/request/FilteredPageRequest.java

@@ -6,6 +6,11 @@
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Sort;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
+
+import edu.tamu.app.model.Note;
+import edu.tamu.app.model.repo.specification.NoteSpecification;
+
 public class FilteredPageRequest {
 
     private int pageNumber;
@@ -22,7 +27,13 @@ public FilteredPageRequest() {
 
     }
 
-    public PageRequest toPageRequest() {
+    @JsonIgnore
+    public NoteSpecification<Note> getSpecification() {
+        return new NoteSpecification<Note>(filters);
+    }
+
+    @JsonIgnore
+    public PageRequest getPageRequest() {
         return new PageRequest(pageNumber - 1, pageSize, new Sort(Sort.Direction.fromString(direction), properties));
     }