Would it make sense if the bean type administration links mirrored those of content types. ie.

Structure
  - Content Types
    - Content type 1
      - Edit
      - Delete
      - Manage Fields
      - Manage Display
  - Blocks
    - Block Types
      - Block type 1
        - Edit
        - Delete
        - Manage Fields
        - Manage Display

Currently the only way to get to these links for each Bean Type is to go through the List page.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

indytechcook’s picture

Yeah, that would make navigation easier in admin/admin_menu and also setting the active menu item.

indytechcook’s picture

Issue tags: +Release blocker

Tagging with Release blocker

indytechcook’s picture

So the first step is here: #1356088: Change Path for block types

But i'm not sure exactly why the admin menus are not being created. The menu_router and menu_links table structure are the same.

alexweber’s picture

As far as I know this is more a limitation of the Entity API's automagic Admin UI stuff than anything else. Check our profile2, they have exactly the same problem. I've just been playing with custom entities this weekend and hit the same roadblock, it's kind of annoying.

indytechcook’s picture

@alexweber

Thanks for the comment but it really doesn't apply here. I don't use entity api amin ui because it requires your bundles being entities themselves. So it should be possible. I did look into it at one time, I even modeled it exactly like ctypes but I couldn't get it to work.

alexweber’s picture

@indytechcook I see... Fair enough! :)

barraponto’s picture

I've fixed the page titles in admin/structure/manage/%bean-type, block/add/%bean-type and block/edit/%bean_delta. I spent a lot of time trying to get the breadcrumbs straight for the admin/structure/manage/%bean-type/{fields,display} but I couldn't understand why it was broken. So I patched a workaround using hook_menu_breadcrumb_alter. It is not pretty, but it works.

barraponto’s picture

Status: Active » Needs review
alexweber’s picture

Status: Needs review » Needs work

@barraponto, I've reviewed the patch and it doesn't seem to address the issue :S

It does, however, address another bug. You should open another issue for this patch!

alexweber’s picture

Status: Needs work » Needs review
FileSize
2.49 KB

Attached is a patch that adds the menu structure suggested by the OP. It doesn't have the Manage Fields/Display links yet.

It isn't as slick as what the authors were trying to do, using token replacements in the menu path but works just the same! :)

indytechcook’s picture

Status: Needs review » Reviewed & tested by the community

Thanks! Looks good. Setting to RTBC so remember to push.

@alexweber woudl you mind elaborating on what bug @barraponto was fixing?

indytechcook’s picture

Status: Reviewed & tested by the community » Fixed
alexweber’s picture

@indytechcook yeah np it was something about breadcrumbs being off, but that should be fixed as a side effect of this patch :)

The other issue was this one #1537456: Better titles for bean/add/%bean-type and bean/edit/%bean-delta

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