fix #542

Feuermagier · Jul 1, 2024 · 41e3ede · 41e3ede
1 parent 6f797bb
commit 41e3ede
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/autograder-core/src/main/java/de/firemage/autograder/core/check/oop/UtilityClassCheck.java b/autograder-core/src/main/java/de/firemage/autograder/core/check/oop/UtilityClassCheck.java
@@ -27,10 +27,8 @@ public static boolean isUtilityClass(CtClass<?> ctClass) {
                 && !ctClass.getMethods().isEmpty()
                 // all methods should be static
                 && ctClass.getMethods().stream().allMatch(CtMethod::isStatic)
-                // all fields should be static and effectively final (no assignments)
-                && ctClass.getFields().stream().allMatch(
-                    ctField -> ctField.isStatic() && SpoonUtil.isEffectivelyFinal(ctField)
-                )
+                // all fields should be static
+                && ctClass.getFields().stream().allMatch(CtModifiable::isStatic)
                 // the class should not extend anything
                 && ctClass.getSuperclass() == null
                 // the class should not implement anything