[Nitro] [PATCH] Separate get_params and post_params

George Moschovitis george.moschovitis at gmail.com
Mon Dec 18 06:19:31 EST 2006


btw, why is it useful to separate get/post params? if u use them
through params (which in your latest implementation just selects from
one collection or the other anyway)?

-g.

On 12/18/06, George Moschovitis <george.moschovitis at gmail.com> wrote:
> this patch is not complete. there are more fixes needed to make this work.
> btw, shouldnt we merge  the post and get parameters as you originally suggested?
>
> -g.
>
> On 12/17/06, Noé Rubinstein <noe.rubinstein at gmail.com> wrote:
> > Same but params and params= are now post_params and post_params= if
> > method is POST, get_params and get_params= otherwise. (thanks Kashia)
> > Sorry for the flood.
> >
> > 2006/12/17, Noé Rubinstein <noe.rubinstein at gmail.com>:
> > > Same but with post precedence other get to prevent from hacking. (thanks manv)
> > >
> > > 2006/12/17, George Moschovitis <george.moschovitis at gmail.com>:
> > > > Interesting,
> > > >
> > > > I will have a look at this...
> > > >
> > > > -g.
> > > >
> > > > On 12/17/06, Noé Rubinstein <noe.rubinstein at gmail.com> wrote:
> > > > > This patch separates GET params and POST params.
> > > > > request.post_params # now returns POST params
> > > > > request.get_params # now returns GET params
> > > > > request.params # Compatibility method returning the result of
> > > > > post_params.merge(get_params)
> > > > > _______________________________________________
> > > > > Nitro-general mailing list
> > > > > Nitro-general at rubyforge.org
> > > > > http://rubyforge.org/mailman/listinfo/nitro-general
> > > > >
> > > >
> > > >
> > > > --
> > > > http://blog.gmosx.com
> > > > http://cull.gr
> > > > http://www.joy.gr
> > > > http://nitroproject.org
> > > > _______________________________________________
> > > > Nitro-general mailing list
> > > > Nitro-general at rubyforge.org
> > > > http://rubyforge.org/mailman/listinfo/nitro-general
> > > >
> > >
> > >
> >
> > _______________________________________________
> > Nitro-general mailing list
> > Nitro-general at rubyforge.org
> > http://rubyforge.org/mailman/listinfo/nitro-general
> >
> >
> >
>
>
> --
> http://blog.gmosx.com
> http://cull.gr
> http://www.joy.gr
> http://nitroproject.org
>


-- 
http://blog.gmosx.com
http://cull.gr
http://www.joy.gr
http://nitroproject.org


More information about the Nitro-general mailing list