Realmz-Castle / Realmz-Remake

1 stars 2 forks source link

Merging fixes from Samuel #1

Closed daerogami closed 2 months ago

daerogami commented 2 months ago

Issues I reported were as follows:

Campaign selection:

City of Bywater (one-character party):

Responses from Samuel included

the "rug dungeon floor" error is normal, it's from an old test i didnt remove, not a bug in the core. The "assigning " by clicking the bars next to the portraits is just how I coded, I could change it. Yea, i didnt up a max items limit (yet). the outline thing is def a bug i had ignored. The NPC ally used to work, I must have broke it since. Maybe I had moved that function to anther script ?

The combat non-walk-able floor and invalid index 90 ones are definitely bugs

the add_pc_or_npc_ally_to_battle_map thing was just a typo

that out of bound index error happens outside of combat too....

I found the problem with is_map_tile_walkable_by_char , the problem was in GameGlobals.find_path, when it checked if the destination point has a creature on it (making it the target) i temporarily unblocks its position and the rest of the area it covers ... but it was checking the CombatButton's (the sprite) size, not the combat button's creature 's size in tiles for y in range(who.creature.size.y) : used to have just who.size.y the turn ending when you do an invalid action like walking into a wall during combat was a sort of crash-proof for buggy AI, the problem was the decide action state not checking if your input led to a possible action before transitioning to the animation state fixed too