Revert "O3-3002: Queue Module - REST endpoints can be accessed withou…

…t authen…" (#72) This reverts commit a6f91d0.
openmrs · Apr 25, 2024 · c64ba23 · c64ba23
1 parent a6f91d0
commit c64ba23
Show file tree

Hide file tree

Showing 6 changed files with 3 additions and 373 deletions.
diff --git a/api/src/main/java/org/openmrs/module/queue/api/QueueEntryService.java b/api/src/main/java/org/openmrs/module/queue/api/QueueEntryService.java
@@ -17,14 +17,12 @@
 import org.openmrs.Location;
 import org.openmrs.Visit;
 import org.openmrs.VisitAttributeType;
-import org.openmrs.annotation.Authorized;
 import org.openmrs.api.APIException;
 import org.openmrs.module.queue.api.search.QueueEntrySearchCriteria;
 import org.openmrs.module.queue.api.sort.SortWeightGenerator;
 import org.openmrs.module.queue.model.Queue;
 import org.openmrs.module.queue.model.QueueEntry;
 import org.openmrs.module.queue.model.QueueEntryTransition;
-import org.openmrs.module.queue.utils.PrivilegeConstants;
 
 public interface QueueEntryService {
 
@@ -34,7 +32,6 @@ public interface QueueEntryService {
 	 * @param uuid uuid of the queue entry to be returned.
 	 * @return {@link org.openmrs.module.queue.model.QueueEntry}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ENTRIES })
 	Optional<QueueEntry> getQueueEntryByUuid(@NotNull String uuid);
 
 	/**
@@ -43,7 +40,6 @@ public interface QueueEntryService {
 	 * @param id queueEntryId - the id of the queue entry to retrieve.
 	 * @return {@link org.openmrs.module.queue.model.QueueEntry}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ENTRIES })
 	Optional<QueueEntry> getQueueEntryById(@NotNull Integer id);
 
 	/**
@@ -52,30 +48,27 @@ public interface QueueEntryService {
 	 * @param queueEntry the queue entry to be saved
 	 * @return saved {@link org.openmrs.module.queue.model.QueueEntry}
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ENTRIES })
 	QueueEntry saveQueueEntry(@NotNull QueueEntry queueEntry);
 
 	/**
 	 * Transitions a queue entry by ending one queue entry and creating a new queue entry that starts at
 	 * that time
-	 *
+	 * 
 	 * @param queueEntryTransition the queueEntryTransition
 	 * @return the new QueueEntry that is created
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ENTRIES })
 	QueueEntry transitionQueueEntry(@NotNull QueueEntryTransition queueEntryTransition);
 
 	/**
 	 * Undos a transition to the input queue entry by voiding it and making its previous queue entry
 	 * active by setting the previous entry's end time to null.
-	 *
+	 * 
 	 * @see QueueEntryService#getPreviousQueueEntry(QueueEntry)
 	 * @param queueEntry the queue entry to undo transition to. Must be active
 	 * @return the previous queue entry, re-activated
 	 * @throws IllegalArgumentException if the previous queue entry does not exist
 	 * @throws IllegalStateException if multiple previous entries are identified
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ENTRIES })
 	QueueEntry undoTransition(@NotNull QueueEntry queueEntry);
 
 	/**
@@ -84,7 +77,6 @@ public interface QueueEntryService {
 	 * @param queueEntry the queue entry to be voided
 	 * @param voidReason the reason for voiding the queue entry
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ENTRIES })
 	void voidQueueEntry(@NotNull QueueEntry queueEntry, String voidReason);
 
 	/**
@@ -93,35 +85,30 @@ public interface QueueEntryService {
 	 * @param queueEntry queue entry to be deleted
 	 * @throws org.openmrs.api.APIException
 	 */
-	@Authorized({ PrivilegeConstants.PURGE_QUEUE_ENTRIES })
 	void purgeQueueEntry(@NotNull QueueEntry queueEntry) throws APIException;
 
 	/**
 	 * @return {@link List} of queue entries that match the given %{@link QueueEntrySearchCriteria}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ENTRIES })
 	List<QueueEntry> getQueueEntries(@NotNull QueueEntrySearchCriteria searchCriteria);
 
 	/**
 	 * @return {@link Long} count of queue entries that match the given
 	 *         %{@link QueueEntrySearchCriteria}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ENTRIES })
 	Long getCountOfQueueEntries(@NotNull QueueEntrySearchCriteria searchCriteria);
 
 	/**
 	 * @param location
 	 * @param queue
 	 * @return VisitQueueNumber - used to identify patients in the queue instead of using patient name
 	 */
