Problem/Motivation

The migration to Layout Builder does not support static contexts. The Core Context modules provides support for non-required contexts, but static context values are not dealt with at all.

Proposed resolution

Migrate static contexts into the correct destinations:

  • Required contexts need to be passed in the block configuration of each component.
  • Optional contexts need to be attached to the entity view display, or whatever is carrying the section list.

Remaining tasks

Do it and add tests.

User interface changes

None.

API changes

None.

Data model changes

None.

Release notes snippet

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

phenaproxima created an issue. See original summary.

phenaproxima’s picture

Status: Active » Needs review
Issue tags: +Needs tests
FileSize
6.3 KB

Will it fail? Will it pass? Let's find out.

phenaproxima’s picture

This should fail fewer tests.

phenaproxima’s picture

And this should fix the final failure.

phenaproxima’s picture

Now with basic low-level test coverage. Still needs higher-level functional coverage, though.

phenaproxima’s picture

Removing a bit of incorrect detritus.

phenaproxima’s picture

Okay, apparently now we have a forced dependency on core_context. Joy!

phenaproxima’s picture

Status: Needs review » Postponed
phenaproxima’s picture

Status: Postponed » Needs review
FileSize
9.53 KB

Okay, committed that. Let's see if this flies.

phenaproxima’s picture

Added high-level functional test coverage for nodes.

phenaproxima’s picture

...and for users.

phenaproxima’s picture

...and taxonomy terms.

  • phenaproxima committed 49efe77 on 8.x-5.x
    Issue #3091783 by phenaproxima: Support migrating static contexts
    
phenaproxima’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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