Mandatory targets

[LevelX]

I had to modify the condition to take care about minNumTargets = 0.
However, that doesn't handle situations where minNumTargets > 0 && minNumTargets < maxNumtargets, where the "Done" Button has to work also.
Seems like there is no such call of the method yet.

Code: Select all

    public TargetCardInLibrary(int minNumTargets, int maxNumTargets, FilterCard filter) {
        super(minNumTargets, maxNumTargets, Zone.LIBRARY, filter);
       
        this.setRequired(minNumTargets > 0 && !filter.hasPredicates());
        this.librarySearchLimit = Integer.MAX_VALUE;
    }

[quercitron]

I'm not sure it should be handled this way. I think required status and number of targets should be independent on this level. If target is required, it just means that at least minNumTargets have to be chosen, if possible.
I'll investigate how this issue can be solved.

[LevelX]

I'm not sure it should be handled this way. I think required status and number of targets should be independent on this level. If target is required, it just means that at least minNumTargets have to be chosen, if possible.

Agreed, but it didn't work.

[LevelX]

Had to do some more change to target to fix bug with Kruphix’s Insight and probably more cards.

https://github.com/magefree/mage/commit/5ae1f6306bb1ea7db6c9e74cb6c21f3a20d133da

[quercitron]

I've moved handling of cases when minNumTargets < maxNumTargets to HumanPlayer implementation. I hope it will work right.