-	@Authorized({ org.openmrs.util.PrivilegeConstants.ADD_VISITS, org.openmrs.util.PrivilegeConstants.EDIT_VISITS })
 	String generateVisitQueueNumber(@NotNull Location location, @NotNull Queue queue, @NotNull Visit visit,
 	        @NotNull VisitAttributeType visitAttributeType);
 
 	/**
 	 * Closes all active queue entries
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ENTRIES })
 	void closeActiveQueueEntries();
 
 	/**
@@ -148,6 +135,5 @@ String generateVisitQueueNumber(@NotNull Location location, @NotNull Queue queue
 	 * @return the previous queue entry, null otherwise.
 	 * @throws IllegalStateException if multiple previous queue entries are identified
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ENTRIES })
 	QueueEntry getPreviousQueueEntry(@NotNull QueueEntry queueEntry);
 }
diff --git a/api/src/main/java/org/openmrs/module/queue/api/QueueRoomService.java b/api/src/main/java/org/openmrs/module/queue/api/QueueRoomService.java
@@ -14,73 +14,23 @@
 import java.util.List;
 import java.util.Optional;
 
-import org.openmrs.annotation.Authorized;
 import org.openmrs.api.APIException;
 import org.openmrs.module.queue.api.search.QueueRoomSearchCriteria;
 import org.openmrs.module.queue.model.QueueRoom;
-import org.openmrs.module.queue.utils.PrivilegeConstants;
 
 public interface QueueRoomService {
 
-	/**
-	 * Gets a queue room given a uuid.
-	 *
-	 * @param uuid the uuid of the queue room to be returned.
-	 * @return {@link org.openmrs.module.queue.model.QueueRoom}
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	Optional<QueueRoom> getQueueRoomByUuid(@NotNull String uuid);
 
-	/**
-	 * Gets a queue room by id.
-	 *
-	 * @param id the id of the queue room to retrieve.
-	 * @return {@link org.openmrs.module.queue.model.QueueRoom}
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	Optional<QueueRoom> getQueueRoomById(@NotNull int id);
 
-	/**
-	 * Gets a List of all Queue Rooms.
-	 *
-	 * @return {@link List} of all queue rooms
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	List<QueueRoom> getAllQueueRooms();
 
-	/**
-	 * Saves a queue room
-	 *
-	 * @param queueRoom the queue room to be saved
-	 * @return saved {@link org.openmrs.module.queue.model.QueueRoom}
-	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ROOMS })
 	QueueRoom saveQueueRoom(@NotNull QueueRoom queueRoom);
 
-	/**
-	 * Gets a List of all Queue Rooms that match the given QueueRoomSearchCriteria.
-	 *
-	 * @return {@link List} of queue rooms that match the given
-	 *         {@link org.openmrs.module.queue.api.search.QueueRoomSearchCriteria}
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	List<QueueRoom> getQueueRooms(QueueRoomSearchCriteria searchCriteria);
 
-	/**
-	 * Retires a queue room.
-	 *
-	 * @param queueRoom the queue room to retire
-	 * @param retireReason the reason for retiring the queue room
-	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ROOMS })
-	void retireQueueRoom(@NotNull QueueRoom queueRoom, String retireReason);
+	void retireQueueRoom(@NotNull QueueRoom queueRoom, String voidReason);
 
-	/**
-	 * Completely remove a queue room from the database
-	 *
-	 * @param queueRoom queue room to be deleted
-	 * @throws org.openmrs.api.APIException
-	 */
-	@Authorized({ PrivilegeConstants.PURGE_QUEUE_ROOMS })
 	void purgeQueueRoom(@NotNull QueueRoom queueRoom) throws APIException;
 }
diff --git a/api/src/main/java/org/openmrs/module/queue/api/QueueService.java b/api/src/main/java/org/openmrs/module/queue/api/QueueService.java
@@ -14,11 +14,9 @@
 import java.util.List;
 import java.util.Optional;
 
-import org.openmrs.annotation.Authorized;
 import org.openmrs.api.APIException;
 import org.openmrs.module.queue.api.search.QueueSearchCriteria;
 import org.openmrs.module.queue.model.Queue;
