It is currently 26 Apr 2024, 15:47
   
Text Size

[fixed]Lost Legacy forces selection from library

Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins

[fixed]Lost Legacy forces selection from library

Postby Korath » 02 Oct 2021, 16:57

Describe the Bug:
When I cast Lost Legacy to test name selection, naming Abjure, a card that isn't in the AI's deck, I'm continuously prompted to "Select a card named Abjure to exile." showing its library.

Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
dev 51b6a316a, duel.

What exactly should be the correct behavior/interaction?
If I click "Done", the dialog should go away, no matter what. Selecting a card when searching a hidden zone is never mandatory even if, unlike here, a selectable card is in it. (And even if the zone's currently revealed by some other effect.)

Are any other cards possibly affected by this bug?
It's plainly somewhere in lobotomy_effect(), not card_lost_legacy() itself.
Last edited by drool66 on 20 Nov 2021, 05:46, edited 2 times in total.
Reason: fixed
User avatar
Korath
DEVELOPER
 
Posts: 3707
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1106 times

Re: Lost Legacy forces selection from library

Postby drool66 » 03 Oct 2021, 06:33

I'm really sorry about this, Korath - it's a probationary change I made to new_global_tutor() for the Ikoria drafts in 73cc6a, specifically for Fire Prophecy I think. For the moment it returns -2 if a player can tutor but chooses not to, as opposed to -1 if he cannot tutor and therefore doesn't. lobotomy_effect() is one of the 80 or so places that compares the output of new_global_tutor() to != -1; comparing to >= 0 fixes this. I pushed earlier than I had intended because Aswan jaguar needed the addresses in ManalinkEh.asm for his cards. This will be cleared up before Ikoria is finalized.
User avatar
drool66
Programmer
 
Posts: 1163
Joined: 25 Nov 2010, 22:38
Has thanked: 186 times
Been thanked: 267 times

Re: [fixed]Lost Legacy forces selection from library

Postby drool66 » 20 Nov 2021, 05:47

Fixed in e87950a
User avatar
drool66
Programmer
 
Posts: 1163
Joined: 25 Nov 2010, 22:38
Has thanked: 186 times
Been thanked: 267 times

Re: [fixed]Lost Legacy forces selection from library

Postby Aswan jaguar » 20 Nov 2021, 08:12

Just a reminder that you didn't push this or any the other commits yet, so their id will change if someone else pushes a commit.
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8078
Joined: 13 May 2010, 12:17
Has thanked: 730 times
Been thanked: 458 times

Re: [fixed]Lost Legacy forces selection from library

Postby drool66 » 20 Nov 2021, 22:06

Oh shoot, I'm seeing them as pushed - commit is "Complete IKO et al drafts from 73cc6ad" [e87950a] 14.11.21. You don't see that one?
(it's included in that commit because I made a change to new_global_tutor() with the IKO drafts that caused this bug)
User avatar
drool66
Programmer
 
Posts: 1163
Joined: 25 Nov 2010, 22:38
Has thanked: 186 times
Been thanked: 267 times

Re: [fixed]Lost Legacy forces selection from library

Postby Korath » 20 Nov 2021, 23:16

It's there, complete with irritating byte-order-mark at the start of the commit message.
User avatar
Korath
DEVELOPER
 
Posts: 3707
Joined: 02 Jun 2013, 05:57
Has thanked: 496 times
Been thanked: 1106 times

Re: [fixed]Lost Legacy forces selection from library

Postby Aswan jaguar » 21 Nov 2021, 06:49

I am sorry, my bad I thought this was about a new commit and I didn't check the commit id as I should have done. :oops:
---
Trying to squash some bugs and playtesting.
User avatar
Aswan jaguar
Super Tester Elite
 
Posts: 8078
Joined: 13 May 2010, 12:17
Has thanked: 730 times
Been thanked: 458 times


Return to Archived Reports

Who is online

Users browsing this forum: No registered users and 30 guests

cron

Who is online

In total there are 30 users online :: 0 registered, 0 hidden and 30 guests (based on users active over the past 10 minutes)
Most users ever online was 4143 on 23 Jan 2024, 08:21

Users browsing this forum: No registered users and 30 guests

Login Form