In testing, I found that the old url paths no longer included the group that the content belonged to. There was also an erronious assumption that the trusted contacts url path would be of the first trusted contact, when it really should be the author.

The following patch should clean all of this up and make urls nice again.

CommentFileSizeAuthor
#2 2292927-2-url-aliases-fix.patch6.08 KBdevin carlson
#1 2292927-url-aliases-fix.patch6.46 KBjaperry

Comments

japerry’s picture

japerry
Location KOMK
commented
StatusFileSize
new 2292927-url-aliases-fix.patch6.46 KB
devin carlson’s picture

devin carlson commented
StatusFileSize
new 2292927-2-url-aliases-fix.patch6.08 KB

An updated patch which corrects the commons_groups update function name (I've got some experience with that :P) and makes some minor coding style improvements to the token implementation.

I didn't run into any issues in testing with content posted to a single group, multiple groups or to private contacts with og_access enabled. I think there may be a better alias than "trusted-content" for content posted to trusted contacts but I don't have a problem with it and couldn't come up with any suggestions.

  • japerry committed c29f214 on 7.x-3.x 
    Issue #2292927 by japerry, devincarlson: Cleanup URL aliases.
japerry’s picture

japerry
Location KOMK
commented
Status: Needs review » Fixed

Thanks Devin, fixed!

Status: Fixed » Closed (fixed)

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