You are now required to confirm when you are disabling a rule, and it states "you will be unable to undo this!". While you can easily can, so beside being inaccurate its not needed since its not a deletion (for which confirmation should be reserved).

CommentFileSizeAuthor
#6 1029600-disable_confirmation.patch2.39 KBtwistor

Comments

fago’s picture

Component: User Interface » User interface

>you will be unable to undo this!

Agreed, undoing it isn't that hard ;) However, I don't think scratching the confirmation page is so a good idea as it would be vulnerable to CSRF attacks. We would have to make this secure with a token or so.

Bojhan’s picture

It sounds like the confirmation page, is a hack to avoid CSRF attacks :P

fago’s picture

At least it's the simplest way to avoid them ;)

Bojhan’s picture

Status: Active » Needs work

Ok, well seems like this needs work.

rszrama’s picture

I noticed this and thought it odd as well. At the very least, we can get rid of that default message. The standard solution used elsewhere is to actually put in a hidden form and have the link submit that to toggle the status. I use Fasttoggle for this for node administration.

twistor’s picture

Status: Needs work » Needs review
StatusFileSize
new2.39 KB

Here's the simple approach. At least this makes disabling things less scary.

fago’s picture

Title: Disable rule should have no confirmation » Disable rule has wrong confirmation text
Status: Needs review » Fixed

thanks committed.

@no-confirmation: Please open another issue for that if you think its still necessary.

rowbotony’s picture

Hi, I also experienced this and freaked out a bit... "cannon be undone" is rather harsh, when in fact "disable" and "enable" rules can be undone, hope @twistor's patch removes the scary message :)

Status: Fixed » Closed (fixed)

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