![Image](https://forums.factorio.com/images/ext/96fb134804750fefe92f550f8b7f3e90.png)
[0.18.22] Improve error message about wrong next_upgrade
- SuperSandro2000
- Filter Inserter
- Posts: 742
- Joined: Sun Jan 12, 2014 3:54 am
- Contact:
[0.18.22] Improve error message about wrong next_upgrade
I think I get what this message means but it is not obvious on the first or second read. It could get some love and improvement in the wording.
![Image](https://forums.factorio.com/images/ext/96fb134804750fefe92f550f8b7f3e90.png)
![Image](https://forums.factorio.com/images/ext/96fb134804750fefe92f550f8b7f3e90.png)
Please call me simply Sandro.
My Main Mods: Sandro's fixes, Expanded Rocket Payloads Touched by an AngelBob and more can be found here
![Image](http://steamsignature.com/profile/default/76561198169442639.png)
My Main Mods: Sandro's fixes, Expanded Rocket Payloads Touched by an AngelBob and more can be found here
![Image](http://steamsignature.com/profile/default/76561198169442639.png)
Re: [0.18.22] Improve error message about wrong next_upgrade
That's one of the more verbose messages in the game. I don't really know how it could be made "better" (aside from removing the result result typo).
Entity must produce no item result or mine to an item that isn't hidden.
What exactly is missing from that?
Also, this isn't a bug but a suggestion.
Entity must produce no item result or mine to an item that isn't hidden.
What exactly is missing from that?
Also, this isn't a bug but a suggestion.
If you want to get ahold of me I'm almost always on Discord.
Re: [0.18.22] Improve error message about wrong next_upgrade
That hurts my head a little bit ... but also it seems wrong? The mining result must have exactly one item product, and that item must not be hidden. Or maybe the check in code is wrong and it should allow entities that don't have any item product in their mining result?
Re: [0.18.22] Improve error message about wrong next_upgrade
The check in the code does allow that. That's what the error says: it must produce nothing, or must produce an item that isn't hidden.posila wrote: ↑Wed May 06, 2020 5:41 pmThat hurts my head a little bit ... but also it seems wrong? The mining result must have exactly one item product, and that item must not be hidden. Or maybe the check in code is wrong and it should allow entities that don't have any item product in their mining result?
If you want to get ahold of me I'm almost always on Discord.
Re: [0.18.22] Improve error message about wrong next_upgrade
As far as I can reason about the code, the check doesn't allow for "produce nothing".
The check is
and hidden() does
Must be minable, must have exactly 1 product, which must be item, which must be visible.
And I didn't see any early exit that would skip this check entirely.
The check is
Code: Select all
if (prototype->minableProperties.hidden())
Code: Select all
bool MinableProperties::hidden() const
{
return this->minable &&
this->products.size() == 1 &&
!this->products.front()->getItemID().isZero() &&
!this->products.front()->getItemID().getPrototype()->isVisible();
}
And I didn't see any early exit that would skip this check entirely.
Re: [0.18.22] Improve error message about wrong next_upgrade
If the thing doesn't have any item products when mined ::hidden() will return false and the code will continue.
On the next line it checks if !minable and if it is, then either it has zero items when mined or produces a non-hidden item.
On the next line it checks if !minable and if it is, then either it has zero items when mined or produces a non-hidden item.
If you want to get ahold of me I'm almost always on Discord.
Re: [0.18.22] Improve error message about wrong next_upgrade
*facepalm*, yep, you are correct.
Re: [0.18.22] Improve error message about wrong next_upgrade
What item conflicts exactly? where did the error occur? Help me help you help us all.