WPIntell

Source evidence

Debug Errors in Version 1.5.9

FeedPress · support · 2014-08-05T04:43:00+00:00

mixedsentiment
mediumseverity
0.78relevance
3replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

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

4 / 33 rows with source links

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

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

29 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
Jeremiah resolved
Good Day! Just wanted to let you know about the following errors: PHP Notice: Undefined index: HTTP_USER_AGENT in /httpd/www/wp-content/plugins/feedpress/feedpress.php on line 803 PHP Notice: Undefined index: HTTP_USER_AGENT in /httpd/www/wp-content/plugins/feedpress/feedpress.php on line 804 PHP Notice: Undefined index: HTTP_USER_AGENT in /httpd/www/wp-content/plugins/feedpress/feedpress.php on line 805 PHP Notice: Undefined index: HTTP_USER_AGENT in /httpd/www/wp-content/plugins/feedpress/feedpress.php on line 806 I think you should be able to resolve the error by capturing it in a variable and using the variable instead. Untested Example: // Handle feed redirections $user_agent = isset($_SERVER['HTTP_USER_AGENT']) ? $_SERVER['HTTP_USER_AGENT'] : ''; if (!preg_match('/feedpress/i', $user_agent) && !preg_match('/uri\.lv/i', $user_agent) && !preg_match('/feedvalidator/i', $user_agent) && !preg_match('/googlebot/i', $user_agent)) { add_action('template_redirect', 'feedpress_redirect', 1); } https://wordpress.org/plugins/feedpress/ Thanks for spotting that. I’ve fixed it in the current version (I didn’t make a new version because we don’t recommend having PHP with notices enabled in production so it’s probably not a major issue). I’m testing the UA variable in the statement to avoid preg_matches on an empty var. Thanks again! You’re welcome, and thank you for the quick response. I’m just going through a client’s site identifying and cleaning up debug log errors and thought I’d let you know. Keep up the good work! Thanks a lot!

Comments

3 shown
maximevalette 2014-08-05T07:54:00+00:00

Thanks for spotting that. I’ve fixed it in the current version (I didn’t make a new version because we don’t recommend having PHP with notices enabled in production so it’s probably not a major issue). I’m testing the UA variable in the statement to avoid preg_matches on an empty var. Thanks again!

Jeremiah 2014-08-05T12:58:00+00:00

You’re welcome, and thank you for the quick response. I’m just going through a client’s site identifying and cleaning up debug log errors and thought I’d let you know. Keep up the good work!

maximevalette 2014-08-05T12:59:00+00:00

Thanks a lot!