The generated links are no longer working since yesterday, as twitter.com made a change. I don't know, what to change in order to make this nice modul run again.

A similiar problem in another tool was solved, maybe it helps to determine, what needs to be changed?
http://drupal.org/node/1161974

Thanks in advance!

Comments

icecreamyou’s picture

icecreamyou commented
Status: Active » Fixed

Committed a fix and created new releases in the 6.x-4.x and 7.x-4.x branches. The 5.x branch has not been maintained since Drupal 7 came out, but you can change it yourself by changing the URL in line 110 of tweet.module from http://twitter.com/home to just http://twitter.com/.

Allthegearnoidea’s picture

Allthegearnoidea commented

Thanks for that fix.. Changing the URL worked.

Thanks again.

MSVZebras’s picture

MSVZebras commented

Thanks for your quick reply, I've changed it and at least that is working again. But what is still strange, is the fact, that blanks from the title and the tweet settings (from the format-field) are changed to "+". This is also something, that happened since the new twitter version came out.

Any idea how to fix this? :o) - Thanks a lot!

icecreamyou’s picture

icecreamyou commented
Status: Fixed » Needs work

Ah, crap. I think that changed even since the last update. The fix just requires changing + to %20 in the URL -- will get to it when I get a chance.

MSVZebras’s picture

MSVZebras commented

I think that this is something in this section then?

//Spaces are manually encoded to plus signs for clarity of whitespace at the end of the tweet.
  $format = str_replace(array('%2523', '%2526', '%252F', '%20'), array('%23', '%26', '%2F', '+'), $format);

...but I honestly don't know, what to change, as I'm not into such stuff ;) - I tried to change the + into %20 but that didn't worked for me.

icecreamyou’s picture

icecreamyou commented

You just have to drop the last replacement:

  $format = str_replace(array('%2523', '%2526', '%252F'), array('%23', '%26', '%2F'), $format);
MSVZebras’s picture

MSVZebras commented

Thanks again for your quick response, but that didn't helped me, I still have the plusses instead of blanks. Maybe something else, that triggers the replacement/format?

icecreamyou’s picture

icecreamyou commented
Status: Needs work » Fixed

Then you did it wrong. I can't help you more than that; 5.x isn't supported.

Fixed in 6.x-dev; this problem does not exist in 7.x.

MSVZebras’s picture

MSVZebras commented

If anyone has the same problem with plus-signs in the title instead of blanks after the twitter-update and when clicking on the tweet-icon of a node, that's what I did now...

Find this in the tweet.module:

$format = str_replace(array('%2523', '%2526', '%252F', '%20'), array('%23', '%26', '%2F', '+'), $format);

and change it to this with adding one rule:

$format = str_replace(array('%2523', '%2526', '%252F', '%20', '+'), array('%23', '%26', '%2F', '+', '%20'), $format);

That will replace the + to blanks, so no more issues with the title then.

Allthegearnoidea’s picture

Allthegearnoidea commented

Thanks for this, works perfectly now.

Thanks for sticking with it.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.