Import fails to authenticate, tweet_feed doesn't handle the error

Created on 14 July 2023, over 1 year ago

Problem/Motivation

Importing a feed using tweet_feed 7.x-1.x currently hits an error processing the Twitter responses:

TypeError: array_pop(): Argument #1 ($array) must be of type array, stdClass given in array_pop() [error]
(line 274 of tweet_feed.module).

I added print_r($tdata) just above that line and this is what it looks like:

stdClass Object
(
    [statuses] => stdClass Object
        (
            [errors] => Array
                (
                    [0] => stdClass Object
                        (
                            [code] => 32
                            [message] => Could not authenticate you.
                        )

                )

        )

)

Proposed resolution

Fix the bug.

Remaining tasks

Fix the bug.

User interface changes

Tweet import failures are handled gracefully and don't WSOD.

API changes

n/a

Data model changes

n/a

🐛 Bug report

Status

Needs review

Version

1.0

Component

Code

Created by

🇺🇸United States DamienMcKenna NH, USA

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @DamienMcKenna
Status changed to Needs review over 1 year ago12:08pm 14 July 2023
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
The problem appears to be that the data converted from json_decode() is stored as a stdClass objects, instead of an array, so passing the second argument to json_decode() to make it convert the data to an array resolves the problem and allows cron to continue.
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
A better approach - look for errors before processing individual tweets.
Comment over 1 year ago →
🇺🇸United States DamienMcKenna NH, USA
Better again - log the error via watchdog().

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024