fixed #374 Pixel-Layout: Add Task - Action Buttons

Author: thomaswoehlke

src/main/java/org/woehlke/java/simpleworklist/domain/ApplicationErrorController.java

@@ -20,70 +20,71 @@
 public class ApplicationErrorController implements ErrorController {
 
     @ExceptionHandler
-    @RequestMapping(path="", method={ GET, POST, PUT,  HEAD, PATCH, DELETE, OPTIONS, TRACE  })
+    @RequestMapping(path="", method={ GET, POST, PUT,  HEAD, PATCH, DELETE, OPTIONS, TRACE })
     public String handleError(
         HttpServletRequest request
     ) {
-        log.info("handleError");
         String errorMessage = (String) request.getAttribute(ERROR_MESSAGE);
         if(errorMessage!=null){
-            log.warn("errorMessage :"+errorMessage);
+            errorMessage = errorMessage.strip();
+            log.warn("handleError - errorMessage: "+errorMessage);
         }
         Integer statusCode = (Integer) request.getAttribute(ERROR_STATUS_CODE);
         if(statusCode != null){
             HttpStatus httpStatus = HttpStatus.valueOf(statusCode);
-            log.warn(httpStatus.value()+""+httpStatus.getReasonPhrase());
+            log.warn(httpStatus.value()+" "+httpStatus.getReasonPhrase());
+            String redirectLoginPageWithError = "redirect:/user/login?login_error=1";
             switch (httpStatus){
                 case NOT_FOUND:
                     log.warn("##################################################");
-                    log.warn("           404 NOT_FOUND");
+                    log.warn("#          404 NOT_FOUND                         #");
                     log.warn("##################################################");
                     return "error/error-404";
                 case INTERNAL_SERVER_ERROR:
                     log.warn("##################################################");
-                    log.warn("           500 INTERNAL_SERVER_ERROR");
+                    log.warn("#          500 INTERNAL_SERVER_ERROR             #");
                     log.warn("##################################################");
                     return "error/error-500";
                 case UNAUTHORIZED:
                     log.warn("##################################################");
-                    log.warn("           401 UNAUTHORIZED");
+                    log.warn("#          401 UNAUTHORIZED                      #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                    return redirectLoginPageWithError;
                 case METHOD_NOT_ALLOWED:
                     log.warn("##################################################");
-                    log.warn("          405 METHOD_NOT_ALLOWED");
+                    log.warn("#         405 METHOD_NOT_ALLOWED                 #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
                 case FORBIDDEN:
                     log.warn("##################################################");
-                    log.warn("          403 FORBIDDEN");
+                    log.warn("#         403 FORBIDDEN                          #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
                 case REQUEST_TIMEOUT:
                     log.warn("##################################################");
-                    log.warn("          408 REQUEST_TIMEOUT");
+                    log.warn("#         408 REQUEST_TIMEOUT                    #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
                 case CONFLICT:
                     log.warn("##################################################");
-                    log.warn("          409 CONFLICT");
+                    log.warn("#         409 CONFLICT                           #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
                 case PRECONDITION_FAILED:
                     log.warn("##################################################");
-                    log.warn("          412 PRECONDITION_FAILED");
+                    log.warn("#         412 PRECONDITION_FAILED                #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
                 case URI_TOO_LONG:
                     log.warn("##################################################");
-                    log.warn("          414 URI_TOO_LONG");
+                    log.warn("#         414 URI_TOO_LONG                       #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
                 case UNSUPPORTED_MEDIA_TYPE:
                     log.warn("##################################################");
-                    log.warn("          415 UNSUPPORTED_MEDIA_TYPE");
+                    log.warn("#         415 UNSUPPORTED_MEDIA_TYPE             #");
                     log.warn("##################################################");
-                    return "redirect:/user/login?login_error=1";
+                  return redirectLoginPageWithError;
             }
         }
         Throwable exception = (Throwable) request.getAttribute(ERROR_EXCEPTION);