Pathauto bulk update for taxonomy terms (also forum terms) is missing a condition to actually advance on the database result set during batch, causing it generate aliases only for the first 25 terms. The bug additionally requires to only have for some vocabularies a pattern and no default pattern set, resulting in no aliases generated for these terms.

CommentFileSizeAuthor
#4 pathauto-taxonomy-bulk-1279270.patch1.09 KBsmk-ka
#1 pathauto-taxonomy-bulk-1279270.patch1.11 KBsmk-ka

Comments

smk-ka’s picture

smk-ka commented
StatusFileSize
new pathauto-taxonomy-bulk-1279270.patch1.11 KB

This patch adds the last processed term id as additional condition, just like the node etc. bulk updates do with the node id.

smk-ka’s picture

smk-ka commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, pathauto-taxonomy-bulk-1279270.patch, failed testing.

smk-ka’s picture

smk-ka commented
Status: Needs work » Needs review
StatusFileSize
new pathauto-taxonomy-bulk-1279270.patch1.09 KB

(With Unix-style line endings)

dave reid’s picture

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

I think the forum code also has this same problem. Could you add that to be fixed as well?

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Reviewed & tested by the community

I also had an empty default pattern. Applied the patch and it solved the problem for me.

dave reid’s picture

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

Nevermind, I see the patch covers the forum condition.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Reviewed & tested by the community » Fixed

Thank you very much! Committed #4 to Git: http://drupalcode.org/project/pathauto.git/commit/11a8cd6. Does not need to be backported at all.

Status: Fixed » Closed (fixed)

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

BeaPower’s picture

BeaPower commented

sub