As of Drupal 7.14, the field_bundle_settings was split into per-bundle variables.

content_migrate_install() was loading the monolithic variable, unsetting relevant parts, and resaving it when deleting a bundle. This isn't necessary, as each individual variable can be set.

CommentFileSizeAuthor
#4 cck-n1571406-4.patch2.45 KBdamienmckenna
#1 cck-1571406-1.patch2.45 KBtim.plunkett

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Active » Needs review
StatusFileSize
new cck-1571406-1.patch2.45 KB

This should work. Untested.

mandreato’s picture

mandreato commented
Status: Needs review » Reviewed & tested by the community

#1 seems to work apart from the last three rows which try to add a newline at the end of the file.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented

The file is currently missing the EOF character. In some text editors, it looks like a new line.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new cck-n1571406-4.patch2.45 KB

Rerolled, just to fix the extremely minor thing Tim mentioned.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

BTW because the new field_bundle_settings functionality was added in 7.14 should this add a requirement so it only works with core versions >= 7.14?

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented
Version: 7.x-2.x-dev » 7.x-3.x-dev
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Committed in 9276cd7. Feel free to submit another patch if the core version dependency is a problem.

Status: Fixed » Closed (fixed)

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