[pLog-svn] r6688 - plog/branches/lifetype-1.2/class/action

[Jon Daley]

 	Yeah, I wondered about this - I think there is a filter later, and 
so that's why I left it.  We should probably remove the filter if we 
aren't going to allow it at all.  Though when I thought about it, I 
thought the filter solution was a better one - so then the user doesn't 
get an error, but it just does the search.

On Tue, 1 Jul 2008, mark at devel.lifetype.net wrote:

> Author: mark
> Date: 2008-07-01 01:41:15 -0400 (Tue, 01 Jul 2008)
> New Revision: 6688
>
> Modified:
>   plog/branches/lifetype-1.2/class/action/defaultaction.class.php
>   plog/branches/lifetype-1.2/class/action/searchaction.class.php
> Log:
> We should not allow the html in searchTerms.
>
> Modified: plog/branches/lifetype-1.2/class/action/defaultaction.class.php
> ===================================================================
> --- plog/branches/lifetype-1.2/class/action/defaultaction.class.php	2008-07-01 05:37:16 UTC (rev 6687)
> +++ plog/branches/lifetype-1.2/class/action/defaultaction.class.php	2008-07-01 05:41:15 UTC (rev 6688)
> @@ -30,7 +30,7 @@
>         {
> 			$this->BlogAction( $actionInfo, $request );
>
> -			$this->registerFieldValidator( "searchTerms", new StringValidator( true ), true );
> +			$this->registerFieldValidator( "searchTerms", new StringValidator(), true );
> 			$this->registerFieldValidator( "postCategoryId", new IntegerValidator(), true );
> 			$this->registerFieldValidator( "postCategoryName", new StringValidator( ), true );
> 			$this->registerFieldValidator( "userId", new IntegerValidator(), true );
>
> Modified: plog/branches/lifetype-1.2/class/action/searchaction.class.php
> ===================================================================
> --- plog/branches/lifetype-1.2/class/action/searchaction.class.php	2008-07-01 05:37:16 UTC (rev 6687)
> +++ plog/branches/lifetype-1.2/class/action/searchaction.class.php	2008-07-01 05:41:15 UTC (rev 6688)
> @@ -23,7 +23,7 @@
>             $this->BlogAction( $actionInfo, $request );
>
> 			// data validation
> -			$this->registerFieldValidator( "searchTerms", new StringValidator( true ));
> +			$this->registerFieldValidator( "searchTerms", new StringValidator());
> 			$this->setValidationErrorView( new ErrorView( $this->_blogInfo, "error_incorrect_search_terms" ));
>         }
>
>
> _______________________________________________
> pLog-svn mailing list
> pLog-svn at devel.lifetype.net
> http://limedaley.com/mailman/listinfo/plog-svn
>

-- 
Jon Daley
http://jon.limedaley.com
~~
There isn't any problem in child-rearing
that cannot be solved with duct tape.
-- Alan Wagstaff