attached patch makes content type deletions work w/ the deletion API. a few things:

  1. this patch is waiting on http://drupal.org/node/154046 -- so you'll need to apply that to test this properly.
  2. the deletion type name i picked was 'node_type', and the id is the type.
CommentFileSizeAuthor
#4 dapi_content_types_1.patch3.48 KBhunmonk
#2 dapi_content_types_0.patch3.53 KBhunmonk
dapi_content_types.patch3.55 KBhunmonk

Comments

pwolanin’s picture

pwolanin commented
Status: Needs review » Needs work

A couple minor problems -

there are several extra blank lines in function node_type_delete_confirm.

Also in this function the line'form' => $form, seems unneeded and generates the notice:

notice: Undefined variable: form in /Users/Shared/www/drupal6/modules/node/content_types.inc on line 399.

otherwise, seems to work fine - content type is deleted as expected.

hunmonk’s picture

hunmonk commented
Status: Needs work » Needs review
StatusFileSize
new dapi_content_types_0.patch3.53 KB

attached patch fixes the previously mentioned issues.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

Tested, seems to work fine. Code looks good. RTBC.

hunmonk’s picture

hunmonk commented
StatusFileSize
new dapi_content_types_1.patch3.48 KB

re-rolled for confirm form changes. tested and working.

hunmonk’s picture

hunmonk commented
Assigned: hunmonk » Unassigned
webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Closed (won't fix)

Deletion API was rolled back.