Even admins make mistakes and click the wrong link. Confirm forms are nice for this so here's a confirmation before deleting views fusions. This patch also gets rid of the arg() calls in favor of getting the arguments from the menu system which is also nice.

I also have a port of this for 5 ready for when 5 is ready.

CommentFileSizeAuthor
views_fusion_confirm_delete.patch3.29 KBneclimdul
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Version: 4.7.x-1.x-dev » 5.x-1.x-dev
Status: Needs review » Active

thanks, I've fixed some bugs (same vids, wrong formid) and committed it to 4.7.x. I'd appreciate a 5.x port..

neclimdul’s picture

neclimdul
He/Him
CreditAttribution: neclimdul commented
Version: 5.x-1.x-dev » master
Status: Active » Patch (to be ported)

I'll get one together this evening. (Can you make a -5-dev version so we can use it as an issue version?)

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented
Version: master » 5.x-1.x-dev

thanks. there is already one.. :)

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented

any updates on this?

fago’s picture

fago
Primary language German
Location Vienna
CreditAttribution: fago commented

hm!?