Just wanted to let the maintainers of pathauto that I've taken on a maintainership role of the path_redirect module and I'm working on getting things up to shape. The path_redirect integration with pathauto has been broken for a while it seems and the API will be changing slightly. I will post a patch shortly when I get it all working and release an official version of path_redirect with the API change.

CommentFileSizeAuthor
#3 365992-pathauto-path-redirect-integration-D6.patch2.13 KBdave reid

Comments

greggles’s picture

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

Great news!

path_redirect integration with pathauto has been broken for a while

In any specific way? I'm using it on many sites without (noticeable) issue.

dave reid’s picture

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

I had it reported at #316411: Pathauto integration not working, but I'm not sure exactly how it's broken (I ran some basic click-around-the-site tests and it didn't work for me before any of the API changes). I'm working on getting a SimpleTest test case written specifically for the pathauto integration so I can figure it out exactly.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Version: 7.x-1.x-dev » 6.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new 365992-pathauto-path-redirect-integration-D6.patch2.13 KB

Here's the patch that matches the path_redirect API functions with current path_redirect 6.x-1.x. Ideally I'd like to time official releases of each of our modules at the same time so that there is as little problem with incompatibilities as possible (although I have made the API changes flexible to accept the old function parameters).

dave reid’s picture

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

I'm also happy to report that my little test case for pathauto integration is working perfectly now. :)

Freso’s picture

Freso commented
Status: Needs review » Fixed

Committed to 6.x-2.x and 6.x-1.x. When are you planning on making a release with the new API?

dave reid’s picture

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

It's actually out right now in my beta releases, but I made sure that it was backwards-compatible. I really don't need to change anything. It would be nice to time 'official' releases together, though.

Status: Fixed » Closed (fixed)

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