github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-11-15 19:19:33 +00:00
Code Issues Projects Releases Packages Wiki Activity

* Coat of Arms - FIxed that the boost effect also worked with non creature types (e.g. Basic land types if some lands were animated).

Browse source
This commit is contained in:
LevelX2 2016-05-22 16:24:45 +02:00
parent b565b832b0
commit 949662de16
1 changed files with 6 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
Mage.Sets/src/mage/sets/magic2010/CoatOfArms.java
View file

@ -44,6 +44,7 @@ import mage.constants.Zone;
import mage.filter.common.FilterCreaturePermanent; import mage.filter.common.FilterCreaturePermanent;
import mage.game.Game; import mage.game.Game;
import mage.game.permanent.Permanent; import mage.game.permanent.Permanent;
import mage.util.CardUtil;
/** /**
* *
@ -87,11 +88,6 @@ class CoatOfArmsEffect extends ContinuousEffectImpl {
return new CoatOfArmsEffect(this); return new CoatOfArmsEffect(this);
} }
@Override
public void init(Ability source, Game game) {
super.init(source, game);
}
@Override @Override
public boolean apply(Game game, Ability source) { public boolean apply(Game game, Ability source) {
List<Permanent> permanents = game.getBattlefield().getActivePermanents(filter, source.getControllerId(), game); List<Permanent> permanents = game.getBattlefield().getActivePermanents(filter, source.getControllerId(), game);
@ -112,9 +108,11 @@ class CoatOfArmsEffect extends ContinuousEffectImpl {
for (Permanent permanent : permanents) { for (Permanent permanent : permanents) {
if (!permanent.getId().equals(target.getId())) { if (!permanent.getId().equals(target.getId())) {
for (String subtype : targetSubtype) { for (String subtype : targetSubtype) {
if (permanent.hasSubtype(subtype)) { if (!CardUtil.isNonCreatureSubtype(subtype)) {
amount++; if (permanent.hasSubtype(subtype)) {
break; amount++;
break;
}
} }
} }
} }