Get rid of deprecated classes and methods

wimdeblauwe · Nov 25, 2024 · 489d84f · 489d84f
1 parent be7da3a
commit 489d84f
Show file tree

Hide file tree

Showing 25 changed files with 45 additions and 2,369 deletions.
diff --git a/...boot/src/main/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxHandlerInterceptor.java b/...boot/src/main/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxHandlerInterceptor.java
@@ -31,31 +31,16 @@ public void afterCompletion(HttpServletRequest request, HttpServletResponse resp
 
         HtmxResponse htmxResponse = RequestContextUtils.getHtmxResponse(request);
         if (htmxResponse != null) {
-            addHxTriggerHeaders(response, HtmxResponseHeader.HX_TRIGGER, htmxResponse.getTriggersInternal());
-            addHxTriggerHeaders(response, HtmxResponseHeader.HX_TRIGGER_AFTER_SETTLE, htmxResponse.getTriggersAfterSettleInternal());
-            addHxTriggerHeaders(response, HtmxResponseHeader.HX_TRIGGER_AFTER_SWAP, htmxResponse.getTriggersAfterSwapInternal());
-
-            if (htmxResponse.getLocation() != null) {
-                HtmxLocation location = htmxResponse.getLocation();
-                if (location.hasContextData()) {
-                    location.setPath(RequestContextUtils.createUrl(request, location.getPath(), htmxResponse.isContextRelative()));
-                    setHeaderJsonValue(response, HtmxResponseHeader.HX_LOCATION, location);
-                } else {
-                    response.setHeader(HtmxResponseHeader.HX_LOCATION.getValue(), RequestContextUtils.createUrl(request, location.getPath(), htmxResponse.isContextRelative()));
-                }
-            }
+            addHxTriggerHeaders(response, HtmxResponseHeader.HX_TRIGGER, htmxResponse.getTriggers());
+            addHxTriggerHeaders(response, HtmxResponseHeader.HX_TRIGGER_AFTER_SETTLE, htmxResponse.getTriggersAfterSettle());
+            addHxTriggerHeaders(response, HtmxResponseHeader.HX_TRIGGER_AFTER_SWAP, htmxResponse.getTriggersAfterSwap());
+
             if (htmxResponse.getReplaceUrl() != null) {
                 response.setHeader(HtmxResponseHeader.HX_REPLACE_URL.getValue(), RequestContextUtils.createUrl(request, htmxResponse.getReplaceUrl(), htmxResponse.isContextRelative()));
             }
             if (htmxResponse.getPushUrl() != null) {
                 response.setHeader(HtmxResponseHeader.HX_PUSH_URL.getValue(), RequestContextUtils.createUrl(request, htmxResponse.getPushUrl(), htmxResponse.isContextRelative()));
             }
-            if (htmxResponse.getRedirect() != null) {
-                response.setHeader(HtmxResponseHeader.HX_REDIRECT.getValue(), RequestContextUtils.createUrl(request, htmxResponse.getRedirect(), htmxResponse.isContextRelative()));
-            }
-            if (htmxResponse.isRefresh()) {
-                response.setHeader(HtmxResponseHeader.HX_REFRESH.getValue(), "true");
-            }
             if (htmxResponse.getRetarget() != null) {
                 response.setHeader(HtmxResponseHeader.HX_RETARGET.getValue(), htmxResponse.getRetarget());
             }

diff --git a/...n/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxHandlerMethodAnnotationHandler.java b/...n/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxHandlerMethodAnnotationHandler.java
@@ -5,13 +5,10 @@
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
 import org.springframework.core.annotation.AnnotatedElementUtils;
-import org.springframework.http.HttpHeaders;
 
 import java.lang.reflect.Method;
 import java.time.Duration;
 
-import static io.github.wimdeblauwe.htmx.spring.boot.mvc.HtmxResponseHeader.*;
-
 /**
  * A handler for processing htmx annotations present on exception handler methods.
  *
@@ -56,56 +53,56 @@ private void setHxPushUrl(HttpServletRequest request, HttpServletResponse respon
         HxPushUrl methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxPushUrl.class);
         if (methodAnnotation != null) {
             if (HtmxValue.TRUE.equals(methodAnnotation.value())) {
-                setHeader(response, HX_PUSH_URL, getRequestUrl(request));
+                setHeader(response, HtmxResponseHeader.HX_PUSH_URL, getRequestUrl(request));
             } else {
-                setHeader(response, HX_PUSH_URL, RequestContextUtils.createUrl(request, methodAnnotation.value(), methodAnnotation.contextRelative()));
+                setHeader(response, HtmxResponseHeader.HX_PUSH_URL, RequestContextUtils.createUrl(request, methodAnnotation.value(), methodAnnotation.contextRelative()));
             }
         }
     }
 
     private void setHxRedirect(HttpServletRequest request, HttpServletResponse response, Method method) {
         HxRedirect methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxRedirect.class);
         if (methodAnnotation != null) {
-            setHeader(response, HX_REDIRECT, RequestContextUtils.createUrl(request, methodAnnotation.value(), methodAnnotation.contextRelative()));
+            setHeader(response, HtmxResponseHeader.HX_REDIRECT, RequestContextUtils.createUrl(request, methodAnnotation.value(), methodAnnotation.contextRelative()));
         }
     }
 
     private void setHxReplaceUrl(HttpServletRequest request, HttpServletResponse response, Method method) {
         HxReplaceUrl methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxReplaceUrl.class);
         if (methodAnnotation != null) {
             if (HtmxValue.TRUE.equals(methodAnnotation.value())) {
-                setHeader(response, HX_REPLACE_URL, getRequestUrl(request));
+                setHeader(response, HtmxResponseHeader.HX_REPLACE_URL, getRequestUrl(request));
             } else {
-                setHeader(response, HX_REPLACE_URL, RequestContextUtils.createUrl(request, methodAnnotation.value(), methodAnnotation.contextRelative()));
+                setHeader(response, HtmxResponseHeader.HX_REPLACE_URL, RequestContextUtils.createUrl(request, methodAnnotation.value(), methodAnnotation.contextRelative()));
             }
         }
     }
 
     private void setHxReswap(HttpServletResponse response, Method method) {
         HxReswap methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxReswap.class);
         if (methodAnnotation != null) {
-            setHeader(response, HX_RESWAP, convertToReswap(methodAnnotation));
+            setHeader(response, HtmxResponseHeader.HX_RESWAP, convertToReswap(methodAnnotation));
         }
     }
 
     private void setHxRetarget(HttpServletResponse response, Method method) {
         HxRetarget methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxRetarget.class);
         if (methodAnnotation != null) {
-            setHeader(response, HX_RETARGET, methodAnnotation.value());
+            setHeader(response, HtmxResponseHeader.HX_RETARGET, methodAnnotation.value());
         }
     }
 
     private void setHxReselect(HttpServletResponse response, Method method) {
         HxReselect methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxReselect.class);
         if (methodAnnotation != null) {
-            setHeader(response, HX_RESELECT, methodAnnotation.value());
+            setHeader(response, HtmxResponseHeader.HX_RESELECT, methodAnnotation.value());
         }
     }
 
     private void setHxTrigger(HttpServletResponse response, Method method) {
         HxTrigger methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxTrigger.class);
         if (methodAnnotation != null) {
-            setHeader(response, convertToHeader(methodAnnotation.lifecycle()), methodAnnotation.value());
+            setHeader(response, HtmxResponseHeader.HX_TRIGGER, methodAnnotation.value());
         }
     }
 
@@ -126,20 +123,7 @@ private void setHxTriggerAfterSwap(HttpServletResponse response, Method method)
     private void setHxRefresh(HttpServletResponse response, Method method) {
         HxRefresh methodAnnotation = AnnotatedElementUtils.findMergedAnnotation(method, HxRefresh.class);
         if (methodAnnotation != null) {
-            setHeader(response, HX_REFRESH, HtmxValue.TRUE);
-        }
-    }
-
-    private HtmxResponseHeader convertToHeader(HxTriggerLifecycle lifecycle) {
-        switch (lifecycle) {
-            case RECEIVE:
-                return HX_TRIGGER;
-            case SETTLE:
-                return HX_TRIGGER_AFTER_SETTLE;
-            case SWAP:
-                return HX_TRIGGER_AFTER_SWAP;
-            default:
-                throw new IllegalArgumentException("Unknown lifecycle:" + lifecycle);
+            setHeader(response, HtmxResponseHeader.HX_REFRESH, HtmxValue.TRUE);
         }
     }
 

diff --git a/...in/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxHandlerMethodArgumentResolver.java b/...in/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxHandlerMethodArgumentResolver.java
@@ -9,10 +9,6 @@
 import org.springframework.web.method.support.HandlerMethodArgumentResolver;
 import org.springframework.web.method.support.ModelAndViewContainer;
 
-import java.util.Objects;
-
-import static io.github.wimdeblauwe.htmx.spring.boot.mvc.HtmxRequestHeader.*;
-
 public class HtmxHandlerMethodArgumentResolver implements HandlerMethodArgumentResolver {
 
     @Override

diff --git a/...ot/src/main/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxMvcAutoConfiguration.java b/...ot/src/main/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxMvcAutoConfiguration.java
@@ -63,7 +63,6 @@ public void addArgumentResolvers(List<HandlerMethodArgumentResolver> resolvers)
 
     @Override
     public void addReturnValueHandlers(List<HandlerMethodReturnValueHandler> handlers) {
-        handlers.add(new HtmxResponseHandlerMethodReturnValueHandler(viewResolverObjectFactory.getObject(), localeResolverObjectFactory, objectMapper));
         handlers.add(new HtmxViewMethodReturnValueHandler(viewResolverObjectFactory.getObject(), localeResolverObjectFactory.getObject()));
     }
 

diff --git a/htmx-spring-boot/src/main/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxRequest.java b/htmx-spring-boot/src/main/java/io/github/wimdeblauwe/htmx/spring/boot/mvc/HtmxRequest.java
@@ -1,10 +1,10 @@
 package io.github.wimdeblauwe.htmx.spring.boot.mvc;
 
-import static io.github.wimdeblauwe.htmx.spring.boot.mvc.HtmxRequestHeader.*;
-
 import jakarta.servlet.http.HttpServletRequest;
 import org.springframework.lang.Nullable;
 
+import static io.github.wimdeblauwe.htmx.spring.boot.mvc.HtmxRequestHeader.*;
+
 /**
  * This class can be used as a controller method argument to access
  * the <a href="https://htmx.org/reference/#request_headers">htmx Request Headers</a>.
@@ -188,98 +188,36 @@ public static final class Builder {
         private Builder() {
         }
 
-        /**
-         * @deprecated use {@link #boosted(boolean)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withBoosted(boolean boosted) {
-            return boosted(boosted);
-        }
-
         public Builder boosted(boolean boosted) {
             this.boosted = boosted;
             return this;
         }
 
-        /**
-         * @deprecated use {@link #currentUrl(String)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withCurrentUrl(String currentUrl) {
-            this.currentUrl = currentUrl;
-            return this;
-        }
-
         public Builder currentUrl(String currentUrl) {
             this.currentUrl = currentUrl;
             return this;
         }
 
-        /**
-         * @deprecated use {@link #historyRestoreRequest(boolean)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withHistoryRestoreRequest(boolean historyRestoreRequest) {
-            this.historyRestoreRequest = historyRestoreRequest;
-            return this;
-        }
-
         public Builder historyRestoreRequest(boolean historyRestoreRequest) {
             this.historyRestoreRequest = historyRestoreRequest;
             return this;
         }
 
-        /**
-         * @deprecated use {@link #promptResponse(String)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withPromptResponse(String promptResponse) {
-            this.promptResponse = promptResponse;
-            return this;
-        }
-
         public Builder promptResponse(String promptResponse) {
             this.promptResponse = promptResponse;
             return this;
         }
 
-        /**
-         * @deprecated use {@link #target(String)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withTarget(String target) {
-            this.target = target;
-            return this;
-        }
-
         public Builder target(String target) {
             this.target = target;
             return this;
         }
 
-        /**
-         * @deprecated use {@link #triggerName(String)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withTriggerName(String triggerName) {
-            this.triggerName = triggerName;
-            return this;
-        }
-
         public Builder triggerName(String triggerName) {
             this.triggerName = triggerName;
             return this;
         }
 
-        /**
-         * @deprecated use {@link #triggerId(String)} instead. Will be removed in 4.0.
-         */
-        @Deprecated
-        public Builder withTriggerId(String triggerId) {
-            this.triggerId = triggerId;
-            return this;
-        }
-
         public Builder triggerId(String triggerId) {
             this.triggerId = triggerId;
             return this;