PHPCAS-109 Add try&catch around all the runtime errors. If we run in cli (phunit mode) we rethrow them again and during normal webserver mode we simply exit()

git-svn-id: https://source.jasig.org/cas-clients/phpcas/trunk@24935 f5dbab47-78f9-eb45-b975-e544023573eb

Author: jfritschi

source/CAS.php

@@ -1635,6 +1635,20 @@ public static function setNoClearTicketsFromUrl() {
 		$PHPCAS_CLIENT->setNoClearTicketsFromUrl();
 		phpCAS :: traceEnd();
 	}
+	
+	/**
+	 * 
+	 * Force phpcas to thow Exceptions instead of calling exit()
+	 */
+	public static function throwExceptionsInsteadOfExiting(){
+		global $PHPCAS_CLIENT;
+		phpCAS :: traceBegin();
+		if (!is_object($PHPCAS_CLIENT)) {
+			phpCAS :: error('this method should only be called after ' . __CLASS__ . '::client() or' . __CLASS__ . '::proxy()');
+		}
+		$PHPCAS_CLIENT->throwExceptionsInsteadOfExiting();
+		phpCAS :: traceEnd();
+	}
 
 	/** @} */
 

source/CAS/AuthenticationException.php

@@ -46,7 +46,22 @@ public function __construct($client,$failure,$cas_url,$no_response,$bad_response
 			phpCAS::trace('CAS response: '.$cas_response);
 		}
 		$client->printHTMLFooter();
-		phpCAS::traceExit();
+		phpCAS::traceEnd();
+	}
+	
+	/**
+	 * Decide what to do with the exception
+	 * @param CAS_Client $client
+	 */
+	public function handleException(CAS_Client $client){
+		phpCAS::traceBegin();
+		if($client->isThrowingExceptionsEnabled() || php_sapi_name() === 'cli'){
+			phpCAS::traceEnd();
+			throw $this;
+		}else{
+			phpCAS::traceExit();
+			exit();
+		}
 	}
 
 }