Allow query string in RequestPath conditions

Created on 1 December 2018, over 6 years ago
Updated 30 January 2023, over 2 years ago

Problem/Motivation

Want to make a block visibility condition with a query string.

Proposed resolution

Extend the RequestPath condition and the PathMatcher accordingly.
Add logic that normalizes query string order and accounts for facet keys where index can change.

Remaining tasks

Review, commit.

User interface changes

RequestPath config form has option to consider query string.

API changes

Pure API addition with no BC breaks.

Data model changes

None.

Release notes snippet

TBD

Feature request
Status

Needs work

Version

10.1

Component
Base 

Last updated about 8 hours ago

Created by

🇩🇪Germany geek-merlin Freiburg, Germany

Live updates comments and jobs are added and updated live.
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.

  • The Needs Review Queue Bot tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

    Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

    Consult the Drupal Contributor Guide to find step-by-step guides for working with issues.

  • First commit to issue fork.
  • 🇺🇸United States jsutta United States

    I was hoping to use this functionality for an enhancement to a site I'm working on, but the code in the patch no longer works. Nevertheless, I added it as the basis for an enhancement to a feature branch on this issue.

    In the meantime, anyone looking for this functionality should use the Condition Query contributed module.

Production build 0.71.5 2024