fix: only append ?v to asset links that have no query params

Agent-Logs-Url: https://github.com/halo-dev/halo/sessions/7742a189-6a24-448d-8614-a276ebddb8d5

Co-authored-by: ruibaby <21301288+ruibaby@users.noreply.github.com>

Author: Copilot

application/src/main/java/run/halo/app/theme/ThemeLinkBuilder.java

@@ -40,7 +40,7 @@ protected String processLink(IExpressionContext context, String link) {
             var path = PathUtils.combinePath(THEME_PREVIEW_PREFIX, theme.getName(), link);
             var uriComponents = UriComponentsBuilder.fromUriString(path).build();
             if (StringUtils.isNotBlank(theme.getVersion())
-                && uriComponents.getQueryParams().get("v") == null) {
+                && uriComponents.getQueryParams().isEmpty()) {
                 return UriComponentsBuilder.fromUriString(path)
                     .queryParam("v", theme.getVersion())
                     .build().toString();

application/src/test/java/run/halo/app/theme/ThemeLinkBuilderTest.java

@@ -82,14 +82,20 @@ void processAssetsLinkWithoutVersion() {
     }
 
     @Test
-    void processAssetsLinkWithExistingVersionParam() {
-        // link that already has a v= parameter should not get another one appended
+    void processAssetsLinkWithExistingQueryParams() {
+        // link that already has any query parameter should not get ?v appended
         ThemeLinkBuilder themeLinkBuilder =
             new ThemeLinkBuilder(getTheme(true), externalUrlSupplier);
 
+        // already has v param
         String link = "/assets/css/style.css?v=custom";
         String processed = themeLinkBuilder.processLink(null, link);
         assertThat(processed).isEqualTo("/themes/test-theme/assets/css/style.css?v=custom");
+
+        // has a different custom query param the theme itself added
+        link = "/assets/css/style.css?foo=bar";
+        processed = themeLinkBuilder.processLink(null, link);
+        assertThat(processed).isEqualTo("/themes/test-theme/assets/css/style.css?foo=bar");
     }
 
     @Test