Bug Reports (snapshot builds)

[krevett]

Yet another bug (sorry for these multiple posts I'm currently playing and post as I notice ) still using today's snapshot: A flashbacked spell that is countered by a spell or ability isn't exiled as it should per the rules.

I quote myself because this bug is very annoying, if the AI flashback a spell and I choose to counter it, the AI can flashback it the same turn if it has the ressources because the spell isn't exiled...It makes countering flashbacked spell very poor

[timmermac]

Avatar switching bug still exists as of r14436.

[krevett]

Avatar switching bug still exists as of r14436.

In fact, when I select my avatars in properties and test in constructed they are well placed, but if I play a match in quest mode they get switched and remain switched after that.

[Doublestrike]

Ahh, the problem is in quest mode. Thanks, Krevett, god is in the details.

Fixed.

[krevett]

I don't know if it was fixed in today's snapshot but if that's the case the avatar still switch when going into quest mode (r14456) I report but perhaps the fix will be in tomorrow's snapshot
Anyway thanks everyone for this wonderful project!!

Edit: Just to tell that the way avatars are displayed in the options takes much longer to load than the first time they were included...

Edit2: Sorry, just reset the avatars and now it seems to works properly, my preference file was inversed

Edit3: Eventually, after some match in quest mode, my avatar is inversed with the one I choosed for AI in my preferences but it's not inversed with the icon of my quest oponnent

[friarsol]

r14463
When the AI plays a land with a triggered ability, the trigger isn't flushed to the stack before the AI continues to play. This can create a situation where the AI casts a Sorcery and a Land trigger appears on top/bottom (I forget which) of it.

[krevett]

In fact the avatar switch bug now seems to switch between those i've chosen for the AI and myself in settings each time I restart forge.

[gos]

No need to change all of the earlier decks, people can place them into the /decks/ directory and Max's converter will convert and then move them into the /decks/constructed/ directory automatically.

I have to admit, I'm not too happy with the converter renaming all the .dck files...

[Chris H.]

No need to change all of the earlier decks, people can place them into the /decks/ directory and Max's converter will convert and then move them into the /decks/constructed/ directory automatically.

I have to admit, I'm not too happy with the converter renaming all the .dck files...

`
Can you give us an example of what you are seeing?

[gos]

No need to change all of the earlier decks, people can place them into the /decks/ directory and Max's converter will convert and then move them into the /decks/constructed/ directory automatically.

I have to admit, I'm not too happy with the converter renaming all the .dck files...

Can you give us an example of what you are seeing?

Example:

I put a file named Ali Aintrazi's Heartless Rites.dck in the /res/decks/ folder.
After I start forge, and the converter has done its work, the file is in the /res/decks/constructed/ folder, but it's named AliAintrazisHeart.dck.

Worse:

I put the following files in the /res/decks/ folder:
Crypt of Agadeem, Budget Version 2.dck
Crypt of Agadeem, Budget Version 3.dck
Crypt of Agadeem, Final Budget Version.dck
Crypt of Agadeem, Non-Budget Version.dck
Crypt of Agadeem, Version 1.dck

After the conversion, I have the following decks in the /res/decks/constructed/ folder:
CryptofAgadeemBu.dck
CryptofAgadeemFi.dck
CryptofAgadeemNo.dck
CryptofAgadeemVe.dck

That first one is Budget Version 3 - I haven't found Budget Version 2, and I have a sinking feeling that it's been overwritten...

[gos]

That first one is Budget Version 3 - I haven't found Budget Version 2, and I have a sinking feeling that it's been overwritten...

Yeah, it looks like it's permanently gone and needs to be recreated. I have all the decks I compile from the wizards.com columns in database form, but I have no overview of any other decks that may or may not have been deleted permanently just now.

[Chris H.]

That first one is Budget Version 3 - I haven't found Budget Version 2, and I have a sinking feeling that it's been overwritten...

Yeah, it looks like it's permanently gone and needs to be recreated. I have all the decks I compile from the wizards.com columns in database form, but I have no overview of any other decks that may or may not have been deleted permanently just now.

`
I think there is some code which limits the file names in forge when we save a deck in the deck editor. The length limit may be 20 characters? And space characters (and other special chars) are removed when the deck is saved. I guess that the converter is using the same code?

We can edit the file name of a deck outside of forge and make the name longer and add space characters.

[friarsol]

We can edit the file name of a deck outside of forge and make the name longer and add space characters.

That's not really a solution. Stripping possibly harmful characters and spaces is one thing, but truncating and overwriting without warning/renaming is really bad.

[Chris H.]

We can edit the file name of a deck outside of forge and make the name longer and add space characters.

That's not really a solution. Stripping possibly harmful characters and spaces is one thing, but truncating and overwriting without warning/renaming is really bad.

`
I did not mean this to be taken as a solution. It is more of an explanation on how people could rename their decks outside of forge and then with this new version find that they are having the same problem as Gos. I myself have renamed my constructed decks outside of forge. This may end up more widespread.

[Max mtg]

@14501

Code: Select all

    public final String getBestFileName() {
        return this.getName().replaceAll("[^-_$#@.{[()]} a-zA-Z0-9]", "");
    }

Its older edition was truncating filenames for decks some time ago.

To recreate your decks move all files from res/decks/constructed to res/decks and let deckparser assign some better names