Skip to content

Conversation

@seto1
Copy link
Collaborator

@seto1 seto1 commented Aug 5, 2025

コンテンツ管理にはゴミ箱機能のために論理削除が使われています。
現在、論理削除実行時にBcUploadBehavior->beforeDelete経由でアイキャッチが削除されるなど、以下のissueに記載の問題が存在します。

対策として、論理削除時にはイベントの発火や関連データの削除を行わないよう調整しています。
ご確認お願いします。

issue

#4230
#4234

@seto1 seto1 requested a review from Copilot August 5, 2025 09:25

This comment was marked as outdated.

@seto1 seto1 requested a review from Copilot August 5, 2025 09:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

このPRは論理削除(soft delete)時にイベント実行や関連データの削除を行わないように改修することを目的としています。現在のsoft delete実装では、物理削除と同様にbeforeDelete/afterDeleteイベントや関連データのカスケード削除が実行されていましたが、これを論理削除時には実行しないように変更しています。

  • 論理削除(soft delete)処理からイベント実行とカスケード削除を除去
  • 物理削除(hard delete)メソッドにオプション引数を追加し、イベント実行とカスケード削除機能を移行
  • ArrayObjectのimportを追加してオプション処理を改善

@seto1 seto1 changed the title WIP: 論理削除時にイベントの実行や関連データの削除を行わない 論理削除時にイベントの実行や関連データの削除を行わない Aug 5, 2025
@ryuring
Copy link
Collaborator

ryuring commented Aug 7, 2025

@seto1 イベント処理を物理削除に移行したということですね、ありがとうございます。マージします。

@ryuring ryuring merged commit c5b7f9c into baserproject:5.2.x Aug 7, 2025
1 check passed
@momofff momofff added this to the 5.2.0 milestone Aug 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

3 participants