do not edit the zone currently iterated on

[ebbit1q]

Related Ticket(s)

• reverts Fix regression from #4281 which caused crash if card was null #4340 and Fix regression from #4281 which caused crash with QList and GCC race time #4341 which were fixes that didn't work

Short roundup of the initial problem

unattaching cards invokes the moveCard function which changes the
internal cards list

this can cause the iterator to become invalidated which will crash but
because of the data not always being moved it will often still work as
intended, giving the idea that it is random

What will change with this Pull Request?

• cards will no longer be unattached when a player leaves play if both cards are controlled by the same player
  • this is unnoticeable to clients as in the same event that player will have all their cards deleted regardless

[ZeldaZach]

This makes sense