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

Bug 316053

Summary: renaming remote project leads to ProjectNotConfiguredException
Product: [Tools] PTP Reporter: Vivian Kong <vivkong>
Component: RDTAssignee: Vivian Kong <vivkong>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 CC: g.watson
Version: 4.0Flags: vivkong: review+
g.watson: review+
Target Milestone: 4.0   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
proposed patch
none
updated patch none

Description Vivian Kong CLA 2010-06-07 15:19:35 EDT
Created attachment 171320 [details]
proposed patch

Project names are used as the keys for the service model manager. When the new project name is used to look up services it results in ProjectNotConfiguredExceptions being thrown.  We need a listener for project rename/delete events, so when a project is renamed/deleted the service model can be updated.

Greg, can you please review this patch?  Thanks.
Comment 1 Greg Watson CLA 2010-06-07 16:56:39 EDT
Looking at this now.
Comment 2 Greg Watson CLA 2010-06-07 17:35:12 EDT
1. The remap interface needs to be added to IServiceModelManager and documented.

2. The plugin version needs to be incremented to 2.0

3. @since 2.0 tags need to be added to remap and ProjectChangeListener

Otherwise I'm ok with the patch.
Comment 3 Vivian Kong CLA 2010-06-07 17:38:32 EDT
(In reply to comment #2)
> 1. The remap interface needs to be added to IServiceModelManager and
> documented.
> 
> 2. The plugin version needs to be incremented to 2.0
> 
> 3. @since 2.0 tags need to be added to remap and ProjectChangeListener
> 
> Otherwise I'm ok with the patch.

Thanks Greg!
Comment 4 Vivian Kong CLA 2010-06-07 18:35:36 EDT
Created attachment 171348 [details]
updated patch
Comment 5 Vivian Kong CLA 2010-06-07 18:39:57 EDT
Fixed in HEAD.