Merge pull request #25651 from ZehMatt/reduce-logging

Fix #25229: Excessive game action logging

distribution/changelog.txt

@@ -13,6 +13,7 @@
 - Fix: [#9895] Stand-up coaster gets wrong intensity boost from the synchronisation bonus.
 - Fix: [#22484] Lingering ghost entrance after placing park entrance.
 - Fix: [#24952] Duplicate track designs when running via Steam without having RCT1 linked.
+- Fix: [#25229] Excessive logging of game actions, reduced to top-level game actions and filters ghost related ones.
 - Fix: [#25524] The track construction arrow does not immediately change position when deleting track pieces.
 - Fix: [#25552] Clear Scenery does not show an error message about insufficient money if cash is negative.
 - Fix: [#25565] Chairlift station ends are missing tunnels at certain rotations.

src/openrct2/actions/GameAction.cpp

@@ -350,7 +350,10 @@ namespace OpenRCT2::GameActions
             }
 
             ActionLogContext logContext;
-            LogActionBegin(logContext, action);
+            if (topLevel && !flags.has(CommandFlag::ghost))
+            {
+                LogActionBegin(logContext, action);
+            }
 
             // Execute the action, changing the game state
             result = action->Execute(gameState);
@@ -362,8 +365,10 @@ namespace OpenRCT2::GameActions
                 // Script hooks may now have changed the game action result...
             }
 #endif
-
-            LogActionFinish(logContext, action, result);
+            if (topLevel && !flags.has(CommandFlag::ghost))
+            {
+                LogActionFinish(logContext, action, result);
+            }
 
             // If not top level just give away the result.
             if (!topLevel)