0001268: Game freezes when Treasure Chest grants level

(0003549)
Ivan (developer)
2013-05-24 18:30

For me it works - both player and AI can pick chests without any bugs. Using latest trunk.

Although I've found another evil evil bug that may be related - new report incoming.

(0003550)
Warmonger (administrator)
2013-05-24 19:03
edited on: 2013-05-24 19:06

Okay, I rebuild recent revision and now it works for human, but still not for AI.

(0003555)
anonymous (viewer)
2013-05-25 21:43

Works for me.
Can you attach the map / savegame, where pressing "end turn" triggers this?

(0003556)
Tow (developer)
2013-05-25 21:44

@up
It's me.

(0003561)
Zamolxis (viewer)
2013-05-27 10:15
edited on: 2013-05-27 10:23

@Tow: I'm getting a crash after fighting at Pandora's Box, leveling up both battle experience and Pandora bonus. Can it be related and I should wait for this to be fixed first, or shall I prepare a separate report?

EDIT: It's not related to spell casting (just to rule that out, I played the battle without spells). It's the VCMI_Tests_2012 map - Pandora Box just above Rion at map start.

(0003562)
Tow (developer)
2013-05-27 10:17

I'll check this in a moment.

(0003563)
Tow (developer)
2013-05-27 10:21

I just checked — I don't get a crash when hero levels up both from defeating Pandora's Box creatures and exp bonus. Some other factors may be involved.
Please create a report.

(0003564)
Zamolxis (viewer)
2013-05-27 10:36

0001271 created

(0003573)
Warmonger (administrator)
2013-05-28 05:03

Uploaded savegame. It starts endless loop once red Barbarian picks the chest.

Playing with autoSkip option if that could make a difference.

(0003574)
Tow (developer)
2013-05-28 10:02

Which revision was used to produce this save? I doesn't work for me neither with trunk nor 0.92b.

(0003575)
Warmonger (administrator)
2013-05-28 11:13

Oops, I forgot to get rid of whole array of mods in first place :P

(0003576)
Warmonger (administrator)
2013-05-28 11:18

Try now.

(0003577)
Tow (developer)
2013-05-28 11:32

Finally reproduced!
Many thanks for that! :)

(0003578)
Tow (developer)
2013-05-28 14:03

One more request: could you upload the h3m with map used for this game?

(0003579)
Warmonger (administrator)
2013-05-28 16:35

Done. I use this map quite a lot for about two years ;)

(0003582)
Tow (developer)
2013-05-28 22:49

The map is invalid. It contains only 2 allowed heroes. Therefore it is impossible to properly give starting heroes to all the players.

The bug was caused because one of players was given a hero that was imprisoned somewhere else on map. Duplicate hero = distaster sooner or later. [Recent chagnes to hero serialization made it much sooner.]

Fixed in r3391. Now VCMI will use heroes that are not allowed but will be careful about not taking the ones already used (normally or in prison).

(0003583)
Warmonger (administrator)
2013-05-29 04:00

In OH3, if imprisoned hero is allowed, you are free to choose him when starting game. In such case, the prison becomes empty.

Also, I complained about prison being hero object like three years ago :P

Notes
(0003549) Ivan (developer) 2013-05-24 18:30	For me it works - both player and AI can pick chests without any bugs. Using latest trunk. Although I've found another evil evil bug that may be related - new report incoming.

(0003550) Warmonger (administrator) 2013-05-24 19:03 edited on: 2013-05-24 19:06	Okay, I rebuild recent revision and now it works for human, but still not for AI.

(0003555) anonymous (viewer) 2013-05-25 21:43	Works for me. Can you attach the map / savegame, where pressing "end turn" triggers this?

(0003556) Tow (developer) 2013-05-25 21:44	@up It's me.

(0003561) Zamolxis (viewer) 2013-05-27 10:15 edited on: 2013-05-27 10:23	@Tow: I'm getting a crash after fighting at Pandora's Box, leveling up both battle experience and Pandora bonus. Can it be related and I should wait for this to be fixed first, or shall I prepare a separate report? EDIT: It's not related to spell casting (just to rule that out, I played the battle without spells). It's the VCMI_Tests_2012 map - Pandora Box just above Rion at map start.

(0003562) Tow (developer) 2013-05-27 10:17	I'll check this in a moment.

(0003563) Tow (developer) 2013-05-27 10:21	I just checked — I don't get a crash when hero levels up both from defeating Pandora's Box creatures and exp bonus. Some other factors may be involved. Please create a report.

(0003564) Zamolxis (viewer) 2013-05-27 10:36	0001271 created

(0003573) Warmonger (administrator) 2013-05-28 05:03	Uploaded savegame. It starts endless loop once red Barbarian picks the chest. Playing with autoSkip option if that could make a difference.

(0003574) Tow (developer) 2013-05-28 10:02	Which revision was used to produce this save? I doesn't work for me neither with trunk nor 0.92b.

(0003575) Warmonger (administrator) 2013-05-28 11:13	Oops, I forgot to get rid of whole array of mods in first place :P

(0003576) Warmonger (administrator) 2013-05-28 11:18	Try now.

(0003577) Tow (developer) 2013-05-28 11:32	Finally reproduced! Many thanks for that! :)

(0003578) Tow (developer) 2013-05-28 14:03	One more request: could you upload the h3m with map used for this game?

(0003579) Warmonger (administrator) 2013-05-28 16:35	Done. I use this map quite a lot for about two years ;)

(0003582) Tow (developer) 2013-05-28 22:49	The map is invalid. It contains only 2 allowed heroes. Therefore it is impossible to properly give starting heroes to all the players. The bug was caused because one of players was given a hero that was imprisoned somewhere else on map. Duplicate hero = distaster sooner or later. [Recent chagnes to hero serialization made it much sooner.] Fixed in r3391. Now VCMI will use heroes that are not allowed but will be careful about not taking the ones already used (normally or in prison).

(0003583) Warmonger (administrator) 2013-05-29 04:00	In OH3, if imprisoned hero is allowed, you are free to choose him when starting game. In such case, the prison becomes empty. Also, I complained about prison being hero object like three years ago :P

Issue History
Date Modified	Username	Field	Change
2013-05-24 17:21	Warmonger	New Issue
2013-05-24 18:30	Ivan	Note Added: 0003549
2013-05-24 19:03	Warmonger	Note Added: 0003550
2013-05-24 19:06	Warmonger	Note Edited: 0003550	View Revisions
2013-05-25 21:43	anonymous	Note Added: 0003555
2013-05-25 21:43	anonymous	Status	new => feedback
2013-05-25 21:44	Tow	Note Added: 0003556
2013-05-25 21:44	Tow	Assigned To	=> Tow
2013-05-25 21:44	Tow	Status	feedback => assigned
2013-05-27 10:15	Zamolxis	Note Added: 0003561
2013-05-27 10:15	Zamolxis	Status	assigned => feedback
2013-05-27 10:17	Tow	Note Added: 0003562
2013-05-27 10:17	Zamolxis	Note Edited: 0003561	View Revisions
2013-05-27 10:21	Tow	Note Added: 0003563
2013-05-27 10:23	Zamolxis	Note Edited: 0003561	View Revisions
2013-05-27 10:36	Zamolxis	Note Added: 0003564
2013-05-28 05:02	Warmonger	File Added: Loop1268.rar
2013-05-28 05:03	Warmonger	Note Added: 0003573
2013-05-28 05:03	Warmonger	Status	feedback => assigned
2013-05-28 10:02	Tow	Note Added: 0003574
2013-05-28 11:13	Warmonger	Note Added: 0003575
2013-05-28 11:17	Warmonger	File Deleted: Loop1268.rar
2013-05-28 11:17	Warmonger	File Added: Loop1268.rar
2013-05-28 11:18	Warmonger	Note Added: 0003576
2013-05-28 11:32	Tow	Note Added: 0003577
2013-05-28 11:32	Tow	Status	assigned => confirmed
2013-05-28 14:03	Tow	Note Added: 0003578
2013-05-28 16:34	Warmonger	File Added: CoTD 252x252 001.h3m
2013-05-28 16:35	Warmonger	Note Added: 0003579
2013-05-28 22:49	Tow	Note Added: 0003582
2013-05-28 22:49	Tow	Status	confirmed => resolved
2013-05-28 22:49	Tow	Fixed in Version	=> 0.93
2013-05-28 22:49	Tow	Resolution	open => fixed
2013-05-29 04:00	Warmonger	Note Added: 0003583
2014-05-30 17:43	beegee	Status	resolved => closed

Relationships