[11:48:05] * Dominus bows before wjp
[11:48:25] <Dominus> you were right in insisting we take a closer look
[11:48:28] <Dominus> :)
[12:07:37] <Dominus> so we'd need to modify usecode/intrinsics.cc:1956 USECODE_INTRINSIC(remove_item) to delete items in locked containers as well, right? Or does it need to search first and something else needs to be modified?
[12:10:03] <Dominus> hmm, or actually usecode/ucinternal.cc void Usecode_internal::remove_item
[13:10:00] <wjp> I'm trying to remember if BG/SI let you pay vendors with gold from inside locked containers
[13:29:45] <wjp> thanks for investigating, by the way
[13:39:14] <Dominus> wjp, I'm quite proud of so easily finding the regression ;)
[13:39:36] <Dominus> good point about vendors and locked containers
[13:40:07] <Dominus> I'll try later. should be quite easy to test
[15:35:21] <Dominus> wjp, you cannot pay vendors with gold from inside locked chests
[15:35:56] <Dominus> BUT I think that fails because it first checks if there is enough available gold before going to the deletion part
[18:37:04] <Dominus> Malignant, I wonder, did you check what happens with the unsealed box? because if opened the box the scroll gets correctly deleted (when I commented your ucinternal change)