Bug 276095 - [fiximprove] : MTJ extension points should handle dynamic add/remove of plugins
Summary: [fiximprove] : MTJ extension points should handle dynamic add/remove of plugins
Status: CLOSED FIXED
Alias: None
Product: MTJ (Archived)
Classification: Tools
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 1.0.1   Edit
Assignee: David Marques CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks:
 
Reported: 2009-05-13 10:18 EDT by Gorkem Ercan CLA
Modified: 2009-10-28 08:45 EDT (History)
4 users (show)

See Also:


Attachments
Patch with partial solution to deviceimporter extension point (3.92 KB, patch)
2009-08-14 14:20 EDT, Rafael Amaral CLA
no flags Details | Diff
Patch with bug fix (35.96 KB, patch)
2009-08-27 15:12 EDT, Rafael Amaral CLA
wtv368: iplog+
wtv368: review+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gorkem Ercan CLA 2009-05-13 10:18:37 EDT
Most (if not all ) extension points of the MTJ do not support plugins that are added/removed during runtime. I think this is a required feature on a P2 & Pulsar world. Perhaps separate  reports should be created for all the extension points that does not support dynamic add/remove and should. I am aware of at least jadextensions and securitypermission extension points that needs to support this.
Comment 1 Rafael Amaral CLA 2009-08-12 13:02:26 EDT
Hello All,

I am planning to supply a fix for this bug.

Regards,

Rafael Amaral
Comment 2 Rafael Amaral CLA 2009-08-14 14:20:30 EDT
Created attachment 144565 [details]
Patch with partial solution to deviceimporter extension point

Hello all,

This patch has a partial solution to deviceimporter extension point, 
but with this solution if you have already imported some devices that need
an specific extension to be imported, and in runtime you remove that extension, 
you get an ClassNotFoundException because the classes from that extension
are not there anymore.

I would like to know your opinion about how should I deal with this situation?

Regards,

Rafael Amaral
Comment 3 Rafael Amaral CLA 2009-08-27 15:12:45 EDT
Created attachment 145843 [details]
Patch with bug fix

Hello All,

Follows attached the patch with the bug resolution. Can anyone please revise it?

Regards,

Rafael Amaral
Comment 4 Rafael Amaral CLA 2009-08-27 16:01:45 EDT
I, Rafael Amaral have created the patch 145843 solely based on existing (EPL licensed) code in the MTJ project, and wrote all new code 100% myself, without using any other open source or proprietary source code as a basis for my work.

   I am making my contribution available under the terms of the Eclipse Public License (EPL) to be included in the codebase of the MTJ project.
Comment 5 David Marques CLA 2009-08-27 16:56:16 EDT
Patch reviewed and code committed.
Comment 6 Diego Madruga Sandin CLA 2009-10-28 08:45:46 EDT
Released