Updated: Comment #0

Problem/Motivation

We introduced the weird "bundle_prefix" concept to support the weirdness that comment.module used to do with regard to bundles. comment.module no longer does this after #731724: Convert comment settings into a field to make them work with CMI and non-node entities

Proposed resolution

Remove the remnants of "bundle_prefix".

API changes

Relative to D7 none, as there was no such general concept in the Entity API.
Relative to HEAD: no more bundle_prefix for entities.

#731724: Convert comment settings into a field to make them work with CMI and non-node entities

CommentFileSizeAuthor
#1 no-more-bundle-prefix.patch2.65 KBtstoeckler
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tstoeckler’s picture

Status: Active » Needs review
FileSize
2.65 KB

Here we go.

larowlan’s picture

+1 rtbc

yched’s picture

Way cool. RTBC +1 if green.

jibran’s picture

Status: Needs review » Reviewed & tested by the community

RTBC as per #2 and #3.

fago’s picture

Great change!

andypost’s picture

+1 here

catch’s picture

Status: Reviewed & tested by the community » Fixed

Full of win. I think I'm partially responsible for introducing bundle prefix in the first place so nice to see it ripped out again!

Committed/pushed to 8.x, thanks!

Status: Fixed » Closed (fixed)

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