MantisBT - VCMI
View Issue Details
0001240VCMIModspublic2013-03-14 11:122014-05-30 17:43
Orzi 
Warmonger 
normaltweakalways
closedfixed 
0.92 
0.93 
0001240: Upgraded Assids attack until enemy die
Assids from Cove (upgraded ones) have a really major bug. Their special ability, instead of allowing them of attack twice (as intended and as in HotA), makes them attack the enemy many times until their opponent die. Can you fix it, or just change that ability to standard double strike?
No tags attached.
Issue History
2013-03-14 11:12OrziNew Issue
2013-03-14 14:23WarmongerAssigned To => Warmonger
2013-03-14 14:23WarmongerStatusnew => assigned
2013-03-14 14:24WarmongerPriorityimmediate => normal
2013-03-14 14:24WarmongerSeveritymajor => crash
2013-03-14 17:02AlexVinSNote Added: 0003444
2013-03-14 17:41OrziNote Added: 0003446
2013-03-14 18:00WarmongerNote Added: 0003447
2013-03-14 18:00WarmongerSeveritycrash => tweak
2013-03-14 18:00WarmongerCategoryMechanics - Battles => Mods
2013-03-14 18:08AlexVinSNote Added: 0003448
2013-03-14 18:11WarmongerNote Added: 0003449
2013-03-14 18:11IvanNote Added: 0003450
2013-03-14 18:12IvanNote Edited: 0003450bug_revision_view_page.php?bugnote_id=3450#r2155
2013-03-14 19:17OrziNote Deleted: 0003446
2013-04-01 09:42WarmongerNote Added: 0003480
2013-04-01 09:42WarmongerStatusassigned => resolved
2013-04-01 09:42WarmongerFixed in Version => 0.93
2013-04-01 09:42WarmongerResolutionopen => fixed
2014-05-30 17:43beegeeStatusresolved => closed

Notes
(0003444)
AlexVinS   
2013-03-14 17:02   
Setting at least "val" in config may be enough

 
   {
    "type" : "ADDITIONAL_ATTACK"
   }
(0003447)
Warmonger   
2013-03-14 18:00   
[facepalm]

Alex is right, there is missing value. Adding it restores correct behaviour.
Can't check it right now, but it's likely that bonuses default to -1 value. It should, however, work with value of bonus equal to zero.
(0003448)
AlexVinS   
2013-03-14 18:08   
There was also possible problem with uninitialized value. In case of unset "val"
 Bonus->val may contain garbage. I fixed this in 3293.
(0003449)
Warmonger   
2013-03-14 18:11   
Bonus constructor should initialize value on its own. This is preferred solution as works for all cases.
(0003450)
Ivan   
2013-03-14 18:11   
(edited on: 2013-03-14 18:12)
Initialization should be fixed already:
http://sourceforge.net/apps/trac/vcmi/changeset/3293 [^]

Old parsing code: if "val" is missing - bonus value is uninitialized (random)
New code: if "val" is missing - value is 0

+ there is explicit initialization to 0 in Bonus constructor

Oups. I'm not fast enough :)

(0003480)
Warmonger   
2013-04-01 09:42   
Assids work fine in my game, so I assume it's fixed as Ivan mentioned. Still, more of a problem with mod than game itself.