プロジェクト

全般

プロフィール

Backport(バックポート) #3716

mail_to 関数を用いるときに空白が + に変換されてしまう

isao sano9年以上前に追加. 9年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2013-08-09
期日:
進捗率:

100%


説明

概要

mail_to 関数を用いるときに空白が + に変換されてしまう.
mailto スキームを用いる場合は + ではなく %20 にすべきだが, symfony の UrlHelper では urlencode() を用いるため + に変換されてしまい,空白が + で表示されてしまう.

http://www.ietf.org/rfc/rfc2368.txt

修正案

diff --git a/lib/vendor/symfony/lib/helper/UrlHelper.php b/lib/vendor/symfony/lib/helper/UrlHelper.php
index d0bc4c1..4995c49 100644
--- a/lib/vendor/symfony/lib/helper/UrlHelper.php
+++ b/lib/vendor/symfony/lib/helper/UrlHelper.php
@@ -498,7 +498,7 @@ function mail_to($email, $name = '', $options = array(), $default_value = array(
   $default = array();
   foreach ($default_tmp as $key => $value)
   {
-    $default[] = urlencode($key).'='.urlencode($value);
+    $default[] = rawurlencode($key).'='.rawurlencode($value);
   }
   $options = count($default) ? '?'.implode('&', $default) : '';


関連するチケット

コピー元 OpenPNE 3 - Bug(バグ) #3383: mail_to 関数を用いるときに空白が + に変換されてしまう Won't fix(対応せず) 2013-08-09

関係しているリビジョン

リビジョン 27421698 (差分)
isao sano9年以上前に追加

repair the function that changes space into symbol +(fixes #3716, BP from #3383)

リビジョン b2e07c01
Shinichi Urabe9年以上前に追加

Merge pull request #216 from isaosano/t-3716

repair the function that changes space into symbol +(fixes #3716, BP from #3383)

リビジョン a4b6b479 (差分)
isao sanoほぼ4年前に追加

(fixed #4178) repair the function that changes space into symbol +(fixes #3716, BP from #3383)

リビジョン d282edf9 (差分)
isao sanoほぼ4年前に追加

(fixed #4429, BP from #4178) repair the function that changes space into symbol +(fixes #3716, BP from #3383)

履歴

#1 isao sano9年以上前に更新

  • コピー元 Bug(バグ) #3383: mail_to 関数を用いるときに空白が + に変換されてしまう を追加

#2 isao sano9年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#3 isao sano9年以上前に更新

  • ステータスAccepted(着手) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

https://github.com/openpne/OpenPNE3/pull/216
にてプルリクエストを行いました

#4 Shinichi Urabe9年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

#5 isao sano9年以上前に更新

  • 担当者isao sano にセット

#6 isao sano9年以上前に更新

  • ステータスPending Testing(テスト待ち) から Pending Merge(マージ待ち) に変更
  • 進捗率70 から 80 に変更

単体試験完了しました。
マージお願いたします。

#7 Shinichi Urabe9年以上前に更新

  • ステータスPending Merge(マージ待ち) から Fixed(完了) に変更
  • 進捗率80 から 100 に変更

他の形式にエクスポート: Atom PDF