Opened 12 years ago

Closed 12 years ago

Last modified 2 years ago

#6160 closed defect (fixed)

SCI: QfG3: Both the player and Yesufu are declared winners

Reported by: bluegr Owned by: sluicebox
Priority: normal Component: Engine: SCI
Version: Keywords: script
Cc: Game: Quest for Glory 3

Description

This is a resubmission of bug #6154 submitted to the allura tracker by user CaptainJei: https://sourceforge.net/p/scummvm/bugs/6154/

I am playing as a paladin and competing with Yesufu in the Initiation contest. During the first contest (the Twisted Tree puzzle), I take the vine and click in on the spear. The hero throws two spears and is declared the winner by the elder. Following that Yesufu runs up to the elder and is also declared the winner. No points are scored. I have attached a save game. All you need to do is click the vine and then click the spears with the vine to reproduce it. ScummVM version: 1.6.0git1853-gf2fa69 (Oct 1 2012 10:27:59) Language: English Version: floppy Platform: Windows 7

Ticket imported from: #3575570. Ticket imported from: bugs/6160.

Attachments (1)

qfg3.000 (66.2 KB ) - added by bluegr 12 years ago.

Download all attachments as: .zip

Change History (4)

by bluegr, 12 years ago

Attachment: qfg3.000 added

comment:1 by bluegr, 12 years ago

Owner: set to bluegr
Resolution: fixed
Status: newclosed

comment:2 by bluegr, 12 years ago

Many thanks for the bug report. This was an issue with an incorrect workaround for an uninitialized script variable. Fixed in commit dbb5021 - the fix should be available in the next daily build

comment:3 by sluicebox, 2 years ago

Owner: changed from bluegr to sluicebox

This is now fixed with a script patch: https://github.com/scummvm/scummvm/commit/f45a5eff90af49a4295583d0c23f0a449bf8fb49

The problem with this and ticket #5277 was that using a workaround hard-coded the winner. Changing the value only changed who the winner would always be. A script patch is necessary to properly store and retrieve the dynamic value.

Note: See TracTickets for help on using tickets.