Opened 11 years ago
Closed 10 years ago
#6357 closed defect (fixed)
DRACI: Object lost from inventory after saving
| Reported by: | SF/greencis | Owned by: | sev- |
|---|---|---|---|
| Priority: | normal | Component: | Engine: Draci |
| Version: | Keywords: | ||
| Cc: | Game: | Dragon History |
Description
WinXP 32bit, ScummVM 1.7.0git540-g49ec202 (Jun 15 2013 07:44:11) Android 2.3.6, ScummVM 1.7.0git540-g49ec202 (Jun 15 2013 06:13:14)
In Dragon History, you can lose an object from the inventory. That makes the game not completable.
How to replicate: 1) choose one object in the inventory and exit from the inventory; 2) call Global Main Menu and save the game; 3) quit from the game; 4) launch ScummVM again and load the savegame you made before. You'll see that the object you had selected before disappeared. And the game is not completable now!
I've attached savegames from English Dragon History. The first savegame was made before choosing an object (in my occasion - a cigar), the second one was made after choosing a cigar (there's no more cigar in the inventory).
Ticket imported from: #3614551. Ticket imported from: bugs/6357.
Attachments (2)
Change History (8)
by , 11 years ago
comment:1 by , 11 years ago
Replicated with latest git master on Linux x86_64 with attached savegames.
I think the issues is that the cursor object is not being saved/restored, or forced back to the inventory upon saving, thus it is lost. This is probably a bug present in the original, but would need to try the original interpreter under DOSBox to be sure.
comment:2 by , 11 years ago
Checked the Draci engine code and I think this is because _currentItem is not saved... https://github.com/scummvm/scummvm/blob/master/engines/draci/game.h#L362
Neither is there a check for NULL/empty and if not, a force to inventory prior to save as I indicated or a save (sync) here: https://github.com/scummvm/scummvm/blob/master/engines/draci/game.cpp#L1526
A patch should be possible, though not sure of the best solution here.
comment:3 by , 11 years ago
| Owner: | set to |
|---|
comment:6 by , 10 years ago
| Owner: | changed from to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
Savegame before choosing a cigar...