@dhunink
Thank you for reporting the issue.
Can you provide some more detail about which theme is being used and which WordPress version are you using?
Also, can you list other plugins that are being used on the sites for menus?
On line 124, a function is overriding the default behaviour of menus via filter and as per documentation, the filter walker_nav_menu_start_el should be providing $item object.
https://developer.wordpress.org/reference/hooks/walker_nav_menu_start_el/
If some other plugin is generating menus, without adhering to WordPress’ standard, and not providing that object, then it would be causing the warning.
In the meantime I’ll add an additional check in the plugin to see if the $item object has certain properties or not, to rectify this issue.
Hi!
I’ve done some more debugging and nailed it down to specific situation where there’s no menu set for a menu location. Happens a lot when the plugin is installed on a new installation where the user did not yet created menu’s and/or assigned these menu’s to a menu location.
In that case the $item object is indeed empty, which is normal behavior. But the code always assumes the $item object consists of something.
My suggestion:
add a simple check in the function that checks if the $item object has the desired content. If not, the function does not need to continue.
Replacing :
if ( 'gs_sim' !== $item->object ) {
with:
if (!isset($item->object) || $item->object !== 'gs_sim'){
and replacing
if ('FULL HTML OUTPUT' === $item->post_title ) {
with
if (isset($item->post_title) && 'FULL HTML OUTPUT' === $item->post_title ) {
In the file: wp-content/plugins/shortcode-in-menus/includes/class-shortcode-in-menus.php
Stops this plugin from filling the log with thousands of error lines per day.
@gagan0123 are you looking to fix this, following the suggestion made by @hbwarper?
@dhunink
Yes, I’ll be adding this patch and will provide an update by tomorrow.
@hbwarper
Thanks for the patch 🙂 , would it be okay if I use your name in the list of contributors for the plugin?
