Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 360300

Summary: decouple monitor.ui from context.core
Product: z_Archived Reporter: Steffen Pingel <steffen.pingel>
Component: MylynAssignee: Project Inbox <mylyn-triaged>
Status: RESOLVED WONTFIX QA Contact:
Severity: enhancement    
Priority: P1 CC: krzysztof.daniel, shawn.minto, zephyrin.soh
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Linux   
Whiteboard:
Bug Depends on:    
Bug Blocks: 360299    

Description Steffen Pingel CLA 2011-10-08 01:22:32 EDT
The monitor.ui bundle should not have a direct dependency on context.core to avoid a cycle in the build dependency graph between Mylyn Context and Mylyn Commons and to improve reusability of the monitor components.

To maintain backwards compatibility a new extension point in monitor.ui should be created that is invoked by AbstractUserInteractionMonitor:

IInteractionEventProcessor {

  /** Sets properties of builder. */
  void processNavigation(InteractionEventBuilder builder, String partId, Object targetElement, String kind, boolean contributeToContext)
 
}

InteractionEventBuilder {

  Kind kind; String structureKind; String handle; String originId; String navigatedRelation;

  IInteractionEvent createEvent();

}
Comment 1 Steffen Pingel CLA 2011-10-08 15:26:39 EDT
See http://review.mylyn.org/#change,79.
Comment 2 Steffen Pingel CLA 2012-05-03 08:47:14 EDT
*** Bug 378343 has been marked as a duplicate of this bug. ***
Comment 3 Steffen Pingel CLA 2015-01-07 17:11:00 EST
Migrated review: https://git.eclipse.org/r/#/c/5638/
Comment 4 Sam Davis CLA 2016-04-05 17:34:55 EDT
Closing as part of backlog cleanup. Please reopen if you are working on this.