I noticed a bunch of weird links at the top of my administration menu, sometimes they are filtered out but in one case they even showed up in the UI and messed it up pretty badly.

The current code is adding possibly quite a lot of menu links for non-existing menu link parents, we should check for that explicitly.

We could also try to make it more intelligent and match on the path, but better to not define them at all than in a messed up way.

This affects block_content_type in core, as well as file_entity and flag from what I've seen.

CommentFileSizeAuthor
#2 admin_toolbar-check-parent-2873577-2.patch965 bytesberdir

Comments

Berdir created an issue. See original summary.

berdir’s picture

Status: Active » Needs review
StatusFileSize
new965 bytes
gg24’s picture

Status: Needs review » Needs work

Hi @Berdir,

Please mention the steps to reproduce this issue. I am not able to reproduce it as you mentioned this does not happen every time.
Please attach an image as well if possible. This would help me in understanding this issue in a better way.

Thanks!

berdir’s picture

Status: Needs work » Needs review

Possible that this only happens with file_entity and not block_content, all you need to do is enable both file_entity and admin_toolbar_tools and then /admin/structure/menu/manage/admin shows a bunch of weird top-level menu links.

adriancid’s picture

Status: Needs review » Needs work

Hi @Berdir I' testing now the last dev version with the file_entity module enabled but I can't see the problem, can you re-test and add a image if the problem persists?

adriancid’s picture

Status: Needs work » Postponed (maintainer needs more info)
adriancid’s picture

Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

I'm closing this as we can't reproduce the issue, if the problem persists please feel free to reopen and update the status.

adriancid’s picture