17123 1 true 2009-01-13T18:05:48-08:00 17123 2009-01-18T00:00:00-08:00 27290 70 nestedrecursive-params 1 9779 resolved http params patch 2009-01-16T08:31:06-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken There's been many requests for nested/recursive params support a la Rails. e.g., when the query string or form post includes keys with subscripts, they are parsed into a series of Hash/Array structures: @@@ foo[bar]=bling&foo[baz]=bop @@@ yields: @@@ { :foo => { :bar => 'bling', :baz => 'bop' } } @@@ This is very likely to go into Rack core at some point but it would be worth adding to Sinatra in the meantime as a convenience. There's been many requests for nested/recursive params support a la Rails. e.g., when the query string or form post includes keys with subscripts, they are parsed into a series of Hash/Array structures: @@@ foo[bar]=bling&foo[baz]=bop @@@ yields: @@@ { :foo => { :bar => 'bling', :baz => 'bop' } } @@@ This is very likely to go into Rack core at some point but it would be worth adding to Sinatra in the meantime as a convenience. <div><p>There's been many requests for nested/recursive params support a la Rails. e.g., when the query string or form post includes keys with subscripts, they are parsed into a series of Hash/Array structures:</p> <pre><code> foo[bar]=bling&amp;foo[baz]=bop </code></pre> <p>yields:</p> <pre><code> { :foo =&gt; { :bar =&gt; 'bling', :baz =&gt; 'bop' } } </code></pre> <p>This is very likely to go into Rack core at some point but it would be worth adding to Sinatra in the meantime as a convenience.</p></div> 17123 0 There's been many requests for nested/recursive params support a la Rails. e.g., when the query string or form post includes keys with subscripts, they are parsed into a series of Hash/Array structures: @@@ foo[bar]=bling&foo[baz]=bop @@@ yields: @@@ { :foo => { :bar => 'bling', :baz => 'bop' } } @@@ This is very likely to go into Rack core at some point but it would be worth adding to Sinatra in the meantime as a convenience. <div><p>There's been many requests for nested/recursive params support a la Rails. e.g., when the query string or form post includes keys with subscripts, they are parsed into a series of Hash/Array structures:</p> <pre><code> foo[bar]=bling&amp;foo[baz]=bop </code></pre> <p>yields:</p> <pre><code> { :foo =&gt; { :bar =&gt; 'bling', :baz =&gt; 'bop' } } </code></pre> <p>This is very likely to go into Rack core at some point but it would be worth adding to Sinatra in the meantime as a convenience.</p></div> false 2009-01-13T18:05:49-08:00 17123 --- {} 27290 70 nestedrecursive-params 0 9779 new http params 2009-01-13T18:05:49-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 0 The following thread includes an example implementation: http://groups.google.com/group/sinatrarb/browse_thread/thread/9077403d7f150277/4d1085e1f0b0cc7b This has also been discussed heavily on the Rack list with lots of performance tips: http://groups.google.com/group/rack-devel/browse_thread/thread/1a9b8dc431bff499/3b0f8ccec6a6f918 <div><p>The following thread includes an example implementation:</p> <p><a href="http://groups.google.com/group/sinatrarb/browse_thread/thread/9077403d7f150277/4d1085e1f0b0cc7b"> http://groups.google.com/group/s...</a></p> <p>This has also been discussed heavily on the Rack list with lots of performance tips:</p> <p><a href="http://groups.google.com/group/rack-devel/browse_thread/thread/1a9b8dc431bff499/3b0f8ccec6a6f918"> http://groups.google.com/group/r...</a></p></div> false 2009-01-13T18:07:56-08:00 17123 --- {} 27290 70 nestedrecursive-params 0 9779 new http params 2009-01-13T18:08:00-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 0 false 2009-01-13T18:08:37-08:00 17123 --- :title: Nested/Recursive Params 27290 70 nestedrecursive-params 0 9779 new http params 2009-01-13T18:08:41-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 0 Oops. I don't want to take this on for 0.9.0. If someone else wants to get a branch and start getting something stable together we might be able to move it up. <div><p>Oops. I don't want to take this on for 0.9.0. If someone else wants to get a branch and start getting something stable together we might be able to move it up.</p></div> false 2009-01-13T18:26:59-08:00 17123 --- :milestone: 27290 27725 70 nestedrecursive-params 0 9779 new http params 2009-01-13T18:27:00-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.1 17123 0 Here's a stab at it, using manveru's implementation from rack's ML. <div><p>Here's a stab at it, using manveru's implementation from rack's ML.</p></div> false 2009-01-14T06:48:07-08:00 17123 --- :tag: http params 27725 70 nestedrecursive-params 0 9779 new http params patch 2009-01-14T06:48:09-08:00 278 ronin-278 (at lighthouseapp) Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.1 17123 1 false 2009-01-14T06:57:40-08:00 17123 --- :title: Nested/Recursive Params (e.g., "post[title]=Hello") 27725 70 nestedrecursive-params 0 9779 new http params patch 2009-01-14T06:57:44-08:00 278 ronin-278 (at lighthouseapp) Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.1 17123 1 MMmm. I like this patch. Moving to 0.9.0. Anyone else care to review / comment? <div><p>MMmm. I like this patch. Moving to 0.9.0. Anyone else care to review / comment?</p></div> false 2009-01-14T07:44:49-08:00 17123 --- :milestone: 27725 27290 70 nestedrecursive-params 0 9779 new http params patch 2009-01-14T07:44:52-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 1 Looks OK to me. +1 for applying/inclusion. <div><p>Looks OK to me. +1 for applying/inclusion.</p></div> false 2009-01-14T11:42:07-08:00 17123 --- {} 27290 70 nestedrecursive-params 0 9779 new http params patch 2009-01-14T11:42:12-08:00 15365 Markus Prinz Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 1 Okay, I had to make some slight tweaks but this is pretty true to foca's original patch: http://github.com/rtomayko/sinatra/commit/nestedparams Differences from foca's patch: 1. Performs param nesting once at the beginning of the request instead of on each matching route. 2. Added a testcase for deeply nested params. I feel pretty good about it. Let's see if we can get some feedback on it today and then maybe we'll merge as gravy on 0.9.0. <div><p>Okay, I had to make some slight tweaks but this is pretty true to foca's original patch:</p> <p><a href="http://github.com/rtomayko/sinatra/commit/nestedparams">http://github.com/rtomayko/sinat...</a></p> <p>Differences from foca's patch:</p> <ol> <li> <p>Performs param nesting once at the beginning of the request instead of on each matching route.</p> </li> <li> <p>Added a testcase for deeply nested params.</p> </li> </ol> <p>I feel pretty good about it. Let's see if we can get some feedback on it today and then maybe we'll merge as gravy on 0.9.0.</p></div> false 2009-01-15T04:37:46-08:00 17123 --- {} 27290 70 nestedrecursive-params 0 9779 new http params patch 2009-01-15T04:37:50-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 1 false 2009-01-15T04:37:54-08:00 17123 --- :state: new 27290 70 nestedrecursive-params 0 9779 open http params patch 2009-01-15T04:37:55-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken 17123 1 Nest params is in for 0.9.0. Thanks foca. <div><p>Nest params is in for 0.9.0. Thanks foca.</p></div> true 2009-01-16T08:31:05-08:00 17123 --- :state: open 27290 70 nestedrecursive-params 0 9779 resolved http params patch 2009-01-16T08:31:06-08:00 17123 Ryan Tomayko Ryan Tomayko Ryan Tomayko http://sinatra.lighthouseapp.com/projects/9779/tickets/70 0.9.0 Hoboken c077b082a1ec035cabf71787ca92004e09d1715d text/plain 2009-01-14T06:48:07-08:00 0001-Allow-nested-hash-parameters.patch 78304 2816 278 http://sinatra.lighthouseapp.com/attachments/78304/0001-Allow-nested-hash-parameters.patch