Closed (fixed)
Project:
Flag
Version:
6.x-2.x-dev
Component:
Views integration
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
9 Feb 2009 at 11:25 UTC
Updated:
22 Dec 2010 at 13:10 UTC
Jump to comment: Most recent file
Comments
Comment #1
mooffie commentedI'm attaching a schema of how this can be done. This is a rip off of once-functional old patch I had laying around.
Comment #2
mooffie commented'field' should point to 'fcid', of course, not to 'content_id'. (That snippet was written long ago, when we didn't yet have a primary key on that table.)
Comment #3
Rosamunda commentedsubscribing!
Comment #4
mooffie commentedAnother reason to expose this table:
If we expose this table, we will be able to do a "summary" on the flaggings done during the past X days/hours/etc.
Comment #5
mooffie commentedAnother reason to expose this table:
The patch achieves this by exposing three relationships: "The flagged node", "The flagged comment", The flagged user".
Comment #6
mooffie commentedAnother reason to expose this table:
(The "flagged node" relationship I mentioned in the previous reply solves this as well.)
Comment #7
mooffie commentedThere are many reasons to exposes this table. The code here complements the "flagging flaggings" feature which in itself solves many feature requests.
Comment #8
mitchell commentedTagging as 'flag 1.2' to try to help draw more attention to it.
Also added #464970: Natively support aliases for base tables.
Comment #9
mitchell commentedMarked #433308: Get all nodes from all users I have flagged? (Flag Users) as a duplicate of this issue.
Moving.
Comment #10
camaromelt commentedI feel this would be a great addition as I am wanting to also view the number of flags a node has in a given timeframe and then cause the top flagged item to be hard posted to a front page. Any thoughts if this idea is on the same wavelength? I'd love to get involved but I am totally newb with PHP.
Comment #11
jday commentedI would like the table exposed in views so that we can create a view of all the users who have flagged a particular node.
Comment #12
quicksketchYou can do that already. See the handbook page on this exact task: http://drupal.org/node/326308
Comment #13
jday commentedok almost have it, the handbook says
"you can easily turn it into a block. Add a "Block" display. Since blocks don't get arguments from the URL, you'll have to add the "Node ID from URL" default argument."
but I don't see this "Node ID from URL" option, where is it?
Nevermind, found it under "Provide default argument"
Comment #14
awolfey commentedPlease read #859822: Views integration for private flags module and then come back here.
I currently have time and funding to do the flag views integration rewrite. Views is not really a specialty but I should be able to figure it out.
But I'd like to have buy in from the module maintainers before I start. If not, I can still do what I need for my project. It really should be views though.
Please test the module posted here #750578: Patch creates hook_flag_delete() and let me know what you think. I plan to have this done one way or another in the next few weeks.
Thanks
Comment #15
mooffie commentedWhat's described in this thread isn't a rewrite. A rewrite entails writing an "upgrade path" for all the old views and to rewrite the handbook pages, and we don't have energy for this.
What's described here is a supplemental (or "alternative"), that could even reside in a separate module. The existing Views integration isn't modified.
I've read it. You wrote:
And this proposal (#371432) makes this possible. Don't you think so?
I can't speak on behalf of Nate. As for me, I've been AWOL so I'm not a factor. But it'd be easier for me to spend the few hours needed to complete this task than to make various promises. I may have good news for you: it's possible that I'll need to implement this anyway, to accommodate #871064: Making flaggings fieldable better.
Comment #16
awolfey commentedMoofie, [#871064 ] sounds great for D7, but I need this in the next few weeks for D6. Will you be working on anything for d6 in that time frame?
I think your suggestion of make views integration a separate module makes good sense for my project, so could go down that road. No sense in duplicating our efforts, so let me know what you think.
Comment #17
henrijs.seso commentedfollowing
Comment #18
a_c_m commentedfollowing as well :)
Comment #19
nunoveloso commentedanother follow
Comment #20
bleen commentedsubscribe
Comment #21
mooffie commentedComment #22
mooffie commented#981912: List flags & its count user is falgged with - Views & Flag was marked a dup of this issue.
(It contains nice examples of views that are hard to make with the current Views support.)
Comment #23
mooffie commented(Fixed changing the title by mistake.)
Comment #24
mooffie commentedI bet you lost hope, after two years, this moment would come :-)
There's now an alternative, better Views support for Flag:
Flag Vista
Comment #25
mooffie commented(Some may ask why wasn't "Flag Vista" posted as a mega-patch for Flag. Because it would have taken years to review it and write code to convert existing, old views. I suspect changing the Catholic doctrine would be easier. Instead, we should decouple Flag's built-in Views support into a separate module to give users the chance to simply not have it.)
Comment #26
henrijs.seso commentedlol