Warning: Missing argument 2 for Nelio_Content_External_Featured_Image

Hi!

Thanks for the heads up. I’m quite surprised you’re getting this warning. The function that’s “causing” the warning (maybe_add_efi_as_background) has several parameters and is used as a callback from a WordPress filer:

add_filter( 'post_thumbnail_html', array( $this, 'maybe_add_efi_as_background' ), 10, 4 );

If you take a look at WordPress docs, you’ll see that this filter has more than one parameter:

apply_filters( 'post_thumbnail_html',
  string $html, int $post_id, string $post_thumbnail_id,
  string|array $size, string $attr )

so I don’t understand why you’re getting the warning (the second argument shouldn’t be missing). Anyway, here’s how you can fix it until we release a new version of the plugin:

1. Edit the file public/class-nelio-content-external-featured-image-public.php in our plugin
2. Go to line 285, where function maybe_add_efi_as_background is defined
3. The second parameter is $post_id. Set a default value for this parameter: $post_id = 0
4. The final result should look like this:

public function maybe_add_efi_as_background( $html, $post_id = 0, …

Alternatively, you can disable warnings in your server (it’s usually a good idea to disable debug options in a production site).

Thanks again for your feedback.

Regards,
David

• This reply was modified 8 years, 10 months ago by David Aguilera.

Great, @capson!