Opened 4 years ago
Last modified 4 years ago
#12565 new feature request
PRIVATE: Casebook inaccesible
Reported by: | TrivialBalderdash | Owned by: | neuromancer |
---|---|---|---|
Priority: | low | Component: | Engine: Private |
Version: | Keywords: | ||
Cc: | Game: | Private Eye |
Description
The casebook on the menu/radio/phone/map screen does not allow for reviewing choices made during the game. Locations are shown, and inventory can be accessed using the tab at the top, but none of the pages in between can be viewed. In the corner location to turn the page, only a down arrow is displayed, which returns to the main screen. (See attachment)
This is not required to complete the game, and almost feels like a bonus feature. Though someone might want to review their case history as they are trying to solve the case.
Attachments (3)
Change History (12)
by , 4 years ago
Attachment: | Casebook.png added |
---|
by , 4 years ago
Attachment: | Casebook2.png added |
---|
comment:1 by , 4 years ago
comment:2 by , 4 years ago
You are right, this was never implemented and it is optional. Still, it's clear it is useful to review the case information during the game. I decided to prioritize the rest of the gameplay, since this feature is hardcoded in the binary and I had to understand how to how each decision is recorded (I suspect this is also hardcoded) and allow to replay each video.
by , 4 years ago
Attachment: | nodups.diff added |
---|
comment:3 by , 4 years ago
Can you try that patch? (You will need to restart your game, old saves won't be useful)
comment:4 by , 4 years ago
Ok, I gave this a try.
The inventory list in the casebook is blank. I can still collect inventory and it still is viewable in the cabinet in Marlowe's office. I played through part of the game, collecting more items, saving the game, reloading, then restarting ScummVM and reloading. In all cases the Inventory page of the casebook was blank.
Disclaimer: This is the first time I've attempted to compile ScummVM and also to apply a patch. My confidence in my ability to do this correctly is low.
I decided to use Linux (Debian). I cloned the repository -
git clone git://github.com/scummvm/scummvm.git
I downloaded the diff file to the directory just created and ran:
patch -p1 < nodups.diff
This did appear to update the funcs.cpp file.
Then:
./configure make clean make -j2
It seemed to complete correctly, since I could start ScummVM and run the game.
comment:5 by , 4 years ago
Yes, I can confirm this. Your procedure is correct, I will send you another (better) patch soon.
comment:6 by , 4 years ago
I identified the issue. It's failing to render some masks on master
, so I will commit there a fix soon.
comment:7 by , 4 years ago
I think this is fixed. To re-test from your scummvm cloned repository:
git checkout . git pull make -j2
Please let me know if the duplicated entries are gone (you will need to start a new game).
comment:8 by , 4 years ago
Success! This seems to have resolved the duplicate inventory entries. I tried several combinations of saves, reloads, and restarts, and everything was displaying correctly.
And thank you for the git instructions! I appreciate the coaching as I'm learning how this works.
comment:9 by , 4 years ago
Great, thanks for the testing. Please keep this issue open since the casebook is still not fully implemented.
Additionally, the list of locations links to the pages, as shown in the second attachment.