From 4057cc2859b0e448b72fd6845daa529ec0ff3a0b Mon Sep 17 00:00:00 2001
From: Tyler Moore <mooretc2@miamioh.edu>
Date: Thu, 4 Oct 2018 14:16:52 -0700
Subject: [PATCH] Implementing new function in all classes implementing Player

---
 Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java | 4 ++++
 Mage.Tests/src/test/java/org/mage/test/stub/PlayerStub.java   | 3 +++
 Mage/src/main/java/mage/players/Player.java                   | 2 +-
 Mage/src/main/java/mage/players/PlayerImpl.java               | 3 +--
 4 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java b/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java
index 0fa4a84a094..05837e862f4 100644
--- a/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java
+++ b/Mage.Tests/src/test/java/org/mage/test/player/TestPlayer.java
@@ -2072,6 +2072,10 @@ public class TestPlayer implements Player {
         return computerPlayer.searchLibrary(target, game, targetPlayerId, triggerEvents);
     }
 
+    public void lookAtAllLibraries(Ability source, Game game) {
+        computerPlayer.lookAtAllLibraries(source, game);
+    }
+
     @Override
     public boolean flipCoin(Game game) {
         return computerPlayer.flipCoin(game);
diff --git a/Mage.Tests/src/test/java/org/mage/test/stub/PlayerStub.java b/Mage.Tests/src/test/java/org/mage/test/stub/PlayerStub.java
index 810c068ccf6..360bd942514 100644
--- a/Mage.Tests/src/test/java/org/mage/test/stub/PlayerStub.java
+++ b/Mage.Tests/src/test/java/org/mage/test/stub/PlayerStub.java
@@ -597,6 +597,9 @@ public class PlayerStub implements Player {
         return false;
     }
 
+    @Override
+    public void lookAtAllLibraries(Ability source, Game game) {}
+
     @Override
     public boolean canPlayLand() {
         return false;
diff --git a/Mage/src/main/java/mage/players/Player.java b/Mage/src/main/java/mage/players/Player.java
index f6f30cf08e5..ec86ad550c1 100644
--- a/Mage/src/main/java/mage/players/Player.java
+++ b/Mage/src/main/java/mage/players/Player.java
@@ -357,7 +357,7 @@ public interface Player extends MageItem, Copyable<Player> {
      * @param game
      * @return
      */
-    boolean lookAtAllLibraries(Ability source, Game game);
+    void lookAtAllLibraries(Ability source, Game game);
 
     boolean canPlayLand();
 
diff --git a/Mage/src/main/java/mage/players/PlayerImpl.java b/Mage/src/main/java/mage/players/PlayerImpl.java
index 9b202e4466c..53053fb0466 100644
--- a/Mage/src/main/java/mage/players/PlayerImpl.java
+++ b/Mage/src/main/java/mage/players/PlayerImpl.java
@@ -2478,7 +2478,7 @@ public abstract class PlayerImpl implements Player, Serializable {
     }
 
     @Override
-    public boolean lookAtAllLibraries(Ability source, Game game) {
+    public void lookAtAllLibraries(Ability source, Game game) {
         for(UUID playerId : game.getState().getPlayersInRange(this.getId(), game)){
             Player player = game.getPlayer(playerId);
             String playerName = this.getName().equals(player.getName()) ? "Your " : player.getName() + "'s ";
@@ -2486,7 +2486,6 @@ public abstract class PlayerImpl implements Player, Serializable {
             Cards cardsInLibrary = new CardsImpl(player.getLibrary().getTopCards(game, player.getLibrary().size()));
             lookAtCards(playerName, cardsInLibrary, game);
         }
-        return true;
     }
 
     private boolean handleLibraryCastableCards(Library library, Game game, UUID targetPlayerId) {