see:
function node_multiple_delete_confirm
function node_revision_delete
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | dapi_remnants.patch | 18.17 KB | asimmonds |
see:
function node_multiple_delete_confirm
function node_revision_delete
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | dapi_remnants.patch | 18.17 KB | asimmonds |
Comments
Comment #1
asimmonds commentedHave included in this patch the code from the CVS revision immediately previous to the DAPI commit for:
- comment_multiple_delete_confirm()
- node_multiple_delete_confirm()
- node_revision_delete()
The will bring this code back to before DAPI was introduced.
Tested multiple comment and node deletions, as well as a node revision deletion.
also have included the removal of the DAPI functions:
- _drupal_delete()
- _drupal_delete_execute()
- drupal_redirect()
from common.inc
A grep of the core code shows no usage of these 3 functions, apart from actions.(inc|module) which has a open issue for this and other problems.
Comment #2
AjK commentedAs of now patch still applies, however an offset is required for comment.module hunk #1:
Hunk #1 succeeded at 1284 (offset -12 lines).
Tested all the delete functions this patch touches and they now work fine (which were broken without patch).
Comment #3
gábor hojtsyThanks, committed!
Comment #4
(not verified) commented