admin batch delete broken, node_delete needs reworked [#36250]

this patch fixes several problems with drupal's delete cycle, the main one being that admin batch delete was broken. the main cause of this was that node_delete could not be called directly anymore after the forms API upgrade. this patch fixes that, and makes node_delete exactly that--a function which deletes a node. the confirm screen has been seperated out from the act of deletion, which allows admin/node or any other module to call node_delete on it's own if necessary. in the new structure, pass the nid to be deleted, as follows:

node_delete($nid)

it's not longer necessary to pass a confirm element to bypass the confirm screen, and passing an array will no longer work--the array structure has been eliminated since it's no longer necessary.

Comment	File	Size	Author
	node_delete.patch	6.94 KB	hunmonk

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Dries CreditAttribution: Dries commented 4 November 2005 at 19:26

The following modules use node_delete():

acidfree/acidfree.module
acidfree/class_album.inc
aggregator2/TODO.txt
aggregator2/aggregator2.module
cck/content.module
copyright/copyright.module
eventrepeat/eventrepeat.module
flexinode/flexinode.module
groups/profile_node.module
i18n/i18n.inc
import_export/import_export.module
jobsearch/patches/node.module-4.6.2
jobsearch/patches/node.module-4.6.3
naggregator/naggregator.module
node_aggregator/feed.module
quickpost/quickpost.module
simpletest/tests/page_creation.test
simpletest/tests/page_view.test
simpletest/tests/taxonomy.module.test
spam/spam.module
tagnode/tagnode.module
upload_image/upload_image.module
user_related_content/user_related_content.module
webform/webform.module
workspace/workspace.module