octgn / OCTGN

Online Card and Tabletop Gaming Network
http://www.octgn.net
Other
359 stars 129 forks source link

Shared Deck Not Loading [mtg] #2233

Closed hawkerm closed 3 months ago

hawkerm commented 5 months ago

Describe the bug Trying to load a decklist with Shared="True" and getting a The given key was not present in the dictionary error. Couldn't see how to move things into the global tab in the editor for the magic set list, so was modifying the xml for that property. And while deck loads in editor, it won't load on the table. Changing the property back to a non-shared deck and it works fine. Thought maybe had to do with the section name, but de-duping them or renaming them doesn't do anything (though not having a Main section when loading a non-shared deck seems to have a problem too).

To Reproduce Steps to reproduce the behavior:

  1. Install mtg game 3.4.394
  2. Modify Main deck to have Shared="True" (see xml below for file)
  3. Start LAN game
  4. Load deck
  5. See error

Expected behavior Able to load shared deck for tower formats. Related #1005, #755, #959, #326

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Additional context Add any other context about the problem here.

<?xml version="1.0" encoding="utf-8" standalone="yes"?>
<deck game="a6c8d2e8-7cd8-11dd-8f94-e62b56d89593">
  <section name="Main" shared="True">
    <card qty="10" id="414d3cae-b8cf-4d53-bd6b-1aa83a828ba9">Dandân</card>
    <card qty="2" id="8d42d7aa-7f53-4cfc-842a-086aab2448d1">Brainstorm</card>
    <card qty="2" id="0aa81390-4e0b-484b-a5be-a9449cd41860">Magical Hack</card>
    <card qty="8" id="63453ed9-5cf1-4cad-b173-a067f22a4405">Memory Lapse</card>
    <card qty="2" id="a0f0c20c-184e-4d27-ae8b-933abb6fee0c">Metamorphose</card>
    <card qty="2" id="952eb6ae-a530-4f4f-92f0-a6602beaa7b2">Mind Bend</card>
    <card qty="2" id="63754036-d51e-47bb-925b-564d9dc922ff">Narset's Reversal</card>
    <card qty="2" id="04dc842c-3250-44ed-906c-38f14bf1f0e2">Predict</card>
    <card qty="2" id="9a04e9be-48be-440e-9825-cfffd4c2b1a4">Repulse</card>
    <card qty="2" id="3df927c0-9aa9-450b-ab2a-ae12c5489d57">Supplant Form</card>
    <card qty="2" id="ba5dac3d-4b49-44c4-a7b2-0a99485252c9">Unsubstantiate</card>
    <card qty="2" id="98291778-2ec2-47e2-ac99-5f8cfbb3cf24">Chart a Course</card>
    <card qty="2" id="a375ec24-4841-4792-ad58-f29cdf0d1bbb">Diminishing Returns</card>
    <card qty="2" id="e88ae6bf-9c58-4543-ba66-19ea41d01e9b">Gone Missing</card>
    <card qty="2" id="281a685a-bd02-43bf-8700-2207c65bbbb1">Mystic Retrieval</card>
    <card qty="4" id="a3584150-4f09-4c15-b13e-707bd91b3004">Take Inventory</card>
    <card qty="2" id="7b7610f3-f182-404e-80b9-ccd94e174db0">Flood Plain</card>
    <card qty="2" id="c5651e5e-2314-4e3d-a2c8-2579239314e1">Halimar Depths</card>
    <card qty="18" id="0c4eaecf-dd4c-45ab-9b50-2abe987d35d4">Island</card>
    <card qty="2" id="666f455e-3a3d-475d-b67a-a1fdd74820eb">Izzet Boilerworks</card>
    <card qty="2" id="d8ddab06-aff7-4c40-bcaa-10cbfe899dd9">Lonely Sandbar</card>
    <card qty="2" id="170e792c-80d5-4775-ad95-37614574ab84">Mystic Sanctuary</card>
    <card qty="2" id="e48d55ff-10d0-4d9b-9202-02ebb2137953">Remote Isle</card>
    <card qty="2" id="00f8ec3c-d2cb-477c-a7e8-ff497df646d6">Temple of Epiphany</card>
  </section>
  <section name="Sideboard" shared="False" />
  <section name="Command Zone" shared="False" />
  <section name="Planes/Schemes" shared="False" />
  <notes><![CDATA[]]></notes>
</deck>
Wirox commented 3 months ago

Hey. Not all games support the shared deck and MtG is one of them, and you can't just edit the deck's XML to make it work. If you'd like MtG to support the shared deck, you could post such a suggestion here: https://github.com/brine/mtg-octgn

brine commented 3 months ago

Closing, as this is not an issue within OCTGN