Copying from

            // @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);
#6 drupal8.comment-node-bundle.6.patch776 bytessun
PASSED: [[SimpleTest]]: [MySQL] 32,915 pass(es).
[ View ]
drupal.comment-node-bundle.0.patch951 bytessun
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch drupal.comment-node-bundle.0.patch. See the log in the details link for more information.
[ View ]


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

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
new776 bytes
PASSED: [[SimpleTest]]: [MySQL] 32,915 pass(es).
[ View ]

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.