Grimrock

Is there a list of bugs in LoG2 2.2.4 ?
( similar to https://github.com/JKos/log2doc/wiki )

I'm asking because I just (re)discovered a bug with
ItemComponent.onEquipItem(self, champion, slot)

If you pick up an item with the mouse cursor, and then click on a champion portrait, the item is inserted into the inventory, but onEquipItem() isn't called.

Zo Kath Ra wrote:If you pick up an item with the mouse cursor, and then click on a champion portrait, the item is inserted into the inventory, but onEquipItem() isn't called.

If you manually drop the item into a backpack slot (or through the portrait) - the equip hooks are not called.
Pretty sure onEquipItem & onUnequipItem are only called if items are placed into the "equipped" item slots 1 - 12

... but definitely a great idea to list up known bugs (and provide possible solutions)

I know minmay has listed and detailed many bugs for a number of areas / components in his InfoDump
viewtopic.php?f=22&t=13948

Im trying to find work arounds for RunePanel & CastSpell components currently
(cut / pasted from minmays infoDump ...)

One would think that the equipItem hooks would only fire if the item was equipped, but in practice, they fire when the item occupies any of the 32 inventory slots... except if they were dropped on the portrait; as mentioned in the OP. This is a bit of a pain.

It is possible to place an if condition in the hook, that detects if the slot is not one of the on-body inventory slots. This could effectively make the hook only act when the item is on the body.

There is another bug with these hooks that happens when a connector is attached to either hook for a spawned object. When equipped, or unequipped, the game crashes for not finding the object's map component.

Isaac wrote:One would think that the equipItem hooks would only fire if the item was equipped, but in practice, they fire when the item occupies any of the 32 inventory slots... except if they were dropped on the portrait; as mentioned in the OP. This is a bit of a pain.
It is possible to place an if condition in the hook, that detects if the slot is not one of the on-body inventory slots. This could effectively make the hook only act when the item is on the body.
There is another bug with these hooks that happens when a connector is attached to either hook for a spawned object. When equipped, or unequipped, the game crashes for not finding the object's map component.

Ah yup totally my bad guys - I rushed a test without reading what my onEquipItem hook actually did...
it starts out with an if condition checking for the handslots 1 and 2 - just as Isaac mentioned That connector bug does sound annoying
Again, I think its a great idea to create a reference list for known bugs/solutions

- MonsterChargeComponent accumulates errors in its movement whenever it changes animations (up to the frame delay each time), making it a little slower than it should be.
- FrozenMonsterComponent resets the monster's materials when it ends, without applying material overrides again, so monsters with material overrides will appear to lose them when they freeze/unfreeze
- Destroying a SurfaceComponent that contains items will leave the items in a broken state
- The scripting interface sandbox can be escaped by various methods
- Odd values of stats and base power give you a smaller damage increase than even values of stats and base power (going from 10 to 11 strength is not as good as going from 11 to 12 strength)
- HealthComponents immune to the "physical" damage type are also completely immune to all melee, projectile, and firearm attacks
- On levels containing a WaterSurfaceComponent, various debugDraw features are only drawn to the upper left quadrant of the screen and the reflections
- An object that triggers one of its connectors while not on a map will crash
- Zhandul's Orb still claims to provide a bonus to fire spells, but doesn't
- The Assassin's Dagger still claims to have life leech, but doesn't
- Setting a monster's level to above 1 will give a bigger max health bonus than its current health bonus (so e.g. a monster at 100/100 health might end up at 200/400)
- In Isle of Nex, there is a script_entity that changes the world position of a turtle nest, but as turtle nests have minimalSaveState, this world position change does not persist across save/load
- I've encountered a very rare null pointer dereference at instruction 0x004b43af when creating entities.
- convertToItemOnImpact converts stacks to a single item, and if the item hits an obstacle, it can spawn the new item on the other side of the obstacle
- ItemComponent:getFuel() crashes
- onComputeCooldown doesn't work for RunePanelComponent or CastSpellComponent
- Editor copy and paste can delete existing objects or even crash if the pasted objects have replacesWall/replacesFloor
- If an entity id collision occurs in the editor, connectors will no longer display actions until the dungeon is reloaded (even after the collision is fixed)
- While using a camera other than the default one, saving will result in a bogus warning message
- The miniature dungeon editor preview window reports its size incorrectly (but the fullscreen preview is fine)
- ButtonComponent still triggers its onActivate hook when disabled
- Music cuts off samples when it loops
- math.chromacity() doesn't work from the user scripting interface
- string.count() and string.split() enter infinite loops if their second argument is the empty string
- If there is a pedestal behind the party, casting Meteor Storm will strike the pedestal with one of the projectiles, hurting the party
- ModelComponent:getReflection() and ModelComponent:setReflection() don't work
- The alchemy panel crashes if it attempts to create an item with a custom gfxAtlas
- The expRate property of EquipmentItemComponent always reports itself as 25% in the tooltip, even when it isn't
- TimerComponent:getTimerInterval() requires a number argument, even though it doesn't use it
- Connectors with invalid targets can taint savegames when activated
- SurfaceComponents with a normal not pointing straight up (because of rotation) will not correctly compute the place an item should go when placed on the surface with the mouse
- Double-sided materials are only lit properly on one side
- Refraction buffer doesn't include particles, leading to particles under water or behind portals being invisible

@minmay
That is a handy list to have.

- HealthComponents immune to the "physical" damage type are also completely immune to all melee, projectile, and firearm attacks

There seems to be a logic to this one, as all of these attacks are usually physical; are there any that aren't?

Yes, those attacks can have any damage type. See torch, ethereal_blade, venomfang_pick's secondary attack, meteor_hammer, lightning_bow's secondary attack, lightning_blade, fire_blade, ancient_claymore.

Now THAT is a list !!

Some of those bugs already have work arounds / fixes (eg - ButtonComponent still triggers its onActivate hook when disabled ) ...
These we could post as they are discussed.... and then for others there would be no fix other than implementing entire
reworkings (eg - The alchemy panel crashes if it attempts to create an item with a custom gfxAtlas) ...
for which we can provide links to other threads with the relevent mod files / asset packs
And I guess there would be some we can mark off the list easily as: No chance of fixing/preventing

Most of these are not that difficult to fix. The music looping bug, refraction buffer limitation, and null pointer dereference are probably things we are stuck with forever, however, since Lua can't get anywhere near the relevant code. The refraction buffer thing isn't a total deal breaker because you can use sortOffset to force particles to appear in front of refractive materials when necessary, and the null pointer crash is extremely rare (I haven't tried to reverse engineer the executable because that would be insane, but my guess is that a library is doing something dumb).

I found a very weird bug yesterday when testing the preview of Eternal Recurrence. On reloading the saved game, the game crashed with an error saying an object was referencing another object and it could not find it. I inspected the save file with my save editor tool and found the objects with the logged ids. It was a chest and a lock pick, the lock pick was in my inventory and the chest still had it listed as inside the chest... What the hell ??? I spawned randomly 2 to 4 items in chests in the previous version, I reduced it to 2 to 3 and I did not get any similar error so far... Is this a bug of the game engine ?