I found an issue when the Bulk update SQL use CAST as CHAR, which doesn't work in postgresql. We need to CAST as VARCHAR. Then it works. The patch is only on the taxonomy. I am not certain if this works on mysql. So it is merely and example of a fix in one of the SQL statement. Perhaps someone know the two db's a bit better and can offer a complete solution.

CommentFileSizeAuthor
pathauto_taxonomy_pgsql.patch898 bytespearcec

Comments

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Thanks for the patch!

Which version of PostgreSQL are you using?

We did lots of work on this with pgsql folks in #212327: cast the concatenated join value to be able to use the url_alias index so I'm hesitant to change this.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Status: Active » Postponed (maintainer needs more info)

...

pearcec’s picture

pearcec commented
Status: Postponed (maintainer needs more info) » Needs review

Unfortunately at the moment our dev environment is 7.4. But production is 8.2. I am almost certain I ran into this issue on 7.4.

I don't know when or if I will run into this issue again. But should we change the status to "patch (code needs review)" and just let it sit in case someone else has an issue? I don't feel compelled to make 7.4 work.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented
Title: Postgresql Bulk Update » Postgresql 7.4 Bulk Update

If this only impacts 7.4 (I imagine that is the case) then I think we might want to just dismiss it...

But, for now, yes, let's leave the issue open.

justin3’s picture

justin3 commented

Works for PG 8.2 !

I had the issue with bulk update returning just /taxonomy/term for a new alias, then stopping- ie not updating any more taxonomy aliases even on future updates.

Then I switched to pathauto 6.x-1.x-dev - which fixed that error, but pathauto bulk update was still not adding new taxo terms- until this patch fixed it.

thanks!

justin3’s picture

justin3 commented

Just had to make this same fix for pathauto_node.inc to get the bulk updates to continue for nodes

line 100 - change CHAR to VARCHAR

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
commented

Anyone use pgsql8.3 to be able to test it? I feel confident that this works for some people...

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented

Duplicate of #319070: Provide a wrapper for cross-db and cross-field-type concatenation (PosgreSQL)

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Closed (duplicate)