Anonymous | Login | 2024-11-21 14:58 UTC |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0000961 | VCMI | Mechanics - Battles | public | 2012-05-18 18:12 | 2014-05-30 17:41 | ||||
Reporter | Ivan | ||||||||
Assigned To | Ivan | ||||||||
Priority | low | Severity | tweak | Reproducibility | always | ||||
Status | closed | Resolution | fixed | ||||||
Platform | x86_64 | OS | Ubuntu Linux | OS Version | 12.04 (precise) | ||||
Product Version | 0.88 | ||||||||
Target Version | Fixed in Version | 0.89 | |||||||
Summary | 0000961: Wrong terrain in battle | ||||||||
Description | Map Arrogance, sawmill guardians near starting town. This battle always happens on clouds with expert Air magic bonus. I don't see any source of clouds nearby - correct terrain should be normal dirt. | ||||||||
Tags | No tags attached. | ||||||||
Attached Files | |||||||||
Notes | |
(0002494) Tow (developer) 2012-05-18 22:58 |
There are two issues here. Object from another map level (underground) were incorrectly considered for battlefield overlays and that is fixed in r2691. There is also another problem — terrain overlays can have various shapes, not only 8x6. The coveringAt method was used for checking if the given tile is covered by overlay. It depended on coverage maps that were disabled in r2550 as part of fix for 0000866. Coverage maps are not only GUI hint, they also affect game logic. I'm wondering, if it would be a good idea to programatically recreate all coverage maps (it shouldn't be hard to check where def is transparent and where it is not) and put that info into some JSON config. What do you think? |
(0002500) Ivan (developer) 2012-05-19 08:13 |
I was thinking about turning all objects shapes into rectangles (as in editor) on loading using existing msk's. IIRC this will work with all overlays. I'll see what I can do without breaking 0000866 If not - then I'll move them to json one way or another. |
(0002501) Ivan (developer) 2012-05-19 11:53 |
Should be fixed in rev. 2694 Instead of blockmaps\coverage maps code uses width and height values from msk. It should be correct for all overlays. |
Issue History | |||
Date Modified | Username | Field | Change |
2012-05-18 18:12 | Ivan | New Issue | |
2012-05-18 22:46 | Tow | Product Version | 0.dev => 0.88 |
2012-05-18 22:47 | Tow | Relationship added | related to 0000866 |
2012-05-18 22:58 | Tow | Note Added: 0002494 | |
2012-05-18 22:58 | Tow | Assigned To | => Tow |
2012-05-18 22:58 | Tow | Status | new => feedback |
2012-05-19 08:13 | Ivan | Note Added: 0002500 | |
2012-05-19 08:13 | Ivan | Status | feedback => assigned |
2012-05-19 08:14 | Ivan | Assigned To | Tow => Ivan |
2012-05-19 11:53 | Ivan | Note Added: 0002501 | |
2012-05-19 11:53 | Ivan | Status | assigned => resolved |
2012-05-19 11:53 | Ivan | Fixed in Version | => 0.89 |
2012-05-19 11:53 | Ivan | Resolution | open => fixed |
2014-05-30 17:41 | beegee | Status | resolved => closed |
Copyright © 2000 - 2024 MantisBT Team |