After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 516178 - Scheme error during progress dialog causes freeze
Scheme error during progress dialog causes freeze
Status: VERIFIED FIXED
Product: GnuCash
Classification: Other
Component: Import - QIF
git-master
Other All
: Normal normal
: ---
Assigned To: Derek Atkins
Derek Atkins
Depends on:
Blocks: backport
 
 
Reported: 2008-02-13 06:16 UTC by Charles Day
Modified: 2018-06-29 22:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Easy-to-read patch (svn -x -w) (2.35 KB, patch)
2008-02-14 05:22 UTC, Charles Day
none Details | Review
Proposed patch (16.37 KB, patch)
2008-02-14 05:24 UTC, Charles Day
committed Details | Review

Description Charles Day 2008-02-13 06:16:02 UTC
Please describe the problem:
If a Scheme error occurs during in the qif-to-gnc procedure while a progress bar is visible, the druid freezes and GnuCash must be killed.


Steps to reproduce:
1. Cause a Scheme error to occur during the qif-to-gnc procedure after a progress dialog is shown. For example, edit qif-to-gnc.scm and insert a bad expression such as (qwerty) in the code after the call to gnc-progress-dialog-new.
2. Attempt to import a QIF file that is large enough to cause a progress bar to appear. For example, use the QIF file attached to bug 515888.

Actual results:
The druid becomes frozen when the error occurs.

Expected results:
The error is handled in the same manner as when there isn't a progress dialog.

Does this happen every time?
Yes.

Other information:
Comment 1 Charles Day 2008-02-14 05:22:17 UTC
Created attachment 105199 [details] [review]
Easy-to-read patch (svn -x -w)

Don't apply this patch. Just read it. It is just to make it easier to find the real substance of the patch by ignoring whitespace changes (and there were many, due to indentation from wrapping the entire existing procedure in a new "let").

The full patch, including the whitespace changes, will attached next.
Comment 2 Charles Day 2008-02-14 05:24:12 UTC
Created attachment 105200 [details] [review]
Proposed patch

This is the one to apply.
Comment 3 Charles Day 2008-02-23 05:40:56 UTC
Committed as r16954. Awaiting backport for 2.2.4.
Comment 4 Andreas Köhler 2008-02-28 22:27:31 UTC
Applied to branches/2.2 as r16972 for GnuCash 2.2.4.
Thanks a lot!
Comment 5 John Ralls 2018-06-29 22:01:09 UTC
GnuCash bug tracking has moved to a new Bugzilla host. This bug has been copied to https://bugs.gnucash.org/show_bug.cgi?id=516178. Please update any external references or bookmarks.