| Summary: | [History View] History view does not link to selection in Synchronize view | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Markus Keller <markus.kell.r> | ||||
| Component: | Team | Assignee: | Michael Valenta <Michael.Valenta> | ||||
| Status: | VERIFIED FIXED | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | P3 | CC: | daniel_megert, ekuleshov | ||||
| Version: | 3.2 | Keywords: | helpwanted | ||||
| Target Milestone: | 3.3 M4 | ||||||
| Hardware: | PC | ||||||
| OS: | Windows XP | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Markus Keller
I20060725-0800 This seems to work for me: i) I have a bunch of team related projects that have incoming changes. I made a bunch of changes to a different project in my workspace (to ensure I have both incoming and outgoing changes). ii) I switched over to the Team Sync perspective, which has the History View visible as a default. iii) Performed a sync and got some incoming and outgoing changes. Turned on linking in the History View, clicked on both incoming and outgoing changes => watched History View update as expected. I also tried cycling through all of the models modes (All Models, Java and Workspace) and through the various filtering modes (incoming, outgoing, conflicts). iv) Finally I tried opening the Sync View and History View while in the Java perspective. Followed step iii again and watched the History View update. Am I missing anything here that you did? Did you click on a new file? Did you have multiple instances of the History View open? Was the view pinned? I tried to reproduce in a fresh workspace, and I think I found the deciding factor: I'm still using a pinned old (not model-based) synchronization. To reproduce, you have to switch off 'Preferences > Team > CVS > Synchronize/Compare > Allow models (e.g. Java) to participate in synchronizations' and then create a new synchronization. That did it! I'll take a look... *** Bug 154024 has been marked as a duplicate of this bug. *** This applies only to the non-model based Synchronize pages. As such, we do not plan on addressing it. Patches will be accepted. Created attachment 55299 [details]
Patch
This is a minimal patch. If you think SyncInfoModelElement should adapt to IResource in general, please move it to SyncInfoModelElement#getAdapter(..).
We don't want SyncInfoModelElement to adapt to IResource so the patch you have provided is the way to go. I will put it in today. Fix released to HEAD. Thanks. Verified |