Closed (fixed)
Project:
Universally Unique IDentifier
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
28 Jun 2011 at 18:10 UTC
Updated:
2 Sep 2012 at 19:51 UTC
Jump to comment: Most recent file
Comments
Comment #1
robloachShould this be associated with the referencing content instead?
Comment #3
skwashd commentedI'm not sure if this is needed. I tend to think of path aliases as configuration. How would this be used?
Comment #4
robloachIf 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.
Comment #5
dave reidUntil we have UUID resolution for paths in core (paths default to node/[node:uuid]) I don't think this is worth implementing.
Comment #6
skwashd commentedI tend to agree with @davereid on this one. @dixon_'s entity work may help this. Marking this as postponed for now.
Comment #7
skwashd commentedThere's a module for that - http://drupal.org/project/url_alias The module allows you to export your url_aliases with the node.
Comment #8
neochief commentedI've just committed import/export support of path aliases to UUID. Currently only node and taxonomy aliases should work.
Comment #9
skwashd commentedThis was marked as wonfix. The patch needs to be reverted.
Comment #10
dixon_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
Comment #11
aspilicious commentedI 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!
Comment #12
skwashd commentedI 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.
Comment #13
aspilicious commentedThis is al we need I think to revert this
Comment #14
alexiswatson commentedLooks sane after a manual review; we're testing it in-house now.
Comment #15
skwashd commentedI've applied the rollback patch.