The default value for shorturl_domain is HTTP_HOST

variable_get('shorturl_domain', $_SERVER['HTTP_HOST']);

This actually fails validation if you go to the settings page the first time and just save.

I think it should be changed as below

variable_get('shorturl_domain', 'http://' . $_SERVER['HTTP_HOST']);

in both the settings form and the shorturl_shorten() function.

The patch is from my dev version - which is getting slowly further from the CVS version

CommentFileSizeAuthor
short_url_default_domain.patch1.08 KBseanburlington

Comments

irakli’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.