It is currently 01 May 2025, 04:06
   
Text Size

cards with the whenever keyword

Post MTG Forge Related Programming Questions Here

Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins

Re: cards with the whenever keyword

Postby Sloth » 19 Sep 2010, 09:30

Chris H. wrote: If we delete the

Code: Select all
 + " Token"
`
we end up with a token that is named like the other Thrull token. Does anyone have any thoughts on this matter?
Well, we should delete + " Token", to correct it ruleswise. The problem with the pictures of tokens would have arisen sooner or later, since there are more cases of tokens with the same name but different stats. So let's not hack it in, but solve the problem, where it has to be solved: in the picture assigning.

PS: I can remember some discussions on another forum, whether token is part of the name of a token. The rules say it is not, but it's understandable that such an error sneaked in here.
User avatar
Sloth
Programmer
 
Posts: 3498
Joined: 23 Jun 2009, 19:40
Has thanked: 125 times
Been thanked: 507 times

Re: cards with the whenever keyword

Postby Chris H. » 19 Sep 2010, 14:01

Once I had a free moment I was able to deduce that there were two different Thrull tokens and my tokens folder is now up to date. I ran a test deck and created both types of Thrull tokens and they used both pictures.

I deleted the + " Token" on the SVN. We still have a couple of minor issues with this keyword. It is a long code block but it works fairly well at this time.

There was a recent bug report with an error exception and it looked like the AI was casting a counterspell and the spell that was countered may have had a wheneverKeyword ability that could not correctly resolve.
User avatar
Chris H.
Forge Moderator
 
Posts: 6320
Joined: 04 Nov 2008, 12:11
Location: Mac OS X Yosemite
Has thanked: 644 times
Been thanked: 643 times

Re: cards with the whenever keyword

Postby Sloth » 19 Sep 2010, 14:56

Sloth wrote:Well, we should delete + " Token", to correct it ruleswise. The problem with the pictures of tokens would have arisen sooner or later, since there are more cases of tokens with the same name but different stats. So let's not hack it in, but solve the problem, where it has to be solved: in the picture assigning.
Uh, nevermind. I wasn't aware that tokens picture files already have color and P/T as part of their name.

It's great when problems are already solved!
User avatar
Sloth
Programmer
 
Posts: 3498
Joined: 23 Jun 2009, 19:40
Has thanked: 125 times
Been thanked: 507 times

Previous

Return to Developer's Corner

Who is online

Users browsing this forum: No registered users and 17 guests


Who is online

In total there are 17 users online :: 0 registered, 0 hidden and 17 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 17 guests

Login Form