[fixed]Ferrovore wrong prompt menu
Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins
[fixed]Ferrovore wrong prompt menu
by Aswan jaguar » 17 Jul 2018, 12:59
Describe the Bug:
Ferrovore's prompt menu wrongly says "Select a card."
Which card did behave improperly?
Ferrovore
Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
Manalink dev 778ccb5 version - duel
What exactly should be the correct behavior/interaction?
Ferrovore's prompt menu is "Select an artifact." or "Select an artifact to sacrifice."
Are any other cards possibly affected by this bug?
-
Ferrovore's prompt menu wrongly says "Select a card."
Which card did behave improperly?
Ferrovore
Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
Manalink dev 778ccb5 version - duel
What exactly should be the correct behavior/interaction?
Ferrovore's prompt menu is "Select an artifact." or "Select an artifact to sacrifice."
Are any other cards possibly affected by this bug?
-
- Attachments
-
- ferrovore select a card.rar
- (3.22 KiB) Downloaded 130 times
Last edited by Aswan jaguar on 27 Aug 2019, 15:01, edited 2 times in total.
Reason: fixed
Reason: fixed
---
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: Ferrovore wrong prompt menu
by Aswan jaguar » 13 Jan 2019, 16:47
I tested and AI seems not broken this time.
---
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]Ferrovore wrong prompt menu
by Korath » 18 Jan 2019, 03:59
You made it illegal to sacrifice Ferrovore to itself, assuming of course that it's become an artifact from another effect. The previous test_definition_t set not_me specifically so that the AI wouldn't assume it could pump itself an extra +3/+0 by saccing it to itself.
Why not fix generic_activated_ability_with_sacrifice_as_cost() so it uses new_sacrifice()'s already-existing implementation to generate the prompt, rather than fixing it on every single card that calls it? All you're doing is making more work for yourself and creating new opportunities for typos (as you did with Ragamuffyn, for example).
Why not fix generic_activated_ability_with_sacrifice_as_cost() so it uses new_sacrifice()'s already-existing implementation to generate the prompt, rather than fixing it on every single card that calls it? All you're doing is making more work for yourself and creating new opportunities for typos (as you did with Ragamuffyn, for example).
-
Korath - DEVELOPER
- Posts: 3707
- Joined: 02 Jun 2013, 05:57
- Has thanked: 496 times
- Been thanked: 1106 times
Re: [fixed]Ferrovore wrong prompt menu
by Aswan jaguar » 18 Jan 2019, 17:18
Thanks, for keeping an eye on things.
- Ok, I reverted Ferrovore back "to sacrifice Ferrovore to itself" implementation in commit 14d1a2c2.
- I can't find new_sacrifice function unless you mean new_sacrifice_impl which I tried to follow with no success.
- For Ragamuffyn if you mean the "int card_Raggamuffyn" typo, that was there already and frustrated me when I was looking for the card. I don't understand how to fix that. I also met 3-4 other cards with bugs like that with "s" in the end added or missing.
- Ok, I reverted Ferrovore back "to sacrifice Ferrovore to itself" implementation in commit 14d1a2c2.
- I can't find new_sacrifice function unless you mean new_sacrifice_impl which I tried to follow with no success.
- For Ragamuffyn if you mean the "int card_Raggamuffyn" typo, that was there already and frustrated me when I was looking for the card. I don't understand how to fix that. I also met 3-4 other cards with bugs like that with "s" in the end added or missing.
---
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: Ferrovore wrong prompt menu
by Korath » 25 Jan 2019, 20:36
No, I meant the typo in the prompt you added.
For misspelled function names, you can fix these by grepping for the function name (possibly prepended by an underscore) and changing it everywhere it appears:
I've mostly never bothered to correct such errors, since doing so still isn't always enough to find a card's implementation. Some examples, just off the top of my head:
For misspelled function names, you can fix these by grepping for the function name (possibly prepended by an underscore) and changing it everywhere it appears:
- Code: Select all
...\manalink\master\src>git grep -wnH _?card_raggamuffyn
git grep -wnH _?card_raggamuffyn
ManalinkEh.asm:4819:extern _card_raggamuffyn
ManalinkEh.asm:16634: jmp near _card_raggamuffyn ; 2006e07
cards/dissension.c:3168:int card_raggamuffyn(int player, int card, event_t event){
I've mostly never bothered to correct such errors, since doing so still isn't always enough to find a card's implementation. Some examples, just off the top of my head:
- There's no guarantee that, just because there's a function named card_allied_reinforcements, that Allied Reinforcements uses it instead of, say, card_ayli_eternal_pilgrim().
- It does no good to search for card_prodigal_pyromancer when it actually reuses card_prodigal_sorcerer() (as do six other cards).
- If a card gets functional errata (like Ajani's Pridemate just did) or exhibits a bug, and another card shares its function but needs not to change, then just fixing card_ajanis_pridemate breaks the other card. (As it happens, it was made to be the same as Bloodbond Vampire instead of different from it, but it could've just as easily gone the other way.)
-
Korath - DEVELOPER
- Posts: 3707
- Joined: 02 Jun 2013, 05:57
- Has thanked: 496 times
- Been thanked: 1106 times
Re: [fixed]Ferrovore wrong prompt menu
by Aswan jaguar » 27 Aug 2019, 15:01
I don't know how change the ~140 entries automatically (if that is possible) to lose the char prompt value which errors in each entry, so I prefer to change only the ones I meet that need a correct prompt.Korath wrote:Why not fix generic_activated_ability_with_sacrifice_as_cost() so it uses new_sacrifice()'s already-existing implementation to generate the prompt, rather than fixing it on every single card that calls it? All you're doing is making more work for yourself and creating new opportunities for typos (as you did with Ragamuffyn, for example).
Ferrovore fixed in commit ef267b93.
---
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
6 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 32 guests