MantisBT - VCMI
View Issue Details
0000277VCMIGUI - Hero screen / Exchange windowpublic2009-12-02 09:162010-07-13 10:59
Steven Aus 
OnionKnight 
normalcrashsometimes
closedfixed 
0.75 
0.8 
0000277: There is a reasonable likelihood of crashes when moving more than 1 or 2 artifacts in the hero transfer screen.
In the attached save, Jenova and Ufretin are in boats and they both have a fair number of artifacts which can't really be exchanged very easily. =)
No tags attached.
related to 0000309closed  Crash when we have only 1 artifact in the backpack and we try to select it 
7z HeroExchangeCrashes.7z (109,510) 2009-12-02 09:16
https://bugs.vcmi.eu/file_download.php?file_id=130&type=bug
7z Necklace Crash.7z (109,370) 2009-12-02 09:39
https://bugs.vcmi.eu/file_download.php?file_id=131&type=bug
Issue History
2009-12-02 09:16Steven AusNew Issue
2009-12-02 09:16Steven AusFile Added: HeroExchangeCrashes.7z
2009-12-02 09:38Steven AusNote Added: 0000348
2009-12-02 09:39Steven AusFile Added: Necklace Crash.7z
2009-12-02 09:40Steven AusNote Edited: 0000348bug_revision_view_page.php?bugnote_id=348#r430
2009-12-02 09:41Steven AusNote Edited: 0000348bug_revision_view_page.php?bugnote_id=348#r431
2009-12-02 21:23OnionKnightStatusnew => assigned
2009-12-02 21:23OnionKnightAssigned To => OnionKnight
2009-12-02 21:25OnionKnightNote Added: 0000365
2009-12-03 00:11OnionKnightNote Added: 0000374
2009-12-03 00:11OnionKnightStatusassigned => feedback
2009-12-03 02:05anonymousNote Added: 0000375
2009-12-03 02:06anonymousNote Deleted: 0000375
2009-12-03 02:07Steven AusNote Added: 0000376
2009-12-03 11:56OnionKnightStatusfeedback => resolved
2009-12-03 11:56OnionKnightResolutionopen => fixed
2009-12-03 11:57OnionKnightStatusresolved => feedback
2009-12-03 11:57OnionKnightResolutionfixed => reopened
2009-12-03 11:57OnionKnightStatusfeedback => resolved
2009-12-03 11:57OnionKnightFixed in Version => 0.89
2009-12-03 11:57OnionKnightResolutionreopened => fixed
2009-12-22 05:44OnionKnightRelationship addedhas duplicate 0000309
2009-12-22 10:48ZamolxisRelationship replacedrelated to 0000309
2010-07-13 10:59ZamolxisNote Added: 0001077
2010-07-13 10:59ZamolxisStatusresolved => closed

Notes
(0000348)
Steven Aus   
2009-12-02 09:38   
(edited on: 2009-12-02 09:41)
I have narrowed down one source of crashes. When left-clicking (right-clicking appears to be fine) on the Necklace of Swiftness in Jenova's backpack (Necklace Crash), either in her hero screen or in the transfer screen, it always crashes. Some other source of artifact transfer crashes might be more intermittent, as I don't think the Necklace of Swiftness is the only artifact transfer or left-clicking that crashes, just that in the attached save it *always* crashes when that artifact is left-clicked on.

(0000365)
OnionKnight   
2009-12-02 21:25   
I was worried there would be a whole lot of bug reports regarding the artifact screen, since the 0.75 version was in the middle of a significant rewrite of that part. I'll look into it and see if the problem is still there.
(0000374)
OnionKnight   
2009-12-03 00:11   
It looks like it's not possible to move the last artifact from a backpack, this causes a division by zero when it tries to update the backpack information. I thought I had tested this, but it's likely it got reintroduced somewhere on the way when I figured that calling scrollBackpack(0) was a good way to update backpack information and then never really tested it again.
This problem is fixed, but I'll leave it open since the report is not clear if this is the only problem.
(0000376)
Steven Aus   
2009-12-03 02:07   
I think this issue can be closed. I moved a heap of artifacts between Jenova and Ufretin, always making sure I didn't try to left-click on the last artifact in a hero's backpack, and it didn't crash at all. I think it's just that division by zero error for the last backpack item as you say.
(0001077)
Zamolxis   
2010-07-13 10:59   
Cannot reproduce anymore in 0.81. Probably fixed indeed. Please reopen if it reoccurs.