Backport(バックポート) #3709
opToolkit::writeCacheFile() にある複数の不具合の修正
開始日:
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)
が実行されるようにする
関連するチケット
関係しているリビジョン
履歴
#1 isao sano が9年以上前に更新
- コピー元 Bug(バグ) #3302: opToolkit::writeCacheFile() にある複数の不具合の修正 を追加
#4 isao sano が9年以上前に更新
- ステータス を Accepted(着手) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
https://github.com/openpne/OpenPNE3/pull/202
にてプルリクエストしました
#6 Shinichi Urabe が9年以上前に更新
- ステータス を Pending Review(レビュー待ち) から Rejected(差し戻し) に変更
#7 Shinichi Urabe が9年以上前に更新
- ステータス を Rejected(差し戻し) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
ステータス間違えました
レビュー済みです
#9 Shinichi Urabe が9年以上前に更新
- ステータス を Pending Merge(マージ待ち) から Fixed(完了) に変更
- 進捗率 を 80 から 100 に変更