Ignore specific paths

I very much am in favour of the least configurable settings possible to meet the majority use case. This setting is clear and harmless in it's default empty state and provides a potentially very useful option so I'm in favour of the feature.

That said, this is a new feature so have a strong feeling that we shouldn't be using 'blacklist' - see also #2993575: Remove usage of "blacklist", "whitelist", use better terms instead → in core with regards to existing features where it is way more complicated and compatibility issues etc. I think ignorelist would be clearest - and then can use nice verb tenses like ignored etc.

Additionally, does need a re-roll due to other changes committed in the last couple weeks.

1) Rerolled the patch
2) Replace blacklist with denylist
3) We still call "path_alias.whitelist", but it comes from path_alias module

Thanks! I like it. Going to leave open for a few days and if no one else has opinions I'll probably commit it early next week.

Hi @LOBsTerr,

Tested the patch, it applied cleanly.

urvasi@urvasi-Inspiron-15-3552:/var/www/html/contribution/drupal/web/modules/contrib/subpathauto-2964786$ git apply -v 2964786-11.patch
Checking patch config/schema/subpathauto.schema.yml...
Checking patch src/Form/SettingsForm.php...
Checking patch src/PathProcessor.php...
Checking patch subpathauto.links.menu.yml...
Checking patch subpathauto.services.yml...
Checking patch tests/src/Functional/SubPathautoFunctionalTest.php...
Checking patch tests/src/Kernel/SubPathautoKernelTest.php...
Checking patch tests/src/Unit/SubPathautoTest.php...
Applied patch config/schema/subpathauto.schema.yml cleanly.
Applied patch src/Form/SettingsForm.php cleanly.
Applied patch src/PathProcessor.php cleanly.
Applied patch subpathauto.links.menu.yml cleanly.
Applied patch subpathauto.services.yml cleanly.
Applied patch tests/src/Functional/SubPathautoFunctionalTest.php cleanly.
Applied patch tests/src/Kernel/SubPathautoKernelTest.php cleanly.
Applied patch tests/src/Unit/SubPathautoTest.php cleanly.

@NickDickinsonWilde, committing the changes to save your time and efforts.

Thanks

@NickDickinsonWilde Can we already merge it ?
It also would be nice to have a new release tagged.
Thanks

Hey. Great work on this guys. Have been using the solution in https://www.drupal.org/project/subpathauto/issues/2964786 ✨ Ignore specific paths RTBC till now but don't think I can use it on the 10 compatible version.

Did this make it into the official release yet?

It is not possible to exclude 'node/*/edit' pages
Updated patch #11

Thanks for this great feature.

I had some trouble understanding how it worked, so I think this needs a bit of documentation and some clearer UI texts.

I would change the label to:

"Ignore the following paths"

And I would change the description to:

"These paths will be ignored by subpathauto. Use for example '/node/*/edit' to ensure that subpathauto ignores the node edit path. You can use wilcards(*). Start each path with a forward slash '/'. Enter one path per line."

I've read you discussed how to name the list and you settled for Denylist. Would "Ignore list" not be a better name for this?