Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jan 18 20:18
    phobosd edited #6201
  • Jan 18 20:17
    phobosd edited #6201
  • Jan 18 19:51
    JayDi85 labeled #6201
  • Jan 18 19:51
    JayDi85 labeled #6201
  • Jan 18 19:49
    phobosd opened #6201
  • Jan 18 16:39

    JayDi85 on master

    Additional to prev commit (compare)

  • Jan 18 16:37

    JayDi85 on master

    * UI: fixed that some user defi… (compare)

  • Jan 18 16:09
    jeffwadsworth labeled #6200
  • Jan 18 14:05
    philo-jh opened #6200
  • Jan 18 09:02

    JayDi85 on master

    Merge fix (compare)

  • Jan 18 08:03

    JayDi85 on xmage_1.4.42V2

    (compare)

  • Jan 18 07:42

    JayDi85 on master

    Prepare new release (compare)

  • Jan 18 07:24

    JayDi85 on master

    Revert (compare)

  • Jan 18 07:12

    JayDi85 on master

    * Mogis, God of Slaughter - fix… (compare)

  • Jan 18 07:01

    JayDi85 on master

    Tests: improved logs in verify … Fixed multiple card texts; (compare)

  • Jan 18 06:01
    JayDi85 edited #6169
  • Jan 18 06:00
    JayDi85 edited #6169
  • Jan 18 05:59
    JayDi85 edited #6169
  • Jan 18 05:52

    JayDi85 on master

    Return old working version of f… (compare)

  • Jan 18 05:43

    JayDi85 on master

    UI: choose dialog improves (see… (compare)

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
Oleg Agafonov
@JayDi85
shot_190924_060240.png
shot_190924_060317.png
shot_190924_060404.png
Just add support of maxNumberOfTargets as example above -- that's all.
Evan Kranzler
@theelk801
wow
that's it
thank you
Evan Kranzler
@theelk801
so I'm testing it, and it doesn't work
doesn't prevent you from choosing more than the maximum amount
Oleg Agafonov
@JayDi85
You must add same code in all other places like possibleTargets, canChoose, etc
canTarget, canChoose, possibleTargets
Evan Kranzler
@theelk801
alright
so just add if (getMaxNumberOfTargets() > 0 && getTargets().size() >= getMaxNumberOfTargets()) {return false;}to the boolean methods?
Oleg Agafonov
@JayDi85
nope
you can choose already choosen
if... return getTargets.contain(objectId)
Evan Kranzler
@theelk801
ah ok
do that for both boolean methods?
Oleg Agafonov
@JayDi85
yearh
hmm
Evan Kranzler
@theelk801
wait, for canChoose
don't I just return false there?
Oleg Agafonov
@JayDi85
I don't think so... if you reach max limit then you can choose between already selected, but not new
Evan Kranzler
@theelk801
do you mind implementing the changes and pushing them?
Oleg Agafonov
@JayDi85
nope, thats for you)
but canTarget code... you must test (is it need limit code or not)
Evan Kranzler
@theelk801
I implemented the changes you suggested but I can still select more than the maximum
Oleg Agafonov
@JayDi85
well, then I'll debug it later
it's can be canTarget problem
Evan Kranzler
@theelk801
I'm going to bed
but thanks for the help
Oleg Agafonov
@JayDi85
TargetAmout was improved by magefree/mage@bad26b1
Fenhl
@fenhl
hey, where are the XMage client preferences stored?