Providing views integration with flag_notes fields will provide ability for users to create like a My Bookmarks space that provides URL, page title, and notes that user stores against the flag
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 1022818_flag_page_flag_note_views_relationship.patch | 3.75 KB | alexpott |
| #6 | flag_note_relationship.jpg | 24.79 KB | alexpott |
| #5 | flag_note_relationship.jpg | 54.72 KB | alexpott |
| #4 | flag_page-1022818_2.patch | 1.23 KB | Ravi.J |
| #1 | flag_page-1022818.patch | 1.1 KB | Ravi.J |
Comments
Comment #1
Ravi.J commentedPatch attached
Comment #2
Ravi.J commentedComment #3
alexpottI think this needs to be wrapped with
Other than that this makes perfect sense.
Comment #4
Ravi.J commentedRevised patch
Comment #5
alexpottI'm not sure that the relationship options make sense. See image attached.
Especially the line "Include only flagged content" as means is "Include only pages with flag notes". To limit the flag_page views to only flagged content you need to include a different relationship.
Need to write a relationship handler or override the flag_handler_relationship_content for this purpose.
Comment #6
alexpottThe patch attached provides a new relationship handler which has improved configuration options and messages (see attached image)
Comment #7
rickvug commentedThe patch in #6 is working well for me. I'm now able to build a "my bookmarks" block that use's a user's individual bookmark titles provided by flag note. No problems encountered.
Comment #8
rickvug commentedCode looks clean and follows all standards from what I can see. Seeing as how two developers have touched this patch and it is working for multiple users I'm marking RTBC.
Comment #9
alexpottFeature included 6.x-2.1 release.