When fixing filenames for existing files it would be useful to be able to redirect visitors, who may have bookmarked the old paths or otherwise shared the URLs, to new the new ones. To that end, it would be very useful if transliteration_retroactive() could:

  • Check if path_redirect is installed,
  • If so, add a {path_redirect} record for the file.
CommentFileSizeAuthor
#7 881644.patch984 bytesamateescu
#2 transliteration-support_path_redirect-881644-3.patch1.04 KBjwilson3

Comments

dave reid’s picture

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

Would be great, as long as the proper APIs are used and not direct db queries to {path_redirect}. That bit us multiple times with pathauto.

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Status: Active » Needs review
StatusFileSize
new transliteration-support_path_redirect-881644-3.patch1.04 KB

Please have a look...

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Looks simple, looks good. Anyone else care to test?

bsztreha’s picture

bsztreha commented

I also need this feature, i test and seems ok!
Patch created the alias, and it works

jwilson3’s picture

jwilson3
Primary language English
Location Ecuador
commented
Status: Needs review » Reviewed & tested by the community

In light of #4, might I venture to RTBC my own patch, or is that going to far?

amateescu’s picture

amateescu commented
Version: 6.x-3.0 » 7.x-3.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Tested this one manually and it worked as advertised so I commited #2 to 6.x-3.x.

Needs to be ported to 7.x-3.x for the Redirect module.

amateescu’s picture

amateescu commented
Assigned: Unassigned » amateescu
Status: Patch (to be ported) » Needs review
StatusFileSize
new 881644.patch984 bytes

Started to work on this.

amateescu’s picture

amateescu commented
Status: Needs review » Fixed

Commited #7 to 7.x-3.x.

http://drupalcode.org/project/transliteration.git/commit/62f9274

Status: Fixed » Closed (fixed)

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