getValidCards + getTargetableCards to getValidCardsToTarget

[Hanmac]

We should clean up the parts of the code that uses getValidCards + TargetRestriction + getTargetableCards with getValidCardsToTarget ?

like this part?

forge/forge-game/src/main/java/forge/game/staticability/StaticAbilityMustTarget.java

Lines 44 to 47 in 831d61f

	TargetRestrictions tgt = currentAbility.getTargetRestrictions();
	List<ZoneType> zone = tgt.getZone();
	List<Card> validCards = CardLists.getValidCards(game.getCardsIn(zone), tgt.getValidTgts(), currentAbility.getActivatingPlayer(), currentAbility.getHostCard(), currentAbility);
	List<Card> choices = CardLists.getTargetableCards(validCards, currentAbility);

[Jetz72]

Just going by method names, currentAbility.getTargetRestrictions().getValidTgts(), [getTargetableCards + an ability], [getValidCards + a valid-target restriction] , and [getValidCardsToTarget + an ability]... all sound like they should mean the same thing. Having to use some combination of them together makes these sections kinda incomprehensible. Definitely in favor of simplifying where possible, and adding documentation or changing method names where simplifying is not possible.