MantisBT - VCMI
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0002366VCMIMechanics - Battlespublic2016-01-06 13:252017-07-03 20:49
ReporterSXX 
Assigned ToFeniksFire 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version0.98f 
Target VersionFixed in Version1.next 
Summary0002366: Spells: targeted obstacle-creating spells must placeable on moat
DescriptionCurrently you can't place force field or fire wall in moat while this should be possible.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0002186closed AVS Fire Wall: can be placed within castle wall 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-01-06 13:25SXXNew Issue
2016-01-06 13:26SXXNote Added: 0006250
2016-01-06 13:47SXXRelationship addedrelated to 0002186
2016-01-06 13:47SXXAssigned To => AVS
2016-01-06 13:47SXXStatusnew => assigned
2016-01-06 14:53AVSNote Added: 0006252
2016-01-06 16:43SXXNote Added: 0006254
2017-06-21 18:11FeniksFireNote Added: 0007097
2017-06-21 18:12FeniksFireNote Edited: 0007097bug_revision_view_page.php?bugnote_id=7097#r3360
2017-07-03 20:49SXXNote Added: 0007100
2017-07-03 20:49SXXStatusassigned => resolved
2017-07-03 20:49SXXFixed in Version => 1.next
2017-07-03 20:49SXXResolutionopen => fixed
2017-07-03 20:49SXXAssigned ToAVS => FeniksFire

Notes
(0006250)
SXX   
2016-01-06 13:26   
I'm working on related moat issues (0000726), but not sure if going to fix it.
So let's it'll be there.
(0006252)
AVS   
2016-01-06 14:53   
This one may be tricky. Need to check regressions caused by 2 obstacles on same hex.
(0006254)
SXX   
2016-01-06 16:43   
I'll likely fix damage handling of more than one obstacles, but not sure what other problems are there. If you know what else need to be fixed give me some hint.
(0007097)
FeniksFire   
2017-06-21 18:11   
(edited on: 2017-06-21 18:12)
I solved it some time ago, I will throw it together with other patches when I finish them.
(0007100)
SXX   
2017-07-03 20:49   
Fixed: https://github.com/vcmi/vcmi/pull/318 [^]