Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.It doesn't matter what you set the depth to. Subpathauto will find it! The problem is subpathauto_url_inbound_alter() calls subpathauto_lookup_subpath() which calls drupal_get_normal_path() which calls subpathauto_url_inbound_alter(). Wash, rinse, repeat until path is found.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | 1247966-subpath-recursive-and-depth-bug.patch | 5.19 KB | dave reid |
| #5 | 1247966-4-path-recursion.patch | 3.36 KB | twistor |
| #4 | 1247966-2-path-recursion.patch | 1.87 KB | twistor |
| #3 | 1247966-1-path-recursion.patch | 628 bytes | twistor |
| #1 | 1247966-path-recursion.patch | 563 bytes | twistor |
Comments
Comment #1
twistor commentedComment #2
twistor commentedComment #3
twistor commentedThe last patch broke some things. Redirects for one.
Comment #4
twistor commentedSo I realized that the string walking in subpathauto_lookup_subpath() doesn't work at all. This changes that to something that's simpler for me to understand and works.
Comment #5
twistor commentedAdded test to illustrate the problem.
Comment #6
dave reidThanks for helping push this. I merged the test into the existing test and cleaned some code up. I think this should be 100% now.
Comment #7
dave reidThe big problem with the patch in #5 is that once it finds a matching sub-path it will return the path prefix in reversed order. On your site if you were to go to $alias . '/a/b' - you would see that $_GET['q'] would be 'node/[nid]/b/a'.
Comment #8
dave reid#6 committed to Git.