Wael Nasreddine wrote:
> Hello,
>
> My boss always assign the target version when he opens issues by email, and I always have to edit all issues he (or I) opened via email to set the correct target version and remove it from the description manually.
>
> It would be easier if redmine parses the "Version:" tag and assign it correctly.
>
> Thanks

--------------------------------------------------------------------------------

--------------------------------------------------------------------------------
Ummmm Guys!! What's going on?
--------------------------------------------------------------------------------
Wael Nasreddine wrote:
> Ummmm Guys!! What's going on?

It seems Alexsandro is trying out the redmine here rather than on the demo site and I reverted it, sorry for not stating it explicitly.
--------------------------------------------------------------------------------
Felix Schäfer wrote:
> Wael Nasreddine wrote:
> > Ummmm Guys!! What's going on?
>
> It seems Alexsandro is trying out the redmine here rather than on the demo site and I reverted it, sorry for not stating it explicitly.

I kinda figured it out when I saw his Test defect :D, anyway no problem thanks for fixing it fast, hopefully it'll make it to redmine fast too :)
--------------------------------------------------------------------------------
I've created a patch myself, the "Version" keyword is now parsed and correctly assigned, however the *version* must already exist, if the version does not already exist, it's ignored...

I plan on to extending it to create a Target Version if one doesn't exist, but I prefer if Eric or someone else do this part because I'm still too new to RoR...
--------------------------------------------------------------------------------
I would appreciate it if someone would remove the first posted file because the unit test file is not indented correctly (using tabs not spaces)

Sorry.. :(
--------------------------------------------------------------------------------
Damn IDEs :(... voilà, the correctly indented file, saved you 2 minutes...

P.S: It would be interesting if I could remove my own attachments...
--------------------------------------------------------------------------------
Tagged for [[BugMash-10]].
--------------------------------------------------------------------------------
I'm getting some test failures with this patch. Can you update to the latest Redmine trunk, reapply your patch, and try running the MailHandler's tests?

<pre>
1) Failure:
test_add_issue_note(MailHandlerTest) [test/unit/mail_handler_test.rb:253]:
<false> is not true.

2) Failure:
test_add_issue_note_should_send_email_notification(MailHandlerTest) [test/unit/mail_handler_test.rb:277]:
<false> is not true.

3) Failure:
test_add_issue_note_with_attribute_changes(MailHandlerTest) [test/unit/mail_handler_test.rb:262]:
<false> is not true.

</pre>
--------------------------------------------------------------------------------
Added in r4394. The default keyword is "Fixed version" but r4395 lets you use localized keywords.
--------------------------------------------------------------------------------

relates,Closed,5594,Improve MailHandler's keyword handling