Posted by j0k3z on February 11, 2010 at 3:58am
| Project: | Twitter Pull |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
Would it be possible to use this module to grab the last X tweets from a specific list of twitter users?
Comments
#1
Attached 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().
#2
I 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!
#3
Thanks 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:
<?php print twitter_pull_render('@josuevalencia/doodo', 20); ?>#4
Tanks 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.
#5
Oh 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:
<?php print twitter_pull_render('@josuevalencia/doodo', NULL, 20); ?>To include a title, put quotations marks (single or double) around it:
<?php print twitter_pull_render('@josuevalencia/doodo', 'Tweets', 20); ?>#6
Thank you, it works fine for me ;-)
#7
@detour,
patch looks fine to me. You have CVS access, now, so do you mind to, please commit it?
Thank you.
#8
Alright, I've committed this patch to the dev branch. A new development release should show up in the next day.
#9
Automatically closed -- issue fixed for 2 weeks with no activity.