Only update preview state once preview is rendering

Author: jojoe77777

src/main/java/me/voidxwalker/worldpreview/WorldPreview.java

@@ -22,6 +22,7 @@ public class WorldPreview  implements ClientModInitializer {
    public static ClientPlayerEntity player;
    public static ClientWorld clientWord;
    public static boolean inPreview;
+   public static boolean renderingPreview;
    public static BlockPos spawnPos;
    public static int kill=0;
    public static int playerSpawn;

src/main/java/me/voidxwalker/worldpreview/mixin/client/MinecraftClientMixin.java

@@ -144,4 +144,14 @@ private void worldpreview_outputInWorldState(CallbackInfo info) {
             StateOutputHelper.outputState("inworld,gamescreenopen");
         }
     }
+
+    @Inject(method = "render", at = @At(value = "INVOKE", target="Lnet/minecraft/client/util/Window;swapBuffers()V", shift = At.Shift.AFTER))
+    private void worldpreview_actuallyInPreview(boolean tick, CallbackInfo ci) {
+        if (WorldPreview.inPreview && !WorldPreview.renderingPreview) {
+            WorldPreview.renderingPreview = true;
+            StateOutputHelper.outputState("previewing," + StateOutputHelper.loadingProgress);
+            WorldPreview.log(Level.INFO, "Starting Preview at (" + WorldPreview.player.getX() + ", " + (double) Math.floor(WorldPreview.player.getY()) + ", " + WorldPreview.player.getZ() + ")");
+        }
+    }
+
 }

src/main/java/me/voidxwalker/worldpreview/mixin/client/WorldGenerationProgressLoggerMixin.java

@@ -18,7 +18,7 @@ public abstract class WorldGenerationProgressLoggerMixin {
     private void worldpreview_outputGenerationState(ChunkPos pos, ChunkStatus status, CallbackInfo info) {
         // Using the getProgressPercentage to recalculate is slightly unoptimized but prevents needing to do locals capture, making it easier to port this mixin.
         StateOutputHelper.loadingProgress = MathHelper.clamp(getProgressPercentage(), 0, 100);
-        StateOutputHelper.outputState((WorldPreview.inPreview ? "previewing," : "generating,") + StateOutputHelper.loadingProgress);
+        StateOutputHelper.outputState((WorldPreview.renderingPreview ? "previewing," : "generating,") + StateOutputHelper.loadingProgress);
     }
 
     @Shadow

src/main/java/me/voidxwalker/worldpreview/mixin/client/render/LevelLoadingScreenMixin.java

@@ -89,8 +89,6 @@ public void worldpreview_render(MatrixStack matrices, int mouseX, int mouseY, fl
                     WorldPreview.camera.update(WorldPreview.world, WorldPreview.player, this.client.options.perspective > 0, this.client.options.perspective == 2, 0.2F);
                     WorldPreview.player.refreshPositionAndAngles(WorldPreview.player.getX(), WorldPreview.player.getY() - 1.5, WorldPreview.player.getZ(), 0.0F, 0.0F);
                     WorldPreview.inPreview=true;
-                    StateOutputHelper.outputState("previewing," + StateOutputHelper.loadingProgress);
-                    WorldPreview.log(Level.INFO,"Starting Preview at ("+ WorldPreview.player.getX() + ", "+(double)Math.floor(WorldPreview.player.getY())+ ", "+ WorldPreview.player.getZ()+")");
                 }
                 MatrixStack matrixStack = new MatrixStack();
                 matrixStack.peek().getModel().multiply(this.worldpreview_getBasicProjectionMatrix());

src/main/java/me/voidxwalker/worldpreview/mixin/server/MinecraftServerMixin.java

@@ -137,6 +137,7 @@ public void worldpreview_kill(CallbackInfo ci){
     @Inject(method="runServer",at=@At(value="INVOKE",target="Lnet/minecraft/server/MinecraftServer;setupServer()Z",shift = At.Shift.AFTER), cancellable = true)
     public void worldpreview_kill2(CallbackInfo ci){
         WorldPreview.inPreview=false;
+        WorldPreview.renderingPreview=false;
         LockSupport.unpark(((MinecraftClientMixin)MinecraftClient.getInstance()).invokeGetThread());
         if(WorldPreview.kill==1){
             ci.cancel();