Failure/foul-magics
1
0
Fork 0
forked from External/mage
Code Pull requests Releases Packages Activity Actions

fix #11456 (Become Brutes)

Browse source 
CreateRoleAttachedTargetEffect changed to use all targets, not just first
This commit is contained in:
xenohedron 2023-11-25 02:20:11 -05:00
parent 9f3777a82f
commit 867a8f54b0
2 changed files with 36 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

17
Mage/src/main/java/mage/abilities/effects/common/CreateRoleAttachedTargetEffect.java
View file

@ -9,6 +9,8 @@ import mage.game.Game;
import mage.game.permanent.Permanent;
import mage.game.permanent.token.Token;
import java.util.UUID;
/**
* @author TheElk801
*/
@ -33,13 +35,16 @@ public class CreateRoleAttachedTargetEffect extends OneShotEffect {
@Override
public boolean apply(Game game, Ability source) {
Permanent permanent = game.getPermanent(getTargetPointer().getFirst(game, source));
if (permanent == null) {
return false;
boolean result = false;
for (UUID targetId : getTargetPointer().getTargets(game, source)) {
Permanent permanent = game.getPermanent(targetId);
if (permanent != null) {
Token token = roleType.createToken(permanent, game, source);
// The token may not be created, for instance if the creature has protection from enchantments.
result |= !token.getLastAddedTokenIds().isEmpty();
}
}
Token token = roleType.createToken(permanent, game, source);
// The token may not be created, for instance if the creature has protection from enchantments.
return !token.getLastAddedTokenIds().isEmpty();
return result;
}
@Override