[fixed]Lost Legacy forces selection from library
Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins
[fixed]Lost Legacy forces selection from library
by 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.
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
Reason: fixed
-
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
by 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.
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
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
by drool66 » 20 Nov 2021, 05:47
Fixed in e87950a
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
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
by 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.
Trying to squash some bugs and playtesting.
-
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
by 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)
(it's included in that commit because I made a change to new_global_tutor() with the IKO drafts that caused this bug)
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
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
by Korath » 20 Nov 2021, 23:16
It's there, complete with irritating byte-order-mark at the start of the commit message.
-
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
by 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.
---
Trying to squash some bugs and playtesting.
Trying to squash some bugs and playtesting.
-
Aswan jaguar - Super Tester Elite
- Posts: 8078
- Joined: 13 May 2010, 12:17
- Has thanked: 730 times
- Been thanked: 458 times
7 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 30 guests