[pcgen-commit-svn] SF.net SVN: pcgen: [2877] Trunk/pcgen/code/src/java/pcgen/core/ PlayerCharacter.java

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

Revision: 2877
          http://svn.sourceforge.net/pcgen/?rev=2877&view=rev
Author:   jdempsey
Date:     2007-05-07 18:18:03 -0700 (Mon, 07 May 2007)

Log Message:
-----------
Fix bug: 5.11.13 specific monster class and dragon
Issue#: 1706067

Modified Paths:
--------------
    Trunk/pcgen/code/src/java/pcgen/core/PlayerCharacter.java

Modified: Trunk/pcgen/code/src/java/pcgen/core/PlayerCharacter.java
===================================================================
--- Trunk/pcgen/code/src/java/pcgen/core/PlayerCharacter.java	2007-05-08 01:13:18 UTC (rev 2876)
+++ Trunk/pcgen/code/src/java/pcgen/core/PlayerCharacter.java	2007-05-08 01:18:03 UTC (rev 2877)
@@ -8598,12 +8598,15 @@
 			// the equipment they belong to.
 			if (pobj != null && !(pobj instanceof EquipmentModifier))
 			{
-				pobj.activateBonuses(this);
-
-				// TODO - Class bonuses only get added if level is greater than
-				// zero. Is this check required? Should it be part of
-				// getPObjectList()?
-				ret.addAll(pobj.getActiveBonuses(this));
+				// Class bonuses are only included if the level is greater than 0
+				// This is because 0 levels of a class can be added to access spell casting etc
+				if (!(pobj instanceof PCClass)
+					|| ((PCClass) pobj).getLevel() > 0)
+				{
+					pobj.activateBonuses(this);
+	
+					ret.addAll(pobj.getActiveBonuses(this));
+				}
 			}
 		}
 


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