Where communities thrive


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

    JayDi85 on master

    Test framework: added more alia… (compare)

  • 10:02

    JayDi85 on master

    Test framework: added alias sup… (compare)

  • 02:03
    JayDi85 edited #5451
  • 02:02

    JayDi85 on master

    Logs cleanup (removed database … (compare)

  • 02:01

    JayDi85 on master

    Test framework: improved aliase… (compare)

  • Jan 26 20:54
    dragonfyre23 opened #6227
  • Jan 26 16:52
    JayDi85 assigned #6222
  • Jan 26 16:39

    JayDi85 on master

    Merge fix (compare)

  • Jan 26 16:37

    JayDi85 on master

    Tests for #6222 (compare)

  • Jan 26 15:56
    JayDi85 closed #6223
  • Jan 26 15:56
    JayDi85 commented #6223
  • Jan 26 15:56

    JayDi85 on master

    * Thassa's Oracle - fixed wrong… (compare)

  • Jan 26 15:55
    JayDi85 assigned #6223
  • Jan 26 15:53
    JayDi85 closed #6225
  • Jan 26 15:53
    JayDi85 commented #6225
  • Jan 26 15:53
    JayDi85 labeled #6225
  • Jan 26 15:53
    JayDi85 assigned #6225
  • Jan 26 15:52

    JayDi85 on master

    * Vivien's Arkbow - fixed that … (compare)

  • Jan 26 15:39
    JayDi85 edited #6169
  • Jan 26 15:39
    JayDi85 edited #6169
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
Oleg Agafonov
@JayDi85
@fenhl if you make good and popular custom set then it's can be added to xmage
xmage's server support to load custom sets (e.g. official don't have set but unofficial server have it -- users connects to server, download new cards and play on it)