Jasig/phpCAS — Issue #7: Move all parameter validation out of the phpCAS class or make the phpCAS testable
Author: adamfranco
Created: 2011-10-26 20:23:07+00:00
State: open

*[Migrated from PHPCAS-127](https://issues.jasig.org/browse/PHPCAS-127)*

Description
=========

The phpCAS class should only validate that the client exists all other input validation and sequence validation should be done in the CAS_Client where it is testable.

Alternatively, the phpCAS class could be made testable by allowing resetting of its variables.

Joachim Fritschi added a comment - 06/Sep/11 1:52 PM - edited
--------------------------------------------------------------------------------------

Ok sounds like a plan. But if it's possible don't move the validation code directly into the CAS_Cclient class. I would really like to start stripping this class down. 3000 lines is just a bit to much for my taste.
Maybe we can put this in some external Validator class?