fix handling of parameters with multiple values in Mojolicious::Plugi…

…n::TagHelpers
mojolicious · Sep 2, 2015 · d7a308d · d7a308d
1 parent 1c448e3
commit d7a308d
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/Changes b/Changes
@@ -1,6 +1,8 @@
 
-6.18  2015-08-31
+6.18  2015-09-02
   - Improved portability of slurp function in Mojo::Util.
+  - Fixed handling of parameters with multiple values in
+    Mojolicious::Plugin::TagHelpers.
   - Fixed Makefile.PL to be compliant with version 2 of the CPAN distribution
     metadata specification. (Grinnz)
 

diff --git a/lib/Mojolicious/Plugin/TagHelpers.pm b/lib/Mojolicious/Plugin/TagHelpers.pm
@@ -80,7 +80,7 @@ sub _input {
     }
 
     # Others
-    else { $attrs{value} = $values[0] }
+    else { $attrs{value} = $values[-1] }
   }
 
   return _validation($c, $name, 'input', name => $name, %attrs);

diff --git a/t/mojolicious/tag_helper_lite_app.t b/t/mojolicious/tag_helper_lite_app.t
@@ -254,7 +254,7 @@ $t->get_ok('/multibox?foo=two&foo=one')->status_is(200)->content_is(<<EOF);
 EOF
 
 # Advanced form with values
-$t->get_ok('/form/lala?a=2&b=0&c=2&d=3&escaped=1%22+%222')->status_is(200)
+$t->get_ok('/form/lala?a=3&a=2&b=0&c=2&d=3&escaped=1%22+%222')->status_is(200)
   ->content_is(<<EOF);
 <form action="/links" method="post">
   <input name="foo">