Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 338496 - Minor tweaks to update configuration marker
Summary: Minor tweaks to update configuration marker
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: m2e (show other bugs)
Version: unspecified   Edit
Hardware: PC Mac OS X - Carbon (unsup.)
: P3 minor (vote)
Target Milestone: ---   Edit
Assignee: Milos Kleint CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 340095 (view as bug list)
Depends on:
Blocks:
 
Reported: 2011-02-28 21:18 EST by Pascal Rapicault CLA
Modified: 2021-04-19 13:26 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 Pascal Rapicault CLA 2011-02-28 21:18:40 EST
When a project configuration needs to be updated, an error marker is being created on the project.
Several comments:
- Should the error be located on the pom instead of the project?
- Should there be a quick fix to allow the user to resolve the problem?
Comment 1 Pascal Rapicault CLA 2011-02-28 21:19:45 EST
Note that we also talked about having a preference that would automatically trigger the project configuration upon changes, so it may be an argument to not do much on this and rather solve the other issue.
Comment 2 Igor Fedorenko CLA 2011-02-28 22:01:31 EST
(In reply to comment #0)
> When a project configuration needs to be updated, an error marker is being
> created on the project.
> Several comments:
> - Should the error be located on the pom instead of the project?

We discussed this with Milos. This is not a problem with pom.xml, so we decided project was more appropriate location for this marker.


> - Should there be a quick fix to allow the user to resolve the problem?

Yes, quick-fix is a good idea.


(In reply to comment #1)
> Note that we also talked about having a preference that would automatically
> trigger the project configuration upon changes, so it may be an argument to not
> do much on this and rather solve the other issue.

Automatic project configuration update is a hard problem to solve. We need to account for any possible manual changes to project configuration. Dealing with major project configuration changes (eg, packaging war->eclipse-plugin) requires working implementation of #unconfigure, which is not there. 


Also, changes that require configuration update are relatively infrequent, so I think it may be good enough just to show the update is required.
Comment 3 Milos Kleint CLA 2011-03-07 03:49:21 EST
In the netbeans codebase, I've done automatic project configuration updates. Not entirely transferable here but here's how it worked for reference:

1. on IDE start all opened projects loaded (a necessary step as there is no other model than the POM)
2. on settings.xml file changes, all opened projects reload.
3. on pom.xml file change, the respective project reloads. AFAIK I never gone as far as updating all child projects when a parent changes.. not sure though..
4. There is still a "Refresh" action on the project's popup menu, so if users detect a problem, they can fix it.

Didn't get any complaints for this setup.  Maybe I would if I removed the Refresh button :)
Comment 4 Igor Fedorenko CLA 2011-03-15 19:44:24 EDT
*** Bug 340095 has been marked as a duplicate of this bug. ***
Comment 5 Milos Kleint CLA 2011-03-16 10:09:34 EDT
(In reply to comment #4)
> *** Bug 340095 has been marked as a duplicate of this bug. ***

since the 340095 is marked as duplicate, are we in agreement that the fix to this issue is the addition of quick fix ?
Comment 6 Igor Fedorenko CLA 2011-03-16 10:21:11 EDT
Yes. Lets do quick-fix now and re-evaluate if we need anything more elaborate based on user feedback later.
Comment 7 Milos Kleint CLA 2011-03-17 06:21:22 EDT
http://git.eclipse.org/c/m2e/m2e-core.git/commit/?h=milos_m7_buffer&id=468d6ebccf45cf0fc61d5c0597c7ca475c17dd58

implements the quickfix, not in master yet.
Comment 8 Milos Kleint CLA 2011-03-18 03:46:24 EDT
in master now
Comment 9 Denis Roy CLA 2021-04-19 13:26:52 EDT
Moved to https://github.com/eclipse-m2e/m2e-core/issues/