MTGProxyPrinter

Check-in [9deba291b1]
Login

Check-in [9deba291b1]

Many hyperlinks are disabled.
Use anonymous login to enable hyperlinks.

Overview
Comment:The Archidekt deck list downloader now accepts URLs starting with "https://www."
Downloads: Tarball | ZIP archive | SQL archive
Timelines: family | ancestors | descendants | both | trunk
Files: files | file ages | folders
SHA3-256: 9deba291b12b319fb258620eb9e5f8ec1a9c1a486bb127145df7e61d877909b2
User & Date: thomas 2024-09-22 18:07:45
Context
2024-10-01
15:54
Deck list downloaders for CubeCobra and Manabox.app now accepts the "http://www." prefix. check-in: e6312ee8a9 user: thomas tags: trunk
2024-09-22
18:07
The Archidekt deck list downloader now accepts URLs starting with "https://www." check-in: 9deba291b1 user: thomas tags: trunk
2024-09-16
09:07
MainWindow: Fix a broken call to tr(), which was still using an f-string and wasn't picked up by lupdate. check-in: cd7aee3234 user: thomas tags: trunk
Changes
Hide Diffs Unified Diffs Ignore Whitespace Patch

Changes to mtg_proxy_printer/decklist_downloader.py.

297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
        if self.found_deck_tag:
            self.decklist_json = data
            self.found_deck_tag = False


class ArchidektDownloader(DecklistDownloader):
    DECKLIST_PATH_RE = re.compile(
        r"https://archidekt\.com/decks/(?P<deck_id>\d+).*?"
    )
    PARSER_CLASS = ScryfallCSVParser
    APPLICABLE_WEBSITES = "Archidekt (archidekt.com)"

    def map_to_download_url(self, decklist_url: str) -> str:
        return decklist_url








|







297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
        if self.found_deck_tag:
            self.decklist_json = data
            self.found_deck_tag = False


class ArchidektDownloader(DecklistDownloader):
    DECKLIST_PATH_RE = re.compile(
        r"https://(www\.)?archidekt\.com/decks/(?P<deck_id>\d+).*?"
    )
    PARSER_CLASS = ScryfallCSVParser
    APPLICABLE_WEBSITES = "Archidekt (archidekt.com)"

    def map_to_download_url(self, decklist_url: str) -> str:
        return decklist_url

Changes to tests/test_decklist_downloader.py.

104
105
106
107
108
109
110



111
112
113
114
115
116
117
    yield DeckstatsDownloader, "https://deckstats.net/decks/57872/450232-tapland#show__spoiler"
    yield DeckstatsDownloader, "https://deckstats.net/decks/51910/1430827-"  # An untitled deck
    yield DeckstatsDownloader, "https://deckstats.net/decks/51910/1430827"  # Missing required hyphen, added internally

    # Archidekt
    yield ArchidektDownloader, "https://archidekt.com/decks/8"
    yield ArchidektDownloader, "https://archidekt.com/decks/4296325"




    # TCGPlayer Infinite
    yield TCGPlayerDownloader, "https://infinite.tcgplayer.com/magic-the-gathering/deck/Azorius-Hammer/468532"
    yield TCGPlayerDownloader, "https://infinite.tcgplayer.com/magic-the-gathering/deck/Azorius-Hammer/468532/"

    # CubeCobra
    yield CubeCobraDownloader, "https://cubecobra.com/cube/list/gilpauper"







>
>
>







104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
    yield DeckstatsDownloader, "https://deckstats.net/decks/57872/450232-tapland#show__spoiler"
    yield DeckstatsDownloader, "https://deckstats.net/decks/51910/1430827-"  # An untitled deck
    yield DeckstatsDownloader, "https://deckstats.net/decks/51910/1430827"  # Missing required hyphen, added internally

    # Archidekt
    yield ArchidektDownloader, "https://archidekt.com/decks/8"
    yield ArchidektDownloader, "https://archidekt.com/decks/4296325"
    yield ArchidektDownloader, "https://archidekt.com/decks/4296325/some_name"
    yield ArchidektDownloader, "https://www.archidekt.com/decks/4296325"
    yield ArchidektDownloader, "https://www.archidekt.com/decks/4296325/some_name"

    # TCGPlayer Infinite
    yield TCGPlayerDownloader, "https://infinite.tcgplayer.com/magic-the-gathering/deck/Azorius-Hammer/468532"
    yield TCGPlayerDownloader, "https://infinite.tcgplayer.com/magic-the-gathering/deck/Azorius-Hammer/468532/"

    # CubeCobra
    yield CubeCobraDownloader, "https://cubecobra.com/cube/list/gilpauper"