Project

General

Profile

Backport(バックポート) #3658

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

Added by isao sano almost 7 years ago. Updated over 6 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Assignee:
Target version:
Start date:
2014-07-15
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 cfe22bf7 (diff)
Added by 誠二 天重 almost 7 years ago

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

History

#1 Updated by isao sano almost 7 years ago

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

#2 Updated by 誠二 天重 almost 7 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to 誠二 天重

#3 Updated by 誠二 天重 almost 7 years ago

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

#4 Updated by Shinichi Urabe almost 7 years ago

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

レビューOKです

#5 Updated by isao sano over 6 years ago

  • Status changed from Pending Testing(テスト待ち) to Pending Merge(マージ待ち)
  • % Done changed from 70 to 80

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

#6 Updated by Shinichi Urabe over 6 years ago

  • Status changed from Pending Merge(マージ待ち) to Fixed(完了)
  • % Done changed from 80 to 100

マージしました

Also available in: Atom PDF