Now that we have #841804: entity_uri() should be used wherever possible committed and am very close on #288639: Taxonomy term aliases are build against incorrect source paths there should actually be no reason to special case the handling of forum module. Since we will be using taxonomy_term_path/entity_uri everywhere, we can just re-use the existing taxonomy pathauto support. Yay for more simple code!

CommentFileSizeAuthor
#8 863980-pathauto-forum-taxonomy-wedding.patch9.38 KBdave reid
#7 863980-pathauto-forum-taxonomy-wedding.patch9.98 KBdave reid
#6 863980-pathauto-forum-taxonomy-wedding-D7.patch9.98 KBdave reid
#5 863980-pathauto-forum-taxonomy-wedding-D7.patch8 KBdave reid
#2 863980-pathauto-forum-taxonomy-wedding-D7.patch7.16 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Priority: Normal » Major
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs work
StatusFileSize
new 863980-pathauto-forum-taxonomy-wedding-D7.patch7.16 KB

Hrmf, the taxonomy and forum modules keep separate UIs, so I'm not sure how much of this makes sense to do. This definitely makes things a little more difficult if we want to generalize with entity information.

...

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Assigned: dave reid » Unassigned
Priority: Major » Normal

Bumping down in priority

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Assigned: Unassigned » dave reid
Priority: Normal » Major

Bumping back to major as I would really like this solved before 7.0.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Needs review
StatusFileSize
new 863980-pathauto-forum-taxonomy-wedding-D7.patch8 KB

And now here's the working patch with update function. This is ready to roll for me.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 863980-pathauto-forum-taxonomy-wedding-D7.patch9.98 KB

Without setting a forum variable this time.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 863980-pathauto-forum-taxonomy-wedding.patch9.98 KB
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 863980-pathauto-forum-taxonomy-wedding.patch9.38 KB
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Priority: Major » Normal

This can wait until after a 7.0 release.

Scott J’s picture

Scott J commented

OT:
What would be really good, is for Drupal core to stop special-casing Forum module (and Blog for that matter), and just make their functionality available to ALL content and taxonomy types.

mably’s picture

mably commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.