Bugzilla – Full Text Bug Listing
| Summary: | [CDO] Allow Adapters to be notified during loading | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Modeling] EMF | Reporter: | Alexandre Borgoltz <alexandre.borgoltz> | ||||
| Component: | cdo.core | Assignee: | Eike Stepper <stepper> | ||||
| Status: | CLOSED FIXED | QA Contact: | Eike Stepper <stepper> | ||||
| Severity: | enhancement | ||||||
| Priority: | P3 | CC: | alexandre.borgoltz | ||||
| Version: | 4.1 | ||||||
| Target Milestone: | --- | ||||||
| Hardware: | PC | ||||||
| OS: | Windows Vista | ||||||
| Whiteboard: | |||||||
| Attachments: |
|
||||||
|
Description
Alexandre Borgoltz
I'm adding the following new API: - CDOView.Options.isLoadNotificationEnabled() - CDOView.Options.setLoadNotificationEnabled(boolean) The default value is false (i.e. the former behaviour) commit 7071cb1cd5f9a14529f3be4c8ad319848aa94715 Please test carefully because we don't have tests for this in our suites (other than regression tests)! Created attachment 210149 [details]
Patch - Failing Test case for Load notification API
Here's the requested patch.
It adds org.eclipse.emf.cdo.tests.LoadNotificationTest, that is failing, because constuctor-time-registered adapters are not notified.
Alexandre, I can't seem to apply your patch: C:/Users/Stepper/AppData/Local/Temp/~attachment-5480502489365861950.patch:841: trailing whitespace. * C:/Users/Stepper/AppData/Local/Temp/~attachment-5480502489365861950.patch:872: trailing whitespace. * C:/Users/Stepper/AppData/Local/Temp/~attachment-5480502489365861950.patch:879: trailing whitespace. * C:/Users/Stepper/AppData/Local/Temp/~attachment-5480502489365861950.patch:887: trailing whitespace. * C:/Users/Stepper/AppData/Local/Temp/~attachment-5480502489365861950.patch:897: trailing whitespace. * error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/Model6Factory.java:18 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/Model6Factory.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/Model6Package.java:23 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/Model6Package.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/impl/Model6FactoryImpl.java:20 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/impl/Model6FactoryImpl.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/impl/Model6PackageImpl.java:20 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/legacy/model6/impl/Model6PackageImpl.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/Model6Factory.java:15 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/Model6Factory.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/Model6Package.java:24 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/Model6Package.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/impl/Model6FactoryImpl.java:18 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/impl/Model6FactoryImpl.java: patch does not apply error: patch failed: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/impl/Model6PackageImpl.java:25 error: plugins/org.eclipse.emf.cdo.tests.model6/src/org/eclipse/emf/cdo/tests/model6/impl/Model6PackageImpl.java: patch does not apply Nevertheless I've managed to modify model6.ecore and the generated GImpl classes: commit 7005d35ba2bb85c295b2e25d1126e4a539f24036 Can you please attach your test case in plain Java format? Closing. |