プロジェクト

全般

プロフィール

Backport(バックポート) #3973

#2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる)」により一部修正が巻き戻ってしまっている.

kaoru nishizoe11ヶ月前に追加. 7ヶ月前に更新.

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

100%


説明

概要

以下のチケット及びコミットが巻き戻ってしまっている.

#1565 「管理画面のSNS設定にアクセスできない(特定のモジュールのページにアクセスできない)」
f30b06229f8dbffc3a3a663f881df3776378c67d
10a1e4f16ae0532478db6495b2e87f9dcfee316f

巻き戻った原因のチケット及びコミットは以下のものである.

チケット: #2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる) 」
64a72113b7cc6eaf39a21ac66efcfc8289354e0c

修正案

OpenPNE 内での lib/vendor 以下にあるライブラリのアップデート方法や独自パッチの管理方法を検討する必要がある.
とりあえず以下のパッチを利用することで本チケットの問題は解決できる.

  $ git diff 90716878b97e49424b0eb6a5fa69161771e06b2c...10a1e4f16ae0532478db6495b2e87f9dcfee316f lib/vendor/symfony/ > t1565.patch
  $ vim t1565.patch
  $ patch -p1 < t1565.patch

t1565.patch
diff --git a/lib/vendor/symfony/lib/request/sfWebRequest.class.php b/lib/vendor/symfony/lib/request/sfWebRequest.class.php
index c2205f6..0c817fd 100644
--- a/lib/vendor/symfony/lib/request/sfWebRequest.class.php
+++ b/lib/vendor/symfony/lib/request/sfWebRequest.class.php
@@ -239,8 +239,8 @@ class sfWebRequest extends sfRequest
   public function getPathInfo()
   {
     $pathInfo = '';
-
     $pathArray = $this->getPathInfoArray();
+    $isIis = (isset($pathArray['SERVER_SOFTWARE']) && false !== stripos($pathArray['SERVER_SOFTWARE'], 'iis'));

     // simulate PATH_INFO if needed
     $sf_path_info_key = $this->options['path_info_key'];
@@ -261,14 +261,14 @@ class sfWebRequest extends sfRequest
     else
     {
       $pathInfo = $pathArray[$sf_path_info_key];
-      if ($relativeUrlRoot = $this->getRelativeUrlRoot())
+      if ($isIis && $relativeUrlRoot = $this->getRelativeUrlRoot())
       {
         $pathInfo = preg_replace('/^'.str_replace('/', '\\/', $relativeUrlRoot).'\//', '', $pathInfo);
       }
     }

     // for IIS
-    if (isset($_SERVER['SERVER_SOFTWARE']) && false !== stripos($_SERVER['SERVER_SOFTWARE'], 'iis') && $pos = stripos($pathInfo, '.php'))
+    if ($isIis && $pos = stripos($pathInfo, '.php'))
     {
       $pathInfo = substr($pathInfo, $pos + 4);
     }


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2780: #2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる)」により一部修正が巻き戻ってしまっている. Fixed(完了) 2012-02-07

関係しているリビジョン

リビジョン d0c6c1f5 (差分)
Yuya Watanabe11ヶ月前に追加

(fixes #3973, BP from #2780) add patch to symfony-1.4.13 and repatch #1565 to symfony for avoding http://trac.symfony-project.org/ticket/9061 problem

(cherry picked from commit 732ca6fe5c6cebc36d0e7bc82d236d1f7ce209f7)

リビジョン cdf99818
kaoru nishizoe7ヶ月前に追加

Merge pull request #342 from upsilon/t-3973

(fixes #3973, BP from #2780) add patch to symfony-1.4.13 and repatch #1565 to symfony for avoding http://trac.symfony-project.org/ticket/9061 problem

履歴

#1 kaoru nishizoe11ヶ月前に更新

  • 関連している Bug(バグ) #2780: #2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる)」により一部修正が巻き戻ってしまっている. を追加

#2 Youichi Kimura11ヶ月前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 担当者Youichi Kimura にセット
  • 進捗率0 から 50 に変更

下記の通り Pull Request を作成しました
https://github.com/openpne/OpenPNE3/pull/342

#3 Youichi Kimura11ヶ月前に更新

  • 説明 を更新 (diff)

732ca6fe5c6cebc36d0e7bc82d236d1f7ce209f7 に合わせて手順を修正

#4 kaoru nishizoe10ヶ月前に更新

  • 対象バージョンOpenPNE 3.8.21 から OpenPNE 3.8.x に変更

#5 Shinichi Urabe8ヶ月前に更新

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

#7 kaoru nishizoe7ヶ月前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.22 に変更

#8 Chiharu Nakajima7ヶ月前に更新

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

テストOKです

#9 kaoru nishizoe7ヶ月前に更新

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

マージしました

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