Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 360300 - decouple monitor.ui from context.core
Summary: decouple monitor.ui from context.core
Status: RESOLVED WONTFIX
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: PC Linux
: P1 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 378343 (view as bug list)
Depends on:
Blocks: 360299
  Show dependency tree
 
Reported: 2011-10-08 01:22 EDT by Steffen Pingel CLA
Modified: 2016-04-05 17:34 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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.