From 9bcfced7d245063b4291f7ab4313111879a30a03 Mon Sep 17 00:00:00 2001
From: Agetian <Agetian@269b9781-a132-4a9b-9d4e-f004f1b56b58>
Date: Wed, 27 Sep 2017 17:02:23 +0000
Subject: [PATCH] - Fixed a NPE in DiscardEffect.

git-svn-id: http://svn.slightlymagic.net/forge/trunk@35801 269b9781-a132-4a9b-9d4e-f004f1b56b58
---
 .../java/forge/game/ability/effects/DiscardEffect.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/forge-game/src/main/java/forge/game/ability/effects/DiscardEffect.java b/forge-game/src/main/java/forge/game/ability/effects/DiscardEffect.java
index 9346f58e..afb31549 100644
--- a/forge-game/src/main/java/forge/game/ability/effects/DiscardEffect.java
+++ b/forge-game/src/main/java/forge/game/ability/effects/DiscardEffect.java
@@ -293,11 +293,11 @@ public class DiscardEffect extends SpellAbilityEffect {
 
                     CardCollectionView toBeDiscarded = validCards.isEmpty() ? null : chooser.getController().chooseCardsToDiscardFrom(p, sa, validCards, min, max);
 
-                    if (toBeDiscarded.size() > 1) {
-                        toBeDiscarded = GameActionUtil.orderCardsByTheirOwners(p.getGame(), toBeDiscarded, ZoneType.Graveyard);
-                    }
-
                     if (toBeDiscarded != null) {
+                        if (toBeDiscarded.size() > 1) {
+                            toBeDiscarded = GameActionUtil.orderCardsByTheirOwners(p.getGame(), toBeDiscarded, ZoneType.Graveyard);
+                        }
+
                         if (mode.startsWith("Reveal") ) {
                             p.getController().reveal(toBeDiscarded, ZoneType.Hand, p,
                                     chooser + " has chosen " + (toBeDiscarded.size() == 1 ? "this card" : "these cards")  + " from ");