Pathauto supports creating new redirects through Path Redirect when changing the alias. However, the language was not passed along to path_redirect_save(). Fixed with this patch.

CommentFileSizeAuthor
#4 921188-4.patch1.08 KBroderik
pathauto-redirect-language.patch481 bytesdavid lesieur

Comments

Status: Needs review » Needs work

The last submitted patch, pathauto-redirect-language.patch, failed testing.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 6.x-1.4 » 6.x-1.x-dev
Status: Needs work » Needs review

Seems reasonable.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Assigned: Unassigned » dave reid
roderik’s picture

roderik
Primary language Dutch
Location Amsterdam,NL / Budapest,HU
commented
StatusFileSize
new 921188-4.patch1.08 KB

I guess we should take the language of the existing alias, not of the (new state of the) object/node being saved.

Patch applies to 6.x-1.x as well as 6.x-2.x (with offset). I don't see any problems.

(...except with the first call to _pathauto_set_alias() which is present in the 1.x series, but that one doesn't have to do with this patch per se. So I'm ignoring it.)

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Tested and confirmed this is a good change. Committed to Git:
http://drupalcode.org/project/pathauto.git/commit/87d98fe
http://drupalcode.org/project/pathauto.git/commit/875f2b1

Status: Fixed » Closed (fixed)

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