Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 01:26
    ElchananHaas edited #8100
  • 01:25
    ElchananHaas opened #8100
  • Aug 04 19:17
    jeffwadsworth assigned #8044
  • Aug 04 19:13
    jeffwadsworth commented #7842
  • Aug 04 18:40

    JayDi85 on master

    * Target colored cards/creature… Tests: added serialization test… (compare)

  • Aug 04 17:50
    JayDi85 closed #8099
  • Aug 04 17:50
    JayDi85 commented #8099
  • Aug 04 17:50
    JayDi85 labeled #8099
  • Aug 04 17:49
    woodscode opened #8099
  • Aug 04 17:43
    github-actions[bot] commented #8098
  • Aug 04 17:43
    alexander-novo opened #8098
  • Aug 04 16:46
    github-actions[bot] commented #8097
  • Aug 04 16:45
    Jipsu opened #8097
  • Aug 04 16:30
    jeffwadsworth commented #7842
  • Aug 04 16:05
    EllNubNub edited #8096
  • Aug 04 16:04
    EllNubNub closed #8096
  • Aug 04 15:59
    EllNubNub opened #8096
  • Aug 04 14:14
    github-actions[bot] commented #8062
  • Aug 04 14:14
    Jipsu commented #8062
  • Aug 04 02:23
    jeffwadsworth commented on b6a3f7e
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?
Fenhl
@fenhl
ah, found it (for Windows, in the registry at Computer\HKEY_CURRENT_USER\Software\JavaSoft\Prefs\mage\client)
Evan Kranzler
@theelk801
@JayDi85 awesome, thanks again for doing that
Zach H
@ZeldaZach
You got it working?
Evan Kranzler
@theelk801
yeah he made it work
although I need to apply the changes to TargetCreaturePermanentAmount so I can finally implement Forked Lightning
Fenhl
@fenhl
hey, what's the opinion on custom sets here?
there's that Star Wars set, but I'm considering XMage for a custom format with multiple sets
and I'm wondering if that's something that could go in the main repo or if a fork would be better
Zach H
@ZeldaZach
Probably a fork
That’s not really official and most people won’t want non official cards
Fenhl
@fenhl
XMage already has unofficial cards
Evan Kranzler
@theelk801
yeah we do have the star wars cards for some reason but adding new sets potentially leads to issues so we prefer not to
Fenhl
@fenhl
makes sense
are there any custom sets implemented in Java using the plugin system?
@Lymia has an example but it's in Scala which doesn't help much with copy/pasting card abilities