Card Ligatures and YOU!

[friarsol]

So, it looks like we might have a small issue with AE ligature conversions to Ae. For the Kaladesh card, "Aether Tradewinds" everywhere in Forge it's listed as AEther Tradewinds. I think for now, we can just adjust the edition file to be the existing version, but we should come up with a plan to convert the ligature across the board. We can probably do this for all ligature cards at the same time, but basically we'll need to convert editions files AND on-the-fly loading of decks.

Anyone interested in taking that on?

[Agetian]

By the way, as a little related observation, this might also require the conversion of card picture names *or* exceptional treating of AE/Ae in card picture file names in the code (the latter would be preferable, I guess), otherwise the cards with AE in their names won't show up correctly on any case-sensitive operating system (which in case of Forge means Linux, MacOS, anything BSD, and Android).

Hmm, so basically we need all the cards that had AE in them to now have Ae, right?
What should this cover? For example:

* editions files (AE->Ae everywhere);
* decks (deck loader should be able to load both the AE version and the Ae version);
* card pictures (card picture loader should be able to load both the AE version and the Ae version to operate correctly on case-sensitive operating systems).

Anything else I'm missing here?

- Agetian

[Hanmac]

i think the most easy way would be to allow the card to have an Alternate Name where it does look for it too.

like if it does not find "Aether Tradewinds" it does try "AEther Tradewinds".

that would fix the problems with converting for now.

[Marek14]

To make it simpler, note that all cards with the AE ligature contain the word "AEther", with just a single exception (AErathi Berserker).

[Hanmac]

i added "AltName" to Card scripts so this is possible:

Code: Select all

Name:Aetherflame Wall
AltName:AEtherflame Wall


what works: without changing the set files, the card is found under its new name. (Set has AEtherflame Wall, but it gets Aetherflame Wall)
That will work with already created Deck files.

What does not work yet:
i didn't manage to get it with the Imagekey loading.
it should try Name first and if that fails it should try AltName.
but it didn't do as i want yet.

[Agetian]

Nice job so far, Hanmac! It'd be important to get it working with ImageKey properly as well since pretty much any non-Windows system, including Android, is otherwise going to suffer from missing images when being unable to load the image by the alt-name key...

- Agetian

[Hanmac]

yeah i know ;P thats why i said that ImageKey is not converted yet to understand AltName (i have linux)

i need to see how many functions i need to rewrite to make it working like that. probably everywhere "getImageKey" is used.