Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2499#discussion_r171863515
--- Diff:
nifi-nar-bundles/nifi-jms-bundle/nifi-jms-processors/src/main/java/org/apache/nifi/jms/cf/JMSConnectionFactoryProvider.java
---
@@ -257,21 +256,26 @@ private void
setConnectionFactoryProperties(ConfigurationContext context) {
*/
private void setProperty(String propertyName, Object propertyValue) {
String methodName = this.toMethodName(propertyName);
- Method method = Utils.findMethod(methodName,
this.connectionFactory.getClass());
- if (method != null) {
+ Method[] methods = Utils.findMethods(methodName,
this.connectionFactory.getClass());
+ if (methods != null && methods.length < 0) {
--- End diff --
methods.length can never be less than 0 here... was that supposed to be > 0
perhaps?
---
