Github user jugi92 commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2499#discussion_r171875056
  
    --- 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 --
    
    Yes sorry, it was supposed to be methods.length > 0
    I will change and open a new pull request


---

Reply via email to