Method bonusTo

code/src/java/pcgen/core/analysis/BonusCalc.java:55–137

(PObject po, String aType, String aName, final Object obj,
		final Collection<BonusObj> aBonusList, final PlayerCharacter aPC)

	}

	public static double bonusTo(PObject po, String aType, String aName, final Object obj,
		final Collection<BonusObj> aBonusList, final PlayerCharacter aPC)
	{
		if ((aBonusList == null) || (aBonusList.isEmpty()))
		{
			return 0;
		}

		double retVal = 0;

		aType = aType.toUpperCase();
		aName = aName.toUpperCase();

		final String aTypePlusName = aType + '.' + aName + '.';

		if (!BonusCalc.dontRecurse && (po instanceof Ability) && (AbilityUtilities.isFeat(obj))
			&& !Globals.checkRule(RuleConstants.FEATPRE))
		{
			// SUCK!  This is horrid, but bonusTo is actually recursive with respect to
			// passesPreReqToGain and there is no other way to do this without decomposing the
			// dependencies.  I am loathe to break working code.
			// This addresses bug #709677 -- Feats give bonuses even if you no longer qualify
			BonusCalc.dontRecurse = true;

			boolean returnZero = false;

			if (!po.qualifies(aPC, po))
			{
				returnZero = true;
			}

			BonusCalc.dontRecurse = false;

			if (returnZero)
			{
				return 0;
			}
		}

		int iTimes = 1;

		if (aPC != null && "VAR".equals(aType))
		{
			iTimes = Math.max(1, aPC.getConsolidatedAssociationList(po).size());
		}

		for (BonusObj bonus : aBonusList)
		{
			String bString = bonus.toString().toUpperCase();

			if (aPC != null && !aPC.getConsolidatedAssociationList(po).isEmpty())
			{
				int span = 4;
				int idx = bString.indexOf("%VAR");

				if (idx == -1)
				{
					idx = bString.indexOf("%LIST|");