chseet_fantasy_combined has hardcoded Rage information

Description

SF ID: 1818896
Last Update: Settings changed ( eddyanthony )
Details:

I'm seeing lots of hardcoding of values for Barbarian Rage stuff all over the place, instead of referencing the VARs from the LST files.

I'm going to tackle some basics, but I'm going to need a better OS monkey to go behind me and polish it.
Submitted:

Andrew McDougall ( tir-gwaith ) - 2007-10-23 19:12:10 EDT

Date: 2008-12-05 08:30:51 EST
Sender: kariannaSourceForge.net DonorProject Admin & DonorAccepting Donations
Hide

Bounce

Date: 2008-02-08 10:59:45 EST
Sender: zaisterSourceForge.net Subscriber and DonorProject Donor
Hide

Barbarian Rage stuff also seems to be hardcoded into the preview tab

Date: 2008-02-08 10:49:01 EST
Sender: kariannaSourceForge.net DonorProject Admin & DonorAccepting Donations
Hide

How far did you get with this Tir? I should be able to give you a hand..

Environment

None

Activity

Show:
Andrew Maitland
December 11, 2011, 3:37 AM

Kar - you able to investigate this at all?

Andrew Maitland
September 11, 2012, 5:23 PM

function RagePrint()
{

%VAR.RageTimes.GTEQ.1

var strMod = |VAR.RageStrBonus|;
var conMod = |VAR.RageConBonus|;
var willMod = |VAR.RageMorale|;
tempHTML = '<br /><table width="100%" cellspacing="0" cellpadding="2"><tr><td class="c9wB" colspan="2">BARBARIAN RAGE<\/td><\/tr>';
tempHTML = tempHTML+'<tr><td colspan="2" class="l9s">The Barbarian temporarily gains 'modPlusMinus(strMod)' to Strength, 'modPlusMinus(conMod)' to Constitution, and a 'modPlusMinus(willMod)' morale bonus on Will saves, but suffers a -2 penalty to AC. ';
// Do it this way since we want to show the same length regardless of current rage state
tempHTML = tempHTML+'A fit of rage lasts for ';
if ( RageStatus == "Raging" )
tempHTML = tempHTML+(3+AbArr[CONindex].tempmodifier)+' rounds. ';
else
tempHTML = tempHTML+(3+AbArr[CONindex].tempmodifier+conMod/2)+' rounds. ';
if ( |VAR.BarbarianRageLVL| < 20 )
tempHTML = tempHTML+'At the end of the rage, the barbarian is fatigued (-2 to Strength, -2 to Dexterity, can&rsquo;t charge or run) for the duration of that encounter. ';
else
tempHTML = tempHTML+'A 20th level Barbarian is NOT fatigued after leaving a rage. ';
tempHTML = tempHTML+'The barbarian can only fly into a rage once per encounter, and only |VAR.RageTimes.INTVAL| times per day (determined by level). Entering a rage takes no time itself, but the barbarian can only do it during his action.<\/td><\/tr><\/table>';
document.getElementById('RageDiv').innerHTML = tempHTML;

%

Andrew Maitland
September 11, 2012, 5:28 PM

These are the changes made in the base.xml - just advise how to alter the script above using this - I'm thinking we should be able to throw this in there in place of the text:

ABILITYALL.Special Ability.0.TYPE=RageDescription.ASPECT.RageDescription

Here is the full base.xml text I tweaked. NOTE: It has backwards compatibility.

<!-- Pathfinder -->

%VAR.RageLVL.GTEQ.1

<!-- If character can Rage -->
<rage>
<uses_per_day>

VAR.RageDuration.INTVAL

</uses_per_day>
<uses_per_day.title>Rounds/day</uses_per_day.title>
</rage>

%

<!-- Character Rage -->

%VAR.RageTimes.GTEQ.1

<!-- If character can Rage -->
<rage>
<uses_per_day>

VAR.RageTimes.INTVAL

</uses_per_day>
<uses_per_day.title>Uses per day</uses_per_day.title>

IIF(ABILITYALL.Special Ability.0.TYPE=RageDescription.HASASPECT.RageDescription:Y)
<description>

ABILITYALL.Special Ability.0.TYPE=RageDescription.ASPECT.RageDescription

</description>

ELSE

<description>The Barbarian gains +|VAR.RageStrBonus.INTVAL| to Strength, +|VAR.RageConBonus.INTVAL| to Constitution, and a +|VAR.RageMorale.INTVAL| morale bonus on Will saves, but suffers a -2 penalty to AC for |VAR.RageConBonus.INTVAL+3| rounds. At the end of the rage, the barbarian is fatigued (-2 to Strength, -2 to Dexterity, can't charge or run) for the duration of that encounter. The barbarian can only rage once per encounter. Entering a rage takes no time itself, but the barbarian can only do it during his action.</description>

ENDIF

</rage>

Andrew Maitland
October 27, 2012, 6:34 PM

Completed: At revision: 18027

Assignee

Andrew Maitland

Reporter

David Bender

Labels

None

Epic/Theme

None

Pending User Input

None

Fix versions

Priority

Major
Configure