Overview

Navigating to /admin/modules causes:

The website encountered an unexpected error. Try again later.

Symfony\Component\Routing\Exception\MissingMandatoryParametersException: Some mandatory parameters are missing ("entity_type", "entity") to generate a URL for route "experience_builder.experience_builder". in Drupal\Core\Routing\UrlGenerator->doGenerate() (line 187 of core/lib/Drupal/Core/Routing/UrlGenerator.php).
Drupal\Core\Routing\UrlGenerator->getInternalPathFromRoute() (Line: 300)
Drupal\Core\Routing\UrlGenerator->generateFromRoute() (Line: 108)
Drupal\Core\Render\MetadataBubblingUrlGenerator->generateFromRoute() (Line: 765)
Drupal\Core\Url->toString() (Line: 183)
Drupal\Core\Utility\LinkGenerator->generate() (Line: 102)
Drupal\Core\Render\Element\Link::preRenderLink()
call_user_func_array() (Line: 113)
Drupal\Core\Render\Renderer->doTrustedCallback() (Line: 870)
…

Proposed resolution

Stop specifying a configure route.

User interface changes

No more crash.

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

Wim Leers created an issue. See original summary.

wim leers’s picture

Jesse also found this.

wim leers’s picture

Status: Active » Needs review

TIL configure_params exists by reading \Drupal\system\Form\ModulesListForm::buildRow() 🤪 — with a sole example in core: core/modules/system/tests/modules/system_test/system_test.info.yml.

This restores the same behavior as we previously had.

wim leers’s picture

Title: Regression caused by #3462247: `configure` route in *.info.yml needs params » Regression caused by #3462247: `configure` route in *.info.yml missing mandatory parameters
Status: Needs review » Reviewed & tested by the community

#5 fixes it, but it may still result in a 404 if the first node is not yet created.

So I just updated the MR to instead make the configure link point to /admin/structure/component 👍 (And that will continue to be relevant even after #3463999: Auto-create/update Component config entities for all discovered SDCs that meet XB's minimum criteria lands — it'll show the person who just installed XB which components were automatically detected and configured for use with XB 😊)

wim leers’s picture

Status: Reviewed & tested by the community » Needs review

Actually, this needs approval from two more people 😇

tedbow’s picture

Status: Needs review » Reviewed & tested by the community

  • tedbow committed 25b99dc0 on 0.x authored by Wim Leers
    Issue #3466531 by Wim Leers, jessebaker: Regression caused by #3462247...
tedbow’s picture

Status: Reviewed & tested by the community » Fixed

super small change to I merged

Status: Fixed » Closed (fixed)

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