MantisBT - VCMI
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0002422VCMIMechanics - Battlespublic2016-03-02 12:372022-04-12 11:04
ReporterKantor 
Assigned ToAVS 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version0.98g 
Target VersionFixed in Version0.99 
Summary0002422: Not all hexes can be used to place mines
DescriptionTrying to fill battlefield with mines will always leave some hexes untouched.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filesjpg minefield.jpg (549,103) 2016-03-02 12:37
https://bugs.vcmi.eu/file_download.php?file_id=2472&type=bug
jpg

jpg minefield 2.jpg (525,165) 2016-03-02 20:51
https://bugs.vcmi.eu/file_download.php?file_id=2473&type=bug
jpg
Issue History
Date ModifiedUsernameFieldChange
2016-03-02 12:37KantorNew Issue
2016-03-02 12:37KantorFile Added: minefield.jpg
2016-03-02 17:04AVSNote Added: 0006465
2016-03-02 17:04AVSAssigned To => AVS
2016-03-02 17:04AVSStatusnew => feedback
2016-03-02 17:16WarmongerNote Added: 0006466
2016-03-02 17:24AVSNote Added: 0006467
2016-03-02 18:31AVSNote Added: 0006468
2016-03-02 18:31AVSStatusfeedback => assigned
2016-03-02 20:51KantorNote Added: 0006469
2016-03-02 20:51KantorFile Added: minefield 2.jpg
2016-03-03 10:48AVSNote Added: 0006470
2016-09-23 05:21SXXNote Added: 0006724
2016-09-23 05:21SXXStatusassigned => resolved
2016-09-23 05:21SXXFixed in Version => 0.99
2016-09-23 05:21SXXResolutionopen => fixed
2022-04-12 11:04PovelitelStatusresolved => closed

Notes
(0006465)
AVS   
2016-03-02 17:04   
I think this this not a bug.
(0006466)
Warmonger   
2016-03-02 17:16   
Not sure about mines, but certainly you could fill entire battlefield with Quick Sand. Did that many times ;)
(0006467)
AVS   
2016-03-02 17:24   
@Warmonger, We have explicit check "(hex.getX() > 2 && hex.getX() < 14)" for both Land Mine and Quick Sand ant it looks like done on purpose, is it just wrong?
(0006468)
AVS   
2016-03-02 18:31   
Fixed in https://github.com/vcmi/vcmi/pull/198. [^] But still not sure )
(0006469)
Kantor   
2016-03-02 20:51   
^ Now both mines and quick sand hexes can appear outside the battlefield :)
(0006470)
AVS   
2016-03-03 10:48   
Fixed the fix
(0006724)
SXX   
2016-09-23 05:21   
Tested. Fixed in PR198:
https://github.com/vcmi/vcmi/pull/198 [^]