Description
Testmerges
582, 442, 545, 579, 559, 597, 610, 472
Round ID
3604
Description of the bug
Attempted to add ventcrawling to a Working Joe (which I had done previously, though it had been a while) and had a good few minutes of troubleshooting. Even after I had stripped down and added what few items were left to the canEnterVentWith list, I was still getting the same error message: "We cannot be carrying items or have items equipped when vent crawling!"
Ultimately found the culprit: while /atom/movable/emissive_blocker is included in human and human subtype mobs' contents list by default, it is not included within the canEnterVentWith list by default.
What's the difference with what should have happened?
The error message cites only held or equipped items, and the other default and potential contents that wouldn't display in a human mob's inventory (embryos, implants, limbs) are included in canEnterVentWith by default. So presumably the intent is to simply require a human mob to strip to take advantage of being granted the ventcrawl verb via GM/staff intervention, making the omission of the emissive blocker an unintended consequence of emissive lighting implementation.
How do we reproduce this bug?
- Spawn strip and discard any starting equipment (glasses, scarves etc)
- Add verb: /mob/living/proc/ventcrawl
- Attempt to ventcrawl
...
Issue Bingo
- Issue could be reproduced at least once
- Issue happened in a recent (less than 7 days ago) round
- Couldn't find an existing issue about this (https://github.com/cmss13-devs/cmss13/issues)