Making actions instance methods on Player for cleaner Ruby syntax

[radar]

Before

action = Magic::Actions::ActivateAbility.new(player: p1, permanent: island, ability: island.activated_abilities.first)

After

p1.activate_ability(ability: island.activated_abilities.first)

Abilities are now tied to the permanent they're from. IOW, going from ability -> permanent now rather than permanent -> abilities.