Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 335218 - Extension point for registering EObject, EPackage, model editor
Summary: Extension point for registering EObject, EPackage, model editor
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: EMF-Facet (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Nicolas Bros CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-01-24 12:08 EST by Fabien Giquel CLA
Modified: 2020-05-01 11:27 EDT (History)
3 users (show)

See Also:
gdupe: indigo+


Attachments
org.eclipse.emf.facet.emf.util.doc (25.05 KB, application/x-zip-compressed)
2011-01-28 04:59 EST, Nicolas Bros CLA
nicolas.bros: iplog+
Details
org.eclipse.emf.facet.util.emf.core.tests (11.81 KB, application/x-zip-compressed)
2011-01-28 05:33 EST, Nicolas Bros CLA
nicolas.bros: iplog+
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Fabien Giquel CLA 2011-01-24 12:08:59 EST
Here is an entry for planned Indigo feature : providing some extension point which allows the user to register some editor/browser. Such an information will be used in some specific modeling feature (such as EPackage view, or MoDisco discovery feature).
Such editor is dedicated either to EPackage, EObject, or Resource input.
Comment 1 Fabien Giquel CLA 2011-01-24 12:13:12 EST
This enhancement has been commited into SVN in org.eclipse.emf.facet.util.emf.ui (rev 252).
There are still improvments to do :
- try to move the extension point to org.eclipse.emf.facet.util.emf.core
- complete the preferences page
Comment 2 Fabien Giquel CLA 2011-01-25 04:14:22 EST
Did not success in moving the extension point from
org.eclipse.emf.facet.util.emf.ui to org.eclipse.emf.facet.util.emf.core since
the registry class uses JFace IPreferenceStore interface. To go further
registry class should also be registered through some additional extension
point.
Comment 3 Fabien Giquel CLA 2011-01-27 04:46:51 EST
Memo : On the aspect "try to move the extension point to org.eclipse.emf.facet.util.emf.core" we should try to use the API 
String text = Platform.getPreferencesService()
Comment 4 Nicolas Bros CLA 2011-01-27 11:06:41 EST
Fixed in revision 3804.

To manipulate preferences, it uses InstanceScope, DefaultScope and ScopedPreferenceStore to adapt the InstanceScope to the FieldEditorPreferencePage.
Comment 5 Gregoire Dupe CLA 2011-01-27 12:38:25 EST
Hi Nicolas,

Please, can you:
 - precise where to find the documentation,
 - precise where to find the unit tests,
 - add one line in the contributors list of each file you’ve modified,
 - add a readme file to explain why you expose an internal package (when this is the case).

Regards,
Gregoire
Comment 6 Nicolas Bros CLA 2011-01-28 04:46:22 EST
I put the documentation in:
https://dev.eclipse.org/svnroot/modeling/org.eclipse.emft.facet/trunk/doc/org.eclipse.emf.facet.emf.util.doc

Committed in revision 263.
Comment 7 Nicolas Bros CLA 2011-01-28 04:59:57 EST
Created attachment 187820 [details]
org.eclipse.emf.facet.emf.util.doc

attaching new doc plug-in (for IP log)
Comment 8 Nicolas Bros CLA 2011-01-28 05:33:00 EST
Created attachment 187824 [details]
org.eclipse.emf.facet.util.emf.core.tests

I added a new test plug-in : org.eclipse.emf.facet.util.emf.core.tests.
Committed in revision 266.
Comment 9 Nicolas Bros CLA 2011-01-31 10:04:27 EST
Marking as fixed.
Comment 10 Gregoire Dupe CLA 2011-02-01 03:44:07 EST
Hello,

The plug-in org.eclipse.emf.facet.emf.util.doc is not correctly named: it should be named org.eclipse.emf.facet.util.emf.doc.

Regards,
Gregoire Dupe
Comment 11 Nicolas Bros CLA 2011-02-01 04:06:41 EST
> The plug-in org.eclipse.emf.facet.emf.util.doc is not correctly named: it
> should be named org.eclipse.emf.facet.util.emf.doc.

Fixed in revision 272.
Comment 12 Gregoire Dupe CLA 2011-06-06 11:37:25 EDT
This bug can be closed.
Comment 13 Gregoire Dupe CLA 2011-06-06 11:37:37 EDT
This bug can be closed.