- integrate with the Location module or the Geo module?
- option to remove location similar to Twitter's policy (whatever that might happen to be)
Would love to be able to build views based on tweet location, such as a map.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | twitter-644954.patch | 11.82 KB | derhasi |
Comments
Comment #1
sirkitree commented++ this would be some sweet functionality!
Comment #2
benklocek commented+1 subscribe
Comment #3
derhasi commentedI added a patch for implementing storage of longitude and latitude of the tweet. Additionaly there are views fields for both values, that can be used by Openlayers & Co. This should fix the title Store tweet location.
Removing a location from twitter.com would be a new issue for the API, I guess. Implemenation with location's or geo's storage options should be an issue for a contrib module.
It would be nice someone could review the patch.
P.S.: sorry for the whitespace changes, but as it is part of the Drupal coding standards I configured my IDE to convert it automatically.
Comment #4
pribeh commentedsubscribe
Comment #5
derhasi commentedpribeh, did you test the attached patch of #3?
Comment #6
pribeh commentedHi darhasi. I will get to testing this today and report back. I'm just setting things up on a dev site to test this.
Comment #7
pribeh commentedTesting of this patch will have to wait until tomorrow. Thanks derhasi for putting your efforts into this. I do plan on using this on a live site if this works out.
Comment #8
greg.harveyWow, just looking in to this for a client. Good to find the beginnings of a patch. I'll need to do what Richard needed to do, so if we follow this through with our customer we'll be able to test this and hopefully get it finished.
For now, subscribe and +1!
Comment #9
mrf commentedStill seems like a good idea to me, but should be done on 7.x-5.x-dev first.
Comment #10
jasonflaherty commentedJust following up with this request. Was it implemented in 7.x-5.x-dev or into the latest module?
Comment #11
damienmckenna@jasonflaherty: It hasn't been added to any branch yet.
Comment #12
damienmckennaComment #13
damienmckenna