magge-faf / faf-moderator-client-develop-modified

magge's modified Mordor makes moderating mischievous members much more majestic.
MIT License
0 stars 2 forks source link

Add important game-related information to the top of the Moderator Events log #38

Closed IndexLibrorumProhibitorum closed 4 months ago

IndexLibrorumProhibitorum commented 5 months ago

Information such as whether a game is ranked, whether a game is on full-share, partial-share, or no-share, and whether cheating was turned on is rather useful to handle reports.

As it is now, while I can confirm that a player Ctrl-K'd their units, I will have to check through the FAF client to see if this happened in a full-share game, since doing this in a no-share game is not technically against the rules. This information is present in the replay information, so it shouldn't be too much effort to add this in.

magge-faf commented 4 months ago

@IndexLibrorumProhibitorum Anything that I should change/add/remove that is important for the moderation process?

If cheats were enabled, it will show it in the first line in the chat log with an !-indicator, otherwise I ignore it to avoid text noise:

        if (Boolean.parseBoolean(cheatsEnabled)) {
            report.append("[!] Cheats Enabled: ").append(cheatsEnabled).append("\n");
        }

same for non-default values for Common Army (If you remember our union-control-thing recently ...)

image

image

image

IndexLibrorumProhibitorum commented 4 months ago

Looks good and complete! I like the [!] indicator. All 3 screenshots show team 2 and 3 rather than team 1 and 2, is that intentional?

magge-faf commented 4 months ago

Thanks - Yes, that is from the replay-data itself. TMK Team 1 are spectators, and Team 0 are civilians (or vice versa?). That is just how it is treated within the replay.

Edit: I will release a version with the recent changes. Git should auto-close the other issues as well, when it gets released.