PHP Warning: strpos(): Empty needle

  • Resolved breezynetworks

    (@breezynetworks)


    7 years, 4 months ago

    Hello,

    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

    Please can you help?

Viewing 6 replies - 1 through 6 (of 6 total)
  • Thread Starter breezynetworks

    (@breezynetworks)

    7 years, 4 months ago

    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 🙂

    Thread Starter breezynetworks

    (@breezynetworks)

    7 years, 4 months ago

    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.

    Israel Barragan

    (@reedyseth)

    7 years, 4 months ago

    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.

    Thread Starter breezynetworks

    (@breezynetworks)

    7 years, 4 months ago

    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!

    Israel Barragan

    (@reedyseth)

    7 years, 4 months ago

    Awesome, glad that I could help.

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

    Israel Barragan

    (@reedyseth)

    7 years, 3 months ago

    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

Viewing 6 replies - 1 through 6 (of 6 total)

The topic ‘PHP Warning: strpos(): Empty needle’ is closed to new replies.

Tags