Project

General

Profile

Actions

Defect #2122

closed

URL in send_reminders email for 'View all issues' is incorrect.

Added by Robert Chady over 15 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Email notifications
Target version:
Start date:
2008-11-01
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

In the emails send_reminder sends out, the URL inserted for the 'View all issues' link is not correct if you have the project installed in a sub-directory. All other URLs that get sent out are correct, just not this one. This is based on r1646 so it is a ways behind trunk, but it looks to do the same thing in HEAD too.

Actions #1

Updated by Robert Chady over 15 years ago

Related to this issue, I also found that emails generated from SVN commits also has the incorrect URL, namely it is missing the sub-directory in the URL: /issues/show/### instead of /redmine/issues/show/###.

On a hunch I tried to update the 'host' setting in settings to include the /redmine so url_for() will build the correct URl. Unfortunately, this results in normal email notices of activity going out with /redmine/redmine/issues/show/### in the URL.

So, why are URLs getting built one way for SVN commits and send_reminders, but another way for normal activity emails?

Actions #2

Updated by Jean-Philippe Lang over 15 years ago

  • Category set to Email notifications
  • Status changed from New to Resolved
  • Target version set to 0.8
  • Affected version (unused) set to 0.7.3
  • Resolution set to Fixed
  • Affected version set to 0.7.3

This should be fixed in r1989. Add /redmine to your host setting, it should no longer be added twice to the url when sending emails from the web app.

Feedback is welcome.

Actions #3

Updated by Eric Davis over 15 years ago

  • Status changed from Resolved to 7
  • Resolution deleted (Fixed)

r1989 caused a test failure because :skip_relative_url_root is added to the url, "https://mydomain.foo/issues/show/1?skip_relative_url_root=true". Is the :skip_relative_url_root option available in mailer views?

Actions #4

Updated by Jean-Philippe Lang over 15 years ago

  • Status changed from 7 to Closed
  • Resolution set to Fixed

Indeed. UrlRewriter#url_for doesn't work as I expected.
r1992 should fix it.

Actions

Also available in: Atom PDF