external-repos/forge
1
0
Fork 0
mirror of https://github.com/Card-Forge/forge.git synced 2025-11-19 04:08:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch 'choosenum' into 'master'

Browse Source 
fix ChooseNumberEffect range for random

See merge request core-developers/forge!5253
This commit is contained in:
Michael Kamensky
2021-08-23 16:06:24 +00:00
parent 0e1033a075 1a2229afe6
commit f052372d22
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
forge-game/src/main/java/forge/game/ability/effects/ChooseNumberEffect.java
View File

@@ -54,7 +54,8 @@ public class ChooseNumberEffect extends SpellAbilityEffect {
if ((tgt == null) || p.canBeTargetedBy(sa)) { if ((tgt == null) || p.canBeTargetedBy(sa)) {
int chosen; int chosen;
if (random) { 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); p.getGame().getAction().notifyOfValue(sa, p, Integer.toString(chosen), null);
} else { } else {
String title = sa.hasParam("ListTitle") ? sa.getParam("ListTitle") : Localizer.getInstance().getMessage("lblChooseNumber"); String title = sa.hasParam("ListTitle") ? sa.getParam("ListTitle") : Localizer.getInstance().getMessage("lblChooseNumber");