Project

General

Profile

Backport（バックポート） #3764

Backport（バックポート） #3767: fileを参照するテーブル(activity_imageなど)のレコード削除時に対応するfileレコードが削除されない

バナー設定でアップロードした画像を削除してもDBから削除されない

Added by Chiharu Nakajima over 7 years ago. Updated over 7 years ago.

Status:
Fixed（完了）
Priority:
Normal（通常）
Assignee:
Youichi Kimura
Target version:
OpenPNE 3.8.15
Start date:
2015-01-06
Due date:
% Done:

100%

Description

再現手順

  1. 管理画面の「デザイン設定」→「バナー設定」(/pc_backend.php/design/banner) を開く
  2. 「バナー画像を追加」を開き、適当な画像をアップロードする
  3. バナー設定のページから、先ほど追加した画像を削除する
  4. 管理画面のアップロード画像リスト (/pc_backend.php/monitoring) に、削除したはずのバナー画像が残ったままとなっている

原因

親チケット #3738 を参照。

Related issues

Copied from OpenPNE 3 - Bug（バグ） #3755: バナー設定でアップロードした画像を削除してもDBから削除されない Won't fix（対応せず） 2015-01-06

Associated revisions

Revision c8a69310 (diff)
Added by Chiharu Nakajima over 7 years ago

(refs #3764, BP from #3755) fix BannerImage::delete() doesn't delete related File

History

#1 Updated by Chiharu Nakajima over 7 years ago

  • Copied from Bug（バグ） #3755: バナー設定でアップロードした画像を削除してもDBから削除されない added

#2 Updated by Chiharu Nakajima over 7 years ago

  • Status changed from Pending Review（レビュー待ち） to New（新規）
  • % Done changed from 50 to 0

#3 Updated by Chiharu Nakajima over 7 years ago

  • Status changed from New（新規） to Pending Review（レビュー待ち）
  • % Done changed from 0 to 50

プルリクエストしました。
https://github.com/openpne/OpenPNE3/pull/242

#4 Updated by Rimpei Ogawa over 7 years ago

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

#5 Updated by isao sano over 7 years ago

  • Parent task set to #3767

#6 Updated by isao sano over 7 years ago

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

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

#7 Updated by isao sano over 7 years ago

今回の修正により発生したわけではありませんが、試験中にバナー画像の削除に関するバグを発見したので別チケット( #3786 )を作成しました。

#8 Updated by Rimpei Ogawa over 7 years ago

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

Also available in: Atom PDF