MantisBT - VCMI
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000889VCMIMechanics - Objectspublic2012-03-01 14:312022-04-12 09:17
ReporterZamolxis 
Assigned ToSXX 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version0.87 
Target VersionFixed in Version0.98f 
Summary0000889: Angel Wings don't work anymore
DescriptionHero with Angel Wings cannot fly anymore. Can be reproduced with Astral in the attached saved game. File is from 0.87b, but the bug was introduced in 0.87 (in 0.86 the artifact still worked).
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0000627closed SXX Fly bug 
related to 0000476closed SXX Fly does not ignore all obstacles when there's a reasonably alternative available on land (+ other issues with the Fly spell) 
has duplicate 0001780closed SXX Angel Wings artifact not working 
related to 0002191resolved SXX Boots of Levitation don't work anymore 
Attached Files7z 87b_AngelWings.7z (73,694) 2012-03-01 14:31
https://bugs.vcmi.eu/file_download.php?file_id=855&type=bug
Issue History
Date ModifiedUsernameFieldChange
2012-03-01 14:31ZamolxisNew Issue
2012-03-01 14:31ZamolxisStatusnew => assigned
2012-03-01 14:31ZamolxisAssigned To => Warmonger
2012-03-01 14:31ZamolxisFile Added: 87b_AngelWings.7z
2012-03-01 14:36WarmongerRelationship addedrelated to 0000627
2012-03-01 14:36WarmongerRelationship addedrelated to 0000476
2012-03-01 14:37WarmongerNote Added: 0002262
2012-03-01 14:37WarmongerAssigned ToWarmonger =>
2012-03-01 14:38WarmongerStatusassigned => new
2012-03-01 14:47ZamolxisNote Added: 0002263
2012-03-01 15:13ZamolxisNote Edited: 0002263bug_revision_view_page.php?bugnote_id=2263#r1517
2012-03-01 23:30TowNote Added: 0002267
2012-03-01 23:30TowStatusnew => confirmed
2013-03-01 22:56ZamolxisNote Added: 0003421
2013-03-01 23:27IvanNote Added: 0003423
2013-03-02 00:12ZamolxisNote Added: 0003425
2013-03-02 09:11IvanNote Added: 0003426
2014-05-03 13:30WarmongerRelationship addedhas duplicate 0001780
2015-03-11 22:57SXXAssigned To => SXX
2015-03-11 22:57SXXStatusconfirmed => assigned
2015-06-21 21:20AVSRelationship addedrelated to 0002191
2015-06-21 21:20AVSRelationship deletedrelated to 0002191
2015-06-21 21:21AVSRelationship addedrelated to 0002191
2015-11-24 10:19SXXNote Added: 0006101
2015-11-24 10:19SXXStatusassigned => resolved
2015-11-24 10:19SXXFixed in Version => 0.98f
2015-11-24 10:19SXXResolutionopen => fixed
2022-04-12 09:17PovelitelStatusresolved => closed

Notes
(0002262)
Warmonger   
2012-03-01 14:37   
As far as I know, Fly has been disabled at some point due to related issues.
(0002263)
Zamolxis   
2012-03-01 14:47   
(edited on: 2012-03-01 15:13)
Possibly. Anyway, as long as the official spreadsheet marks both the artifact and related spell as implemented, then from a tester's perspective this is a bug (a minor one though, so it can definitely wait for a future release).
(0002267)
Tow   
2012-03-01 23:30   
I removed it when I was improving pathfinder. The "flying" handling code was only making pathfinder more messy and needed a total rewrite anyway. Spending time to keep broken code would be wasteful.
(0003421)
Zamolxis   
2013-03-01 22:56   
Shall we maybe add a note to the Item Implementation List that Fly spell & Angel Wings are disabled?
(0003423)
Ivan   
2013-03-01 23:27   
Wow. You've managed to make note exactly 1 year after previous change :)

On topic - that list was extremely useful during development of 0.7-0.8 series but now I'm not sure - most of missing features are not objects like what we have in list but rather pieces of missing mechanics which sometimes need complex description (like 0001166).

Perhaps more efficient way will be using bug tracker for them - add "missing feature" tag (as opposed to feature requests) or to keep some "master report" and make all bugs like this as its children.
(0003425)
Zamolxis   
2013-03-02 00:12   
A year indeed. That explains why I almost forgot about it... and went to "the list". But it also rang a bell so I came here. But newer testers may get confused.

Though you're the coder(s), so maybe see with Tow & co what's most practical from your perspective.

As a tester I found the list very practical, but it would remain practical if a note is made for what is implemented, but for some reason disabled or "in the works".
(0003426)
Ivan   
2013-03-02 09:11   
I just don't see that list will be easy to use for both players and devs now when more than 90% of objects are implemented.

Plus we have some objects that are partially implemented (like anti-magic garrison).

Keeping all of this in bugtracker with appropriate descriptions is better idea IMO.

And for players we can provide bug filter that will display only missing features.
(0006101)
SXX   
2015-11-24 10:19   
Fly implemented:
https://github.com/vcmi/vcmi/commit/a85106289181b87af2be857d0d89d5a09dda1934 [^]