You need to authenticate at least one Twitter account in order to use the Twitter API. Please fill out the OAuth fields at Twitter Settings and then return here.

The link there to go to Twitter Settings is absolute and go to http://www.MYSITE.com/admin/config/services/twitter/settings but the real link should be relative to include my subfolder: http://www.MYSITE.com/SUBFOLDER/admin/config/services/twitter/settings

CommentFileSizeAuthor
#6 twitter-status-report-links-now-use-l-1884064-6.patch1.14 KBjgSnell
#4 twitter-proper-use-of-l-1884064-4.patch4.39 KBe0ipso

Comments

warmth’s picture

warmth commented
Version: 7.x-5.2 » 7.x-5.3
warmth’s picture

warmth commented

Same for:

Tweets are pulled from Twitter by running cron. You can view the full list of tweets at the Tweets view.

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Assigned: Unassigned » e0ipso

Same bug also found in Twitter install requirements and Twitter Actions (You first need to authenticate a Twitter account for posting to Twitter).

e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Version: 7.x-5.3 » 7.x-5.x-dev
Status: Active » Needs review
StatusFileSize
new twitter-proper-use-of-l-1884064-4.patch4.39 KB

This patch should fix all six cases.

13rac1’s picture

13rac1 commented
Status: Needs review » Needs work

Patch doesn't apply. Please make a git patch.

$ git apply twitter-proper-use-of-l-1884064-4.patch 
error: patch failed: twitter.pages.inc:84
error: twitter.pages.inc: patch does not apply

$ patch -p1 < twitter-proper-use-of-l-1884064-4.patch 
patching file twitter.install
patching file twitter.pages.inc
Hunk #1 succeeded at 89 with fuzz 1 (offset 5 lines).
Hunk #2 FAILED at 105.
1 out of 2 hunks FAILED -- saving rejects to file twitter.pages.inc.rej
patching file twitter_actions/twitter_actions.module
13rac1’s picture

13rac1 commented
Issue summary: View changes

update

jgSnell’s picture

jgSnell commented
Issue summary: View changes
Status: Needs work » Needs review
StatusFileSize
new twitter-status-report-links-now-use-l-1884064-6.patch1.14 KB

All of these mentioned appear to have been fixed except for the link on the report status page. Patch attached.

dgeo’s picture

dgeo commented
Status: Needs review » Reviewed & tested by the community

Same bug here, #6 patch applies to 7.x-5.x branch and corrects the link... please commit !

… and thank you jgSnell !

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Needs review
Parent issue: » #2402307: Plan for Twitter v7.x-5.9 release

Triggering the testbot.

The last submitted patch, 4: twitter-proper-use-of-l-1884064-4.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 6: twitter-status-report-links-now-use-l-1884064-6.patch, failed testing.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: e0ipso » Unassigned

  • DamienMcKenna committed 4a89dcd on 7.x-5.x authored by jgSnell 
    Issue #1884064 by jgSnell, e0ipso: Link path to configure is wrong in...

  • DamienMcKenna committed f3e8a30 on 6.x-5.x authored by jgSnell 
    Issue #1884064 by jgSnell, e0ipso: Link path to configure is wrong in...

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Fixed

Committed to 6.x-5.x, 7.x-5.x and 7.x-6.x. Thanks!

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Closed a duplicate: #2236737: hook_requirements doesn't respect Drupal base path

Status: Fixed » Closed (fixed)

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