fix player keys in BasketCourt

front/components/editor/BasketCourt.tsx

@@ -23,7 +23,7 @@ export function BasketCourt({
             {players.map((player) => {
                 return (
                     <CourtPlayer
-                        key={player.id}
+                        key={player.team + player.role}
                         player={player}
                         onChange={onPlayerChange}
                         onRemove={() => onPlayerRemove(player)}

front/components/editor/CourtPlayer.tsx

@@ -43,7 +43,6 @@ export default function CourtPlayer({
                 setY(y)
 
                 onChange({
-                    id: player.id,
                     rightRatio: x,
                     bottomRatio: y,
                     team: player.team,

front/tactic/Player.ts

@@ -1,12 +1,6 @@
 import { Team } from "./Team"
 
 export interface Player {
-    /**
-     * unique identifier of the player.
-     * This identifier must be unique to the associated court.
-     */
-    id: number
-
     /**
      * the player's team
      * */

front/views/Editor.tsx

@@ -128,7 +128,6 @@ function EditorView({
             return [
                 ...players,
                 {
-                    id: players.length,
                     team: element.team,
                     role: element.key,
                     rightRatio: x,
@@ -151,11 +150,7 @@ function EditorView({
                         default_value={name}
                         on_validated={(new_name) => {
                             onNameChange(new_name).then((success) => {
-                                if (success) {
+                                success ? setStyle({}) : setStyle(ERROR_STYLE)
-                                    setStyle({})
-                                } else {
-                                    setStyle(ERROR_STYLE)
-                                }
                             })
                         }}
                     />
@@ -192,7 +187,9 @@ function EditorView({
                             onPlayerChange={(player) => {
                                 setPlayers((players) => {
                                     const idx = players.findIndex(
-                                        (p) => p.id === player.id,
+                                        (p) =>
+                                            p.team === player.team &&
+                                            p.role === player.role,
                                     )
                                     return players.toSpliced(idx, 1, player)
                                 })
@@ -200,7 +197,9 @@ function EditorView({
                             onPlayerRemove={(player) => {
                                 setPlayers((players) => {
                                     const idx = players.findIndex(
-                                        (p) => p.id === player.id,
+                                        (p) =>
+                                            p.team === player.team &&
+                                            p.role === player.role,
                                     )
                                     return players.toSpliced(idx, 1)
                                 })