Bug(バグ) #1632
完了
[PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分
Hidenori Goto さんが約14年前に追加.
7年以上前に更新.
説明
1. 末尾スラッシュの除去処理がサブディレクトリのプレフィックスを考慮していない。
opApplicationConfiguration::getAppScriptName()の以下の処理
if ('/' === $prefix)
{
$prefix = '';
}
は、後続の処理でスラッシュとファイル名を連結するために$prefixの末尾のスラッシュを除去するための処理と思われるのですが、ルートディレクトリのパターンでしか処理されていません。
2. $isNoScriptNameがtrueの場合とのインターフェイスの整合性
$isNoScriptNameがtrueの場合、$prefixをそのままreturnしていますが、$prefixの末尾のスラッシュの扱いについて、このメソッドの戻り値の一貫性が崩れていると思われます。
(このメソッドからは、末尾スラッシュなしのパスが返されるというインターフェイスになっているという認識)
※sfRouting::fixGeneratedUrl()でprefixとの結合を行っていますが、ここでprefixの末尾にスラッシュがないことを前提としているのかどうかまでは分からなかったため、調べています。
パッチを添付しました。
ファイル
関連するチケット
3 (0件未完了 — 3件完了)
- 対象バージョン を OpenPNE 3.7.0 にセット
- 題名 を opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 から [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 に変更
- 対象バージョン を OpenPNE 3.7.0 から 252 に変更
- 対象バージョン を 252 から OpenPNE 3.8.x に変更
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 担当者 を 誠二 天重 にセット
- 進捗率 を 0 から 50 に変更
- 3.6 で発生するか を Unknown (未調査) にセット
- 3.8 で発生するか を Unknown (未調査) にセット
- 対象バージョン を OpenPNE 3.8.x から OpenPNE 3.8.12 に変更
- コピー先 Backport(バックポート) #3633: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 を追加
- 対象バージョン を OpenPNE 3.8.12 から OpenPNE 3.8.x に変更
- 対象バージョン を OpenPNE 3.8.x から OpenPNE 3.9.0-old に変更
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
- コピー先 Backport(バックポート) #3658: [PATCH] opApplicationConfiguration::getAppScriptName()で、prefixの末尾スラッシュ除去処理が不十分 を追加
- ステータス を Pending Testing(テスト待ち) から Won't fix(対応せず) に変更
- 進捗率 を 70 から 0 に変更
OpenPNE 3.8.12 にて対応済みであったため、対応せずとします。
他の形式にエクスポート: Atom
PDF