[pcgen-commit-svn] SF.net SVN: pcgen:[15686] sandbox/uisync/code/src/java/pcgen/io/ PCGVer2Parser.java

[<jde...@us...>]

Revision: 15686
          http://pcgen.svn.sourceforge.net/pcgen/?rev=15686&view=rev
Author:   jdempsey
Date:     2011-12-23 04:27:10 +0000 (Fri, 23 Dec 2011)
Log Message:
-----------
Add logging of character source errors.

Modified Paths:
--------------
    sandbox/uisync/code/src/java/pcgen/io/PCGVer2Parser.java

Modified: sandbox/uisync/code/src/java/pcgen/io/PCGVer2Parser.java
===================================================================
--- sandbox/uisync/code/src/java/pcgen/io/PCGVer2Parser.java	2011-12-20 19:37:40 UTC (rev 15685)
+++ sandbox/uisync/code/src/java/pcgen/io/PCGVer2Parser.java	2011-12-23 04:27:10 UTC (rev 15686)
@@ -227,11 +227,16 @@
 	{
 		buildPcgLineCache(lines);
 
-		if (!cache.containsKey(TAG_GAMEMODE)
-				|| !cache.containsKey(TAG_CAMPAIGN))
+		if (!cache.containsKey(TAG_GAMEMODE))
 		{
+			Logging.errorPrint("Character does not have game mode information.");
 			return null;
 		}
+		if (!cache.containsKey(TAG_CAMPAIGN))
+		{
+			Logging.errorPrint("Character does not have campaign information.");
+			return null;
+		}
 		String line = cache.get(TAG_GAMEMODE).get(0);
 		String requestedMode = line.substring(TAG_GAMEMODE.length() + 1);
 		GameMode mode = SystemCollections.getGameModeNamed(requestedMode);
@@ -249,6 +254,7 @@
 		//if mode == null still then a game mode was not found
 		if (mode == null)
 		{
+			Logging.errorPrint("Character's game mode entry was not valid: " + line);
 			return null;
 		}
 		/*
@@ -262,6 +268,7 @@
 		List<Campaign> campaigns = getCampaignList(cache.get(TAG_CAMPAIGN));
 		if (campaigns.isEmpty())
 		{
+			Logging.errorPrint("Character's campaign entry was empty.");
 			return null;
 		}
 		return FacadeFactory.createSourceSelection(mode, campaigns);

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.