mluzarow / mluzarow/MethHelper

Crashing in cook off after 5-10 min

Open
#10 4 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Lua
Stars
5
Forks
2
PR merge metrics
No merged PRs in 30d

Description

Crash log:

Sun Aug 23 17:26:49 2020

Application has crashed: C++ exception
mods/MethHelper-v1.1.0/lua/MethHelper.lua:126: attempt to concatenate a nil value

SCRIPT STACK

queue_dialog() @mods/MethHelper-v1.1.0/lua/MethHelper.lua:220
queue_narrator_dialog() lib/managers/dialogmanager.lua:99
sync_end_assault() lib/managers/hudmanagerpd2.lua:1127
lib/network/handlers/unitnetworkhandler.lua:2897

=================================================
Sun Aug 23 17:32:19 2020

Application has crashed: C++ exception
mods/MethHelper-v1.1.0/lua/MethHelper.lua:126: attempt to concatenate a nil value

SCRIPT STACK

queue_dialog() @mods/MethHelper-v1.1.0/lua/MethHelper.lua:220
queue_narrator_dialog() lib/managers/dialogmanager.lua:99
sync_start_assault() lib/managers/hudmanagerpd2.lua:1112
lib/network/handlers/unitnetworkhandler.lua:2887

==============================================
I'm using Gurkoel's fork.
Line 126: Line 126: managers.chat:send_message(1, '[SilentMethMagic]', '[' .. countAddedIngredients(currentRecipeList) .. '/3] [' .. ingredient_dialog[id] .. ']', Color.green)

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at mods/MethHelper-v1.1.0/lua/MethHelper.lua:126 and inspect how queue_dialog() at line 220 supplies countAddedIngredients(currentRecipeList) and ingredient_dialog[id]. Follow the related dialogmanager.lua, hudmanagerpd2.lua, and unitnetworkhandler.lua stack entries while reproducing the crash in Cook Off. Done means the reported nil concatenation crash no longer occurs during the affected assault transitions.

Written by the indexing model from the issue text.

Assessment

Tech stack
lua
Domain
game-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
32/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.