Use Preconditions.checkNotNull for null checks across DataFixerRegistry and AetherDataFixersProperties to enforce consistent validation.

Author: Splatcrafter

aether-datafixers-spring-boot-starter/src/main/java/de/splatgames/aether/datafixers/spring/AetherDataFixersProperties.java

@@ -22,6 +22,7 @@
 
 package de.splatgames.aether.datafixers.spring;
 
+import com.google.common.base.Preconditions;
 import de.splatgames.aether.datafixers.spring.config.DataFixerDomainProperties;
 import de.splatgames.aether.datafixers.spring.config.DynamicOpsFormat;
 import org.jetbrains.annotations.NotNull;
@@ -197,7 +198,7 @@ public DynamicOpsFormat getDefaultFormat() {
      * @throws NullPointerException if defaultFormat is {@code null}
      */
     public void setDefaultFormat(@NotNull final DynamicOpsFormat defaultFormat) {
-        this.defaultFormat = defaultFormat;
+        this.defaultFormat = Preconditions.checkNotNull(defaultFormat, "defaultFormat must not be null");
     }
 
     /**
@@ -246,7 +247,7 @@ public Map<String, DataFixerDomainProperties> getDomains() {
      * @throws NullPointerException if domains is {@code null}
      */
     public void setDomains(@NotNull final Map<String, DataFixerDomainProperties> domains) {
-        this.domains = domains;
+        this.domains = Preconditions.checkNotNull(domains, "domains must not be null");
     }
 
     /**
@@ -269,7 +270,7 @@ public ActuatorProperties getActuator() {
      * @throws NullPointerException if actuator is {@code null}
      */
     public void setActuator(@NotNull final ActuatorProperties actuator) {
-        this.actuator = actuator;
+        this.actuator = Preconditions.checkNotNull(actuator, "actuator must not be null");
     }
 
     /**
@@ -292,7 +293,7 @@ public MetricsProperties getMetrics() {
      * @throws NullPointerException if metrics is {@code null}
      */
     public void setMetrics(@NotNull final MetricsProperties metrics) {
-        this.metrics = metrics;
+        this.metrics = Preconditions.checkNotNull(metrics, "metrics must not be null");
     }
 
     /**

aether-datafixers-spring-boot-starter/src/main/java/de/splatgames/aether/datafixers/spring/autoconfigure/DataFixerRegistry.java

@@ -22,6 +22,7 @@
 
 package de.splatgames.aether.datafixers.spring.autoconfigure;
 
+import com.google.common.base.Preconditions;
 import de.splatgames.aether.datafixers.core.AetherDataFixer;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
@@ -148,6 +149,8 @@ public class DataFixerRegistry {
      * @throws NullPointerException     if domain or fixer is {@code null}
      */
     public void register(@NotNull final String domain, @NotNull final AetherDataFixer fixer) {
+        Preconditions.checkNotNull(domain, "domain must not be null");
+        Preconditions.checkNotNull(fixer, "fixer must not be null");
         if (this.fixers.containsKey(domain)) {
             throw new IllegalArgumentException(
                     "DataFixer already registered for domain: " + domain
@@ -170,6 +173,7 @@ public void register(@NotNull final String domain, @NotNull final AetherDataFixe
      */
     @Nullable
     public AetherDataFixer get(@NotNull final String domain) {
+        Preconditions.checkNotNull(domain, "domain must not be null");
         return this.fixers.get(domain);
     }
 
@@ -191,6 +195,7 @@ public AetherDataFixer get(@NotNull final String domain) {
      */
     @NotNull
     public AetherDataFixer require(@NotNull final String domain) {
+        Preconditions.checkNotNull(domain, "domain must not be null");
         final AetherDataFixer fixer = this.fixers.get(domain);
         if (fixer == null) {
             throw new IllegalArgumentException(
@@ -254,6 +259,7 @@ public Set<String> getDomains() {
      * @throws NullPointerException if domain is {@code null}
      */
     public boolean contains(@NotNull final String domain) {
+        Preconditions.checkNotNull(domain, "domain must not be null");
         return this.fixers.containsKey(domain);
     }