Skip to content

Handle empty AutoplaceSpecification.control#18

Open
robertcampion wants to merge 1 commit into
KirkMcDonald:masterfrom
robertcampion:master
Open

Handle empty AutoplaceSpecification.control#18
robertcampion wants to merge 1 commit into
KirkMcDonald:masterfrom
robertcampion:master

Conversation

@robertcampion
Copy link
Copy Markdown

Version 2.0.21 added the following bugfix:

Fixed that agricultural tower would show all nauvis tiles as unplantable. Nauvis grass, dirt and red desert are now specifically plantable for tree seeds and highlight as green. (https://forums.factorio.com/122065)

The fix added an autoplace specification to the tree-plant prototype without a control field (code here). This is allowed since control is optional according to the API. However this causes nil to be passed as a key to the append function here, resulting in a "table index is nil" error.

This fix ignores the plant entity if its autoplace.control field is empty, which restores the previous behavior where tree-plant was ignored.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant