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

Bug #278787 reported by Siegfried Gevatter
4
Affects Status Importance Assigned to Milestone
ideatorrent
Fix Committed
Low
Unassigned

Bug Description

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

Changed in ideatorrent:
importance: Undecided → Wishlist
Revision history for this message
Sayak Banerjee (sayakb-deactivatedaccount) wrote :

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

Changed in ideatorrent:
assignee: nobody → sayakb
importance: Wishlist → Low
status: New → In Progress
Revision history for this message
Sayak Banerjee (sayakb-deactivatedaccount) wrote :

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

Thanks you

Changed in ideatorrent:
status: In Progress → Fix Committed
Revision history for this message
Saivann Carignan (oxmosys) wrote :

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!

Revision history for this message
Sayak Banerjee (sayakb-deactivatedaccount) wrote :

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.

Revision history for this message
Sayak Banerjee (sayakb-deactivatedaccount) wrote :

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.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.