Hide unpublished/disabled terms

Descr WIP

Attaching a patch adding respect of parent enable check of term link.

This is considered needed in case of conflict with work done in other places like this one: https://www.drupal.org/files/issues/2023-10-13/taxonomy_menu-hide_empty_... →

Hi, I've Tested MR- MR !9 mergeable on Drupal version- 10.1.9-dev. The MR was applied successfully and looks good to me.

Testing Steps:

• Set up Drupal 10.1.9.
• Download/Enable the Taxonomy Menu module.
• Create Taxonomy Menu> Create new terms> Published & unpublished. You can see unpublished terms are also showing in the menu, Added before results
• Download the shared patch & apply.
• Go back to the site home & Reverify the menu.

Testing Results:
After applying the patch, the Unpublished terms are not showing now.

Hi, we've met the same issue on a use case to render a taxonomy menu by linkset feature.

We're using Drupal 10.2.5, taxonomy_menu 3.6

The test in merge request does not update the menu item shown in linkset endpoint, please see the gif below where I'd unpublished the term.

Steps with old patch:

1. Unpublished term
2. Refresh to check menu item disabled, yes
3. Refresh endpoint of linkset menu to have menu item disappear, still exists

I've figured out another fix that successfully updates the menu item's "enabled" property while also affects linkset endpoint.

Steps with new patch:

1. Unpublished term
2. Refresh to check menu item disabled, yes
3. Refresh endpoint of linkset menu to have menu item disappear, the unpublished menu item disappears as well

We definitely want to get this in the next release.

Can someone please update the MR to include the latest changes in the 8.x-3.x branch?

Also, this needs test coverage to make sure the change isn't accidentally broken later on.

Thank you.

MR updated: Added most recent commits from 8.x-3.x, created test case for unpublished taxonomy term

Great work, thank you!

Committed.

FYI after discussing it with the security team it was decided that this fit under PSA-2023-07-12 → . In short, while there is a security aspect to this, it is such an unlikely scenario to happen that it was deemed fixable in public. As such, there won't be an advisory for this fix.

The change has caused a big regression for us on translatable vocabularies. Now every taxonomy menu item is shown multiple times, one item for each translation. So one term "Research" is shown three times if the term is translated into two additional languages.

Thank you for the clarification, szediler.

Not all of the patch changes made it in, so it's not possible to get or alter the enabled item. I'll open a new issue.

Automatically closed - issue fixed for 2 weeks with no activity.