[JAMES-721] MBoxMailRepository.remove(String key) causes ClassCastException

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.0, 2.3.0
• Fix Version/s: 2.3.1
• Component/s: MailStore & MailRepository
• Assignee: Norman Maurer
• Reporter: Joachim Draeger
• Created: 02/Dec/06 19:33
• Updated: 21/Nov/07 08:31
• Resolved: 04/Jan/07 12:49

Description

It calls
ArrayList keys = new ArrayList();
keys.add(key);
this.remove(keys);
although remove(Collection) expects Mail objects

02/12/06 20:25:14 ERROR pop3server: Some deleted messages were not removed: java.lang.String
java.lang.ClassCastException: java.lang.String
at org.apache.james.mailrepository.MBoxMailRepository$3.messageAction(MBoxMailRepository.java:702)
at org.apache.james.mailrepository.MBoxMailRepository.parseMboxFile(MBoxMailRepository.java:277)
at org.apache.james.mailrepository.MBoxMailRepository.remove(MBoxMailRepository.java:691)
at org.apache.james.mailrepository.MBoxMailRepository.remove(MBoxMailRepository.java:767)

public void remove(Mail mail)

{ // Convert the message into a key Vector delVec = new Vector(); delVec.addElement(mail); remove(delVec); }

Works, but the comment is wrong.