[DeckSync] Match server rows by identity and hash the whole deck

Sync keyed local files to server rows by path, but the server names a row
after the deck, not after the file, so any deck whose file name and deck name
differ uploaded again on every run. A second pass then brought the server copy
back as an extra local file.

- Match by the sidecar's row id, then guid, then content, and only then by the
  server path, so renaming either side no longer duplicates a deck.
- Key the server tree by row id: the server allows several rows with the same
  name in one folder, and keying by path silently dropped all but one.
- Stop discarding rows whose name contains '/' ("U/W Control"). They stay
  matchable, and only the download file name is sanitised.
- Build the download path from the folder plus a sanitised row name, including
  every folder segment, so a hostile server name cannot escape the deck folder.
- Download a colliding server row under a unique name and count it, instead of
  skipping it without a trace.
- Hash the whole deck instead of the card hash: SHA-1 over the native deck
  string covers name, printings, banner, tokens and sideboard plans. The client
  and the server hash the exact bytes they exchange, so both agree.

Adds deck_content_hash_test for the client/server hashing invariant and for the
name-only change the card hash cannot see.

cockatrice/src/interface/deck_sync/deck_sync_profile.h

@@ -9,15 +9,16 @@
 #include <QString>
 
 /**
- * @struct DeckSyncProfile
+ * @class DeckSyncProfile
  * @brief Identifies which server and login a sync sidecar belongs to.
  *
  * A local deck folder can sync against several servers. Every server login
  * gets its own profile, and every profile gets its own sidecar file, so
  * switching servers never mixes up remote deck ids.
  */
-struct DeckSyncProfile
+class DeckSyncProfile
 {
+public:
     QString host;     ///< Server hostname as entered by the user.
     int port = 0;     ///< Server port.
     QString username; ///< Login name on that server.

cockatrice/src/interface/deck_sync/deck_sync_sidecar.cpp

@@ -37,11 +37,12 @@ bool DeckSyncSidecar::load()
             continue;
         }
         const QJsonObject entry = it.value().toObject();
-        const DeckSyncRecord record{.guid = entry.value("guid").toString(),
-                                    .remoteId = entry.value("remoteId").toInt(-1),
-                                    .baseHash = entry.value("baseHash").toString(),
-                                    .lastSyncSecs = static_cast<qint64>(entry.value("lastSync").toDouble()),
-                                    .deleted = entry.value("deleted").toBool(false)};
+        DeckSyncRecord record;
+        record.guid = entry.value("guid").toString();
+        record.remoteId = entry.value("remoteId").toInt(-1);
+        record.baseHash = entry.value("baseHash").toString();
+        record.lastSyncSecs = static_cast<qint64>(entry.value("lastSync").toDouble());
+        record.deleted = entry.value("deleted").toBool(false);
         recordMap.insert(it.key(), record);
     }
     return true;

libcockatrice_deck_list/libcockatrice/deck_list/deck_diff.cpp

@@ -70,13 +70,14 @@ DeckDiffResult diff(const DeckList &oldDeck, const DeckList &newDeck)
         const int oldCount = oldCounts.value(key, 0);
         const ZoneCardInfo newInfo = newInfos.value(key, ZoneCardInfo());
         if (oldCount != newInfo.count) {
-            const DeckCardChange change{.zone = key.first,
-                                        .cardName = key.second,
-                                        .oldCount = oldCount,
-                                        .newCount = newInfo.count,
-                                        .cardSetShortName = newInfo.cardSetShortName,
-                                        .cardSetNumber = newInfo.cardSetNumber,
-                                        .cardProviderId = newInfo.cardProviderId};
+            DeckCardChange change;
+            change.zone = key.first;
+            change.cardName = key.second;
+            change.oldCount = oldCount;
+            change.newCount = newInfo.count;
+            change.cardSetShortName = newInfo.cardSetShortName;
+            change.cardSetNumber = newInfo.cardSetNumber;
+            change.cardProviderId = newInfo.cardProviderId;
             result.changes.append(change);
         }
     }