Jump to content

fatherbushido

WFG Retired
  • Posts

    1.148
  • Joined

  • Days Won

    4

Posts posted by fatherbushido

  1. 49 minutes ago, Nescio said:

    No? So health is not an integer? Interesting. What about other values, e.g. resources?

    health is decimal

    resources are integers (not need to cut the unit in which we count it, else we had to count it in another unit, if someone think what I say didn't mean anything, he is right, that's why we have to have integer resources)

    • Like 1
  2. On 24/10/2017 at 4:54 PM, Nescio said:

    Although buying and selling seems to work, I still get a warning I don't understand whenever I start or load a game:

    
    WARNING: JavaScript warning: simulation/components/Barter.js line 45 reference to undefined property multiplier.buy[resource]

    Furthermore, I still want to disable bartering resources other than silver directly; only silver ought to be used for buying or selling other resources.

    That's a really annoying bug.

    When putting warn(uneval(this.template.BarterMultiplier)) in the Init of the Player component, we see that the 5th resources is not taken into account. I hope it's because we miss something when using the resource feature (that would be the best).

    I will read the doc on trac.

    edit: I can't understand right now. Let's say it's magical since we find an explication.

  3. 48. Yes that's this one. (The tooltip should be change accordingly as those modifications are already handled in Template.js).

    Don't consider that patch as officialy recommended. It is not reviewed.

    (For example, take care of rounding issues, if you use only integers addition it will be fine).

    45. Your modified files are on github?

  4. 1 hour ago, bb_ said:

    Is the string support really broken in there?,

    In my recent diff I propose to change some misleading array scopes, like `["foo bar"]=="foo+bar"==[["foo", "bar"]]` in where the first syntax is is misleading. And doesn't `"foo bar"` or `"["foo, "bar"]` represents the OR here?

    Yes for specific affects, there is no support of the "+". You can try (or update) the related test of DeriveModificationsFromTech in test_Technologies.js

  5. 1 hour ago, Nescio said:

    48. What is the syntax for internal AND and OR operators in technology json file modifications?

    
    {"value": "Loot/silver", "add": 1, "affects": "Healer"}

    E.g. I want it to affect champion infantry instead of healer.

    https://trac.wildfiregames.com/wiki/Technology_Templates

    It's a string but the "spaces = OR , + = AND" thing doesn't work here. (Should it? see @bb_ recent diff).

    The only thing you can do with those specific affects is something like "Champion Infantry" which will indeed apply to entity matching Champion AND Infantry classes.

    If you want to do an OR with specific affects, you have to use a new line.

    So all is doable.

    (Also have you set that modification in your mod? And what do you want to do with it, as modifying the loot is not always the good idea with techs?).

×
×
  • Create New...