josehernandezfintecheandomx commented on code in PR #2308:
URL: https://github.com/apache/fineract/pull/2308#discussion_r864852644


##########
fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/config/FineractProperties.java:
##########
@@ -60,5 +60,25 @@ public static class FineractModeProperties {
         public boolean isReadOnlyMode() {
             return readEnabled && !writeEnabled && !batchEnabled;
         }
+
+        public boolean isWriteOnlyMode() {
+            return !readEnabled && writeEnabled && !batchEnabled;
+        }
+
+        public boolean isBatchOnlyMode() {
+            return !readEnabled && !writeEnabled && batchEnabled;
+        }
+
+        public boolean isReadInstance() {

Review Comment:
   Due mixing the setup, you could have Read only or Read/Write or all and in 
all cases the instance is a Read instance



##########
fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/exceptionmapper/InvalidTenantIdentifierExceptionMapper.java:
##########
@@ -42,7 +42,7 @@ public class InvalidTenantIdentifierExceptionMapper 
implements ExceptionMapper<I
 
     @Override
     public Response toResponse(@SuppressWarnings("unused") final 
InvalidTenantIdentiferException e) {
-        return 
Response.status(Status.UNAUTHORIZED).entity(ApiGlobalErrorResponse.invalidTenantIdentifier())
+        return 
Response.status(Status.METHOD_NOT_ALLOWED).entity(ApiGlobalErrorResponse.invalidTenantIdentifier())

Review Comment:
   updated



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to