Re-enable MergeConsumables fix

This commit is contained in:
Tyfon
2024-08-30 13:40:00 -07:00
parent d481368cce
commit a7071cd572

View File

@@ -25,14 +25,10 @@ public class FixTraderControllerSimulateFalsePatch : ModulePatch
ref ItemOperation __result, ref ItemOperation __result,
bool __runOriginal) bool __runOriginal)
{ {
// TODO: The following commented code is necessary for compatibility with any mod that prefix patches and returns false, if (!__runOriginal)
// specifically MergeConsumables. However, that mod currently doesn't implement Rollback() on its operations, {
// which breaks multi-select and has very nasty side effects. return __runOriginal;
// Until that is fixed, leaving this as commented because while this breaks that mod, there are no game-breaking effects. }
// if (!__runOriginal)
// {
// return __runOriginal;
// }
TargetItemOperation opStruct; TargetItemOperation opStruct;
opStruct.targetItem = targetItem; opStruct.targetItem = targetItem;