Closed (fixed)
Project:
Twitter Pull
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
11 Feb 2010 at 03:58 UTC
Updated:
14 Apr 2010 at 23:30 UTC
Jump to comment: Most recent file
Comments
Comment #1
arithmetric commentedAttached is a patch that adds support for Twitter lists. This allows getting a feed from a Twitter user's list by using something like '@acquia/acquians' as the Twitter key in a call to twitter_pull_render().
Comment #2
josue.valencia commentedI have applied the patch and it works fine. I tried to put some arguments but I can't make it work. Example:
<?php print twitter_pull_render ('@josuevalencia/doodo', $num_items = 20); ?>I hardcoded $num_items to 10 in twitter_pull.module to test passing arguments in php function.
You can see te module in action: http://doodo.estudiobaobab.com/blog
Maybe I am doing something wrong.
Nice module!
Comment #3
arithmetric commentedThanks for testing this patch. I'm using this change on a production site too, so I'm marking it reviewed and tested.
To use the argument that limits the number of tweets displayed, you should add it like this:
Comment #4
josue.valencia commentedTanks so much.
I've got to add the 'title' argument as well because it doesn't display any tweets without it, only 'No tweets' message. So, this is the code I am using in http://www.estudiobaobab.com/blog:
<?php print twitter_pull_render ('@n3storm/estudiobaobab', Tweets, 10); ?>I've paste this code in a block and it works fine. I don't know yet if this can cause some performance issues.
Comment #5
arithmetric commentedOh yes, there's a mistake in my code above. The title argument is missing, and if you want to use the default, you must include NULL as a placeholder. So the code should be:
To include a title, put quotations marks (single or double) around it:
Comment #6
josue.valencia commentedThank you, it works fine for me ;-)
Comment #7
irakli commented@detour,
patch looks fine to me. You have CVS access, now, so do you mind to, please commit it?
Thank you.
Comment #8
arithmetric commentedAlright, I've committed this patch to the dev branch. A new development release should show up in the next day.