Twitter protocol - retweets are missing tweet body

[omniwolf]

Hi guys,

Just updated to the development set of plugins (I'm using the plugin updater and chose development, so i'm not running a mix of prod and dev).
I have noticed that the twitter protocol now truncates retweets.
Where it should say:

abcnews: RT @abcnewsPerth Perth news newsy news blah blah

it just says:

abcnews: RT @abcnewsPerth

All retweets seem to have this issue - the message only contains the tweeter's name, but not their actual tweet.
I'm using the group chat feature, if that helps.

Thanks.

[omniwolf]

bump!

If i reload the non development version of miranda and twitter, then retweets work again, so i know it's not a Twitter API issue, it's a miranda issue.
I had a quick look at the source, the RT code is from line 228 in the twitter.cpp code.  Ghazan made some changes to this code about 5 weeks ago here:

http://trac.miranda-ng.org/changeset/13835/

and i guess he didn't test it. 

[Wishmaster]

http://trac.miranda-ng.org/changeset/13835/ and i guess he didn't test it.

Hm, I don't use Twitter, but quickly looking at that commit, it seems to be correct (assuming that the previous code worked, I didn't check the exact JSON format). Maybe I'll add some more logging.

[Robyer]

I think it didn't worked even before. I was using twitter plugin for few months and I never saw correct retweet's text (at least I don't remember that). So it must be older change...

[Wishmaster]

I believe I fixed it, here is x64 version, please test!

PS: It was appearently r13248 (after "// here we grab the "retweeted_status" um.. section?  it's in here that all the info we need is")

[Robyer]

Wishmaster, could you maybe commit your changes?

Or provide also 32 bit version, please.

[Wishmaster]

Wishmaster, could you maybe commit your changes? Or provide also 32 bit version, please.

Done

[Robyer]

Wishmaster, hmm, probably it's still not fixed.

I'm seeing just "RT @ " as content of retweeted tweets.

[Wishmaster]

Please try this patch

[Wishmaster]

Please try this patch

Okay, I just comitted it. Please give feedback.

[omniwolf]

If you can post a 64bit compilation i can test

[watcher]

omniwolf, you can already test it, its in updater, check for updates.

[omniwolf]

hmm i tried and there was no update, but it's come through this morning.
The issue is fixed, thanks