Thanks, I've committed a change to use <event> as well for the scheduler.
For backward compatibility <eventId> still works there, of course.

Florent

On Wed, Sep 23, 2009 at 11:00 AM, Sean Radford <[email protected]> wrote:
> Hi,
>
> Ok, that does work.
>
> However, thought I would point out that it is inconsistent with the
> SchedulerRegistryService, as here you specify an *eventId*
>
> e.g.
>
> <extension
>
> target="org.nuxeo.ecm.platform.scheduler.core.service.SchedulerRegistryService"
>   point="schedule">
>   <schedule id="myTestSchedule">
>     <username>system</username>
>     <password></password>
>     <eventId>acmeDemoEvent</eventId>
>
> Should one of them be changed to make things consistent? Would you like me
> to raise a JIRA?
>
> Regards,
>
> Sean
>
> Florent Guillaume wrote:
>>
>> This is wrong:
>>    <eventId>acmeDemoEvent</eventId>
>> It should be:
>>    <event>acmeDemoEvent</event>
>>
>> Did you find this in some of our documentation, or is it just a mistake?
>>
>> Florent
>>
>>
>> On Fri, Sep 18, 2009 at 6:28 PM, Sean Radford <[email protected]>
>> wrote:
>>
>>>
>>> Hi,
>>>
>>> I would have thought that the following:
>>>
>>> <?xml version="1.0"?>
>>> <component name="acme.demo.Listener">
>>>  <extension target="org.nuxeo.ecm.core.event.EventServiceComponent"
>>> point="listener">
>>>  <listener name="demoEventListener" async="true" postCommit="true"
>>>    class="acme.demo.DemoEventListener">
>>>    <eventId>acmeDemoEvent</eventId>
>>>  </listener>
>>>  </extension>
>>> </component>
>>>
>>>
>
> --
> Dr. Sean Radford, MBBS, MSc
> http://www.tacola.com/
> t: +44 (0)845 KEY HELP
> t: +44 (0)845 539 4357
> m: +44 (0)7802 24 24 86
>
>



-- 
Florent Guillaume, Head of R&D, Nuxeo
Open Source, Java EE based, Enterprise Content Management (ECM)
http://www.nuxeo.com   http://www.nuxeo.org   +33 1 40 33 79 87
_______________________________________________
ECM mailing list
[email protected]
http://lists.nuxeo.com/mailman/listinfo/ecm
To unsubscribe, go to http://lists.nuxeo.com/mailman/options/ecm

Reply via email to