fix npe

closedcontour · Oct 19, 2018 · f14c982 · f14c982
1 parent 1266dd2
commit f14c982
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/projects/app/src/pages/components/PlayerChooser.tsx b/projects/app/src/pages/components/PlayerChooser.tsx
@@ -4,7 +4,7 @@ import * as React from "react";
 export interface PlayerChooserProps {
   players: IPlayer[];
   selectedPlayer: IPlayer | undefined | null;
-  onPlayerChanged(player: IPlayer | undefined): void;
+  onPlayerChanged(player: IPlayer | null): void;
 }
 
 export class PlayerChooser extends React.PureComponent<PlayerChooserProps, {}> {
@@ -27,6 +27,6 @@ export class PlayerChooser extends React.PureComponent<PlayerChooserProps, {}> {
   private handlePlayerChooserChange = (event: React.ChangeEvent<HTMLSelectElement>) => {
     const playerId = event.target.value;
     const player = this.props.players.find(p => p.id.toString() === playerId);
-    this.props.onPlayerChanged(playerId === "" ? undefined : player);
+    this.props.onPlayerChanged(playerId === "" || !player ? null : player);
   };
 }