Including $base_url in the generated url in the emails doesn't work since Drupal's url() function will add in the subdirectory drupal is installed in, so you get the subdirectory twice, like so:

http://www.mozdev.org/drupal/drupal/comment/edit/40
$base_url ---^^--- url()

Where it should be:
http://www.mozdev.org/drupal/comment/edit/40

CommentFileSizeAuthor
spam-5.x-1.1-2-url.patch3.84 KBdwarner

Comments

dwarner’s picture

dwarner commented

Non-monospace text screwed up my explanation of what was $base_url and what was url();

For this URL: http://www.mozdev.org/drupal/drupal/comment/edit/40
$base_url = http://www.mozdev.org/drupal
url('comment/edit/40') = /drupal/comment/edit/40

jeremy’s picture

jeremy commented
Version: 5.x-1.1-2 » 5.x-3.x-dev
Status: Active » Needs review

I'll review this fix for the 3.x development branch.

jeremy’s picture

jeremy commented
Status: Needs review » Fixed

Bug not applicable to 3.x version of the spam module, closing.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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