It is currently 03 Jul 2025, 17:58
   
Text Size

About merging Draft and Sealed decks

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins

Re: About merging Draft and Sealed decks

Postby 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 :)
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

Postby 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

Postby 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
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

Postby jeffwadsworth » 19 Feb 2012, 20:24

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
Works fine now. Thanks.
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

Postby Chris H. » 20 Feb 2012, 14:54

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.
User avatar
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

Postby Max mtg » 20 Feb 2012, 17:57

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.
Not related to deckcode.
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

Postby 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

Postby moomarc » 21 Feb 2012, 05:50

Max mtg wrote:
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.
Not related to deckcode.
It's a small change I made to display card names in editor. - replace AE with ligature Æ as it is printed on cards
Thanks Max. Not sure what the problem was the other day but looks great now. =D>
-Marc
User avatar
moomarc
Pixel Commander
 
Posts: 2091
Joined: 04 Jun 2010, 15:22
Location: Johannesburg, South Africa
Has thanked: 371 times
Been thanked: 372 times

Previous

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 17 guests

Main Menu

User Menu

Our Partners


Who is online

In total there are 17 users online :: 0 registered, 0 hidden and 17 guests (based on users active over the past 10 minutes)
Most users ever online was 5050 on 26 Jun 2025, 06:02

Users browsing this forum: No registered users and 17 guests

Login Form