Closed (fixed)
Project:
Flag
Version:
6.x-1.x-dev
Component:
Flag core
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Jun 2009 at 19:35 UTC
Updated:
11 Oct 2009 at 22:25 UTC
Jump to comment: Most recent file
Comments
Comment #1
quicksketchNice! Just the way I'd want it implemented. One thing though, we shouldn't be filtering by the current user. In a rare situation, it might be possible to have a user that didn't have any flags listed on the node type, but another user does for that same type. Then the administrator that rearranges the fields would end up rearranging all the fields except for the Flag fieldset.
So basically, we should just list the flag fieldset based on any flags existing with the "show_on_form" property, rather than filtering to the current user.
Comment #2
sirkitree commentedThis more of what you had in mind?
Comment #3
quicksketchNice, exactly. I applied the patch and found that the "$node" variable didn't exist, it should be $form['#node']. After making that change everything worked perfectly. Committed attached patch. Thanks sirkitree!
Comment #5
Gabriel R. commentedHey, it works, thanks :-)