Warning message for missing arguments

  • Warning: Missing argument 2 for SimplyExclude::se_widget_getarchives_where() in ../wp-content/plugins/simply-exclude/simplyexclude.php on line 2526

    Getting this error on the front end even if WP_DEBUG set to false.

    Please replace
    function se_widget_getarchives_where($where_sql, $args)
    with
    function se_widget_getarchives_where($where_sql, $args = array())

    https://www.ads-software.com/plugins/simply-exclude/

Viewing 8 replies - 1 through 8 (of 8 total)
  • Plugin Author Paul Menard

    (@pmenard)

    @naresh Vachhani

    I have a patch for this but waiting on another user to confirm a different issue. should be released this coming weekend.

    interesting note here on your comment

    “Getting this error on the front end even if WP_DEBUG set to false.” this is not something controlled by Simply-Exclude. I know you know this so my comment is more for other users. If your hosting is displaying these PHP Warnings I would have them turn off PHP error_reporting as this can expose information about your hosting which can be used by others.

    Another interesting issue on this reported fix. Per your correct suggestion the second function argument is optional and needs a default valid. What I find confusing is within the WordPress core there is only one call to the filter ‘get archives_where’. This is from wp-includes/general-template.php line 1374. It is this filter there Simply-Exclude hooks into and connects to the plugin function se_widget_getarchives_where().

    If you view this this hook call (see below) you will see there is a second argument passed ‘$r’. So it is confusing as to why there is a need to have this set to a default value within the plugin code.

    $where = apply_filters( ‘getarchives_where’, “WHERE post_type = ‘post’ AND post_status = ‘publish'”, $r );

    P-

    Paul, has the fix been pushed? I’m also encountering this error on a page, and I’m running Simply Exclude 2.0.6.4.

    I ask because I see you mentioned this 6 hours ago and marked the subject as resolved, and there was also an update pushed around that time, but I’m not sure if 2.0.6.4 was the upcoming patch you mentioned in your comment.

    Thanks!

    wp-content/plugins/simply-exclude/simplyexclude.php on line 2527

    Problem also here, please fix inmediately.

    Do you have a version archive, so that we can roll back till then?

    This problem still exists with version “2.0.6.4”

    I will try to do a rollback from your archive page…

    https://www.ads-software.com/plugins/simply-exclude/developers/

    Yes, rollback to older version fixes this problem.

    @paul. Please do not mark issuses as “solved” if they are not. Especially if it is a critical issue like in this case!

    Plugin Author Paul Menard

    (@pmenard)

    Sorry about that. Somehow the fix did not get included. Just pushed 2.0.6.5. Let me know.

    Plugin Author Paul Menard

    (@pmenard)

    Appears somehow a beta version of the plugin was sent out instead of the simpler fix for 2.0.6.4 and 2.0.6.5. Hold tight. Trying to sort out the mess this morning. Will have 2.0.6.6 out shortly. Apologies.

    well,

    “[quack] happens” ??

    Thank you for fixing the issue!

    Great plugin anyway!

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘Warning message for missing arguments’ is closed to new replies.