Copying from admin_menu.map.inc:

            // @todo Make Comment module expose the original node type bundle,
            //   pretty please.
            if (drupal_substr($bundle_name, 0, 13) == 'comment_node_') {
              $bundle_name = drupal_substr($bundle_name, 13);
            }
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sun’s picture

drupal.comment-node-bundle.0.patch queued for re-testing.

dixon_’s picture

Version: 7.x-dev » 8.x-dev
sun’s picture

sun’s picture

drupal.comment-node-bundle.0.patch queued for re-testing.

Status: Needs review » Needs work
Issue tags: +Quick fix, +Needs backport to D7

The last submitted patch, drupal.comment-node-bundle.0.patch, failed testing.

sun’s picture

Status: Needs work » Needs review
FileSize
776 bytes

Re-rolled against latest branch head.

dixon_’s picture

Status: Needs review » Reviewed & tested by the community

To me this looks like a sensible addition. The patch is very simple, and looks good to me.

dixon_’s picture

The patch applies without offset to D7. So should be ready for straight backport.

catch’s picture

Version: 8.x-dev » 7.x-dev

Looks good to me as well, I'm pretty sure I've had to figure out the node type back from the comment before as well.

Committed to 8.x, moving to 7.x for webchick to consider.

webchick’s picture

Status: Reviewed & tested by the community » Fixed

This seems like a pretty harmless and convenient API addition. I'm comfortable committing to 7.x as well.

Committed/pushed. Thanks!

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix, -Needs backport to D7

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