It would be useful to have a feature that allows for combining multiple feeds into one. This might only make sense for Twitter and Facebook so I think it probably makes sense to start there. I'll plan to follow up with a patch here.

CommentFileSizeAuthor
#2 combined_feeds_block-2678338-2.patch9.3 KBtandroid

Comments

tandroid created an issue. See original summary.

tandroid’s picture

Status: Active » Needs review
StatusFileSize
new9.3 KB

Patch attached for testing.

Anonymous’s picture

Status: Needs review » Needs work

This is a great idea for a new feature! I've downloaded this module, enabled it and applied the patch from #2. There are a few issues that I've encountered and configuration ideas I had in the process of setting up:

  1. When I try to apply the patch, I get Reversed (or previously applied) patch detected!. Being that this is the first time I've downloaded and installed this module, I'm unsure of why this would be happening.
  2. After clearing the cache, the combine tab will show up when editing the social feed settings, but it looks like the facebook link disappears from the admin menu for some reason. Combine is in its place.
  3. It might make sense to have the twitter and facebook checkboxes already selected. It would also be great to include instagram as an option.
  4. When adding the block to the page, numerous notices pop up on a bunch of different lines within the .module file (this is a condensed list):
    Notice: Undefined index: type in socialfeed_facebook_data() (line 293 of socialfeed.module).
    Notice: Undefined index: type in socialfeed_facebook_data() (line 298 of socialfeed.module).
    Notice: Undefined index: link in socialfeed_facebook_data() (line 307 of socialfeed.module).
    Notice: Undefined index: meta in socialfeed_combined_sort() (line 534 of socialfeed.module).
    Warning: usort(): Array was modified by the user comparison function in socialfeed_combined_posts() (line 523 of socialfeed.module).
  5. I noticed that the patch dropped the combine.tpl.php file into the root folder instead of the templates directory. I manually moved the file over and the block rendered on the page. Both the twitter and facebook items are showing as expected, ordered by date.

Hopefully this helps! Let me know if you'd like any other information provided.

hemangi.gokhale’s picture

I think this patch is a great idea, but I'm struggling to get it to apply. Looks like it's in need of a re-roll at this point.

hemangi.gokhale’s picture

Assigned: tandroid » Unassigned
Status: Needs work » Closed (outdated)

Marking this issue as closed. Please note that Drupal 7, 8, or 9 releases are no longer officially supported. If anyone continues to experience this problem with the newer releases of Drupal 10 or 11, please feel free to open a new issue. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.