プロジェクト

全般

プロフィール

Backport(バックポート) #3710

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

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

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

100%


説明

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) が実行されるようにする

関連するチケット

コピー元 OpenPNE 3 - Bug(バグ) #3302: opToolkit::writeCacheFile() にある複数の不具合の修正 Won't fix(対応せず) 2013-01-17

関係しているリビジョン

リビジョン 97c86446 (差分)
isao sano9年以上前に追加

fix using result of void function in opToolkit::writeCacheFile() (fixes #3710, BP from #3302)

リビジョン b1424eae
Shinichi Urabe9年以上前に追加

Merge pull request #211 from isaosano/t-3710

fix using result of void function in opToolkit::writeCacheFile() (fixes #3710, BP from #3302)

履歴

#1 isao sano9年以上前に更新

  • コピー元 Bug(バグ) #3302: opToolkit::writeCacheFile() にある複数の不具合の修正 を追加

#2 isao sano9年以上前に更新

  • ステータスPending Review(レビュー待ち) から New(新規) に変更
  • 担当者 を削除 (Youichi Kimura)
  • 進捗率50 から 0 に変更

#3 isao sano9年以上前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

#4 isao sano9年以上前に更新

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

https://github.com/openpne/OpenPNE3/pull/211
にてプルリクエストを行いました

#5 Shinichi Urabe9年以上前に更新

  • 担当者isao sano にセット

担当を設定ください(差し戻し対応が誰か不明になるため)

#6 Shinichi Urabe9年以上前に更新

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

#7 isao sano9年以上前に更新

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

単体試験完了しました。
マージお願いたします。

#8 Shinichi Urabe9年以上前に更新

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

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