As of June 11, 2013, the version of Twitter Pull installed with OPIC does not work. This patch updates that module and adds the Twitter and OAuth modules to the install profile to minimize impact on OPIC and provide a path to continue using any code built on top of the Twitter Pull module.

There were several options listed in the issue addressing this update below, but this one appeared to be the lowest impact for existing OPIC installations.

#1781024: Make the Twitter Pull module work when Twitter turns off the version 1 API

Patch to follow...

CommentFileSizeAuthor
#9 2060595-openpublic-filter-7.patch38.37 KBhefox
#8 2022583-twitter_pull-update-8-do-not-test.patch1.79 KBhefox
#7 2022583-twitter_pull-update-7.patch1.8 KBhefox
#5 2022583-twitter_pull-update-5.patch1.8 KBhefox
#4 2022583-twitter_pull-update-4.patch1.8 KBhefox
#1 openpublic-twitter_pull-update-2022583-01.patch761 bytesarpieb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arpieb’s picture

arpieb CreditAttribution: arpieb commented
Status: Active » Needs review
FileSize
openpublic-twitter_pull-update-2022583-01.patch761 bytes

... and here's the patch.

Grayside’s picture

Grayside CreditAttribution: Grayside commented
Priority: Normal » Major

As this was a functioning part of OpenPublic that is now blank and buggy, this seems a bit higher priority to fix.

The patch eyeballs good to me, though the release notes should be sure to mention that API credentials are now needed.

hefox’s picture

hefox CreditAttribution: hefox commented
Status: Needs review » Needs work

Doesn't do enough; should be an update hook to enable new modules at least imo

#2051533: Enable twitter module via update hook if exists

hefox’s picture

hefox CreditAttribution: hefox commented
Status: Needs work » Needs review
FileSize
2022583-twitter_pull-update-4.patch1.8 KB
hefox’s picture

hefox CreditAttribution: hefox commented
FileSize
2022583-twitter_pull-update-5.patch1.8 KB

non-https patch

hefox’s picture

hefox CreditAttribution: hefox commented
Status: Needs review » Needs work

oauth => oauth_common >.O

hefox’s picture

hefox CreditAttribution: hefox commented
Status: Needs work » Needs review
FileSize
2022583-twitter_pull-update-7.patch1.8 KB
hefox’s picture

hefox CreditAttribution: hefox commented
FileSize
2022583-twitter_pull-update-8-do-not-test.patch1.79 KB

Ignore, need one that works with [#2060595[

hefox’s picture

hefox CreditAttribution: hefox commented
FileSize
2060595-openpublic-filter-7.patch38.37 KB

need newer version of features

hefox’s picture

hefox CreditAttribution: hefox commented
Issue summary: View changes
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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