Vote #64082
完了URL in send_reminders email for 'View all issues' is incorrect.
0%
説明
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.
journals
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?
--------------------------------------------------------------------------------
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.
--------------------------------------------------------------------------------
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?
--------------------------------------------------------------------------------
Indeed. UrlRewriter#url_for doesn't work as I expected.
r1992 should fix it.
--------------------------------------------------------------------------------