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

Persistance of lists does not work #33

Open
shrnjad opened this issue Apr 22, 2021 · 6 comments
Open

Persistance of lists does not work #33

shrnjad opened this issue Apr 22, 2021 · 6 comments
Labels
bug Something isn't working

Comments

@shrnjad
Copy link

shrnjad commented Apr 22, 2021

Describe the bug
When adding an entry to a list, the change will not be persisted.

To Reproduce
Start the attached project. Click on 'Add entry to list' -> Click on 'Save Ink' -> Close program and reopen -> Click on 'Load Ink' and observe the text in the label.

Expected behavior
After the steps outline above, the label should say that the entry is contained in the list.

Ink files
test.ink in attached project

Environment:

  • OS: Windows, also observed in web-browsers on Windows, mobile devices and Mac
  • Godot version: 3.2.3
  • Inklecate version: 0.9.0 (I think)
  • inkgd version: 0.2.1

Additional context
It can be fixed by _story._get_runtime().dont_save_default_values = false

Attached Godot project
inkgd_issue.zip

@ephread ephread added the bug Something isn't working label Nov 27, 2021
@ephread
Copy link
Owner

ephread commented Dec 22, 2021

@shrnjad thank you so much for reporting the issue and attaching a project.

Sorry it took so long to tackle this. I believe the issue is fixed in 0.4.0, would you be able to double check? The attached project seems to behave properly.

@shrnjad
Copy link
Author

shrnjad commented Jan 11, 2022

@ephread So sorry for the late reply. We are currently in the early stage of our current project (Hauma again). I would test it a bit later when I get around to implementing persistance if that is ok and we are not in a hurry.

@shrnjad shrnjad changed the title Persistance of losts does not work Persistance of lists does not work Jan 11, 2022
@shrnjad
Copy link
Author

shrnjad commented Jan 25, 2022

So I tried to test with version 0.4.2 but could not get the test project to run. (I downloaded it from the link in this issue)

It is also not obvious what the problem is while debugging. The problem happens at this line:

grafik

It runs into some recursive error that happens at this place in ink_list.gd:
grafik

Has the interface for lists changed? Or what could be broken?

@ephread
Copy link
Owner

ephread commented Jan 25, 2022

It's my fault, types are broken, I was overzealous in adding them. Thanks for uncovering the issue! I wish GDscript would support optionals.

It runs into some recursive error that happens

The recursive error is misleading, it happens because the debugger keeps retrieving the value from the getter to display it in the inspector. I consider this a bug in Godot, but I've been too lazy to report it.

I'll fix it, publish a new version and keep you posted. Sorry for the inconvenience 😞

@shrnjad
Copy link
Author

shrnjad commented Jan 26, 2022

No worries, I am happy to support you! 💪

@ephread
Copy link
Owner

ephread commented Jan 26, 2022

Thanks!

Note to self, the recursive problem with the debugger is fixed in Godot 4 alpha 1!

SuccessKid

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants