[Nitro] render :index broken
George Moschovitis
george.moschovitis at gmail.com
Sat Sep 15 13:42:37 EDT 2007
thanks, for this patch. Is it possible that you add to the spec file:
test/raw/dispatcher.rb
a test for this?
thank you.
-g.
On 9/15/07, Arne Brasseur <arne at arnebrasseur.net> wrote:
> Patch included. This solves redirecting to the index method of the root
> controller.
>
> (ab)
>
> Arne Brasseur schreef:
> > When doing "render :index " in a controller action, the following gives
> > me an error:
> >
> > #dispatcher.rb line 142
> >
> > def dispatch(uri, method = :get)
> > # Extract the query string.
> >
> > path, query = uri.split("?", 2) # <------- uri == "" then
> > path == query == nil
> >
> > path ||= "" # <------ This solves the problem
> >
> > # Try to route the path.
> >
> > path = @router.route(path) if @router
> >
> > # The characters after the last '.' in the path form the
> > # extension that itself represents the expected response
> > # content type.
> >
> > ext = File.extname(path)[1..-1] || "html" # <------------ extname
> > don't like no nil
> > ....
> >
> > So if uri.length == 0, then uri.split('?') returns [], so path and query
> > are both nil. The call to File.extname fails because of this.
> >
> > I solved this by adding the path ||= "", does this look like a good
> > solution? Should I make a patch?
> >
> > thx,
> > (ab)
> >
> >
>
>
> --
> Arne Brasseur
> http://www.arnebrasseur.net
> arne at arnebrasseur.net
>
>
> _______________________________________________
> Nitro-general mailing list
> Nitro-general at rubyforge.org
> http://rubyforge.org/mailman/listinfo/nitro-general
>
>
--
http://www.me.gr
http://phidz.com
http://blog.gmosx.com
http://cull.gr
http://www.joy.gr
http://nitroproject.org
More information about the Nitro-general
mailing list