MantisBT - VCMI
View Issue Details
0001667VCMIMechanics - Battlespublic2014-01-20 19:132023-04-12 23:23
anoleyn 
 
normalminoralways
newreopened 
0.94 
0.95 
0001667: Elixir of Life not working properly
Elixir of Life gives a health bonus to undead and non living cratures although it should not (as states in the description). Additionally, most probably the 25% bonus value is also granted on the basis of the bonuses of the artifact's components not only on the crature's base health value. For example, Phoenixes should have a bonus of:

200 * 25% + 4 = 54

but have:

(200 + 4) * 25% + 4 = 55

Attached screen shows both problems on a Ghost Dragon.
No tags attached.
png Elixir.png (433,066) 2014-01-20 19:13
https://bugs.vcmi.eu/file_download.php?file_id=1660&type=bug
png
Issue History
2014-01-20 19:13anoleynNew Issue
2014-01-20 19:13anoleynFile Added: Elixir.png
2014-01-20 19:14WarmongerAssigned To => Warmonger
2014-01-20 19:14WarmongerStatusnew => assigned
2014-02-28 17:51WarmongerNote Added: 0004541
2014-02-28 17:51WarmongerStatusassigned => resolved
2014-02-28 17:51WarmongerFixed in Version => 0.95
2014-02-28 17:51WarmongerResolutionopen => fixed
2014-03-18 10:44anoleynNote Added: 0004597
2014-03-18 10:44anoleynStatusresolved => feedback
2014-03-18 10:44anoleynResolutionfixed => reopened
2014-03-18 11:52WarmongerNote Added: 0004598
2014-09-02 09:55WarmongerCategoryMechanics - Other => Mechanics - Battles
2023-04-12 23:23IvanAssigned ToWarmonger =>
2023-04-12 23:23IvanStatusfeedback => new

Notes
(0004541)
Warmonger   
2014-02-28 17:51   
Fixed in r3784, though it's workaround.
(0004597)
anoleyn   
2014-03-18 10:44   
Patrtially fixed - still gives bonus to non-living creatures like golems etc.

Maybe creating a new creature property 'IS_LIVING' and making 'type of creature' mendatory in creature config would solve the problem?
(0004598)
Warmonger   
2014-03-18 11:52   
Yes it would, but it would also introduce more problems than it's worth - all creatures in existing mods do not have this property.

Ideally, it should be possible to make excluding limiter opposite to "has bonus of type" limiter.