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.

Sign up for GitHub

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

Jump to bottom

bug(core): gmcp.Char event handler attempt to index field 'State' (nil value) #39

Closed
iLPdev opened this issue Feb 17, 2023 · 2 comments
Closed

bug(core): gmcp.Char event handler attempt to index field 'State' (nil value) #39

iLPdev opened this issue Feb 17, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@iLPdev
Copy link
Owner

iLPdev commented Feb 17, 2023

on load of clean profile running v1.0.0, I receive...

[ LUA ] - error in event handler for gmcp.Char:
<[string "Script: prs-core"]:81: attempt to index field 'State' (a nil value)>

can anyone replicate? @eterchun?
@iLPdev iLPdev added the bug Something isn't working label Feb 17, 2023
eterchun added a commit to eterchun/prs that referenced this issue Feb 17, 2023
@eterchun
Fix for iLPdev#39 - adding checks before processing
550c22c 
Adding checks before processing to avoid errors in logs.
@eterchun
Copy link
Collaborator

Apparently I fixed this locally after submitting the code. We get the same effect but avoid the error messages by adding checks at the top for the existence of gmcp.Char.State, etc.

iLPdev pushed a commit that referenced this issue Feb 17, 2023
@eterchun
Fix for #39 - adding checks before processing (#40)
b4e4d40 
Adding checks before processing to avoid errors in logs.
@iLPdev
Copy link
Owner Author

iLPdev commented Feb 17, 2023

closed by #40

@iLPdev iLPdev closed this as completed Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@eterchun @iLPdev