Merge pull request #58 from TAMULib/sprint5-staging

Sprint 5 staging

Author: jcreel

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

@@ -41,21 +41,21 @@ public ApiResponse getById(@PathVariable Long id) {
     }
 
     @RequestMapping("/create")
-    @PreAuthorize("hasRole('SERVICE_MANAGER')")
+    @PreAuthorize("hasRole('WEB_MANAGER')")
     @WeaverValidation(business = { @WeaverValidation.Business(value = CREATE) })
     public ApiResponse create(@WeaverValidatedModel Note note, @WeaverCredentials Credentials credentials) throws UserNotFoundException {
         return new ApiResponse(SUCCESS, noteRepo.create(note, credentials));
     }
 
     @RequestMapping("/update")
-    @PreAuthorize("hasRole('SERVICE_MANAGER')")
+    @PreAuthorize("hasRole('WEB_MANAGER')")
     public ApiResponse update(@WeaverValidatedModel Note note) {
         return new ApiResponse(SUCCESS, noteRepo.update(note));
     }
 
     @Transactional
     @RequestMapping("/remove")
-    @PreAuthorize("hasRole('SERVICE_MANAGER')")
+    @PreAuthorize("hasRole('WEB_MANAGER')")
     public ApiResponse remove(@WeaverValidatedModel Note note) {
         noteRepo.delete(note);
         return new ApiResponse(SUCCESS);

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

@@ -40,21 +40,21 @@ public ApiResponse getById(@PathVariable Long id) {
     }
 
     @RequestMapping("/create")
-    @PreAuthorize("hasRole('WEB_MANAGER')")
+    @PreAuthorize("hasAnyRole('ADMIN','SERVICE_ADMIN','NOTICE_MANAGER')")
     @WeaverValidation(business = { @WeaverValidation.Business(value = CREATE) })
     public ApiResponse create(@WeaverValidatedModel Notification notification) {
         return new ApiResponse(SUCCESS, notificationRepo.create(notification));
     }
 
     @RequestMapping("/update")
-    @PreAuthorize("hasRole('WEB_MANAGER')")
+    @PreAuthorize("hasAnyRole('ADMIN','SERVICE_ADMIN','NOTICE_MANAGER')")
     @WeaverValidation(business = { @WeaverValidation.Business(value = UPDATE) })
     public ApiResponse update(@WeaverValidatedModel Notification notification) {
         return new ApiResponse(SUCCESS, notificationRepo.update(notification));
     }
 
     @RequestMapping("/remove")
-    @PreAuthorize("hasRole('WEB_MANAGER')")
+    @PreAuthorize("hasAnyRole('ADMIN','SERVICE_ADMIN','NOTICE_MANAGER')")
     @WeaverValidation(business = { @WeaverValidation.Business(value = DELETE) })
     public ApiResponse remove(@WeaverValidatedModel Notification notification) {
         notificationRepo.delete(notification);

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

@@ -17,6 +17,7 @@
 import edu.tamu.app.model.Service;
 import edu.tamu.app.model.repo.IdeaRepo;
 import edu.tamu.app.model.repo.ServiceRepo;
+import edu.tamu.app.model.request.FilteredPageRequest;
 import edu.tamu.app.model.request.IssueRequest;
 import edu.tamu.app.model.request.ServiceRequest;
 import edu.tamu.app.service.ProjectService;
@@ -52,6 +53,12 @@ public ApiResponse getPublicServices() {
         return new ApiResponse(SUCCESS, serviceRepo.findByIsPublicOrderByStatusDescNameAsc(true));
     }
 
+    @RequestMapping("/page")
+    @PreAuthorize("hasRole('ANONYMOUS')")
+    public ApiResponse page(@RequestBody FilteredPageRequest filteredPageRequest) {
+        return new ApiResponse(SUCCESS, serviceRepo.findAll(filteredPageRequest.getServiceSpecification(), filteredPageRequest.getPageRequest()));
+    }
+
     @RequestMapping("/{id}")
     @PreAuthorize("hasRole('ANONYMOUS')")
     public ApiResponse getService(@PathVariable Long id) {

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

@@ -78,7 +78,7 @@ public ApiResponse getUser(@WeaverUser User user) {
      * @throws Exception
      */
     @RequestMapping
-    @PreAuthorize("hasRole('WEB_MANAGER')")
+    @PreAuthorize("hasRole('ADMIN')")
     public ApiResponse allUsers() throws Exception {
         return new ApiResponse(SUCCESS, userRepo.findAll());
     }
@@ -90,7 +90,7 @@ public ApiResponse allUsers() throws Exception {
      * @throws Exception
      */
     @RequestMapping("/update")
-    @PreAuthorize("hasRole('WEB_MANAGER')")
+    @PreAuthorize("hasRole('ADMIN')")
     public ApiResponse updateUser(@RequestBody User user) throws Exception {
         user = userRepo.save(user);
         simpMessagingTemplate.convertAndSend("/channel/user", new ApiResponse(SUCCESS, userRepo.findAll()));

src/main/java/edu/tamu/app/enums/Role.java

@@ -4,6 +4,6 @@
 
 public enum Role implements IRole {
 
-    ROLE_ADMIN, ROLE_WEB_MANAGER, ROLE_SERVICE_MANAGER, ROLE_STAFF, ROLE_USER, ROLE_ANONYMOUS
+    ROLE_ADMIN, ROLE_SERVICE_ADMIN, ROLE_SERVICE_MANAGER, ROLE_WEB_MANAGER, ROLE_NOTICE_MANAGER, ROLE_STAFF, ROLE_USER, ROLE_ANONYMOUS
 
 }

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

@@ -2,6 +2,9 @@
 
 import java.util.List;
 
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.jpa.domain.Specification;
 import org.springframework.data.jpa.repository.JpaRepository;
 
 import edu.tamu.app.enums.Status;
@@ -10,6 +13,8 @@
 
 public interface ServiceRepo extends JpaRepository<Service, Long>, ServiceRepoCustom {
 
+    public Page<Service> findAll(Specification<Service> specification, Pageable pageable);
+
     public List<Service> findByIsPublicOrderByStatusDescNameAsc(Boolean isPublic);
 
     public List<Service> findByIsAuto(Boolean isAuto);

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

@@ -49,11 +49,18 @@ public Predicate toPredicate(Root<E> root, CriteriaQuery<?> query, CriteriaBuild
             }
         }
 
-        query.orderBy(cb.desc(root.get("lastModified")));
+        toPredicateDefaultQueryOrderBy(root, query, cb);
 
         return builder.build(cb);
     }
 
+    /**
+     * Allow implementing classes to control order by in case lastModified is non-existent.
+     */
+    protected void toPredicateDefaultQueryOrderBy(Root<E> root, CriteriaQuery<?> query, CriteriaBuilder cb) {
+        query.orderBy(cb.desc(root.get("lastModified")));
+    }
+
     private class PredicateBuilder {
 
         private final Map<String, List<Predicate>> predicates;

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

@@ -0,0 +1,19 @@
+package edu.tamu.app.model.repo.specification;
+
+import java.util.Map;
+
+import javax.persistence.criteria.CriteriaBuilder;
+import javax.persistence.criteria.CriteriaQuery;
+import javax.persistence.criteria.Root;
+
+public class ServiceSpecification<E> extends AbstractSpecification<E> {
+
+    public ServiceSpecification(Map<String, String[]> filters) {
+        super(filters);
+    }
+
+    @Override
+    protected void toPredicateDefaultQueryOrderBy(Root<E> root, CriteriaQuery<?> query, CriteriaBuilder cb) {
+        query.orderBy(cb.desc(root.get("name")));
+    }
+}

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

@@ -13,9 +13,11 @@
 import edu.tamu.app.model.FeatureProposal;
 import edu.tamu.app.model.Idea;
 import edu.tamu.app.model.Note;
+import edu.tamu.app.model.Service;
 import edu.tamu.app.model.repo.specification.FeatureProposalSpecification;
 import edu.tamu.app.model.repo.specification.IdeaSpecification;
 import edu.tamu.app.model.repo.specification.NoteSpecification;
+import edu.tamu.app.model.repo.specification.ServiceSpecification;
 
 public class FilteredPageRequest {
 
@@ -42,6 +44,11 @@ public IdeaSpecification<Idea> getIdeaSpecification() {
         return new IdeaSpecification<Idea>(filters);
     }
 
+    @JsonIgnore
+    public ServiceSpecification<Service> getServiceSpecification() {
+        return new ServiceSpecification<Service>(filters);
+    }
+
     @JsonIgnore
     public FeatureProposalSpecification<FeatureProposal> getFeatureProposalSpecification() {
         return new FeatureProposalSpecification<FeatureProposal>(filters);

src/main/java/edu/tamu/app/service/SystemMonitorService.java

@@ -48,7 +48,7 @@ public void updateAll() {
                 if (serviceStatus != service.getStatus()) {
                     logger.debug("Updating the status of [" + service.getName() + "] to: " + serviceStatus);
                     service.setStatus(serviceStatus);
-                    serviceRepo.save(service);
+                    serviceRepo.update(service);
                 }
             } catch (MalformedURLException e) {
                 logger.error("Did not check the status of [" + service.getName() + "] due to a malformed URL: " + service.getServiceUrl());