Skip to content

Commit

Permalink
Fixed a NPE when running replacePlayers on islands
Browse files Browse the repository at this point in the history
  • Loading branch information
OmerBenGera committed Jan 28, 2023
1 parent cd02916 commit e2374cb
Showing 1 changed file with 1 addition and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1464,9 +1464,8 @@ public boolean transferIsland(SuperiorPlayer superiorPlayer) {
}

@Override
public void replacePlayers(SuperiorPlayer originalPlayer, @Nullable SuperiorPlayer newPlayer) {
public void replacePlayers(SuperiorPlayer originalPlayer, @Nullable SuperiorPlayer newPlayer) {F

This comment has been minimized.

Copy link
@ArDiDEx

ArDiDEx Jan 29, 2023

there's an F for no reason here that would prevent building it

Preconditions.checkNotNull(originalPlayer, "originalPlayer parameter cannot be null.");
Preconditions.checkNotNull(newPlayer, "newPlayer parameter cannot be null.");
Preconditions.checkState(originalPlayer != newPlayer, "originalPlayer and newPlayer cannot equal.");

Log.debug(Debug.REPLACE_PLAYER, owner, originalPlayer, newPlayer);
Expand Down

0 comments on commit e2374cb

Please sign in to comment.