It's actually supposed to be a link. But it's supposed to be the full link, not just the file name. (we could choose to change that policy of course)

I noticed it as well on that mailing and stared at the code for a while but didn't decide on a fix I was happy with. I need to think about where else we might want absolute URLs instead of relative ones.

I've noticed that some email tools don't like the bug references in the email. Should they (the URL) be protected by brackets (<>)?

what do you mean by "don't like" ? what do they do?

I noticed that Mail.app on OSX seems to include the line of === signs under the bug link as part of the link which makes absolutely no sense since there is a carriage return in between them.

Mail.app was the one I noticed as well. I also reported it as a bug to Apple.

In my humble opinion the buglink (0002995) should be written as # and the number, not as a link in the email. Ofcourse, this could be made into a configurable option. but "tainting" the comments with url's in the email i doubt would make anyone happy :)

actually, I think I agree. The link to the bug on which the action was performed needs to be there but the rest are better displayed as links after you go view the bug.

Here is my fix for this (protecting the URLs):

% diff mantis/core/string_api.php original_src/mantisbt/core/string_api.php
303c303
< return "<" . config_get( 'path' ) . string_get_bug_view_url( $p_bug_id, $p_user_id ) . ">";

          return config_get( 'path' ) . string_get_bug_view_url( $p_bug_id, $p_user_id );

original_src is a CVS snapshot from 8/15

thraxisp - was this fix applied - can this be resolved now?

This seems to have been fixed in the latest versions (probably of Apple Mail). The other mail tools I have access to seem to recognize the URLs now.

I have found other mail tools that don't seem to accept split URLs (Eudora).

The original angle bracket enclosure was not added, but seemed to work in some areas.

I'll implement

Here is my patch:

Index: core/email_api.php

RCS file: /cvsroot/mantisbt/mantisbt/core/email_api.php,v
retrieving revision 1.89
diff -r1.89 email_api.php
784c784
< $t_message .= $p_visible_bug_data['email_bug_view_url'] . "\n";

                  $t_message .= '<' . $p_visible_bug_data['email_bug_view_url'] . "> \n";

Fixed brackets in CVS.