-import org.openmrs.module.queue.utils.PrivilegeConstants;
 
 /**
  * This interface defines methods for Queue objects
@@ -31,7 +29,6 @@ public interface QueueService {
 	 * @param uuid uuid of the queue to be returned.
 	 * @return {@link org.openmrs.module.queue.model.Queue}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUES })
 	Optional<Queue> getQueueByUuid(@NotNull String uuid);
 
 	/**
@@ -40,7 +37,6 @@ public interface QueueService {
 	 * @param id queueId - the id of the queue to retrieve.
 	 * @return {@link org.openmrs.module.queue.model.Queue}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUES })
 	Optional<Queue> getQueueById(@NotNull Integer id);
 
 	/**
@@ -49,7 +45,6 @@ public interface QueueService {
 	 * @param queue the queue to be saved
 	 * @return saved {@link org.openmrs.module.queue.model.Queue}
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUES })
 	Queue createQueue(@NotNull Queue queue);
 
 	/**
@@ -58,19 +53,16 @@ public interface QueueService {
 	 * @param queue the queue to be saved
 	 * @return saved {@link org.openmrs.module.queue.model.Queue}
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUES })
 	Queue saveQueue(@NotNull Queue queue);
 
 	/**
 	 * @return all queues
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUES })
 	List<Queue> getAllQueues();
 
 	/**
 	 * @return {@link List} of queues that match the given %{@link QueueSearchCriteria}
 	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUES })
 	List<Queue> getQueues(@NotNull QueueSearchCriteria searchCriteria);
 
 	/**
@@ -79,7 +71,6 @@ public interface QueueService {
 	 * @param queue the queue to retire
 	 * @param retireReason the reason for voiding the queue
 	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUES })
 	void retireQueue(@NotNull Queue queue, String retireReason);
 
 	/**
@@ -88,6 +79,5 @@ public interface QueueService {
 	 * @param queue queue to be deleted
 	 * @throws APIException <strong>Should</strong> delete the given queue from the database
 	 */
-	@Authorized({ PrivilegeConstants.PURGE_QUEUES })
 	void purgeQueue(@NotNull Queue queue) throws APIException;
 }
diff --git a/api/src/main/java/org/openmrs/module/queue/api/RoomProviderMapService.java b/api/src/main/java/org/openmrs/module/queue/api/RoomProviderMapService.java
@@ -14,74 +14,24 @@
 import java.util.List;
 import java.util.Optional;
 
-import org.openmrs.annotation.Authorized;
 import org.openmrs.api.APIException;
 import org.openmrs.module.queue.api.search.RoomProviderMapSearchCriteria;
 import org.openmrs.module.queue.model.RoomProviderMap;
-import org.openmrs.module.queue.utils.PrivilegeConstants;
 
 public interface RoomProviderMapService {
 
-	/**
-	 * Gets a room provider map given a uuid.
-	 *
-	 * @param uuid the uuid of the room provider map to be returned.
-	 * @return {@link org.openmrs.module.queue.model.RoomProviderMap}
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	Optional<RoomProviderMap> getRoomProviderMapByUuid(@NotNull String uuid);
 
-	/**
-	 * Gets a room provider map by id.
-	 *
-	 * @param id the id of the room provider map to retrieve.
-	 * @return {@link org.openmrs.module.queue.model.RoomProviderMap}
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	Optional<RoomProviderMap> getRoomProviderMapById(@NotNull int id);
 
-	/**
-	 * Saves a room provider map
-	 *
-	 * @param roomProviderMap the room provider map to be saved
-	 * @return saved {@link org.openmrs.module.queue.model.RoomProviderMap}
-	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ROOMS })
 	RoomProviderMap saveRoomProviderMap(@NotNull RoomProviderMap roomProviderMap);
 
-	/**
-	 * Gets a List of all Room Provider Maps.
-	 *
-	 * @return {@link List} of all room provider maps
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	List<RoomProviderMap> getAllRoomProviderMaps();
 
-	/**
-	 * Gets a List of all Room Provider Maps that match the given RoomProviderMapSearchCriteria.
-	 *
-	 * @return {@link List} of room provider maps that match the given
-	 *         {@link org.openmrs.module.queue.api.search.RoomProviderMapSearchCriteria}
-	 */
-	@Authorized({ PrivilegeConstants.GET_QUEUE_ROOMS })
 	List<RoomProviderMap> getRoomProviderMaps(RoomProviderMapSearchCriteria searchCriteria);
 
-	/**
-	 * Voids a room provider map
-	 *
-	 * @param roomProviderMap the room provider map to void
-	 * @param voidReason the reason for voiding the room provider map
-	 */
-	@Authorized({ PrivilegeConstants.MANAGE_QUEUE_ROOMS })
 	void voidRoomProviderMap(@NotNull RoomProviderMap roomProviderMap, String voidReason);
 
-	/**
-	 * Completely remove a room provider map from the database
-	 *
-	 * @param roomProviderMap room provider map to be deleted
-	 * @throws org.openmrs.api.APIException
-	 */
-	@Authorized({ PrivilegeConstants.PURGE_QUEUE_ROOMS })
 	void purgeRoomProviderMap(@NotNull RoomProviderMap roomProviderMap) throws APIException;
 
 }
diff --git a/api/src/main/java/org/openmrs/module/queue/utils/PrivilegeConstants.java b/api/src/main/java/org/openmrs/module/queue/utils/PrivilegeConstants.java