Adding the bundles can be done in the entity info hook directly by using a database query instead of the load functions. I've changed it to an implementation that's a simpler version of how node.module does it.

#1440600: model_entity_info_alter() should not depend on Entity API helper functions. is a similar change, but didn't go as far as to remove the alter hook.

CommentFileSizeAuthor
#5 model-remove-entity-info-alter-2225895-5.patch1.96 KBvegantriathlete
#5 interdiff.2-5.txt608 bytesvegantriathlete
#2 model-remove-entity-info-alter-2225895-2.patch1.82 KBarvola
#1 model-remove-entity-info-alter-2225895-1.patch1.82 KBarvola
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arvola’s picture

arvola CreditAttribution: arvola commented
Assigned: arvola » Unassigned
Status: Active » Needs review
FileSize
model-remove-entity-info-alter-2225895-1.patch1.82 KB
arvola’s picture

arvola CreditAttribution: arvola commented
FileSize
model-remove-entity-info-alter-2225895-2.patch1.82 KB

First patch had a whitespace error.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented

There is a mistake in your patch.
Look at

$return['contact']['bundles'][$type] = array(

should be
$return['model']['bundles'][$type] = array(

Anybody’s picture

Anybody
Primary language German
Location Porta Westfalica
CreditAttribution: Anybody commented
Status: Needs review » Needs work

Confirming typo from #3. After fixing this the patch is working and good!
+1 for this. arvola could you upload a corrected patch?

vegantriathlete’s picture

vegantriathlete
he/him/his
Primary language English
Location Lakewood, CO
CreditAttribution: vegantriathlete as a volunteer commented
Status: Needs work » Needs review
FileSize
interdiff.2-5.txt608 bytes
model-remove-entity-info-alter-2225895-5.patch1.96 KB

Looks like the file was missing the EOL character on the last line as well.