chamilo / chamilo-lms

main/inc/lib/nusoap/class.soap_transport_http.php

Spacing +136 added lines, -136 removed lines

@@ -26,18 +26,18 @@ 
 	var $incoming_cookies = array();

 	var $outgoing_payload = '';

 	var $incoming_payload = '';

-	var $response_status_line;	// HTTP response status line

+	var $response_status_line; // HTTP response status line

 	var $useSOAPAction = true;

 	var $persistentConnection = false;

-	var $ch = false;	// cURL handle

-	var $ch_options = array();	// cURL custom options

-	var $use_curl = false;		// force cURL use

-	var $proxy = null;			// proxy information (associative array)

+	var $ch = false; // cURL handle

+	var $ch_options = array(); // cURL custom options

+	var $use_curl = false; // force cURL use

+	var $proxy = null; // proxy information (associative array)

 	var $username = '';

 	var $password = '';

 	var $authtype = '';

 	var $digestRequest = array();

-	var $certRequest = array();	// keys must be cainfofile (optional), sslcertfile, sslkeyfile, passphrase, certpassword (optional), verifypeer (optional), verifyhost (optional)

+	var $certRequest = array(); // keys must be cainfofile (optional), sslcertfile, sslkeyfile, passphrase, certpassword (optional), verifypeer (optional), verifyhost (optional)

 								// cainfofile: certificate authority file, e.g. '$pathToPemFiles/rootca.pem'

 								// sslcertfile: SSL certificate file, e.g. '$pathToPemFiles/mycert.pem'

 								// sslkeyfile: SSL key file, e.g. '$pathToPemFiles/mykey.pem'

@@ -54,7 +54,7 @@ 
 	* @param boolean $use_curl Whether to try to force cURL use

 	* @access public

 	*/

-	function __construct($url, $curl_options = NULL, $use_curl = false){

+	function __construct($url, $curl_options = NULL, $use_curl = false) {

 		parent::__construct();

 		$this->debug("ctor url=$url use_curl=$use_curl curl_options:");

 		$this->appendDebug($this->varDump($curl_options));

@@ -63,7 +63,7 @@ 
 			$this->ch_options = $curl_options;

 		}

 		$this->use_curl = $use_curl;

-		preg_match('/\$Revisio' . 'n: ([^ ]+)/', $this->revision, $rev);

+		preg_match('/\$Revisio'.'n: ([^ ]+)/', $this->revision, $rev);

 		$this->setHeader('User-Agent', $this->title.'/'.$this->version.' ('.$rev[1].')');

 	}

 

@@ -115,19 +115,19 @@ 
 		$this->url = $url;

 

 		$u = parse_url($url);

