Global Redirect in core

moshe weitzman - October 9, 2006 - 00:58
Project:Drupal
Version:7.x-dev
Component:node system
Category:feature request
Priority:normal
Assigned:Unassigned
Status:postponed
Description

seems to be that this is sensible default behavior for core. is there a core patch yet? if not, please submit.

#1

nicholasThompson - October 9, 2006 - 08:32

If only I knew where to start for that kinda thing. I'll take a peek around - see what I can find.

#2

greggles - October 9, 2006 - 23:49

Personally and as pathauto maintainer I think we'd be best off if urls could have at a max 2 aliases: the internal alias and 1 external alias and then any other aliases are implemented as some form of redirect (301, 302, etc.).

This has several benefits:

1. Performance - drupal wouldn't need to load the behemoth url_alias table because the alias could be stored in the node/ and similar locations
2. No longer any opportunity to have content at multiple nodes which is a problem both for usability (Jakob Nielsen writes about this) and for SEO
3. It makes pathauto simpler to maintain :)

That is something I'd like to work on for 6.x (isn't it nice to know that the next version will be 6.x!)

This node is certainly an improvement towards that direction of limiting the public face of Drupal and helps spread the point that multiple aliases for a node is a "bad thing".

#3

FiReaNG3L - October 10, 2006 - 00:23

I totally agree with your suggestions greggles (especially getting rid of the huge url_alias table, and probably the 20-50 database calls per page to it :)). There's no use that i can think of for 2 aliases pointing to the same content, and core 301 redirect for node/nid should be default.

Great stuff!

#4

nicholasThompson - October 10, 2006 - 08:35

Getting the alias into the node table is a fantastic idea. I dont know why this hasn't been done already.

Just an idea for performance - is it not worth generating the url_alias table as an array (internal URL as the key, external as the value) and then dumping it in the cache table? At least then there is just one hit to pull them all out...

Also - about submitting this module to core, is there not a code freeze?

#5

drawk - October 10, 2006 - 08:45

Yeah, but it should still be submitted. Whether it makes it into 5 or 6 depends, I guess, on if it is seen as a feature or a bug fix.

#6

FiReaNG3L - October 10, 2006 - 15:46

I see it as a bugfix personally, because from a search engine optimization point of view, it kills sites. Google really doesn't like 2 'pages' (aliases) having the same content on the same site, especially if its for every piece of content.

#7

nicholasThompson - January 15, 2007 - 22:45
Status:active» postponed

#8

Pancho - January 3, 2008 - 13:50
Status:postponed» active

Too late for D6 now, let's get this at least into D7! Might work on this next week.

#9

sun - April 14, 2008 - 02:12

Subscribing. The de-slashing feature of GlobalRedirect might also be possible at .htaccess level.

#10

nicholasThompson - April 14, 2008 - 08:45

The deslashing is DEFINATELY possible at htaccess level, however the question about de-slashing is surely up to the person running the website? What if yuo WANT slashed on the end of the URL?

#11

Freso - August 15, 2008 - 13:36
Title:patch for core» Global Redirect in core
Project:Global Redirect» Drupal
Version:HEAD» 7.x-dev
Component:Code» node system

If you want Global Redirect's functionality in core... shouldn't the issue be against core?

#12

Freso - August 15, 2008 - 13:37
Category:task» feature request

#13

sun - August 27, 2008 - 16:21
Status:active» postponed

Based on the reported bugs that have been reported since release 1.2 I'm setting this to postponed until GR has matured.

GR badly needs some love from i18n, custom_url_rewrite as well as URL alias experts - please step in if you belong to this group.

 
 

Drupal is a registered trademark of Dries Buytaert.