Try finding the constructor instead of using unsafe reflection

CitizensDev · Sep 23, 2019 · deac714 · deac714
1 parent b174e10
commit deac714
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/net/citizensnpcs/api/persistence/PersistenceLoader.java b/src/main/java/net/citizensnpcs/api/persistence/PersistenceLoader.java
@@ -1,5 +1,6 @@
 package net.citizensnpcs.api.persistence;
 
+import java.lang.reflect.Constructor;
 import java.lang.reflect.Field;
 import java.lang.reflect.ParameterizedType;
 import java.lang.reflect.Type;
@@ -254,7 +255,9 @@ private static void ensureDelegateLoaded(Class<? extends Persister<?>> delegateC
         if (loadedDelegates.containsKey(delegateClass))
             return;
         try {
-            loadedDelegates.put(delegateClass, delegateClass.newInstance());
+            Constructor<? extends Persister<?>> constructor = delegateClass.getConstructor();
+            constructor.setAccessible(true);
+            loadedDelegates.put(delegateClass, constructor.newInstance());
         } catch (Exception e) {
             e.printStackTrace();
             loadedDelegates.put(delegateClass, null);