Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.
Related Issues
#731724: Convert comment settings into a field to make them work with CMI and non-node entities
Comment | File | Size | Author |
---|---|---|---|
#1 | no-more-bundle-prefix.patch | 2.65 KB | tstoeckler |
Comments
Comment #1
tstoecklerHere we go.
Comment #2
larowlan+1 rtbc
Comment #3
yched CreditAttribution: yched commentedWay cool. RTBC +1 if green.
Comment #4
jibranRTBC as per #2 and #3.
Comment #5
fagoGreat change!
Comment #6
andypost+1 here
Comment #7
catchFull 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!