The option "Delete Protection: Disallow deletion of nodes for users without administer nodes permission." is sort of moot since you can set Delete perms per content type in the regular Drupal permissions admin.

CommentFileSizeAuthor
#3 wikitools_remove-delete-protection_263475-3.patch3.21 KBadd1sun
#2 wikitools_remove-delete-protection_263475-2.patch2.13 KBadd1sun

Comments

cwgordon7’s picture

cwgordon7 commented

True. That was an oversight on my part during the porting.

add1sun’s picture

add1sun commented
Status: Active » Needs review
StatusFileSize
new wikitools_remove-delete-protection_263475-2.patch2.13 KB

Taking it out. Patch against 6 branch.

add1sun’s picture

add1sun commented
StatusFileSize
new wikitools_remove-delete-protection_263475-3.patch3.21 KB

Ah, buggers. Forget to check the incs. This one also removes it from the settings page.

cwgordon7’s picture

cwgordon7 commented

Looks good... I need to do some testing though. In fact, I've been meaning to write simpletests for this module for a while now...

bsherwood’s picture

bsherwood commented

What actually happens if you have this enabled? Does it still try to "protect" the node from deletion? Does it conflict with Drupal's Node API?

Thanks!

cwgordon7’s picture

cwgordon7 commented
Status: Needs review » Fixed

Fixed, thanks! (Sorry for the delay!)

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.