Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Hey there,
I just stumbled across an issue which is more Drupal related.
Nevertheless, it is annoying, so I fixed it.
The problem is, that if you exclude paths like "node/*" all paths will be excluded that start with "node/". That also included "node/add/[NODETYPE]" pages.
To prevent such cases, I added a new text field to insert paths into that would annihilate of the exclude field.
Example:
Excludeded Urls:
node/*
Included:
node/add/blog
Result:
node/123132 -> No redirect
node/356/edit -> No redirect
node/add/blog -> Redirect
I hope this helps someone someday :)
Patch will follow in the first comment.
Greetings
func0der
| Comment | File | Size | Author |
|---|---|---|---|
| #7 | r4032login-hungry_regex-2327145-7.patch | 3.36 KB | rooby |
Comments
Comment #1
func0der CreditAttribution: func0der commentedAnd the promised patch.
Comment #2
func0der CreditAttribution: func0der commentedComment #3
func0der CreditAttribution: func0der commentedFixed Copy&Paste mistake -.-
Sorry.
Comment #4
csc4 CreditAttribution: csc4 commentedLooks like a very useful addition
Comment #5
rooby CreditAttribution: rooby commentedAn alternative solution could be to implement something like the context module uses, where a ~ prefix negates, so you could do this in the current noredirect field:
node/*
~node/add/blog
That's a pattern I've seen used in a few modules.
Comment #6
rooby CreditAttribution: rooby commentedThis patch no longer applies. It also needs to be applied with
patch -p7
instead of
patch -p1
Comment #7
rooby CreditAttribution: rooby commentedHere is a reroll for latest dev. No changes.
Seems to work for my small amount of testing.