Conditional Flags works fine with Javascript flags, but if you change link type to normal or confirmation form it just doesn't work.

CommentFileSizeAuthor
#4 conditional_flags-2129977-4.patch2.67 KBdanepowell

Comments

sebas5384’s picture

Category: Bug report » Feature request

I see this as a feature request, so i will love to review patches :)

Just wondering if even the javascript didn't work for other link types, the reaction unflag/flag still applies in the background, can you tell me?

Cheers!

Plazmus’s picture

Hey, thank for your reply. If that's how it suppose to work, then I would suggest to put that on the module's description page.

I can't provide a patch as I am not a developer.
It is worth to mention that we can actually achieve the same functionality with Rules, but this time only with no javascript links, but I guess it's not that elegant solution.

sebas5384’s picture

Yes! I'm going to write about this in the description page, thanks for the suggestion!

danepowell’s picture

Category: Feature request » Bug report
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new2.67 KB

The problem is that Flag removed hook_flag() in D7 and replaced it with hook_flag_flag() and hook_flag_unflag(). The attached patch fixes this.

sebas5384’s picture

Hey Dane! thanks for the patch.

I'll review it, and if it works, then i make an other release ;)

Cheers!