Closed (fixed)
Project:
Pathauto
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
30 Aug 2010 at 12:19 UTC
Updated:
21 Oct 2015 at 15:04 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
gordon commentedattach patch
Comment #2
gordon commentedThe hook changed on me. Fixed.
Comment #4
dave reidNew features are applied to 7.x-1.x only and then backported to 6.x-2.x. The 6.x-1.x branch is closed to feature requests.
Comment #5
hefox commented(#728004: Allow abitrary modules to alter the pipeline is the issue about the hook; subscribe)
Comment #6
cyberwolf commentedSubscribing.
Comment #7
bc commentedsubscribing
Comment #8
kehan commentedsubscribing
Comment #9
dave reidThis also doesn't account for any multi-lingual variables.
Comment #10
joachim commentedAh yes, I see in pathauto_pattern_load_by_entity():
So this patch would need to consider pathauto_node_{$bundle}_pattern for the current node types being exported, and also pathauto_node_{$bundle}_{$language}_pattern, presumably for all languages enabled on the site, or more likely all the variables of that form that are defined.
Comment #11
dave reidComment #12
dave reidThis should work for node types and taxonomy vocabularies.
Comment #13
dave reidComment #15
dave reidCommitted #12 to 7.x-1.x.
Comment #16
joelpittetSeems this tries to force pathauto into all my existing features... is this intended?
Comment #17
dave reidI don't know how Features handles this hook with existing features. But I know this is how we should have supported exporting it from the start. You can always change it and export the settings to another feature, but this matches the behavior of other content-type controlled variables.
Comment #18
joelpittetSeems to be producing a bunch of these exclude groupings in different node related features as I ignore the suggestions:
Comment #19
dave reid@joelpittet: Does your content type feature also include all the taxonomy vocabularies exported along with it? Or also have excluded all the vocabularies?
Comment #20
joelpittetVocabularies excluded as well.
Comment #21
dave reidHrm, I'm not sure what to tell you or if this is even a problem. That seems to be expected of Features when you opt-out of the auto-exporting/grouping of variables with the content type / vocabularies.
Comment #22
joelpittetThanks will maybe bring it up in the features queue. Thought I'd share a concern after this was committed.