From: SourceForge.net <no...@so...> - 2008-11-03 21:16:48
|
Bugs item #1783640, was opened at 2007-08-28 22:24 Message generated for change (Settings changed) made by karianna You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384719&aid=1783640&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: Other Group: To be fixed For 5.16.0 Status: Open Resolution: None Priority: 7 Private: No Submitted By: LegacyKing (amaitland) >Assigned to: Tom Parker (thpr) Summary: Parenthesis in a formula caues parse mistake Initial Comment: You have exposed a code bug where a parenthesis in a formula in any ADD will cause a parse failure and force the formula to 1. TM, bug please. Please use the following as a workaround (though you can use a variable other than PERSONALFEATADDER if you wish) 1<tab>ADD:FEAT|PERSONALFEATADDER|Dodge,Track,Power Attack<tab>DEFINE:PERSONALFEATADDER|if(TL>1,3,1) TP. -- OP: Hi guys, I am trying to get a class to apply 3 feats at first level or a choice of one of the three if your character level is greater than 1. I have put this line into the class file. 1<tab>ADD:FEAT|if(TL=1,3,1)|FooFeat1,FooFeat2,FooFeat3 It seems to always evaluate the if() to 1. 1<tab>ADD:FEAT|if(TL=1,1,3)|FooFeat1,FooFeat2,FooFeat3 This line will still only give the choice of 1 feat. 1<tab>ADD:FEAT|3|FooFeat1,FooFeat2,FooFeat3 This line will give the choice of all three feats. Has anyone else had similar problems? OS: XP sp2 Java: 1.5 Thanks Craig - I think Equal as boolean operator under JEP needs to use "==" instead of "=". Try whether "if(TL==1,3,1)" works. Cheerio, Frank Kliewe PCGen Content Silverback - I tried your suggestion ... still no luck :( I also tried the following: 1<tab>ADD:FEAT|if(TL>1,3,1)|FooFeat1,FooFeat2,FooFeat3 1<tab>ADD:FEAT|if(TL<1,1,3)|FooFeat1,FooFeat2,FooFeat3 1<tab>ADD:FEAT|if(TL=classlevel("Fooclass"),3,1)|FooFeat1,FooFeat2,FooFeat3 No luck with any of these either. If I use the following: 1<tab>ADD:FEAT|TL+1|FooFeat1,FooFeat2,FooFeat3 I get a selection of 2 of the feats. Totally confused, as this should be fairly simple. ---------------------------------------------------------------------- Comment By: Martijn Verburg (karianna) Date: 2008-11-03 16:00 Message: What about 5.16 Tom, do we move this? ---------------------------------------------------------------------- Comment By: Tom Parker (thpr) Date: 2007-09-15 17:40 Message: Logged In: YES user_id=1037926 Originator: NO This will naturally "fix" in 6.0 - the parens are used as syntax detection (old vs. new) and the old syntax is used internally, which makes this very difficult to fix in 5.14 ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=384719&aid=1783640&group_id=25576 |