Apache
/
activemq
mirror of https://github.com/apache/activemq.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

https://issues.apache.org/jira/browse/AMQ-5525 - worked around the issue with blueprint converter - all. All of the activemq commands have the same single arg signature, so the conversion should be trivial, doing the conversion inline in the default case

This commit is contained in:
gtully 2015-01-19 14:34:37 +00:00
parent efe68aa032
commit 548aa1e415
1 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
activemq-karaf/src/main/java/org/apache/activemq/karaf/commands/ActiveMQCommand.java
View File

@ -129,6 +129,25 @@ public class ActiveMQCommand extends AbstractCommand implements CompletableFunct
}
}
if (argumentValues.size() == 1 && arguments.size() == 1) {
Object val = argumentValues.values().iterator().next();
// short circut convert via blueprint... cause all our commands match this
// bluepring was failing to convert the last long param to a string for browse
// where dest is a numeric value - activemq-karaf-itests
// see: org.apache.activemq.karaf.itest.ActiveMQBrokerFeatureTest.test()
if (val instanceof List) {
Field field = arguments.values().iterator().next();
List<Object> values = (List<Object>) val;
ArrayList<String> convertedValues = new ArrayList<String>(values.size());
for (Object o : values) {
convertedValues.add(String.valueOf(o));
}
field.setAccessible(true);
field.set(action, convertedValues);
return true;
}
}
for (Map.Entry<Argument, Object> entry : argumentValues.entrySet()) {
Field field = arguments.get(entry.getKey());
Object value = convert(action, session, entry.getValue(), field.getGenericType());