[pLog-svn] r6088 - plog/branches/lifetype-1.2/class/security
plogworld at jon.limedaley.com
Thu Nov 29 09:17:36 EST 2007
On Thu, 29 Nov 2007, Mark Wu wrote:
>> Well, hopefully Paul will chime in here. I am pretty
>> sure I understand what you did. Did you read Paul's (I
>> assume it was Paul) comment about why he added the second
>> loop? I think we need to run the entire pipeline twice,
>> particularly those filters that ran the first time.
> Yep, I read his comments about add second run , that why I think the change
> is okay for us. Becasue I can make sure all pipeline filters run once. No
> more no less :D
I don't understand, if plugin A runs and trains the comment as
non-spam, and then plugin B runs and trains as spam, in the two-run
method, plugin A would get run again and be able to clean up his bad train
of the comment.
Your method doesn't allow for that, correct?
>> The thing that we need to have is the "executed" flag
>> passed into each filter, and then each filter can decide
>> whether it wants to run twice or not.
> Yes, this is what I did in rev. 6088.
Hrm, I'll guess I'll have to look more into the code. Plugins
never run twice in your new method, so what you just said is incorrect?
> BTW, did you test it yet? Does it work for you?
I don't use any plugins that would require the functionality that
Paul added. I don't expect it broke anything that I do, since I only use
hidden input. Only running once restricts what plugins can do, and
depending on what letter they start with, depends on if they run in the
correct order or not.
As computer scientists, we don't like to think that we're fools.
We like to think we are better than that.
-- Professor Tygar
More information about the pLog-svn