Fix post-init methods not being run

- Since subclass converters were added, config post-init methods were not working as intended due to a mistake
- Enable asynchronous file writes to sql databases for BlockDataManager

Author: boxbeam

res/plugin.yml

@@ -1,6 +1,6 @@
 name: RedLib
 main: redempt.redlib.RedLib
-version: 2022-02-02 02:53
+version: 2022-02-15 22:13
 author: Redempt
 api-version: 1.13
 load: STARTUP

src/redempt/redlib/RedLib.java

@@ -50,7 +50,7 @@ public static String msg(String msg) {
 	 * @return An instance of RedLib if it is a plugin dependency, or your plugin if RedLib is shaded
 	 */
 	public static Plugin getInstance() {
-		return redLib != null ? redLib : getCallingPlugin();
+		return redLib != null ? redLib : JavaPlugin.getProvidingPlugin(RedLib.class);
 	}
 
 	private static int getMidVersion() {

src/redempt/redlib/blockdata/BlockDataManager.java

@@ -180,7 +180,8 @@ private CompletableFuture<Void> save(ChunkPosition pos, boolean force) {
 		blocks.forEach((k, v) -> {
 			map.put(k.toString(), v.data);
 		});
-		return backend.save(pos, map.toString());
+		String data = map.toString();
+		return backend.save(pos, data);
 	}
 
 	private CompletableFuture<Void> unload(ChunkPosition pos) {
@@ -203,6 +204,12 @@ public void remove(DataBlock db) {
 		Optional.ofNullable(dataBlocks.get(cpos)).ifPresent(m -> m.remove(db.getBlockPosition()));
 	}
 
+	/**
+	 * Moves a DataBlock to a new location asynchronously
+	 * @param db The DataBlock whose data should be moved
+	 * @param location The Block to move the data to
+	 * @return A CompletableFuture for the moving task
+	 */
 	public CompletableFuture<DataBlock> moveAsync(DataBlock db, Block location) {
 		remove(db);
 		ChunkPosition cpos = new ChunkPosition(location);

src/redempt/redlib/blockdata/backend/SQLiteBackend.java

@@ -37,6 +37,7 @@ public SQLiteBackend(Path path) {
 			e.printStackTrace();
 		}
 		helper = new SQLHelper(SQLHelper.openSQLite(path));
+		helper.execute("PRAGMA synchronous = OFF;");
 		helper.executeUpdate("CREATE TABLE IF NOT EXISTS data (x INT, z INT, world STRING, data TEXT, PRIMARY KEY (x, z, world));");
 		helper.setCommitInterval(5 * 20 * 60);
 	}

src/redempt/redlib/config/instantiation/FieldSummary.java

@@ -74,7 +74,6 @@ public static FieldSummary getFieldSummary(ConversionManager manager, Class<?> c
 			List<ConfigField> fields = new ArrayList<>();
 			Map<ConfigField, TypeConverter<?>> converters = new HashMap<>();
 			Method postInit = null;
-			
 			while (clazz != null && (staticContext || clazz.isAnnotationPresent(ConfigMappable.class) || Instantiator.isRecord(clazz))) {
 				for (Field field : clazz.getDeclaredFields()) {
 					int mod = field.getModifiers();
@@ -110,12 +109,11 @@ public static FieldSummary getFieldSummary(ConversionManager manager, Class<?> c
 						pos++;
 					}
 				}
+				if (!staticContext && postInit == null) {
+					postInit = getPostInitMethod(clazz);
+				}
 				clazz = clazz.getSuperclass();
 			}
-			
-			if (!staticContext) {
-				postInit = getPostInitMethod(clazz);
-			}
 			return new FieldSummary(fields, converters, configPath, configPathConverter, postInit);
 		} catch (NoSuchMethodException e) {
 			e.printStackTrace();