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 ]


tim.plunkett’s picture

Status:Active» Needs review
new1.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.