Empty Needle Warning

  • Resolved CandleFOREX

    (@candleforex)


    11 years, 10 months ago

    Hi,

    First of all thank you for such a great plugin. We use your plugin instead of the W3 Total Cache JS and CSS compression (it breaks the site).

    Anyway, there is a little problem with the plugin since the last update.

    The problem is that we get an empty needle warning. Here are two examples:

    [10-May-2013 19:33:09 UTC] PHP Warning: strpos() [function.strpos]: Empty needle in /home/USERNAME/public_html/wp-content/plugins/autoptimize/classes/autoptimizeScripts.php on line 229
    [10-May-2013 19:33:09 UTC] PHP Warning: strpos() [function.strpos]: Empty needle in /home/USERNAME/public_html/wp-content/plugins/autoptimize/classes/autoptimizeScripts.php on line 255

    It seems every load of the page generates 20 of these lines. This is not a lot, but when you have hundreds of people a day doing a page loads, this can end up with an error log of several hundred MB a week.

    Any chance you can put out a fix for this please?

    Our site if you need it is:
    https://MetaTraderProgramming.com

    Thanks

    https://www.ads-software.com/extend/plugins/autoptimize/

Viewing 9 replies - 1 through 9 (of 9 total)
  • Plugin Author Frank Goossens

    (@futtta)

    Don’t have that error, but the dontmove-array changes with the value of the exclusion-list in the autoptimize-admin screen. What values did you configure there CandleForex? And do (or don’t) you get errors complaining about the array_merge on line 31?

    Thread Starter CandleFOREX

    (@candleforex)

    Wow..I wasnt expecting such a fast reply thanks!

    What I have for the “Exclude scripts from autoptimize:” area is:

    s_sid,smowtion_size,sc_project,WAU_,wau_add,comment-form-quicktags,edToolbar,ch_client,buzz,

    Plugin Author Frank Goossens

    (@futtta)

    can you remove the last comma and see what happens?

    Plugin Author Frank Goossens

    (@futtta)

    Better yet; open up wp-content/plugins/autoptimize/classes/autoptimizeScripts.php and on line 30 change

    $exclJSArr = array_map('trim',explode(",",$excludeJS));

    into

    $exclJSArr = array_filter(array_map('trim',explode(",",$excludeJS)));

    This should solve PHP warnings about empty needles/ delimiters.

    Thread Starter CandleFOREX

    (@candleforex)

    My apologies on the delay.

    Thank you these changes have been made a few minutes ago.
    Will update you as to how it goes (logs run every 24 hours so we got to wait).

    Thank you!

    Plugin Author Frank Goossens

    (@futtta)

    anything in your logs CandleForex?

    Thread Starter CandleFOREX

    (@candleforex)

    Thank you very much for the reminder!

    No, after I made that code change, the error went away totally.
    So again thanks!

    Maybe in the next version of your plugin, you could include this fix?
    Would probabblly cut down on support requests, or perhaps it was just us?

    Plugin Author Frank Goossens

    (@futtta)

    Thanks for your feedback, this will most certainly be included in the next version!

    Plugin Author Frank Goossens

    (@futtta)

    I just pushed out 1.6.4 with this fix in it, hope you like it ??

Viewing 9 replies - 1 through 9 (of 9 total)
  • The topic ‘Empty Needle Warning’ is closed to new replies.