[MGNLOBS-9] Remove deprecated ObservationUtil class Created: 06/Apr/10 Updated: 04/Nov/15 Resolved: 04/Nov/15 |
|
| Status: | Closed |
| Project: | Observation |
| Component/s: | None |
| Affects Version/s: | 1.1, 2.0.1 |
| Fix Version/s: | 1.3.x, 2.0.x |
| Type: | Task | Priority: | Major |
| Reporter: | Jan Haderka | Assignee: | Unassigned |
| Resolution: | Won't Do | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Template: |
|
| Acceptance criteria: |
Empty
|
| Task DoR: |
Empty
|
| Date of First Response: |
| Comments |
| Comment by Magnolia International [ 14/Dec/10 ] |
|
This is still open, but as far as I can tell, changes have been merged back to core, except for the dubious changes introduced with rev 16566 for "supporting synchronous event listeners". (SynchEventListener inner class, instanciateSynchEventListener method). This is used in info.magnolia.module.observation.ObservationModule#addListenerConfiguration, which itself seems a little convoluted, and introduces a dependency to Jackrabbit. org.apache.jackrabbit.core.observation.SynchronousEventListener is a markup interface used internally by Jackrabbit (this should be documented), but our SynchEventListener does strictly nothing else than delegating to the given listener. Since we're targeting a new major version of the observation module, how about finally removing this class altogether ? (and only keep the extra synchronous-stuff, or move that to core as well) |
| Comment by Michael Mühlebach [ 04/Nov/15 ] |
|
Given the thousands of other issues we have open that are more highly requested, we won't be able to address this issue in the foreseeable future. Instead we will focus on issues with a higher impact, and more votes. |