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.

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

juampynr’s picture

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.

yanniboi’s picture

Status: Active » Needs review
FileSize
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.

yanniboi’s picture

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

juampynr’s picture

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

yanniboi’s picture

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!

juampynr’s picture

Version: 7.x-6.x-dev » 7.x-5.x-dev
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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