[ Note: the included patch is against a file which already has #861016: Prevent urls from getting language treatment and #881164: Some pathauto stuff has the wrong 'src' URL applied. I hope that's OK. Let me know if not.]

PostgreSQL does not have the CONCAT function. So I changed it with a REPLACE function.

This has the added bonus of keeping the expected 'alias.src' value in sync with the output from openid_provider_user_path() (formerly known as openid_provider_user_url()) . It's not in sync currently, which caused #881164.
(And the use of '%user' as a replacement string, is arbitrary. The only reason I picked it, is that this is also used as an argument to openid_provider_user_path(), elsewhere in the code.)

The number 10 in the CAST ... AS CHAR(10) is arbitrary. (The maximum length of an uid.) Note you cannot do 'CAST ... AS CHAR' in Postgres, since that'll shorten the uid to one character. (Search drupal.org for 'cast varchar' if you want more examples.)

CommentFileSizeAuthor
openid_provider.pgsql_.patch1.1 KBroderik

Comments

grub3’s picture

grub3 commented
Status: Needs review » Reviewed & tested by the community

Thanks for the fix. I applied the patch manually. And it works.

anarcat’s picture

anarcat commented
Status: Reviewed & tested by the community » Fixed

patch committed and pushed to 6.x branch.

Status: Fixed » Closed (fixed)

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