aboutsummaryrefslogtreecommitdiff
path: root/web/pw-server/src
diff options
context:
space:
mode:
authorIlion Beyst <ilion.beyst@gmail.com>2022-10-16 12:42:20 +0200
committerIlion Beyst <ilion.beyst@gmail.com>2022-10-16 12:42:20 +0200
commit03cdbbc003b102156194568a7b8fc52d636f87f4 (patch)
tree3a6bcb4532e26f7be324eaaeeb1cc7ac35d6d25e /web/pw-server/src
parent79479ba8babfc40a0d0731fec41b2e8426fbe7f2 (diff)
downloadplanetwars.dev-03cdbbc003b102156194568a7b8fc52d636f87f4.tar.xz
planetwars.dev-03cdbbc003b102156194568a7b8fc52d636f87f4.zip
only show playerlog to bot owner
Diffstat (limited to 'web/pw-server/src')
-rw-r--r--web/pw-server/src/routes/matches/[match_id].svelte69
1 files changed, 44 insertions, 25 deletions
diff --git a/web/pw-server/src/routes/matches/[match_id].svelte b/web/pw-server/src/routes/matches/[match_id].svelte
index b3d9f01..1d18375 100644
--- a/web/pw-server/src/routes/matches/[match_id].svelte
+++ b/web/pw-server/src/routes/matches/[match_id].svelte
@@ -35,46 +35,61 @@
matchLog = await apiClient.getText(`/api/matches/${matchData["id"]}/log`);
});
- let selectedPlayer;
-
- $: matchPlayerSelectItems = matchData["players"].map((player: any, index: number) => ({
- color: PLAYER_COLORS[index],
- value: index,
- playerId: index + 1, // stoopid player number + 1
- label: player["bot_name"],
- }));
+ $: playersWithVisibleLog = matchData["players"]
+ .map((player: any, index: number) => ({
+ color: PLAYER_COLORS[index],
+ value: index,
+ playerId: index + 1, // stoopid player number + 1
+ displayName: player["bot_name"] || "player",
+ matchPlayer: player,
+ }))
+ .filter((item) => canSeePlayerLog($currentUser, item.matchPlayer));
// TODO: refactor match logs so that users can no longer get match logs for other players.
- function currentUserCanSeeStdErr(matchPlayer: object): boolean {
+ function canSeePlayerLog(user: object | null, matchPlayer: object): boolean {
if (!matchPlayer["owner_id"]) {
return true;
}
- console.log(matchPlayer, $currentUser);
- return matchPlayer["owner_id"] === $currentUser?.["user_id"];
+ return matchPlayer["owner_id"] === user?.["user_id"];
}
- $: showStdErr =
- !!selectedPlayer && currentUserCanSeeStdErr(matchData["players"][selectedPlayer["value"]]);
+ // using the same value here causes svelte to freeze
+ let dropdownSelectedPlayer: any;
+ let selectedPlayer: any;
+ $: if (playersWithVisibleLog.length == 1) {
+ selectedPlayer = playersWithVisibleLog[0];
+ } else {
+ selectedPlayer = dropdownSelectedPlayer;
+ }
</script>
<div class="container">
<Visualizer {matchLog} {matchData} />
<div class="output-pane">
<div class="player-select">
- <Select
- items={matchPlayerSelectItems}
- clearable={false}
- searchable={false}
- bind:value={selectedPlayer}
- >
- <div slot="item" let:item>
- <span style:color={item.color}>{item.label}</span>
- </div>
- </Select>
+ {#if playersWithVisibleLog.length == 1}
+ <h3 class="player-log-header">
+ player log for
+ <span style:color={selectedPlayer["color"]}>{selectedPlayer.displayName}</span>
+ </h3>
+ {:else}
+ <Select
+ items={playersWithVisibleLog}
+ label="displayName"
+ clearable={false}
+ searchable={false}
+ bind:value={dropdownSelectedPlayer}
+ placeholder="Select player to see logs"
+ >
+ <div slot="item" let:item>
+ <span style:color={item.color}>{item.displayName}</span>
+ </div>
+ </Select>
+ {/if}
</div>
<div class="player-log">
- <PlayerLog {matchLog} playerId={selectedPlayer?.playerId} {showStdErr} />
+ <PlayerLog {matchLog} playerId={selectedPlayer?.["playerId"]} />
</div>
</div>
</div>
@@ -90,7 +105,11 @@
}
.player-select {
- padding: 20px;
+ padding: 0 20px;
+ }
+
+ .player-log-header {
+ color: #eee;
}
.player-log {