"Delete ALL his/her _____" should ask for confirmation

[Siegfried Gevatter]

The "Delete ALL his/her [ideas/solutions/comments]" admin tools on user profiles should ask for confirmation before deleting everything...

[Sayak Banerjee (sayakb-deactivatedaccount)]

Agreed. Although this link is not used often, this should still ask for confirmation.

[Sayak Banerjee (sayakb-deactivatedaccount)]

Fix committed with r527 on personal branch.
If you have a local installation of ideatorrent, please checkout and suggest any changes.

Thanks you

[Saivann Carignan (oxmosys)]

Sayak Banerjee : If found a fix for this bug too today before I found that you already had it fixed in your branch. In case that this interest you, I found a little javascript trick that doesn't require to add a large javascript function, all that is needed is to add :

onclick="javascript:return confirm('Are you sure you want to delete ALL user ideas?');"

I attach a diff that contains the fix, in case it can be helpful.

Thanks for your great work on ideatorrent!

[Sayak Banerjee (sayakb-deactivatedaccount)]

Yes ofcourse, that should work as well. I just added this to have an embedded confirmation since IdeaTorrent did not have any popup confirmations at all.

[Sayak Banerjee (sayakb-deactivatedaccount)]

Using this method will still need a function to add the confirmation dialog value to a variable and return it.
If done this way, once the user clicks "No", the confirmation dialog will not be shows at all for further clicks.