attached patch converts filter deletions to the deletion API. tested as working.

CommentFileSizeAuthor
dapi_filter.patch4.1 KBhunmonk

Comments

drewish’s picture

drewish commented

subscribing. couldn't test it right now because of some other errors in the filter module.

eaton’s picture

eaton commented
Status: Needs review » Reviewed & tested by the community

Ran this through and it works as advertised. Same queries, same results.

eaton’s picture

eaton commented
Status: Reviewed & tested by the community » Postponed

With DeletionAPI pushed off until D7, this patch is postponed as well.

hunmonk’s picture

hunmonk commented
Assigned: hunmonk » Unassigned
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Postponed » Closed (won't fix)

I've been marking these won't fix... since the deletion API was rolled back and won't be put in D7 unless it undergoes a significant refactoring, all of these sub-patches won't be anything like this in the end.