Legend

Glad you like the code and its working for you!

Regarding the bug, it sounds familiar... did you already finish the whole series? Because I know I fixed some bugs around targeting in the lesson “Spell & Abilities” – check out the section header “More Refactoring”.

Thanks for the gif, I can confirm that I see the same issue. Here's what is happening...

In the "ClickToPlayCardController", when you click off of the manual targeted card, it goes back to the hand so you can see the board better and make it easier to actually select a target. This is the "ShowTargetState" state. It looks similar to what happens when you deselect a card, so that could be a bit confusing, and it would probably help if we added a different position off to the side or something.

The next time you click, it assigns the selection (null in this case) and completes the flow via ConfirmState etc. The logic for determining the validity of a target should exists in the TargetSystem. The TargetSystem is permitting this spell to play because of the way that it is defined. The card's target's "required" flag will be false because I forgot to implement it. If you add something like this to the "Card10"'s "target" body:

"required": true,

And add something like this to the DeckFactory:

if (targetData.ContainsKey("required")) {
target.required = Convert.ToBoolean(targetData["required"]);
}

Then you should be able to fix this bug. Hope that helps!