[Rseding91] [2.0.42] Incomplete recipe-category migration

This subforum contains all the issues which we already resolved.
lyvgbfh
Fast Inserter
Fast Inserter
Posts: 175
Joined: Fri Jul 10, 2020 6:48 pm
Contact:

[Rseding91] [2.0.42] Incomplete recipe-category migration

Post by lyvgbfh »

The attached save fails to load after a recipe-category removal, even if a migration is written.

To reproduce, use the attached version of pyalienlife (it is only modified to have a migration added) and sync the attached save.

Boskid: "CustomChooseElemButton stores recipe id's not as ID's but as property tree, and those are not covered by migrations"

For the conversation in the playtesting discord that led to this report, see here
Attachments
factorio-current - Copy (3).log
(87.22 KiB) Downloaded 34 times
3.0.43.json
it is 3.0.36.json in the log
(344 Bytes) Downloaded 24 times
pyalienlife_3.0.43.zip
(1.3 MiB) Downloaded 17 times
py-migrated-2.zip
(11.27 MiB) Downloaded 16 times
Rseding91
Factorio Staff
Factorio Staff
Posts: 15251
Joined: Wed Jun 11, 2014 5:23 am
Contact:

Re: [2.0.42] Incomplete recipe-category migration

Post by Rseding91 »

I started looking into this, and it seems like it's gong to be a lot more complicated to fix than I originally thought.

Would it be acceptable that if loading migrates something - which would cause an error in these filters - to simply clear the filters and require the mod to re-set them?
If you want to get ahold of me I'm almost always on Discord.
lyvgbfh
Fast Inserter
Fast Inserter
Posts: 175
Joined: Fri Jul 10, 2020 6:48 pm
Contact:

Re: [Rseding91] [2.0.42] Incomplete recipe-category migration

Post by lyvgbfh »

I'm not sure I fully understand the cause, is this a LuaPrototypeFilter stored in storage/global? If so, I think clearing it would be fine, though a log warning would be appreciated.
Rseding91
Factorio Staff
Factorio Staff
Posts: 15251
Joined: Wed Jun 11, 2014 5:23 am
Contact:

Re: [Rseding91] [2.0.42] Incomplete recipe-category migration

Post by Rseding91 »

It’s a LuaProrotypeFilter stored in the map save file.
If you want to get ahold of me I'm almost always on Discord.
lyvgbfh
Fast Inserter
Fast Inserter
Posts: 175
Joined: Fri Jul 10, 2020 6:48 pm
Contact:

Re: [Rseding91] [2.0.42] Incomplete recipe-category migration

Post by lyvgbfh »

What kind of effect does the clearing of it have to the end user?
Rseding91
Factorio Staff
Factorio Staff
Posts: 15251
Joined: Wed Jun 11, 2014 5:23 am
Contact:

Re: [Rseding91] [2.0.42] Incomplete recipe-category migration

Post by Rseding91 »

The visible options shown in the GUI when a player clicks the custom choose elem button.
If you want to get ahold of me I'm almost always on Discord.
lyvgbfh
Fast Inserter
Fast Inserter
Posts: 175
Joined: Fri Jul 10, 2020 6:48 pm
Contact:

Re: [Rseding91] [2.0.42] Incomplete recipe-category migration

Post by lyvgbfh »

That shouldn't be a big deal to work around if it's cleared, thanks!
Post Reply

Return to “Resolved Problems and Bugs”