Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 330163 - [Hibernate] TeneoHibernateMappingProvider#getMapping() does not remove EcorePackage/EResourcePackage
Summary: [Hibernate] TeneoHibernateMappingProvider#getMapping() does not remove EcoreP...
Status: CLOSED INVALID
Alias: None
Product: EMF
Classification: Modeling
Component: cdo.core (show other bugs)
Version: 4.0   Edit
Hardware: Macintosh Mac OS X
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Martin Taal CLA
QA Contact: Eike Stepper CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-13 01:47 EST by Eike Stepper CLA
Modified: 2019-11-08 03:13 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 Eike Stepper CLA 2010-11-13 01:47:49 EST
Cloned from: 330075: TeneoHibernateMappingProvider#getMapping() does not remove EcorePackage/EResourcePackage
https://bugs.eclipse.org/bugs/show_bug.cgi?id=330075

In method TeneoHibernateMappingProvider#getMapping() the packages EcorePackage and EResourcePackage are intended to be be removed:

----------------------------------------------------------------------------------
    // translate the list of EPackages to an array
    final List<EPackage> epacks = getHibernateStore().getPackageHandler().getEPackages();
    // remove the ecore and resource package
    epacks.remove(EcorePackage.eINSTANCE);
    epacks.remove(EresourcePackage.eINSTANCE);
----------------------------------------------------------------------------------
 
This does not work (or at least in the scenario I had) this way, since the HibernateStore's package handler retrieves another instance then the singleton instance the plugins provide.

Instead of trying to remove the packages by instance from the packs list they have to be removed by their nsURI. The instances in epacks that match EcorePackage.eNS_URI and EResourcePackage.eNS_URI must be searched and removed.
Comment 1 Martin Taal CLA 2011-05-15 07:56:18 EDT
This code does not exist anymore, closing the issue.