Fixes #336

Author: bogdan

CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [2.3.7]
+
+* Fixed serialization of empty `Array` and empty `Hash`. Fixes [#336](https://github.com/railsware/js-routes/issues/336).
+
+``` javascript
+blog_path({filters: {}, columns: []}) // => /blog
+```
+
 ## [2.3.6]
 
 * Support new Rails 8.1 nil parameter serialization.

lib/routes.js

@@ -140,7 +140,10 @@ RubyVariables.WRAPPER(
                     if (prefix) {
                         key = prefix + "[" + key + "]";
                     }
-                    result.push(this.default_serializer(prop, key));
+                    const subvalue = this.default_serializer(prop, key);
+                    if (subvalue.length) {
+                        result.push(subvalue);
+                    }
                 }
             }
             else {

lib/routes.ts

@@ -263,7 +263,10 @@ RubyVariables.WRAPPER(
             if (prefix) {
               key = prefix + "[" + key + "]";
             }
-            result.push(this.default_serializer(prop, key));
+            const subvalue = this.default_serializer(prop, key);
+            if (subvalue.length) {
+              result.push(subvalue);
+            }
           }
         } else {
           result.push(

spec/js_routes/rails_routes_compatibility_spec.rb

@@ -191,6 +191,10 @@
       expectjs("Routes.inbox_path(1, {hello: ['world', 'mars']})").to eq(test_routes.inbox_path(1, :hello => [:world, :mars]))
     end
 
+    it "should support empty array get parameters" do
+      expectjs("Routes.inboxes_path({ a: [], b: {} })").to eq(test_routes.inboxes_path({ a: [], b: {} }))
+    end
+
     context "object without prototype" do
       before(:each) do
         evaljs("let params = Object.create(null); params.q = 'hello';")