Project

General

Profile

Backport(バックポート) #3633

[PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分

Added by 誠二 天重 about 5 years ago. Updated about 5 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2010-10-01
Due date:
% Done:

100%


Description

状況

1. 末尾スラッシュの除去処理がサブディレクトリのプレフィックスを考慮していない。

opApplicationConfiguration::getAppScriptName()の以下の処理

    if ('/' === $prefix)
    {
      $prefix = '';
    }

は、後続の処理でスラッシュとファイル名を連結するために$prefixの末尾のスラッシュを除去するための処理と思われるのですが、ルートディレクトリのパターンでしか処理されていません。

2. $isNoScriptNameがtrueの場合とのインターフェイスの整合性

$isNoScriptNameがtrueの場合、$prefixをそのままreturnしていますが、$prefixの末尾のスラッシュの扱いについて、このメソッドの戻り値の一貫性が崩れていると思われます。
(このメソッドからは、末尾スラッシュなしのパスが返されるというインターフェイスになっているという認識)

※sfRouting::fixGeneratedUrl()でprefixとの結合を行っていますが、ここでprefixの末尾にスラッシュがないことを前提としているのかどうかまでは分からなかったため、調べています。

対応

パッチを添付しました。

0001-fixed-opApplicationConfiguration.patch View (1.01 KB) Hidenori Goto, 2010-10-01 10:16


Related issues

Copied from OpenPNE 3 - Bug(バグ) #1632: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 Won't fix(対応せず) 2010-10-01

Associated revisions

Revision b7886fa2 (diff)
Added by 誠二 天重 about 5 years ago

fixed removal of trailing slash is not enough. (refs 3633, BP from #1632)

History

#1 Updated by 誠二 天重 about 5 years ago

  • Copied from Bug(バグ) #1632: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 added

#2 Updated by 誠二 天重 about 5 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/148
にてプルリクエスト。

#3 Updated by Shinichi Urabe about 5 years ago

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

レビューOKです

#4 Updated by Shinichi Urabe about 5 years ago

  • Assignee set to 誠二 天重

#5 Updated by isao sano about 5 years ago

トピックブランチでの単体テストは完了しました。
問題ありません。

stable-3.8.xでの結合テスト完了後に本チケットを「終了」にします。

#6 Updated by Shinichi Urabe about 5 years ago

https://github.com/openpne/OpenPNE3/pull/148 を stable-3.8.x に取り込みました

#7 Updated by isao sano about 5 years ago

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

結合試験完了しました。
問題ありませんのでステータスを「Fixed」いたします。

Also available in: Atom PDF