only access WorldPreset in client distribution

PssbleTrngle · Aug 19, 2022 · 612dfa5 · 612dfa5
1 parent 619ee7f
commit 612dfa5
Show file tree

Hide file tree

Showing 8 changed files with 24 additions and 10 deletions.
diff --git a/.github/workflows/sonar.yml b/.github/workflows/sonar.yml
@@ -35,6 +35,13 @@ jobs:
           key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle') }}
           restore-keys: ${{ runner.os }}-gradle
 
+      - name: Update mod version
+        uses: christian-draeger/[email protected]
+        with:
+          path: 'gradle.properties'
+          property: 'mod_version'
+          value: ${{ github.sha }}
+
       - name: Build and analyze
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

diff --git a/Common/src/main/java/possible_triangle/skygrid/mixin/MinecraftServerMixin.java b/Common/src/main/java/possible_triangle/skygrid/mixin/MinecraftServerMixin.java
@@ -16,7 +16,7 @@ public class MinecraftServerMixin {
     @Inject(at = @At("RETURN"), method = "reloadResources(Ljava/util/Collection;)Ljava/util/concurrent/CompletableFuture;", cancellable = true)
     public void onReload(Collection<String> packs, CallbackInfoReturnable<CompletableFuture<Void>> callback) {
         var self = (MinecraftServer) (Object) this;
-        callback.getReturnValue().thenAccept($ -> XMLResource.Companion.reload(self));
+        callback.setReturnValue(callback.getReturnValue().thenAccept($ -> XMLResource.Companion.reload(self)));
     }
 
 }
diff --git a/...angle/skygrid/mixin/WorldPresetMixin.java → ...le/skygrid/mixin/WorldPresetAccessor.java b/...angle/skygrid/mixin/WorldPresetMixin.java → ...le/skygrid/mixin/WorldPresetAccessor.java
@@ -7,7 +7,7 @@
 import java.util.List;
 
 @Mixin(WorldPreset.class)
-public interface WorldPresetMixin {
+public interface WorldPresetAccessor {
 
     @Accessor("PRESETS")
     static List<WorldPreset> presets() {

diff --git a/Fabric/src/main/kotlin/possible_triangle/skygrid/SkygridFabric.kt b/Fabric/src/main/kotlin/possible_triangle/skygrid/SkygridFabric.kt
@@ -3,15 +3,14 @@ package possible_triangle.skygrid
 import kotlinx.serialization.ExperimentalSerializationApi
 import net.fabricmc.api.ClientModInitializer
 import net.fabricmc.api.ModInitializer
-import net.fabricmc.fabric.api.client.item.v1.ItemTooltipCallback
 import net.fabricmc.fabric.api.command.v1.CommandRegistrationCallback
 import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents
 import net.fabricmc.fabric.api.resource.ResourceManagerHelper
 import net.minecraft.server.packs.PackType
 import nl.adaptivity.xmlutil.ExperimentalXmlUtilApi
 import possible_triangle.skygrid.command.SkygridCommand
 import possible_triangle.skygrid.data.XMLResource
-import possible_triangle.skygrid.mixin.WorldPresetMixin
+import possible_triangle.skygrid.mixin.WorldPresetAccessor
 import possible_triangle.skygrid.platform.FabricPlatformHelper
 import possible_triangle.skygrid.world.SkygridGenerator
 
@@ -21,10 +20,12 @@ import possible_triangle.skygrid.world.SkygridGenerator
 object SkygridFabric : ModInitializer, ClientModInitializer {
 
     override fun onInitialize() {
+        SkygridMod.LOGGER.info("Common Initialize")
         SkygridMod.init()
         SkygridMod.setup()
         FabricPlatformHelper.register()
 
+
         XMLResource.register {
             val listener = FabricReloadListener(it, it.path)
             ResourceManagerHelper.get(PackType.SERVER_DATA).registerReloadListener(listener)
@@ -34,11 +35,11 @@ object SkygridFabric : ModInitializer, ClientModInitializer {
         ServerLifecycleEvents.SERVER_STOPPING.register { XMLResource.clear() }
 
         CommandRegistrationCallback.EVENT.register { it, _ -> SkygridCommand.register(it) }
-
     }
 
     override fun onInitializeClient() {
-        WorldPresetMixin.presets().add(SkygridGenerator)
+        SkygridMod.LOGGER.info("Client Initialize")
+        WorldPresetAccessor.presets().add(SkygridGenerator)
     }
 
 }
diff --git a/Fabric/src/main/resources/fabric.mod.json b/Fabric/src/main/resources/fabric.mod.json
@@ -20,12 +20,18 @@
         "adapter": "kotlin",
         "value": "possible_triangle.skygrid.SkygridFabric"
       }
+    ],
+    "client": [
+      {
+        "adapter": "kotlin",
+        "value": "possible_triangle.skygrid.SkygridFabric"
+      }
     ]
   },
   "mixins": [
     "skygrid.fabric.mixins.json"
   ],
-  "accessWidener" : "skygrid.accesswidener",
+  "accessWidener": "skygrid.accesswidener",
   "depends": {
     "fabricloader": ">=0.12",
     "fabric": "*",

diff --git a/Fabric/src/main/resources/skygrid.fabric.mixins.json b/Fabric/src/main/resources/skygrid.fabric.mixins.json
@@ -10,7 +10,7 @@
     "WorldGenSettingsMixin"
   ],
   "client": [
-    "WorldPresetMixin"
+    "WorldPresetAccessor"
   ],
   "injectors": {
     "defaultRequire": 1

diff --git a/build.gradle b/build.gradle
@@ -37,7 +37,7 @@ subprojects {
                     'Timestampe'              : System.currentTimeMillis(),
                     'Built-On-Java'           : "${System.getProperty('java.vm.version')} (${System.getProperty('java.vm.vendor')})",
                     'Build-On-Minecraft'      : minecraft_version,
-                    //"MixinConfigs": "${mod_id}.${project.name.toLowerCase()}.mixins.json"
+                    "MixinConfigs": "${mod_id}.${project.name.toLowerCase()}.mixins.json"
             ])
         }
     }

diff --git a/gradle.properties b/gradle.properties
@@ -1,5 +1,5 @@
 # Project
-mod_version=3.0.7
+mod_version=0.0.0-dev
 release_type=release
 group=com.possible_triangle