You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@grails.apache.org by jd...@apache.org on 2026/05/14 14:18:18 UTC

(grails-core) 02/02: Fix bad compareTo vs equals implementation

This is an automated email from the ASF dual-hosted git repository.

jdaugherty pushed a commit to branch 8.0.x-hibernate7
in repository https://gitbox.apache.org/repos/asf/grails-core.git

commit b02d617d5be6d7c0cf64fb21f3cfeda10c91ff15
Author: James Daugherty <jd...@jdresources.net>
AuthorDate: Thu May 14 09:32:42 2026 -0400

    Fix bad compareTo vs equals implementation
---
 .../groovy/org/apache/grails/data/testing/tck/domains/Person.groovy   | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/grails-datamapping-tck/src/main/groovy/org/apache/grails/data/testing/tck/domains/Person.groovy b/grails-datamapping-tck/src/main/groovy/org/apache/grails/data/testing/tck/domains/Person.groovy
index 9f11a39911..895df89aed 100644
--- a/grails-datamapping-tck/src/main/groovy/org/apache/grails/data/testing/tck/domains/Person.groovy
+++ b/grails-datamapping-tck/src/main/groovy/org/apache/grails/data/testing/tck/domains/Person.groovy
@@ -80,6 +80,8 @@ class Person implements Serializable, Comparable<Person>, AsyncEntity<Person> {
 
     @Override
     int compareTo(Person t) {
-        age <=> t.age
+        age <=> t.age ?:
+                firstName <=> t.firstName ?:
+                        lastName <=> t.lastName
     }
 }