Bundle info has been moved to a new hook hook_entity_bundle_info() and the menu_link entity type doesn't even need to explicitely declare any bundle at the moment.

CommentFileSizeAuthor
#1 1937384.patch572 bytesamateescu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amateescu’s picture

Status: Active » Needs review
FileSize
572 bytes

And the patch.

swentel’s picture

Status: Needs review » Reviewed & tested by the community
amateescu’s picture

Status: Reviewed & tested by the community » Needs review
Issue tags: +Quick fix

Can't get any 'quicker' fix than this :)

amateescu’s picture

Status: Needs review » Reviewed & tested by the community

Sorry, x-post.

andypost’s picture

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Needs work
Issue tags: -Quick fix

Where is this bundle defined then? I see user defines it with this hook, but no such hook for menu items? http://api.drupal.org/api/drupal/core%21includes%21entity.api.php/functi...

amateescu’s picture

Status: Needs work » Reviewed & tested by the community
Issue tags: +Quick fix

It's provided by default in entity_get_bundles().

Gábor Hojtsy’s picture

Why do user module define it then?

webchick’s picture

Status: Reviewed & tested by the community » Needs review

Looks like this still needs discussion.

Gábor Hojtsy’s picture

I've been responded to on IRC that people don't know either. However a module doing something relatively useless should not be a reason for this to do it too. One thing that user module does is it specifies a path for the account settings (as the bundle config path?) which is kind of odd. Although if that is useful for some UI interaction, then maybe menu module should do the same?

swentel’s picture

That path is used by Field UI so the 'Manage fields' and 'Manage Display' screens are placed correctly, so that's only usefull if your entity is fieldable.

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Thanks!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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