fix url_for api compatibility issues

sharifulin / mojolicious-plugin-i18n

Internationalization Plugin for Mojolicious 5.x and higher

11 stars 11 forks source link

fix url_for api compatibility issues #13

Closed alexeiras closed 10 years ago

alexeiras commented 10 years ago

Hey!

As of version 1.1, the url_for helper does not comply with the original interface, incorrectly capturing the passed-in arguments. e.g: passing a list, passing target + hashref...

I add some tests unveiling this issue and provide a fix.

Cheers Alex

sharifulin commented 10 years ago

Hi, thanks, but the test with "maito:" doesn't work

"en/mailto:sri@example.com" instead of "mailto:sri@example.com"

alexeiras commented 10 years ago

Hi Anatoly! Thanks for your rapid response! That test worked for me, but I'll look into it to guarantee correct handling of absolute urls.

Cheers