Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The API URLs for most Twitter operations must be versioned, eg. https://api.twitter.com/
1.1/statuses/update.json
However, the API URLs for oAuth operations must not versioned, eg. https://api.twitter.com/oauth/authenticate
This has also been discussed in #1809720: On cron: Unexpected error: Sorry, that page does not exist
Comment | File | Size | Author |
---|---|---|---|
#7 | 1814610-proper-api-urls-6.x-3.x.patch | 1.77 KB | fizk |
#1 | 1814610-proper-api-urls.patch | 1.44 KB | fizk |
Comments
Comment #1
fizk CreditAttribution: fizk commentedHere's a patch.
Comment #2
fizk CreditAttribution: fizk commentedComment #3
fizk CreditAttribution: fizk commentedComment #3.0
fizk CreditAttribution: fizk commentedClarify description.
Comment #4
tbenice CreditAttribution: tbenice commentedPatch #1 worked. Thanks.
Comment #5
ytsuhako CreditAttribution: ytsuhako commentedPatch #1 worked me too. thank you!
Comment #6
juampynr CreditAttribution: juampynr commentedCommitted. Many thanks.
http://drupalcode.org/project/twitter.git/commitdiff/67814a1
¿Any chance to backport this for Drupal 6?
Comment #7
fizk CreditAttribution: fizk commentedPatch backported to 6.x-3.x-dev
Comment #8
joestewart CreditAttribution: joestewart commented@juampy - the patch in #7 applies cleanly to 6.x-3.x and corrects the issue.
Comment #9
scottrouse CreditAttribution: scottrouse commentedAgreed. Patch in #7 works for 6.x-3.x-dev.
Comment #10
xurizaemonCommitted in 118d859
Comment #11
juampynr CreditAttribution: juampynr commentedMany thanks @fizk!
Comment #12
fizk CreditAttribution: fizk commented@juampy, @grobot, Thanks :) Could you also take a look at #1814642: Invalid handling of exceptions causes fatal errors on PHP5.2
Comment #13
lyricnz CreditAttribution: lyricnz commentedSuggest create a release including this fix, as twitter API is broken without it.
Comment #14
xurizaemonIMO this should go forward into 7.x-4.x also, but if #1569982: Integrate with Libraries API is retained then doing so is outside the scope of this module.
Comment #15
fizk CreditAttribution: fizk commentedWhatever the outcome of #1569982: Integrate with Libraries API is, I think the fixes in this issue should be released immediately.
Comment #16
xurizaemonAgreed. Happy to make this happen today. Issue for release co-ordination: #1820138: Release co-ordination for Twitter 7.x-3.3 & 6.x-3.0
Comment #17.0
(not verified) CreditAttribution: commentedLink to other discussion