About merging Draft and Sealed decks
Post MTG Forge Related Programming Questions Here
	Moderators: timmermac, Agetian, friarsol, Blacksmith, KrazyTheFox, 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 84 guests