Skip to content

Ignore duplicate access transformers#412

Merged
budak7273 merged 3 commits into
satisfactorymodding:dev-1.2from
Jarno458:fix_accesor_transformer_duplicates
May 1, 2026
Merged

Ignore duplicate access transformers#412
budak7273 merged 3 commits into
satisfactorymodding:dev-1.2from
Jarno458:fix_accesor_transformer_duplicates

Conversation

@Jarno458
Copy link
Copy Markdown
Contributor

@Jarno458 Jarno458 commented Apr 9, 2026

If two mods define the same accessor transformer, it will generate a duplicates causing builds to fail

found with the following access transformer in two different mods

Accessor=(Class="USubsystemActorManager", Property="SubsystemActors")

@budak7273 budak7273 moved this from 🆕 New to 👀 In review in Mod Loader (SML), Alpakit, ExampleMod, and Engine Apr 9, 2026
@budak7273 budak7273 changed the title Fix accesor transformer duplicates Ignore duplicate access transformers May 1, 2026
@budak7273
Copy link
Copy Markdown
Member

Would probably be better to do the Distinct in AccessTransformerTable.GetFor, so any future uses (if they ever exist) don't run into the same issues
But LGTM
https://discord.com/channels/555424930502541343/562722670974599227/1499459313750773851

Will merge once Jarno takes a look

Copy link
Copy Markdown
Contributor

@Th3Fanbus Th3Fanbus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, I like Robb's approach because of its simplicity.

@Jarno458
Copy link
Copy Markdown
Contributor Author

Jarno458 commented May 1, 2026

Tested works correct, also if multiple mods have the same unused transformer

@budak7273 budak7273 merged commit 3dba2d1 into satisfactorymodding:dev-1.2 May 1, 2026
1 check failed
@github-project-automation github-project-automation Bot moved this from 👀 In review to ✅ Completed in Mod Loader (SML), Alpakit, ExampleMod, and Engine May 1, 2026
@Jarno458 Jarno458 deleted the fix_accesor_transformer_duplicates branch May 2, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants