Skip to content
This repository has been archived by the owner on Mar 30, 2023. It is now read-only.

BaseValidationRule missing sanitize method #8

Open
GoogleCodeExporter opened this issue Mar 13, 2015 · 4 comments
Open

BaseValidationRule missing sanitize method #8

GoogleCodeExporter opened this issue Mar 13, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

Please add a sanitize method to the BaseValidationRule


Original issue reported on code.google.com by mike.bob...@gmail.com on 9 Oct 2009 at 7:34

@GoogleCodeExporter
Copy link
Author

Original comment by mike.bob...@gmail.com on 9 Oct 2009 at 7:41

@GoogleCodeExporter
Copy link
Author

De-allocating all defects to me. 

Original comment by vande...@gmail.com on 14 Jul 2010 at 9:27

@GoogleCodeExporter
Copy link
Author

I went ahead and took the simplest route here, but I've added that method. It 
currently throws a ValidationAvailabilityException (my best guess) unless 
overridden with a real implementation in a child class. I felt that was safer 
than trying to guess what a default "safe" value would be for all possible 
implementations.

I also commented out the global $ESAPI line from the constructor because it 
wasn't used in the constructor or the majority of BaseValidationRule's child 
classes. The 3 that did use it also called it individually. 

Original comment by Fen...@gmail.com on 22 Jul 2010 at 4:15

Attachments:

@GoogleCodeExporter
Copy link
Author

All these issues must be dealt with before 1.0. New issues will be on a case by 
case basis as to whether we hold up 1.0 or not. 

Original comment by vande...@gmail.com on 17 Jun 2011 at 3:36

  • Added labels: Milestone-Release1.0

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant