#3507 closed defect (fixed)
[PATCH] some technologies and auras don't affect all damage types
Reported by: | Palaxin | Owned by: | scythetwirler |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 19 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Description (last modified by )
For some unit types with mixed damage (e.g. spear infantry or slingers) only one damage type is affected by the technology "Will to fight" and by the Leonidas hero aura.
I added the missing damage types in the patch. Note that there may be more files to be corrected.
Attachments (3)
Change History (17)
by , 8 years ago
Attachment: | patch_attack_v1.7z added |
---|
comment:1 by , 8 years ago
Description: | modified (diff) |
---|
comment:2 by , 8 years ago
Palaxin you should upload a patch/diff with your changes. Use "svn diff" in command line or use TortoiseSVN when working on Windows. See: http://trac.wildfiregames.com/wiki/SubmittingPatches
comment:3 by , 8 years ago
Sorry and thank you for the link. I do not have experience with such tools and did only edit the templates in my A18 installation folder so far, but I saw in GitHub that the problem still exists. Nevertheless I will try to use SVN and get the .diff files.
comment:4 by , 8 years ago
You already found the build instructions for the svn? http://trac.wildfiregames.com/wiki/BuildInstructions Please make your patches from the newest svn version, thats easier for developers.
comment:5 by , 8 years ago
Also notice you don't need to compile on windows, as the repository comes with a precompiled .exe file (autobuild, which is updated more or less often enough).
by , 8 years ago
Attachment: | attack_v1.patch added |
---|
comment:6 by , 8 years ago
This patch is made with tortoiseSVN from the newest SVN. I will try to compile myself, otherwise I use the autobild (I use Windows 10). Thank you for your help.
comment:7 by , 8 years ago
If you ask me, the Leonidas aura should give defensive and not offensive stats. The mission at Thermopylae was to resist against the Persian invaders as long as possible and not to slaughter them as there was no chance to defeat them anyway. So I would add 2-3 armor instead of +20% attack. But I'm not in the position to make such changes so I hope this patch makes you happy.
comment:8 by , 8 years ago
I wonder if it can't be simplified, so that we would only have to change one line? Can't we just make all "Male" units to modify all "Attack" values or something like that?
comment:9 by , 8 years ago
There are several other technologies which always list the three damage types. So this would be very convenient, but I'm afraid someone else would have to implement this (perhaps in attack.js?)
comment:10 by , 8 years ago
Keywords: | patch review added |
---|---|
Milestone: | Backlog → Alpha 20 |
by , 8 years ago
Attachment: | attack_v2.patch added |
---|
I have simplified it a bit. It didn't work when I used "Male" or "Male Unit" for "affects". Is there a possiblity to do that? However, this would not cause the intended changes. For example we have a slave template, and slaves should definitely not get bonus attack through this technology though they are males. I added "Heros", so basically all human fighting units should profit from this tech now.
comment:13 by , 8 years ago
Keywords: | review removed |
---|---|
Milestone: | Alpha 20 → Alpha 19 |
Thanks for the patch. :) Not a big problem, but in the future, try to generate your patches from the root of the repository (that has the folders binaries, build, libraries, source etc.) for convenience.
patch for "Will to fight" technology and Leonides template