MantisBT - VCMI
View Issue Details
0001230VCMIMechanics - Adventure Mappublic2013-03-05 11:442015-10-17 11:45
Ivan 
SXX 
normalfeatureN/A
resolvedfixed 
 
0.98e 
0001230: Missing creature ability: Nomads allow free movement on sand
This H3 creature ability is still missing.
No tags attached.
related to 0001371closed Warmonger Missing creature ability: Thieves should give permanent Visions effect 
related to 0002098assigned SXX CGameHandler::moveArtifact don't send information about changed bonuses to client 
related to 0001231resolved Warmonger Missing creature ability: Crystal Dragons give +3 crystals/week 
Issue History
2013-03-05 11:44IvanNew Issue
2013-03-05 18:11WarmongerRelationship addedrelated to 0001231
2013-03-05 18:13WarmongerNote Added: 0003437
2013-07-22 08:22WarmongerRelationship addedrelated to 0001371
2013-07-22 08:31WarmongerSeveritymajor => feature
2014-03-29 20:40WarmongerNote Added: 0004628
2014-03-29 20:53WarmongerAssigned To => Warmonger
2014-03-29 20:53WarmongerStatusnew => assigned
2014-03-29 20:53WarmongerAssigned ToWarmonger =>
2014-03-29 20:54WarmongerStatusassigned => new
2015-10-14 00:20SXXAssigned To => SXX
2015-10-14 00:20SXXStatusnew => assigned
2015-10-14 00:23SXXCategoryMechanics - Battles => Mechanics - Adventure Map
2015-10-14 01:22SXXNote Added: 0005942
2015-10-14 01:23SXXRelationship addedrelated to 0002098
2015-10-14 14:53SXXNote Added: 0005946
2015-10-14 22:28SXXNote Added: 0005947
2015-10-17 11:45SXXNote Added: 0005959
2015-10-17 11:45SXXStatusassigned => resolved
2015-10-17 11:45SXXFixed in Version => 0.98e
2015-10-17 11:45SXXResolutionopen => fixed

Notes
(0003437)
Warmonger   
2013-03-05 18:13   
Needs stack->hero prpagator, but most likely also pathfinder checking for specific terrain penalties - not only general "Pathfinding" secondary skill as it is now.
(0004628)
Warmonger   
2014-03-29 20:40   
Just figured out that "HERO" propagator already exists, will check this.
(0005942)
SXX   
2015-10-14 01:22   
Some implementation there:
https://github.com/ArseniyShestakov/vcmi/commit/9cfb7a86b7b24b535fe32d4739c5400c89f28315 [^]

A bit of work on client still needed to make it update path information when Nomads added or removed from army, but I think it's related to "heroBonusChanged ".
(0005946)
SXX   
2015-10-14 14:53   
With Warmonger suggestion applied an updated version:
https://github.com/ArseniyShestakov/vcmi/commit/737b579e71a3d1b4a04d3c882d02aaa6df81e41f [^]
(0005947)
SXX   
2015-10-14 22:28   
Pull request:
https://github.com/vcmi/vcmi/pull/123 [^]
As far as I tested new bonus shouldn't affect saves, but anyway.
(0005959)
SXX   
2015-10-17 11:45   
Fixed:
https://github.com/vcmi/vcmi/commit/8c9f8f22d9467439859b2fdc87cd82251f73383c [^]