MantisBT - VCMI
View Issue Details
0001264VCMIGUI - Battlespublic2013-04-27 06:422014-05-30 17:43
Warmonger 
Tow 
normalmajoralways
closedfixed 
0.92 
0.930.93 
0001264: ONLY_ENEMY_ARMY bonus range does not work
All bonuses with this range don't work in battle, unfortunatelly. It seems that they are corrctly attached to battle node, but for some reason are not visible in game.
Use attached map and enter battle with Arch Devil / Ghost Dragon. Check enemy luck and morale which should be reduced by 1, but are not.
No tags attached.
? Testy12 abilities.h3m (9,128) 2013-04-27 06:42
https://bugs.vcmi.eu/file_download.php?file_id=1263&type=bug
Issue History
2013-04-27 06:42WarmongerNew Issue
2013-04-27 06:42WarmongerFile Added: Testy12 abilities.h3m
2013-04-27 06:46WarmongerSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=2185#r2185
2013-05-19 20:34TowNote Added: 0003539
2013-05-19 20:34TowStatusnew => resolved
2013-05-19 20:34TowFixed in Version => 0.93
2013-05-19 20:34TowResolutionopen => fixed
2013-05-19 20:34TowAssigned To => Tow
2013-05-19 20:34TowCategoryMechanics - Battles => GUI - Battles
2013-05-20 10:48WarmongerNote Added: 0003540
2013-05-20 10:51TowNote Added: 0003541
2013-05-20 10:53WarmongerNote Added: 0003542
2013-05-20 10:58TowNote Added: 0003543
2014-05-30 17:43beegeeStatusresolved => closed

Notes
(0003539)
Tow   
2013-05-19 20:34   
Spent so much time on debugging bonus system and it was a trival bug — window used CStackInstance to print morale/luck, not the CStack… The bonuses worked fine all the time. Life is so cruel.

Fixed in r3378.
(0003540)
Warmonger   
2013-05-20 10:48   
You say so? I noticed that finally artifacts with ONLY_ENEMY_ARMY work as expected, but they have nothing to do with morale.
(0003541)
Tow   
2013-05-20 10:51   
I checked the Ghost Dragon / Arch Devil and it works for me.
Let me know if there is any counter-case.
ONLY_ENEMY_ARMY is a terrible workaround that shouldn't be part of bonus system at all. (Yet it is damn useful so can't be removed.) There might be a case where it causes issue, but this bug isn't a one.
(0003542)
Warmonger   
2013-05-20 10:53   
I mean, it did not work before, after recent commit it does. So it doesn't seem to be only issue with window, something else has changed in the meantime.
(0003543)
Tow   
2013-05-20 10:58   
It did work. Well, almost. Ghost Dragon was buggy due to 2 bugs:
1 entry in ZCRTRAIT was bugfully interpreted as +1 to morale (I fixed to -1)
2 you added entry to JSON config that gave -1 to morale (I commented it out)

Neither of bonuses was visible in the creature window and their sum was 0, so mechanically there was no visisble change to morale.

Still, the issues were unrelated to ONLY_ENEMY_ARMY. ;)