fix(war): pass the war accept event its guilds in the right order

[darbyjack]

Summary

GuildWarAcceptEvent takes its two guilds challenger-then-defender and passes the defender to GuildEvent as event.guild. CommandWar.accept() passed them the other way round.

 fun accept(player: Player, guild: Guild) {
   val challenge = challengeHandler.getChallenge(guild) ?: ...
   val challenger = challenge.challenger
+  // challenge.defender == guild, so guild is the defender
-  val event = GuildWarAcceptEvent(player, guild, challenger)
+  val event = GuildWarAcceptEvent(player, challenger, guild)

What a listener saw before and after:

property	before	after
event.challenger	defender	challenger
event.defender	challenger	defender
event.guild	challenger	defender

Evidence

accept() refuses unless challenge.defender == guild, so the command's guild is always the defender. Position 2 is challenger and position 3 is defender, so the old call handed each guild the other's slot.

The two sibling call sites already pass challenger first and are untouched:

CommandWar.challenge  GuildWarChallengeEvent(player, guild, targetGuild)   # guild is the challenger
CommandWar.deny       GuildWarDeclineEvent(player, challenger, guild)       # correct
CommandWar.accept     GuildWarAcceptEvent(player, guild, challenger)       # swapped

./gradlew test testJava11 passes.

Testing

No test was added. Nothing under src/test referenced GuildWarAcceptEvent and there were no war command tests to extend. Covering this would need a new pattern for driving a BaseCommand subclass: the handler reaches for Bukkit.getPluginManager(), and the code after the event fires needs ACF's currentCommandManager, which does not exist without a server. The existing suite tests handlers and domain objects directly rather than commands, so this would be new scaffolding, which is out of scope here.

Merge Danger

Door: two-way

Reverts with a single commit.

Blast Radius: plugin API consumers

GuildWarAcceptEvent's constructor signature is unchanged, so nothing breaks at compile time. Listeners reading challenger, defender or guild get the correct guild going forward. Listeners that worked around the swap will need updating.