New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed double escape of & in mail_to helper #1654

Merged
merged 1 commit into from Apr 13, 2014

Conversation

Projects
None yet
2 participants
@rmm5t
Contributor

rmm5t commented Apr 13, 2014

  • This only manifests itself with mail_to links that have more than
    one query option (subject, body, cc, bcc)
  • Escaping & in mail_to is unnecessary, because it is already later
    handled when link_to is called
Fixed double escape of & in mail_to helper
* This only manifests itself with mail_to links that have more than one
  query option (subject, body, cc, bcc)
* Escaping & in mail_to is unnecessary, because it is already later
  handled when link_to is called
@nesquena

This comment has been minimized.

Show comment
Hide comment
@nesquena

nesquena Apr 13, 2014

Member

Thanks!

Member

nesquena commented Apr 13, 2014

Thanks!

nesquena added a commit that referenced this pull request Apr 13, 2014

Merge pull request #1654 from rmm5t/fix-mailto-double-amp
Fixed double escape of & in mail_to helper

@nesquena nesquena merged commit 2eed826 into padrino:master Apr 13, 2014

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details

@rmm5t rmm5t deleted the rmm5t:fix-mailto-double-amp branch Apr 13, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment