No Subscribe2 notification override displayed in edit mode

  • So,

    The “Subscribe2 notification override” block is empty with the latest wordpress + s2 plugin, also there is a corresponding error message in the logs:

    PHP Warning: call_user_func() expects parameter 1 to be a valid callback, class ‘S2_Admin’ does not have a method ‘s2-meta-box’ in ../wp-admin/includes/template.php on line 1050,…

    IT look like the function name setup in a callback is invalid, i searched and found s2-meta-box in class-s2-admin.php and made the following modification:

    line 290 of classes/class-s2-admin.php:
    add_meta_box( ‘subscribe2’, __( ‘Subscribe2 Notification Override’, ‘subscribe2’ ), array( &$this, ‘s2-meta-box’ ), $s2_post_type, ‘advanced’ );

    to
    add_meta_box( ‘subscribe2’, __( ‘Subscribe2 Notification Override’, ‘subscribe2’ ), array( &$this, ‘s2_override_meta’ ), $s2_post_type, ‘advanced’ );

    Which is the name of the function just following, look like it’s working

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

  • @doyousoft

    Your fix is perfectly valid – great bug catch. I’ll try to contact someone with commit privileges to get this fixed in the next version.

    I made this exact change and now I get this on any wp-admin page. Any ideas what I messed up?

    Parse error: syntax error, unexpected ‘Notification’ (T_STRING) in /home/content/25/2981525/html/warblogle/wp-content/plugins/subscribe2/classes/class-s2-admin.php on line 291

    @warblogle

    I suspect you have copied and pasted the code from above which is using textured quotes so ` instead of ‘. THat’s not valid PHP and is therefore giving you a syntax error. Use straight quotes and it should resolve the issue.

    Thread Starter silmaril

    (@doyousoft)

    Yes,

    the only change to be made is replacing the string “s2-meta-box” with “s2_override_meta”.

    That worked. Thanks!

    FCCC

    (@brightcrossfire)

    I also have this error message:
    Warning: call_user_func() expects parameter 1 to be a valid callback, class ‘S2_Admin’ does not have a method ‘s2_meta_box’ in /home/fcccorg/public_html/wp-admin/includes/template.php on line 1045

    Could you please explain where I can search to find the string “s2-meta-box” so I can replace it?

    Thank you!

    Thread Starter silmaril

    (@doyousoft)

    i don’t see what more you need, everything is in the thread …

    FCCC

    (@brightcrossfire)

    I’m completely new to WP so I’m not sure where I can locate the error in order to correct it. Thank you for your patience.

Viewing 8 replies - 1 through 8 (of 8 total)
  • The topic ‘No Subscribe2 notification override displayed in edit mode’ is closed to new replies.