Opened 4 months ago

Closed 3 months ago

#15643 closed defect (outdated)

GUI: Load button is disabled even when saves are present

Reported by: ctoroman Owned by: digitall
Priority: high Component: GUI
Version: Keywords:
Cc: Game: Kyrandia 1

Description

ScummVM 2.10.0git11945-g7186f8e6a5b (Jan 1 2025 04:39:27)
OS: Windows 10

This happened after commits:
https://github.com/scummvm/scummvm/commit/979a17edc420ec945be284398271a292deda6624
https://github.com/scummvm/scummvm/commit/a86405389a405616e261fee21ba2b5e29b1a5851

(list_after_commit.jpg)

After the cancellation of these fixes, everything works fine.
(list_before_commit_1.jpg and list_before_commit_2.jpg)

Attachments (3)

list_after_commit.JPG (54.3 KB ) - added by ctoroman 4 months ago.
list_before_commit_1.JPG (52.2 KB ) - added by ctoroman 4 months ago.
list_before_commit_2.JPG (102.8 KB ) - added by ctoroman 4 months ago.

Download all attachments as: .zip

Change History (11)

by ctoroman, 4 months ago

Attachment: list_after_commit.JPG added

by ctoroman, 4 months ago

Attachment: list_before_commit_1.JPG added

by ctoroman, 4 months ago

Attachment: list_before_commit_2.JPG added

comment:1 by lotharsm, 4 months ago

Priority: normalhigh

comment:2 by digitall, 3 months ago

The screenshots provided show this occurring with Legend of Kyrandia CD/DOS/Russian. This may be an issue with the KYRA engine APIs for save/load state. Will try replicating...

comment:3 by digitall, 3 months ago

Component: --Unset--GUI
Game: Kyrandia 1

comment:4 by ctoroman, 3 months ago

Legend of Kyrandia is just an example. The problem is also with Gobliiins 5, Full Throttle...

comment:5 by criezy, 3 months ago

Note that the code causing this issue was reverted on January 2 as more issues were found (related to performances). I may revisit this feature, so this bug report is still useful as a reminder that there was an issue with the original code and to pay special care to this. But since the issue should no longer present in current code I think we can close this?

comment:6 by ctoroman, 3 months ago

If the problem is solved, then we can close this, of course. Thanks)

comment:7 by digitall, 3 months ago

@criezy: As this is "fixed" by removal of the code in the current master and was not included in the v2.9.0 release, am going to close this bug as fixed... though as you said, future reference if the code removed is restored in future with fixes.

@ctoroman: Thanks. Please reopen this bug if/when this occurs again and/or the code is re-added and this re-occurs.

comment:8 by digitall, 3 months ago

Owner: set to digitall
Resolution: outdated
Status: newclosed
Note: See TracTickets for help on using tickets.