Using latest from CVS, I cannot delete nodes from the admin/node page. The admin/node/delete/nid URL seems to be broken. I get a confirmation page, but the node persists despite my confirmation.

CommentFileSizeAuthor
#2 node_delete_0.txt386 byteschx
#1 node_delete.txt382 byteschx

Comments

chx’s picture

chx commented
Assigned: Unassigned » chx
Priority: Normal » Critical
StatusFileSize
new node_delete.txt382 bytes

I've found this very confusing. Confusing, yes, but after looking at node.module especially node_admin, not surprising at all. The results from the confirmation form is just not taken care of. So here's my proposal: redirect the result of the confirmation to /node/nid/delete. However, if you do not like this approach, I have written a few lines for node_admin which also fixes the problem.

chx’s picture

chx commented
Assigned: chx » Unassigned
Priority: Critical » Normal
StatusFileSize
new node_delete_0.txt386 bytes

Forgot about outgoing aliasing, now I use the url() function. Works nicely with i18n URL translation.

chx’s picture

chx commented
Assigned: Unassigned » chx
Priority: Normal » Critical

For some reason the issue system unassigned me from this. Sorry, just starting to issue my bugfixes.

dries’s picture

dries commented

This has been fixed two days ago.

Anonymous’s picture

(not verified) commented