NormalizedMessageImpl.getAttachment() returns DataHandler when it should return
null
------------------------------------------------------------------------------------
Key: SM-1186
URL: https://issues.apache.org/activemq/browse/SM-1186
Project: ServiceMix
Issue Type: Bug
Components: servicemix-core
Affects Versions: 3.2.1
Environment: any
Reporter: Alex Vigdor
The current implementation of getAttachment() in NormalizedMessageImpl always
returns a DataHandler if the message has any attachments, even if it does not
have an attachment with the given ID:
public DataHandler getAttachment(String id) {
if (attachments != null) {
return new DataHandler((DataSource) attachments.get(id));
}
return null;
}
According to the JBI javadocs, getAttachment should return "DataHandler
representing attachment content, or null if an attachment with the specified
identifier is not found".
Thus, the implementation should add an additional check
if(attachments !=null && attachments.get(id) != null){
return new DataHandler((DataSource) attachments.get(id));
}
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.