Defect #2122
closed
URL in send_reminders email for 'View all issues' is incorrect.
Added by Robert Chady almost 16 years ago.
Updated over 3 years ago.
Category:
Email notifications
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.
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?
- 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.
- 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?
- Status changed from 7 to Closed
- Resolution set to Fixed
Indeed. UrlRewriter#url_for doesn't work as I expected.
r1992 should fix it.
Also available in: Atom
PDF