プロジェクト

全般

プロフィール

Bug(バグ) #3302

完了

opToolkit::writeCacheFile() にある複数の不具合の修正

Youichi Kimura さんがほぼ12年前に追加. 7年以上前に更新.

ステータス:
Won't fix(対応せず)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2013-01-17
期日:
進捗率:

0%

予定工数:
3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

説明

Overview (概要)

lib/util/opToolkit.class.php 内の opToolkit::writeCacheFile() に下記のようなコードがある。

if ($filesystem->copy($tmpFile, $pathToCacheFile, array('override' => true)))
{
  $filesystem->remove($tmpFile);
}

このコードには下記のような複数の問題があるため、この修正を行う。

  • if の条件に、値を返さない sfFilesystem::copy() を指定している
  • もし sfFilesystem::copy() が成功した場合に true を返すようなメソッドであることを期待していたとしても、$tmpFile のファイルは常に削除されるべきであるため if による分岐は不要である (失敗した場合でも一時ファイルは削除すべき)

Spec (仕様)

  • if による分岐を無くし、常に $filesystem->remove($tmpFile) が実行されるようにする

関連するチケット 2 (0件未完了2件完了)

コピー先 OpenPNE 3 - Backport(バックポート) #3709: opToolkit::writeCacheFile() にある複数の不具合の修正Fixed(完了)isao sano2013-01-17

操作
コピー先 OpenPNE 3 - Backport(バックポート) #3710: opToolkit::writeCacheFile() にある複数の不具合の修正Fixed(完了)isao sano2013-01-17

操作

Youichi Kimura さんがほぼ12年前に更新

一部勘違いな記述があったため削除

Youichi Kimura さんがほぼ12年前に更新

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

更新履歴 5f41d4ddca13ec656c638f283861481b842f1f8c で適用されました。

isao sano さんが約10年前に更新

isao sano さんが約10年前に更新

Shinichi Urabe さんが約10年前に更新

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

isao sano さんが7年以上前に更新

  • ステータスPending Testing(テスト待ち) から Won't fix(対応せず) に変更
  • 進捗率70 から 0 に変更

OpenPNE 3.8.14 にて対応済みであったため、対応せずとします。

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