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

[Paul Westbrook]

Hello,
    Thank you.  I will make the change to the plugins.  The reason  
that I put the code to make sure the variable was defined was to  
prevent php warnings.

--Paul


On Mar 20, 2007, at 9:25 AM, Oscar Renalias wrote:

> 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.
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn

--
Paul Westbrook
paul at westbrooks.org
<http://www.westbrooks.org>