Tin Pavlinic opened SPR-16141 and commented

The attached test fails. There are two issues:

1) StandardTypeComparator's canCompare is wrong, in the sense that it answers true to comparisons between integers and strings, but compare crashes.

2) Possibly as a workaround for (1) Operator.equalityCheck does checks to make sure the comparison is safe instead of delegating to canCompare.

The result is that implementing a custom TypeComparator does not always result in it being used in equality checks.


Affects: 5.0.1

Attachments: - AlternativeComparatorTests.java (1.96 kB)

Referenced from: pull request https://github.com/spring-projects/spring-framework/pull/1581

Comment From: bclozel

Closing in favor of #1581