Menu
▾
▴
[Pcgen-trackers_OutputSheets] [ pcgen-Output Sheet Bugs-1740513 ] Gap in Fantasy Combined OS
|
From: SourceForge.net <no...@so...> - 2007-06-23 14:40:53
|
Output Sheet Bugs item #1740513, was opened at 2007-06-20 19:28 Message generated for change (Comment added) made by karianna You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=748296&aid=1740513&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: Output Sheet Error >Group: To be Fixed for 5.12 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Phantom of Krankor (y2krankor) Assigned to: Martijn Verburg (karianna) Summary: Gap in Fantasy Combined OS Initial Comment: When outputting using the d20/fantasy_csheet_combined, there's a huge gap between Armor and Shield and Shield and Items entries on the left side. This appears in 1,2 and 3-4 page styles. ---------------------------------------------------------------------- >Comment By: Martijn Verburg (karianna) Date: 2007-06-23 15:40 Message: Logged In: YES user_id=252169 Originator: NO Will take a look myself ---------------------------------------------------------------------- Comment By: Chuck Pint (chuckpint) Date: 2007-06-23 14:40 Message: Logged In: YES user_id=901169 Originator: NO Ok, I'm able to reproduce the problem with both 5.10.2 and 5.12.0 RC3. This shows up with both the "combined" and "compact" Output Sheets. I'm been able to isolate it down to the fact the failing OS have "|FOR.0.100.1. (...) ,1|" while the OS that do not fail have "|FOR,%armor,0,COUNT[EQTYPE.Armor]-1,1,0|" or the like. The OS that have the first kind of FOR token also use it for weapons blocks as well. The weapons blocks output just fine. So I'm thinking that there is a strange interaction between the FOR termination and the ARMOR token. Also as a side note, the docs for the ARMOR token are wrong. The first parameter for ARMOR.x.y.z, is optional. There are many cases of ARMOR.%.NAME is the various OS. As noted in the docs, MERGEALL is assumed if not listed. I'm going to need a code monkey to look at the code and see what is broken here. I'm assigning this back to Karianna so it gets reassigned properly. ---------------------------------------------------------------------- Comment By: Phantom of Krankor (y2krankor) Date: 2007-06-22 17:40 Message: Logged In: YES user_id=407252 Originator: YES For simplicity (and better proof), I've rebuilt that PC with only SRD stuff. ---------------------------------------------------------------------- Comment By: Phantom of Krankor (y2krankor) Date: 2007-06-22 17:39 Message: Logged In: YES user_id=407252 Originator: YES File Added: Dwarf Fighter (SRD) 1.pcg ---------------------------------------------------------------------- Comment By: Phantom of Krankor (y2krankor) Date: 2007-06-22 17:38 Message: Logged In: YES user_id=407252 Originator: YES File Added: currentPC33544.htm ---------------------------------------------------------------------- Comment By: Chuck Pint (chuckpint) Date: 2007-06-22 01:39 Message: Logged In: YES user_id=901169 Originator: NO There are exactly 99 empty Armor and Shield listings embedded into the OS. There is a FOR loop of 1 to 100, but it should terminate if there are not more items of type "Armor" or "Shield". Those are showing up as the "gap". Could you post your character file as well? I'm wondering if there is something strange in that file. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=748296&aid=1740513&group_id=25576 |