Earlier versions of the codebase created records for entities that are now no longer supported, e.g. comment, file (when using file_entity), mailchimp_list, profile2_type, taxonomy_vocabulary, and possibly others. Compile an authoritative list of these entity types and then build an update script to purge the {metatag} table of matching records.
| Comment | File | Size | Author |
|---|---|---|---|
| #11 | metatag-n1857116-11.patch | 333 bytes | damienmckenna |
| #9 | metatag-n1857116-9.patch | 744 bytes | damienmckenna |
| #7 | metatag-n1857116-7.patch | 326 bytes | damienmckenna |
| #3 | metatag-n1857116-3.patch | 1.05 KB | damienmckenna |
| #1 | metatag-n1857116.patch | 822 bytes | damienmckenna |
Comments
Comment #1
damienmckennaRather than manually tracking a manual list of unsupported entities, just using entity_get_info() should be enough.
Comment #2
damienmckennaCommon entities that could be safely manually purged:
Comment #3
damienmckennaA patch that just purges the items listed in #2 above.
Comment #4
damienmckennaCommitted.
Comment #5
damienmckennaMarking this as Postponed as I'll have to come back to it later for a final cleanup.
Comment #6
dave reidMeta tags are perfectly valid with file entities when used with File entity module. Why should those be removed?
Comment #7
damienmckennaBecause the file entity configuration doesn't have 'metatags' = TRUE assigned. I've added #1857334: Enable Metatag support for File entity? to make a decision and, for now, will remove 'file' records from the to-purge list.
Comment #8
damienmckennaI've committed the patch from #7 and putting this back on the Postponed stack to be dealt with later.
Comment #9
damienmckennaBetter reporting for data deletion.
Comment #10
damienmckennaCommitted.
Comment #11
damienmckennaOne more tweak - get rid of the profile2 records.
Comment #12
damienmckennaI've committed the patch from 11, leaving it as Postponed for now.
Comment #13
damienmckennaNo need to tag something as being a release blocker if I'm postponing it.
Comment #14
damienmckennaHonestly I think this is good enough for now.