Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Update DSL.pm #508

Closed
wants to merge 1 commit into from

3 participants

@unickuity

fix params method that was not taking parameters into account

@unickuity unickuity Update DSL.pm
fix params method that was not taking parametes into account
4e8e4f5
@xsawyerx
Owner

Isn't that what param is for?

@unickuity

my %querystring_params = params('query');
my %route_params = params('route');
my %post_params = params('body');
the parameter is the source in that case

@veryrusty
Owner

Note that this a dup of #479 and the Pr in #480.

@unickuity

I agree

@xsawyerx
Owner

Merged, thanks guys! :)

@xsawyerx xsawyerx closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 20, 2013
  1. @unickuity

    Update DSL.pm

    unickuity authored
    fix params method that was not taking parametes into account
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/Dancer2/Core/DSL.pm
View
2  lib/Dancer2/Core/DSL.pm
@@ -294,7 +294,7 @@ sub uri_for { shift->request->uri_for(@_) }
sub splat { shift->request->splat }
-sub params { shift->request->params }
+sub params { shift->request->params(@_) }
sub param { shift->request->param(@_) }
Something went wrong with that request. Please try again.