Path aliases are not possible to use as default front page [#204344]

As stated above

Edit Site Information - Default front page

I am wanting to change the default home page from "node" to a value "home"

I have created a page, and using clean url's, set the url as "home"

I have tested that the url is valid by going to http://mysite.com/home - I get the page

When I put "home" as the value in the default path I get an error

The path 'home' is either invalid or you do not have access to it.

I am logged is as user 1

I have tried a different url path name - did not resolve

I have only one page of content so far - tried to add a second - did not resolve either

Hope this helps

Alex

Comment	File	Size	Author
#6	path_front.patch	934 bytes	catch
#6
#1	system.path_.admin_.patch	950 bytes	marcingy
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

marcingy CreditAttribution: marcingy commented 30 December 2007 at 22:11

Priority:

Normal

» Critical

File	Size
system.path_.admin_.patch	950 bytes

This issue is because the patch alias wasn't being converted into the true path before being checkd for validity.

Patch provided

Comment #2

marcingy CreditAttribution: marcingy commented 30 December 2007 at 22:11

Status:

Active

» Needs review

Comment #3

marcingy CreditAttribution: marcingy commented 30 December 2007 at 22:14

Version:

6.0-rc1

» 6.x-dev

Comment #4

Rowanw CreditAttribution: Rowanw commented 31 December 2007 at 02:06

Status:

Needs review

» Needs work

(Stripping trailing CRs from patch.)

Otherwise the patch appears to work.

However the extra notice isn't really necessary (below) as it doesn't change anything for the user.

"The menu system stores system paths only, but will use the URL alias for display. x has been stored as node/1"

Comment #5

marcingy CreditAttribution: marcingy commented 31 December 2007 at 07:36

I did this as this replicates how the menu admin screens work for adding a menu item. I'll re-roll the patch in morning to remove the CRS.

Comment #6

catch

he/him

English

CreditAttribution: catch commented 31 December 2007 at 14:22

Status:

Needs work

» Reviewed & tested by the community

File	Size
path_front.patch	934 bytes

OK I tested this and it works fine. Agree 100% with using the same logic and string as the menu system (and since it's an identical string it should be ok post freeze I guess).

Trailing CRs don't stop things from being committed afaik, but here's one without anyway. Don't credit me for it if it gets committed.

Comment #7

Gábor Hojtsy

he/him

Hungarian

Hungary

CreditAttribution: Gábor Hojtsy commented 31 December 2007 at 14:43

Title:	Edit Site Information - Default front page	» Path aliases are not possible to use as default front page
Priority:	Critical	» Normal
Status:	Reviewed & tested by the community	» Fixed

- I would not call this bug critical, it does not stop Drupal from working in any case. Also retitled so we have a proper title for the issue.

- Reviewed the patch, and although we can argue about the default front page not being part of the menu system, we are definitely dealing with a menu system design artifact here, so the message was right there IMHO.

Committed. Thanks.