Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error on "twitter search" #280

Open
hillz1 opened this issue Feb 24, 2015 · 6 comments
Open

Error on "twitter search" #280

hillz1 opened this issue Feb 24, 2015 · 6 comments
Labels

Comments

@hillz1
Copy link

hillz1 commented Feb 24, 2015

When I execute "twitter search mykeyword" it returns this error,http://s15.postimg.org/3wwdegnkr/angel_beats_bg5.jpg seems that you're still using API v1 on that twitter search, please update it to the latest version

@RouxRC RouxRC added the CLI label Feb 26, 2015
@Zearin
Copy link
Contributor

Zearin commented Dec 15, 2015

👍 Just happened to me, too. Weird…!

@RouxRC
Copy link
Member

RouxRC commented Dec 15, 2015

The whole Search action through the command line client is completely outdated (see https://github.com/sixohsix/twitter/blob/master/twitter/cmdline.py#L469-L486 ) and should be completely rewirtten using the classical api route, it should be quite easy to do.

@OAuthBringer
Copy link

So, I've pushed a hotfix on my fork of the repo here

I've got something working for my own purposes for the moment, but I was unable to track the source of the issue. There are some trickle down formatting issues that will need to be resolved in the API update including date formats and key lookups.

The redefinition of twitter.domain is now unnecessary as the search api follows the same root domain as everything else. The issue is that search is being added as a uriparts somewhere between the SearchAction and the TwitterCall. I'm assuming this is in the search method itself, but I was unable to identify where this is actually defined, and ultimately I got tired of running down the rabbit hole. I'd be happy to patch it if someone who knows the lib a bit more can point me in the direciton of the Twitter().search() method definition, or explain how/where it is that the search term is being added to uriparts

I tried overwriting it in SearchAction itself, but this is too early in the process.

@RouxRC
Copy link
Member

RouxRC commented Jul 24, 2016

Hi,
Cool you started fixing it. However, there's no reason to do such a dirty hack, you can simply call the twitter search route by doing : twitter.search.tweets(args). @sixohsix built this lib so that calling any route remains generic whatever twitter decides to change :) (basically undefined methods are recursively added to uriparts)

@RouxRC
Copy link
Member

RouxRC commented Jul 24, 2016

@OAuthBringer
Copy link

Ah, that clarifies things a great deal. That is pretty straightforward.
I'll patch that and push something up soon. I still want to understand the
inputs as some of the search action options are non functional.

On Sun, Jul 24, 2016, 9:36 AM Benjamin Ooghe-Tabanou <
[email protected]> wrote:

see here
https://github.com/sixohsix/twitter/blob/master/twitter/api.py#L195-L208


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#280 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIZ79cmXh8MzdZlIGfo3J-_-O1lOErMeks5qY3hVgaJpZM4Dkq4N
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants