Disable MergeConsumables compat change until that mod implements Rollback()

This commit is contained in:
Tyfon
2024-08-22 12:18:54 -07:00
parent 495922a8e1
commit 3f8ef95685

View File

@@ -25,10 +25,14 @@ public class FixTraderControllerSimulateFalsePatch : ModulePatch
ref ItemOperation __result, ref ItemOperation __result,
bool __runOriginal) bool __runOriginal)
{ {
if (!__runOriginal) // TODO: The following commented code is necessary for compatibility with any mod that prefix patches and returns false,
{ // specifically MergeConsumables. However, that mod currently doesn't implement Rollback() on its operations,
return __runOriginal; // which breaks multi-select and has very nasty side effects.
} // 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;