always check for @Config.Ignore before processing field (#71)

GTNewHorizons · Sep 13, 2024 · 6a39dbe · 6a39dbe
1 parent ecd7d92
commit 6a39dbe
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/main/java/com/gtnewhorizon/gtnhlib/config/ConfigurationManager.java b/src/main/java/com/gtnewhorizon/gtnhlib/config/ConfigurationManager.java
@@ -170,6 +170,10 @@ private static void processConfigInternal(Class<?> configClass, String category,
                 || foundCategory && cat.requiresWorldRestart();
 
         for (val field : configClass.getDeclaredFields()) {
+            if (field.isAnnotationPresent(Config.Ignore.class)) {
+                continue;
+            }
+
             if (instance != null && Modifier.isStatic(field.getModifiers())) {
                 throw new ConfigException(
                         "Illegal config field: " + field.getName()
@@ -178,10 +182,6 @@ private static void processConfigInternal(Class<?> configClass, String category,
                                 + ": Static field in instance context! Did you forget an @Config.Ignore annotation?");
             }
 
-            if (field.getAnnotation(Config.Ignore.class) != null) {
-                continue;
-            }
-
             field.setAccessible(true);
 
             if (!ConfigFieldParser.canParse(field)) {