Spring A wrong logical for ConvertersForPair#getConverter

Affects: \<5.2.3>

if (!(converter instanceof ConditionalGenericConverter) ||
        ((ConditionalGenericConverter) converter).matches(sourceType, targetType)) {
    return converter;
}

it should be

if ((converter instanceof ConditionalGenericConverter) &&
        ((ConditionalGenericConverter) converter).matches(sourceType, targetType)) {
    return converter;
}

Comment From: sbrannen

If you feel that the boolean logic is incorrect, please provide a failing test case which demonstrates that the current implementation is flawed and that your proposed change fixes the flaw.

Thanks

Comment From: spring-projects-issues

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.