chamilo / chamilo-lms

main/auth/cas/logout.php

Spacing +1 added lines, -1 removed lines

@@ -9,5 +9,5 @@
 require_once('authcas.php');
 global $cas_auth_ver, $cas_auth_server, $cas_auth_port, $cas_auth_uri;
 
-phpCAS::client($cas_auth_ver,$cas_auth_server,$cas_auth_port,$cas_auth_uri);
+phpCAS::client($cas_auth_ver, $cas_auth_server, $cas_auth_port, $cas_auth_uri);
 phpCAS::logout();

main/auth/cas/authcas.php

Spacing +6 added lines, -6 removed lines

@@ -41,13 +41,13 @@ 
     }
 
 	if (!is_object($PHPCAS_CLIENT)) {
-		phpCAS::client($cas_auth_ver,$cas_auth_server,$cas_auth_port,$cas_auth_uri);
+		phpCAS::client($cas_auth_ver, $cas_auth_server, $cas_auth_port, $cas_auth_uri);
 		phpCAS::setNoCasServerValidation();
 	}
 	$auth = phpCAS::checkAuthentication();
 
 	if ($auth) {
-		$login= trim(phpCAS::getUser());
+		$login = trim(phpCAS::getUser());
 		/*
 		   Get user  attributes. Here are the attributes for crdp platform
 		   sn => name
@@ -79,7 +79,7 @@ 
 			default:
 				$status=5; // Student
 		}*/
-		if (!$logout){
+		if (!$logout) {
 		    // get user info from username
 		    $tab_user_info = api_get_user_info($login);
 
@@ -92,7 +92,7 @@ 
                         $chamilo_user = extldap_get_chamilo_user($ldapuser);
                         $chamilo_user['user_id'] = $tab_user_info['user_id'];
                         $chamilo_user['status'] = $tab_user_info['status'];
-        				UserManager::update_user ($chamilo_user["user_id"], $chamilo_user["firstname"], $chamilo_user["lastname"], $login, null, null, $chamilo_user["email"], $chamilo_user["status"], '', '', '', '', 1, null, 0, null,'') ;
+        				UserManager::update_user($chamilo_user["user_id"], $chamilo_user["firstname"], $chamilo_user["lastname"], $login, null, null, $chamilo_user["email"], $chamilo_user["status"], '', '', '', '', 1, null, 0, null, '');
 		            }
 		        }
 		        return $login;
@@ -105,7 +105,7 @@ 
 		            case PLATFORM_AUTH_SOURCE :
 		                // user will have to modify firstname, lastname, email in chamilo profil edit
 		                $userdata = get_lang("EditInProfil");
-          				UserManager::create_user($userdata, $userdata, '5', $userdata, $login, 'casplaceholder', '','','','',CAS_AUTH_SOURCE);
+          				UserManager::create_user($userdata, $userdata, '5', $userdata, $login, 'casplaceholder', '', '', '', '', CAS_AUTH_SOURCE);
           				$user_added = $login;
 		                break;
 		            case LDAP_AUTH_SOURCE :
@@ -163,7 +163,7 @@ 
  *
  * @see online_logout()
  */
-function cas_logout($uinfo=null, $location=null)
+function cas_logout($uinfo = null, $location = null)
 {
     global $cas_auth_ver, $cas_auth_server, $cas_auth_port, $cas_auth_uri;
     global $PHPCAS_CLIENT;

main/auth/cas/logincas.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         $firstpage = $_GET['firstpage'];
         setcookie("GotoCourse", $firstpage);
     }
-    if (!is_object($PHPCAS_CLIENT) ) {
+    if (!is_object($PHPCAS_CLIENT)) {
         phpCAS::client(
             $cas_auth_ver,
             $cas_auth_server,

main/auth/cas/lib/CAS/PGTStorage/pgt-file.php

Spacing +13 added lines, -13 removed lines

@@ -131,17 +131,17 @@ 
    *

    * @public

    */

-  function PGTStorageFile($cas_parent,$format,$path)

+  function PGTStorageFile($cas_parent, $format, $path)

     {

       phpCAS::traceBegin();

       // call the ancestor's constructor

       $this->PGTStorage($cas_parent);

 

-      if (empty($format) ) $format = CAS_PGT_STORAGE_FILE_DEFAULT_FORMAT;

-      if (empty($path) ) $path = CAS_PGT_STORAGE_FILE_DEFAULT_PATH;

+      if (empty($format)) $format = CAS_PGT_STORAGE_FILE_DEFAULT_FORMAT;

+      if (empty($path)) $path = CAS_PGT_STORAGE_FILE_DEFAULT_PATH;

 

       // check that the path is an absolute path

-      if (getenv("OS")=="Windows_NT"){

+      if (getenv("OS") == "Windows_NT") {

       	

       	 if (!preg_match('`^[a-zA-Z]:`', $path)) {

 	     	phpCAS::error('an absolute path is needed for PGT storage to file');

@@ -151,13 +151,13 @@ 
       else

       {

       

-      	if ( $path[0] != '/' ) {

+      	if ($path[0] != '/') {

 			phpCAS::error('an absolute path is needed for PGT storage to file');

       	}

 

       	// store the path (with a leading and trailing '/')      

-      	$path = preg_replace('|[/]*$|','/',$path);

-      	$path = preg_replace('|^[/]*|','/',$path);

+      	$path = preg_replace('|[/]*$|', '/', $path);

+      	$path = preg_replace('|^[/]*|', '/', $path);

       }

       

       $this->_path = $path;

@@ -186,7 +186,7 @@ 
     {

       phpCAS::traceBegin();

       // if the storage has already been initialized, return immediatly

-      if ( $this->isInitialized() )

+      if ($this->isInitialized())

 	return;

       // call the ancestor's method (mark as initialized)

       parent::init();

@@ -222,12 +222,12 @@ 
    *

    * @public

    */

-  function write($pgt,$pgt_iou)

+  function write($pgt, $pgt_iou)

     {

       phpCAS::traceBegin();

       $fname = $this->getPGTIouFilename($pgt_iou);

-      if ( $f=fopen($fname,"w") ) {

-	if ( fputs($f,$pgt) === FALSE ) {

+      if ($f = fopen($fname, "w")) {

+	if (fputs($f, $pgt) === FALSE) {

 	  phpCAS::error('could not write PGT to `'.$fname.'\'');

 	}

 	fclose($f);

@@ -252,10 +252,10 @@ 
       phpCAS::traceBegin();

       $pgt = FALSE;

       $fname = $this->getPGTIouFilename($pgt_iou);

-      if ( !($f=fopen($fname,"r")) ) {

+      if (!($f = fopen($fname, "r"))) {

 	phpCAS::trace('could not open `'.$fname.'\'');

       } else {

-	if ( ($pgt=fgets($f)) === FALSE ) {

+	if (($pgt = fgets($f)) === FALSE) {

 	  phpCAS::trace('could not read PGT from `'.$fname.'\'');

 	} 

 	fclose($f);

main/auth/cas/lib/CAS/PGTStorage/pgt-db.php

Spacing +9 added lines, -9 removed lines

@@ -57,7 +57,7 @@ 
    * @hideinitializer

    * @private

    */

-  var $_url='';

+  var $_url = '';

 

   /**

    * This method returns the PEAR DB URL to use to connect to the database.

@@ -159,18 +159,18 @@ 
    *

    * @public

    */

-  function PGTStorageDB($cas_parent,$user,$password,$database_type,$hostname,$port,$database,$table)

+  function PGTStorageDB($cas_parent, $user, $password, $database_type, $hostname, $port, $database, $table)

     {

       phpCAS::traceBegin();

 

       // call the ancestor's constructor

       $this->PGTStorage($cas_parent);

 

-      if ( empty($database_type) ) $database_type = CAS_PGT_STORAGE_DB_DEFAULT_DATABASE_TYPE;

-      if ( empty($hostname) ) $hostname = CAS_PGT_STORAGE_DB_DEFAULT_HOSTNAME;

-      if ( $port==0 ) $port = CAS_PGT_STORAGE_DB_DEFAULT_PORT;

-      if ( empty($database) ) $database = CAS_PGT_STORAGE_DB_DEFAULT_DATABASE;

-      if ( empty($table) ) $table = CAS_PGT_STORAGE_DB_DEFAULT_TABLE;

+      if (empty($database_type)) $database_type = CAS_PGT_STORAGE_DB_DEFAULT_DATABASE_TYPE;

+      if (empty($hostname)) $hostname = CAS_PGT_STORAGE_DB_DEFAULT_HOSTNAME;

+      if ($port == 0) $port = CAS_PGT_STORAGE_DB_DEFAULT_PORT;

+      if (empty($database)) $database = CAS_PGT_STORAGE_DB_DEFAULT_DATABASE;

+      if (empty($table)) $table = CAS_PGT_STORAGE_DB_DEFAULT_TABLE;

 

       // build and store the PEAR DB URL

       $this->_url = $database_type.':'.'//'.$user.':'.$password.'@'.$hostname.':'.$port.'/'.$database;

@@ -192,7 +192,7 @@ 
     {

       phpCAS::traceBegin();

       // if the storage has already been initialized, return immediatly

-      if ( $this->isInitialized() )

+      if ($this->isInitialized())

 		return;

       // call the ancestor's method (mark as initialized)

       parent::init();

@@ -205,7 +205,7 @@ 
 

       // try to connect to the database

       $this->_link = DB::connect($this->getURL());

-      if ( DB::isError($this->_link) ) {

+      if (DB::isError($this->_link)) {

 	phpCAS::error('could not connect to database ('.DB::errorMessage($this->_link).')');

       }

       var_dump($this->_link);

main/auth/cas/lib/CAS/PGTStorage/pgt-main.php

Spacing +3 added lines, -3 removed lines

@@ -63,7 +63,7 @@ 
   function PGTStorage($cas_parent)

     {

       phpCAS::traceBegin();

-      if ( !$cas_parent->isProxy() ) {

+      if (!$cas_parent->isProxy()) {

 	phpCAS::error('defining PGT storage makes no sense when not using a CAS proxy'); 

       }

       phpCAS::traceEnd();

@@ -107,7 +107,7 @@ 
    * @private

    * @deprecated not used.

    */

-  var $_error_message=FALSE;

+  var $_error_message = FALSE;

 

   /**

    * This method sets en error message, which can be read later by 

@@ -185,7 +185,7 @@ 
    *

    * @protected

    */

-  function write($pgt,$pgt_iou)

+  function write($pgt, $pgt_iou)

     {

       phpCAS::error(__CLASS__.'::'.__FUNCTION__.'() should never be called'); 

     }

main/auth/cas/lib/CAS/languages/languages.php

Spacing +6 added lines, -6 removed lines

@@ -12,13 +12,13 @@
 /**

  * a phpCAS string index

  */

-define("CAS_STR_USING_SERVER",                1);

-define("CAS_STR_AUTHENTICATION_WANTED",       2);

-define("CAS_STR_LOGOUT",                      3);

+define("CAS_STR_USING_SERVER", 1);

+define("CAS_STR_AUTHENTICATION_WANTED", 2);

+define("CAS_STR_LOGOUT", 3);

 define("CAS_STR_SHOULD_HAVE_BEEN_REDIRECTED", 4);

-define("CAS_STR_AUTHENTICATION_FAILED",       5);

-define("CAS_STR_YOU_WERE_NOT_AUTHENTICATED",  6);

-define("CAS_STR_SERVICE_UNAVAILABLE",         7);

+define("CAS_STR_AUTHENTICATION_FAILED", 5);

+define("CAS_STR_YOU_WERE_NOT_AUTHENTICATED", 6);

+define("CAS_STR_SERVICE_UNAVAILABLE", 7);

 //@}

 

 ?>
\ No newline at end of file

main/auth/cas/lib/CAS/domxml-php4-to-php5.php

Spacing +229 added lines, -229 removed lines

@@ -42,178 +42,178 @@ 
 	http://alexandre.alapetite.net/doc-alex/domxml-php4-php5/
 */
 
-define('DOMXML_LOAD_PARSING',0);
-define('DOMXML_LOAD_VALIDATING',1);
-define('DOMXML_LOAD_RECOVERING',2);
-define('DOMXML_LOAD_SUBSTITUTE_ENTITIES',4);
+define('DOMXML_LOAD_PARSING', 0);
+define('DOMXML_LOAD_VALIDATING', 1);
+define('DOMXML_LOAD_RECOVERING', 2);
+define('DOMXML_LOAD_SUBSTITUTE_ENTITIES', 4);
 //define('DOMXML_LOAD_COMPLETE_ATTRS',8);
-define('DOMXML_LOAD_DONT_KEEP_BLANKS',16);
+define('DOMXML_LOAD_DONT_KEEP_BLANKS', 16);
 
-function domxml_new_doc($version) {return new php4DOMDocument();}
-function domxml_new_xmldoc($version) {return new php4DOMDocument();}
-function domxml_open_file($filename,$mode=DOMXML_LOAD_PARSING,&$error=null)
+function domxml_new_doc($version) {return new php4DOMDocument(); }
+function domxml_new_xmldoc($version) {return new php4DOMDocument(); }
+function domxml_open_file($filename, $mode = DOMXML_LOAD_PARSING, &$error = null)
 {
-	$dom=new php4DOMDocument($mode);
-	$errorMode=(func_num_args()>2)&&defined('LIBXML_VERSION');
+	$dom = new php4DOMDocument($mode);
+	$errorMode = (func_num_args() > 2) && defined('LIBXML_VERSION');
 	if ($errorMode) libxml_use_internal_errors(true);
-	if (!$dom->myDOMNode->load($filename)) $dom=null;
+	if (!$dom->myDOMNode->load($filename)) $dom = null;
 	if ($errorMode)
 	{
-		$error=array_map('_error_report',libxml_get_errors());
+		$error = array_map('_error_report', libxml_get_errors());
 		libxml_clear_errors();
 	}
 	return $dom;
 }
-function domxml_open_mem($str,$mode=DOMXML_LOAD_PARSING,&$error=null)
+function domxml_open_mem($str, $mode = DOMXML_LOAD_PARSING, &$error = null)
 {
-	$dom=new php4DOMDocument($mode);
-	$errorMode=(func_num_args()>2)&&defined('LIBXML_VERSION');
+	$dom = new php4DOMDocument($mode);
+	$errorMode = (func_num_args() > 2) && defined('LIBXML_VERSION');
 	if ($errorMode) libxml_use_internal_errors(true);
-	if (!$dom->myDOMNode->loadXML($str)) $dom=null;
+	if (!$dom->myDOMNode->loadXML($str)) $dom = null;
 	if ($errorMode)
 	{
-		$error=array_map('_error_report',libxml_get_errors());
+		$error = array_map('_error_report', libxml_get_errors());
 		libxml_clear_errors();
 	}
 	return $dom;
 }
-function html_doc($html_doc,$from_file=false)
+function html_doc($html_doc, $from_file = false)
 {
-	$dom=new php4DOMDocument();
-	if ($from_file) $result=$dom->myDOMNode->loadHTMLFile($html_doc);
-	else $result=$dom->myDOMNode->loadHTML($html_doc);
+	$dom = new php4DOMDocument();
+	if ($from_file) $result = $dom->myDOMNode->loadHTMLFile($html_doc);
+	else $result = $dom->myDOMNode->loadHTML($html_doc);
 	return $result ? $dom : null;
 }
-function html_doc_file($filename) {return html_doc($filename,true);}
-function xmldoc($str) {return domxml_open_mem($str);}
-function xmldocfile($filename) {return domxml_open_file($filename);}
-function xpath_eval($xpath_context,$eval_str,$contextnode=null) {return $xpath_context->xpath_eval($eval_str,$contextnode);}
-function xpath_new_context($dom_document) {return new php4DOMXPath($dom_document);}
-function xpath_register_ns($xpath_context,$prefix,$namespaceURI) {return $xpath_context->myDOMXPath->registerNamespace($prefix,$namespaceURI);}
-function _entityDecode($text) {return html_entity_decode(strtr($text,array('''=>'\'')),ENT_QUOTES,'UTF-8');}
-function _error_report($error) {return array('errormessage'=>$error->message,'nodename'=>'','line'=>$error->line,'col'=>$error->column)+($error->file==''?array():array('directory'=>dirname($error->file),'file'=>basename($error->file)));}
+function html_doc_file($filename) {return html_doc($filename, true); }
+function xmldoc($str) {return domxml_open_mem($str); }
+function xmldocfile($filename) {return domxml_open_file($filename); }
+function xpath_eval($xpath_context, $eval_str, $contextnode = null) {return $xpath_context->xpath_eval($eval_str, $contextnode); }
+function xpath_new_context($dom_document) {return new php4DOMXPath($dom_document); }
+function xpath_register_ns($xpath_context, $prefix, $namespaceURI) {return $xpath_context->myDOMXPath->registerNamespace($prefix, $namespaceURI); }
+function _entityDecode($text) {return html_entity_decode(strtr($text, array('''=>'\'')), ENT_QUOTES, 'UTF-8'); }
+function _error_report($error) {return array('errormessage'=>$error->message, 'nodename'=>'', 'line'=>$error->line, 'col'=>$error->column) + ($error->file == '' ? array() : array('directory'=>dirname($error->file), 'file'=>basename($error->file))); }
 
 class php4DOMAttr extends php4DOMNode
 {
 	function __get($name)
 	{
-		if ($name==='name') return $this->myDOMNode->name;
+		if ($name === 'name') return $this->myDOMNode->name;
 		else return parent::__get($name);
 	}
-	function name() {return $this->myDOMNode->name;}
+	function name() {return $this->myDOMNode->name; }
 	function set_content($text) {}
 	//function set_value($content) {return $this->myDOMNode->value=htmlspecialchars($content,ENT_QUOTES);}
-	function specified() {return $this->myDOMNode->specified;}
-	function value() {return $this->myDOMNode->value;}
+	function specified() {return $this->myDOMNode->specified; }
+	function value() {return $this->myDOMNode->value; }
 }
 
 class php4DOMDocument extends php4DOMNode
 {
-	function php4DOMDocument($mode=DOMXML_LOAD_PARSING)
+	function php4DOMDocument($mode = DOMXML_LOAD_PARSING)
 	{
-		$this->myDOMNode=new DOMDocument();
-		$this->myOwnerDocument=$this;
-		if ($mode & DOMXML_LOAD_VALIDATING) $this->myDOMNode->validateOnParse=true;
-		if ($mode & DOMXML_LOAD_RECOVERING) $this->myDOMNode->recover=true;
-		if ($mode & DOMXML_LOAD_SUBSTITUTE_ENTITIES) $this->myDOMNode->substituteEntities=true;
-		if ($mode & DOMXML_LOAD_DONT_KEEP_BLANKS) $this->myDOMNode->preserveWhiteSpace=false;
+		$this->myDOMNode = new DOMDocument();
+		$this->myOwnerDocument = $this;
+		if ($mode & DOMXML_LOAD_VALIDATING) $this->myDOMNode->validateOnParse = true;
+		if ($mode & DOMXML_LOAD_RECOVERING) $this->myDOMNode->recover = true;
+		if ($mode & DOMXML_LOAD_SUBSTITUTE_ENTITIES) $this->myDOMNode->substituteEntities = true;
+		if ($mode & DOMXML_LOAD_DONT_KEEP_BLANKS) $this->myDOMNode->preserveWhiteSpace = false;
 	}
 	function add_root($name)
 	{
 		if ($this->myDOMNode->hasChildNodes()) $this->myDOMNode->removeChild($this->myDOMNode->firstChild);
-		return new php4DOMElement($this->myDOMNode->appendChild($this->myDOMNode->createElement($name)),$this->myOwnerDocument);
+		return new php4DOMElement($this->myDOMNode->appendChild($this->myDOMNode->createElement($name)), $this->myOwnerDocument);
 	}
-	function create_attribute($name,$value)
+	function create_attribute($name, $value)
 	{
-		$myAttr=$this->myDOMNode->createAttribute($name);
-		$myAttr->value=htmlspecialchars($value,ENT_QUOTES);
-		return new php4DOMAttr($myAttr,$this);
+		$myAttr = $this->myDOMNode->createAttribute($name);
+		$myAttr->value = htmlspecialchars($value, ENT_QUOTES);
+		return new php4DOMAttr($myAttr, $this);
 	}
-	function create_cdata_section($content) {return new php4DOMNode($this->myDOMNode->createCDATASection($content),$this);}
-	function create_comment($data) {return new php4DOMNode($this->myDOMNode->createComment($data),$this);}
-	function create_element($name) {return new php4DOMElement($this->myDOMNode->createElement($name),$this);}
-	function create_element_ns($uri,$name,$prefix=null)
+	function create_cdata_section($content) {return new php4DOMNode($this->myDOMNode->createCDATASection($content), $this); }
+	function create_comment($data) {return new php4DOMNode($this->myDOMNode->createComment($data), $this); }
+	function create_element($name) {return new php4DOMElement($this->myDOMNode->createElement($name), $this); }
+	function create_element_ns($uri, $name, $prefix = null)
 	{
-		if ($prefix==null) $prefix=$this->myDOMNode->lookupPrefix($uri);
-		if (($prefix==null)&&(($this->myDOMNode->documentElement==null)||(!$this->myDOMNode->documentElement->isDefaultNamespace($uri)))) $prefix='a'.sprintf('%u',crc32($uri));
-		return new php4DOMElement($this->myDOMNode->createElementNS($uri,$prefix==null ? $name : $prefix.':'.$name),$this);
+		if ($prefix == null) $prefix = $this->myDOMNode->lookupPrefix($uri);
+		if (($prefix == null) && (($this->myDOMNode->documentElement == null) || (!$this->myDOMNode->documentElement->isDefaultNamespace($uri)))) $prefix = 'a'.sprintf('%u', crc32($uri));
+		return new php4DOMElement($this->myDOMNode->createElementNS($uri, $prefix == null ? $name : $prefix.':'.$name), $this);
 	}
-	function create_entity_reference($content) {return new php4DOMNode($this->myDOMNode->createEntityReference($content),$this);} //By Walter Ebert 2007-01-22
-	function create_processing_instruction($target,$data=''){return new php4DomProcessingInstruction($this->myDOMNode->createProcessingInstruction($target,$data),$this);}
-	function create_text_node($content) {return new php4DOMText($this->myDOMNode->createTextNode($content),$this);}
-	function document_element() {return parent::_newDOMElement($this->myDOMNode->documentElement,$this);}
-	function dump_file($filename,$compressionmode=false,$format=false)
+	function create_entity_reference($content) {return new php4DOMNode($this->myDOMNode->createEntityReference($content), $this); } //By Walter Ebert 2007-01-22
+	function create_processing_instruction($target, $data = '') {return new php4DomProcessingInstruction($this->myDOMNode->createProcessingInstruction($target, $data), $this); }
+	function create_text_node($content) {return new php4DOMText($this->myDOMNode->createTextNode($content), $this); }
+	function document_element() {return parent::_newDOMElement($this->myDOMNode->documentElement, $this); }
+	function dump_file($filename, $compressionmode = false, $format = false)
 	{
-		$format0=$this->myDOMNode->formatOutput;
-		$this->myDOMNode->formatOutput=$format;
-		$res=$this->myDOMNode->save($filename);
-		$this->myDOMNode->formatOutput=$format0;
+		$format0 = $this->myDOMNode->formatOutput;
+		$this->myDOMNode->formatOutput = $format;
+		$res = $this->myDOMNode->save($filename);
+		$this->myDOMNode->formatOutput = $format0;
 		return $res;
 	}
-	function dump_mem($format=false,$encoding=false)
+	function dump_mem($format = false, $encoding = false)
 	{
-		$format0=$this->myDOMNode->formatOutput;
-		$this->myDOMNode->formatOutput=$format;
-		$encoding0=$this->myDOMNode->encoding;
-		if ($encoding) $this->myDOMNode->encoding=$encoding;
-		$dump=$this->myDOMNode->saveXML();
-		$this->myDOMNode->formatOutput=$format0;
-		if ($encoding) $this->myDOMNode->encoding= $encoding0=='' ? 'UTF-8' : $encoding0; //UTF-8 is XML default encoding
+		$format0 = $this->myDOMNode->formatOutput;
+		$this->myDOMNode->formatOutput = $format;
+		$encoding0 = $this->myDOMNode->encoding;
+		if ($encoding) $this->myDOMNode->encoding = $encoding;
+		$dump = $this->myDOMNode->saveXML();
+		$this->myDOMNode->formatOutput = $format0;
+		if ($encoding) $this->myDOMNode->encoding = $encoding0 == '' ? 'UTF-8' : $encoding0; //UTF-8 is XML default encoding
 		return $dump;
 	}
 	function free()
 	{
 		if ($this->myDOMNode->hasChildNodes()) $this->myDOMNode->removeChild($this->myDOMNode->firstChild);
-		$this->myDOMNode=null;
-		$this->myOwnerDocument=null;
+		$this->myDOMNode = null;
+		$this->myOwnerDocument = null;
 	}
-	function get_element_by_id($id) {return parent::_newDOMElement($this->myDOMNode->getElementById($id),$this);}
+	function get_element_by_id($id) {return parent::_newDOMElement($this->myDOMNode->getElementById($id), $this); }
 	function get_elements_by_tagname($name)
 	{
-		$myDOMNodeList=$this->myDOMNode->getElementsByTagName($name);
-		$nodeSet=array();
-		$i=0;
+		$myDOMNodeList = $this->myDOMNode->getElementsByTagName($name);
+		$nodeSet = array();
+		$i = 0;
 		if (isset($myDOMNodeList))
-			while ($node=$myDOMNodeList->item($i++)) $nodeSet[]=new php4DOMElement($node,$this);
+			while ($node = $myDOMNodeList->item($i++)) $nodeSet[] = new php4DOMElement($node, $this);
 		return $nodeSet;
 	}
-	function html_dump_mem() {return $this->myDOMNode->saveHTML();}
-	function root() {return parent::_newDOMElement($this->myDOMNode->documentElement,$this);}
-	function xinclude() {return $this->myDOMNode->xinclude();}
-	function xpath_new_context() {return new php4DOMXPath($this);}
+	function html_dump_mem() {return $this->myDOMNode->saveHTML(); }
+	function root() {return parent::_newDOMElement($this->myDOMNode->documentElement, $this); }
+	function xinclude() {return $this->myDOMNode->xinclude(); }
+	function xpath_new_context() {return new php4DOMXPath($this); }
 }
 
 class php4DOMElement extends php4DOMNode
 {
-	function add_namespace($uri,$prefix)
+	function add_namespace($uri, $prefix)
 	{
-		if ($this->myDOMNode->hasAttributeNS('http://www.w3.org/2000/xmlns/',$prefix)) return false;
+		if ($this->myDOMNode->hasAttributeNS('http://www.w3.org/2000/xmlns/', $prefix)) return false;
 		else
 		{
-			$this->myDOMNode->setAttributeNS('http://www.w3.org/2000/xmlns/','xmlns:'.$prefix,$uri); //By Daniel Walker 2006-09-08
+			$this->myDOMNode->setAttributeNS('http://www.w3.org/2000/xmlns/', 'xmlns:'.$prefix, $uri); //By Daniel Walker 2006-09-08
 			return true;
 		}
 	}
-	function get_attribute($name) {return $this->myDOMNode->getAttribute($name);}
-	function get_attribute_node($name) {return parent::_newDOMElement($this->myDOMNode->getAttributeNode($name),$this->myOwnerDocument);}
+	function get_attribute($name) {return $this->myDOMNode->getAttribute($name); }
+	function get_attribute_node($name) {return parent::_newDOMElement($this->myDOMNode->getAttributeNode($name), $this->myOwnerDocument); }
 	function get_elements_by_tagname($name)
 	{
-		$myDOMNodeList=$this->myDOMNode->getElementsByTagName($name);
-		$nodeSet=array();
-		$i=0;
+		$myDOMNodeList = $this->myDOMNode->getElementsByTagName($name);
+		$nodeSet = array();
+		$i = 0;
 		if (isset($myDOMNodeList))
-			while ($node=$myDOMNodeList->item($i++)) $nodeSet[]=new php4DOMElement($node,$this->myOwnerDocument);
+			while ($node = $myDOMNodeList->item($i++)) $nodeSet[] = new php4DOMElement($node, $this->myOwnerDocument);
 		return $nodeSet;
 	}
-	function has_attribute($name) {return $this->myDOMNode->hasAttribute($name);}
-	function remove_attribute($name) {return $this->myDOMNode->removeAttribute($name);}
-	function set_attribute($name,$value)
+	function has_attribute($name) {return $this->myDOMNode->hasAttribute($name); }
+	function remove_attribute($name) {return $this->myDOMNode->removeAttribute($name); }
+	function set_attribute($name, $value)
 	{
 		//return $this->myDOMNode->setAttribute($name,$value); //Does not return a DomAttr
-		$myAttr=$this->myDOMNode->ownerDocument->createAttribute($name);
-		$myAttr->value=htmlspecialchars($value,ENT_QUOTES); //Entity problem reported by AL-DesignWorks 2007-09-07
+		$myAttr = $this->myDOMNode->ownerDocument->createAttribute($name);
+		$myAttr->value = htmlspecialchars($value, ENT_QUOTES); //Entity problem reported by AL-DesignWorks 2007-09-07
 		$this->myDOMNode->setAttributeNode($myAttr);
-		return new php4DOMAttr($myAttr,$this->myOwnerDocument);
+		return new php4DOMAttr($myAttr, $this->myOwnerDocument);
 	}
 	/*function set_attribute_node($attr)
 	{
@@ -222,186 +222,186 @@ 
 	}*/
 	function set_name($name)
 	{
-		if ($this->myDOMNode->prefix=='') $newNode=$this->myDOMNode->ownerDocument->createElement($name);
-		else $newNode=$this->myDOMNode->ownerDocument->createElementNS($this->myDOMNode->namespaceURI,$this->myDOMNode->prefix.':'.$name);
-		$myDOMNodeList=$this->myDOMNode->attributes;
-		$i=0;
+		if ($this->myDOMNode->prefix == '') $newNode = $this->myDOMNode->ownerDocument->createElement($name);
+		else $newNode = $this->myDOMNode->ownerDocument->createElementNS($this->myDOMNode->namespaceURI, $this->myDOMNode->prefix.':'.$name);
+		$myDOMNodeList = $this->myDOMNode->attributes;
+		$i = 0;
 		if (isset($myDOMNodeList))
-			while ($node=$myDOMNodeList->item($i++))
-				if ($node->namespaceURI=='') $newNode->setAttribute($node->name,$node->value);
-				else $newNode->setAttributeNS($node->namespaceURI,$node->nodeName,$node->value);
-		$myDOMNodeList=$this->myDOMNode->childNodes;
+			while ($node = $myDOMNodeList->item($i++))
+				if ($node->namespaceURI == '') $newNode->setAttribute($node->name, $node->value);
+				else $newNode->setAttributeNS($node->namespaceURI, $node->nodeName, $node->value);
+		$myDOMNodeList = $this->myDOMNode->childNodes;
 		if (isset($myDOMNodeList))
-			while ($node=$myDOMNodeList->item(0)) $newNode->appendChild($node);
-		$this->myDOMNode->parentNode->replaceChild($newNode,$this->myDOMNode);
-		$this->myDOMNode=$newNode;
+			while ($node = $myDOMNodeList->item(0)) $newNode->appendChild($node);
+		$this->myDOMNode->parentNode->replaceChild($newNode, $this->myDOMNode);
+		$this->myDOMNode = $newNode;
 		return true;
 	}
-	function tagname() {return $this->tagname;}
+	function tagname() {return $this->tagname; }
 }
 
 class php4DOMNode
 {
 	public $myDOMNode;
 	public $myOwnerDocument;
-	function php4DOMNode($aDomNode,$aOwnerDocument)
+	function php4DOMNode($aDomNode, $aOwnerDocument)
 	{
-		$this->myDOMNode=$aDomNode;
-		$this->myOwnerDocument=$aOwnerDocument;
+		$this->myDOMNode = $aDomNode;
+		$this->myOwnerDocument = $aOwnerDocument;
 	}
 	function __get($name)
 	{
 		switch ($name)
 		{
 			case 'type': return $this->myDOMNode->nodeType;
-			case 'tagname': return ($this->myDOMNode->nodeType===XML_ELEMENT_NODE) ? $this->myDOMNode->localName : $this->myDOMNode->tagName; //Avoid namespace prefix for DOMElement
+			case 'tagname': return ($this->myDOMNode->nodeType === XML_ELEMENT_NODE) ? $this->myDOMNode->localName : $this->myDOMNode->tagName; //Avoid namespace prefix for DOMElement
 			case 'content': return $this->myDOMNode->textContent;
 			case 'value': return $this->myDOMNode->value;
 			default:
-				$myErrors=debug_backtrace();
-				trigger_error('Undefined property: '.get_class($this).'::$'.$name.' ['.$myErrors[0]['file'].':'.$myErrors[0]['line'].']',E_USER_NOTICE);
+				$myErrors = debug_backtrace();
+				trigger_error('Undefined property: '.get_class($this).'::$'.$name.' ['.$myErrors[0]['file'].':'.$myErrors[0]['line'].']', E_USER_NOTICE);
 				return false;
 		}
 	}
-	function add_child($newnode) {return append_child($newnode);}
-	function add_namespace($uri,$prefix) {return false;}
-	function append_child($newnode) {return self::_newDOMElement($this->myDOMNode->appendChild($this->_importNode($newnode)),$this->myOwnerDocument);}
-	function append_sibling($newnode) {return self::_newDOMElement($this->myDOMNode->parentNode->appendChild($this->_importNode($newnode)),$this->myOwnerDocument);}
+	function add_child($newnode) {return append_child($newnode); }
+	function add_namespace($uri, $prefix) {return false; }
+	function append_child($newnode) {return self::_newDOMElement($this->myDOMNode->appendChild($this->_importNode($newnode)), $this->myOwnerDocument); }
+	function append_sibling($newnode) {return self::_newDOMElement($this->myDOMNode->parentNode->appendChild($this->_importNode($newnode)), $this->myOwnerDocument); }
 	function attributes()
 	{
-		$myDOMNodeList=$this->myDOMNode->attributes;
-		if (!(isset($myDOMNodeList)&&$this->myDOMNode->hasAttributes())) return null;
-		$nodeSet=array();
-		$i=0;
-		while ($node=$myDOMNodeList->item($i++)) $nodeSet[]=new php4DOMAttr($node,$this->myOwnerDocument);
+		$myDOMNodeList = $this->myDOMNode->attributes;
+		if (!(isset($myDOMNodeList) && $this->myDOMNode->hasAttributes())) return null;
+		$nodeSet = array();
+		$i = 0;
+		while ($node = $myDOMNodeList->item($i++)) $nodeSet[] = new php4DOMAttr($node, $this->myOwnerDocument);
 		return $nodeSet;
 	}
 	function child_nodes()
 	{
-		$myDOMNodeList=$this->myDOMNode->childNodes;
-		$nodeSet=array();
-		$i=0;
+		$myDOMNodeList = $this->myDOMNode->childNodes;
+		$nodeSet = array();
+		$i = 0;
 		if (isset($myDOMNodeList))
-			while ($node=$myDOMNodeList->item($i++)) $nodeSet[]=self::_newDOMElement($node,$this->myOwnerDocument);
+			while ($node = $myDOMNodeList->item($i++)) $nodeSet[] = self::_newDOMElement($node, $this->myOwnerDocument);
 		return $nodeSet;
 	}
-	function children() {return $this->child_nodes();}
-	function clone_node($deep=false) {return self::_newDOMElement($this->myDOMNode->cloneNode($deep),$this->myOwnerDocument);}
+	function children() {return $this->child_nodes(); }
+	function clone_node($deep = false) {return self::_newDOMElement($this->myDOMNode->cloneNode($deep), $this->myOwnerDocument); }
 	//dump_node($node) should only be called on php4DOMDocument
-	function dump_node($node=null) {return $node==null ? $this->myOwnerDocument->myDOMNode->saveXML($this->myDOMNode) : $this->myOwnerDocument->myDOMNode->saveXML($node->myDOMNode);}
-	function first_child() {return self::_newDOMElement($this->myDOMNode->firstChild,$this->myOwnerDocument);}
-	function get_content() {return $this->myDOMNode->textContent;}
-	function has_attributes() {return $this->myDOMNode->hasAttributes();}
-	function has_child_nodes() {return $this->myDOMNode->hasChildNodes();}
-	function insert_before($newnode,$refnode) {return self::_newDOMElement($this->myDOMNode->insertBefore($this->_importNode($newnode),$refnode==null?null:$refnode->myDOMNode),$this->myOwnerDocument);}
-	function is_blank_node() {return ($this->myDOMNode->nodeType===XML_TEXT_NODE)&&preg_match('%^\s*$%',$this->myDOMNode->nodeValue);}
-	function last_child() {return self::_newDOMElement($this->myDOMNode->lastChild,$this->myOwnerDocument);}
-	function new_child($name,$content)
+	function dump_node($node = null) {return $node == null ? $this->myOwnerDocument->myDOMNode->saveXML($this->myDOMNode) : $this->myOwnerDocument->myDOMNode->saveXML($node->myDOMNode); }
+	function first_child() {return self::_newDOMElement($this->myDOMNode->firstChild, $this->myOwnerDocument); }
+	function get_content() {return $this->myDOMNode->textContent; }
+	function has_attributes() {return $this->myDOMNode->hasAttributes(); }
+	function has_child_nodes() {return $this->myDOMNode->hasChildNodes(); }
+	function insert_before($newnode, $refnode) {return self::_newDOMElement($this->myDOMNode->insertBefore($this->_importNode($newnode), $refnode == null ? null : $refnode->myDOMNode), $this->myOwnerDocument); }
+	function is_blank_node() {return ($this->myDOMNode->nodeType === XML_TEXT_NODE) && preg_match('%^\s*$%', $this->myDOMNode->nodeValue); }
+	function last_child() {return self::_newDOMElement($this->myDOMNode->lastChild, $this->myOwnerDocument); }
+	function new_child($name, $content)
 	{
-		$mySubNode=$this->myDOMNode->ownerDocument->createElement($name);
+		$mySubNode = $this->myDOMNode->ownerDocument->createElement($name);
 		$mySubNode->appendChild($this->myDOMNode->ownerDocument->createTextNode(_entityDecode($content)));
 		$this->myDOMNode->appendChild($mySubNode);
-		return new php4DOMElement($mySubNode,$this->myOwnerDocument);
+		return new php4DOMElement($mySubNode, $this->myOwnerDocument);
 	}
-	function next_sibling() {return self::_newDOMElement($this->myDOMNode->nextSibling,$this->myOwnerDocument);}
-	function node_name() {return ($this->myDOMNode->nodeType===XML_ELEMENT_NODE) ? $this->myDOMNode->localName : $this->myDOMNode->nodeName;} //Avoid namespace prefix for DOMElement
-	function node_type() {return $this->myDOMNode->nodeType;}
-	function node_value() {return $this->myDOMNode->nodeValue;}
-	function owner_document() {return $this->myOwnerDocument;}
-	function parent_node() {return self::_newDOMElement($this->myDOMNode->parentNode,$this->myOwnerDocument);}
-	function prefix() {return $this->myDOMNode->prefix;}
-	function previous_sibling() {return self::_newDOMElement($this->myDOMNode->previousSibling,$this->myOwnerDocument);}
-	function remove_child($oldchild) {return self::_newDOMElement($this->myDOMNode->removeChild($oldchild->myDOMNode),$this->myOwnerDocument);}
-	function replace_child($newnode,$oldnode) {return self::_newDOMElement($this->myDOMNode->replaceChild($this->_importNode($newnode),$oldnode->myDOMNode),$this->myOwnerDocument);}
-	function replace_node($newnode) {return self::_newDOMElement($this->myDOMNode->parentNode->replaceChild($this->_importNode($newnode),$this->myDOMNode),$this->myOwnerDocument);}
-	function set_content($text) {return $this->myDOMNode->appendChild($this->myDOMNode->ownerDocument->createTextNode(_entityDecode($text)));} //Entity problem reported by AL-DesignWorks 2007-09-07
+	function next_sibling() {return self::_newDOMElement($this->myDOMNode->nextSibling, $this->myOwnerDocument); }
+	function node_name() {return ($this->myDOMNode->nodeType === XML_ELEMENT_NODE) ? $this->myDOMNode->localName : $this->myDOMNode->nodeName; } //Avoid namespace prefix for DOMElement
+	function node_type() {return $this->myDOMNode->nodeType; }
+	function node_value() {return $this->myDOMNode->nodeValue; }
+	function owner_document() {return $this->myOwnerDocument; }
+	function parent_node() {return self::_newDOMElement($this->myDOMNode->parentNode, $this->myOwnerDocument); }
+	function prefix() {return $this->myDOMNode->prefix; }
+	function previous_sibling() {return self::_newDOMElement($this->myDOMNode->previousSibling, $this->myOwnerDocument); }
+	function remove_child($oldchild) {return self::_newDOMElement($this->myDOMNode->removeChild($oldchild->myDOMNode), $this->myOwnerDocument); }
+	function replace_child($newnode, $oldnode) {return self::_newDOMElement($this->myDOMNode->replaceChild($this->_importNode($newnode), $oldnode->myDOMNode), $this->myOwnerDocument); }
+	function replace_node($newnode) {return self::_newDOMElement($this->myDOMNode->parentNode->replaceChild($this->_importNode($newnode), $this->myDOMNode), $this->myOwnerDocument); }
+	function set_content($text) {return $this->myDOMNode->appendChild($this->myDOMNode->ownerDocument->createTextNode(_entityDecode($text))); } //Entity problem reported by AL-DesignWorks 2007-09-07
 	//function set_name($name) {return $this->myOwnerDocument->renameNode($this->myDOMNode,$this->myDOMNode->namespaceURI,$name);}
-	function set_namespace($uri,$prefix=null)
+	function set_namespace($uri, $prefix = null)
 	{//Contributions by Daniel Walker 2006-09-08
-		$nsprefix=$this->myDOMNode->lookupPrefix($uri);
-		if ($nsprefix==null)
+		$nsprefix = $this->myDOMNode->lookupPrefix($uri);
+		if ($nsprefix == null)
 		{
-			$nsprefix= $prefix==null ? $nsprefix='a'.sprintf('%u',crc32($uri)) : $prefix;
-			if ($this->myDOMNode->nodeType===XML_ATTRIBUTE_NODE)
+			$nsprefix = $prefix == null ? $nsprefix = 'a'.sprintf('%u', crc32($uri)) : $prefix;
+			if ($this->myDOMNode->nodeType === XML_ATTRIBUTE_NODE)
 			{
-				if (($prefix!=null)&&$this->myDOMNode->ownerElement->hasAttributeNS('http://www.w3.org/2000/xmlns/',$nsprefix)&&
-					($this->myDOMNode->ownerElement->getAttributeNS('http://www.w3.org/2000/xmlns/',$nsprefix)!=$uri))
+				if (($prefix != null) && $this->myDOMNode->ownerElement->hasAttributeNS('http://www.w3.org/2000/xmlns/', $nsprefix) &&
+					($this->myDOMNode->ownerElement->getAttributeNS('http://www.w3.org/2000/xmlns/', $nsprefix) != $uri))
 				{//Remove namespace
-					$parent=$this->myDOMNode->ownerElement;
+					$parent = $this->myDOMNode->ownerElement;
 					$parent->removeAttributeNode($this->myDOMNode);
-					$parent->setAttribute($this->myDOMNode->localName,$this->myDOMNode->nodeValue);
-					$this->myDOMNode=$parent->getAttributeNode($this->myDOMNode->localName);
+					$parent->setAttribute($this->myDOMNode->localName, $this->myDOMNode->nodeValue);
+					$this->myDOMNode = $parent->getAttributeNode($this->myDOMNode->localName);
 					return;
 				}
-				$this->myDOMNode->ownerElement->setAttributeNS('http://www.w3.org/2000/xmlns/','xmlns:'.$nsprefix,$uri);
+				$this->myDOMNode->ownerElement->setAttributeNS('http://www.w3.org/2000/xmlns/', 'xmlns:'.$nsprefix, $uri);
 			}
 		}
-		if ($this->myDOMNode->nodeType===XML_ATTRIBUTE_NODE)
+		if ($this->myDOMNode->nodeType === XML_ATTRIBUTE_NODE)
 		{
-			$parent=$this->myDOMNode->ownerElement;
+			$parent = $this->myDOMNode->ownerElement;
 			$parent->removeAttributeNode($this->myDOMNode);
-			$parent->setAttributeNS($uri,$nsprefix.':'.$this->myDOMNode->localName,$this->myDOMNode->nodeValue);
-			$this->myDOMNode=$parent->getAttributeNodeNS($uri,$this->myDOMNode->localName);
+			$parent->setAttributeNS($uri, $nsprefix.':'.$this->myDOMNode->localName, $this->myDOMNode->nodeValue);
+			$this->myDOMNode = $parent->getAttributeNodeNS($uri, $this->myDOMNode->localName);
 		}
-		elseif ($this->myDOMNode->nodeType===XML_ELEMENT_NODE)
+		elseif ($this->myDOMNode->nodeType === XML_ELEMENT_NODE)
 		{
-			$NewNode=$this->myDOMNode->ownerDocument->createElementNS($uri,$nsprefix.':'.$this->myDOMNode->localName);
+			$NewNode = $this->myDOMNode->ownerDocument->createElementNS($uri, $nsprefix.':'.$this->myDOMNode->localName);
 			foreach ($this->myDOMNode->attributes as $n) $NewNode->appendChild($n->cloneNode(true));
 			foreach ($this->myDOMNode->childNodes as $n) $NewNode->appendChild($n->cloneNode(true));
-			$xpath=new DOMXPath($this->myDOMNode->ownerDocument);
-			$myDOMNodeList=$xpath->query('namespace::*[name()!="xml"]',$this->myDOMNode); //Add old namespaces
-			foreach ($myDOMNodeList as $n) $NewNode->setAttributeNS('http://www.w3.org/2000/xmlns/',$n->nodeName,$n->nodeValue); 
-			$this->myDOMNode->parentNode->replaceChild($NewNode,$this->myDOMNode);
-			$this->myDOMNode=$NewNode;
+			$xpath = new DOMXPath($this->myDOMNode->ownerDocument);
+			$myDOMNodeList = $xpath->query('namespace::*[name()!="xml"]', $this->myDOMNode); //Add old namespaces
+			foreach ($myDOMNodeList as $n) $NewNode->setAttributeNS('http://www.w3.org/2000/xmlns/', $n->nodeName, $n->nodeValue); 
+			$this->myDOMNode->parentNode->replaceChild($NewNode, $this->myDOMNode);
+			$this->myDOMNode = $NewNode;
 		}
 	}
 	function unlink_node()
 	{
-		if ($this->myDOMNode->parentNode!=null)
+		if ($this->myDOMNode->parentNode != null)
 		{
-			if ($this->myDOMNode->nodeType===XML_ATTRIBUTE_NODE) $this->myDOMNode->parentNode->removeAttributeNode($this->myDOMNode);
+			if ($this->myDOMNode->nodeType === XML_ATTRIBUTE_NODE) $this->myDOMNode->parentNode->removeAttributeNode($this->myDOMNode);
 			else $this->myDOMNode->parentNode->removeChild($this->myDOMNode);
 		}
 	}
-	protected function _importNode($newnode) {return $this->myOwnerDocument===$newnode->myOwnerDocument ? $newnode->myDOMNode : $this->myOwnerDocument->myDOMNode->importNode($newnode->myDOMNode,true);} //To import DOMNode from another DOMDocument
-	static function _newDOMElement($aDOMNode,$aOwnerDocument)
+	protected function _importNode($newnode) {return $this->myOwnerDocument === $newnode->myOwnerDocument ? $newnode->myDOMNode : $this->myOwnerDocument->myDOMNode->importNode($newnode->myDOMNode, true); } //To import DOMNode from another DOMDocument
+	static function _newDOMElement($aDOMNode, $aOwnerDocument)
 	{//Check the PHP5 DOMNode before creating a new associated PHP4 DOMNode wrapper
-		if ($aDOMNode==null) return null;
+		if ($aDOMNode == null) return null;
 		switch ($aDOMNode->nodeType)
 		{
-			case XML_ELEMENT_NODE: return new php4DOMElement($aDOMNode,$aOwnerDocument);
-			case XML_TEXT_NODE: return new php4DOMText($aDOMNode,$aOwnerDocument);
-			case XML_ATTRIBUTE_NODE: return new php4DOMAttr($aDOMNode,$aOwnerDocument);
-			case XML_PI_NODE: return new php4DomProcessingInstruction($aDOMNode,$aOwnerDocument);
-			default: return new php4DOMNode($aDOMNode,$aOwnerDocument);
+			case XML_ELEMENT_NODE: return new php4DOMElement($aDOMNode, $aOwnerDocument);
+			case XML_TEXT_NODE: return new php4DOMText($aDOMNode, $aOwnerDocument);
+			case XML_ATTRIBUTE_NODE: return new php4DOMAttr($aDOMNode, $aOwnerDocument);
+			case XML_PI_NODE: return new php4DomProcessingInstruction($aDOMNode, $aOwnerDocument);
+			default: return new php4DOMNode($aDOMNode, $aOwnerDocument);
 		}
 	}
 }
 
 class php4DomProcessingInstruction extends php4DOMNode
 {
-	function data() {return $this->myDOMNode->data;}
-	function target() {return $this->myDOMNode->target;}
+	function data() {return $this->myDOMNode->data; }
+	function target() {return $this->myDOMNode->target; }
 }
 
 class php4DOMText extends php4DOMNode
 {
 	function __get($name)
 	{
-		if ($name==='tagname') return '#text';
+		if ($name === 'tagname') return '#text';
 		else return parent::__get($name);
 	}
-	function tagname() {return '#text';}
-	function set_content($text) {$this->myDOMNode->nodeValue=$text; return true;}
+	function tagname() {return '#text'; }
+	function set_content($text) {$this->myDOMNode->nodeValue = $text; return true; }
 }
 
 if (!defined('XPATH_NODESET'))
 {
-	define('XPATH_UNDEFINED',0);
-	define('XPATH_NODESET',1);
-	define('XPATH_BOOLEAN',2);
-	define('XPATH_NUMBER',3);
-	define('XPATH_STRING',4);
+	define('XPATH_UNDEFINED', 0);
+	define('XPATH_NODESET', 1);
+	define('XPATH_BOOLEAN', 2);
+	define('XPATH_NUMBER', 3);
+	define('XPATH_STRING', 4);
 	/*define('XPATH_POINT',5);
 	define('XPATH_RANGE',6);
 	define('XPATH_LOCATIONSET',7);
@@ -413,36 +413,36 @@ 
 {
 	private $myDOMNodelist;
 	public $nodeset;
-	public $type=XPATH_UNDEFINED;
+	public $type = XPATH_UNDEFINED;
 	public $value;
-	function php4DOMNodelist($aDOMNodelist,$aOwnerDocument)
+	function php4DOMNodelist($aDOMNodelist, $aOwnerDocument)
 	{
 		if (!isset($aDOMNodelist)) return; 
-		elseif (is_object($aDOMNodelist)||is_array($aDOMNodelist))
+		elseif (is_object($aDOMNodelist) || is_array($aDOMNodelist))
 		{
-			if ($aDOMNodelist->length>0)
+			if ($aDOMNodelist->length > 0)
 			{
-				$this->myDOMNodelist=$aDOMNodelist;
-				$this->nodeset=array();
-				$this->type=XPATH_NODESET;
-				$i=0;
-				while ($node=$this->myDOMNodelist->item($i++)) $this->nodeset[]=php4DOMNode::_newDOMElement($node,$aOwnerDocument);
+				$this->myDOMNodelist = $aDOMNodelist;
+				$this->nodeset = array();
+				$this->type = XPATH_NODESET;
+				$i = 0;
+				while ($node = $this->myDOMNodelist->item($i++)) $this->nodeset[] = php4DOMNode::_newDOMElement($node, $aOwnerDocument);
 			}
 		}
-		elseif (is_int($aDOMNodelist)||is_float($aDOMNodelist))
+		elseif (is_int($aDOMNodelist) || is_float($aDOMNodelist))
 		{
-			$this->type=XPATH_NUMBER;
-			$this->value=$aDOMNodelist;
+			$this->type = XPATH_NUMBER;
+			$this->value = $aDOMNodelist;
 		}
 		elseif (is_bool($aDOMNodelist))
 		{
-			$this->type=XPATH_BOOLEAN;
-			$this->value=$aDOMNodelist;
+			$this->type = XPATH_BOOLEAN;
+			$this->value = $aDOMNodelist;
 		}
 		elseif (is_string($aDOMNodelist))
 		{
-			$this->type=XPATH_STRING;
-			$this->value=$aDOMNodelist;
+			$this->type = XPATH_STRING;
+			$this->value = $aDOMNodelist;
 		}
 	}
 }
@@ -454,46 +454,46 @@ 
 	function php4DOMXPath($dom_document)
 	{
 		//TODO: If $dom_document is a DomElement, make that default $contextnode and modify XPath. Ex: '/test'
-		$this->myOwnerDocument=$dom_document->myOwnerDocument;
-		$this->myDOMXPath=new DOMXPath($this->myOwnerDocument->myDOMNode);
+		$this->myOwnerDocument = $dom_document->myOwnerDocument;
+		$this->myDOMXPath = new DOMXPath($this->myOwnerDocument->myDOMNode);
 	}
-	function xpath_eval($eval_str,$contextnode=null)
+	function xpath_eval($eval_str, $contextnode = null)
 	{
-		if (method_exists($this->myDOMXPath,'evaluate')) $xp=isset($contextnode) ? $this->myDOMXPath->evaluate($eval_str,$contextnode->myDOMNode) : $this->myDOMXPath->evaluate($eval_str);
-		else $xp=isset($contextnode) ? $this->myDOMXPath->query($eval_str,$contextnode->myDOMNode) : $this->myDOMXPath->query($eval_str);
-		$xp=new php4DOMNodelist($xp,$this->myOwnerDocument);
-		return ($xp->type===XPATH_UNDEFINED) ? false : $xp;
+		if (method_exists($this->myDOMXPath, 'evaluate')) $xp = isset($contextnode) ? $this->myDOMXPath->evaluate($eval_str, $contextnode->myDOMNode) : $this->myDOMXPath->evaluate($eval_str);
+		else $xp = isset($contextnode) ? $this->myDOMXPath->query($eval_str, $contextnode->myDOMNode) : $this->myDOMXPath->query($eval_str);
+		$xp = new php4DOMNodelist($xp, $this->myOwnerDocument);
+		return ($xp->type === XPATH_UNDEFINED) ? false : $xp;
 	}
-	function xpath_register_ns($prefix,$namespaceURI) {return $this->myDOMXPath->registerNamespace($prefix,$namespaceURI);}
+	function xpath_register_ns($prefix, $namespaceURI) {return $this->myDOMXPath->registerNamespace($prefix, $namespaceURI); }
 }
 
 if (extension_loaded('xsl'))
 {//See also: http://alexandre.alapetite.net/doc-alex/xslt-php4-php5/
-	function domxml_xslt_stylesheet($xslstring) {return new php4DomXsltStylesheet(DOMDocument::loadXML($xslstring));}
-	function domxml_xslt_stylesheet_doc($dom_document) {return new php4DomXsltStylesheet($dom_document);}
-	function domxml_xslt_stylesheet_file($xslfile) {return new php4DomXsltStylesheet(DOMDocument::load($xslfile));}
+	function domxml_xslt_stylesheet($xslstring) {return new php4DomXsltStylesheet(DOMDocument::loadXML($xslstring)); }
+	function domxml_xslt_stylesheet_doc($dom_document) {return new php4DomXsltStylesheet($dom_document); }
+	function domxml_xslt_stylesheet_file($xslfile) {return new php4DomXsltStylesheet(DOMDocument::load($xslfile)); }
 	class php4DomXsltStylesheet
 	{
 		private $myxsltProcessor;
 		function php4DomXsltStylesheet($dom_document)
 		{
-			$this->myxsltProcessor=new xsltProcessor();
+			$this->myxsltProcessor = new xsltProcessor();
 			$this->myxsltProcessor->importStyleSheet($dom_document);
 		}
-		function process($dom_document,$xslt_parameters=array(),$param_is_xpath=false)
+		function process($dom_document, $xslt_parameters = array(), $param_is_xpath = false)
 		{
-			foreach ($xslt_parameters as $param=>$value) $this->myxsltProcessor->setParameter('',$param,$value);
-			$myphp4DOMDocument=new php4DOMDocument();
-			$myphp4DOMDocument->myDOMNode=$this->myxsltProcessor->transformToDoc($dom_document->myDOMNode);
+			foreach ($xslt_parameters as $param=>$value) $this->myxsltProcessor->setParameter('', $param, $value);
+			$myphp4DOMDocument = new php4DOMDocument();
+			$myphp4DOMDocument->myDOMNode = $this->myxsltProcessor->transformToDoc($dom_document->myDOMNode);
 			return $myphp4DOMDocument;
 		}
-		function result_dump_file($dom_document,$filename)
+		function result_dump_file($dom_document, $filename)
 		{
-			$html=$dom_document->myDOMNode->saveHTML();
-			file_put_contents($filename,$html);
+			$html = $dom_document->myDOMNode->saveHTML();
+			file_put_contents($filename, $html);
 			return $html;
 		}
-		function result_dump_mem($dom_document) {return $dom_document->myDOMNode->saveHTML();}
+		function result_dump_mem($dom_document) {return $dom_document->myDOMNode->saveHTML(); }
 	}
 }
 ?>

main/auth/cas/lib/CAS/client.php

Spacing +196 added lines, -196 removed lines

@@ -80,9 +80,9 @@ 
 	 */
 	function HTMLFilterOutput($str)
 		{
-		$str = str_replace('__CAS_VERSION__',$this->getServerVersion(),$str);
-		$str = str_replace('__PHPCAS_VERSION__',phpCAS::getVersion(),$str);
-		$str = str_replace('__SERVER_BASE_URL__',$this->getServerBaseURL(),$str);
+		$str = str_replace('__CAS_VERSION__', $this->getServerVersion(), $str);
+		$str = str_replace('__PHPCAS_VERSION__', phpCAS::getVersion(), $str);
+		$str = str_replace('__SERVER_BASE_URL__', $this->getServerBaseURL(), $str);
 		echo $str;
 		}
 	
@@ -136,7 +136,7 @@ 
 	function printHTMLFooter()
 		{
 		$this->HTMLFilterOutput(empty($this->_output_footer)
-			?('<hr><address>phpCAS __PHPCAS_VERSION__ '.$this->getString(CAS_STR_USING_SERVER).' <a href="__SERVER_BASE_URL__">__SERVER_BASE_URL__</a> (CAS __CAS_VERSION__)</a></address></body></html>')
+			? ('<hr><address>phpCAS __PHPCAS_VERSION__ '.$this->getString(CAS_STR_USING_SERVER).' <a href="__SERVER_BASE_URL__">__SERVER_BASE_URL__</a> (CAS __CAS_VERSION__)</a></address></body></html>')
 					:$this->_output_footer);
 		}
 	
@@ -193,7 +193,7 @@ 
 	 */
 	function getLang()
 		{
-		if ( empty($this->_lang) )
+		if (empty($this->_lang))
 			$this->setLang(PHPCAS_LANG_DEFAULT);
 		return $this->_lang;
 		}
@@ -223,8 +223,8 @@ 
 		// call CASclient::getLang() to be sure the language is initialized
 		$this->getLang();
 		
-		if ( !isset($this->_strings[$str]) ) {
-			trigger_error('string `'.$str.'\' not defined for language `'.$this->getLang().'\'',E_USER_ERROR);
+		if (!isset($this->_strings[$str])) {
+			trigger_error('string `'.$str.'\' not defined for language `'.$this->getLang().'\'', E_USER_ERROR);
 		}
 		return $this->_strings[$str];
 		}
@@ -243,8 +243,8 @@ 
 		// include the corresponding language file
 		include_once(dirname(__FILE__).'/languages/'.$lang.'.php');
 		
-		if ( !is_array($this->_strings) ) {
-			trigger_error('language `'.$lang.'\' is not implemented',E_USER_ERROR);
+		if (!is_array($this->_strings)) {
+			trigger_error('language `'.$lang.'\' is not implemented', E_USER_ERROR);
 		}
 		$this->_lang = $lang;
 		}
@@ -331,7 +331,7 @@ 
 	function getServerBaseURL()
 		{ 
 		// the URL is build only when needed
-		if ( empty($this->_server['base_url']) ) {
+		if (empty($this->_server['base_url'])) {
 			$this->_server['base_url'] = 'https://'
 				.$this->getServerHostname()
 				.':'
@@ -350,15 +350,15 @@ 
 	 * @return a URL.
 	 * @private
 	 */
-	function getServerLoginURL($gateway=false,$renew=false) {
+	function getServerLoginURL($gateway = false, $renew = false) {
 		phpCAS::traceBegin();
 		// the URL is build only when needed
-		if ( empty($this->_server['login_url']) ) {
+		if (empty($this->_server['login_url'])) {
 			$this->_server['login_url'] = $this->getServerBaseURL();
 			$this->_server['login_url'] .= 'login?service=';
 			// $this->_server['login_url'] .= preg_replace('/&/','%26',$this->getURL());
 			$this->_server['login_url'] .= urlencode($this->getURL());
-			if($renew) {
+			if ($renew) {
 				// It is recommended that when the "renew" parameter is set, its value be "true"
 				$this->_server['login_url'] .= '&renew=true';
 			} elseif ($gateway) {
@@ -426,7 +426,7 @@ 
 	function getServerServiceValidateURL()
 		{ 
 		// the URL is build only when needed
-		if ( empty($this->_server['service_validate_url']) ) {
+		if (empty($this->_server['service_validate_url'])) {
 			switch ($this->getServerVersion()) {
 				case CAS_VERSION_1_0:
 					$this->_server['service_validate_url'] = $this->getServerBaseURL().'validate';
@@ -448,7 +448,7 @@ 
 		{
 		phpCAS::traceBegin();
 		// the URL is build only when needed
-		if ( empty($this->_server['saml_validate_url']) ) {
+		if (empty($this->_server['saml_validate_url'])) {
 			switch ($this->getServerVersion()) {
 				case SAML_VERSION_1_1:
 					$this->_server['saml_validate_url'] = $this->getServerBaseURL().'samlValidate';
@@ -466,7 +466,7 @@ 
 	function getServerProxyValidateURL()
 		{ 
 		// the URL is build only when needed
-		if ( empty($this->_server['proxy_validate_url']) ) {
+		if (empty($this->_server['proxy_validate_url'])) {
 			switch ($this->getServerVersion()) {
 				case CAS_VERSION_1_0:
 					$this->_server['proxy_validate_url'] = '';
@@ -488,7 +488,7 @@ 
 	function getServerProxyURL()
 		{ 
 		// the URL is build only when needed
-		if ( empty($this->_server['proxy_url']) ) {
+		if (empty($this->_server['proxy_url'])) {
 			switch ($this->getServerVersion()) {
 				case CAS_VERSION_1_0:
 					$this->_server['proxy_url'] = '';
@@ -509,7 +509,7 @@ 
 	function getServerLogoutURL()
 		{ 
 		// the URL is build only when needed
-		if ( empty($this->_server['logout_url']) ) {
+		if (empty($this->_server['logout_url'])) {
 			$this->_server['logout_url'] = $this->getServerBaseURL().'logout';
 		}
 		return $this->_server['logout_url']; 
@@ -547,7 +547,7 @@ 
 	function isHttps() {
 		//if ( isset($_SERVER['HTTPS']) && !empty($_SERVER['HTTPS']) ) {
 		//0.4.24 by Hinnack
-		if ( isset($_SERVER['HTTPS']) && !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == 'on') {
+		if (isset($_SERVER['HTTPS']) && !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == 'on') {
 			return true;
 		} else {
 			return false;
@@ -582,45 +582,45 @@ 
 		phpCAS::traceBegin();
 		
 		// the redirect header() call and DOM parsing code from domxml-php4-php5.php won't work in PHP4 compatibility mode
-		if (version_compare(PHP_VERSION,'5','>=') && ini_get('zend.ze1_compatibility_mode')) {
+		if (version_compare(PHP_VERSION, '5', '>=') && ini_get('zend.ze1_compatibility_mode')) {
 			phpCAS::error('phpCAS cannot support zend.ze1_compatibility_mode. Sorry.');
 		}
 		// skip Session Handling for logout requests and if don't want it'
 		if ($start_session && !$this->isLogoutRequest()) {
 			phpCAS::trace("Starting session handling");
 			// Check for Tickets from the CAS server
-			if (empty($_GET['ticket'])){
+			if (empty($_GET['ticket'])) {
 				phpCAS::trace("No ticket found");
 				// only create a session if necessary
 				if (!session_id()) {
 					phpCAS::trace("No session found, creating new session");
 					session_start();
 				}
-			}else{
+			} else {
 				phpCAS::trace("Ticket found");
 				// We have to copy any old data before renaming the session
 				if (session_id()) {
 					phpCAS::trace("Old active session found, saving old data and destroying session");
 					$old_session = $_SESSION;
 					session_destroy();	
-				}else{
+				} else {
 					session_start();
 					phpCAS::trace("Starting possible old session to copy variables");
 					$old_session = $_SESSION;
 					session_destroy();	
 				}
 				// set up a new session, of name based on the ticket
-				$session_id = preg_replace('/[^\w]/','',$_GET['ticket']);
-				phpCAS::LOG("Session ID: " . $session_id);
+				$session_id = preg_replace('/[^\w]/', '', $_GET['ticket']);
+				phpCAS::LOG("Session ID: ".$session_id);
 				session_id($session_id);
 				session_start();
 				// restore old session vars
-				if(isset($old_session)){
+				if (isset($old_session)) {
 					phpCAS::trace("Restoring old session vars");
 					$_SESSION = $old_session;
 				}
 			}
-		}else{
+		} else {
 			phpCAS::trace("Skipping session creation");
 		}
 		
@@ -631,7 +631,7 @@ 
 		//check version
 		switch ($server_version) {
 			case CAS_VERSION_1_0:
-				if ( $this->isProxy() )
+				if ($this->isProxy())
 					phpCAS::error('CAS proxies are not supported in CAS '
 						.$server_version);
 				break;
@@ -648,35 +648,35 @@ 
 		$this->_server['version'] = $server_version;
 		
 		// check hostname
-		if ( empty($server_hostname) 
-				|| !preg_match('/[\.\d\-abcdefghijklmnopqrstuvwxyz]*/',$server_hostname) ) {
+		if (empty($server_hostname) 
+				|| !preg_match('/[\.\d\-abcdefghijklmnopqrstuvwxyz]*/', $server_hostname)) {
 			phpCAS::error('bad CAS server hostname (`'.$server_hostname.'\')');
 		}
 		$this->_server['hostname'] = $server_hostname;
 		
 		// check port
-		if ( $server_port == 0 
-			|| !is_int($server_port) ) {
+		if ($server_port == 0 
+			|| !is_int($server_port)) {
 			phpCAS::error('bad CAS server port (`'.$server_hostname.'\')');
 		}
 		$this->_server['port'] = $server_port;
 		
 		// check URI
-		if ( !preg_match('/[\.\d\-_abcdefghijklmnopqrstuvwxyz\/]*/',$server_uri) ) {
+		if (!preg_match('/[\.\d\-_abcdefghijklmnopqrstuvwxyz\/]*/', $server_uri)) {
 			phpCAS::error('bad CAS server URI (`'.$server_uri.'\')');
 		}
 		// add leading and trailing `/' and remove doubles      
-		$server_uri = preg_replace('/\/\//','/','/'.$server_uri.'/');
+		$server_uri = preg_replace('/\/\//', '/', '/'.$server_uri.'/');
 		$this->_server['uri'] = $server_uri;
 		
 		// set to callback mode if PgtIou and PgtId CGI GET parameters are provided 
-		if ( $this->isProxy() ) {
-			$this->setCallbackMode(!empty($_GET['pgtIou'])&&!empty($_GET['pgtId']));
+		if ($this->isProxy()) {
+			$this->setCallbackMode(!empty($_GET['pgtIou']) && !empty($_GET['pgtId']));
 		}
 		
-		if ( $this->isCallbackMode() ) {
+		if ($this->isCallbackMode()) {
 			//callback mode: check that phpCAS is secured
-			if ( !$this->isHttps() ) {
+			if (!$this->isHttps()) {
 				phpCAS::error('CAS proxies must be secured to use phpCAS; PGT\'s will not be received from the CAS server');
 			}
 		} else {
@@ -684,33 +684,33 @@ 
 			$ticket = (isset($_GET['ticket']) ? $_GET['ticket'] : null);
 			switch ($this->getServerVersion()) {
 				case CAS_VERSION_1_0: // check for a Service Ticket
-					if( preg_match('/^ST-/',$ticket) ) {
+					if (preg_match('/^ST-/', $ticket)) {
 						phpCAS::trace('ST \''.$ticket.'\' found');
 						//ST present
 						$this->setST($ticket);
 						//ticket has been taken into account, unset it to hide it to applications
 						unset($_GET['ticket']);
-					} else if ( !empty($ticket) ) {
+					} else if (!empty($ticket)) {
 						//ill-formed ticket, halt
 						phpCAS::error('ill-formed ticket found in the URL (ticket=`'.htmlentities($ticket).'\')');
 					}
 					break;
 				case CAS_VERSION_2_0: // check for a Service or Proxy Ticket
-					if( preg_match('/^[SP]T-/',$ticket) ) {
+					if (preg_match('/^[SP]T-/', $ticket)) {
 						phpCAS::trace('ST or PT \''.$ticket.'\' found');
 						$this->setPT($ticket);
 						unset($_GET['ticket']);
-					} else if ( !empty($ticket) ) {
+					} else if (!empty($ticket)) {
 						//ill-formed ticket, halt
 						phpCAS::error('ill-formed ticket found in the URL (ticket=`'.htmlentities($ticket).'\')');
 					} 
 					break;
 				case SAML_VERSION_1_1: // SAML just does Service Tickets
-					if( preg_match('/^[SP]T-/',$ticket) ) {
+					if (preg_match('/^[SP]T-/', $ticket)) {
 						phpCAS::trace('SA \''.$ticket.'\' found');
 						$this->setSA($ticket);
 						unset($_GET['ticket']);
-					} else if ( !empty($ticket) ) {
+					} else if (!empty($ticket)) {
 						//ill-formed ticket, halt
 						phpCAS::error('ill-formed ticket found in the URL (ticket=`'.htmlentities($ticket).'\')');
 					}
@@ -763,7 +763,7 @@ 
 	 */
 	function getUser()
 		{
-		if ( empty($this->_user) ) {
+		if (empty($this->_user)) {
 			phpCAS::error('this method should be used only after '.__CLASS__.'::forceAuthentication() or '.__CLASS__.'::isAuthenticated()');
 		}
 		return $this->_user;
@@ -790,7 +790,7 @@ 
 		{ $this->_attributes = $attributes; }
 	
 	function getAttributes() {
-		if ( empty($this->_user) ) { // if no user is set, there shouldn't be any attributes also...
+		if (empty($this->_user)) { // if no user is set, there shouldn't be any attributes also...
 			phpCAS::error('this method should be used only after '.__CLASS__.'::forceAuthentication() or '.__CLASS__.'::isAuthenticated()');
 		}
 		return $this->_attributes;
@@ -802,8 +802,8 @@ 
 	function hasAttribute($key)
 		{ return (is_array($this->_attributes) && array_key_exists($key, $this->_attributes)); }
 	
-	function getAttribute($key)	{
-		if($this->hasAttribute($key)) {
+	function getAttribute($key) {
+		if ($this->hasAttribute($key)) {
 			return $this->_attributes[$key];
 		}
 	}
@@ -814,14 +814,14 @@ 
 	 * If not, redirect to CAS
 	 * @public
 	 */
-	function renewAuthentication(){
+	function renewAuthentication() {
 		phpCAS::traceBegin();
 		// Either way, the user is authenticated by CAS
-		if( isset( $_SESSION['phpCAS']['auth_checked'] ) )
+		if (isset($_SESSION['phpCAS']['auth_checked']))
 			unset($_SESSION['phpCAS']['auth_checked']);
-		if ( $this->isAuthenticated() ) {
+		if ($this->isAuthenticated()) {
 			phpCAS::trace('user already authenticated; renew');
-			$this->redirectToCas(false,true);
+			$this->redirectToCas(false, true);
 		} else {
 			$this->redirectToCas();
 		}
@@ -838,7 +838,7 @@ 
 		{
 		phpCAS::traceBegin();
 		
-		if ( $this->isAuthenticated() ) {
+		if ($this->isAuthenticated()) {
 			// the user is authenticated, nothing to be done.
 			phpCAS::trace('no need to authenticate');
 			$res = TRUE;
@@ -883,7 +883,7 @@ 
 	function checkAuthentication()
 		{
 		phpCAS::traceBegin();
-		if ( $this->isAuthenticated() ) {
+		if ($this->isAuthenticated()) {
             phpCAS::trace('user is authenticated');
 			$res = TRUE;
 		} else if (isset($_SESSION['phpCAS']['auth_checked'])) {
@@ -897,7 +897,7 @@ 
 			//	    // never reached
 			//	    $res = FALSE;
 			// avoid a check against CAS on every request
-			if (! isset($_SESSION['phpCAS']['unauth_count']) )
+			if (!isset($_SESSION['phpCAS']['unauth_count']))
 				$_SESSION['phpCAS']['unauth_count'] = -2; // uninitialized
 			
 			if (($_SESSION['phpCAS']['unauth_count'] != -2 && $this->_cache_times_for_auth_recheck == -1) 
@@ -943,43 +943,43 @@ 
 		$res = FALSE;
 		$validate_url = '';
 		
-		if ( $this->wasPreviouslyAuthenticated() ) {
+		if ($this->wasPreviouslyAuthenticated()) {
 			// the user has already (previously during the session) been
 			// authenticated, nothing to be done.
 			phpCAS::trace('user was already authenticated, no need to look for tickets');
 			$res = TRUE;
 		}
 		else {
-			if ( $this->hasST() ) {
+			if ($this->hasST()) {
 				// if a Service Ticket was given, validate it
 				phpCAS::trace('ST `'.$this->getST().'\' is present');
-				$this->validateST($validate_url,$text_response,$tree_response); // if it fails, it halts
+				$this->validateST($validate_url, $text_response, $tree_response); // if it fails, it halts
 				phpCAS::trace('ST `'.$this->getST().'\' was validated');
-				if ( $this->isProxy() ) {
-					$this->validatePGT($validate_url,$text_response,$tree_response); // idem
+				if ($this->isProxy()) {
+					$this->validatePGT($validate_url, $text_response, $tree_response); // idem
 					phpCAS::trace('PGT `'.$this->getPGT().'\' was validated');
 					$_SESSION['phpCAS']['pgt'] = $this->getPGT();
 				}
 				$_SESSION['phpCAS']['user'] = $this->getUser();
 				$res = TRUE;
 			}
-			elseif ( $this->hasPT() ) {
+			elseif ($this->hasPT()) {
 				// if a Proxy Ticket was given, validate it
 				phpCAS::trace('PT `'.$this->getPT().'\' is present');
-				$this->validatePT($validate_url,$text_response,$tree_response); // note: if it fails, it halts
+				$this->validatePT($validate_url, $text_response, $tree_response); // note: if it fails, it halts
 				phpCAS::trace('PT `'.$this->getPT().'\' was validated');
-				if ( $this->isProxy() ) {
-					$this->validatePGT($validate_url,$text_response,$tree_response); // idem
+				if ($this->isProxy()) {
+					$this->validatePGT($validate_url, $text_response, $tree_response); // idem
 					phpCAS::trace('PGT `'.$this->getPGT().'\' was validated');
 					$_SESSION['phpCAS']['pgt'] = $this->getPGT();
 				}
 				$_SESSION['phpCAS']['user'] = $this->getUser();
 				$res = TRUE;
 			}
-			elseif ( $this->hasSA() ) {
+			elseif ($this->hasSA()) {
 				// if we have a SAML ticket, validate it.
 				phpCAS::trace('SA `'.$this->getSA().'\' is present');
-				$this->validateSA($validate_url,$text_response,$tree_response); // if it fails, it halts
+				$this->validateSA($validate_url, $text_response, $tree_response); // if it fails, it halts
 				phpCAS::trace('SA `'.$this->getSA().'\' was validated');
 				$_SESSION['phpCAS']['user'] = $this->getUser();
 				$_SESSION['phpCAS']['attributes'] = $this->getAttributes();
@@ -993,7 +993,7 @@ 
 				// if called with a ticket parameter, we need to redirect to the app without the ticket so that CAS-ification is transparent to the browser (for later POSTS)
 				// most of the checks and errors should have been made now, so we're safe for redirect without masking error messages.
 				header('Location: '.$this->getURL());
-				phpCAS::log( "Prepare redirect to : ".$this->getURL() );
+				phpCAS::log("Prepare redirect to : ".$this->getURL());
 			}
 		}
 		
@@ -1006,7 +1006,7 @@ 
 	 * @return true if authenticated based soley on $_SESSION variable
 	 * @since 0.4.22 by Brendan Arnold
 	 */
-	function isSessionAuthenticated ()
+	function isSessionAuthenticated()
 		{
 		return !empty($_SESSION['phpCAS']['user']);
 		}
@@ -1025,28 +1025,28 @@ 
 		{
 		phpCAS::traceBegin();
 		
-		if ( $this->isCallbackMode() ) {
+		if ($this->isCallbackMode()) {
 			$this->callback();
 		}
 		
 		$auth = FALSE;
 		
-		if ( $this->isProxy() ) {
+		if ($this->isProxy()) {
 			// CAS proxy: username and PGT must be present
-			if ( $this->isSessionAuthenticated() && !empty($_SESSION['phpCAS']['pgt']) ) {
+			if ($this->isSessionAuthenticated() && !empty($_SESSION['phpCAS']['pgt'])) {
 				// authentication already done
 				$this->setUser($_SESSION['phpCAS']['user']);
 				$this->setPGT($_SESSION['phpCAS']['pgt']);
 				phpCAS::trace('user = `'.$_SESSION['phpCAS']['user'].'\', PGT = `'.$_SESSION['phpCAS']['pgt'].'\''); 
 				$auth = TRUE;
-			} elseif ( $this->isSessionAuthenticated() && empty($_SESSION['phpCAS']['pgt']) ) {
+			} elseif ($this->isSessionAuthenticated() && empty($_SESSION['phpCAS']['pgt'])) {
 				// these two variables should be empty or not empty at the same time
 				phpCAS::trace('username found (`'.$_SESSION['phpCAS']['user'].'\') but PGT is empty');
 				// unset all tickets to enforce authentication
 				unset($_SESSION['phpCAS']);
 				$this->setST('');
 				$this->setPT('');
-			} elseif ( !$this->isSessionAuthenticated() && !empty($_SESSION['phpCAS']['pgt']) ) {
+			} elseif (!$this->isSessionAuthenticated() && !empty($_SESSION['phpCAS']['pgt'])) {
 				// these two variables should be empty or not empty at the same time
 				phpCAS::trace('PGT found (`'.$_SESSION['phpCAS']['pgt'].'\') but username is empty'); 
 				// unset all tickets to enforce authentication
@@ -1058,10 +1058,10 @@ 
 			}
 		} else {
 			// `simple' CAS client (not a proxy): username must be present
-			if ( $this->isSessionAuthenticated() ) {
+			if ($this->isSessionAuthenticated()) {
 				// authentication already done
 				$this->setUser($_SESSION['phpCAS']['user']);
-				if(isset($_SESSION['phpCAS']['attributes'])){
+				if (isset($_SESSION['phpCAS']['attributes'])) {
 					$this->setAttributes($_SESSION['phpCAS']['attributes']);
 				}
 				phpCAS::trace('user = `'.$_SESSION['phpCAS']['user'].'\''); 
@@ -1082,15 +1082,15 @@ 
 	 * @param $renew true to force the authentication with the CAS server
 	 * @public
 	 */
-	function redirectToCas($gateway=false,$renew=false){
+	function redirectToCas($gateway = false, $renew = false) {
 		phpCAS::traceBegin();
-		$cas_url = $this->getServerLoginURL($gateway,$renew);
+		$cas_url = $this->getServerLoginURL($gateway, $renew);
 		header('Location: '.$cas_url);
-		phpCAS::log( "Redirect to : ".$cas_url );
+		phpCAS::log("Redirect to : ".$cas_url);
 		
 		$this->printHTMLHeader($this->getString(CAS_STR_AUTHENTICATION_WANTED));
 		
-		printf('<p>'.$this->getString(CAS_STR_SHOULD_HAVE_BEEN_REDIRECTED).'</p>',$cas_url);
+		printf('<p>'.$this->getString(CAS_STR_SHOULD_HAVE_BEEN_REDIRECTED).'</p>', $cas_url);
 		$this->printHTMLFooter();
 		
 		phpCAS::traceExit();
@@ -1108,20 +1108,20 @@ 
 		$cas_url = $this->getServerLogoutURL();
 		$paramSeparator = '?';
 		if (isset($params['url'])) {
-			$cas_url = $cas_url . $paramSeparator . "url=" . urlencode($params['url']); 
+			$cas_url = $cas_url.$paramSeparator."url=".urlencode($params['url']); 
 			$paramSeparator = '&';
 		}
 		if (isset($params['service'])) {
-			$cas_url = $cas_url . $paramSeparator . "service=" . urlencode($params['service']); 
+			$cas_url = $cas_url.$paramSeparator."service=".urlencode($params['service']); 
 		}
 		header('Location: '.$cas_url);
-		phpCAS::log( "Prepare redirect to : ".$cas_url );
+		phpCAS::log("Prepare redirect to : ".$cas_url);
 		
 		session_unset();
 		session_destroy();
 		
 		$this->printHTMLHeader($this->getString(CAS_STR_LOGOUT));
-		printf('<p>'.$this->getString(CAS_STR_SHOULD_HAVE_BEEN_REDIRECTED).'</p>',$cas_url);
+		printf('<p>'.$this->getString(CAS_STR_SHOULD_HAVE_BEEN_REDIRECTED).'</p>', $cas_url);
 		$this->printHTMLFooter();
 		
 		phpCAS::traceExit();
@@ -1151,7 +1151,7 @@ 
 	 * By default, only the CAs server (declared in the constructor) will be allowed.
 	 * @public
 	 */
-	function handleLogoutRequests($check_client=true, $allowed_clients=false) {
+	function handleLogoutRequests($check_client = true, $allowed_clients = false) {
 		phpCAS::traceBegin();
 		if (!$this->isLogoutRequest()) {
 			phpCAS::log("Not a logout request");
@@ -1162,7 +1162,7 @@ 
 		phpCAS::log("SAML REQUEST: ".$_POST['logoutRequest']);
 		if ($check_client) {
 			if (!$allowed_clients) {
-				$allowed_clients = array( $this->getServerHostname() ); 
+				$allowed_clients = array($this->getServerHostname()); 
 			}
 			$client_ip = $_SERVER['REMOTE_ADDR'];
 			$client = gethostbyaddr($client_ip);
@@ -1188,21 +1188,21 @@ 
 		}
 		// Extract the ticket from the SAML Request
 		preg_match("|<samlp:SessionIndex>(.*)</samlp:SessionIndex>|", $_POST['logoutRequest'], $tick, PREG_OFFSET_CAPTURE, 3);
-		$wrappedSamlSessionIndex = preg_replace('|<samlp:SessionIndex>|','',$tick[0][0]);
-		$ticket2logout = preg_replace('|</samlp:SessionIndex>|','',$wrappedSamlSessionIndex);
+		$wrappedSamlSessionIndex = preg_replace('|<samlp:SessionIndex>|', '', $tick[0][0]);
+		$ticket2logout = preg_replace('|</samlp:SessionIndex>|', '', $wrappedSamlSessionIndex);
 		phpCAS::log("Ticket to logout: ".$ticket2logout);
-		$session_id = preg_replace('/[^\w]/','',$ticket2logout);
+		$session_id = preg_replace('/[^\w]/', '', $ticket2logout);
 		phpCAS::log("Session id: ".$session_id);
 		
 		// destroy a possible application session created before phpcas
-		if(session_id()){
+		if (session_id()) {
 			session_unset();
 			session_destroy();
 		}
 		// fix session ID
 		session_id($session_id);
-		$_COOKIE[session_name()]=$session_id;
-		$_GET[session_name()]=$session_id;
+		$_COOKIE[session_name()] = $session_id;
+		$_GET[session_name()] = $session_id;
 		
 		// Overwrite session
 		session_start();	
@@ -1338,18 +1338,18 @@ 
 	 *
 	 * @private
 	 */
-	function validateST($validate_url,&$text_response,&$tree_response)
+	function validateST($validate_url, &$text_response, &$tree_response)
 		{
 		phpCAS::traceBegin();
 		// build the URL to validate the ticket
 		$validate_url = $this->getServerServiceValidateURL().'&ticket='.$this->getST();
-		if ( $this->isProxy() ) {
+		if ($this->isProxy()) {
 			// pass the callback url for CAS proxies
 			$validate_url .= '&pgtUrl='.$this->getCallbackURL();
 		}
 		
 		// open and read the URL
-		if ( !$this->readURL($validate_url,''/*cookies*/,$headers,$text_response,$err_msg) ) {
+		if (!$this->readURL($validate_url, ''/*cookies*/, $headers, $text_response, $err_msg)) {
 			phpCAS::trace('could not open URL \''.$validate_url.'\' to validate ('.$err_msg.')');
 			$this->authError('ST not validated',
 				$validate_url,
@@ -1359,7 +1359,7 @@ 
 		// analyze the result depending on the version
 		switch ($this->getServerVersion()) {
 			case CAS_VERSION_1_0:
-				if (preg_match('/^no\n/',$text_response)) {
+				if (preg_match('/^no\n/', $text_response)) {
 					phpCAS::trace('ST has not been validated');
 					$this->authError('ST not validated',
 						$validate_url,
@@ -1367,7 +1367,7 @@ 
 						FALSE/*$bad_response*/,
 						$text_response);
 				}
-				if (!preg_match('/^yes\n/',$text_response)) {
+				if (!preg_match('/^yes\n/', $text_response)) {
 					phpCAS::trace('ill-formed response');
 					$this->authError('ST not validated',
 						$validate_url,
@@ -1376,12 +1376,12 @@ 
 						$text_response);
 				}
 				// ST has been validated, extract the user name
-				$arr = preg_split('/\n/',$text_response);
+				$arr = preg_split('/\n/', $text_response);
 				$this->setUser(trim($arr[1]));
 				break;
 			case CAS_VERSION_2_0:
 				// read the response of the CAS server into a DOM object
-				if ( !($dom = domxml_open_mem($text_response))) {
+				if (!($dom = domxml_open_mem($text_response))) {
 					phpCAS::trace('domxml_open_mem() failed');
 					$this->authError('ST not validated',
 						$validate_url,
@@ -1390,7 +1390,7 @@ 
 						$text_response);
 				}
 				// read the root node of the XML tree
-				if ( !($tree_response = $dom->document_element()) ) {
+				if (!($tree_response = $dom->document_element())) {
 					phpCAS::trace('document_element() failed');
 					$this->authError('ST not validated',
 						$validate_url,
@@ -1399,7 +1399,7 @@ 
 						$text_response);
 				}
 				// insure that tag name is 'serviceResponse'
-				if ( $tree_response->node_name() != 'serviceResponse' ) {
+				if ($tree_response->node_name() != 'serviceResponse') {
 					phpCAS::trace('bad XML root node (should be `serviceResponse\' instead of `'.$tree_response->node_name().'\'');
 					$this->authError('ST not validated',
 						$validate_url,
@@ -1407,9 +1407,9 @@ 
 						TRUE/*$bad_response*/,
 						$text_response);
 				}
-				if ( sizeof($success_elements = $tree_response->get_elements_by_tagname("authenticationSuccess")) != 0) {
+				if (sizeof($success_elements = $tree_response->get_elements_by_tagname("authenticationSuccess")) != 0) {
 					// authentication succeded, extract the user name
-					if ( sizeof($user_elements = $success_elements[0]->get_elements_by_tagname("user")) == 0) {
+					if (sizeof($user_elements = $success_elements[0]->get_elements_by_tagname("user")) == 0) {
 						phpCAS::trace('<authenticationSuccess> found, but no <user>');
 						$this->authError('ST not validated',
 							$validate_url,
@@ -1421,7 +1421,7 @@ 
 					phpCAS::trace('user = `'.$user);
 					$this->setUser($user);
 					
-				} else if ( sizeof($failure_elements = $tree_response->get_elements_by_tagname("authenticationFailure")) != 0) {
+				} else if (sizeof($failure_elements = $tree_response->get_elements_by_tagname("authenticationFailure")) != 0) {
 					phpCAS::trace('<authenticationFailure> found');
 					// authentication failed, extract the error code and message
 					$this->authError('ST not validated',
@@ -1468,7 +1468,7 @@ 
 	 *
 	 * @private
 	 */
-	function validateSA($validate_url,&$text_response,&$tree_response)
+	function validateSA($validate_url, &$text_response, &$tree_response)
 		{
 		phpCAS::traceBegin();
 		
@@ -1476,7 +1476,7 @@ 
 		$validate_url = $this->getServerSamlValidateURL();
 		
 		// open and read the URL
-		if ( !$this->readURL($validate_url,''/*cookies*/,$headers,$text_response,$err_msg) ) {
+		if (!$this->readURL($validate_url, ''/*cookies*/, $headers, $text_response, $err_msg)) {
 			phpCAS::trace('could not open URL \''.$validate_url.'\' to validate ('.$err_msg.')');
 			$this->authError('SA not validated', $validate_url, TRUE/*$no_response*/);
 		}
@@ -1488,7 +1488,7 @@ 
 			case SAML_VERSION_1_1:
 				
 				// read the response of the CAS server into a DOM object
-				if ( !($dom = domxml_open_mem($text_response))) {
+				if (!($dom = domxml_open_mem($text_response))) {
 					phpCAS::trace('domxml_open_mem() failed');
 					$this->authError('SA not validated',
 						$validate_url,
@@ -1497,7 +1497,7 @@ 
 						$text_response);
 				}
 				// read the root node of the XML tree
-				if ( !($tree_response = $dom->document_element()) ) {
+				if (!($tree_response = $dom->document_element())) {
 					phpCAS::trace('document_element() failed');
 					$this->authError('SA not validated',
 						$validate_url,
@@ -1506,7 +1506,7 @@ 
 						$text_response);
 				}
 				// insure that tag name is 'Envelope'
-				if ( $tree_response->node_name() != 'Envelope' ) {
+				if ($tree_response->node_name() != 'Envelope') {
 					phpCAS::trace('bad XML root node (should be `Envelope\' instead of `'.$tree_response->node_name().'\'');
 					$this->authError('SA not validated',
 						$validate_url,
@@ -1515,7 +1515,7 @@ 
 						$text_response);
 				}
 				// check for the NameIdentifier tag in the SAML response
-				if ( sizeof($success_elements = $tree_response->get_elements_by_tagname("NameIdentifier")) != 0) {
+				if (sizeof($success_elements = $tree_response->get_elements_by_tagname("NameIdentifier")) != 0) {
 					phpCAS::trace('NameIdentifier found');
 					$user = trim($success_elements[0]->get_content());
 					phpCAS::trace('user = `'.$user.'`');
@@ -1553,7 +1553,7 @@ 
 		$result = FALSE;
 		
 		if (isset($_SESSION[SAML_ATTRIBUTES])) {
-			phpCAS::trace("session attrs already set.");  //testbml - do we care?
+			phpCAS::trace("session attrs already set."); //testbml - do we care?
 		}
 		
 		$attr_array = array();
@@ -1565,21 +1565,21 @@ 
 			$nodelist = $xPath->xpath_eval("//saml:Attribute");
 			$attrs = $nodelist->nodeset;
 			phpCAS::trace($text_response);
-			foreach($attrs as $attr){
+			foreach ($attrs as $attr) {
 				$xres = $xPath->xpath_eval("saml:AttributeValue", $attr);
 				$name = $attr->get_attribute("AttributeName");
 				$value_array = array();
-				foreach($xres->nodeset as $node){
+				foreach ($xres->nodeset as $node) {
 					$value_array[] = $node->get_content();
 					
 				}
-				phpCAS::trace("* " . $name . "=" . $value_array);
+				phpCAS::trace("* ".$name."=".$value_array);
 				$attr_array[$name] = $value_array;
 			}
 			$_SESSION[SAML_ATTRIBUTES] = $attr_array;
 			// UGent addition...
-			foreach($attr_array as $attr_key => $attr_value) {
-				if(count($attr_value) > 1) {
+			foreach ($attr_array as $attr_key => $attr_value) {
+				if (count($attr_value) > 1) {
 					$this->_attributes[$attr_key] = $attr_value;
 				}
 				else {
@@ -1741,14 +1741,14 @@ 
 	function getCallbackURL()
 		{
 		// the URL is built when needed only
-		if ( empty($this->_callback_url) ) {
+		if (empty($this->_callback_url)) {
 			$final_uri = '';
 			// remove the ticket if present in the URL
 			$final_uri = 'https://';
 			/* replaced by Julien Marchal - v0.4.6
 			 * $this->uri .= $_SERVER['SERVER_NAME'];
 			 */
-			if(empty($_SERVER['HTTP_X_FORWARDED_SERVER'])){
+			if (empty($_SERVER['HTTP_X_FORWARDED_SERVER'])) {
 				/* replaced by teedog - v0.4.12
 				 * $final_uri .= $_SERVER['SERVER_NAME'];
 				 */
@@ -1760,13 +1760,13 @@ 
 			} else {
 				$final_uri .= $_SERVER['HTTP_X_FORWARDED_SERVER'];
 			}
-			if ( ($this->isHttps() && $_SERVER['SERVER_PORT']!=443)
-					|| (!$this->isHttps() && $_SERVER['SERVER_PORT']!=80) ) {
+			if (($this->isHttps() && $_SERVER['SERVER_PORT'] != 443)
+					|| (!$this->isHttps() && $_SERVER['SERVER_PORT'] != 80)) {
 				$final_uri .= ':';
 				$final_uri .= $_SERVER['SERVER_PORT'];
 			}
 			$request_uri = $_SERVER['REQUEST_URI'];
-			$request_uri = preg_replace('/\?.*$/','',$request_uri);
+			$request_uri = preg_replace('/\?.*$/', '', $request_uri);
 			$final_uri .= $request_uri;
 			$this->setCallbackURL($final_uri);
 		}
@@ -1799,7 +1799,7 @@ 
 		$pgt = $_GET['pgtId'];
 		phpCAS::trace('Storing PGT `'.$pgt.'\' (id=`'.$pgt_iou.'\')');
 		echo '<p>Storing PGT `'.$pgt.'\' (id=`'.$pgt_iou.'\').</p>';
-		$this->storePGT($pgt,$pgt_iou);
+		$this->storePGT($pgt, $pgt_iou);
 		$this->printHTMLFooter();
 		phpCAS::traceExit();
 		exit();
@@ -1834,7 +1834,7 @@ 
 	function initPGTStorage()
 		{
 		// if no SetPGTStorageXxx() has been used, default to file
-		if ( !is_object($this->_pgt_storage) ) {
+		if (!is_object($this->_pgt_storage)) {
 			$this->setPGTStorageFile();
 		}
 		
@@ -1850,12 +1850,12 @@ 
 	 *
 	 * @private
 	 */
-	function storePGT($pgt,$pgt_iou)
+	function storePGT($pgt, $pgt_iou)
 		{
 		// ensure that storage is initialized
 		$this->initPGTStorage();
 		// writes the PGT
-		$this->_pgt_storage->write($pgt,$pgt_iou);
+		$this->_pgt_storage->write($pgt, $pgt_iou);
 		}
 	
 	/**
@@ -1884,16 +1884,16 @@ 
 	 *
 	 * @public
 	 */
-	function setPGTStorageFile($format='',
-		$path='')
+	function setPGTStorageFile($format = '',
+		$path = '')
 		{
 		// check that the storage has not already been set
-		if ( is_object($this->_pgt_storage) ) {
+		if (is_object($this->_pgt_storage)) {
 			phpCAS::error('PGT storage already defined');
 		}
 		
 		// create the storage object
-		$this->_pgt_storage = new PGTStorageFile($this,$format,$path);
+		$this->_pgt_storage = new PGTStorageFile($this, $format, $path);
 		}
 	
 	/**
@@ -1922,15 +1922,15 @@ 
 							 $table)
 		{
 		// check that the storage has not already been set
-		if ( is_object($this->_pgt_storage) ) {
+		if (is_object($this->_pgt_storage)) {
 			phpCAS::error('PGT storage already defined');
 		}
 		
 		// warn the user that he should use file storage...
-		trigger_error('PGT storage into database is an experimental feature, use at your own risk',E_USER_WARNING);
+		trigger_error('PGT storage into database is an experimental feature, use at your own risk', E_USER_WARNING);
 		
 		// create the storage object
-		$this->_pgt_storage = new PGTStorageDB($this,$user,$password,$database_type,$hostname,$port,$database,$table);
+		$this->_pgt_storage = new PGTStorageDB($this, $user, $password, $database_type, $hostname, $port, $database, $table);
 		}
 	
 	// ########################################################################
@@ -1949,11 +1949,11 @@ 
 	 *
 	 * @private
 	 */
-	function validatePGT(&$validate_url,$text_response,$tree_response)
+	function validatePGT(&$validate_url, $text_response, $tree_response)
 		{
 		// here cannot use phpCAS::traceBegin(); alongside domxml-php4-to-php5.php
 		phpCAS::log('start validatePGT()');
-		if ( sizeof($arr = $tree_response->get_elements_by_tagname("proxyGrantingTicket")) == 0) {
+		if (sizeof($arr = $tree_response->get_elements_by_tagname("proxyGrantingTicket")) == 0) {
 			phpCAS::trace('<proxyGrantingTicket> not found');
 			// authentication succeded, but no PGT Iou was transmitted
 			$this->authError('Ticket validated but no PGT Iou transmitted',
@@ -1965,7 +1965,7 @@ 
 			// PGT Iou transmitted, extract it
 			$pgt_iou = trim($arr[0]->get_content());
 			$pgt = $this->loadPGT($pgt_iou);
-			if ( $pgt == FALSE ) {
+			if ($pgt == FALSE) {
 				phpCAS::trace('could not load PGT');
 				$this->authError('PGT Iou was transmitted but PGT could not be retrieved',
 					$validate_url,
@@ -1995,7 +1995,7 @@ 
 	 *
 	 * @private
 	 */
-	function retrievePT($target_service,&$err_code,&$err_msg)
+	function retrievePT($target_service, &$err_code, &$err_msg)
 		{
 		phpCAS::traceBegin();
 		
@@ -2010,7 +2010,7 @@ 
 		$cas_url = $this->getServerProxyURL().'?targetService='.urlencode($target_service).'&pgt='.$this->getPGT();
 		
 		// open and read the URL
-		if ( !$this->readURL($cas_url,''/*cookies*/,$headers,$cas_response,$err_msg) ) {
+		if (!$this->readURL($cas_url, ''/*cookies*/, $headers, $cas_response, $err_msg)) {
 			phpCAS::trace('could not open URL \''.$cas_url.'\' to validate ('.$err_msg.')');
 			$err_code = PHPCAS_SERVICE_PT_NO_SERVER_RESPONSE;
 			$err_msg = 'could not retrieve PT (no response from the CAS server)';
@@ -2020,38 +2020,38 @@ 
 		
 		$bad_response = FALSE;
 		
-		if ( !$bad_response ) {
+		if (!$bad_response) {
 			// read the response of the CAS server into a DOM object
-			if ( !($dom = @domxml_open_mem($cas_response))) {
+			if (!($dom = @domxml_open_mem($cas_response))) {
 				phpCAS::trace('domxml_open_mem() failed');
 				// read failed
 				$bad_response = TRUE;
 			} 
 		}
 		
-		if ( !$bad_response ) {
+		if (!$bad_response) {
 			// read the root node of the XML tree
-			if ( !($root = $dom->document_element()) ) {
+			if (!($root = $dom->document_element())) {
 				phpCAS::trace('document_element() failed');
 				// read failed
 				$bad_response = TRUE;
 			} 
 		}
 		
-		if ( !$bad_response ) {
+		if (!$bad_response) {
 			// insure that tag name is 'serviceResponse'
-			if ( $root->node_name() != 'serviceResponse' ) {
+			if ($root->node_name() != 'serviceResponse') {
 				phpCAS::trace('node_name() failed');
 				// bad root node
 				$bad_response = TRUE;
 			} 
 		}
 		
-		if ( !$bad_response ) {
+		if (!$bad_response) {
 			// look for a proxySuccess tag
-			if ( sizeof($arr = $root->get_elements_by_tagname("proxySuccess")) != 0) {
+			if (sizeof($arr = $root->get_elements_by_tagname("proxySuccess")) != 0) {
 				// authentication succeded, look for a proxyTicket tag
-				if ( sizeof($arr = $root->get_elements_by_tagname("proxyTicket")) != 0) {
+				if (sizeof($arr = $root->get_elements_by_tagname("proxyTicket")) != 0) {
 					$err_code = PHPCAS_SERVICE_OK;
 					$err_msg = '';
 					phpCAS::trace('original PT: '.trim($arr[0]->get_content()));
@@ -2063,7 +2063,7 @@ 
 				}
 			} 
 			// look for a proxyFailure tag
-			else if ( sizeof($arr = $root->get_elements_by_tagname("proxyFailure")) != 0) {
+			else if (sizeof($arr = $root->get_elements_by_tagname("proxyFailure")) != 0) {
 				// authentication failed, extract the error
 				$err_code = PHPCAS_SERVICE_PT_FAILURE;
 				$err_msg = 'PT retrieving failed (code=`'
@@ -2105,7 +2105,7 @@ 
 	 *
 	 * @private
 	 */
-	function readURL($url,$cookies,&$headers,&$body,&$err_msg)
+	function readURL($url, $cookies, &$headers, &$body, &$err_msg)
 		{
 		phpCAS::traceBegin();
 		$headers = '';
@@ -2117,7 +2117,7 @@ 
 		// initialize the CURL session
 		$ch = curl_init($url);
 		
-		if (version_compare(PHP_VERSION,'5.1.3','>=')) {
+		if (version_compare(PHP_VERSION, '5.1.3', '>=')) {
 			//only avaible in php5
 			curl_setopt_array($ch, $this->_curl_options);
 		} else {
@@ -2137,7 +2137,7 @@ 
 			curl_setopt($ch, CURLOPT_CAINFO, $this->_cas_server_ca_cert);
 			curl_setopt($ch, CURLOPT_VERBOSE, '1');
 			phpCAS::trace('CURL: Set all required opts for mutual authentication ------');
-		} else if ($this->_cas_server_cert != '' ) {
+		} else if ($this->_cas_server_cert != '') {
 			curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 1);
 			curl_setopt($ch, CURLOPT_SSLCERT, $this->_cas_server_cert);
 		} else if ($this->_cas_server_ca_cert != '') {
@@ -2154,12 +2154,12 @@ 
 		$this->_curl_headers = array(); // empty the headers array
 		curl_setopt($ch, CURLOPT_HEADERFUNCTION, array($this, '_curl_read_headers'));
 		// add cookies headers
-		if ( is_array($cookies) ) {
-			curl_setopt($ch,CURLOPT_COOKIE,implode(';',$cookies));
+		if (is_array($cookies)) {
+			curl_setopt($ch, CURLOPT_COOKIE, implode(';', $cookies));
 		}
 		// add extra stuff if SAML
 		if ($this->hasSA()) {
-			$more_headers = array ("soapaction: http://www.oasis-open.org/committees/security",
+			$more_headers = array("soapaction: http://www.oasis-open.org/committees/security",
 				"cache-control: no-cache",
 				"pragma: no-cache",
 				"accept: text/xml",
@@ -2173,18 +2173,18 @@ 
 			curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
 		}
 		// perform the query
-		$buf = curl_exec ($ch);
+		$buf = curl_exec($ch);
 		//phpCAS::trace('CURL: Call completed. Response body is: \''.$buf.'\'');
-		if ( $buf === FALSE ) {
+		if ($buf === FALSE) {
 			phpCAS::trace('curl_exec() failed');
 			$err_msg = 'CURL error #'.curl_errno($ch).': '.curl_error($ch);
 			//phpCAS::trace('curl error: '.$err_msg);
 			// close the CURL session
-			curl_close ($ch);
+			curl_close($ch);
 			$res = FALSE;
 		} else {
 			// close the CURL session
-			curl_close ($ch);
+			curl_close($ch);
 			
 			$headers = $this->_curl_headers;
 			$body = $buf;
@@ -2209,7 +2209,7 @@ 
 		$sa = $this->getSA();
 		//phpCAS::trace("SA: ".$sa);
 		
-		$body=SAML_SOAP_ENV.SAML_SOAP_BODY.SAMLP_REQUEST.SAML_ASSERTION_ARTIFACT.$sa.SAML_ASSERTION_ARTIFACT_CLOSE.SAMLP_REQUEST_CLOSE.SAML_SOAP_BODY_CLOSE.SAML_SOAP_ENV_CLOSE;
+		$body = SAML_SOAP_ENV.SAML_SOAP_BODY.SAMLP_REQUEST.SAML_ASSERTION_ARTIFACT.$sa.SAML_ASSERTION_ARTIFACT_CLOSE.SAMLP_REQUEST_CLOSE.SAML_SOAP_BODY_CLOSE.SAML_SOAP_ENV_CLOSE;
 		
 		phpCAS::traceEnd($body);
 		return ($body);
@@ -2240,36 +2240,36 @@ 
 	 *
 	 * @public
 	 */
-	function serviceWeb($url,&$err_code,&$output)
+	function serviceWeb($url, &$err_code, &$output)
 		{
 		phpCAS::traceBegin();
 		// at first retrieve a PT
-		$pt = $this->retrievePT($url,$err_code,$output);
+		$pt = $this->retrievePT($url, $err_code, $output);
 		
 		$res = TRUE;
 		
 		// test if PT was retrieved correctly
-		if ( !$pt ) {
+		if (!$pt) {
 			// note: $err_code and $err_msg are filled by CASClient::retrievePT()
 			phpCAS::trace('PT was not retrieved correctly');
 			$res = FALSE;
 		} else {
 			// add cookies if necessary
-			if ( is_array($_SESSION['phpCAS']['services'][$url]['cookies']) ) {
-				foreach ( $_SESSION['phpCAS']['services'][$url]['cookies'] as $name => $val ) { 
+			if (is_array($_SESSION['phpCAS']['services'][$url]['cookies'])) {
+				foreach ($_SESSION['phpCAS']['services'][$url]['cookies'] as $name => $val) { 
 					$cookies[] = $name.'='.$val;
 				}
 			}
 			
 			// build the URL including the PT
-			if ( strstr($url,'?') === FALSE ) {
+			if (strstr($url, '?') === FALSE) {
 				$service_url = $url.'?ticket='.$pt;
 			} else {
 				$service_url = $url.'&ticket='.$pt;
 			}
 			
 			phpCAS::trace('reading URL`'.$service_url.'\'');
-			if ( !$this->readURL($service_url,$cookies,$headers,$output,$err_msg) ) {
+			if (!$this->readURL($service_url, $cookies, $headers, $output, $err_msg)) {
 				phpCAS::trace('could not read URL`'.$service_url.'\'');
 				$err_code = PHPCAS_SERVICE_NOT_AVAILABLE;
 				// give an error message
@@ -2280,15 +2280,15 @@ 
 			} else {
 				// URL has been fetched, extract the cookies
 				phpCAS::trace('URL`'.$service_url.'\' has been read, storing cookies:');
-				foreach ( $headers as $header ) {
+				foreach ($headers as $header) {
 					// test if the header is a cookie
-					if ( preg_match('/^Set-Cookie:/',$header) ) {
+					if (preg_match('/^Set-Cookie:/', $header)) {
 						// the header is a cookie, remove the beginning
-						$header_val = preg_replace('/^Set-Cookie: */','',$header);
+						$header_val = preg_replace('/^Set-Cookie: */', '', $header);
 						// extract interesting information
-						$name_val = strtok($header_val,'; ');
+						$name_val = strtok($header_val, '; ');
 						// extract the name and the value of the cookie
-						$cookie_name = strtok($name_val,'=');
+						$cookie_name = strtok($name_val, '=');
 						$cookie_val = strtok('=');
 						// store the cookie 
 						$_SESSION['phpCAS']['services'][$url]['cookies'][$cookie_name] = $cookie_val;
@@ -2321,28 +2321,28 @@ 
 	 *
 	 * @public
 	 */
-	function serviceMail($url,$service,$flags,&$err_code,&$err_msg,&$pt)
+	function serviceMail($url, $service, $flags, &$err_code, &$err_msg, &$pt)
 		{
 		phpCAS::traceBegin();
 		// at first retrieve a PT
-		$pt = $this->retrievePT($service,$err_code,$output);
+		$pt = $this->retrievePT($service, $err_code, $output);
 		
 		$stream = FALSE;
 		
 		// test if PT was retrieved correctly
-		if ( !$pt ) {
+		if (!$pt) {
 			// note: $err_code and $err_msg are filled by CASClient::retrievePT()
 			phpCAS::trace('PT was not retrieved correctly');
 		} else {
 			phpCAS::trace('opening IMAP URL `'.$url.'\'...');
-			$stream = @imap_open($url,$this->getUser(),$pt,$flags);
-			if ( !$stream ) {
+			$stream = @imap_open($url, $this->getUser(), $pt, $flags);
+			if (!$stream) {
 				phpCAS::trace('could not open URL');
 				$err_code = PHPCAS_SERVICE_NOT_AVAILABLE;
 				// give an error message
 				$err_msg = sprintf($this->getString(CAS_STR_SERVICE_UNAVAILABLE),
 					$service_url,
-					var_export(imap_errors(),TRUE));
+					var_export(imap_errors(), TRUE));
 				$pt = FALSE;
 				$stream = FALSE;
 			} else {
@@ -2446,19 +2446,19 @@ 
 	 *
 	 * @private
 	 */
-	function validatePT(&$validate_url,&$text_response,&$tree_response)
+	function validatePT(&$validate_url, &$text_response, &$tree_response)
 		{
 		phpCAS::traceBegin();
 		// build the URL to validate the ticket
 		$validate_url = $this->getServerProxyValidateURL().'&ticket='.$this->getPT();
 		
-		if ( $this->isProxy() ) {
+		if ($this->isProxy()) {
 			// pass the callback url for CAS proxies
 			$validate_url .= '&pgtUrl='.$this->getCallbackURL();
 		}
 		
 		// open and read the URL
-		if ( !$this->readURL($validate_url,''/*cookies*/,$headers,$text_response,$err_msg) ) {
+		if (!$this->readURL($validate_url, ''/*cookies*/, $headers, $text_response, $err_msg)) {
 			phpCAS::trace('could not open URL \''.$validate_url.'\' to validate ('.$err_msg.')');
 			$this->authError('PT not validated',
 				$validate_url,
@@ -2466,7 +2466,7 @@ 
 		}
 		
 		// read the response of the CAS server into a DOM object
-		if ( !($dom = domxml_open_mem($text_response))) {
+		if (!($dom = domxml_open_mem($text_response))) {
 			// read failed
 			$this->authError('PT not validated',
 				$validate_url,
@@ -2475,7 +2475,7 @@ 
 				$text_response);
 		}
 		// read the root node of the XML tree
-		if ( !($tree_response = $dom->document_element()) ) {
+		if (!($tree_response = $dom->document_element())) {
 			// read failed
 			$this->authError('PT not validated',
 				$validate_url,
@@ -2484,7 +2484,7 @@ 
 				$text_response);
 		}
 		// insure that tag name is 'serviceResponse'
-		if ( $tree_response->node_name() != 'serviceResponse' ) {
+		if ($tree_response->node_name() != 'serviceResponse') {
 			// bad root node
 			$this->authError('PT not validated',
 				$validate_url,
@@ -2492,9 +2492,9 @@ 
 				TRUE/*$bad_response*/,
 				$text_response);
 		}
-		if ( sizeof($arr = $tree_response->get_elements_by_tagname("authenticationSuccess")) != 0) {
+		if (sizeof($arr = $tree_response->get_elements_by_tagname("authenticationSuccess")) != 0) {
 			// authentication succeded, extract the user name
-			if ( sizeof($arr = $tree_response->get_elements_by_tagname("user")) == 0) {
+			if (sizeof($arr = $tree_response->get_elements_by_tagname("user")) == 0) {
 				// no user specified => error
 				$this->authError('PT not validated',
 					$validate_url,
@@ -2504,7 +2504,7 @@ 
 			}
 			$this->setUser(trim($arr[0]->get_content()));
 			
-		} else if ( sizeof($arr = $tree_response->get_elements_by_tagname("authenticationFailure")) != 0) {
+		} else if (sizeof($arr = $tree_response->get_elements_by_tagname("authenticationFailure")) != 0) {
 			// authentication succeded, extract the error code and message
 			$this->authError('PT not validated',
 				$validate_url,
@@ -2564,7 +2564,7 @@ 
 		{
 		phpCAS::traceBegin();
 		// the URL is built when needed only
-		if ( empty($this->_url) ) {
+		if (empty($this->_url)) {
 			$final_uri = '';
 			// remove the ticket if present in the URL
 			$final_uri = ($this->isHttps()) ? 'https' : 'http';
@@ -2572,7 +2572,7 @@ 
 			/* replaced by Julien Marchal - v0.4.6
 			 * $this->_url .= $_SERVER['SERVER_NAME'];
 			 */
-			if(empty($_SERVER['HTTP_X_FORWARDED_SERVER'])){
+			if (empty($_SERVER['HTTP_X_FORWARDED_SERVER'])) {
 				/* replaced by teedog - v0.4.12
 				 * $this->_url .= $_SERVER['SERVER_NAME'];
 				 */
@@ -2586,8 +2586,8 @@ 
 			}
 			$final_uri .= $server_name;
 			if (!strpos($server_name, ':')) {
-				if ( ($this->isHttps() && $_SERVER['SERVER_PORT']!=443)
-						|| (!$this->isHttps() && $_SERVER['SERVER_PORT']!=80) ) {
+				if (($this->isHttps() && $_SERVER['SERVER_PORT'] != 443)
+						|| (!$this->isHttps() && $_SERVER['SERVER_PORT'] != 80)) {
 					$final_uri .= ':';
 					$final_uri .= $_SERVER['SERVER_PORT'];
 				}
@@ -2598,11 +2598,11 @@ 
 			
 			if (isset($request_uri[1]) && $request_uri[1])
 			{
-				$query_string	= $this->removeParameterFromQueryString('ticket', $request_uri[1]);
+				$query_string = $this->removeParameterFromQueryString('ticket', $request_uri[1]);
 				
 				// If the query string still has anything left, append it to the final URI
 				if ($query_string !== '')
-					$final_uri	.= "?$query_string";
+					$final_uri .= "?$query_string";
 				
 			}
 			
@@ -2626,7 +2626,7 @@ 
 	 */
 	function removeParameterFromQueryString($parameterName, $queryString)
 	{
-		$parameterName	= preg_quote($parameterName);
+		$parameterName = preg_quote($parameterName);
 		return preg_replace("/&$parameterName(=[^&]*)?|^$parameterName(=[^&]*)?&?/", '', $queryString);
 	}
 
@@ -2661,18 +2661,18 @@ 
 	 *
 	 * @private
 	 */
-	function authError($failure,$cas_url,$no_response,$bad_response='',$cas_response='',$err_code='',$err_msg='')
+	function authError($failure, $cas_url, $no_response, $bad_response = '', $cas_response = '', $err_code = '', $err_msg = '')
 		{
 		phpCAS::traceBegin();
 		
 		$this->printHTMLHeader($this->getString(CAS_STR_AUTHENTICATION_FAILED));
-		printf($this->getString(CAS_STR_YOU_WERE_NOT_AUTHENTICATED),htmlentities($this->getURL()),$_SERVER['SERVER_ADMIN']);
+		printf($this->getString(CAS_STR_YOU_WERE_NOT_AUTHENTICATED), htmlentities($this->getURL()), $_SERVER['SERVER_ADMIN']);
 		phpCAS::trace('CAS URL: '.$cas_url);
 		phpCAS::trace('Authentication failure: '.$failure);
-		if ( $no_response ) {
+		if ($no_response) {
 			phpCAS::trace('Reason: no response from the CAS server');
 		} else {
-			if ( $bad_response ) {
+			if ($bad_response) {
 				phpCAS::trace('Reason: bad response from the CAS server');
 			} else {
 				switch ($this->getServerVersion()) {
@@ -2680,7 +2680,7 @@ 
 						phpCAS::trace('Reason: CAS error');
 						break;
 					case CAS_VERSION_2_0:
-						if ( empty($err_code) )
+						if (empty($err_code))
 							phpCAS::trace('Reason: no CAS error');
 						else
 							phpCAS::trace('Reason: ['.$err_code.'] CAS error: '.$err_msg);