Request path condition negation doesn't work

Issue fork bothive-3359932

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:

About issue forks

Comments

BramDriesen created an issue. See original summary.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented

Cloning issue into the bothive queue. Since the same negate logic is implemented the negate option probably doesn't work here as well.

BramDriesen credited cgoffin.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Issue tags: +Novice

Setting as novice, just look at the merge request/patch in #3359569: Request path condition negation doesn't work and replicate the fix into this repository.

Adding credit for cgoffin for bringing the issue up.

Kaustab_Roy made their first commit to this issue’s fork.

Kaustab_Roy opened merge request !13

kaustab_roy’s picture

kaustab_roy commented
Status: Active » Needs review

Added changes as per comment #4.

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Tested it out and confirm it's working! thanks

bramdriesen’s picture

bramdriesen
he/him
Primary language Dutch
Location Belgium 🇧🇪🇪🇺
commented
Status: Reviewed & tested by the community » Fixed
Issue tags: +DrupalCamp Ruhr 2023

Status: Fixed » Closed (fixed)

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