Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Oct 13 23:05

    theelk801 on master

    Implemented Silent Assassin (compare)

  • Oct 12 21:08
    theelk801 closed #6013
  • Oct 12 21:08

    theelk801 on master

    fixed Ayara, First of Locthwain… (compare)

  • Oct 12 20:42
    rystan opened #6013
  • Oct 12 16:11

    theelk801 on master

    updated Oathbreaker banlist updated Canadian Highlander poi… updated Australian Highlander p… (compare)

  • Oct 12 15:44

    theelk801 on master

    added Game Night 2019 Implemented Sphinx of Enlighten… Implemented Highcliff Felidar and 4 more (compare)

  • Oct 12 03:30

    theelk801 on master

    Adding incorrectly-failing test… Fix buggy login in SyrKonradThe… Remove unneeded import in an un… and 1 more (compare)

  • Oct 12 03:30
    theelk801 closed #6012
  • Oct 11 14:00
    apetresc edited #6012
  • Oct 11 14:00
    apetresc edited #6012
  • Oct 11 14:00
    apetresc synchronize #6012
  • Oct 11 13:51
    apetresc edited #6012
  • Oct 11 13:50
    apetresc opened #6012
  • Oct 11 13:09
    Opchip commented #5961
  • Oct 10 21:01

    theelk801 on master

    fixed Deafening Silence (compare)

  • Oct 10 02:59
    Opchip commented #6003
  • Oct 10 02:58
    theelk801 commented #5961
  • Oct 10 02:57
    Opchip commented #5961
  • Oct 09 12:07
    luzaasta opened #6011
  • Oct 09 02:50
    theelk801 closed #6010
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(
Alexander Luyten
@imgifty
yes! I started on implementing Commander 19 cards. I'm having some issues running my client though. I get NullPointerExceptions trying to load the CardPluginImpl. Also I had to add Javafx Dependencies to the Maven xml file. Is that correct?
Alexander Luyten
@imgifty
this is the ouput, when I try to run the client: https://hastebin.com/odufonozos.coffeescript
Oleg Agafonov
@JayDi85
You oracle's java 8 to run
Don't try to implement commander 2019 -- @theelk801 already implemented it. There are only very difficult cards (not for novice). The older set -- the easiest missing card to implement.
shot_190818_172504.png
Oleg Agafonov
@JayDi85
shot_190818_172649.png