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

commerce_delete_instances() was loading the monolithic variable, unsetting relevant parts, and resaving it when deleting an entity type or bundle. This isn't necessary, as each individual variable can be deleted.

CommentFileSizeAuthor
#1 commerce-1571376-1.patch1.22 KBtim.plunkett
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tim.plunkett’s picture

Status: Active » Needs review
FileSize
1.22 KB

This should work. Untested.

tim.plunkett’s picture

Category: task » bug

Meant to file this as a bug, since that variable doesn't exist anymore.

bojanz’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

rszrama’s picture

Status: Reviewed & tested by the community » Fixed

Looks great, Tim. Nice catch!

Welcome to the committers log.

Status: Fixed » Closed (fixed)

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