[Pcgen-trackers_Data] [ pcgen-Data Bugs-2181111 ] RPGObjects - Blood and Relics - Clean Up

[SourceForge.net <no...@so...>]

Data Bugs item #2181111, was opened at 2008-10-19 20:14
Message generated for change (Settings changed) made by fluxxdog
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=750091&aid=2181111&group_id=25576

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Dataset needs updating/cleaning
Group: For 5.16.0
>Status: Closed
Resolution: Fixed
Priority: 5
Private: No
Submitted By: LegacyKing (amaitland)
Assigned to: Fluxxdog (fluxxdog)
Summary: RPGObjects - Blood and Relics - Clean Up

Initial Comment:
SEVERE Invalid BONUS has no value: 1|CHECKS|Reflex|Prescience
SEVERE BONUS was given invalid bonus: CHECKS|Reflex|Prescience
LSTERROR Illegal Token 'BONUS' 'CHECKS|Reflex|Prescience' for pcgen.cdom.inst.PCClassLevel Prophet of Fatima(1)
SEVERE Invalid BONUS has no value: 1|COMBAT|INITIATIVE|Prescience
SEVERE BONUS was given invalid bonus: COMBAT|INITIATIVE|Prescience
LSTERROR Illegal Token 'BONUS' 'COMBAT|INITIATIVE|Prescience' for pcgen.cdom.inst.PCClassLevel Prophet of Fatima(1)
SEVERE Invalid BONUS has no value: 1|SKILL|Prophecy|Prescience
SEVERE BONUS was given invalid bonus: SKILL|Prophecy|Prescience
LSTERROR Illegal Token 'BONUS' 'SKILL|Prophecy|Prescience' for pcgen.cdom.inst.PCClassLevel Prophet of Fatima(1)
SEVERE Unconstructed Reference: PCClass Outisder
All Done


----------------------------------------------------------------------

Comment By: Fluxxdog (fluxxdog)
Date: 2008-10-25 16:10

Message:
For the love of...  A PRE check?!?  I thought the tags were case sensitive.
 Case closed.


...  pun intended.

----------------------------------------------------------------------

Comment By: Andrew Wilson (nuance)
Date: 2008-10-25 10:25

Message:
That'll likely be because there is no PRE check for SCIENCE.

When we eventually implement PRE:SCIENCE as the way to say
that then it will understand that prescience is a variable and not#
a PRESCIENCE check.

There is a development spec for this chenge but I believe it's
slated for post 6.0


----------------------------------------------------------------------

Comment By: Fluxxdog (fluxxdog)
Date: 2008-10-25 10:02

Message:
Committed r8204
OK, this one makes no sense.  I've double and triple checked the syntax on
this, and it looked good AFAIK.  There should be no reason it's being
flagged as an error.  Can someone double check this and see if this might
be a bug or just a lack of documentation?  The class line in question is in
bnr2_class_prestige.lst as follows:

1	SAB:Prescience
+%|Prescience							BONUS:VAR|Prescience|(CL+1)/2	BONUS:CHECKS|Reflex|Prescience	BONUS:COMBAT|INITIATIVE|Prescience	BONUS:SKILL|Prophecy|Prescience	DEFINE:Prescience|0

It has been changed to:
1	SAB:Prescience
+%|Prescience							BONUS:VAR|Prescience|(CL+1)/2	BONUS:CHECKS|Reflex|var("Prescience")	BONUS:COMBAT|INITIATIVE|var("Prescience)	BONUS:SKILL|Prophecy|var("Prescience")	DEFINE:Prescience|0

When the var("") function was added, it worked fine, but the documentation
for the DEFINE tag would make it seem to not require this.  So we either
have a code bug (unlikely) or a doc bug (more probable, but...)

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=750091&aid=2181111&group_id=25576