Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 332245 - do not show "See error log for details" if no details are available
Summary: do not show "See error log for details" if no details are available
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 trivial (vote)
Target Milestone: 3.5   Edit
Assignee: Thomas Ehrnhoefer CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-12-09 14:44 EST by Thomas Ehrnhoefer CLA
Modified: 2011-02-17 22:58 EST (History)
1 user (show)

See Also:

Attachments
patch v1. (1.03 KB, patch)
2010-12-09 14:47 EST, Thomas Ehrnhoefer CLA 		no flags Details | Diff
mylyn/context/zip (4.55 KB, application/octet-stream)
2010-12-09 14:47 EST, Thomas Ehrnhoefer CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Ehrnhoefer CLA 2010-12-09 14:44:58 EST 
Currently if for example a QuerySynchronization fails, TasksUiInternal.logAndDisplayStatus() is called, which calls displayStatus() with showLinkToErrorLog true.
This is a problem if the there is only one Status to display, and this Status is neither a MultiStatus nor does it have an exception (and thus a stacktrace to display). The user gets faced with "see error log for details", but there are no additional details visible in the error log.

Currently the QuerySyncJob transforms all stati into a MultiStatus, so even a single Status becomes a MultiStatus, which would need to change, too.
Comment 1 Thomas Ehrnhoefer CLA 2010-12-09 14:47:04 EST 
Created attachment 184893 [details]
patch v1.

this would fix this for my use case
Comment 2 Thomas Ehrnhoefer CLA 2010-12-09 14:47:05 EST 
Created attachment 184894 [details]
mylyn/context/zip
Comment 3 Steffen Pingel CLA 2011-01-07 16:16:54 EST 
Agreed. I have removed the message in the tooltip for now. We should implement a smarter way to drill into the error details, e.g. through a assigning a shortcut.
Comment 4 Steffen Pingel CLA 2011-02-17 22:57:48 EST 
Reopening to fix assignment.
Comment 5 Steffen Pingel CLA 2011-02-17 22:58:01 EST 
Closing.