[pLog-svn] r5923 -plugins/branches/lifetype-1.2/authimage/class/security

[Mark Wu]

Hi Jon:

I think It already take care by the File class. So, we won't get the error
message if the pipeline run twice.

Mark 

> -----Original Message-----
> From: plog-svn-bounces at devel.lifetype.net 
> [mailto:plog-svn-bounces at devel.lifetype.net] On Behalf Of Jon Daley
> Sent: Monday, September 10, 2007 9:01 PM
> To: LifeType Developer List
> Subject: Re: [pLog-svn] r5923 
> -plugins/branches/lifetype-1.2/authimage/class/security
> 
>  	I think you had trouble with this before - that because 
> the pipeline runs twice, you get an error the second time 
> through.  You might need an @ or something to avoid the error?
> 
> On Mon, 10 Sep 2007, mark at devel.lifetype.net wrote:
> 
> > Author: mark
> > Date: 2007-09-10 09:00:10 -0400 (Mon, 10 Sep 2007) New 
> Revision: 5923
> >
> > Modified:
> >   
> > 
> plugins/branches/lifetype-1.2/authimage/class/security/authimagefilter
> > .class.php
> > Log:
> > Delete the image file to prevent user use the same 
> auhtimage to post the comment again and again.
> >
> > Modified: 
> > 
> plugins/branches/lifetype-1.2/authimage/class/security/authimagefilter
> > .class.php 
> > ===================================================================
> > --- 
> plugins/branches/lifetype-1.2/authimage/class/security/authima
> gefilter.class.php	2007-09-08 08:02:20 UTC (rev 5922)
> > +++ 
> plugins/branches/lifetype-1.2/authimage/class/security/authima
> gefilter.class.php	2007-09-10 13:00:10 UTC (rev 5923)
> > @@ -63,7 +63,10 @@
> >                 $locale = $blogInfo->getLocale();
> >                 $result = new PipelineResult( false, 
> AUTHIMAGE_FILTER_MATCH_FOUND, $locale->tr("error_authimage_code") );
> >                 return $result;
> > -            }
> > +            } else {
> > +            	// delete the tempFile to prevent the 
> user use the same authimage code to post comment again
> > +            	File::delete( $tempFile );
> > +        	}
> >
> >             // if everything went fine, we can say so by returning
> >             // a positive PipelineResult object
> >
> > _______________________________________________
> > pLog-svn mailing list
> > pLog-svn at devel.lifetype.net
> > http://limedaley.com/mailman/listinfo/plog-svn
> >
> 
> --
> Jon Daley
> http://jon.limedaley.com/
> 
> No matter where you go,
> there you are.
> -- Buckaroo Bonzai
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn
>