Project

General

Profile

Actions

Defect #11552

closed

MailHandler does not match assignee name with spaces

Added by Kameron Hott over 12 years ago. Updated over 12 years ago.

Status:
Closed
Priority:
Normal
Category:
Email receiving
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

We are working with the imap email ticket creation. It basically works except when the name has a - in it. Example would be First name = "IS -" and Last name = "Operations". If the First name is IS and the Last name = "Operations", the email processes fine. I saw while Googling around that I could use the Login instead which is isoperations. Tried that and it fails.

We are on version 1.4.1.stable.9612(MySql).
Our mail client is Exchange.


Files

defect-11552.patch (2.47 KB) defect-11552.patch William Roush, 2012-07-31 21:11
defect-11552.patch (3.13 KB) defect-11552.patch William Roush, 2012-07-31 21:31
Actions #1

Updated by William Roush over 12 years ago

Confirmed, this test fails when the user has a dash in their first name.

However assigning to the user "dashtest" works fine (the user's log in name).

Actions #2

Updated by William Roush over 12 years ago

William Roush wrote:

Confirmed, this test fails when the user has a dash in their first name.

However assigning to the user "dashtest" works fine (the user's log in name).

Actually let me correct myself, the problem is that there is a space in the first name, I think Redmine is splitting on that space, and fails to look up the user.

Setting the name to "IS- Operations" works fine.

So I think it's doing first name "IS", last name "- Operations".

Actions #3

Updated by Jean-Philippe Lang over 12 years ago

I think Redmine is splitting on that space, and fails to look up the user

You're perfectly right.

Actions #4

Updated by William Roush over 12 years ago

Alright, here is a patch that adds a test that replicates the issue and a fix (Redmine 2.0 default branch).

I added the last combined check as a last-ditch check (re-iteration), I'm a bit worried about catching other names by accident so I only want to grab it if all other name look-ups fail first.

Of course the devs can move that somewhere else if they think that isn't a problem.

Actions #6

Updated by Kameron Hott over 12 years ago

  • Status changed from New to Resolved

Thank you, that patch did fix it. The email works now with IS - Operations. Have a great day and thank you for the very quick fix!!

Actions #7

Updated by William Roush over 12 years ago

Edit: sorry, scratch that, I had a filter issue.

Actions #8

Updated by Jean-Philippe Lang over 12 years ago

  • Target version set to 2.1.0
Actions #9

Updated by Jean-Philippe Lang over 12 years ago

  • Subject changed from redmine:email:receive_imap assignee name with a - fails. to MailHandler does not match assignee name with spaces
  • Category set to Email receiving
  • Status changed from Resolved to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Fixed in r10236 with a slightly different approach, thanks for pointing this out.

Actions

Also available in: Atom PDF