[node:og-group-ref:url:relative] pathauto token broken

Even after updating OG, Token, and Entity to the latest dev releases, [node:og-group-ref:url:relative] still ended up stripping slashes. I worked around the issue using [node:og-group-ref:url:path] instead.

Reopening the bug but moving priority to normal.

Regards,
Jeff

@jeff, @bob, is it the same issue, or a separated issue ?

@shushu The issue I'm having is exactly the same as in the description. Cheers, Jeff

Gotcha :-)
After debugging I found out that "[node:og-group-ref:url:relative]" does not fit to the token structure checked by pathauto_clean_token_values(), so it is being cleared.
Considering each of "path|alias|url|url-brief" might have attribute such as "relative", just patching in order to solve our current need seems to be too limited. Any idea how we should solve it generically ?

@shushu I'm going to try and dig deeper. So far, at least, here are the exact steps to reproduce:

Install fresh D7 site, then:
drush dl og pathauto token entity entityreference views_bulk_operations ctools views --dev --destination=sites/all/modules/contrib -y;
drush en entity_token og_ui pathauto -y;
drush dis overlay -y;
drush cc all;

Make Article a group type
Make Basic page a group content type
Set the Maximum number of values users can enter for og_group_ref to 1
Set the Basic page pathauto pattern to be: [node:og-group-ref:url:relative]/page/[node:title]
Create an Article called "test group"
Create a Basic Page called "test group content"
Notice the url for the basic page is: contenttest-group/page/test-group-content

@loopduplicate, I think the direct solution is simple. In pathauto.inc change line #607 to:
if (!preg_match('/(path|alias|url|url-brief|relative)\]$/', $token)) {
and this should solve the problem. You can try this out.

What I am saying, though, that this might not be the right solution, since it covers only the specific case.

@shushu I already worked around this bug for my use case by using [node:og-group-ref:url:path] instead. I don't need a 'direct solution'. :) I meant that I will try and take a closer look at everything later and try to come up with a patch that can be committed to whatever projects make sense. I only have a small amount of free time every day so I have to just do a little bit at a time. I wrote down the steps to reproduce because I don't know when I'll get back to this bug. For anyone else running into this who needs a hotfix, thanks for the 'direct solution' above. :)

Cheers,
Jeff

Yes, I would highly recommend using url:path instead of url:relative in tokens in Pathauto patterns.

Thanks for your feedback, Dave :)

Hi,

Refering to the "When using this pathauto token, I get a broken path. The slashes in the [node:og-group-ref:url:relative] part are removed every time."

Not sure if this is relevant or not but in the URL-Alias settings you can set what characters is going to be removed by path auto. There you can set "do nothing" for the character Slash (/) that way this character wont be stripped away by path auto.

Thanks a lot #13. Disabling slash removal finally fixed my custom join path from node fields.

Adding this note here since it's the relevant google search result:

For anyone who's running into an issue with tokens being stripped of slashes (and who doesn't want to blanket disable this setting), at least in Drupal 8, slashes are removed from tokens UNLESS the token name ENDS with one of the following strings:

path
alias
url
url-brief

For example, [my-special-path] will retain slashes, but [path-special] will have slashes removed.

That's what #2773573: AliasCleaner preg_match for tokens is too broad is about.