From 1a2229afe6b0bb0b5c61f14347ec06be86aeac9a Mon Sep 17 00:00:00 2001 From: Northmoc Date: Mon, 23 Aug 2021 08:04:18 -0400 Subject: [PATCH] fix ChooseNumberEffect range for random --- .../java/forge/game/ability/effects/ChooseNumberEffect.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/forge-game/src/main/java/forge/game/ability/effects/ChooseNumberEffect.java b/forge-game/src/main/java/forge/game/ability/effects/ChooseNumberEffect.java index ae677e85f7c..63e99413a36 100644 --- a/forge-game/src/main/java/forge/game/ability/effects/ChooseNumberEffect.java +++ b/forge-game/src/main/java/forge/game/ability/effects/ChooseNumberEffect.java @@ -54,7 +54,8 @@ public class ChooseNumberEffect extends SpellAbilityEffect { if ((tgt == null) || p.canBeTargetedBy(sa)) { int chosen; if (random) { - chosen = MyRandom.getRandom().nextInt(max - min) + min; + chosen = MyRandom.getRandom().nextInt((max - min) + 1) + min; + //TODO more useful notify for RepeatEach -> ChooseNumber with random p.getGame().getAction().notifyOfValue(sa, p, Integer.toString(chosen), null); } else { String title = sa.hasParam("ListTitle") ? sa.getParam("ListTitle") : Localizer.getInstance().getMessage("lblChooseNumber");