From 12d4cd142938fedea10befe2aefdd0a964e0215c Mon Sep 17 00:00:00 2001
From: Tiago Costa <Nippius@users.noreply.github.com>
Date: Fri, 25 Apr 2025 15:40:08 +0100
Subject: [PATCH] [Fix][Spyder] Cards cannot be moved to empty spots

When playing Spider, after there are no more cards in stock, it becomes impossible to move cards into empty slots.
This commit fixes the issue by making sure the game logic compares the source and destionation by reference, instead of using the number of cards in source and destination.
---
 Solitaire/ViewModels/Pages/SpiderSolitaireViewModel.cs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/Solitaire/ViewModels/Pages/SpiderSolitaireViewModel.cs b/Solitaire/ViewModels/Pages/SpiderSolitaireViewModel.cs
index 89e1519..8bac1a2 100644
--- a/Solitaire/ViewModels/Pages/SpiderSolitaireViewModel.cs
+++ b/Solitaire/ViewModels/Pages/SpiderSolitaireViewModel.cs
@@ -201,10 +201,11 @@ public override bool CheckAndMoveCard(IList<PlayingCardViewModel> from, IList<Pl
         PlayingCardViewModel card, bool checkOnly = false)
     {
         //  The trivial case is where from and to are the same.
-        if (from.SequenceEqual(to))
+        if (ReferenceEquals(from, to))
             return false;
-        
-        if (to.SequenceEqual(Stock))
+
+        // Don't allow player to move cards to the Stock
+        if (ReferenceEquals(to, Stock))
             return false;
         
         //  This is the complicated operation.