MantisBT - VCMI
View Issue Details
0002422VCMIMechanics - Battlespublic2016-03-02 12:372016-09-23 05:21
Kantor 
AVS 
normalminoralways
resolvedfixed 
0.98g 
0.99 
0002422: Not all hexes can be used to place mines
Trying to fill battlefield with mines will always leave some hexes untouched.
No tags attached.
jpg 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
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

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 [^]