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.

#1 commerce-1571376-1.patch1.22 KBtim.plunkett
PASSED: [[SimpleTest]]: [MySQL] 3,567 pass(es). View
Members fund testing for the Drupal project. Drupal Association Learn more


tim.plunkett’s picture

Status: Active » Needs review
1.22 KB
PASSED: [[SimpleTest]]: [MySQL] 3,567 pass(es). View

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.