Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#13318 closed defect (fixed)

SQ3: version 1.018 doesn't display ScumSoft Announcements

Reported by: Doomlazer Owned by: sluicebox
Priority: normal Component: Engine: SCI
Version: Keywords: original
Cc: Game: Space Quest 3

Description (last modified by Doomlazer)

tested in ScummVM version 2.5.1

In SQ3 version 1.018, Script::cue was changed to check that scripts have a client.

Unfortunately, "announce" script in 702, the ScummSoft office region, doesn't have a client, so the 702.txt lines 7 and 8 are never displayed.

Attachments (2)

702Msg7.png (109.7 KB ) - added by Doomlazer 2 years ago.
702Msg8.png (109.5 KB ) - added by Doomlazer 2 years ago.

Download all attachments as: .zip

Change History (8)

by Doomlazer, 2 years ago

Attachment: 702Msg7.png added

by Doomlazer, 2 years ago

Attachment: 702Msg8.png added

comment:1 by Doomlazer, 2 years ago

Summary: SQ3 1.018 doesn't display ScummSoft AnnouncementsSQ3: version 1.018 doesn't display ScummSoft Announcements

comment:2 by Doomlazer, 2 years ago

Description: modified (diff)

comment:3 by Doomlazer, 2 years ago

Description: modified (diff)

comment:4 by sluicebox, 2 years ago

Component: --Unset--Engine: SCI
Keywords: original added; 1.018 removed

Great catch! I agree with your findings, this is an unintentional regression from upgrading to a newer version of the Script class. This affects all subsequent versions, so the localized ones too.

The announcement messages only randomly happen twice per game, at most, which explains why their disappearance didn't get noticed. I don't see why we shouldn't fix this and restore the corporate ambiance. I'll do it after the SQ3 thing I'm working on now.

comment:5 by sluicebox, 2 years ago

Summary: SQ3: version 1.018 doesn't display ScummSoft AnnouncementsSQ3: version 1.018 doesn't display ScumSoft Announcements

just gonna... go ahead and fix that title for you... =)

comment:6 by sluicebox, 2 years ago

Owner: set to sluicebox
Resolution: fixed
Status: newclosed

Fixed in https://github.com/scummvm/scummvm/commit/cd1639019874e2c8e8ad209aa2cfb9f26c0076f3

Thanks for reporting!

Also, this script patch doesn't fix saved games where you've a already entered the office. Those saves are already tainted and the messages probably won't ever appear, at least not for a few years. So don't use those to test or you'll have a long wait! =)

Last edited 2 years ago by sluicebox (previous) (diff)
Note: See TracTickets for help on using tickets.