Bug 335524 - include task id in error message when synchronization fails
Summary: include task id in error message when synchronization fails
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: contributed
Depends on:
Blocks:
 
Reported: 2011-01-26 17:59 EST by Thomas Ehrnhoefer CLA
Modified: 2011-02-01 23:24 EST (History)
1 user (show)

See Also:

Attachments
patch (2.54 KB, patch)
2011-01-26 18:00 EST, Thomas Ehrnhoefer CLA 		no flags Details | Diff
patch v2 (4.43 KB, patch)
2011-01-26 18:02 EST, Thomas Ehrnhoefer CLA 		no flags Details | Diff
patch v3 (4.51 KB, text/plain)
2011-01-26 18:53 EST, Thomas Ehrnhoefer CLA 		no flags Details
patch v3.1 (4.51 KB, patch)
2011-01-26 18:55 EST, Thomas Ehrnhoefer CLA 		steffen.pingel: iplog+
Details | Diff
Show Obsolete (3) 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 2011-01-26 17:59:24 EST 
Currently a failure in synchronizing related tasks will lead to "Synchronization failed" in the error log. The details list the error message from the server, which does not necessarily have more information on what the task is either.
This is not very helpful, as the user has no idea what task failed to synchronize.
Comment 1 Thomas Ehrnhoefer CLA 2011-01-26 18:00:36 EST 
Created attachment 187696 [details]
patch

minimalist approach to add some information. We might want the full Url though, or the repository label as well.
Comment 2 Thomas Ehrnhoefer CLA 2011-01-26 18:02:47 EST 
Created attachment 187697 [details]
patch v2

this one add repo label to the error message
Comment 3 Steffen Pingel CLA 2011-01-26 18:23:15 EST 
Makes sense but I think the task should be shown by default since the ID is less meaningful.
Comment 4 Thomas Ehrnhoefer CLA 2011-01-26 18:53:46 EST 
Created attachment 187702 [details]
patch v3

updated
Comment 5 Thomas Ehrnhoefer CLA 2011-01-26 18:54:35 EST 
(In reply to comment #3)
> Makes sense but I think the task should be shown by default since the ID is less
> meaningful.
I updated the patch, it now logs
Synchronization of taskKey [repo label] failed
Comment 6 Thomas Ehrnhoefer CLA 2011-01-26 18:55:19 EST 
Created attachment 187703 [details]
patch v3.1
Comment 7 Steffen Pingel CLA 2011-02-01 23:24:24 EST 
Thanks! I have applied the patch.
Comment 8 Steffen Pingel CLA 2011-02-01 23:24:48 EST 
Closing.