Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 09:29
    andreacosta edited #6014
  • 09:28
    andreacosta opened #6014
  • 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
Oleg Agafonov
@JayDi85
db creates on startup on missing db... You must delete db folder from mage-server folder, not from mage-client (it's a different).
EikePeace
@EikePeace
It still creates an empty one :/
Oleg Agafonov
@JayDi85
mage-server/db folder is empty? Show your server logs here (delete mage-server/mageserver.log , start server and copy logs here or to https://pastebin.com/ )
EikePeace
@EikePeace
the server is not empty, it doesn't have any cards in the database
THe decks on a testing server are empty for example
Oleg Agafonov
@JayDi85
What you mean by "empty decks"? Server shows missing card error on game start? Or what?
EikePeace
@EikePeace
No, the decks are literally empty
Not even lands exist
The databases are just empty
It is as if no cards would have been ever created
And the test games all end in a draw
Oleg Agafonov
@JayDi85
Show me server's logs file
ingmargoudt
@ingmargoudt
Could anyone check my fix for Jeskai's Infiltrator ?
there was an endless loop that causes a server crash (I think an out of memory or a stackoverflow exception)
ingmargoudt
@ingmargoudt
but I am not sure this fix is that easy..
ingmargoudt
@ingmargoudt

also, I am interested in this:

  @Deprecated
    void discard(int amount, Ability source, Game game);

    Card discardOne(boolean random, Ability source, Game game);

    Cards discard(int amount, boolean random, Ability source, Game game);

    boolean discard(Card card, Ability source, Game game);

The top method is deprecated and internally calls the 3rd method. The 4th method discards a particular card. So since the top method is deprecated , should we remove it and replace all instances with discard(int amount, boolean random, Ability source, Game game); ? And what about discardOne(random...) ?

Oleg Agafonov
@JayDi85
If void discard( uses by few cards then you can remove it, but if it uses in many cards then save it (there are many simplified methods with less params to call for more easy devs).
Evan Kranzler
@theelk801
probably gonna need some help implementing adventure cards, I have an outline of what I've got in mind on the tracking thread #5961
Zach H
@ZeldaZach
I’ll give it a gander tomorrow
I’d reckon it’s similar to aftermath tho
Evan Kranzler
@theelk801
aftermath is way simpler, it's just a split card with an ability that says "cast this only from your graveyard"
this has a whole bunch of extra rules baggage
Evan Kranzler
@theelk801
the important thing is that we can probably implement them in a way where we can add each card even if the overall implementation doesn't work
Evan Kranzler
@theelk801
would anyone be able to help with fixing Leadership Vacuum? it turns out that the engine doesn't support moving cards to the command zone and I'm not sure how to make it work as zone changes are very complicated and confusing
Zach H
@ZeldaZach
Move to exile and use standard replacement?
Evan Kranzler
@theelk801
I can't even find where the commander replacement happens
Oleg Agafonov
@JayDi85
Zone changes works fine. Look at CommanderReplacementEffect
ZoneChangeEvent zEvent = (ZoneChangeEvent) event;
zEvent.setToZone(Zone.COMMAND);
Evan Kranzler
@theelk801
oh, interesting
but how do we make that actually work
Oleg Agafonov
@JayDi85
You can just add command zone support in public boolean moveCards(Set<Card> cards, Zone toZone,
I don't think it different with other zones like library or outside
Maybe code from case OUTSIDE: can be copy pasted for command zone
Oleg Agafonov
@JayDi85
hmm
Real cards move in private static void placeInDestinationZone(ZoneChangeInfo info, Game game) {
 case COMMAND:
                    // There should never be more than one card here.
                    for (Card card : cards.getCards(game)) {
                        game.addCommander(new Commander(card));
                    }
                    break;
Smelling code... Try to copy-paste code from outside zone (see above) and test commanders moves (if all works -- commander can be casted with same effects and corrected extra cost -- then all fine... if not then I can research it later to fix/improve).
Evan Kranzler
@theelk801
can anyone help with updating TargetAmount? Sundering Stroke requires distributing 7 damage among 3 targets, which is currently not doable and I'm having trouble figuring it out
also Forked Lightning basically needs the same thing
Zach H
@ZeldaZach
Do you have “any number of targets” as an option
And then can restrict from there
Evan Kranzler
@theelk801
basically with the exception of the two cards I mentioned, every card that divides something X damage (or counters, or prevention, etc) among "any number of targets" isn't actually any number
Bogardan Hellkite can't target more than 5 things
and the code for handling that assumes that the limit is the amount being distributed
because it usually is
except here it isn't
but it's hard to figure out what does and doesn't need to be changed
also Vivien, Arkbow Ranger can target "up to" 2 target creatures, which is also different since distribution effects usually require at least one target
Oleg Agafonov
@JayDi85
@theelk801 it's already have maxNumberOfTargets
Oleg Agafonov
@JayDi85
You can setup it in cards and improve code in TargetAnyTargetAmount to implement support of that limit in maxNumberOfTargets