From 8078d2d7e3363679874caaa571a1c138438c95e2 Mon Sep 17 00:00:00 2001
From: Daniel Karbach <daniel.karbach@localhorst.tv>
Date: Tue, 15 Mar 2022 19:30:18 +0100
Subject: [PATCH] only show DNFs for finished players

---
 resources/js/components/results/Item.js | 13 +++++++------
 1 file changed, 7 insertions(+), 6 deletions(-)

diff --git a/resources/js/components/results/Item.js b/resources/js/components/results/Item.js
index 0d0d34e..2a1fc8c 100644
--- a/resources/js/components/results/Item.js
+++ b/resources/js/components/results/Item.js
@@ -9,11 +9,11 @@ import { findResult } from '../../helpers/Participant';
 import { maySeeResults } from '../../helpers/permissions';
 import { withUser } from '../../helpers/UserContext';
 
-const getIcon = (result, index) => {
+const getIcon = (result, index, maySee) => {
 	if (!result || !result.has_finished) {
 		return <Icon.PENDING className="text-muted" size="lg" />;
 	}
-	if (result.forfeit) {
+	if (result.forfeit && maySee) {
 		return <Icon.FORFEIT className="text-danger" size="lg" />;
 	}
 	if (index === 0) {
@@ -28,8 +28,8 @@ const getIcon = (result, index) => {
 	return <Icon.FINISHED className="text-success" size="lg" />;
 };
 
-const getTime = (user, tournament, round, result) => {
-	if (!result || !maySeeResults(user, tournament, round)) {
+const getTime = (result, maySee) => {
+	if (!result || !maySee) {
 		return null;
 	}
 	if (result.time) {
@@ -49,13 +49,14 @@ const Item = ({
 	user,
 }) => {
 	const result = findResult(participant, round);
+	const maySee = maySeeResults(user, tournament, round);
 	return <div className="result">
 		<Box user={participant.user} />
 		<div className="status">
 			<span className="time">
-				{getTime(user, tournament, round, result)}
+				{getTime(result, maySee)}
 			</span>
-			{getIcon(result, index)}
+			{getIcon(result, index, maySee)}
 		</div>
 	</div>;
 };
-- 
2.39.2