[pLog-svn] r5135 - in plugins/branches/lifetype-1.2: blogtimes rsd sitemap tagcloud

[Oscar Renalias]

On 3/20/07, Paul Westbrook <paul at westbrooks.org> wrote:
> Hello,
>
>
> On Mar 20, 2007, at 4:22 AM, Oscar Renalias wrote:
>
> > I think we shouldn't make an assumption regarding the event number,
> > which by the way, you should add to 1.2.1 as well as the code needed
> > to generate it.
> >
>
> The code to generate the event was checked in in revision 5134.

Sorry, I'm blind (again)

> > I would modify the plugin so that it first check if the event is
> > available and if it is, then attach itself to this even. If it isn't
> > then nothing should happen. Or what does everybody else think?
> >
>
> How do I check to see if the event is available?  Is checking to see
> if the variable is defined enough?

Yes, I think it should be enough... In fact you already did something like that:

if( defined( "NAME_OF_THE_EVENT" ))
  $this->registerEvent( "NAME_OF_THE_EVENT" );

If we want to get minimalistic, there is even no need to check if the
event exists, as listening to an event that is never thrown will do
exactly nothing.