Problem/Motivation

field_purge_batch() tries to purge fields that have no instances. field_read_instances() does not return instances attached to unknown entity types unless include_inactive is set in the $include_additional parameter. If you delete field instances in the disable hook of a module that provides an entity type, those instances will not be automatically purged. However, the fields will be purged automatically. Every time you uninstall the module, it will leave instances that have no corresponding fields in the field_config_instances table. These orphaned instances can only be deleted manually.

Proposed resolution

Pass include_inactive in the $include_additional parameter before attempting to purge fields.

Remaining tasks

  1. Review patch.
  2. Commit patch.
CommentFileSizeAuthor
#9 drupal-field_purge_batch-2337139-8.patch1.17 KBdarren oh
#7 drupal-field_purge_batch-2337139-7.patch1.16 KBdarren oh
#5 drupal-field_purge_batch-2337139-5.patch1.11 KBdarren oh
#1 drupal-field_purge_batch-2337139.patch1.11 KBdarren oh

Comments

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Issue summary: View changes
StatusFileSize
new drupal-field_purge_batch-2337139.patch1.11 KB
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Assigned: darren oh » Unassigned
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 1: drupal-field_purge_batch-2337139.patch, failed testing.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
StatusFileSize
new drupal-field_purge_batch-2337139-5.patch1.11 KB

Fixed typo.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Needs work » Needs review
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
StatusFileSize
new drupal-field_purge_batch-2337139-7.patch1.16 KB

Changed wrong line.

Status: Needs review » Needs work

The last submitted patch, 7: drupal-field_purge_batch-2337139-7.patch, failed testing.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Needs work » Needs review
StatusFileSize
new drupal-field_purge_batch-2337139-8.patch1.17 KB
darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Title: field_purge_batch() falsely assumes that instances attached to unknown entities do not exist » Fields with instances attached to unknown entities are deleted on cron run
taran2l’s picture

taran2l
Primary language Ukrainian
Location Lviv
commented
Status: Needs review » Closed (duplicate)

Duplicate of #1340390: "Deleted" but "inactive" fields and instances cannot be removed during cron cleanup (e.g. for uninstalled entities).

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Closed (duplicate) » Needs review

This is not a duplicate. The other patch would still allow fields with inactive instances to be deleted before the instances have been deleted.

mcdruid’s picture

mcdruid commented

Aren't the patches effectively exactly the same?

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented

No, this patch fixes field deletion and the other patch fixes instance deletion.

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 7 security and bugfix support has ended as of 5 January 2025. If the issue verifiably applies to later versions, please reopen with details and update the version.