About merging Draft and Sealed decks
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
23 posts
• Page 2 of 2 • 1, 2
Re: About merging Draft and Sealed decks
by Max mtg » 19 Feb 2012, 18:39
What else is there to impove?
- move the /draft/*.draft and /sealed/*.sealed setup files to a better place and make an UI to edit them.
- eliminate direct file reading for deck-related purposes (such as quest precons, challenges, duels)
- a better UI for all deckeditors
- move the /draft/*.draft and /sealed/*.sealed setup files to a better place and make an UI to edit them.
- eliminate direct file reading for deck-related purposes (such as quest precons, challenges, duels)
- a better UI for all deckeditors

Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: About merging Draft and Sealed decks
by jeffwadsworth » 19 Feb 2012, 19:42
Ver 14202:
- | Open
- This is a Crash Report. An error has occurred. Please save this message to a file.
Please follow the instructions at this address to submit this Crash Report, plus what you were doing at the time:
http://tinyurl.com/3zzrnyb
Reporting bugs in Forge is very important. We thank you for your time.
null
Version:
Forge version SVN
OS: Windows 7 Version: 6.1 Architecture: amd64
Java Version: 1.6.0_30 Vendor: Sun Microsystems Inc.
Detailed error trace:
java.lang.NullPointerException
at forge.deck.io.DeckSetSerializer.read(DeckSetSerializer.java:63)
at forge.deck.io.DeckSetSerializer.read(DeckSetSerializer.java:1)
at forge.deck.io.DeckSerializerBase.readAll(DeckSerializerBase.java:106)
at forge.util.FolderMapView.<init>(FolderMapView.java:47)
at forge.util.FolderMap.<init>(FolderMap.java:42)
at forge.deck.CardCollections.<init>(CardCollections.java:28)
at forge.AllZone.getDecks(AllZone.java:97)
at forge.control.home.ControlConstructed.getCustomNames(ControlConstructed.java:325)
at forge.control.home.ControlConstructed.updateDeckLists(ControlConstructed.java:162)
at forge.control.home.ControlConstructed.<init>(ControlConstructed.java:96)
at forge.view.home.ViewConstructed.<init>(ViewConstructed.java:67)
at forge.view.ViewHomeUI.<init>(ViewHomeUI.java:77)
at forge.view.FView.instantiateCachedUIStates(FView.java:164)
at forge.view.FView$2.run(FView.java:103)
at java.awt.event.InvocationEvent.dispatch(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$000(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)
- jeffwadsworth
- Super Tester Elite
- Posts: 1172
- Joined: 20 Oct 2010, 04:47
- Location: USA
- Has thanked: 287 times
- Been thanked: 70 times
Re: About merging Draft and Sealed decks
by Max mtg » 19 Feb 2012, 20:06
option A: please compress and send me your /res/decks folder to investigate
option B: update to 14206 and tell me how forge behaves then
option B: update to 14206 and tell me how forge behaves then
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: About merging Draft and Sealed decks
by jeffwadsworth » 19 Feb 2012, 20:24
Works fine now. Thanks.Max mtg wrote:option A: please compress and send me your /res/decks folder to investigate
option B: update to 14206 and tell me how forge behaves then
- jeffwadsworth
- Super Tester Elite
- Posts: 1172
- Joined: 20 Oct 2010, 04:47
- Location: USA
- Has thanked: 287 times
- Been thanked: 70 times
Re: About merging Draft and Sealed decks
by Chris H. » 20 Feb 2012, 14:54
In message: Re: Bug Reports (snapshot builds)
This may be related to the new deck code.
`moomarc wrote:Just a small bug: the deck editor is diplaying AE cards strangely this morning (it doesn't affect the card info panel though).
This may be related to the new deck code.
-
Chris H. - Forge Moderator
- Posts: 6320
- Joined: 04 Nov 2008, 12:11
- Location: Mac OS X Yosemite
- Has thanked: 644 times
- Been thanked: 643 times
Re: About merging Draft and Sealed decks
by Max mtg » 20 Feb 2012, 17:57
Not related to deckcode.Chris H. wrote:In message: Re: Bug Reports (snapshot builds)`moomarc wrote:Just a small bug: the deck editor is diplaying AE cards strangely this morning (it doesn't affect the card info panel though).
This may be related to the new deck code.
It's a small change I made to display card names in editor. - replace AE with ligature Æ as it is printed on cards
Single class for single responsibility.
- Max mtg
- Programmer
- Posts: 1997
- Joined: 02 Jul 2011, 14:26
- Has thanked: 173 times
- Been thanked: 334 times
Re: About merging Draft and Sealed decks
by jeffwadsworth » 20 Feb 2012, 19:01
Yeah, I thought someone made an enhancement to the text of the Aether's. Looks great to me.
- jeffwadsworth
- Super Tester Elite
- Posts: 1172
- Joined: 20 Oct 2010, 04:47
- Location: USA
- Has thanked: 287 times
- Been thanked: 70 times
Re: About merging Draft and Sealed decks
by moomarc » 21 Feb 2012, 05:50
Thanks Max. Not sure what the problem was the other day but looks great now.Max mtg wrote:Not related to deckcode.Chris H. wrote:In message: Re: Bug Reports (snapshot builds)`moomarc wrote:Just a small bug: the deck editor is diplaying AE cards strangely this morning (it doesn't affect the card info panel though).
This may be related to the new deck code.
It's a small change I made to display card names in editor. - replace AE with ligature Æ as it is printed on cards

-Marc
-
moomarc - Pixel Commander
- Posts: 2091
- Joined: 04 Jun 2010, 15:22
- Location: Johannesburg, South Africa
- Has thanked: 371 times
- Been thanked: 372 times
23 posts
• Page 2 of 2 • 1, 2
Who is online
Users browsing this forum: No registered users and 17 guests