Improve/Refactor add counters log message
Currently AddCountersTargetEffect and AddCountersSourceEffect are writing log messages when counters are added to permanents. The problem is I've had a couple cards in Dominaria where I've had to write custom effects and manually add a log message (Zar Ojanen, Scion of Efrava and Choking Miasma).
I think if we want these log messages, it should be done in CardImpl's addCounters method so it's all done in one place. This would also possibly allow checking for replacement effects and giving accurate log messages which is not checked at all for currently.
I'm probably not going to get to this until Dominaria is finished but wanted to put this here in case anyone has input/suggestions (and also as a reminder to myself).
Something else I just saw. When we write log messages for a player drawing cards, it's done here using game.fireInformEvent rather than game.informPlayers like the counters are doing. Putting this here for future reference in case this needs to be done for counters as well.
https://github.com/magefree/mage/blob/7cde4ab382e93a69043382ec2122bf82e85c83e1/Mage/src/main/java/mage/actions/MageDrawAction.java#L49-L71