Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 343204 - [modeling] Update DoI model elements on Model Property Changes
Summary: [modeling] Update DoI model elements on Model Property Changes
Status: RESOLVED WONTFIX
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Mylyn (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Miles Parker CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 351889
  Show dependency tree
 
Reported: 2011-04-18 16:41 EDT by Benjamin Muskalla CLA
Modified: 2011-12-06 11:50 EST (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Benjamin Muskalla CLA 2011-04-18 16:41:54 EDT
We need to react on changes of model elements in the properties view to update our DoI graph accordingly.

Not sure yet if we want investiage support for EMF Refactorings ( http://www.eclipse.org/modeling/emft/refactor/index.php )
Comment 1 Miles Parker CLA 2011-07-11 20:47:42 EDT
re: Refactorings, My thinking is not. :) EMF Refactoring support is still early incubation, and refactoring plays much less of a part in model development. In a sense with a model much of what we think of as refactoring (entity renaming, moving, etc..) is "built-in" in that there is always one and only one place to change it. One thought on all of this is that we might want to actually hook into the EMF change notification mechanism itself. That would give us much finer grained access than would just listenting to selection events. That will be a lot trickier to implement, but on the other hand it means that we might end up with a generic way of dealing with not just EMF (meta-model level) model editors but EMF generated editors as well. *That* would be cool.
Comment 2 Benjamin Muskalla CLA 2011-07-12 06:30:35 EDT
We should discuss the availability for EMF generated editors anyways.

Otherwise, is EMF Refactoring really to right approach to rely on? My thinking is that they provide the refactoring while we're essentially interested in change deltas of the artifacts (if due to a merge, refactoring or simple attribute change).
Comment 3 Miles Parker CLA 2011-07-12 13:32:39 EDT
(In reply to comment #2)
> We should discuss the availability for EMF generated editors anyways.
> 
> Otherwise, is EMF Refactoring really to right approach to rely on? My thinking
> is that they provide the refactoring while we're essentially interested in
> change deltas of the artifacts (if due to a merge, refactoring or simple
> attribute change).

Exactly. Though I wasn't thinking of external change delta, I was only thinking of internal editor changes. I'm not sure we have to worry about these issues at all. In Mylyn do you keep degree of interest scores across refactorings, even when those refactorings occur as a result of external changes. (Actually I'm not sure how you would be able to do this.) For example, if you've been working on a method, and then you do a merge against master where someone else has renamed that method, would you retain the DOI for that?

Otherwise, if we simply have auniqu path to stuff people are interested, isn't that enough? It's either:

1. Added in which case we couldn't have been interested in it.
2. Modified in which case the same element mapping would work.
3. Deleted in which case we shouldn't find it. :)
4. Had its name changed in which case wouldn't it be ok to ignore it?
Comment 4 Benjamin Muskalla CLA 2011-07-13 12:43:37 EDT
Yes, for the interaction events, we only want to see the selections and changes in the editor.

> 4. Had its name changed in which case wouldn't it be ok to ignore it?
This is how the current JDT bridge is implemented. The idea here would be to improve this situation (maybe this even needs changes to the context framework). The tracking of changes from other sources (eg. EMF refactoring) would be interesting for to "transform" the context the next time you activate the context (eg. based on the refactoring history).
Comment 5 Miles Parker CLA 2011-07-13 13:00:05 EDT
OK, I'm going to change the summary of this one on the basis that refactoring support will be out of scope for this effort. Benny, perhaps open another more general bug on Context for that idea?

We'll focus on selection events here. I think the change notification stuff might be interesting to look at later, especially if we do DSL / text-absed integration, but it would be challenging to impossible to get selection events form that anyway.

So this task now just relates to property editor interactions.
Comment 6 Benjamin Muskalla CLA 2011-07-19 11:00:00 EDT
See related refactoring tasks:

task 344166: Rename package refactoring doesn't update Mylyn task contexts
task 346352: Extract method refactoring should add the newly created method to the context
task 344166: Rename package refactoring doesn't update Mylyn task contexts
Comment 7 Miles Parker CLA 2011-07-21 17:57:18 EDT
(This one is also essentially complete.)
Comment 8 Miles Parker CLA 2011-07-21 18:23:49 EDT
Correction. We will not be doing support for properties. See bug 343201. So I'm marking this one "wont' fix". If anyone has a good reason they want this, please speak up!
Comment 9 Miles Parker CLA 2011-08-17 21:07:58 EDT
Reorganizing.