Project

General

Profile

Bug(バグ) #2780

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

Added by Yuya Watanabe over 7 years ago. Updated over 2 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2012-02-07
Due date:
% Done:

100%

3.6 で発生するか:
Yes (はい)
3.8 で発生するか:
Unknown (未調査)

Description

概要

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

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

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

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

修正案

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

  $ git diff 901694eda4a4aaebc0c32353c4cdc0e763ca0813...10a1e4f16ae0532478db6495b2e87f9dcfee316f lib/vendor/symfony/ > t2780.patch
  $ vim t2780.patch
  $ patch -p1 < t2780.patch

t2780.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);
     }


Related issues

Related to OpenPNE 3 - Backport(バックポート) #2781: #2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる)」により一部修正が巻き戻ってしまっている. Fixed(完了) 2012-02-07
Related to OpenPNE 3 - Bug(バグ) #1565: 管理画面のSNS設定にアクセスできない(特定のモジュールのページにアクセスできない) Fixed(完了) 2010-09-07
Related to OpenPNE 3 - Backport(バックポート) #3973: #2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる)」により一部修正が巻き戻ってしまっている. Fixed(完了) 2016-07-26

Associated revisions

Revision 732ca6fe (diff)
Added by Yuya Watanabe over 7 years ago

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

History

#1 Updated by Yuya Watanabe over 7 years ago

  • Target version set to OpenPNE 3.7.0
  • 3.6 で発生するか changed from Unknown (未調査) to Yes (はい)

#2 Updated by Yuya Watanabe over 7 years ago

  • Description updated (diff)

#3 Updated by Yuya Watanabe over 7 years ago

  • Subject changed from #1565 「管理画面のSNS設定にアクセスできない(特定のモジュールのページにアクセスできない)」の修正が巻き戻ってしまっている. to #2316 「update symfony to 1.4.13 (同梱のsymfonyのバージョンを1.4.13へ上げる)」により一部修正が巻き戻ってしまっている.
  • Description updated (diff)

#4 Updated by Yuya Watanabe over 7 years ago

  • Description updated (diff)

#1026 「Add option not to recreate DB to openpne:install (openpne:install 時に DB を作り直さないオプションを追加する)」
c65b9aa68adeefe685535abb1d116fd08fb8c70e

は Symofony 本体に含まれていたため削除.

#5 Updated by Yuya Watanabe over 7 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Yuya Watanabe

#6 Updated by Yuya Watanabe over 7 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

更新履歴 732ca6fe5c6cebc36d0e7bc82d236d1f7ce209f7 で適用されました。

#7 Updated by Kousuke Ebihara over 7 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

#8 Updated by Shouta Kashiwagi over 7 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

テストOKです。

#9 Updated by kaoru n almost 4 years ago

  • 3.8 で発生するか set to Unknown (未調査)

#10 Updated by kaoru n about 3 years ago

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

Also available in: Atom PDF