-		foreach($u as $k => $v){

+		foreach ($u as $k => $v) {

 			$this->debug("parsed URL $k = $v");

 			$this->$k = $v;

 		}

 

 		// add any GET params to path

-		if(isset($u['query']) && $u['query'] != ''){

-            $this->path .= '?' . $u['query'];

+		if (isset($u['query']) && $u['query'] != '') {

+            $this->path .= '?'.$u['query'];

 		}

 

 		// set default port

-		if(!isset($u['port'])){

-			if($u['scheme'] == 'https'){

+		if (!isset($u['port'])) {

+			if ($u['scheme'] == 'https') {

 				$this->port = 443;

 			} else {

 				$this->port = 80;

@@ -171,7 +171,7 @@ 
 	* @return	boolean true if connected, false if not

 	* @access   private

 	*/

-	function connect($connection_timeout=0,$response_timeout=30){

+	function connect($connection_timeout = 0, $response_timeout = 30) {

 	  	// For PHP 4.3 with OpenSSL, change https scheme to ssl, then treat like

 	  	// "regular" socket.

 	  	// TODO: disabled for now because OpenSSL must be *compiled* in (not just

@@ -195,7 +195,7 @@ 
 		}

 

 		// use persistent connection

-		if($this->persistentConnection && isset($this->fp) && is_resource($this->fp)){

+		if ($this->persistentConnection && isset($this->fp) && is_resource($this->fp)) {

 			if (!feof($this->fp)) {

 				$this->debug('Re-use persistent connection');

 				return true;

@@ -206,20 +206,20 @@ 
 

 		// munge host if using OpenSSL

 		if ($this->scheme == 'ssl') {

-			$host = 'ssl://' . $host;

+			$host = 'ssl://'.$host;

 		}

-		$this->debug('calling fsockopen with host ' . $host . ' connection_timeout ' . $connection_timeout);

+		$this->debug('calling fsockopen with host '.$host.' connection_timeout '.$connection_timeout);

 

 		// open socket

-		if($connection_timeout > 0){

-			$this->fp = @fsockopen( $host, $this->port, $this->errno, $this->error_str, $connection_timeout);

+		if ($connection_timeout > 0) {

+			$this->fp = @fsockopen($host, $this->port, $this->errno, $this->error_str, $connection_timeout);

 		} else {

-			$this->fp = @fsockopen( $host, $this->port, $this->errno, $this->error_str);

+			$this->fp = @fsockopen($host, $this->port, $this->errno, $this->error_str);

 		}

 

 		// test pointer

-		if(!$this->fp) {

-			$msg = 'Couldn\'t open socket connection to server ' . $this->url;

+		if (!$this->fp) {

+			$msg = 'Couldn\'t open socket connection to server '.$this->url;

 			if ($this->errno) {

 				$msg .= ', Error ('.$this->errno.'): '.$this->error_str;

 			} else {

@@ -231,8 +231,8 @@ 
 		}

 

 		// set response timeout

-		$this->debug('set response timeout to ' . $response_timeout);

-		socket_set_timeout( $this->fp, $response_timeout);

+		$this->debug('set response timeout to '.$response_timeout);

+		socket_set_timeout($this->fp, $response_timeout);

 

 		$this->debug('socket connected');

 		return true;

@@ -391,8 +391,8 @@ 
 		$this->debug('cURL connection set up');

 		return true;

 	  } else {

-		$this->setError('Unknown scheme ' . $this->scheme);

-		$this->debug('Unknown scheme ' . $this->scheme);

+		$this->setError('Unknown scheme '.$this->scheme);

+		$this->debug('Unknown scheme '.$this->scheme);

 		return false;

 	  }

 	}

@@ -407,7 +407,7 @@ 
 	* @return	string data

 	* @access   public

 	*/

-	function send($data, $timeout=0, $response_timeout=30, $cookies=NULL) {

+	function send($data, $timeout = 0, $response_timeout = 30, $cookies = NULL) {

 

 		$this->debug('entered send() with data of length: '.strlen($data));

 

@@ -417,12 +417,12 @@ 
 			$this->tryagain = false;

 			if ($tries++ < 2) {

 				// make connnection

-				if (!$this->connect($timeout, $response_timeout)){

+				if (!$this->connect($timeout, $response_timeout)) {

 					return false;

 				}

 

 				// send request

-				if (!$this->sendRequest($data, $cookies)){

+				if (!$this->sendRequest($data, $cookies)) {

 					return false;

 				}

 

@@ -448,7 +448,7 @@ 
 	* @access   public

 	* @deprecated

 	*/

-	function sendHTTPS($data, $timeout=0, $response_timeout=30, $cookies) {

+	function sendHTTPS($data, $timeout = 0, $response_timeout = 30, $cookies) {

 		return $this->send($data, $timeout, $response_timeout, $cookies);

 	}

 

@@ -469,7 +469,7 @@ 
 		$this->appendDebug($this->varDump($certRequest));

 		// cf. RFC 2617

 		if ($authtype == 'basic') {

-			$this->setHeader('Authorization', 'Basic '.base64_encode(str_replace(':','',$username).':'.$password));

+			$this->setHeader('Authorization', 'Basic '.base64_encode(str_replace(':', '', $username).':'.$password));

 		} elseif ($authtype == 'digest') {

 			if (isset($digestRequest['nonce'])) {

 				$digestRequest['nc'] = isset($digestRequest['nc']) ? $digestRequest['nc']++ : 1;

@@ -477,16 +477,16 @@ 
 				// calculate the Digest hashes (calculate code based on digest implementation found at: http://www.rassoc.com/gregr/weblog/stories/2002/07/09/webServicesSecurityHttpDigestAuthenticationWithoutActiveDirectory.html)

 

 				// A1 = unq(username-value) ":" unq(realm-value) ":" passwd

-				$A1 = $username. ':' . (isset($digestRequest['realm']) ? $digestRequest['realm'] : '') . ':' . $password;

+				$A1 = $username.':'.(isset($digestRequest['realm']) ? $digestRequest['realm'] : '').':'.$password;

 

 				// H(A1) = MD5(A1)

 				$HA1 = md5($A1);

 

 				// A2 = Method ":" digest-uri-value

-				$A2 = $this->request_method . ':' . $this->digest_uri;

+				$A2 = $this->request_method.':'.$this->digest_uri;

 

 				// H(A2)

-				$HA2 =  md5($A2);

+				$HA2 = md5($A2);

 

 				// KD(secret, data) = H(concat(secret, ":", data))

 				// if qop == auth:

@@ -503,19 +503,19 @@ 
 				$nonce = isset($digestRequest['nonce']) ? $digestRequest['nonce'] : '';

 				$cnonce = $nonce;

 				if ($digestRequest['qop'] != '') {

-					$unhashedDigest = $HA1 . ':' . $nonce . ':' . sprintf("%08d", $digestRequest['nc']) . ':' . $cnonce . ':' . $digestRequest['qop'] . ':' . $HA2;

+					$unhashedDigest = $HA1.':'.$nonce.':'.sprintf("%08d", $digestRequest['nc']).':'.$cnonce.':'.$digestRequest['qop'].':'.$HA2;

 				} else {

-					$unhashedDigest = $HA1 . ':' . $nonce . ':' . $HA2;

+					$unhashedDigest = $HA1.':'.$nonce.':'.$HA2;

 				}

 

 				$hashedDigest = md5($unhashedDigest);

 

 				$opaque = '';

 				if (isset($digestRequest['opaque'])) {

-					$opaque = ', opaque="' . $digestRequest['opaque'] . '"';

+					$opaque = ', opaque="'.$digestRequest['opaque'].'"';

 				}

 

-				$this->setHeader('Authorization', 'Digest username="' . $username . '", realm="' . $digestRequest['realm'] . '", nonce="' . $nonce . '", uri="' . $this->digest_uri . $opaque . '", cnonce="' . $cnonce . '", nc=' . sprintf("%08x", $digestRequest['nc']) . ', qop="' . $digestRequest['qop'] . '", response="' . $hashedDigest . '"');

+				$this->setHeader('Authorization', 'Digest username="'.$username.'", realm="'.$digestRequest['realm'].'", nonce="'.$nonce.'", uri="'.$this->digest_uri.$opaque.'", cnonce="'.$cnonce.'", nc='.sprintf("%08x", $digestRequest['nc']).', qop="'.$digestRequest['qop'].'", response="'.$hashedDigest.'"');

 			}

 		} elseif ($authtype == 'certificate') {

 			$this->certRequest = $certRequest;

@@ -537,7 +537,7 @@ 
 	* @access   public

 	*/

 	function setSOAPAction($soapaction) {

-		$this->setHeader('SOAPAction', '"' . $soapaction . '"');

+		$this->setHeader('SOAPAction', '"'.$soapaction.'"');

 	}

 

 	/**

@@ -546,7 +546,7 @@ 
 	* @param    string $enc encoding style. supported values: gzip, deflate, or both

 	* @access   public

 	*/

-	function setEncoding($enc='gzip, deflate') {

+	function setEncoding($enc = 'gzip, deflate') {

 		if (function_exists('gzdeflate')) {

 			$this->protocol_version = '1.1';

 			$this->setHeader('Accept-Encoding', $enc);

@@ -599,7 +599,7 @@ 
 	 * @access	private

 	 */

 	function isSkippableCurlHeader(&$data) {

-		$skipHeaders = array(	'HTTP/1.1 100',

+		$skipHeaders = array('HTTP/1.1 100',

 								'HTTP/1.0 301',

 								'HTTP/1.1 301',

 								'HTTP/1.0 302',

@@ -625,7 +625,7 @@ 
 	* @access   public

 	* @deprecated

 	*/

-	function decodeChunked($buffer, $lb){

+	function decodeChunked($buffer, $lb) {

 		// length := 0

 		$length = 0;

 		$new = '';

@@ -637,17 +637,17 @@ 
 			$this->debug('no linebreak found in decodeChunked');

 			return $new;

 		}

-		$temp = substr($buffer,0,$chunkend);

-		$chunk_size = hexdec( trim($temp) );

+		$temp = substr($buffer, 0, $chunkend);

+		$chunk_size = hexdec(trim($temp));

 		$chunkstart = $chunkend + strlen($lb);

 		// while (chunk-size > 0) {

 		while ($chunk_size > 0) {

 			$this->debug("chunkstart: $chunkstart chunk_size: $chunk_size");

-			$chunkend = strpos( $buffer, $lb, $chunkstart + $chunk_size);

+			$chunkend = strpos($buffer, $lb, $chunkstart + $chunk_size);

 

 			// Just in case we got a broken connection

 		  	if ($chunkend == FALSE) {

-		  	    $chunk = substr($buffer,$chunkstart);

+		  	    $chunk = substr($buffer, $chunkstart);

 				// append chunk-data to entity-body

 		    	$new .= $chunk;

 		  	    $length += strlen($chunk);

@@ -655,7 +655,7 @@ 
 			}

 

 		  	// read chunk-data and CRLF

-		  	$chunk = substr($buffer,$chunkstart,$chunkend-$chunkstart);

+		  	$chunk = substr($buffer, $chunkstart, $chunkend - $chunkstart);

 		  	// append chunk-data to entity-body

 		  	$new .= $chunk;

 		  	// length := length + chunk-size

@@ -667,8 +667,8 @@ 
 			if ($chunkend == FALSE) {

 				break; //Just in case we got a broken connection

 			}

-			$temp = substr($buffer,$chunkstart,$chunkend-$chunkstart);

-			$chunk_size = hexdec( trim($temp) );

+			$temp = substr($buffer, $chunkstart, $chunkend - $chunkstart);

+			$chunk_size = hexdec(trim($temp));

 			$chunkstart = $chunkend;

 		}

 		return $new;

@@ -703,7 +703,7 @@ 
 		$this->outgoing_payload = "$req\r\n";

 

 		// loop thru headers, serializing

-		foreach($this->outgoing_headers as $k => $v){

+		foreach ($this->outgoing_headers as $k => $v) {

 			$hdr = $k.': '.$v;

 			$this->debug("HTTP header: $hdr");

 			$this->outgoing_payload .= "$hdr\r\n";

@@ -740,12 +740,12 @@ 
 

 	  if ($this->io_method() == 'socket') {

 		// send payload

-		if(!fputs($this->fp, $this->outgoing_payload, strlen($this->outgoing_payload))) {

+		if (!fputs($this->fp, $this->outgoing_payload, strlen($this->outgoing_payload))) {

 			$this->setError('couldn\'t write message data to socket');

 			$this->debug('couldn\'t write message data to socket');

 			return false;

 		}

-		$this->debug('wrote data to socket, length = ' . strlen($this->outgoing_payload));

+		$this->debug('wrote data to socket, length = '.strlen($this->outgoing_payload));

 		return true;

 	  } else if ($this->io_method() == 'curl') {

 		// set payload

@@ -754,7 +754,7 @@ 
 		// some servers refuse to work with (so we no longer use this method!)

 		//$this->setCurlOption(CURLOPT_CUSTOMREQUEST, $this->outgoing_payload);

 		$curl_headers = array();

-		foreach($this->outgoing_headers as $k => $v){

+		foreach ($this->outgoing_headers as $k => $v) {

 			if ($k == 'Connection' || $k == 'Content-Length' || $k == 'Host' || $k == 'Authorization' || $k == 'Proxy-Authorization') {

 				$this->debug("Skip cURL header $k: $v");

 			} else {

@@ -762,7 +762,7 @@ 
 			}

 		}

 		if ($cookie_str != '') {

-			$curl_headers[] = 'Cookie: ' . $cookie_str;

+			$curl_headers[] = 'Cookie: '.$cookie_str;

 		}

 		$this->setCurlOption(CURLOPT_HTTPHEADER, $curl_headers);

 		$this->debug('set cURL HTTP headers');

@@ -788,62 +788,62 @@ 
 	* @return	string the response (also sets member variables like incoming_payload)

 	* @access   private

 	*/

-	function getResponse(){

+	function getResponse() {

 		$this->incoming_payload = '';

 

 	  if ($this->io_method() == 'socket') {

 	    // loop until headers have been retrieved

 	    $data = '';

-	    while (!isset($lb)){

+	    while (!isset($lb)) {

 

 			// We might EOF during header read.

-			if(feof($this->fp)) {

+			if (feof($this->fp)) {

 				$this->incoming_payload = $data;

-				$this->debug('found no headers before EOF after length ' . strlen($data));

-				$this->debug("received before EOF:\n" . $data);

+				$this->debug('found no headers before EOF after length '.strlen($data));

+				$this->debug("received before EOF:\n".$data);

 				$this->setError('server failed to send headers');

 				return false;

 			}

 

 			$tmp = fgets($this->fp, 256);

 			$tmplen = strlen($tmp);

-			$this->debug("read line of $tmplen bytes: " . trim($tmp));

+			$this->debug("read line of $tmplen bytes: ".trim($tmp));

 

 			if ($tmplen == 0) {

 				$this->incoming_payload = $data;

-				$this->debug('socket read of headers timed out after length ' . strlen($data));

-				$this->debug("read before timeout: " . $data);

+				$this->debug('socket read of headers timed out after length '.strlen($data));

+				$this->debug("read before timeout: ".$data);

 				$this->setError('socket read of headers timed out');

 				return false;

 			}

 

 			$data .= $tmp;

-			$pos = strpos($data,"\r\n\r\n");

-			if($pos > 1){

+			$pos = strpos($data, "\r\n\r\n");

+			if ($pos > 1) {

 				$lb = "\r\n";

 			} else {

-				$pos = strpos($data,"\n\n");

-				if($pos > 1){

+				$pos = strpos($data, "\n\n");

+				if ($pos > 1) {

 					$lb = "\n";

 				}

 			}

 			// remove 100 headers

-			if (isset($lb) && preg_match('/^HTTP\/1.1 100/',$data)) {

+			if (isset($lb) && preg_match('/^HTTP\/1.1 100/', $data)) {

 				unset($lb);

 				$data = '';

 			}//

 		}

 		// store header data

 		$this->incoming_payload .= $data;

-		$this->debug('found end of headers after length ' . strlen($data));

+		$this->debug('found end of headers after length '.strlen($data));

 		// process headers

-		$header_data = trim(substr($data,0,$pos));

-		$header_array = explode($lb,$header_data);

+		$header_data = trim(substr($data, 0, $pos));

+		$header_array = explode($lb, $header_data);

 		$this->incoming_headers = array();

 		$this->incoming_cookies = array();

-		foreach($header_array as $header_line){

-			$arr = explode(':',$header_line, 2);

-			if(count($arr) > 1){

+		foreach ($header_array as $header_line) {

+			$arr = explode(':', $header_line, 2);

+			if (count($arr) > 1) {

 				$header_name = strtolower(trim($arr[0]));

 				$this->incoming_headers[$header_name] = trim($arr[1]);

 				if ($header_name == 'set-cookie') {

@@ -851,20 +851,20 @@ 
 					$cookie = $this->parseCookie(trim($arr[1]));

 					if ($cookie) {

 						$this->incoming_cookies[] = $cookie;

-						$this->debug('found cookie: ' . $cookie['name'] . ' = ' . $cookie['value']);

+						$this->debug('found cookie: '.$cookie['name'].' = '.$cookie['value']);

 					} else {

-						$this->debug('did not find cookie in ' . trim($arr[1]));

+						$this->debug('did not find cookie in '.trim($arr[1]));

 					}

     			}

 			} else if (isset($header_name)) {

 				// append continuation line to previous header

-				$this->incoming_headers[$header_name] .= $lb . ' ' . $header_line;

+				$this->incoming_headers[$header_name] .= $lb.' '.$header_line;

 			}

 		}

 

 		// loop until msg has been received

 		if (isset($this->incoming_headers['transfer-encoding']) && strtolower($this->incoming_headers['transfer-encoding']) == 'chunked') {

-			$content_length =  2147483647;	// ignore any content-length header

+			$content_length = 2147483647; // ignore any content-length header

 			$chunked = true;

 			$this->debug("want to read chunked content");

 		} elseif (isset($this->incoming_headers['content-length'])) {

@@ -872,7 +872,7 @@ 
 			$chunked = false;

 			$this->debug("want to read content of length $content_length");

 		} else {

-			$content_length =  2147483647;

+			$content_length = 2147483647;

 			$chunked = false;

 			$this->debug("want to read content to EOF");

 		}

@@ -884,8 +884,8 @@ 
 				$this->debug("read chunk line of $tmplen bytes");

 				if ($tmplen == 0) {

 					$this->incoming_payload = $data;

-					$this->debug('socket read of chunk length timed out after length ' . strlen($data));

-					$this->debug("read before timeout:\n" . $data);

+					$this->debug('socket read of chunk length timed out after length '.strlen($data));

+					$this->debug("read before timeout:\n".$data);

 					$this->setError('socket read of chunk length timed out');

 					return false;

 				}

@@ -900,8 +900,8 @@ 
 				$this->debug("read buffer of $tmplen bytes");

 				if (($tmplen == 0) && (!feof($this->fp))) {

 					$this->incoming_payload = $data;

-					$this->debug('socket read of body timed out after length ' . strlen($data));

-					$this->debug("read before timeout:\n" . $data);

+					$this->debug('socket read of body timed out after length '.strlen($data));

+					$this->debug("read before timeout:\n".$data);

 					$this->setError('socket read of body timed out');

 					return false;

 				}

@@ -914,8 +914,8 @@ 
 				$this->debug("read chunk terminator of $tmplen bytes");

 				if ($tmplen == 0) {

 					$this->incoming_payload = $data;

-					$this->debug('socket read of chunk terminator timed out after length ' . strlen($data));

-					$this->debug("read before timeout:\n" . $data);

+					$this->debug('socket read of chunk terminator timed out after length '.strlen($data));

+					$this->debug("read before timeout:\n".$data);

 					$this->setError('socket read of chunk terminator timed out');

 					return false;

 				}

@@ -924,21 +924,21 @@ 
 		if (feof($this->fp)) {

 			$this->debug('read to EOF');

 		}

-		$this->debug('read body of length ' . strlen($data));

+		$this->debug('read body of length '.strlen($data));

 		$this->incoming_payload .= $data;

 		$this->debug('received a total of '.strlen($this->incoming_payload).' bytes of data from server');

 

 		// close filepointer

-		if(

+		if (

 			(isset($this->incoming_headers['connection']) && strtolower($this->incoming_headers['connection']) == 'close') ||

-			(! $this->persistentConnection) || feof($this->fp)){

+			(!$this->persistentConnection) || feof($this->fp)) {

 			fclose($this->fp);

 			$this->fp = false;

 			$this->debug('closed socket');

 		}

 

 		// connection was closed unexpectedly

-		if($this->incoming_payload == ''){

+		if ($this->incoming_payload == '') {

 			$this->setError('no response from server');

 			return false;

 		}

@@ -964,7 +964,7 @@ 
 		if ($cErr != '') {

         	$err = 'cURL ERROR: '.curl_errno($this->ch).': '.$cErr.'<br>';

         	// TODO: there is a PHP bug that can cause this to SEGV for CURLINFO_CONTENT_TYPE

-			foreach(curl_getinfo($this->ch) as $k => $v){

+			foreach (curl_getinfo($this->ch) as $k => $v) {

 				$err .= "$k: $v<br>";

 			}

 			$this->debug($err);

@@ -984,44 +984,44 @@ 
 		$savedata = $data;

 		while ($this->isSkippableCurlHeader($data)) {

 			$this->debug("Found HTTP header to skip");

-			if ($pos = strpos($data,"\r\n\r\n")) {

-				$data = ltrim(substr($data,$pos));

-			} elseif($pos = strpos($data,"\n\n") ) {

-				$data = ltrim(substr($data,$pos));

+			if ($pos = strpos($data, "\r\n\r\n")) {

+				$data = ltrim(substr($data, $pos));

+			} elseif ($pos = strpos($data, "\n\n")) {

+				$data = ltrim(substr($data, $pos));

 			}

 		}

 

 		if ($data == '') {

 			// have nothing left; just remove 100 header(s)

 			$data = $savedata;

-			while (preg_match('/^HTTP\/1.1 100/',$data)) {

-				if ($pos = strpos($data,"\r\n\r\n")) {

-					$data = ltrim(substr($data,$pos));

-				} elseif($pos = strpos($data,"\n\n") ) {

-					$data = ltrim(substr($data,$pos));

+			while (preg_match('/^HTTP\/1.1 100/', $data)) {

+				if ($pos = strpos($data, "\r\n\r\n")) {

+					$data = ltrim(substr($data, $pos));

+				} elseif ($pos = strpos($data, "\n\n")) {

+					$data = ltrim(substr($data, $pos));

 				}

 			}

 		}

 

 		// separate content from HTTP headers

-		if ($pos = strpos($data,"\r\n\r\n")) {

+		if ($pos = strpos($data, "\r\n\r\n")) {

 			$lb = "\r\n";

-		} elseif( $pos = strpos($data,"\n\n")) {

+		} elseif ($pos = strpos($data, "\n\n")) {

 			$lb = "\n";

 		} else {

 			$this->debug('no proper separation of headers and document');

 			$this->setError('no proper separation of headers and document');

 			return false;

 		}

-		$header_data = trim(substr($data,0,$pos));

-		$header_array = explode($lb,$header_data);

-		$data = ltrim(substr($data,$pos));

+		$header_data = trim(substr($data, 0, $pos));

+		$header_array = explode($lb, $header_data);

+		$data = ltrim(substr($data, $pos));

 		$this->debug('found proper separation of headers and document');

 		$this->debug('cleaned data, stringlen: '.strlen($data));

 		// clean headers

 		foreach ($header_array as $header_line) {

-			$arr = explode(':',$header_line,2);

-			if(count($arr) > 1){

+			$arr = explode(':', $header_line, 2);

+			if (count($arr) > 1) {

 				$header_name = strtolower(trim($arr[0]));

 				$this->incoming_headers[$header_name] = trim($arr[1]);

 				if ($header_name == 'set-cookie') {

@@ -1029,14 +1029,14 @@ 
 					$cookie = $this->parseCookie(trim($arr[1]));

 					if ($cookie) {

 						$this->incoming_cookies[] = $cookie;

-						$this->debug('found cookie: ' . $cookie['name'] . ' = ' . $cookie['value']);

+						$this->debug('found cookie: '.$cookie['name'].' = '.$cookie['value']);

 					} else {

-						$this->debug('did not find cookie in ' . trim($arr[1]));

+						$this->debug('did not find cookie in '.trim($arr[1]));

 					}

     			}

 			} else if (isset($header_name)) {

 				// append continuation line to previous header

-				$this->incoming_headers[$header_name] .= $lb . ' ' . $header_line;

+				$this->incoming_headers[$header_name] .= $lb.' '.$header_line;

 			}

 		}

 	  }

@@ -1049,7 +1049,7 @@ 
 

  		// see if we need to resend the request with http digest authentication

  		if (isset($this->incoming_headers['location']) && ($http_status == 301 || $http_status == 302)) {

- 			$this->debug("Got $http_status $http_reason with Location: " . $this->incoming_headers['location']);

+ 			$this->debug("Got $http_status $http_reason with Location: ".$this->incoming_headers['location']);

  			$this->setURL($this->incoming_headers['location']);

 			$this->tryagain = true;

 			return false;

@@ -1057,7 +1057,7 @@ 
 

  		// see if we need to resend the request with http digest authentication

  		if (isset($this->incoming_headers['www-authenticate']) && $http_status == 401) {

- 			$this->debug("Got 401 $http_reason with WWW-Authenticate: " . $this->incoming_headers['www-authenticate']);

+ 			$this->debug("Got 401 $http_reason with WWW-Authenticate: ".$this->incoming_headers['www-authenticate']);

  			if (strstr($this->incoming_headers['www-authenticate'], "Digest ")) {

  				$this->debug('Server wants digest authentication');

  				// remove "Digest " from our elements

@@ -1092,10 +1092,10 @@ 
 		}

 

 		// decode content-encoding

-		if(isset($this->incoming_headers['content-encoding']) && $this->incoming_headers['content-encoding'] != ''){

-			if(strtolower($this->incoming_headers['content-encoding']) == 'deflate' || strtolower($this->incoming_headers['content-encoding']) == 'gzip'){

+		if (isset($this->incoming_headers['content-encoding']) && $this->incoming_headers['content-encoding'] != '') {

+			if (strtolower($this->incoming_headers['content-encoding']) == 'deflate' || strtolower($this->incoming_headers['content-encoding']) == 'gzip') {

     			// if decoding works, use it. else assume data wasn't gzencoded

-    			if(function_exists('gzinflate')){

+    			if (function_exists('gzinflate')) {

 					//$timer->setMarker('starting decoding of gzip/deflated content');

 					// IIS 5 requires gzinflate instead of gzuncompress (similar to IE 5 and gzdeflate v. gzcompress)

 					// this means there are no Zlib headers, although there should be

@@ -1104,13 +1104,13 @@ 
 					if ($this->incoming_headers['content-encoding'] == 'deflate') {

 						if ($degzdata = @gzinflate($data)) {

 	    					$data = $degzdata;

-	    					$this->debug('The payload has been inflated to ' . strlen($data) . ' bytes');

+	    					$this->debug('The payload has been inflated to '.strlen($data).' bytes');

 	    					if (strlen($data) < $datalen) {

 	    						// test for the case that the payload has been compressed twice

 		    					$this->debug('The inflated payload is smaller than the gzipped one; try again');

 								if ($degzdata = @gzinflate($data)) {

 			    					$data = $degzdata;

-			    					$this->debug('The payload has been inflated again to ' . strlen($data) . ' bytes');

+			    					$this->debug('The payload has been inflated again to '.strlen($data).' bytes');

 								}

 	    					}

 	    				} else {

@@ -1120,13 +1120,13 @@ 
 					} elseif ($this->incoming_headers['content-encoding'] == 'gzip') {

 						if ($degzdata = @gzinflate(substr($data, 10))) {	// do our best

 							$data = $degzdata;

-	    					$this->debug('The payload has been un-gzipped to ' . strlen($data) . ' bytes');

+	    					$this->debug('The payload has been un-gzipped to '.strlen($data).' bytes');

 	    					if (strlen($data) < $datalen) {

 	    						// test for the case that the payload has been compressed twice

 		    					$this->debug('The un-gzipped payload is smaller than the gzipped one; try again');

 								if ($degzdata = @gzinflate(substr($data, 10))) {

 			    					$data = $degzdata;

-			    					$this->debug('The payload has been un-gzipped again to ' . strlen($data) . ' bytes');

+			    					$this->debug('The payload has been un-gzipped again to '.strlen($data).' bytes');

 								}

 	    					}

 	    				} else {

@@ -1143,14 +1143,14 @@ 
 					$this->setError('The server sent compressed data. Your php install must have the Zlib extension compiled in to support this.');

 				}

 			} else {

-				$this->debug('Unsupported Content-Encoding ' . $this->incoming_headers['content-encoding']);

-				$this->setError('Unsupported Content-Encoding ' . $this->incoming_headers['content-encoding']);

+				$this->debug('Unsupported Content-Encoding '.$this->incoming_headers['content-encoding']);

+				$this->setError('Unsupported Content-Encoding '.$this->incoming_headers['content-encoding']);

 			}

 		} else {

 			$this->debug('No Content-Encoding header');

 		}

 

-		if(strlen($data) == 0){

+		if (strlen($data) == 0) {

 			$this->debug('no data after headers!');

 			$this->setError('no data present after HTTP headers');

 			return false;

@@ -1167,7 +1167,7 @@ 
 	 * @access	public

 	 */

 	function setContentType($type, $charset = false) {

-		$this->setHeader('Content-Type', $type . ($charset ? '; charset=' . $charset : ''));

+		$this->setHeader('Content-Type', $type.($charset ? '; charset='.$charset : ''));

 	}

 

 	/**

@@ -1176,7 +1176,7 @@ 
 	 * @return	boolean whether the request was honored by this method.

 	 * @access	public

 	 */

-	function usePersistentConnection(){

+	function usePersistentConnection() {

 		if (isset($this->outgoing_headers['Accept-Encoding'])) {

 			return false;

 		}

@@ -1197,7 +1197,7 @@ 
 	 * TODO: allow a Set-Cookie string to be parsed into multiple cookies

 	 */

 	function parseCookie($cookie_str) {

-		$cookie_str = str_replace('; ', ';', $cookie_str) . ';';

+		$cookie_str = str_replace('; ', ';', $cookie_str).';';

 		$data = preg_split('/;/', $cookie_str);

 		$value_str = $data[0];

 

@@ -1221,7 +1221,7 @@ 
 

 		$cookie_param = 'path=';

 		$start = strpos($cookie_str, $cookie_param);

-		if ( $start > 0 ) {

+		if ($start > 0) {

 			$path = substr($cookie_str, $start + strlen($cookie_param));

 			$path = substr($path, 0, strpos($path, ';'));

 		} else {

@@ -1240,7 +1240,7 @@ 
 		if ($sep_pos) {

 			$name = substr($value_str, 0, $sep_pos);

 			$value = substr($value_str, $sep_pos + 1);

-			$cookie= array(	'name' => $name,

+			$cookie = array('name' => $name,

 			                'value' => $value,

 							'domain' => $domain,

 							'path' => $path,

@@ -1260,40 +1260,40 @@ 
 	 * @return	string for Cookie-HTTP-Header

 	 * @access	private

 	 */

-	function getCookiesForRequest($cookies, $secure=false) {

+	function getCookiesForRequest($cookies, $secure = false) {

 		$cookie_str = '';

-		if ((! is_null($cookies)) && (is_array($cookies))) {

+		if ((!is_null($cookies)) && (is_array($cookies))) {

 			foreach ($cookies as $cookie) {

-				if (! is_array($cookie)) {

+				if (!is_array($cookie)) {

 					continue;

 				}

 	    		$this->debug("check cookie for validity: ".$cookie['name'].'='.$cookie['value']);

-				if ((isset($cookie['expires'])) && (! empty($cookie['expires']))) {

+				if ((isset($cookie['expires'])) && (!empty($cookie['expires']))) {

 					if (strtotime($cookie['expires']) <= time()) {

 						$this->debug('cookie has expired');

 						continue;

 					}

 				}

-				if ((isset($cookie['domain'])) && (! empty($cookie['domain']))) {

+				if ((isset($cookie['domain'])) && (!empty($cookie['domain']))) {

 					$domain = preg_quote($cookie['domain']);

-					if (! preg_match("'.*$domain$'i", $this->host)) {

+					if (!preg_match("'.*$domain$'i", $this->host)) {

 						$this->debug('cookie has different domain');

 						continue;

 					}

 				}

-				if ((isset($cookie['path'])) && (! empty($cookie['path']))) {

+				if ((isset($cookie['path'])) && (!empty($cookie['path']))) {

 					$path = preg_quote($cookie['path']);

-					if (! preg_match("'^$path.*'i", $this->path)) {

+					if (!preg_match("'^$path.*'i", $this->path)) {

 						$this->debug('cookie is for a different path');

 						continue;

 					}

 				}

-				if ((! $secure) && (isset($cookie['secure'])) && ($cookie['secure'])) {

+				if ((!$secure) && (isset($cookie['secure'])) && ($cookie['secure'])) {

 					$this->debug('cookie is secure, transport is not');

 					continue;

 				}

-				$cookie_str .= $cookie['name'] . '=' . $cookie['value'] . '; ';

-	    		$this->debug('add cookie to Cookie-String: ' . $cookie['name'] . '=' . $cookie['value']);

+				$cookie_str .= $cookie['name'].'='.$cookie['value'].'; ';

+	    		$this->debug('add cookie to Cookie-String: '.$cookie['name'].'='.$cookie['value']);

 			}

 		}

 		return $cookie_str;

main/inc/lib/nusoap/class.xmlschema.php

Spacing +138 added lines, -138 removed lines

@@ -48,7 +48,7 @@ 
 	* @param	string $namespaces namespaces defined in enclosing XML

 	* @access   public

 	*/

-	function __construct($schema='',$xml='',$namespaces=array()){

+	function __construct($schema = '', $xml = '', $namespaces = array()) {

 		parent::__construct();

 		$this->debug('nusoap_xmlschema class instantiated, inside constructor');

 		// files

@@ -60,13 +60,13 @@ 
 		$this->namespaces = array_merge($this->namespaces, $namespaces);

 

 		// parse schema file

-		if($schema != ''){

+		if ($schema != '') {

 			$this->debug('initial schema file: '.$schema);

 			$this->parseFile($schema, 'schema');

 		}

 

 		// parse xml file

-		if($xml != ''){

+		if ($xml != '') {

 			$this->debug('initial xml file: '.$xml);

 			$this->parseFile($xml, 'xml');

 		}

@@ -81,18 +81,18 @@ 
 	* @return boolean

     * @access public

     */

-	function parseFile($xml,$type){

+	function parseFile($xml, $type) {

 		// parse xml file

-		if($xml != ""){

-			$xmlStr = @join("",@file($xml));

-			if($xmlStr == ""){

+		if ($xml != "") {

+			$xmlStr = @join("", @file($xml));

+			if ($xmlStr == "") {

 				$msg = 'Error reading XML from '.$xml;

 				$this->setError($msg);

 				$this->debug($msg);

 			return false;

 			} else {

 				$this->debug("parsing $xml");

-				$this->parseString($xmlStr,$type);

+				$this->parseString($xmlStr, $type);

 				$this->debug("done parsing $xml");

 			return true;

 			}

@@ -107,9 +107,9 @@ 
     * @param	string $type (schema|xml)

 	* @access   private

 	*/

-	function parseString($xml,$type){

+	function parseString($xml, $type) {

 		// parse xml string

-		if($xml != ""){

+		if ($xml != "") {

 

 	    	// Create an XML parser.

 	    	$this->parser = xml_parser_create();

@@ -120,28 +120,28 @@ 
 	    	xml_set_object($this->parser, $this);

 

 	    	// Set the element handlers for the parser.

-			if($type == "schema"){

-		    	xml_set_element_handler($this->parser, 'schemaStartElement','schemaEndElement');

-		    	xml_set_character_data_handler($this->parser,'schemaCharacterData');

-			} elseif($type == "xml"){

-				xml_set_element_handler($this->parser, 'xmlStartElement','xmlEndElement');

-		    	xml_set_character_data_handler($this->parser,'xmlCharacterData');

+			if ($type == "schema") {

+		    	xml_set_element_handler($this->parser, 'schemaStartElement', 'schemaEndElement');

+		    	xml_set_character_data_handler($this->parser, 'schemaCharacterData');

+			} elseif ($type == "xml") {

+				xml_set_element_handler($this->parser, 'xmlStartElement', 'xmlEndElement');

+		    	xml_set_character_data_handler($this->parser, 'xmlCharacterData');

 			}

 

 		    // Parse the XML file.

-		    if(!xml_parse($this->parser,$xml,true)){

+		    if (!xml_parse($this->parser, $xml, true)) {

 			// Display an error message.

 				$errstr = sprintf('XML error parsing XML schema on line %d: %s',

 				xml_get_current_line_number($this->parser),

 				xml_error_string(xml_get_error_code($this->parser))

 				);

 				$this->debug($errstr);

-				$this->debug("XML payload:\n" . $xml);

+				$this->debug("XML payload:\n".$xml);

 				$this->setError($errstr);

 	    	}

 

 			xml_parser_free($this->parser);

-		} else{

+		} else {

 			$this->debug('no xml passed to parseString()!!');

 			$this->setError('no xml passed to parseString()!!');

 		}

@@ -157,9 +157,9 @@ 
 	function CreateTypeName($ename) {

 		$scope = '';

 		for ($i = 0; $i < count($this->complexTypeStack); $i++) {

-			$scope .= $this->complexTypeStack[$i] . '_';

+			$scope .= $this->complexTypeStack[$i].'_';

 		}

-		return $scope . $ename . '_ContainedType';

+		return $scope.$ename.'_ContainedType';

 	}

 

 	/**

@@ -185,7 +185,7 @@ 
 		}

 

 		// get element prefix

-		if($prefix = $this->getPrefix($name)){

+		if ($prefix = $this->getPrefix($name)) {

 			// get unqualified name

 			$name = $this->getLocalPart($name);

 		} else {

@@ -193,31 +193,31 @@ 
         }

 

         // loop thru attributes, expanding, and registering namespace declarations

-        if(count($attrs) > 0){

-        	foreach($attrs as $k => $v){

+        if (count($attrs) > 0) {

+        	foreach ($attrs as $k => $v) {

                 // if ns declarations, add to class level array of valid namespaces

-				if(preg_match('/^xmlns/',$k)){

+				if (preg_match('/^xmlns/', $k)) {

                 	//$this->xdebug("$k: $v");

                 	//$this->xdebug('ns_prefix: '.$this->getPrefix($k));

-                	if($ns_prefix = substr(strrchr($k,':'),1)){

+                	if ($ns_prefix = substr(strrchr($k, ':'), 1)) {

                 		//$this->xdebug("Add namespace[$ns_prefix] = $v");

 						$this->namespaces[$ns_prefix] = $v;

 					} else {

 						$this->defaultNamespace[$pos] = $v;

-						if (! $this->getPrefixFromNamespace($v)) {

-							$this->namespaces['ns'.(count($this->namespaces)+1)] = $v;

+						if (!$this->getPrefixFromNamespace($v)) {

+							$this->namespaces['ns'.(count($this->namespaces) + 1)] = $v;

 						}

 					}

-					if($v == 'http://www.w3.org/2001/XMLSchema' || $v == 'http://www.w3.org/1999/XMLSchema' || $v == 'http://www.w3.org/2000/10/XMLSchema'){

+					if ($v == 'http://www.w3.org/2001/XMLSchema' || $v == 'http://www.w3.org/1999/XMLSchema' || $v == 'http://www.w3.org/2000/10/XMLSchema') {

 						$this->XMLSchemaVersion = $v;

 						$this->namespaces['xsi'] = $v.'-instance';

 					}

 				}

         	}

-        	foreach($attrs as $k => $v){

+        	foreach ($attrs as $k => $v) {

                 // expand each attribute

-                $k = strpos($k,':') ? $this->expandQname($k) : $k;

-                $v = strpos($v,':') ? $this->expandQname($v) : $v;

+                $k = strpos($k, ':') ? $this->expandQname($k) : $k;

+                $v = strpos($v, ':') ? $this->expandQname($v) : $v;

         		$eAttrs[$k] = $v;

         	}

         	$attrs = $eAttrs;

@@ -225,7 +225,7 @@ 
         	$attrs = array();

         }

 		// find status, register data

-		switch($name){

+		switch ($name) {

 			case 'all':			// (optional) compositor content for a complexType

 			case 'choice':

 			case 'group':

@@ -250,41 +250,41 @@ 
 						// no namespace in arrayType attribute value...

 						if ($this->defaultNamespace[$pos]) {

 							// ...so use the default

-							$attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'] = $this->defaultNamespace[$pos] . ':' . $attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'];

+							$attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'] = $this->defaultNamespace[$pos].':'.$attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'];

 						}

 					}

             	}

-                if(isset($attrs['name'])){

+                if (isset($attrs['name'])) {

 					$this->attributes[$attrs['name']] = $attrs;

 					$aname = $attrs['name'];

-				} elseif(isset($attrs['ref']) && $attrs['ref'] == 'http://schemas.xmlsoap.org/soap/encoding/:arrayType'){

+				} elseif (isset($attrs['ref']) && $attrs['ref'] == 'http://schemas.xmlsoap.org/soap/encoding/:arrayType') {

 					if (isset($attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'])) {

 	                	$aname = $attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'];

 	                } else {

 	                	$aname = '';

 	                }

-				} elseif(isset($attrs['ref'])){

+				} elseif (isset($attrs['ref'])) {

 					$aname = $attrs['ref'];

                     $this->attributes[$attrs['ref']] = $attrs;

 				}

 

-				if($this->currentComplexType){	// This should *always* be

+				if ($this->currentComplexType) {	// This should *always* be

 					$this->complexTypes[$this->currentComplexType]['attrs'][$aname] = $attrs;

 				}

 				// arrayType attribute

-				if(isset($attrs['http://schemas.xmlsoap.org/wsdl/:arrayType']) || $this->getLocalPart($aname) == 'arrayType'){

+				if (isset($attrs['http://schemas.xmlsoap.org/wsdl/:arrayType']) || $this->getLocalPart($aname) == 'arrayType') {

 					$this->complexTypes[$this->currentComplexType]['phpType'] = 'array';

                 	$prefix = $this->getPrefix($aname);

-					if(isset($attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'])){

+					if (isset($attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'])) {

 						$v = $attrs['http://schemas.xmlsoap.org/wsdl/:arrayType'];

 					} else {

 						$v = '';

 					}

-                    if(strpos($v,'[,]')){

+                    if (strpos($v, '[,]')) {

                         $this->complexTypes[$this->currentComplexType]['multidimensional'] = true;

                     }

-                    $v = substr($v,0,strpos($v,'[')); // clip the []

-                    if(!strpos($v,':') && isset($this->typemap[$this->XMLSchemaVersion][$v])){

+                    $v = substr($v, 0, strpos($v, '[')); // clip the []

+                    if (!strpos($v, ':') && isset($this->typemap[$this->XMLSchemaVersion][$v])) {

                         $v = $this->XMLSchemaVersion.':'.$v;

                     }

                     $this->complexTypes[$this->currentComplexType]['arrayType'] = $v;

@@ -295,7 +295,7 @@ 
 			break;

 			case 'complexType':

 				array_push($this->complexTypeStack, $this->currentComplexType);

-				if(isset($attrs['name'])){

+				if (isset($attrs['name'])) {

 					// TODO: what is the scope of named complexTypes that appear

 					//       nested within other c complexTypes?

 					$this->xdebug('processing named complexType '.$attrs['name']);

@@ -310,7 +310,7 @@ 
 					//                        minOccurs="0" maxOccurs="unbounded" />

 					//                </sequence>

 					//            </complexType>

-					if(isset($attrs['base']) && preg_match('/:Array$/',$attrs['base'])){

+					if (isset($attrs['base']) && preg_match('/:Array$/', $attrs['base'])) {

 						$this->xdebug('complexType is unusual array');

 						$this->complexTypes[$this->currentComplexType]['phpType'] = 'array';

 					} else {

@@ -318,7 +318,7 @@ 
 					}

 				} else {

 					$name = $this->CreateTypeName($this->currentElement);

-					$this->xdebug('processing unnamed complexType for element ' . $this->currentElement . ' named ' . $name);

+					$this->xdebug('processing unnamed complexType for element '.$this->currentElement.' named '.$name);

 					$this->currentComplexType = $name;

 					//$this->currentElement = false;

 					$this->complexTypes[$this->currentComplexType] = $attrs;

@@ -330,7 +330,7 @@ 
 					//                        minOccurs="0" maxOccurs="unbounded" />

 					//                </sequence>

 					//            </complexType>

-					if(isset($attrs['base']) && preg_match('/:Array$/',$attrs['base'])){

+					if (isset($attrs['base']) && preg_match('/:Array$/', $attrs['base'])) {

 						$this->xdebug('complexType is unusual array');

 						$this->complexTypes[$this->currentComplexType]['phpType'] = 'array';

 					} else {

@@ -349,12 +349,12 @@ 
 						$attrs['form'] = 'qualified';

 					}

 				}

-				if(isset($attrs['type'])){

+				if (isset($attrs['type'])) {

 					$this->xdebug("processing typed element ".$attrs['name']." of type ".$attrs['type']);

-					if (! $this->getPrefix($attrs['type'])) {

+					if (!$this->getPrefix($attrs['type'])) {

 						if ($this->defaultNamespace[$pos]) {

-							$attrs['type'] = $this->defaultNamespace[$pos] . ':' . $attrs['type'];

-							$this->xdebug('used default namespace to make type ' . $attrs['type']);

+							$attrs['type'] = $this->defaultNamespace[$pos].':'.$attrs['type'];

+							$this->xdebug('used default namespace to make type '.$attrs['type']);

 						}

 					}

 					// This is for constructs like

@@ -365,20 +365,20 @@ 
 					//                </sequence>

 					//            </complexType>

 					if ($this->currentComplexType && $this->complexTypes[$this->currentComplexType]['phpType'] == 'array') {

-						$this->xdebug('arrayType for unusual array is ' . $attrs['type']);

+						$this->xdebug('arrayType for unusual array is '.$attrs['type']);

 						$this->complexTypes[$this->currentComplexType]['arrayType'] = $attrs['type'];

 					}

 					$this->currentElement = $attrs['name'];

 					$ename = $attrs['name'];

-				} elseif(isset($attrs['ref'])){

+				} elseif (isset($attrs['ref'])) {

 					$this->xdebug("processing element as ref to ".$attrs['ref']);

 					$this->currentElement = "ref to ".$attrs['ref'];

 					$ename = $this->getLocalPart($attrs['ref']);

 				} else {

-					$type = $this->CreateTypeName($this->currentComplexType . '_' . $attrs['name']);

-					$this->xdebug("processing untyped element " . $attrs['name'] . ' type ' . $type);

+					$type = $this->CreateTypeName($this->currentComplexType.'_'.$attrs['name']);

+					$this->xdebug("processing untyped element ".$attrs['name'].' type '.$type);

 					$this->currentElement = $attrs['name'];

-					$attrs['type'] = $this->schemaTargetNamespace . ':' . $type;

+					$attrs['type'] = $this->schemaTargetNamespace.':'.$type;

 					$ename = $attrs['name'];

 				}

 				if (isset($ename) && $this->currentComplexType) {

@@ -386,12 +386,12 @@ 
 					$this->complexTypes[$this->currentComplexType]['elements'][$ename] = $attrs;

 				} elseif (!isset($attrs['ref'])) {

 					$this->xdebug("add element $ename to elements array");

-					$this->elements[ $attrs['name'] ] = $attrs;

-					$this->elements[ $attrs['name'] ]['typeClass'] = 'element';

+					$this->elements[$attrs['name']] = $attrs;

+					$this->elements[$attrs['name']]['typeClass'] = 'element';

 				}

 			break;

 			case 'enumeration':	//	restriction value list member

-				$this->xdebug('enumeration ' . $attrs['value']);

+				$this->xdebug('enumeration '.$attrs['value']);

 				if ($this->currentSimpleType) {

 					$this->simpleTypes[$this->currentSimpleType]['enumeration'][] = $attrs['value'];

 				} elseif ($this->currentComplexType) {

@@ -399,11 +399,11 @@ 
 				}

 			break;

 			case 'extension':	// simpleContent or complexContent type extension

-				$this->xdebug('extension ' . $attrs['base']);

+				$this->xdebug('extension '.$attrs['base']);

 				if ($this->currentComplexType) {

 					$ns = $this->getPrefix($attrs['base']);

 					if ($ns == '') {

-						$this->complexTypes[$this->currentComplexType]['extensionBase'] = $this->schemaTargetNamespace . ':' . $attrs['base'];

+						$this->complexTypes[$this->currentComplexType]['extensionBase'] = $this->schemaTargetNamespace.':'.$attrs['base'];

 					} else {

 						$this->complexTypes[$this->currentComplexType]['extensionBase'] = $attrs['base'];

 					}

@@ -413,19 +413,19 @@ 
 			break;

 			case 'import':

 			    if (isset($attrs['schemaLocation'])) {

-					$this->xdebug('import namespace ' . $attrs['namespace'] . ' from ' . $attrs['schemaLocation']);

+					$this->xdebug('import namespace '.$attrs['namespace'].' from '.$attrs['schemaLocation']);

                     $this->imports[$attrs['namespace']][] = array('location' => $attrs['schemaLocation'], 'loaded' => false);

 				} else {

-					$this->xdebug('import namespace ' . $attrs['namespace']);

+					$this->xdebug('import namespace '.$attrs['namespace']);

                     $this->imports[$attrs['namespace']][] = array('location' => '', 'loaded' => true);

-					if (! $this->getPrefixFromNamespace($attrs['namespace'])) {

-						$this->namespaces['ns'.(count($this->namespaces)+1)] = $attrs['namespace'];

+					if (!$this->getPrefixFromNamespace($attrs['namespace'])) {

+						$this->namespaces['ns'.(count($this->namespaces) + 1)] = $attrs['namespace'];

 					}

 				}

 			break;

 			case 'include':

 			    if (isset($attrs['schemaLocation'])) {

-					$this->xdebug('include into namespace ' . $this->schemaTargetNamespace . ' from ' . $attrs['schemaLocation']);

+					$this->xdebug('include into namespace '.$this->schemaTargetNamespace.' from '.$attrs['schemaLocation']);

                     $this->imports[$this->schemaTargetNamespace][] = array('location' => $attrs['schemaLocation'], 'loaded' => false);

 				} else {

 					$this->xdebug('ignoring invalid XML Schema construct: include without schemaLocation attribute');

@@ -435,12 +435,12 @@ 
 				$this->xdebug("do nothing for element $name");

 			break;

 			case 'restriction':	// simpleType, simpleContent or complexContent value restriction

-				$this->xdebug('restriction ' . $attrs['base']);

-				if($this->currentSimpleType){

+				$this->xdebug('restriction '.$attrs['base']);

+				if ($this->currentSimpleType) {

 					$this->simpleTypes[$this->currentSimpleType]['type'] = $attrs['base'];

-				} elseif($this->currentComplexType){

+				} elseif ($this->currentComplexType) {

 					$this->complexTypes[$this->currentComplexType]['restrictionBase'] = $attrs['base'];

-					if(strstr($attrs['base'],':') == ':Array'){

+					if (strstr($attrs['base'], ':') == ':Array') {

 						$this->complexTypes[$this->currentComplexType]['phpType'] = 'array';

 					}

 				}

@@ -467,15 +467,15 @@ 
 			break;

 			case 'simpleType':

 				array_push($this->simpleTypeStack, $this->currentSimpleType);

-				if(isset($attrs['name'])){

-					$this->xdebug("processing simpleType for name " . $attrs['name']);

+				if (isset($attrs['name'])) {

+					$this->xdebug("processing simpleType for name ".$attrs['name']);

 					$this->currentSimpleType = $attrs['name'];

-					$this->simpleTypes[ $attrs['name'] ] = $attrs;

-					$this->simpleTypes[ $attrs['name'] ]['typeClass'] = 'simpleType';

-					$this->simpleTypes[ $attrs['name'] ]['phpType'] = 'scalar';

+					$this->simpleTypes[$attrs['name']] = $attrs;

+					$this->simpleTypes[$attrs['name']]['typeClass'] = 'simpleType';

+					$this->simpleTypes[$attrs['name']]['phpType'] = 'scalar';

 				} else {

-					$name = $this->CreateTypeName($this->currentComplexType . '_' . $this->currentElement);

-					$this->xdebug('processing unnamed simpleType for element ' . $this->currentElement . ' named ' . $name);

+					$name = $this->CreateTypeName($this->currentComplexType.'_'.$this->currentElement);

+					$this->xdebug('processing unnamed simpleType for element '.$this->currentElement.' named '.$name);

 					$this->currentSimpleType = $name;

 					//$this->currentElement = false;

 					$this->simpleTypes[$this->currentSimpleType] = $attrs;

@@ -501,29 +501,29 @@ 
 		// bring depth down a notch

 		$this->depth--;

 		// position of current element is equal to the last value left in depth_array for my depth

-		if(isset($this->depth_array[$this->depth])){

+		if (isset($this->depth_array[$this->depth])) {

         	$pos = $this->depth_array[$this->depth];

         }

 		// get element prefix

-		if ($prefix = $this->getPrefix($name)){

+		if ($prefix = $this->getPrefix($name)) {

 			// get unqualified name

 			$name = $this->getLocalPart($name);

 		} else {

         	$prefix = '';

         }

 		// move on...

-		if($name == 'complexType'){

-			$this->xdebug('done processing complexType ' . ($this->currentComplexType ? $this->currentComplexType : '(unknown)'));

+		if ($name == 'complexType') {

+			$this->xdebug('done processing complexType '.($this->currentComplexType ? $this->currentComplexType : '(unknown)'));

 			$this->xdebug($this->varDump($this->complexTypes[$this->currentComplexType]));

 			$this->currentComplexType = array_pop($this->complexTypeStack);

 			//$this->currentElement = false;

 		}

-		if($name == 'element'){

-			$this->xdebug('done processing element ' . ($this->currentElement ? $this->currentElement : '(unknown)'));

+		if ($name == 'element') {

+			$this->xdebug('done processing element '.($this->currentElement ? $this->currentElement : '(unknown)'));

 			$this->currentElement = array_pop($this->elementStack);

 		}

-		if($name == 'simpleType'){

-			$this->xdebug('done processing simpleType ' . ($this->currentSimpleType ? $this->currentSimpleType : '(unknown)'));

+		if ($name == 'simpleType') {

+			$this->xdebug('done processing simpleType '.($this->currentSimpleType ? $this->currentSimpleType : '(unknown)'));

 			$this->xdebug($this->varDump($this->simpleTypes[$this->currentSimpleType]));

 			$this->currentSimpleType = array_pop($this->simpleTypeStack);

 		}

@@ -536,7 +536,7 @@ 
 	* @param    string $data element content

 	* @access   private

 	*/

-	function schemaCharacterData($parser, $data){

+	function schemaCharacterData($parser, $data) {

 		$pos = $this->depth_array[$this->depth - 1];

 		$this->message[$pos]['cdata'] .= $data;

 	}

@@ -546,32 +546,32 @@ 
 	*

 	* @access   public

 	*/

-	function serializeSchema(){

+	function serializeSchema() {

 

 		$schemaPrefix = $this->getPrefixFromNamespace($this->XMLSchemaVersion);

 		$xml = '';

 		// imports

 		if (sizeof($this->imports) > 0) {

-			foreach($this->imports as $ns => $list) {

+			foreach ($this->imports as $ns => $list) {

 				foreach ($list as $ii) {

 					if ($ii['location'] != '') {

-						$xml .= " <$schemaPrefix:import location=\"" . $ii['location'] . '" namespace="' . $ns . "\" />\n";

+						$xml .= " <$schemaPrefix:import location=\"".$ii['location'].'" namespace="'.$ns."\" />\n";

 					} else {

-						$xml .= " <$schemaPrefix:import namespace=\"" . $ns . "\" />\n";

+						$xml .= " <$schemaPrefix:import namespace=\"".$ns."\" />\n";

 					}

 				}

 			}

 		}

 		// complex types

-		foreach($this->complexTypes as $typeName => $attrs){

+		foreach ($this->complexTypes as $typeName => $attrs) {

 			$contentStr = '';

 			// serialize child elements

-			if(isset($attrs['elements']) && (count($attrs['elements']) > 0)){

-				foreach($attrs['elements'] as $element => $eParts){

-					if(isset($eParts['ref'])){

+			if (isset($attrs['elements']) && (count($attrs['elements']) > 0)) {

+				foreach ($attrs['elements'] as $element => $eParts) {

+					if (isset($eParts['ref'])) {

 						$contentStr .= "   <$schemaPrefix:element ref=\"$element\"/>\n";

 					} else {

-						$contentStr .= "   <$schemaPrefix:element name=\"$element\" type=\"" . $this->contractQName($eParts['type']) . "\"";

+						$contentStr .= "   <$schemaPrefix:element name=\"$element\" type=\"".$this->contractQName($eParts['type'])."\"";

 						foreach ($eParts as $aName => $aValue) {

 							// handle, e.g., abstract, default, form, minOccurs, maxOccurs, nillable

 							if ($aName != 'name' && $aName != 'type') {

@@ -587,8 +587,8 @@ 
 				}

 			}

 			// attributes

-			if(isset($attrs['attrs']) && (count($attrs['attrs']) >= 1)){

-				foreach($attrs['attrs'] as $attr => $aParts){

+			if (isset($attrs['attrs']) && (count($attrs['attrs']) >= 1)) {

+				foreach ($attrs['attrs'] as $attr => $aParts) {

 					$contentStr .= "    <$schemaPrefix:attribute";

 					foreach ($aParts as $a => $v) {

 						if ($a == 'ref' || $a == 'type') {

@@ -604,15 +604,15 @@ 
 				}

 			}

 			// if restriction

-			if (isset($attrs['restrictionBase']) && $attrs['restrictionBase'] != ''){

+			if (isset($attrs['restrictionBase']) && $attrs['restrictionBase'] != '') {

 				$contentStr = "   <$schemaPrefix:restriction base=\"".$this->contractQName($attrs['restrictionBase'])."\">\n".$contentStr."   </$schemaPrefix:restriction>\n";

 				// complex or simple content

-				if ((isset($attrs['elements']) && count($attrs['elements']) > 0) || (isset($attrs['attrs']) && count($attrs['attrs']) > 0)){

+				if ((isset($attrs['elements']) && count($attrs['elements']) > 0) || (isset($attrs['attrs']) && count($attrs['attrs']) > 0)) {

 					$contentStr = "  <$schemaPrefix:complexContent>\n".$contentStr."  </$schemaPrefix:complexContent>\n";

 				}

 			}

 			// finalize complex type

-			if($contentStr != ''){

+			if ($contentStr != '') {

 				$contentStr = " <$schemaPrefix:complexType name=\"$typeName\">\n".$contentStr." </$schemaPrefix:complexType>\n";

 			} else {

 				$contentStr = " <$schemaPrefix:complexType name=\"$typeName\"/>\n";

@@ -620,8 +620,8 @@ 
 			$xml .= $contentStr;

 		}

 		// simple types

-		if(isset($this->simpleTypes) && count($this->simpleTypes) > 0){

-			foreach($this->simpleTypes as $typeName => $eParts){

+		if (isset($this->simpleTypes) && count($this->simpleTypes) > 0) {

+			foreach ($this->simpleTypes as $typeName => $eParts) {

 				$xml .= " <$schemaPrefix:simpleType name=\"$typeName\">\n  <$schemaPrefix:restriction base=\"".$this->contractQName($eParts['type'])."\">\n";

 				if (isset($eParts['enumeration'])) {

 					foreach ($eParts['enumeration'] as $e) {

@@ -632,14 +632,14 @@ 
 			}

 		}

 		// elements

-		if(isset($this->elements) && count($this->elements) > 0){

-			foreach($this->elements as $element => $eParts){

+		if (isset($this->elements) && count($this->elements) > 0) {

+			foreach ($this->elements as $element => $eParts) {

 				$xml .= " <$schemaPrefix:element name=\"$element\" type=\"".$this->contractQName($eParts['type'])."\"/>\n";

 			}

 		}

 		// attributes

-		if(isset($this->attributes) && count($this->attributes) > 0){

-			foreach($this->attributes as $attr => $aParts){

+		if (isset($this->attributes) && count($this->attributes) > 0) {

+			foreach ($this->attributes as $attr => $aParts) {

 				$xml .= " <$schemaPrefix:attribute name=\"$attr\" type=\"".$this->contractQName($aParts['type'])."\"\n/>";

 			}

 		}

@@ -654,7 +654,7 @@ 
 		foreach (array_diff($this->usedNamespaces, $this->enclosingNamespaces) as $nsp => $ns) {

 			$el .= " xmlns:$nsp=\"$ns\"";

 		}

-		$xml = $el . ">\n".$xml."</$schemaPrefix:schema>\n";

+		$xml = $el.">\n".$xml."</$schemaPrefix:schema>\n";

 		return $xml;

 	}

 

@@ -664,8 +664,8 @@ 
 	* @param    string $string debug data

 	* @access   private

 	*/

-	function xdebug($string){

-		$this->debug('<' . $this->schemaTargetNamespace . '> '.$string);

+	function xdebug($string) {

+		$this->debug('<'.$this->schemaTargetNamespace.'> '.$string);

 	}

 

     /**

@@ -680,11 +680,11 @@ 
     * @access public

     * @deprecated

     */

-	function getPHPType($type,$ns){

-		if(isset($this->typemap[$ns][$type])){

+	function getPHPType($type, $ns) {

+		if (isset($this->typemap[$ns][$type])) {

 			//print "found type '$type' and ns $ns in typemap<br>";

 			return $this->typemap[$ns][$type];

-		} elseif(isset($this->complexTypes[$type])){

+		} elseif (isset($this->complexTypes[$type])) {

 			//print "getting type '$type' and ns $ns from complexTypes array<br>";

 			return $this->complexTypes[$type]['phpType'];

 		}

@@ -713,7 +713,7 @@ 
     * @see addSimpleType

     * @see addElement

     */

-	function getTypeDef($type){

+	function getTypeDef($type) {

 		//$this->debug("in getTypeDef for type $type");

 		if (substr($type, -1) == '^') {

 			$is_element = 1;

@@ -722,10 +722,10 @@ 
 			$is_element = 0;

 		}

 

-		if((! $is_element) && isset($this->complexTypes[$type])){

+		if ((!$is_element) && isset($this->complexTypes[$type])) {

 			$this->xdebug("in getTypeDef, found complexType $type");

 			return $this->complexTypes[$type];

-		} elseif((! $is_element) && isset($this->simpleTypes[$type])){

+		} elseif ((!$is_element) && isset($this->simpleTypes[$type])) {

 			$this->xdebug("in getTypeDef, found simpleType $type");

 			if (!isset($this->simpleTypes[$type]['phpType'])) {

 				// get info for type to tack onto the simple type

@@ -745,7 +745,7 @@ 
 				}

 			}

 			return $this->simpleTypes[$type];

-		} elseif(isset($this->elements[$type])){

+		} elseif (isset($this->elements[$type])) {

 			$this->xdebug("in getTypeDef, found element $type");

 			if (!isset($this->elements[$type]['phpType'])) {

 				// get info for type to tack onto the element

@@ -770,7 +770,7 @@ 
 				}

 			}

 			return $this->elements[$type];

-		} elseif(isset($this->attributes[$type])){

+		} elseif (isset($this->attributes[$type])) {

 			$this->xdebug("in getTypeDef, found attribute $type");

 			return $this->attributes[$type];

 		} elseif (preg_match('/_ContainedType$/', $type)) {

@@ -792,23 +792,23 @@ 
     * @access public

     * @deprecated

     */

-    function serializeTypeDef($type){

+    function serializeTypeDef($type) {

     	//print "in sTD() for type $type<br>";

-	if($typeDef = $this->getTypeDef($type)){

+	if ($typeDef = $this->getTypeDef($type)) {

 		$str .= '<'.$type;

-	    if(is_array($typeDef['attrs'])){

-		foreach($typeDef['attrs'] as $attName => $data){

+	    if (is_array($typeDef['attrs'])) {

+		foreach ($typeDef['attrs'] as $attName => $data) {

 		    $str .= " $attName=\"{type = ".$data['type']."}\"";

 		}

 	    }

 	    $str .= " xmlns=\"".$this->schema['targetNamespace']."\"";

-	    if(count($typeDef['elements']) > 0){

+	    if (count($typeDef['elements']) > 0) {

 		$str .= ">";

-		foreach($typeDef['elements'] as $element => $eData){

+		foreach ($typeDef['elements'] as $element => $eData) {

 		    $str .= $this->serializeTypeDef($element);

 		}

 		$str .= "</$type>";

-	    } elseif($typeDef['typeClass'] == 'element') {

+	    } elseif ($typeDef['typeClass'] == 'element') {

 		$str .= "></$type>";

 	    } else {

 		$str .= "/>";

@@ -828,22 +828,22 @@ 
     * @access public

     * @deprecated

 	*/

-	function typeToForm($name,$type){

+	function typeToForm($name, $type) {

 		// get typedef

-		if($typeDef = $this->getTypeDef($type)){

+		if ($typeDef = $this->getTypeDef($type)) {

 			// if struct

-			if($typeDef['phpType'] == 'struct'){

+			if ($typeDef['phpType'] == 'struct') {

 				$buffer .= '<table>';

-				foreach($typeDef['elements'] as $child => $childDef){

+				foreach ($typeDef['elements'] as $child => $childDef) {

 					$buffer .= "

 					<tr><td align='right'>$childDef[name] (type: ".$this->getLocalPart($childDef['type'])."):</td>

 					<td><input type='text' name='parameters[".$name."][$childDef[name]]'></td></tr>";

 				}

 				$buffer .= '</table>';

 			// if array

-			} elseif($typeDef['phpType'] == 'array'){

+			} elseif ($typeDef['phpType'] == 'array') {

 				$buffer .= '<table>';

-				for($i=0;$i < 3; $i++){

+				for ($i = 0; $i < 3; $i++) {

 					$buffer .= "

 					<tr><td align='right'>array item (type: $typeDef[arrayType]):</td>

 					<td><input type='text' name='parameters[".$name."][]'></td></tr>";

@@ -900,7 +900,7 @@ 
 	* @access public

 	* @see getTypeDef

 	*/

-	function addComplexType($name,$typeClass='complexType',$phpType='array',$compositor='',$restrictionBase='',$elements=array(),$attrs=array(),$arrayType=''){

+	function addComplexType($name, $typeClass = 'complexType', $phpType = 'array', $compositor = '', $restrictionBase = '', $elements = array(), $attrs = array(), $arrayType = '') {

 		$this->complexTypes[$name] = array(

 	    'name'		=> $name,

 	    'typeClass'	=> $typeClass,

@@ -928,7 +928,7 @@ 
 	* @see nusoap_xmlschema

 	* @see getTypeDef

 	*/

-	function addSimpleType($name, $restrictionBase='', $typeClass='simpleType', $phpType='scalar', $enumeration=array()) {

+	function addSimpleType($name, $restrictionBase = '', $typeClass = 'simpleType', $phpType = 'scalar', $enumeration = array()) {

 		$this->simpleTypes[$name] = array(

 	    'name'			=> $name,

 	    'typeClass'		=> $typeClass,

@@ -949,14 +949,14 @@ 
 	* @access public

 	*/

 	function addElement($attrs) {

-		if (! $this->getPrefix($attrs['type'])) {

-			$attrs['type'] = $this->schemaTargetNamespace . ':' . $attrs['type'];

+		if (!$this->getPrefix($attrs['type'])) {

+			$attrs['type'] = $this->schemaTargetNamespace.':'.$attrs['type'];

 		}

-		$this->elements[ $attrs['name'] ] = $attrs;

-		$this->elements[ $attrs['name'] ]['typeClass'] = 'element';

+		$this->elements[$attrs['name']] = $attrs;

+		$this->elements[$attrs['name']]['typeClass'] = 'element';

 

-		$this->xdebug("addElement " . $attrs['name']);

-		$this->appendDebug($this->varDump($this->elements[ $attrs['name'] ]));

+		$this->xdebug("addElement ".$attrs['name']);

+		$this->appendDebug($this->varDump($this->elements[$attrs['name']]));

 	}

 }

 

src/Chamilo/UserBundle/Entity/Repository/UserRepository.php

Spacing +2 added lines, -2 removed lines

@@ -258,7 +258,7 @@ 
                         INNER JOIN ChamiloUserBundle:User AS U WITH UF.friendUserId = U
                         WHERE
                             U.status != 6 AND
-                            UF.relationType NOT IN(" . USER_RELATION_TYPE_DELETED . ", " . USER_RELATION_TYPE_RRHH . ") AND
+                            UF.relationType NOT IN(" . USER_RELATION_TYPE_DELETED.", ".USER_RELATION_TYPE_RRHH.") AND
                             UF.userId = $currentUserId AND
                             UF.friendUserId != $currentUserId AND
                             U = R.user AND
@@ -282,7 +282,7 @@ 
                 $dql = "SELECT DISTINCT U
                         FROM ChamiloUserBundle:User U
                         INNER JOIN ChamiloCoreBundle:TrackEOnline T WITH U.id = T.loginUserId
-                        WHERE T.loginDate >= '" . $limit_date . "'";
+                        WHERE T.loginDate >= '" . $limit_date."'";
             }
         }
 

plugin/ims_lti/create.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         Display::return_message($plugin->get_lang('ToolAdded'), 'success')
     );
 
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
     exit;
 }
 

plugin/ims_lti/OAuthSimple.php

Spacing +55 added lines, -55 removed lines

@@ -25,7 +25,7 @@ 
      * @param shared_secret (String) The shared secret. This value is also usually provided by the site you wish to use.
 	 * @return OAuthSimple (Object)
      */
-    function __construct ($APIKey = "", $sharedSecret=""){
+    function __construct($APIKey = "", $sharedSecret = "") {
 
         if (!empty($APIKey))
 		{
@@ -65,7 +65,7 @@ 
 	 * @param(string, object) List of parameters for the call, this can either be a URI string (e.g. "foo=bar&gorp=banana" or an object/hash)
 	 * @return OAuthSimple (Object)
 	 */
-    public function setParameters ($parameters=Array()) {
+    public function setParameters($parameters = Array()) {
 
         if (is_string($parameters))
 		{
@@ -77,7 +77,7 @@ 
 		}
         else if (!empty($parameters))
 		{
-			$this->_parameters = array_merge($this->_parameters,$parameters);
+			$this->_parameters = array_merge($this->_parameters, $parameters);
 		}
         if (empty($this->_parameters['oauth_nonce']))
 		{
@@ -101,7 +101,7 @@ 
 		}
         if (empty($this->_parameters['oauth_version']))
 		{
-            $this->_parameters['oauth_version']="1.0";
+            $this->_parameters['oauth_version'] = "1.0";
 		}
 
         return $this;
@@ -113,7 +113,7 @@ 
 	 * @access public
 	 * @see setParameters
 	 */
-    public function setQueryString ($parameters)
+    public function setQueryString($parameters)
     {
         return $this->setParameters($parameters);
     }
@@ -124,13 +124,13 @@ 
      * @param path (String) the fully qualified URI (excluding query arguments) (e.g "http://example.org/foo")
 	 * @return OAuthSimple (Object)
      */
-    public function setURL ($path)
+    public function setURL($path)
 	{
         if (empty($path))
 		{
             throw new OAuthSimpleException('No path specified for OAuthSimple.setURL');
 		}
-        $this->_path=$path;
+        $this->_path = $path;
 
         return $this;
     }
@@ -141,9 +141,9 @@ 
      * @param path (String)
 	 * @see setURL
      */
-    public function setPath ($path)
+    public function setPath($path)
     {
-        return $this->_path=$path;
+        return $this->_path = $path;
     }
 
     /**
@@ -152,14 +152,14 @@ 
      * @param action (String) HTTP Action word.
 	 * @return OAuthSimple (Object)
      */
-    public function setAction ($action)
+    public function setAction($action)
     {
         if (empty($action))
 		{
 			$action = 'GET';
 		}
         $action = strtoupper($action);
-        if (preg_match('/[^A-Z]/',$action))
+        if (preg_match('/[^A-Z]/', $action))
 		{
             throw new OAuthSimpleException('Invalid action specified for OAuthSimple.setAction');
 		}
@@ -174,7 +174,7 @@ 
      * @param signatures (object) object/hash of the token/signature pairs {api_key:, shared_secret:, oauth_token: oauth_secret:}
 	 * @return OAuthSimple (Object)
      */
-    public function signatures ($signatures)
+    public function signatures($signatures)
     {
         if (!empty($signatures) && !is_array($signatures))
 		{
@@ -184,9 +184,9 @@ 
         {
             if (empty($this->_secrets))
             {
-                $this->_secrets=Array();
+                $this->_secrets = Array();
             }
-            $this->_secrets=array_merge($this->_secrets,$signatures);
+            $this->_secrets = array_merge($this->_secrets, $signatures);
         }
         if (isset($this->_secrets['api_key']))
 		{
@@ -231,21 +231,21 @@ 
      * @param method (String) Method of signing the transaction (only PLAINTEXT and SHA-MAC1 allowed for now)
 	 * @return OAuthSimple (Object)
     */
-    public function setSignatureMethod ($method="")
+    public function setSignatureMethod($method = "")
 	{
         if (empty($method))
 		{
             $method = $this->_default_signature_method;
 		}
         $method = strtoupper($method);
-        switch($method)
+        switch ($method)
         {
             case 'PLAINTEXT':
             case 'HMAC-SHA1':
-                $this->_parameters['oauth_signature_method']=$method;
+                $this->_parameters['oauth_signature_method'] = $method;
 				break;
             default:
-                throw new OAuthSimpleException ("Unknown signing method $method specified for OAuthSimple.setSignatureMethod");
+                throw new OAuthSimpleException("Unknown signing method $method specified for OAuthSimple.setSignatureMethod");
 				break;
         }
 
@@ -260,7 +260,7 @@ 
      * @param args (Array) hash of arguments for the call {action, path, parameters (array), method, signatures (array)} all arguments are optional.
 	 * @return (Array) signed values
      */
-    public function sign($args=array())
+    public function sign($args = array())
     {
         if (!empty($args['action']))
 		{
@@ -280,15 +280,15 @@ 
 		}
         if (empty($args['parameters']))
 		{
-            $args['parameters']=array();
+            $args['parameters'] = array();
 		}
         $this->setParameters($args['parameters']);
         $normParams = $this->_normalizedParameters();
 
-        return Array (
+        return Array(
             'parameters' => $this->_parameters,
             'signature' => self::_oauthEscape($this->_parameters['oauth_signature']),
-            'signed_url' => $this->_path . '?' . $normParams,
+            'signed_url' => $this->_path.'?'.$normParams,
             'header' => $this->getHeaderString(),
             'sbs'=> $this->sbs
             );
@@ -304,7 +304,7 @@ 
      * @param args (Array)
 	 * @return $result (String)
     */
-    public function getHeaderString ($args=array())
+    public function getHeaderString($args = array())
     {
         if (empty($this->_parameters['oauth_signature']))
 		{
@@ -314,7 +314,7 @@ 
 
         foreach ($this->_parameters as $pName => $pValue)
         {
-            if (strpos($pName,'oauth_') !== 0)
+            if (strpos($pName, 'oauth_') !== 0)
 			{
                 continue;
 			}
@@ -322,25 +322,25 @@ 
             {
                 foreach ($pValue as $val)
                 {
-                    $result .= $pName .'="' . self::_oauthEscape($val) . '", ';
+                    $result .= $pName.'="'.self::_oauthEscape($val).'", ';
                 }
             }
             else
             {
-                $result .= $pName . '="' . self::_oauthEscape($pValue) . '", ';
+                $result .= $pName.'="'.self::_oauthEscape($pValue).'", ';
             }
         }
 
-        return preg_replace('/, $/','',$result);
+        return preg_replace('/, $/', '', $result);
     }
 
-    private function _parseParameterString ($paramString)
+    private function _parseParameterString($paramString)
     {
-        $elements = explode('&',$paramString);
+        $elements = explode('&', $paramString);
         $result = array();
         foreach ($elements as $element)
         {
-            list ($key,$token) = explode('=',$element);
+            list ($key, $token) = explode('=', $element);
             if ($token)
 			{
                 $token = urldecode($token);
@@ -349,15 +349,15 @@ 
             {
                 if (!is_array($result[$key]))
 				{
-                    $result[$key] = array($result[$key],$token);
+                    $result[$key] = array($result[$key], $token);
 				}
                 else
 				{
-                    array_push($result[$key],$token);
+                    array_push($result[$key], $token);
 				}
             }
             else
-                $result[$key]=$token;
+                $result[$key] = $token;
         }
         return $result;
     }
@@ -375,21 +375,21 @@ 
 
 	    //FIX: urlencode of ~ and '+'
         $string = str_replace(
-            Array('%7E','+'  ), // Replace these
-            Array('~',  '%20'), // with these
+            Array('%7E', '+'), // Replace these
+            Array('~', '%20'), // with these
             $string);
 
         return $string;
     }
 
-    private function _getNonce($length=5)
+    private function _getNonce($length = 5)
     {
         $result = '';
         $cLength = strlen($this->_nonce_chars);
-        for ($i=0; $i < $length; $i++)
+        for ($i = 0; $i < $length; $i++)
         {
-            $rnum = rand(0,$cLength - 1);
-            $result .= substr($this->_nonce_chars,$rnum,1);
+            $rnum = rand(0, $cLength - 1);
+            $result .= substr($this->_nonce_chars, $rnum, 1);
         }
         $this->_parameters['oauth_nonce'] = $result;
 
@@ -402,7 +402,7 @@ 
         {
             throw new OAuthSimpleException('No consumer_key set for OAuthSimple');
         }
-        $this->_parameters['oauth_consumer_key']=$this->_secrets['consumer_key'];
+        $this->_parameters['oauth_consumer_key'] = $this->_secrets['consumer_key'];
 
         return $this->_parameters['oauth_consumer_key'];
     }
@@ -432,7 +432,7 @@ 
 		$normalized_keys = array();
 		$return_array = array();
 
-        foreach ( $this->_parameters as $paramName=>$paramValue) {
+        foreach ($this->_parameters as $paramName=>$paramValue) {
             if (preg_match('/w+_secret/', $paramName) OR
                 $paramName == "oauth_signature") {
                     continue;
@@ -443,9 +443,9 @@ 
 				if (is_array($paramValue))
 				{
 					$normalized_keys[self::_oauthEscape($paramName)] = array();
-					foreach($paramValue as $item)
+					foreach ($paramValue as $item)
 					{
-						array_push($normalized_keys[self::_oauthEscape($paramName)],  self::_oauthEscape($item));
+						array_push($normalized_keys[self::_oauthEscape($paramName)], self::_oauthEscape($item));
 					}
 				}
 				else
@@ -457,54 +457,54 @@ 
 
 		ksort($normalized_keys);
 
-		foreach($normalized_keys as $key=>$val)
+		foreach ($normalized_keys as $key=>$val)
 		{
 			if (is_array($val))
 			{
 				sort($val);
-				foreach($val as $element)
+				foreach ($val as $element)
 				{
-					array_push($return_array, $key . "=" . $element);
+					array_push($return_array, $key."=".$element);
 				}
 			}
 			else
 			{
-				array_push($return_array, $key .'='. $val);
+				array_push($return_array, $key.'='.$val);
 			}
 
         }
         $presig = join("&", $return_array);
         $sig = $this->_generateSignature($presig);
-        $this->_parameters['oauth_signature']=$sig;
+        $this->_parameters['oauth_signature'] = $sig;
         array_push($return_array, "oauth_signature=$sig");
 		return join("&", $return_array);
     }
 
 
-    private function _generateSignature ($parameters="")
+    private function _generateSignature($parameters = "")
     {
         $secretKey = '';
-		if(isset($this->_secrets['shared_secret']))
+		if (isset($this->_secrets['shared_secret']))
 		{
 			$secretKey = self::_oauthEscape($this->_secrets['shared_secret']);
 		}
 
 		$secretKey .= '&';
-		if(isset($this->_secrets['oauth_secret']))
+		if (isset($this->_secrets['oauth_secret']))
 		{
             $secretKey .= self::_oauthEscape($this->_secrets['oauth_secret']);
         }
-        if(!empty($parameters)){
+        if (!empty($parameters)) {
             $parameters = urlencode($parameters);
         }
-        switch($this->_parameters['oauth_signature_method'])
+        switch ($this->_parameters['oauth_signature_method'])
         {
             case 'PLAINTEXT':
-                return urlencode($secretKey);;
+                return urlencode($secretKey); ;
             case 'HMAC-SHA1':
                 $this->sbs = self::_oauthEscape($this->_action).'&'.self::_oauthEscape($this->_path).'&'.$parameters;
 
-                return base64_encode(hash_hmac('sha1',$this->sbs,$secretKey,TRUE));
+                return base64_encode(hash_hmac('sha1', $this->sbs, $secretKey, TRUE));
             default:
                 throw new OAuthSimpleException('Unknown signature method for OAuthSimple');
 				break;

plugin/ims_lti/edit.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         Display::return_message($plugin->get_lang('NoTool'), 'error')
     );
 
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
     exit;
 }
 
@@ -58,7 +58,7 @@ 
         Display::return_message($plugin->get_lang('ToolEdited'), 'success')
     );
 
-    header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+    header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
     exit;
 }
 

plugin/ims_lti/ImsLtiTool.php

Spacing +1 added lines, -1 removed lines

@@ -173,7 +173,7 @@
         $foo = explode('=', $strings);
 
         return [
-            'key' => 'custom_' . $foo[0],
+            'key' => 'custom_'.$foo[0],
             'value' => $foo[1]
         ];
     }

plugin/ims_lti/ImsLtiPlugin.php

Spacing +6 added lines, -6 removed lines

@@ -111,7 +111,7 @@ 
             return false;
         }
 
-        $sql = 'DROP TABLE IF EXISTS ' . self::TABLE_TOOL;
+        $sql = 'DROP TABLE IF EXISTS '.self::TABLE_TOOL;
         Database::query($sql);
 
         return true;
@@ -124,14 +124,14 @@ 
     {
         $button = Display::toolbarButton(
             $this->get_lang('AddExternalTool'),
-            api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/add.php',
+            api_get_path(WEB_PLUGIN_PATH).'ims_lti/add.php',
             'cog',
             'primary'
         );
 
         $this->course_settings = [
             [
-                'name' => $this->get_lang('ImsLtiDescription') . $button . '<hr>',
+                'name' => $this->get_lang('ImsLtiDescription').$button.'<hr>',
                 'type' => 'html'
             ]
         ];
@@ -153,8 +153,8 @@ 
             ->setId($cToolId)
             ->setCId($course->getId())
             ->setName($tool->getName())
-            ->setLink($this->get_name() . '/start.php?' . http_build_query(['id' => $tool->getId()]))
-            ->setImage($this->get_name() . '.png')
+            ->setLink($this->get_name().'/start.php?'.http_build_query(['id' => $tool->getId()]))
+            ->setImage($this->get_name().'.png')
             ->setVisibility(1)
             ->setAdmin(0)
             ->setAddress('squaregray.gif')
@@ -172,7 +172,7 @@ 
         $text = $this->get_lang('ImsLtiDescription');
         $text .= sprintf(
             $this->get_lang('ManageToolButton'),
-            api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php'
+            api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php'
         );
 
         return $text;

plugin/ims_lti/start.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 
 if (empty($toolId)) {
     if (api_is_platform_admin()) {
-        header('Location: ' . api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/list.php');
+        header('Location: '.api_get_path(WEB_PLUGIN_PATH).'ims_lti/list.php');
         exit;
     }
 
@@ -25,7 +25,7 @@ 
 $template = new Template($imsLtiPlugin->get_title());
 $template->assign(
     'launch_url',
-    api_get_path(WEB_PLUGIN_PATH) . 'ims_lti/form.php?' . http_build_query(['id' => $tool->getId()])
+    api_get_path(WEB_PLUGIN_PATH).'ims_lti/form.php?'.http_build_query(['id' => $tool->getId()])
 );
 
 $content = $template->fetch('ims_lti/view/start.tpl');