I've created a patch to support pathauto so you can create aliasses for profile entities.

Comments

barthje created an issue. See original summary.

barthje’s picture

Status: Active » Needs review
StatusFileSize
new971 bytes

Included the patch

Status: Needs review » Needs work

The last submitted patch, 2: profile-pathauto-2942455.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

barthje’s picture

Status: Needs work » Needs review
StatusFileSize
new1015 bytes

Status: Needs review » Needs work

The last submitted patch, 4: profile-pathauto-2942455-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

mglaman’s picture

Title: support for pathauto » Add a path field to support Pathauto
Priority: Normal » Minor

We should be adding the field on the entity itself.

wluisi’s picture

StatusFileSize
new727 bytes

Here's a patch for adding a path base field to the Profile entity itself.

bojanz’s picture

Priority: Minor » Normal

This will need a bit more thought, because our default use case doesn't require path aliases, and showing such a field by default to existing users (such as the ones using Commerce) would be confusing. So we'd at least need to make sure that the path field is hidden by default.

littlepixiez’s picture

StatusFileSize
new27.23 KB

I was having a gander at this as we want to have path aliases for one of our profile types. It looks like Pathauto can add a 'path' field definition to any entity type that doesn't currently have one - if you go to "/admin/config/search/path/settings" and switch it on for "Profile", then you can configure patterns for profile entities this way and it will appear in the form display as expected.

If paths were to be a default in this module, presumably tests and the .info file would need the "Path" module as a dependency, and field definitions would need to be added in an update hook as well as predefined in the entity? Happy to supply a patch and pop the URL alias in the hidden region, but like @bojanz says, I'm not sure all users would need this. :)

rolodmonkey’s picture

StatusFileSize
new618 bytes

I have re-rolled the patch from comment #4.

bojanz’s picture

Category: Feature request » Support request
Status: Needs work » Fixed

I have confirmed that #9 is correct. There is no need for a patch. Just go to /admin/config/search/path/settings and enable Pathauto for Profile.

Thanks, littlepixiez!

Status: Fixed » Closed (fixed)

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

hockey2112’s picture

#9 worked for me, thanks!