Skip to content

fixed mod variants not being created for binary patched mods#19218

Merged
IDCs merged 4 commits intomasterfrom
fix/19184
Jan 8, 2026
Merged

fixed mod variants not being created for binary patched mods#19218
IDCs merged 4 commits intomasterfrom
fix/19184

Conversation

@IDCs
Copy link
Contributor

@IDCs IDCs commented Jan 5, 2026

The addModRule reducer was ignoring the tag identifier when installing new collections - this would cause the rule to be replaced on previous collections if the mod reference would match (which it did as the tag was being replaced).

Only reason why this wasn't a problem in past versions is because we used to prompt the user if he wants a new variant of the matched mod.

testModReference would still fail under any other conditions when trying to differentiate between two binary patched versions of the same mod.

fixed the annoying re-download for externally sourced mods as a bonus.

Apparently it fixed collection rule issues when applying to bundled mods too.

fixes #19184
fixes #18878
fixes #18977

only happens if you're installing a mod which has no patches, and
already have the same mod installed (but with patches)

fixes #19184
@IDCs IDCs requested a review from a team January 5, 2026 15:41
@IDCs IDCs self-assigned this Jan 5, 2026
@IDCs IDCs marked this pull request as draft January 7, 2026 09:06
The addModRule reducer was ignoring the tag identifier when
installing new collections - this would cause the rule to be
replaced on previous collections if the mod reference would match.

Only reason why this wasn't a problem in past versions is because we
used to prompt the user if he wants a new variant or replace the matched
mod.

testModReference would still fail under any other conditions when trying
to differentiate between two binary patched versions of the same mod.

fixed the annoying re-download for externally sourced mods as a bonus.

fixes #19184
fixes #18878
@IDCs IDCs merged commit 9255549 into master Jan 8, 2026
2 checks passed
@IDCs IDCs deleted the fix/19184 branch January 8, 2026 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment