unpoly-rails 3.9.5 → 3.10.0

data/lib/unpoly/rails/change/field_definition.rb

@@ -97,12 +97,20 @@ module Unpoly
 
             define_method "write_#{method}_to_response_headers" do
               value = send(method)
-              stringified = field.stringify(value)
+              if Util.blank?(value)
+                # Blank values like [] have a present serialization ("[]".present? => true),
+                # so we must check the Ruby value here.
+                return
+              end
 
-              if stringified.present? # app servers don't like blank header values
-                header_name = send("#{method}_response_header_name")
-                response.headers[header_name] = stringified
+              stringified = field.stringify(value)
+              if Util.blank?(stringified)
+                # App servers don't like blank header values
+                return
               end
+
+              header_name = send("#{method}_response_header_name")
+              response.headers[header_name] = stringified
             end
 
           end

data/lib/unpoly/rails/change.rb

@@ -66,7 +66,11 @@ module Unpoly
       end
 
       def target_changed?
-        # The target has changed if either:
+        # The target is present if the original request contained an X-Up-Target header.
+        # That means it is also present if it was never changed on the server.
+        #
+        # To check whether the server has *changed* the target, we must look at either condition:
+        #
         # (1) The #target= setter was called, setting @server_target
         # (2) An up[target] param was set to preserve a previously changed target through a redirect.
         (@server_target && @server_target != target_from_request_headers) || !!target_from_params
@@ -256,10 +260,7 @@ module Unpoly
 
           write_expire_cache_to_response_headers
           write_evict_cache_to_response_headers
-
-          if context_changes.present?
-            write_context_changes_to_response_headers
-          end
+          write_context_changes_to_response_headers
 
           if target_changed?
             # Only write the target to the response if it has changed.
@@ -405,18 +406,32 @@ module Unpoly
       end
 
       def fields_as_params
-        params = {}
-
         # When the browser sees a redirect it will automatically resend the request headers
         # from the original request. This means that headers we set on the client (X-Up-Target, X-Up-Mode, etc.)
         # are automatically preserved through redirects.
         #
         # We still need to handle all the response headers that we set on the server.
         # There are encoded as params.
-        params[context_changes_param_name] = serialized_context_changes      # server-set header must be sent if present
-        params[events_param_name]          = serialized_events               # server-set header must be sent if present
-        params[expire_cache_param_name]    = serialized_expire_cache         # server-set header must be sent if present
-        params[evict_cache_param_name]     = serialized_evict_cache          # server-set header must be sent if present
+        params = {}
+
+        if Util.present?(context_changes)
+          # Don't send an empty hash as _up_context_changes=%7B%7D
+          params[context_changes_param_name] = serialized_context_changes
+        end
+
+        if Util.present?(events)
+          # Don't send an empty array as _up_events=%5B%5D
+          params[events_param_name] = serialized_events
+        end
+
+        if Util.present?(expire_cache)
+          # Note that we must support evict_cache == false for up.cache.keep
+          params[expire_cache_param_name] = serialized_expire_cache
+        end
+
+        if Util.present?(evict_cache)
+          params[evict_cache_param_name] = serialized_evict_cache
+        end
 
         if target_changed?
           # Only write the target to the response if it has changed.

data/lib/unpoly/rails/util.rb

@@ -37,6 +37,14 @@ module Unpoly
           unicode_string.gsub(/[[:^ascii:]]/) { |char| "\\u" + char.ord.to_s(16).rjust(4, "0") }
         end
 
+        def blank?(value)
+          value.blank? && value != false
+        end
+
+        def present?(value)
+          !blank?(value)
+        end
+
       end
     end
   end

data/lib/unpoly/rails/version.rb

@@ -3,6 +3,6 @@ module Unpoly
     ##
     # The current version of the unpoly-rails gem.
     # The first 3 digits should match the version of the Unpoly frontend code.
-    VERSION = '3.9.5'
+    VERSION = '3.10.0'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: unpoly-rails
 version: !ruby/object:Gem::Version
-  version: 3.9.5
+  version: 3.10.0
 platform: ruby
 authors:
 - Henning Koch
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-12-13 00:00:00.000000000 Z
+date: 2025-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: railties
@@ -157,7 +157,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.2.3
 signing_key: 
 specification_version: 4
 summary: Rails bindings for Unpoly, the unobtrusive JavaScript framework