Skip to content
Snippets Groups Projects

Draft: Trim the path after decoding.

Open Draft: Trim the path after decoding.
issue/drupal-3456244:3456244-11x-trim-path into 11.x
1 unresolved thread
Open catch requested to merge issue/drupal-3456244:3456244-11x-trim-path into 11.x
1 unresolved thread

Closes #3456244

Merge request reports

Members who can merge are allowed to add commits.

Merged results pipeline #238917 passed

Stage: 🪄 Lint
Stage: 🗜️ Test
Pipeline: drupal

#238919

    Merged results pipeline passed for 28bd8085

    Code Quality is loading
    Test summary results are being parsed
    Full report
    Ready to merge by members who can write to the target branch.

    Merge details

    • The source branch is 1323 commits behind the target branch.
    • 1 commit will be added to 11.x.
    • Source branch will not be deleted.

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
  • catch added 1 commit

    added 1 commit

    • a23da136 - Deprecate the service instead of removing it.

    Compare with previous version

  • Dave Long
  • Dave Long
    Dave Long @longwave started a thread on an old version of the diff
    • core/lib/Drupal/Core/Routing/RouteProvider.php
    463 464 $key_parts[] = '[' . $provider . ']=' . $key_part;
    464 465 }
    465 466
    466 return 'route:' . implode(':', $key_parts) . ':' . $request->getPathInfo();
    467 return 'route:' . implode(':', $key_parts) . ':' . trim(urldecode($request->getPathInfo()));
  • catch added 1 commit

    added 1 commit

    • 946dee73 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • catch added 1 commit

    added 1 commit

    • a4bddc9b - Duplicate the logic more consistently and add a @todo.

    Compare with previous version

  • catch added 104 commits

    added 104 commits

    Compare with previous version

    Toggle commit list
  • catch added 1 commit

    added 1 commit

    • ffe7fe56 - Deprecate in 11.1 instead of 10.4

    Compare with previous version

  • catch marked this merge request as draft

    marked this merge request as draft

    • Please register or sign in to reply