Needs review
Project:
Twitter Pull
Version:
6.x-1.x-dev
Component:
Code
Priority:
Major
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
2 Mar 2010 at 03:50 UTC
Updated:
4 Feb 2012 at 22:26 UTC
I needed a way to create embed the feed into content, so I wrote up the following filter.
It accepts all of the 4 arguments from comma seperated list in the tag [twitter-pull:ARGUMENTS].
From lessions learnt in other modules, I'm using double quote support to escape the title allowing titles like "My account, other title stuff".
Also, rather than returning an error message, I check for the existence of the theme instead, and I use the default (NULL) if not found.
[twitter-pull:URL]
[twitter-pull:URL,TITLE,NUM,THEMEKEY]
These insert Twitter feed content based on the URL, (e.g. @username or #hashtag) and optional title (TITLE), feed number (NUM), and theme key (THEMEKEY) properties.
| Comment | File | Size | Author |
|---|---|---|---|
| twitter_pull-input-filter-wrapper-for-twitter_pull_render .patch | 4.12 KB | alan d. |
Comments
Comment #1
alan d. commentedAs an aside, is there an easy way to create a listing based on a singular list from the user? So rather than @user , I'm looking for @user/list-of-interest
And any reason for the number limit? 20 doesn't sound like much. (I do not know what the Twitter API limits are)
Cheers
Comment #2
irakli commented@Alan D.
Lists are now supported.
20 is the limit of the API itself.
Thank you for the patch.
Comment #3
moshe weitzman commentedInput filter seems handy an in the spirit of this module
Comment #4
irakli commentedBumping up the priority. Would like to get to this as soon as I can.
Patch needs to be re-rolled and we need a similar patch for D7, though :(
Any help would be highly appreciated.
Thanks
Comment #5
cookiesunshinex commentedI tried to apply this patch and received the following:
patching file twitter_pull.module
Hunk #1 FAILED at 39.
Hunk #2 FAILED at 73.
Hunk #3 succeeded at 210 with fuzz 1 (offset 84 lines).
Hunk #4 FAILED at 241.
3 out of 4 hunks FAILED -- saving rejects to file twitter_pull.module.rej
Comment #6
ken hawkins commentedIs there still interest in this? If a new patch was submitted would it be applied?