- changed status to open
Parse weapsOn field
Website should not display the whole 'combat' box when a tower has no attack. Having a check for abilFac == 0, might not be optimal, since there are towers that have no damage but still attack, so if you could try locate the weapsOn field in the xml files and use that, would be cool. :D
This would work in a similar way to the 'magic' box, which doesn't appear when a tower doesn't use mana.
Comments (11)
-
repo owner -
repo owner - changed status to resolved
removing combat box for towers without an attack | closes
#10→ <<cset 472b605be99c>>
-
reporter General Error SQL ERROR [ mysql4 ]
Incorrect integer value: '' for column 'weapsOn' at row 1 [1366]
An SQL error occurred while fetching this page. Please contact the Board Administrator if this problem persists.
-
reporter - changed status to open
Li'l bug :P
-
reporter http://eeve.org/~glowackos/User_Towers/The Steam Engine.twrarc Also, if I may request permission to remove my own posts, I'd appreciate it. :)
-
repo owner Error should be fixed. But the problem is in your twrarc the weapsOn field is not present.
-
reporter oh :( well my tower has no attack, and i just checked in the object editor and weapsOn is 0 for it by default.. (same in tower creation map) I'm guessing GMSI only puts changed values in the xml, so if the value is not present, i think website should default to no combat box. (Hopefully, older tower versions didn't use a different base unit for the tower, as this could lead to inconsistencies..)
-
repo owner This tower ( http://www.eeve.org/YouTD/viewresource.php?tid=669 ) has weapsOn not set in the file. We need to look into it how GMSI handles this. If you add the value by hand it works : )
-
reporter The tower you mentioned has no attack. I dled it and checked, it's weapsOn field is 0 (default) and when parsed with GMSI, has no attack in the testmap. I am unsure why it even has an ability factor set.. The update to it (the one that is in the map) has weapsOn set to 1.
-
reporter However GMSI still bothered balancing its damage and everything, so I guess I'll chop that out of the code as its misleading and may confuse people parsing their towers for the first time.
-
repo owner - changed status to resolved
default value for weapsOn set to 0 | closes
#10→ <<cset 36cd9e88f456>>
Note: Reparsed every tower with a script.
- Log in to comment
Will look into it.