MantisBT

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0002066VCMIMechanics - Adventure Mappublic2015-02-10 22:012015-10-15 02:38
ReporterSXX 
Assigned ToSXX 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version0.97b 
Target VersionFixed in Version0.97c 
Summary0002066: First hero movement after town / non-pickable object visit can't be interrupted
DescriptionSo if you're accidentally sent hero to attack some monster it's impossible to interrupt this movement and hero going to die. Though it's only happen on very first movement of hero after visit and any movement after that can be interrupted just fine.

This problem only happen after visiting objects that spawn dialogue window, but aren't pickable. E.g it's won't happen after visiting/picking Campfire or Treasure Chest.

Also it's will happen even after visiting object you will use other hero before and only then switch back to that hero that visited it.
Steps To Reproduce  - Run normal game in VCMI.
  - Now with hero visit any town or if it's here just open town screen.
  - Close town screen
  - Hero will be automatically selected at this point.
  - Sent this hero to any point on map.
  - Try to interrupt movement using mouse buttons or space bar.

You'll notice it's impossible to interrupt this movement no matter what.

Same will happen after hero visit non-pickable object (Learning stone, etc).
TagsNo tags attached.
Attached Files

- Relationships
related to 0000984closedSXX Hero move on adv. map should be interruptible 

-  Notes
(0005520)
SXX (administrator)
2015-02-15 23:36

So I almost find out about this one. Problem occur because of check in "CPlayerInterface::doMoveHero". There is "curTile" that was added to avoid hero stopping at blocked tile while flying:
https://github.com/vcmi/vcmi/commit/71b73bad530224333e21df1ab51846c7380ed51b [^]

This check itself is broken because it's never change after hero first move. Also this check possible always return "blocked" for the tile where hero itself standing while I not sure about that because there no such problem with removable objects.
(0005523)
SXX (administrator)
2015-02-16 07:28
edited on: 2015-02-16 07:50

So I find out why it's happen. As curTile a pointer to TerrainTile it's become unblocked once hero leave this tile, but this doesn't happen for dwellings as they always remain here and marked as "blocked".

Currently I'm going to find out how fly effect worked with map objects in original to make sure behavior I'm going implement is correct one.

(0005524)
SXX (administrator)
2015-02-16 07:53

Here is H3 behaviour: fly skill prefer to flying above non-blocked object (including corpse!) and it's does let you stop on dwellings entry points except they're blocked by another hero or not allowed to stop on them at all (corpse).
(0005525)
SXX (administrator)
2015-02-16 08:08

Also in H3 it's was possible to stay on some objects like magic well without actually visiting them.
(0005552)
Ivan (developer)
2015-02-20 20:21

>> and it's does let you stop on dwellings entry points except they're blocked by another hero

Isn't this is an obvious bug in H3? Try "capturing" towns this way - IIRC this completely breaks H3 creating ghost armies in town garrison and such.

(IMO possibility to *stop* on visitables without possibility to *move* on them directly is definitely wrong)
(0005560)
SXX (administrator)
2015-02-22 06:39

I'm kind a agree with you on that and it's will make whole fly pathfinding a bit easier, but not sure if I want to make decision to disable that or not.

Hope @Warmonger or other devs may comment on this one too.
(0005609)
SXX (administrator)
2015-03-10 18:55

Should now be fixed with merge of my teleport branch:
https://github.com/vcmi/vcmi/commit/ec879046ca52ecc5c431f4409ff9c3bd75363d26 [^]

As this check didn't work at all and It's was only cause problems with testing teleports I dropped it. In future when movement re-implemented for cheat-free flying and water walking we should make sure to not do same mistake again.

- Issue History
Date Modified Username Field Change
2015-02-10 22:01 SXX New Issue
2015-02-10 22:02 SXX Relationship added related to 0000984
2015-02-15 23:31 SXX Assigned To => SXX
2015-02-15 23:31 SXX Status new => assigned
2015-02-15 23:36 SXX Note Added: 0005520
2015-02-16 07:28 SXX Note Added: 0005523
2015-02-16 07:50 SXX Note Edited: 0005523 View Revisions
2015-02-16 07:53 SXX Note Added: 0005524
2015-02-16 08:08 SXX Note Added: 0005525
2015-02-20 20:21 Ivan Note Added: 0005552
2015-02-22 06:39 SXX Note Added: 0005560
2015-03-10 18:55 SXX Note Added: 0005609
2015-03-10 18:55 SXX Status assigned => resolved
2015-03-10 18:55 SXX Fixed in Version => 0.97c
2015-03-10 18:55 SXX Resolution open => fixed
2015-10-15 02:38 SXX Status resolved => closed

Site | Forums | Wiki | Slack | GitHub


Copyright © 2000 - 2024 MantisBT Team
Hosting provided by DigitalOcean