Revival Protocol restores action points to disoriented units

[Iridar]

The blame is with the Revival Protocol blindly applying X2Effect_RestoreActionPoints to all targets.

The effect itself is poorly coded as well:

simulated protected function OnEffectAdded(const out EffectAppliedData ApplyEffectParameters, XComGameState_BaseObject kNewTargetState, XComGameState NewGameState, XComGameState_Effect NewEffectState)
{
	local XComGameState_Unit TargetUnit;

	TargetUnit = XComGameState_Unit(kNewTargetState);
	if (TargetUnit != none)
	{
		while (TargetUnit.NumActionPoints(class'X2CharacterTemplateManager'.default.StandardActionPoint) < class'X2CharacterTemplateManager'.default.StandardActionsPerTurn)
		{
			TargetUnit.ActionPoints.AddItem(class'X2CharacterTemplateManager'.default.StandardActionPoint);
		}
	}
}

Arguably it should call TargetUnit.GiveStandardActionPoints() instead.

This bug and others is addressed by RM's [WotC] Revival Protocol Fixes.

[Iridar]

Closing this in favor of #1414, which covers more bugs.