Merge pull request #34 from ZorTik/development

Code reformat

Author: ZorTik

core/src/main/java/me/zort/sqllib/SQLDatabaseConnectionImpl.java

@@ -88,6 +88,7 @@ public class SQLDatabaseConnectionImpl extends PooledSQLDatabaseConnection {
     private transient Logger logger;
     @Getter(onMethod_ = {@Nullable, @ApiStatus.Experimental})
     private transient Transaction transaction;
+    private transient SchemaSynchronizer<SQLDatabaseConnection> schemaSynchronizer;
     private int errorCount = 0;
 
     /**
@@ -114,8 +115,9 @@ public SQLDatabaseConnectionImpl(final @NotNull SQLConnectionFactory connectionF
         this.errorStateHandlers = new CopyOnWriteArrayList<>();
         this.transaction = null;
         this.logger = Logger.getGlobal();
-        this.mappingRegistry = new MappingRegistryImpl(this, new SQLSchemaSynchronizer());
+        this.mappingRegistry = new MappingRegistryImpl(this);
 
+        setSchemaSynchronizer(new SQLSchemaSynchronizer());
         enableCaching(CacheManager.noCache());
 
         // Default backup value resolvers.
@@ -205,7 +207,8 @@ public boolean synchronizeModel() {
     @ApiStatus.Experimental
     @Override
     public boolean synchronizeModel(TableSchema entitySchema, String table) {
-        return getSchemaSynchronizer().synchronize(this, entitySchema, getSchemaBuilder(table).buildTableSchema()).isSuccessful();
+        return getSchemaSynchronizer().synchronize(this, entitySchema,
+                getSchemaBuilder(table).buildTableSchema()).isSuccessful();
     }
 
     /**
@@ -221,7 +224,9 @@ public boolean synchronizeModel(TableSchema entitySchema, String table) {
     @ApiStatus.Experimental
     @Override
     public boolean synchronizeModel(Class<?> entity, String table) {
-        return synchronizeModel(new EntitySchemaBuilder(table, entity, getOptions().getNamingStrategy(), this instanceof SQLiteDatabaseConnectionImpl).buildTableSchema(), table);
+        return synchronizeModel(new EntitySchemaBuilder(table, entity,
+                getOptions().getNamingStrategy(),
+                this instanceof SQLiteDatabaseConnectionImpl).buildTableSchema(), table);
     }
 
     /**
@@ -232,13 +237,13 @@ public boolean synchronizeModel(Class<?> entity, String table) {
     @ApiStatus.Experimental
     @Override
     public void setSchemaSynchronizer(SchemaSynchronizer<SQLDatabaseConnection> synchronizer) {
-        mappingRegistry.setSynchronizer(synchronizer);
+        this.schemaSynchronizer = synchronizer;
     }
 
     @ApiStatus.Experimental
     @Override
     public SchemaSynchronizer<SQLDatabaseConnection> getSchemaSynchronizer() {
-        return mappingRegistry.getSynchronizer();
+        return schemaSynchronizer;
     }
 
     @ApiStatus.Experimental

core/src/main/java/me/zort/sqllib/mapping/MappingRegistryImpl.java

@@ -1,31 +1,20 @@
 package me.zort.sqllib.mapping;
 
-import lombok.Getter;
-import lombok.Setter;
-import me.zort.sqllib.SQLDatabaseConnection;
+import lombok.RequiredArgsConstructor;
 import me.zort.sqllib.SQLDatabaseConnectionImpl;
-import me.zort.sqllib.SQLiteDatabaseConnectionImpl;
 import me.zort.sqllib.api.mapping.MappingProxyInstance;
 import me.zort.sqllib.api.mapping.StatementMappingRegistry;
-import me.zort.sqllib.api.model.SchemaSynchronizer;
 
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
 import java.util.concurrent.ConcurrentHashMap;
 
+@RequiredArgsConstructor
 public class MappingRegistryImpl implements StatementMappingRegistry {
 
     private final Map<Class<?>, MappingProxyInstance<?>> proxyWrappers = new ConcurrentHashMap<>();
     private final SQLDatabaseConnectionImpl connection;
-    @Setter
-    @Getter
-    private SchemaSynchronizer<SQLDatabaseConnection> synchronizer;
-
-    public MappingRegistryImpl(SQLDatabaseConnectionImpl connection, SchemaSynchronizer<SQLDatabaseConnection> synchronizer) {
-        this.connection = connection;
-        this.synchronizer = synchronizer;
-    }
 
     @Override
     public void registerProxy(MappingProxyInstance<?> proxyInstance) {