Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 22 20:36
    andli commented #4160
  • Oct 22 01:46

    theelk801 on master

    added Pioneer format (compare)

  • Oct 22 01:42

    theelk801 on master

    updated Pauper ban list updated Standard ban list (compare)

  • Oct 21 14:16
    JayDi85 commented #6016
  • Oct 21 10:04
    andreacosta commented #6016
  • Oct 19 02:37
    scmurray15 commented #5961
  • Oct 17 21:21
    t-schroeder commented #6015
  • Oct 17 21:21
    t-schroeder commented #6015
  • Oct 17 21:20
    t-schroeder commented #6015
  • Oct 17 20:46
    t-schroeder commented #6015
  • Oct 17 20:39
    t-schroeder commented #6015
  • Oct 17 20:37
    t-schroeder commented #6015
  • Oct 17 13:54
    andreacosta closed #6014
  • Oct 17 10:49
    spjspj commented #6015
  • Oct 17 09:39
    t-schroeder commented #6015
  • Oct 17 05:52
    Ebola16 opened #6018
  • Oct 16 23:55
    spjspj labeled #6017
  • Oct 16 23:55
    spjspj opened #6017
  • Oct 16 22:56
    JayDi85 commented #6016
  • Oct 16 19:29
    andreacosta commented #6016
EikePeace
@EikePeace
This message was deleted
public boolean apply(Game game, Ability source) {
        Token token = new WeaponToken();
        token.putOntoBattlefield(1, game, source.getSourceId(), source.getControllerId());
        Permanent token = game.getPermanent(token.getLastAddedToken);
        Permanent permanent = game.getPermanent(source.getFirstTarget());
        if (token != null && permanent != null && !permanent.cantBeAttachedBy(token, game)) {
            game.getState().setValue("attachTo:" + token.getId(), permanent);
            permanent.addAttachment(token.getId(), game);
            return true;
        }
        return false;
    }
Ah, now it works
it gives me an error at the second token and says, that it is already defined
ingmargoudt
@ingmargoudt
You define token twice in the same scope, which Java does not allow. Rename one of them
Oleg Agafonov
@JayDi85
@EikePeace your effect can create multiple weapon tokens in combo... Use token.getLastAddedTokenIds instead token.getLastAddedToken to cycle and attach each of that tokens to target.
EikePeace
@EikePeace
 public ArmAbilityEffect() {
        super(new WeaponToken());

    }

    @Override
    public boolean apply(Game game, Ability source) {
        for (int i=0;  i < this.getLastAddedTokenIds().size(); i++){
            Permanent token = game.getPermanent(getLastAddedTokenIds().get(i));
            Permanent permanent = game.getPermanent(source.getFirstTarget());
            if (token != null && permanent != null && !permanent.cantBeAttachedBy(token, game)) {
                game.getState().setValue("attachTo:" + token.getId(), permanent);
                permanent.addAttachment(token.getId(), game);
            }
        }
        return true;
    }
}
like this?

Now I am just working on getting it to be connected to a EnterTheBattlefield Trigger and targeting the creature that just entered

For this card for example:

EikePeace
@EikePeace

I have a small problem:

My IntelliJ stopped to autocomplete inputs and doesn't higlight stuff anymore

Does anybody have an idea how to fix that?

Zach H
@ZeldaZach
Did you try updating
And make sure your files are within the project
EikePeace
@EikePeace

It suddenly works again after restarting it for the third

Strange ^^"

But yeah, I also updated now, thx :)
Zach H
@ZeldaZach
Cool
ingmargoudt
@ingmargoudt
Happens sometimes. Use Invalidate caches & restart to rebuild indexes
EikePeace
@EikePeace

Now I'm just getting an error in all files in Mage.Verify

For the JsonIgnoreProperties

Sorry for spamming all my problems in here
It cannot resolve the symbol
Oleg Agafonov
@JayDi85
@EikePeace try to find POM file in project explorer (for mage verify), right click on it and press maven -> re-import. that's file must contains <groupId>com.fasterxml.jackson.core</groupId>
EikePeace
@EikePeace
The Import Statement is still red
It tries to import from com.fasterxml.jackson.annotations.JsonIgnoreProperties;
But when I try to type it myself, .annotations isn't a thing that is recognized
Oleg Agafonov
@JayDi85
Well... That's strange. Then try clean install: create new folder, unzip git sources and import/rebuild as maven project.
Oleg Agafonov
@JayDi85
It's will be red until you add that lib to project structure in mage-verify (maven adds it automaticly on maven->re-import).
shot_190812_183802.png
EikePeace
@EikePeace
Now it works
It didn't get added automatically somehow
Thanks :)
ingmargoudt
@ingmargoudt
Food for thought, a while ago I removed the game.getGameState.getwatcher() method and added it to the Game class. Reasoning because a) it is a violation of the Law of Demeter and b) cards and players should not have direct access to the gamestate. I can look at other Objects in the GameImpl class that have similar behaviour
I have 2 weeks off now so I can spend some time again :)
Another thing I could do is to remove the SubTypeList. It basically became a simple ArrayList now. The transitive API is not used anymore
Or make more tests
Evan Kranzler
@theelk801
please feel free to help on the commander cards
they're uh... difficult
Zach H
@ZeldaZach
How are y’all dealing with turn order changes
Evan Kranzler
@theelk801
we're not!
or at least I'm not
or at least I'm not yet
I'm thinking it might join the ranks of cards like Mind Bend where we just don't implement it
we'll see if it ends up on modo
Zach H
@ZeldaZach
I’m looking into it on my side for trice
Oleg Agafonov
@JayDi85
lol, WTF: Exile Aeon Engine: Reverse the game’s turn order. (For example, if play had proceeded clockwise around the table, it now goes counterclockwise.)
Oleg Agafonov
@JayDi85
I'll try to implement change turn order effect
Alexander Luyten
@imgifty
Hello guys! I setup xmage on my ide. Are there any easy bugs/issues I could start helping with?
Oleg Agafonov
@JayDi85
@imgifty you can:
  1. Find old missing card and implement it (if it easy and already exists -- search code by same card texts):
    https://github.com/magefree/mage/labels/tracking%20set

  2. If you can run tests then search private void checkWrongAbilitiesText(Card card, JsonCard ref) { and run verify test for latest C19 commander cards to check texts. You will see wrong/missing abilities (maybe some texts have typo, wrong coded or just missing). Same for other sets too.

  3. You can look at sonar's code report for bugs or fixes: https://sonarcloud.io/dashboard?id=JayDi85_mage

Oleg Agafonov
@JayDi85
Or just look at https://github.com/magefree/mage/issues take some issues and test it to confirm or fix. Some bugs can be easy to fix (as example: wrong type/damage, missing optional choose or nothing happened).
4 - Some cards needs extra info about battlefield state like "make damage equals to creatures controlled". You can add that info to card text as special hints: search code for examples: .addHint(