PHP Warning: strpos(): Empty needle

A bit more information, I have looked that the PHP file and noticed this code on line 57:

if ( ( strpos( $_SERVER[“REQUEST_URI”], $manager_page_permalink ) !== false ) ) {

I think this is happening because I have left the management URL blank (after disabling the virtual management page), but I did this to remove the link from my view source.

Would be alright to remove some code to stop this happening?

Please let me know ??

• This reply was modified 6 years ago by breezynetworks.
• This reply was modified 6 years ago by breezynetworks.
• This reply was modified 6 years ago by breezynetworks.

Just to be clear, my goal is to remove this:

<!– Subscribe to comments Reloaded MP: ‘/comment-subscriptions/’ –>

From the source code on a post, and secondly, to stop the PHP errors from spamming my logs, which removing the management URL seemed to have done.

Hello @breezynetworks

Running the latest version of Subscribe to Comments Reloaded and my PHP logs are being spammed by this every day:

[24-Nov-2018 19:40:34 UTC] PHP Warning: strpos(): Empty needle in /www/wp-content/plugins/subscribe-to-comments-reloaded/wp_subscribe_reloaded.php on line 57

I was able to replicate this issue. I will work on the fix.

As for the comment section there is not nothing I can do, there is a purpose for it. You will have to remove it from the code.

Awesome, glad that I could help.

I ended up leaving the comment management page url in, but removing this from the code:

View post on imgur.com

Thanks!

Awesome, glad that I could help.

Thanks to you, I was not aware of this issue. Glad that you sort the commenting issue.

Hi @

I have fix this issue on the dev version of the code, it will we release on the next update, probably this weekend or the new week.

https://github.com/stcr/subscribe-to-comments-reloaded/commit/8d7740649d67474b1bb5accf41f99a4869b697e1