Feature Unequip selling shorts and restore previous pants.

[fronobulax]

Thread 'r29359 - What's Changed feat: equip selling shorts when autoselling by @midgleyc in #3848 Full Changelog: r29358...r29359'

Saturday at 5:42 PM

What's Changed​

• feat: equip selling shorts when autoselling by @midgleyc in #3848

Full Changelog: r29358...r29359

View on Github

• Github Bot
• Replies: 0
• Forum: Github Changes

AFAIK the decision not to restore the previous outfit when auto-equipping the selling shorts was deliberate. But I have found a case where the assumption that autosell preserved my outfit is baked into my "workflow".

I finish my day by putting on my pajamas. Sometimes I manage my store which will autosell some items. I am left wearing the selling shorts, not my plus adventure pants, as expected.

Posting this to see whether other people would also like a restore or whether I just need to make changes.

 

[Ryo_Sangnoir]

Adding a checkpoint around it is easy enough and shouldn't break anything I think.

 

[fronobulax]

Adding a checkpoint around it is easy enough and shouldn't break anything I think.

Does save/restore a checkpoint cost any more than taking off/putting on pants?

 

[Ryo_Sangnoir]

It's our standard way of restoring equipment.

Mafia will check everything created at the point the checkpoint was created, but only the pants should have changed, so it should only restore the previous pants in practice.