WPIntell

Source evidence

PHP 8 compatibility fix / request

Good Old Twitter Feed Widget · support · 2022-07-20T12:25:00+00:00

neutralsentiment
mediumseverity
0.53relevance
2replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

3 / 23 rows with source links

13.0% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

20 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
gregbob resolved
Hello, Running this plugin on php 8 results in the following error: Warning: Undefined array key “images” in …/wp-content/plugins/good-old-twitter-feed-widget/good_old_twitter_widget.php on line 609 Please consider changing lines 607 – 611 of good_old_twitter_widget.php from if ( $options['images'] && isset($message['entities']['media'][0]['media_url_https']) && $message['entities']['media'][0]['media_url_https'] != '' ) { to if ( !empty($options['images']) && $options['images'] && isset($message['entities']['media'][0]['media_url_https']) && $message['entities']['media'][0]['media_url_https'] != '' ) { Thank you! To be accurate this is only a warning, but php 8 will display it on the frontend by default, even with wp_debug set to “false”. Default error reporting is set to E_ALL in php 8. https://php.watch/versions/8.0/error-display-E_ALL Hi Gregbob, the issue is now fixed in the 1.2.6 release. Thanks for reporting it!

Comments

2 shown
gregbob 2022-07-20T12:34:00+00:00

To be accurate this is only a warning, but php 8 will display it on the frontend by default, even with wp_debug set to “false”. Default error reporting is set to E_ALL in php 8. https://php.watch/versions/8.0/error-display-E_ALL

whiletrue 2022-07-26T10:13:00+00:00

Hi Gregbob, the issue is now fixed in the 1.2.6 release. Thanks for reporting it!