Hi, I just installed Drupal and then opened my homepage. When I click on create a first account and put a username and my email I get this message: "The URL is not valid and cannot be changed"

What should I do?

Please tell me if you need more information. I'm pretty sure my settings are correct. The problem seems to be related to email sendings.

Thanks for your help.

CommentFileSizeAuthor
#12 settings.php_1.patch1.82 KBeaton
#9 settings_slash_2.diff739 byteschx
#8 settings_slash_1.diff735 byteschx
#4 settings_slash_0.diff793 byteschx
#3 settings_slash.diff892 byteschx

Comments

heine’s picture

heine commented
Priority: Critical » Normal

What should I do?

1. Adapt $base_url in default/sites/settings.php. Make sure there's no trailing slash.
2. Tell us more. e.g. from what does the error message originate
3. Use follow up, instead of opening new issues.

NadirG’s picture

NadirG commented
Status: Active » Fixed

Thank you so much!!!!! I removed the slash and it worked, I didn't know the problem could come from this, I hope my message will help other people too!
Cheers,

Nadir

chx’s picture

chx commented
Title: Error message when I create first acount » Move the no slash warning closer to the $base_url
Assigned: Unassigned » chx
Priority: Normal » Minor
Status: Fixed » Needs review
StatusFileSize
new settings_slash.diff892 bytes

Well then.

chx’s picture

chx commented
StatusFileSize
new settings_slash_0.diff793 bytes
heine’s picture

heine commented

ROFL; this was a support request for 4.6.5, where the no trailing slash sentence is directly above $base_url... We can't force people to read.

But still a +1

moshe weitzman’s picture

moshe weitzman commented

people who don't read once probably won't read twice. no need to clutter.

chx’s picture

chx commented
Status: Needs review » Closed (works as designed)

This patch was just me venting over a moron.

chx’s picture

chx commented
Status: Closed (works as designed) » Needs review
StatusFileSize
new settings_slash_1.diff735 bytes

Eaton's idea. Could not resist to move it into a patch.

chx’s picture

chx commented
StatusFileSize
new settings_slash_2.diff739 bytes

Fixed typos.

moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Reviewed & tested by the community

looks good to me. nice rhyme.

Steven’s picture

Steven commented

Is there any reason why we don't just rtrim('/') ?

eaton’s picture

eaton commented
Status: Reviewed & tested by the community » Needs work
StatusFileSize
new settings.php_1.patch1.82 KB

I am a bad, bad man.

But seriously, Steven is right. An rtrim could fix this.

Bèr Kessels’s picture

Bèr Kessels commented

I recall that we discussed this in the passed.

It surely fits the matra "be generous on what you take for input, but strict on output", if we trim any leading slashes of.

however, the point then was, that a single line in the comments "weights" nothing, while an rtrim is run on any pageview. I agree that helping morons, by making the app smarter, is a good thing, but only as long as it has no impact on all the non-morons.

I am all for "wont fixing this".

chx’s picture

chx commented
Status: Needs work » Reviewed & tested by the community

I think a little humour can't hurt.

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed a modified version of this patch to HEAD.

dries’s picture

dries commented
Status: Fixed » Closed (fixed)
beginner’s picture

beginner commented
Version: 4.6.5 » 4.7.2

Wow! Eaton!

Webchick said she learned a lot form your 'trailing slash broadway musical'.
http://drupal.org/node/44987#comment-109568
;)