Error with Web Intents field [#1944174]

When I added the Web Intents field to the Twitter view, I get errors of:

Notice: Undefined property: stdClass::$twitter_id in twitter_views_handler_field_web_intents->render() (line 100 of /.../sites/all/modules/twitter/twitter_views_field_handlers.inc).

This is on a new/test twitter account with a single "test" tweet. After some hours troubleshooting, I found that adding the Twitter status message ID field before the Web Intents field resolved this.

Can a field in a view be marked as dependent on another field? (ie. web_intents depends on twitter_id) If so, consider this a request to do so. If not, maybe this mention will help someone with the same problem in the future.

Comment	File	Size	Author
#2	twitter-error_with_web_intents-1944174-2.patch	756 bytes	yanniboi
#2

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

juampynr CreditAttribution: juampynr commented 17 March 2013 at 05:27

Version:

7.x-5.6

» 7.x-6.x-dev

I could not find a way to find out the Twitter ID automatically. Patches are welcome.

Comment #2

yanniboi CreditAttribution: yanniboi commented 11 April 2013 at 16:19

Status:

Active

» Needs review

File	Size
twitter-error_with_web_intents-1944174-2.patch	756 bytes

Here is a patch against 7.x-5.6.

I just added the twitter_id to the construct function and called it in the query function. Will repatch against version 7.x-6.x shortly.

Comment #3

yanniboi CreditAttribution: yanniboi commented 11 April 2013 at 16:35

Quick head up, the above patch applies cleanly to the latest version of 7.x-6.x.

Comment #4

juampynr CreditAttribution: juampynr commented 11 April 2013 at 16:47

Status:

Needs review

» Fixed

Outstanding @yanniboi, I did not know that using additional_fields was the way to solve that field dependency. Committed.

Thanks!

http://drupalcode.org/project/twitter.git/commit/053b7b7

Comment #5

yanniboi CreditAttribution: yanniboi commented 11 April 2013 at 18:05

Status:

Fixed

» Needs review

As with my post (#22) on #1924478: Invalid Twitter OAuth request, it would be really helpful if you could commit this against the 7.x-5.6 release as well as 7.x-6.x.

Thanks!