UI Improvements, Round 2: Match UI

[friarsol]

I think Max mentioned quests saved between 13703 and 13735 (?) are permanently broken?

But that shouldn't have anything to do with the prefs saving...I'll test again using your clues.

Could be, but I wouldn't think that would affect prefs. I'll let you know if my new quest is broken too.

[Max mtg]

I think Max mentioned quests saved between 13703 and 13735 (?) are permanently broken?

But that shouldn't have anything to do with the prefs saving...I'll test again using your clues.

Could be, but I wouldn't think that would affect prefs. I'll let you know if my new quest is broken too.

I never tried to load that broken quest saves with the updated client, but in theory such saves should load normally without exceptions, as for decks - I expect them to contain no cards after deserialization from incorrecly saved decksections.

[friarsol]

Saving in a new quest didn't work.

[Doublestrike]

I just tested again, in quest mode.

- Old quest, persist across match
- Old quest, persist across restart
- New quest, persist across match
- New quest, persist across restart

No problems.

[goonjamin]

I am also still having the same issue as friarsol, with layout and phase prefs not saving.

What version of java is everyone running? Right now I am using java 6 update 30 (computer is windows 7 x86). I am going to try installing java 7u2 and see if that makes a difference.

[friarsol]

The machine I'm having issues with is my Netbook, Win 7x86 6u25. goonjamin, Java 7 will probably have worse errors as they broke backwards compatibility.

I'll check on my Gaming machine some point today when I have some time.

[friarsol]

Tested my gaming machine Win7x64 java 6u23 same as above. Doesn't save for me in Quest mode. But it does in Constructed..

[Chris H.]

Tested my gaming machine Win7x64 java 6u23 same as above. Doesn't save for me in Quest mode. But it does in Constructed..

`
I ran the same test on my machine and got the same results.

[friarsol]

It looks like forge.view.GuiTopLevel.savePrefs() is only being called from forge.control.match.ControlDock.concede()

I can't even tell where this is called for the Quest? Is there some different function?

[Doublestrike]

Ahh that's the clue that broke the camel's eye of the needle! I've been testing using only the concede button, not the regular game end!

Should be fixed now.