While it might not make sense for everyone to have pathauto on activities, there are valid use cases for it. With this patch I hope to enable support for pathauto and path aliases on crm core activities.

I haven't had time to test the patch, but I'll do so as soon as our dev server is back up.

CommentFileSizeAuthor
#4 pathauto_support_for-2399585-4.patch1.46 KBt0xicCode
#2 pathauto_support_activities-2399585-2.patch7.35 KBlendude
#2 interdiff-2399585-1-2.txt8.83 KBlendude
#1 pathauto_support_for-2399585-1.patch5.29 KBt0xicCode

Comments

t0xicCode’s picture

t0xicCode commented
Status: Active » Needs review
StatusFileSize
new pathauto_support_for-2399585-1.patch5.29 KB
lendude’s picture

lendude
Primary language Dutch
Location Amsterdam
commented
Status: Needs review » Patch (to be ported)
StatusFileSize
new interdiff-2399585-1-2.txt8.83 KB
new pathauto_support_activities-2399585-2.patch7.35 KB

Couple of issues that needed fixing.

Needed to check module_exists. Doing that for both crm_core modules now so removed the dependency from .info
To get the right patterns back from pathauto the call to pathauto needed the real module name not this modules name, otherwise it would return the contact pattern for activities.

Updated the comments a bit to be more verbose.

Fixed some typos.

Committed to the sandbox

lendude’s picture

lendude
Primary language Dutch
Location Amsterdam
commented
Status: Patch (to be ported) » Fixed

Marking as fixed, let me know if your testing turns up anything.

t0xicCode’s picture

t0xicCode commented
Status: Fixed » Needs review
StatusFileSize
new pathauto_support_for-2399585-4.patch1.46 KB

It seems that crm_core 0.980 changed the name of the tokens. Applying this patch on top of the one in #2 completely fixes the issue.

lendude’s picture

lendude
Primary language Dutch
Location Amsterdam
commented
Status: Needs review » Fixed

Yeah I was working on an older version. Updated to latest CRM Core version and got the errors too.

#4 also pushed to sandbox.

Status: Fixed » Closed (fixed)

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