Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Assumption that ingredients[1][1] exists (repost from discussions) #52

Closed
jacobwatkinsgit opened this issue Feb 23, 2020 · 1 comment · Fixed by #54
Closed

[BUG] Assumption that ingredients[1][1] exists (repost from discussions) #52

jacobwatkinsgit opened this issue Feb 23, 2020 · 1 comment · Fixed by #54
Labels
Angels Lib override functionality related Impact: Bug Something could be wrong

Comments

@jacobwatkinsgit
Copy link

Describe the bug
Error on start-up.:

  27.933 Mods to disable:Failed to load mods: __angelsindustries__/data-final-fixes.lua:24: ..._/prototypes/overrides/industries-override-functions.lua:184: attempt to index field '?' (a nil value)
stack traceback:
	..._/prototypes/overrides/industries-override-functions.lua:184: in function 'core_tier_upgrade'
	...lsindustries__/prototypes/overrides/angels-tech-base.lua:200: in main chunk
	[C]: in function 'require'
	__angelsindustries__/data-final-fixes.lua:24: in main chunk
stack traceback:
	[C]: in function 'require'
	__angelsindustries__/data-final-fixes.lua:24: in main chunk

To Reproduce
Information to reproduce the behavior:

  1. Factorio 18.08?
  2. Modlist: Mobile Factory and Angel's Industries and whatever else
  3. Angel's Industries 0.3.1

Additional context
Mobile Factory has a "tier 0" recipe that has no ingredients, and completes when a new player/force is created (multiplayer). An assumption is made on line 184 that an ingredients [1][1] exists, which it will not for a recipe with no ingredients. The author is coding a workaround by having a fake ingredient, but the bug belongs to you.

TimVroom added a commit that referenced this issue Feb 23, 2020
@TimVroom TimVroom linked a pull request Feb 23, 2020 that will close this issue
@LovelySanta LovelySanta added Angels Lib override functionality related Impact: Bug Something could be wrong labels Feb 23, 2020
TimVroom added a commit that referenced this issue Feb 28, 2020
* Issue #52 - Checking if ingredients exist

* Issue #52 - Update version so it reflects properly
@TimVroom
Copy link
Collaborator

fine catch, should be fixed on new release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Angels Lib override functionality related Impact: Bug Something could be wrong
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants