You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
First, I love the preg_replace() idea! And two, thanks for adding the wp_nav_menu_item_custom_fields action hook so Nav Menu Roles and Menu Icons can be compatible.
However, I've found that your action hook's params aren't the same as mine.
You have:
do_action( 'wp_nav_menu_item_custom_fields', $id, $item, $depth, $args );
While I have:
do_action( 'wp_nav_menu_item_custom_fields', $item_id, $item, $depth, $args );
In your case $id will always be zero. I don't know if this has a lot of impact since it's only used by us menu plugin authors, but I thought I'd point it out.
The text was updated successfully, but these errors were encountered:
First, I love the
preg_replace()
idea! And two, thanks for adding thewp_nav_menu_item_custom_fields
action hook so Nav Menu Roles and Menu Icons can be compatible.However, I've found that your action hook's params aren't the same as mine.
You have:
do_action( 'wp_nav_menu_item_custom_fields', $id, $item, $depth, $args );
While I have:
do_action( 'wp_nav_menu_item_custom_fields', $item_id, $item, $depth, $args );
In your case
$id
will always be zero. I don't know if this has a lot of impact since it's only used by us menu plugin authors, but I thought I'd point it out.The text was updated successfully, but these errors were encountered: