Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ファイル削除時に参照しているノートを削除しないようにする #7346

Closed
mei23 opened this issue Mar 13, 2021 · 3 comments
Labels
✨Feature This adds/improves/enhances a feature

Comments

@mei23
Copy link
Contributor

mei23 commented Mar 13, 2021

Summary

#4724 でファイル削除時に参照しているノートも削除するようにしているが
https://github.com/syuilo/misskey/blob/9cf811b048c03d093871b21c0a357813746378af/src/services/drive/delete-file.ts#L89-L112

  • Note.fileIds (Array)のクエリに (このままでは) インデックスが効かないので重い。
  • ちゃんとトランザクションを張る必要があるが、関連するノートの探索もあるのでトランザクションがややこしくなる。
  • リモート起因のノートが削除された際には、どのノートからも参照されなくなったファイルを削除する必要があるが、ファイルが削除されたらノートも削除するの規則もあるとややこしくなる。

ので、ドライブファイルが削除されてもノートには何もしない方が良さそう。
ノートなどがDB上で削除されていたファイルを参照していた場合は参照してなかったことにしてしまう (#7345 でまかなえる)
や v10 のようにドライブファイルは論理削除にしてしまうなど。

@mei23 mei23 added the ✨Feature This adds/improves/enhances a feature label Mar 13, 2021
@syuilo
Copy link
Member

syuilo commented Mar 13, 2021

論理削除にしたくなってきた
ref. https://github.com/syuilo/misskey/issues/7342

@mei23
Copy link
Contributor Author

mei23 commented Mar 13, 2021

ドライブファイルは別に物理削除でいいと思う。
削除した跡を残したい場合、ノートから参照しているドライブファイルがなかったらそれは削除済みだと区別付くので。

@mei23
Copy link
Contributor Author

mei23 commented Feb 12, 2022

解決済み

@mei23 mei23 closed this as completed Feb 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature
Projects
None yet
Development

No branches or pull requests

2 participants