[pLog-svn] r2228 - plugins/trunk/authimage/class/security

[Mark Wu]

Mmmm 

If we put  File::delete( $tempFile ); in authimagefilter.class.php, you will
see even we type the right password, we still get the error page. 

After tracking the code, I found the auhtimage is deleted succeessful in the
first run, and in second run the pLog does not find the authimage file, so,
it just throw a error page ....

That's why I guss the action always run twice ..

Mark

> -----Original Message-----
> From: plog-svn-bounces at devel.plogworld.net 
> [mailto:plog-svn-bounces at devel.plogworld.net] On Behalf Of Jon Daley
> Sent: Wednesday, June 15, 2005 8:33 PM
> To: plog-svn at devel.plogworld.net
> Subject: Re: [pLog-svn] r2228 - plugins/trunk/authimage/class/security
> 
>  	How do you know it is executing twice?  I don't have 
> debugging turned on, so perhaps I am missing something, but 
> with Ben's and the other change you made earlier, I haven't 
> had duplicated files in my directory any more.
> 
> On Wed, 15 Jun 2005 mark at devel.plogworld.net wrote:
> > Author: mark
> > Date: 2005-06-15 05:35:05 +0000 (Wed, 15 Jun 2005) New 
> Revision: 2228
> >
> > Modified:
> >   plugins/trunk/authimage/class/security/authimagefilter.class.php
> > Log:
> > I have to delete "File::delete( $tempFile );" in 
> authimagefilter.class.php. pLog's pipeline will execute 
> "twice", if we detele the file, the pLog will show us the 
> code is incorrect even the comment is add.
> >
> > I still have no idea why our pipeline will execute twice.
> >
> > Mark
> >
> > Modified: 
> > plugins/trunk/authimage/class/security/authimagefilter.class.php
> > ===================================================================
> > --- 
> plugins/trunk/authimage/class/security/authimagefilter.cl
> ass.php	2005-06-15 05:04:01 UTC (rev 2227)
> > +++ 
> plugins/trunk/authimage/class/security/authimagefilter.cl
> ass.php	2005-06-15 05:35:05 UTC (rev 2228)
> > @@ -62,7 +62,6 @@
> >
> >             // if everything went fine, we can say so by returning
> >             // a positive PipelineResult object
> > -            File::delete( $tempFile );
> >             $result = new PipelineResult();
> >             return $result;
> >         }
> >
> > _______________________________________________
> > pLog-svn mailing list
> > pLog-svn at devel.plogworld.net
> > http://devel.plogworld.net/mailman/listinfo/plog-svn
> >
> 
> **************************************************************
> *                           *     "I find television very    *
> *     Jonathan M. Daley     *     educating.  Every time     *
> *                           *  somebody turns on the set, I  *
> *   jondaley at snurgle.org    *   go into the other room and   *
> *                           *          read a book."         *
> * www.snurgle.org/~jondaley *                -- Groucho Marx *
> **************************************************************
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.plogworld.net
> http://devel.plogworld.net/mailman/listinfo/plog-svn