Contrib.social

Remove the optional $request argument from outbound path processor

Open on Drupal.org →
Created on 26 March 2014, about 10 years ago
Updated 17 July 2023, 11 months ago

Updated: Comment #N

Problem/Motivation

OutboundPathProcessorInterface::processOutbound currently accepts an optional $request parameter. The only subsystem requiring the request when processing outbound paths is the various language path processors.

At the same time it is desirable to remove the dependency of UrlGenerator from the request service. In order to do so, outbound path processing needs to be possible without having to inject the request (or request_stack) into the UrlGenerator.

Proposed resolution

  • Remove the optional request parameter from OutboundPathProcessorInterface::processOutbound()

Removing the request dependency from the UrlGenerator completely should be done in a follow-up issue.

Remaining tasks

evaluate and commit

User interface changes

no

API changes

\Drupal\Core\PathProcessor\OutboundPathProcessorInterface::processOutbound() optional argument removal

-  public function processOutbound($path, &$options = array(), Request $request = NULL, CacheableMetadata $cacheable_metadata = NULL);
+  public function processOutbound($path, &$options = array(), CacheableMetadata $cacheable_metadata = NULL);

Beta phase evaluation

<!--Uncomment the relevant rows for the issue. -->
📌 Task
Status

Needs work

Version

11.0 🔥

Component
Base  →

Last updated 1 minute ago

Created by

🇨🇭Switzerland znerol

Live updates comments and jobs are added and updated live.
  • Needs issue summary update

    Issue summaries save everyone time if they are kept up-to-date. See Update issue summary task instructions.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024