I'm creating a Feature for a view which defines a page with the url path 'path-in-english'. Using the core path module, I can define a path alias for other languages, for example 'chemin-en-francais' (path in French). However, it is possible to export views in Features, but, as far as I can tell, not URL aliases defined with the core path module. It would be a nice functionality to have!

CommentFileSizeAuthor
#11 uuid_node_save_path_alias-982818.patch812 bytesandreyjan

Comments

Grayside’s picture

Grayside commented

The problem here is that no path alias referring to content, such as nodes, could properly be exported.
Path aliases to paths consistent across all Drupal sites could conceivably be exported.

For now, it would be easier to wrap some logic around path_set_alias() to set up your aliases on hook_install().

alberto56’s picture

alberto56 commented

Thanks for your answer. Good point indeed. path_set_alias() is the path I decided to go with. The question arises, though, about overriding and resetting Features.

Path aliases to paths consistent across all Drupal sites could conceivably be exported.

Right, and I would add path aliases to views pages as well, as long as the path alias(es) and the views to which they refer co-exist in the same view.

Cheers,

Albert.

robloach’s picture

robloach
he/him
commented
Project: Features » UUID Features Integration
Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Postponed

#1202958: UUIDs for Path/URL Aliases

alberto56’s picture

alberto56 commented

I mentioned this issue in a blog post about features: http://mediatribe.net/en/node/47

q0rban’s picture

q0rban commented

I don't see a need for UUIDs in order to make this exportable. A path can only have one alias, so in a sense, that path itself is it's own unique identifier, or machine name.

q0rban’s picture

q0rban commented

To clarify what I mean, I don't think it would be too hard to write features exportables for url aliases without UUIDs:

features['url_alias'][] = "und:user/login"
features['url_alias'][] = "fr:user/login"

hswong3i’s picture

hswong3i commented
Status: Postponed » Active

Since #1202958: UUIDs for Path/URL Aliases already get fixed shall we review this issue once again?

danithaca’s picture

danithaca commented

Any new update?

halth’s picture

halth
Primary language Portuguese, Brazil
Location Brazil
commented
Issue summary: View changes

Updates?

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
commented

@danithaca & @halth: If there are no new comments, then there are no new updates. Please do not post comments like those above as you're needlessly spamming everyone that is subscribed to the issue. If you'd like to move things along, then either work on developing a patch, or pay someone else to do it. Thanks.

andreyjan’s picture

andreyjan commented
StatusFileSize
new uuid_node_save_path_alias-982818.patch812 bytes

As a quick solution for exported nodes path aliases can be attached patch.

kenorb’s picture

kenorb commented
Status: Active » Needs review
anybody’s picture

anybody
Primary language German
Location Porta Westfalica
commented

#11 works for me. Can we get further feedback? Is someone else using the patch?

petednz’s picture

petednz commented

does this overlap with https://www.drupal.org/node/2220837

aubjr_drupal’s picture

aubjr_drupal commented

@petednz, yes - I've combined the patch from #11 with #23 on https://www.drupal.org/project/uuid_features/issues/2220837 to make the following patch: https://www.drupal.org/files/issues/2018-12-21/uuid_node_save_path_alias...

Per @goron from https://www.drupal.org/project/uuid_features/issues/2220837#comment-1271...

"The patch in #23 together with the patch at https://www.drupal.org/node/982818#comment-10040366 seem to provide the solution to exporting nodes successfully with alias (this patch) as well as rebuilding correctly on features revert (other issue's patch). Seems to me they should be combined and consolidated into one issue."

petednz’s picture

petednz commented

thx aubjr_drupal - hopefully someone can progress this - i will see if i can review it some point

das-peter’s picture

das-peter commented
Status: Needs review » Fixed

Looks good to me - used patch from #2220837.
Committed & pushed.

Status: Fixed » Closed (fixed)

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