Problem/Motivation

In some cases, other systems might use route parameters for storing some metadata about a particular route. One example is layout builder using the following parameter to indicate info about section storage:

'section_storage' => ['layout_builder_tempstore' => TRUE],

Proposed resolution

Content moderation should not throw errors when processing routes that have parameters that do not contain a 'type'.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Sam152 created an issue. See original summary.

Sam152’s picture

The last submitted patch, 2: 3032548-2-TEST-ONLY.patch, failed testing. View results

Status: Needs review » Needs work

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

tim.plunkett’s picture

Status: Needs work » Reviewed & tested by the community

Random fail on the actual patch, correct fail on the TEST ONLY patch.
After a retest it should all be fine.

larowlan’s picture

Status: Reviewed & tested by the community » Fixed

Committed aa00366 and pushed to 8.7.x. Thanks!

  • larowlan committed aa00366 on 8.7.x
    Issue #3032548 by Sam152: ContentModerationRouteSubscriber assumes all...

Status: Fixed » Closed (fixed)

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