Opened 15 years ago
Closed 15 years ago
#3134 closed defect (wontfix)
Download errors not displayed
Reported by: | stoecker | Owned by: | jttt |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | Core | Version: | latest |
Keywords: | Cc: |
Description
The progress dialog does not vanish in case of download errors, but also the error message is not displayed.
Test: try downloading a much to large area.
Attachments (2)
Change History (5)
comment:1 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
by , 15 years ago
Attachment: | down-error.patch added |
---|
comment:2 by , 15 years ago
Priority: | major → minor |
---|---|
Resolution: | fixed |
Status: | closed → reopened |
The error handling while downloading is still fragile. The PleaseWaitDialog may not disappear if an error was thrown. At the moment I have no real User Action to cause this error. But if you apply produce-error.patch a synthetic exception is thrown and JOSM does not catch it correctly.
If this is too academic close this ticket again. I'll open it again if found a user action to cause this error (maybe disturb network connections).
DO NOT INSERT produce-error.patch in SVN....
The problem: The finish function might not be called if after createSubTaskMonitor finishTask for the subTaskMonitor is not called. I tried to correct this (see down-error.patch) but this is not very nice. I think taskFinish of the main ProgressMonitor should in all cases call the doFinish function even if some subtasks are not finished correctly.
comment:3 by , 15 years ago
Resolution: | → wontfix |
---|---|
Status: | reopened → closed |
If this is too academic close this ticket again. I'll open it again if found a user action to cause this error (maybe disturb network connections).
I'm closing the ticket for the time being. Please reopen if necessary.
Fixed in r1875