Commander 2017 spoiler season
Post MTG Forge Related Programming Questions Here
Moderators: timmermac, Blacksmith, KrazyTheFox, Agetian, friarsol, CCGHQ Admins
Re: Commander 2017 spoiler season
by Indigo Dragon » 31 Aug 2017, 12:11
@Agetian
I once used http://svn.slightlymagic.net/websvn/log ... F&isdir=1& (Don't bother clicking, it 404s), which had a handy log feature to show all the updates. It allowed me to keep track of what was updated and what wasn't.
Goblin Hero viewtopic.php?f=26&t=20999&p=218130#p218130 says that he's "not yet sure if it's worth fixing" .
For now I'm watching http://svn.slightlymagic.net/forge/, which is really hard and bare bones.
I'm self-admitted very stupid when it comes to Eclipse, Java, Programming, Coding, etc. I'm trying to learn, but the going is tough. For now, this system of updates is working and I'm very happy with it.
===
Now, for less boring stuff
Nazahn, Revered Bladesmith has a whole other ability "Whenever an equipped creature you control attacks, you may tap target creature defending player controls." This is currently completely missing.
I'm thinking of there being two updates to Planeswalkers. The first and easiest one is to double space loyalty abilities in the Oracle text.
eg. for Chandra, Pyrogenius;
What I have just realised by doing this is that some cards have "[+1]" and some have "[+1]:". Damn. That's what I'm going to do instead, standardize to "[+1]:".
Now for emblems. Refer to the following. /forge/trunk/forge-game/src/main/java/forge/game/ability/effects/effecteffect.java
Then again, there wasn't much gain in going to the moon either. And we choose to change the emblem token names so that it will be standardized and sort able, not because it is easy, but because it is hard.
Thanks and good luck.
Edit: Vessel of Malignity has "SpellDescription$ Target opponent exiles a card from his or her hand. Activate this ability only any time you could cast a sorcery." instead of "Target opponent exiles two cards from his or her hand. Activate this ability only any time you could cast a sorcery."
I once used http://svn.slightlymagic.net/websvn/log ... F&isdir=1& (Don't bother clicking, it 404s), which had a handy log feature to show all the updates. It allowed me to keep track of what was updated and what wasn't.
Goblin Hero viewtopic.php?f=26&t=20999&p=218130#p218130 says that he's "not yet sure if it's worth fixing" .
For now I'm watching http://svn.slightlymagic.net/forge/, which is really hard and bare bones.
I'm self-admitted very stupid when it comes to Eclipse, Java, Programming, Coding, etc. I'm trying to learn, but the going is tough. For now, this system of updates is working and I'm very happy with it.
===
Now, for less boring stuff
Nazahn, Revered Bladesmith has a whole other ability "Whenever an equipped creature you control attacks, you may tap target creature defending player controls." This is currently completely missing.
I'm thinking of there being two updates to Planeswalkers. The first and easiest one is to double space loyalty abilities in the Oracle text.
eg. for Chandra, Pyrogenius;
- Code: Select all
[+2] Chandra, Pyrogenius deals 2 damage to each opponent.
[-3] Chandra, Pyrogenius deals 4 damage to target creature.
[-10] Chandra, Pyrogenius deals 6 damage to target player and each creature he or she controls.
- Code: Select all
[+2] Chandra, Pyrogenius deals 2 damage to each opponent.
[-3] Chandra, Pyrogenius deals 4 damage to target creature.
[-10] Chandra, Pyrogenius deals 6 damage to target player and each creature he or she controls.
- Code: Select all
Oracle:[+2] Chandra, Pyrogenius deals 2 damage to each opponent.\n\n[-3] Chandra, Pyrogenius deals 4 damage to target creature.\n\n[-10] Chandra, Pyrogenius deals 6 damage to target player and each creature he or she controls.
What I have just realised by doing this is that some cards have "[+1]" and some have "[+1]:". Damn. That's what I'm going to do instead, standardize to "[+1]:".
Now for emblems. Refer to the following. /forge/trunk/forge-game/src/main/java/forge/game/ability/effects/effecteffect.java
- Code: Select all
String image;
if (sa.hasParam("Image")) {
image = ImageKeys.getTokenKey(sa.getParam("Image"));
} else if (name.endsWith("emblem")) { // try to get the image from name
image = ImageKeys.getTokenKey(name.replace(",", "").replace(" ", "_").toLowerCase());
} else { // use host image
image = hostCard.getImageKey();
}
Then again, there wasn't much gain in going to the moon either. And we choose to change the emblem token names so that it will be standardized and sort able, not because it is easy, but because it is hard.
Thanks and good luck.
Edit: Vessel of Malignity has "SpellDescription$ Target opponent exiles a card from his or her hand. Activate this ability only any time you could cast a sorcery." instead of "Target opponent exiles two cards from his or her hand. Activate this ability only any time you could cast a sorcery."
Ctrl+C, Ctrl+V
-
Indigo Dragon - Posts: 130
- Joined: 11 May 2016, 10:18
- Location: Australia
- Has thanked: 63 times
- Been thanked: 56 times
Re: Commander 2017 spoiler season
by Agetian » 31 Aug 2017, 12:47
Ah, gotcha!
You can ensure that you're staying on up to date cardsfolder by regularly checking out the SVN repository - you can use something like TortoiseSVN for the purpose, and check out the repository at "http://svn.slightlymagic.net/forge/trunk". That should also provide you with the changelog. Let me know if you need further help with it.
Double spacing for the 'walker abilities sounds like a good change.
Yeah, changing the emblem names might be tricky, though it is doable (some renaming will have to be done though, yeah, including the files on the server).
I'll patch up Nazahn soon
EDIT: Done.
- Agetian
You can ensure that you're staying on up to date cardsfolder by regularly checking out the SVN repository - you can use something like TortoiseSVN for the purpose, and check out the repository at "http://svn.slightlymagic.net/forge/trunk". That should also provide you with the changelog. Let me know if you need further help with it.
Double spacing for the 'walker abilities sounds like a good change.
Yeah, changing the emblem names might be tricky, though it is doable (some renaming will have to be done though, yeah, including the files on the server).
I'll patch up Nazahn soon
EDIT: Done.
- Agetian
- Agetian
- Programmer
- Posts: 3474
- Joined: 14 Mar 2011, 05:58
- Has thanked: 677 times
- Been thanked: 563 times
Re: Commander 2017 spoiler season
by Agetian » 01 Sep 2017, 03:09
@ Indigo Dragon: Oh, btw, Edgar Markov is supposed to have Card.Vampire, not Creature.Vampire, since it says "another Vampire spell", which includes Vampire tribal spells
- Agetian
- Agetian
- Agetian
- Programmer
- Posts: 3474
- Joined: 14 Mar 2011, 05:58
- Has thanked: 677 times
- Been thanked: 563 times
Re: Commander 2017 spoiler season
by Indigo Dragon » 01 Sep 2017, 04:56
@Agetian, time to stop NecroPosting, sent you a PM. Really excited!
Ctrl+C, Ctrl+V
-
Indigo Dragon - Posts: 130
- Joined: 11 May 2016, 10:18
- Location: Australia
- Has thanked: 63 times
- Been thanked: 56 times
49 posts
• Page 4 of 4 • 1, 2, 3, 4
Who is online
Users browsing this forum: No registered users and 56 guests