Add UUID support for the Path module for URL Aliases.

Comments

robloach’s picture

Should this be associated with the referencing content instead?

Status: Needs review » Needs work

The last submitted patch, uuid_path.patch, failed testing.

skwashd’s picture

Status: Needs work » Postponed (maintainer needs more info)

I'm not sure if this is needed. I tend to think of path aliases as configuration. How would this be used?

robloach’s picture

If you have a bunch of nodes with uuids, and you move them over to a new server, you now have all of your nodes correctly on the new server. Their paths, however, just have node/232132 instead of the desired clean URL paths.

dave reid’s picture

Until we have UUID resolution for paths in core (paths default to node/[node:uuid]) I don't think this is worth implementing.

skwashd’s picture

Status: Postponed (maintainer needs more info) » Postponed

I tend to agree with @davereid on this one. @dixon_'s entity work may help this. Marking this as postponed for now.

skwashd’s picture

Status: Postponed » Closed (won't fix)

There's a module for that - http://drupal.org/project/url_alias The module allows you to export your url_aliases with the node.

neochief’s picture

Status: Closed (won't fix) » Fixed

I've just committed import/export support of path aliases to UUID. Currently only node and taxonomy aliases should work.

skwashd’s picture

Status: Fixed » Needs work

This was marked as wonfix. The patch needs to be reverted.

dixon_’s picture

I agree, this patch with all the follow-up commits should probably be reverted in favor of this slightly more robust approach: #1483824: UUID path module

aspilicious’s picture

I do believe we have to revert this. Actually it's harmless to revert because the commits don't look like the original report.
They don't make any new column. They process at runtime so when we revert this and commit the uuid_path module everybody should be fine ;)

4 commits need to be reverted:
-------------------------------
http://drupalcode.org/project/uuid.git/commitdiff/d725f336aa6ed95b68cc33...
http://drupalcode.org/project/uuid.git/commitdiff/be7dba3e5902be62bf6d3c...
http://drupalcode.org/project/uuid.git/commitdiff/a43b5edf92c9cdd1d93d8a...
http://drupalcode.org/project/uuid.git/commit/11c92408d748f9f5e749cc3b4a...

I propose these next steps:
1) I create a revert patch
2) We review this patch and the patch from the other issue
3) If both are rtbc, we apply both patches
4) We party!

skwashd’s picture

I plan to revert this change on Sunday morning AEST (UTC+10) unless it is done sooner by neochief. We need to get the proper implementation of this in.

aspilicious’s picture

Status: Needs work » Needs review
StatusFileSize
new2.79 KB

This is al we need I think to revert this

alexiswatson’s picture

Looks sane after a manual review; we're testing it in-house now.

skwashd’s picture

Status: Needs review » Fixed

I've applied the rollback patch.

Status: Fixed » Closed (fixed)

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