Problem/Motivation

In a few update path tests, we create fixtures for configurable field storages so we can test an update hook that touches their settings.

However, those fixtures do not go through the regular "field install" process (no database tables, no last installed schema entries) because they're not needed for the purpose of the update test.

This is not a problem at the moment, but it will become one once we start using the last installed definitions for entity type and field storage CRUD operations in #2976035: Entity type CRUD operations must use the last installed entity type and field storage definitions .

Proposed resolution

Mark those field storages with `custom_storage = TRUE`.

Remaining tasks

Review.

User interface changes

Nope.

API changes

Nope.

Data model changes

Nope.

Release notes snippet

Nope.

CommentFileSizeAuthor
#2 3031086.patch2.22 KBamateescu
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

amateescu created an issue. See original summary.

amateescu’s picture

Status: Active » Needs review
FileSize
2.22 KB
plach’s picture

Status: Needs review » Reviewed & tested by the community

Makes sense.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 2: 3031086.patch, failed testing. View results

amateescu’s picture

Status: Needs work » Reviewed & tested by the community

Testbot hiccup :)

plach’s picture

  • Gábor Hojtsy committed 14e383a on 8.7.x
    Issue #3031086 by amateescu, plach: Configurable fields that are created...
Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Makes sense. Looks simple and straightforward, thanks!

Status: Fixed » Closed (fixed)

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

quietone’s picture

publish change record