プロジェクト

全般

プロフィール

Backport(バックポート) #3658

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

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

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

100%


説明

状況

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

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

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

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

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

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

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

対応

パッチを添付しました。

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


関連するチケット

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

関係しているリビジョン

リビジョン cfe22bf7 (差分)
誠二 天重9年以上前に追加

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

履歴

#1 isao sano9年以上前に更新

  • コピー元 Bug(バグ) #1632: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 を追加

#2 誠二 天重9年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者誠二 天重 にセット

#3 誠二 天重9年以上前に更新

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

#4 Shinichi Urabe9年以上前に更新

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

レビューOKです

#5 isao sano9年以上前に更新

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

単体試験完了いたしました。問題ありません。

#6 Shinichi Urabe9年以上前に更新

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

マージしました

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