I detest confirmation forms, but in this case it was the quick and easy way to add this functionality without risking security and having to handle the JS replaceWith structure.
I have finally come full circle and what to remove the confirmation form and just use AJAX. This patch does so using security tokens, works without JS and works with multiple Activity2 Views on a page with Delete links.
Haven't decided if we need a .js file or if inline like this is ok.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | activity_delete.patch | 5.82 KB | Scott Reynolds |
| activity_delete.patch | 5.8 KB | Scott Reynolds |
Comments
Comment #1
Scott Reynolds commentedLets not use the keyword 'delete'. this just changes that to deleteSettings.
Comment #2
Scott Reynolds commenteddoesn't work cleanly with the Ajax pager setting :-(. Have to figure out a Views2 and a separate Views3 method for setting the pager.
Comment #3
pribeh commentedBump. Plenty of users have reported hating delete confirmation forms on various things including activity items - on my sites.
Comment #4
sirkitree commentedmoving to needs work until we figure out the views stuff, since views is our main form of display.
Comment #5
madhusudan commentedI tried the patch. it patched correctly.. also deletes activity.. after deleting the page doesn't get refreshed!...
an ajaxified delete confirm will be fine..
looks like I need to roll it back..
Comment #6
_shyD6 reached its EOL back in February 2016, and there is no active release for D6 for this module anymore.
Development or support is not planned for D6. All D6-related issues are marked as outdated in a bunch.
If the issue remains relevant for D10+ versions, merge requests with proposed solutions for a new module version (D10+) are welcome in a new follow-up issue.
Thanks!