chamilo / chamilo-lms

plugin/bbb/lib/bbb.lib.php

Doc Comments +2 added lines, -2 removed lines

@@ -240,7 +240,7 @@ 
     /**
      * Returns a meeting "join" URL
      * @param string The name of the meeting (usually the course code)
-     * @return mixed The URL to join the meeting, or false on error
+     * @return false|string The URL to join the meeting, or false on error
      * @todo implement moderator pass
      * @assert ('') === false
      * @assert ('abcdefghijklmnopqrstuvwxyzabcdefghijklmno') === false
@@ -647,7 +647,7 @@ 
      * Closes a meeting (usually when the user click on the close button from
      * the conferences listing.
      * @param string The internal ID of the meeting (id field for this meeting)
-     * @return void
+     * @return false|null
      * @assert (0) === false
      */
     public function endMeeting($id)

Spacing +36 added lines, -36 removed lines

@@ -210,7 +210,7 @@ 
 
         $params['record'] = api_get_course_setting('big_blue_button_record_and_store', $courseCode) == 1 ? true : false;
         $max = api_get_course_setting('big_blue_button_max_students_allowed', $courseCode);
-        $max =  isset($max) ? $max : -1;
+        $max = isset($max) ? $max : -1;
 
         $params['status'] = 1;
         // Generate a pseudo-global-unique-id to avoid clash of conferences on
@@ -243,23 +243,23 @@ 
             $duration = 300;
 
             $bbbParams = array(
-                'meetingId' => $params['remote_id'], 					// REQUIRED
-                'meetingName' => $meetingName, 	// REQUIRED
-                'attendeePw' => $attendeePassword, 					// Match this value in getJoinMeetingURL() to join as attendee.
-                'moderatorPw' => $moderatorPassword, 					// Match this value in getJoinMeetingURL() to join as moderator.
-                'welcomeMsg' => $welcomeMessage, 					// ''= use default. Change to customize.
-                'dialNumber' => '', 					// The main number to call into. Optional.
-                'voiceBridge' => $params['voice_bridge'], 					// PIN to join voice. Required.
-                'webVoice' => '', 						// Alphanumeric to join voice. Optional.
+                'meetingId' => $params['remote_id'], // REQUIRED
+                'meetingName' => $meetingName, // REQUIRED
+                'attendeePw' => $attendeePassword, // Match this value in getJoinMeetingURL() to join as attendee.
+                'moderatorPw' => $moderatorPassword, // Match this value in getJoinMeetingURL() to join as moderator.
+                'welcomeMsg' => $welcomeMessage, // ''= use default. Change to customize.
+                'dialNumber' => '', // The main number to call into. Optional.
+                'voiceBridge' => $params['voice_bridge'], // PIN to join voice. Required.
+                'webVoice' => '', // Alphanumeric to join voice. Optional.
                 'logoutUrl' =>  $this->logoutUrl,
-                'maxParticipants' => $max, 				// Optional. -1 = unlimitted. Not supported in BBB. [number]
-                'record' => $record, 					// New. 'true' will tell BBB to record the meeting.
-                'duration' => $duration, 				// Default = 0 which means no set duration in minutes. [number]
+                'maxParticipants' => $max, // Optional. -1 = unlimitted. Not supported in BBB. [number]
+                'record' => $record, // New. 'true' will tell BBB to record the meeting.
+                'duration' => $duration, // Default = 0 which means no set duration in minutes. [number]
                 //'meta_category' => '', 				// Use to pass additional info to BBB server. See API docs.
             );
 
             if ($this->debug) {
-                error_log("create_meeting params: ".print_r($bbbParams,1));
+                error_log("create_meeting params: ".print_r($bbbParams, 1));
             }
 
             $status = false;
@@ -272,7 +272,7 @@ 
                 if (isset($result) && strval($result['returncode']) == 'SUCCESS') {
                     if ($this->debug) {
                         error_log(
-                            "create_meeting result: " . print_r($result, 1)
+                            "create_meeting result: ".print_r($result, 1)
                         );
                     }
                     $meeting = $this->joinMeeting($meetingName, true);
@@ -303,7 +303,7 @@ 
 
         $courseId = api_get_course_int_id();
         $sessionId = api_get_session_id();
-        $conditions =  array(
+        $conditions = array(
             'where' => array(
                 'c_id = ? AND session_id = ? AND meeting_name = ? AND status = 1 AND access_url = ?' =>
                     array($courseId, $sessionId, $meetingName, $this->accessUrl)
@@ -312,7 +312,7 @@ 
 
         if ($this->hasGroupSupport()) {
             $groupId = api_get_group_id();
-            $conditions =  array(
+            $conditions = array(
                 'where' => array(
                     'c_id = ? AND session_id = ? AND meeting_name = ? AND group_id = ? AND status = 1 AND access_url = ?' =>
                         array($courseId, $sessionId, $meetingName, $groupId, $this->accessUrl)
@@ -409,7 +409,7 @@ 
 
             if ($this->debug) {
                 error_log(
-                    "meeting is running: " . intval($meetingInfoExists)
+                    "meeting is running: ".intval($meetingInfoExists)
                 );
             }
 
@@ -426,11 +426,11 @@ 
 
         if ($meetingInfoExists) {
             $joinParams = array(
-                'meetingId' => $meetingData['remote_id'],	//	-- REQUIRED - A unique id for the meeting
-                'username' => $this->userCompleteName,	//-- REQUIRED - The name that will display for the user in the meeting
-                'password' => $pass,			//-- REQUIRED - The attendee or moderator password, depending on what's passed here
+                'meetingId' => $meetingData['remote_id'], //	-- REQUIRED - A unique id for the meeting
+                'username' => $this->userCompleteName, //-- REQUIRED - The name that will display for the user in the meeting
+                'password' => $pass, //-- REQUIRED - The attendee or moderator password, depending on what's passed here
                 //'createTime' => api_get_utc_datetime(),			//-- OPTIONAL - string. Leave blank ('') unless you set this correctly.
-                'userID' => api_get_user_id(),				//-- OPTIONAL - string
+                'userID' => api_get_user_id(), //-- OPTIONAL - string
                 'webVoiceConf' => ''	//	-- OPTIONAL - string
             );
             $url = $this->api->getJoinMeetingURL($joinParams);
@@ -439,7 +439,7 @@ 
             $url = $this->logoutUrl;
         }
         if ($this->debug) {
-            error_log("return url :" . $url);
+            error_log("return url :".$url);
         }
 
         return $url;
@@ -480,7 +480,7 @@ 
      */
     public function getAllMeetingsInCourse($courseId, $sessionId, $status)
     {
-        $conditions =  array(
+        $conditions = array(
             'where' => array(
                 'status = ? AND c_id = ? AND session_id = ? ' => array(
                     $status,
@@ -512,7 +512,7 @@ 
         $courseId = api_get_course_int_id();
         $sessionId = api_get_session_id();
 
-        $conditions =  array(
+        $conditions = array(
             'where' => array(
                 'c_id = ? AND session_id = ? AND access_url = ?' => array(
                     $courseId,
@@ -524,7 +524,7 @@ 
 
         if ($this->hasGroupSupport()) {
             $groupId = api_get_group_id();
-            $conditions =  array(
+            $conditions = array(
                 'where' => array(
                     'c_id = ? AND session_id = ? AND group_id = ? AND access_url = ?' =>
                         array($courseId, $sessionId, $groupId, $this->accessUrl)
@@ -533,7 +533,7 @@ 
         }
 
         if ($isGlobalPerUser) {
-            $conditions =  array(
+            $conditions = array(
                 'where' => array(
                     'user_id = ? AND access_url = ?' =>
                         array($this->userId, $this->accessUrl)
@@ -568,7 +568,7 @@ 
             }
             $meetingBBB['end_url'] = $this->endUrl($meetingDB);
 
-            if (isset($meetingBBB['returncode']) && (string)$meetingBBB['returncode'] == 'FAILED') {
+            if (isset($meetingBBB['returncode']) && (string) $meetingBBB['returncode'] == 'FAILED') {
                 if ($meetingDB['status'] == 1 && $this->isConferenceManager()) {
                     $this->endMeeting($meetingDB['id']);
                 }
@@ -737,7 +737,7 @@ 
                     $actionLinksArray[] = $actionLinks;
                     $item['action_links'] = implode('<br />', $actionLinksArray);
                 }
-                $item['show_links']  = implode('<br />', $recordArray);
+                $item['show_links'] = implode('<br />', $recordArray);
                 $item['action_links'] = implode('<br />', $actionLinksArray);
             }
 
@@ -750,11 +750,11 @@ 
 
             if ($meetingDB['status'] == 1) {
                 $joinParams = array(
-                    'meetingId' => $meetingDB['remote_id'],		//-- REQUIRED - A unique id for the meeting
-                    'username' => $this->userCompleteName,	//-- REQUIRED - The name that will display for the user in the meeting
-                    'password' => $pass,			//-- REQUIRED - The attendee or moderator password, depending on what's passed here
-                    'createTime' => '',			//-- OPTIONAL - string. Leave blank ('') unless you set this correctly.
-                    'userID' => '',			//	-- OPTIONAL - string
+                    'meetingId' => $meetingDB['remote_id'], //-- REQUIRED - A unique id for the meeting
+                    'username' => $this->userCompleteName, //-- REQUIRED - The name that will display for the user in the meeting
+                    'password' => $pass, //-- REQUIRED - The attendee or moderator password, depending on what's passed here
+                    'createTime' => '', //-- OPTIONAL - string. Leave blank ('') unless you set this correctly.
+                    'userID' => '', //	-- OPTIONAL - string
                     'webVoiceConf' => ''	//	-- OPTIONAL - string
                 );
                 $item['go_url'] = $this->protocol.$this->api->getJoinMeetingURL($joinParams);
@@ -815,8 +815,8 @@ 
         $pass = $this->getUserMeetingPassword();
 
         $endParams = array(
-            'meetingId' => $meetingData['remote_id'],   // REQUIRED - We have to know which meeting to end.
-            'password' => $pass,        // REQUIRED - Must match moderator pass for meeting.
+            'meetingId' => $meetingData['remote_id'], // REQUIRED - We have to know which meeting to end.
+            'password' => $pass, // REQUIRED - Must match moderator pass for meeting.
         );
         $this->api->endMeetingWithXmlResponseArray($endParams);
         Database::update(
@@ -1067,7 +1067,7 @@ 
      */
     public function redirectToBBB($url)
     {
-        if (file_exists(__DIR__ . '/../config.vm.php')) {
+        if (file_exists(__DIR__.'/../config.vm.php')) {
             // Using VM
             echo Display::url(get_lang('ClickToContinue'), $url);
             exit;

plugin/bbb/lib/bbb_api.php

Doc Comments +3 added lines

@@ -59,6 +59,9 @@
 		$this->_bbbServerBaseUrl 	= CONFIG_SERVER_BASE_URL;
 	}
 
+	/**
+	 * @param string $url
+	 */
 	private function _processXmlResponse($url){
 	/*
 	A private utility method used by other public methods to process XML responses.

Spacing +21 added lines, -21 removed lines

@@ -55,25 +55,25 @@ 
 	*/
 		// BASE CONFIGS - set these for your BBB server in config.php and they will
 		// simply flow in here via the constants:
-		$this->_securitySalt 		= CONFIG_SECURITY_SALT;
-		$this->_bbbServerBaseUrl 	= CONFIG_SERVER_BASE_URL;
+		$this->_securitySalt = CONFIG_SECURITY_SALT;
+		$this->_bbbServerBaseUrl = CONFIG_SERVER_BASE_URL;
 	}
 
-	private function _processXmlResponse($url){
+	private function _processXmlResponse($url) {
 	/*
 	A private utility method used by other public methods to process XML responses.
 	*/
 		if (extension_loaded('curl')) {
-			$ch = curl_init() or die ( curl_error($ch) );
+			$ch = curl_init() or die (curl_error($ch));
 			$timeout = 10;
-			curl_setopt( $ch, CURLOPT_SSL_VERIFYPEER, false);
-			curl_setopt( $ch, CURLOPT_URL, $url );
-			curl_setopt( $ch, CURLOPT_RETURNTRANSFER, 1 );
-			curl_setopt( $ch, CURLOPT_CONNECTTIMEOUT, $timeout);
-			$data = curl_exec( $ch );
-			curl_close( $ch );
-
-			if($data)
+			curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, false);
+			curl_setopt($ch, CURLOPT_URL, $url);
+			curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
+			curl_setopt($ch, CURLOPT_CONNECTTIMEOUT, $timeout);
+			$data = curl_exec($ch);
+			curl_close($ch);
+
+			if ($data)
 				return (new SimpleXMLElement($data));
 			else
 				return false;
@@ -137,10 +137,10 @@ 
 		'&duration='.urlencode($creationParams['duration']);
 		//'&meta_category='.urlencode($creationParams['meta_category']);
 		$welcomeMessage = $creationParams['welcomeMsg'];
-		if(trim($welcomeMessage))
+		if (trim($welcomeMessage))
 			$params .= '&welcome='.urlencode($welcomeMessage);
 		// Return the complete URL:
-		return ( $creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt) );
+		return ($creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt));
 	}
 
 	public function createMeetingWithXmlResponseArray($creationParams) {
@@ -165,7 +165,7 @@ 
 		$xml = $this->_processXmlResponse($this->getCreateMeetingURL($creationParams));
 
         if ($xml) {
-			if($xml->meetingID)
+			if ($xml->meetingID)
 				return array(
 					'returncode' => $xml->returncode,
 					'message' => $xml->message,
@@ -283,7 +283,7 @@ 
 		$meetingId = '1234'		-- REQUIRED - The unique id for the meeting
 		*/
 		$xml = $this->_processXmlResponse($this->getIsMeetingRunningUrl($meetingId));
-		if($xml) {
+		if ($xml) {
 			return array(
 				'returncode' => $xml->returncode,
 				'running' => $xml->running 	// -- Returns true/false.
@@ -310,7 +310,7 @@ 
 		and then handle the results that we get in the XML response.
 		*/
 		$xml = $this->_processXmlResponse($this->getGetMeetingsUrl());
-		if($xml) {
+		if ($xml) {
 			// If we don't get a success code, stop processing and return just the returncode:
 			if ($xml->returncode != 'SUCCESS') {
 				$result = array(
@@ -379,7 +379,7 @@ 
 		);
 		*/
 		$xml = $this->_processXmlResponse($this->getMeetingInfoUrl($infoParams));
-		if($xml) {
+		if ($xml) {
 			// If we don't get a success code or messageKey, find out why:
 			if (($xml->returncode != 'SUCCESS') || ($xml->messageKey == null)) {
 				$result = array(
@@ -454,7 +454,7 @@ 
 		probably be required in user code when 'recording' is set to true.
 		*/
 		$xml = $this->_processXmlResponse($this->getRecordingsUrl($recordingParams));
-		if($xml) {
+		if ($xml) {
 			// If we don't get a success code or messageKey, find out why:
 			if (($xml->returncode != 'SUCCESS') || ($xml->messageKey == null)) {
 				$result = array(
@@ -524,7 +524,7 @@ 
 		);
 		*/
 		$xml = $this->_processXmlResponse($this->getPublishRecordingsUrl($recordingParams));
-		if($xml) {
+		if ($xml) {
 			return array(
 				'returncode' => $xml->returncode,
 				'published' => $xml->published 	// -- Returns true/false.
@@ -557,7 +557,7 @@ 
 		*/
 
 		$xml = $this->_processXmlResponse($this->getDeleteRecordingsUrl($recordingParams));
-		if($xml) {
+		if ($xml) {
 			return array(
 				'returncode' => $xml->returncode,
 				'deleted' => $xml->deleted 	// -- Returns true/false.

Braces +28 added lines, -40 removed lines

@@ -73,10 +73,11 @@ 
 			$data = curl_exec( $ch );
 			curl_close( $ch );
 
-			if($data)
-				return (new SimpleXMLElement($data));
-			else
-				return false;
+			if($data) {
+							return (new SimpleXMLElement($data));
+			} else {
+							return false;
+			}
 		}
 		return (simplexml_load_file($url));
 	}
@@ -85,11 +86,9 @@ 
 		/* Process required params and throw errors if we don't get values */
 		if ((isset($param)) && ($param != '')) {
 			return $param;
-		}
-		elseif (!isset($param)) {
+		} elseif (!isset($param)) {
 			throw new Exception('Missing parameter.');
-		}
-		else {
+		} else {
 			throw new Exception(''.$param.' is required.');
 		}
 	}
@@ -99,8 +98,7 @@ 
 		/* Don't know if we'll use this one, but let's build it in case. */
 		if ((isset($param)) && ($param != '')) {
 			return $param;
-		}
-		else {
+		} else {
 			$param = '';
 			return $param;
 		}
@@ -137,8 +135,9 @@ 
 		'&duration='.urlencode($creationParams['duration']);
 		//'&meta_category='.urlencode($creationParams['meta_category']);
 		$welcomeMessage = $creationParams['welcomeMsg'];
-		if(trim($welcomeMessage))
-			$params .= '&welcome='.urlencode($welcomeMessage);
+		if(trim($welcomeMessage)) {
+					$params .= '&welcome='.urlencode($welcomeMessage);
+		}
 		// Return the complete URL:
 		return ( $creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt) );
 	}
@@ -165,8 +164,8 @@ 
 		$xml = $this->_processXmlResponse($this->getCreateMeetingURL($creationParams));
 
         if ($xml) {
-			if($xml->meetingID)
-				return array(
+			if($xml->meetingID) {
+							return array(
 					'returncode' => $xml->returncode,
 					'message' => $xml->message,
 					'messageKey' => $xml->messageKey,
@@ -176,14 +175,14 @@ 
 					'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded,
 					'createTime' => $xml->createTime
 					);
-			else
-				return array(
+			} else {
+							return array(
 					'returncode' => $xml->returncode,
 					'message' => $xml->message,
 					'messageKey' => $xml->messageKey
 					);
-		}
-		else {
+			}
+		} else {
 			return null;
 		}
 	}
@@ -253,8 +252,7 @@ 
 				'message' => $xml->message,
 				'messageKey' => $xml->messageKey
 				);
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -288,8 +286,7 @@ 
 				'returncode' => $xml->returncode,
 				'running' => $xml->running 	// -- Returns true/false.
 				);
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -317,8 +314,7 @@ 
 					'returncode' => $xml->returncode
 				);
 				return $result;
-			}
-			elseif ($xml->messageKey == 'noMeetings') {
+			} elseif ($xml->messageKey == 'noMeetings') {
 				/* No meetings on server, so return just this info: */
 				$result = array(
 					'returncode' => $xml->returncode,
@@ -326,8 +322,7 @@ 
 					'message' => $xml->message
 				);
 				return $result;
-			}
-			else {
+			} else {
 				// In this case, we have success and meetings. First return general response:
 				$result = array(
 					'returncode' => $xml->returncode,
@@ -348,8 +343,7 @@ 
 					}
 				return $result;
 			}
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -388,8 +382,7 @@ 
 					'message' => $xml->message
 				);
 				return $result;
-			}
-			else {
+			} else {
 				// In this case, we have success and meeting info:
 				$result = array(
 					'returncode' => $xml->returncode,
@@ -418,8 +411,7 @@ 
 					}
 				return $result;
 			}
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -463,8 +455,7 @@ 
 					'message' => $xml->message
 				);
 				return $result;
-			}
-			else {
+			} else {
 				// In this case, we have success and recording info:
 				$result = array(
 					'returncode' => $xml->returncode,
@@ -495,8 +486,7 @@ 
 					}
 				return $result;
 			}
-		}
-		else {
+		} else {
 			return null;
 		}
 	}
@@ -529,8 +519,7 @@ 
 				'returncode' => $xml->returncode,
 				'published' => $xml->published 	// -- Returns true/false.
 				);
-		}
-		else {
+		} else {
 			return null;
 		}
 
@@ -562,8 +551,7 @@ 
 				'returncode' => $xml->returncode,
 				'deleted' => $xml->deleted 	// -- Returns true/false.
 				);
-		}
-		else {
+		} else {
 			return null;
 		}
 

Indentation +403 added lines, -403 removed lines

@@ -44,107 +44,107 @@ 
 
 class BigBlueButtonBN {
 
-	private $_securitySalt;
-	private $_bbbServerBaseUrl;
+    private $_securitySalt;
+    private $_bbbServerBaseUrl;
 
-	/* ___________ General Methods for the BigBlueButton Class __________ */
+    /* ___________ General Methods for the BigBlueButton Class __________ */
 
-	function __construct() {
-	/*
+    function __construct() {
+    /*
 	Establish just our basic elements in the constructor:
 	*/
-		// BASE CONFIGS - set these for your BBB server in config.php and they will
-		// simply flow in here via the constants:
-		$this->_securitySalt 		= CONFIG_SECURITY_SALT;
-		$this->_bbbServerBaseUrl 	= CONFIG_SERVER_BASE_URL;
-	}
-
-	private function _processXmlResponse($url){
-	/*
+        // BASE CONFIGS - set these for your BBB server in config.php and they will
+        // simply flow in here via the constants:
+        $this->_securitySalt 		= CONFIG_SECURITY_SALT;
+        $this->_bbbServerBaseUrl 	= CONFIG_SERVER_BASE_URL;
+    }
+
+    private function _processXmlResponse($url){
+    /*
 	A private utility method used by other public methods to process XML responses.
 	*/
-		if (extension_loaded('curl')) {
-			$ch = curl_init() or die ( curl_error($ch) );
-			$timeout = 10;
-			curl_setopt( $ch, CURLOPT_SSL_VERIFYPEER, false);
-			curl_setopt( $ch, CURLOPT_URL, $url );
-			curl_setopt( $ch, CURLOPT_RETURNTRANSFER, 1 );
-			curl_setopt( $ch, CURLOPT_CONNECTTIMEOUT, $timeout);
-			$data = curl_exec( $ch );
-			curl_close( $ch );
-
-			if($data)
-				return (new SimpleXMLElement($data));
-			else
-				return false;
-		}
-		return (simplexml_load_file($url));
-	}
-
-	private function _requiredParam($param) {
-		/* Process required params and throw errors if we don't get values */
-		if ((isset($param)) && ($param != '')) {
-			return $param;
-		}
-		elseif (!isset($param)) {
-			throw new Exception('Missing parameter.');
-		}
-		else {
-			throw new Exception(''.$param.' is required.');
-		}
-	}
-
-	private function _optionalParam($param) {
-		/* Pass most optional params through as set value, or set to '' */
-		/* Don't know if we'll use this one, but let's build it in case. */
-		if ((isset($param)) && ($param != '')) {
-			return $param;
-		}
-		else {
-			$param = '';
-			return $param;
-		}
-	}
-
-	/* __________________ BBB ADMINISTRATION METHODS _________________ */
-	/* The methods in the following section support the following categories of the BBB API:
+        if (extension_loaded('curl')) {
+            $ch = curl_init() or die ( curl_error($ch) );
+            $timeout = 10;
+            curl_setopt( $ch, CURLOPT_SSL_VERIFYPEER, false);
+            curl_setopt( $ch, CURLOPT_URL, $url );
+            curl_setopt( $ch, CURLOPT_RETURNTRANSFER, 1 );
+            curl_setopt( $ch, CURLOPT_CONNECTTIMEOUT, $timeout);
+            $data = curl_exec( $ch );
+            curl_close( $ch );
+
+            if($data)
+                return (new SimpleXMLElement($data));
+            else
+                return false;
+        }
+        return (simplexml_load_file($url));
+    }
+
+    private function _requiredParam($param) {
+        /* Process required params and throw errors if we don't get values */
+        if ((isset($param)) && ($param != '')) {
+            return $param;
+        }
+        elseif (!isset($param)) {
+            throw new Exception('Missing parameter.');
+        }
+        else {
+            throw new Exception(''.$param.' is required.');
+        }
+    }
+
+    private function _optionalParam($param) {
+        /* Pass most optional params through as set value, or set to '' */
+        /* Don't know if we'll use this one, but let's build it in case. */
+        if ((isset($param)) && ($param != '')) {
+            return $param;
+        }
+        else {
+            $param = '';
+            return $param;
+        }
+    }
+
+    /* __________________ BBB ADMINISTRATION METHODS _________________ */
+    /* The methods in the following section support the following categories of the BBB API:
 	-- create
 	-- join
 	-- end
 	*/
 
-	public function getCreateMeetingUrl($creationParams) {
-		/*
+    public function getCreateMeetingUrl($creationParams) {
+        /*
 		USAGE:
 		(see $creationParams array in createMeetingArray method.)
 		*/
-		$this->_meetingId = $this->_requiredParam($creationParams['meetingId']);
-		$this->_meetingName = $this->_requiredParam($creationParams['meetingName']);
-		// Set up the basic creation URL:
-		$creationUrl = $this->_bbbServerBaseUrl."api/create?";
-		// Add params:
-		$params =
-		'name='.urlencode($this->_meetingName).
-		'&meetingID='.urlencode($this->_meetingId).
-		'&attendeePW='.urlencode($creationParams['attendeePw']).
-		'&moderatorPW='.urlencode($creationParams['moderatorPw']).
-		'&dialNumber='.urlencode($creationParams['dialNumber']).
-		'&voiceBridge='.urlencode($creationParams['voiceBridge']).
-		'&webVoice='.urlencode($creationParams['webVoice']).
-		'&logoutURL='.urlencode($creationParams['logoutUrl']).
-		'&maxParticipants='.urlencode($creationParams['maxParticipants']).
-		'&record='.urlencode($creationParams['record']).
-		'&duration='.urlencode($creationParams['duration']);
-		//'&meta_category='.urlencode($creationParams['meta_category']);
-		$welcomeMessage = $creationParams['welcomeMsg'];
-		if(trim($welcomeMessage))
-			$params .= '&welcome='.urlencode($welcomeMessage);
-		// Return the complete URL:
-		return ( $creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt) );
-	}
-
-	public function createMeetingWithXmlResponseArray($creationParams) {
-		/*
+        $this->_meetingId = $this->_requiredParam($creationParams['meetingId']);
+        $this->_meetingName = $this->_requiredParam($creationParams['meetingName']);
+        // Set up the basic creation URL:
+        $creationUrl = $this->_bbbServerBaseUrl."api/create?";
+        // Add params:
+        $params =
+        'name='.urlencode($this->_meetingName).
+        '&meetingID='.urlencode($this->_meetingId).
+        '&attendeePW='.urlencode($creationParams['attendeePw']).
+        '&moderatorPW='.urlencode($creationParams['moderatorPw']).
+        '&dialNumber='.urlencode($creationParams['dialNumber']).
+        '&voiceBridge='.urlencode($creationParams['voiceBridge']).
+        '&webVoice='.urlencode($creationParams['webVoice']).
+        '&logoutURL='.urlencode($creationParams['logoutUrl']).
+        '&maxParticipants='.urlencode($creationParams['maxParticipants']).
+        '&record='.urlencode($creationParams['record']).
+        '&duration='.urlencode($creationParams['duration']);
+        //'&meta_category='.urlencode($creationParams['meta_category']);
+        $welcomeMessage = $creationParams['welcomeMsg'];
+        if(trim($welcomeMessage))
+            $params .= '&welcome='.urlencode($welcomeMessage);
+        // Return the complete URL:
+        return ( $creationUrl.$params.'&checksum='.sha1("create".$params.$this->_securitySalt) );
+    }
+
+    public function createMeetingWithXmlResponseArray($creationParams) {
+        /*
 		USAGE:
 		$creationParams = array(
 			'name' => 'Meeting Name',	-- A name for the meeting (or username)
@@ -162,34 +162,34 @@ 
 			'meta_category' => '', 		-- Use to pass additional info to BBB server. See API docs to enable.
 		);
 		*/
-		$xml = $this->_processXmlResponse($this->getCreateMeetingURL($creationParams));
+        $xml = $this->_processXmlResponse($this->getCreateMeetingURL($creationParams));
 
         if ($xml) {
-			if($xml->meetingID)
-				return array(
-					'returncode' => $xml->returncode->__toString(),
-					'message' => $xml->message->__toString(),
-					'messageKey' => $xml->messageKey->__toString(),
-					'meetingId' => $xml->meetingID->__toString(),
-					'attendeePw' => $xml->attendeePW->__toString(),
-					'moderatorPw' => $xml->moderatorPW->__toString(),
-					'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded->__toString(),
-					'createTime' => $xml->createTime->__toString()
-					);
-			else
-				return array(
-					'returncode' => $xml->returncode->__toString(),
-					'message' => $xml->message->__toString(),
-					'messageKey' => $xml->messageKey->__toString()
-					);
-		}
-		else {
-			return null;
-		}
-	}
-
-	public function getJoinMeetingURL($joinParams) {
-		/*
+            if($xml->meetingID)
+                return array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'message' => $xml->message->__toString(),
+                    'messageKey' => $xml->messageKey->__toString(),
+                    'meetingId' => $xml->meetingID->__toString(),
+                    'attendeePw' => $xml->attendeePW->__toString(),
+                    'moderatorPw' => $xml->moderatorPW->__toString(),
+                    'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded->__toString(),
+                    'createTime' => $xml->createTime->__toString()
+                    );
+            else
+                return array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'message' => $xml->message->__toString(),
+                    'messageKey' => $xml->messageKey->__toString()
+                    );
+        }
+        else {
+            return null;
+        }
+    }
+
+    public function getJoinMeetingURL($joinParams) {
+        /*
 		NOTE: At this point, we don't use a corresponding joinMeetingWithXmlResponse here because the API
 		doesn't respond on success, but you can still code that method if you need it. Or, you can take the URL
 		that's returned from this method and simply send your users off to that URL in your code.
@@ -203,249 +203,249 @@ 
 			'webVoiceConf' => ''		-- OPTIONAL - string
 		);
 		*/
-		$this->_meetingId = $this->_requiredParam($joinParams['meetingId']);
-		$this->_username = $this->_requiredParam($joinParams['username']);
-		$this->_password = $this->_requiredParam($joinParams['password']);
-		// Establish the basic join URL:
-		$joinUrl = $this->_bbbServerBaseUrl."api/join?";
-		// Add parameters to the URL:
-		$params =
-		'meetingID='.urlencode($this->_meetingId).
-		'&fullName='.urlencode($this->_username).
-		'&password='.urlencode($this->_password).
-		'&userID='.urlencode($joinParams['userID']).
-		'&webVoiceConf='.urlencode($joinParams['webVoiceConf']);
-		// Only use createTime if we really want to use it. If it's '', then don't pass it:
-		if (((isset($joinParams['createTime'])) && ($joinParams['createTime'] != ''))) {
-			$params .= '&createTime='.urlencode($joinParams['createTime']);
-		}
-		// Return the URL:
-		return ($joinUrl.$params.'&checksum='.sha1("join".$params.$this->_securitySalt));
-	}
-
-	public function getEndMeetingURL($endParams) {
-		/* USAGE:
+        $this->_meetingId = $this->_requiredParam($joinParams['meetingId']);
+        $this->_username = $this->_requiredParam($joinParams['username']);
+        $this->_password = $this->_requiredParam($joinParams['password']);
+        // Establish the basic join URL:
+        $joinUrl = $this->_bbbServerBaseUrl."api/join?";
+        // Add parameters to the URL:
+        $params =
+        'meetingID='.urlencode($this->_meetingId).
+        '&fullName='.urlencode($this->_username).
+        '&password='.urlencode($this->_password).
+        '&userID='.urlencode($joinParams['userID']).
+        '&webVoiceConf='.urlencode($joinParams['webVoiceConf']);
+        // Only use createTime if we really want to use it. If it's '', then don't pass it:
+        if (((isset($joinParams['createTime'])) && ($joinParams['createTime'] != ''))) {
+            $params .= '&createTime='.urlencode($joinParams['createTime']);
+        }
+        // Return the URL:
+        return ($joinUrl.$params.'&checksum='.sha1("join".$params.$this->_securitySalt));
+    }
+
+    public function getEndMeetingURL($endParams) {
+        /* USAGE:
 		$endParams = array (
 			'meetingId' => '1234',		-- REQUIRED - The unique id for the meeting
 			'password' => 'mp'			-- REQUIRED - The moderator password for the meeting
 		);
 		*/
-		$this->_meetingId = $this->_requiredParam($endParams['meetingId']);
-		$this->_password = $this->_requiredParam($endParams['password']);
-		$endUrl = $this->_bbbServerBaseUrl."api/end?";
-		$params =
-		'meetingID='.urlencode($this->_meetingId).
-		'&password='.urlencode($this->_password);
-		return ($endUrl.$params.'&checksum='.sha1("end".$params.$this->_securitySalt));
-	}
-
-	public function endMeetingWithXmlResponseArray($endParams) {
-		/* USAGE:
+        $this->_meetingId = $this->_requiredParam($endParams['meetingId']);
+        $this->_password = $this->_requiredParam($endParams['password']);
+        $endUrl = $this->_bbbServerBaseUrl."api/end?";
+        $params =
+        'meetingID='.urlencode($this->_meetingId).
+        '&password='.urlencode($this->_password);
+        return ($endUrl.$params.'&checksum='.sha1("end".$params.$this->_securitySalt));
+    }
+
+    public function endMeetingWithXmlResponseArray($endParams) {
+        /* USAGE:
 		$endParams = array (
 			'meetingId' => '1234',		-- REQUIRED - The unique id for the meeting
 			'password' => 'mp'			-- REQUIRED - The moderator password for the meeting
 		);
 		*/
-		$xml = $this->_processXmlResponse($this->getEndMeetingURL($endParams));
-		if ($xml) {
-			return array(
-				'returncode' => $xml->returncode->__toString(),
-				'message' => $xml->message->__toString(),
-				'messageKey' => $xml->messageKey->__toString()
-				);
-		}
-		else {
-			return null;
-		}
-
-	}
-
-	/* __________________ BBB MONITORING METHODS _________________ */
-	/* The methods in the following section support the following categories of the BBB API:
+        $xml = $this->_processXmlResponse($this->getEndMeetingURL($endParams));
+        if ($xml) {
+            return array(
+                'returncode' => $xml->returncode->__toString(),
+                'message' => $xml->message->__toString(),
+                'messageKey' => $xml->messageKey->__toString()
+                );
+        }
+        else {
+            return null;
+        }
+
+    }
+
+    /* __________________ BBB MONITORING METHODS _________________ */
+    /* The methods in the following section support the following categories of the BBB API:
 	-- isMeetingRunning
 	-- getMeetings
 	-- getMeetingInfo
 	*/
 
-	public function getIsMeetingRunningUrl($meetingId) {
-		/* USAGE:
+    public function getIsMeetingRunningUrl($meetingId) {
+        /* USAGE:
 		$meetingId = '1234'		-- REQUIRED - The unique id for the meeting
 		*/
-		$this->_meetingId = $this->_requiredParam($meetingId);
-		$runningUrl = $this->_bbbServerBaseUrl."api/isMeetingRunning?";
-		$params =
-		'meetingID='.urlencode($this->_meetingId);
-		return ($runningUrl.$params.'&checksum='.sha1("isMeetingRunning".$params.$this->_securitySalt));
-	}
-
-	public function isMeetingRunningWithXmlResponseArray($meetingId) {
-		/* USAGE:
+        $this->_meetingId = $this->_requiredParam($meetingId);
+        $runningUrl = $this->_bbbServerBaseUrl."api/isMeetingRunning?";
+        $params =
+        'meetingID='.urlencode($this->_meetingId);
+        return ($runningUrl.$params.'&checksum='.sha1("isMeetingRunning".$params.$this->_securitySalt));
+    }
+
+    public function isMeetingRunningWithXmlResponseArray($meetingId) {
+        /* USAGE:
 		$meetingId = '1234'		-- REQUIRED - The unique id for the meeting
 		*/
-		$xml = $this->_processXmlResponse($this->getIsMeetingRunningUrl($meetingId));
-		if($xml) {
-			return array(
-				'returncode' => $xml->returncode->__toString(),
-				'running' => $xml->running->__toString() 	// -- Returns true/false.
-			);
-		}
-		else {
-			return null;
-		}
-
-	}
-
-	public function getGetMeetingsUrl() {
-		/* Simply formulate the getMeetings URL
+        $xml = $this->_processXmlResponse($this->getIsMeetingRunningUrl($meetingId));
+        if($xml) {
+            return array(
+                'returncode' => $xml->returncode->__toString(),
+                'running' => $xml->running->__toString() 	// -- Returns true/false.
+            );
+        }
+        else {
+            return null;
+        }
+
+    }
+
+    public function getGetMeetingsUrl() {
+        /* Simply formulate the getMeetings URL
 		We do this in a separate function so we have the option to just get this
 		URL and print it if we want for some reason.
 		*/
-		$getMeetingsUrl = $this->_bbbServerBaseUrl."api/getMeetings?checksum=".sha1("getMeetings".$this->_securitySalt);
-		return $getMeetingsUrl;
-	}
+        $getMeetingsUrl = $this->_bbbServerBaseUrl."api/getMeetings?checksum=".sha1("getMeetings".$this->_securitySalt);
+        return $getMeetingsUrl;
+    }
 
-	public function getMeetingsWithXmlResponseArray() {
-		/* USAGE:
+    public function getMeetingsWithXmlResponseArray() {
+        /* USAGE:
 		We don't need to pass any parameters with this one, so we just send the query URL off to BBB
 		and then handle the results that we get in the XML response.
 		*/
-		$xml = $this->_processXmlResponse($this->getGetMeetingsUrl());
-		if($xml) {
-			// If we don't get a success code, stop processing and return just the returncode:
-			if ($xml->returncode != 'SUCCESS') {
-				$result = array(
-					'returncode' => $xml->returncode->__toString()
-				);
-				return $result;
-			}
-			elseif ($xml->messageKey == 'noMeetings') {
-				/* No meetings on server, so return just this info: */
-				$result = array(
-					'returncode' => $xml->returncode->__toString(),
-					'messageKey' => $xml->messageKey->__toString(),
-					'message' => $xml->message->__toString()
-				);
-				return $result;
-			}
-			else {
-				// In this case, we have success and meetings. First return general response:
-				$result = array(
-					'returncode' => $xml->returncode->__toString(),
-					'messageKey' => $xml->messageKey->__toString(),
-					'message' => $xml->message->__toString()
-				);
-				// Then interate through meeting results and return them as part of the array:
-				foreach ($xml->meetings->meeting as $m) {
-					$result[] = array(
-						'meetingId' => $m->meetingID->__toString(),
-						'meetingName' => $m->meetingName->__toString(),
-						'createTime' => $m->createTime->__toString(),
-						'attendeePw' => $m->attendeePW->__toString(),
-						'moderatorPw' => $m->moderatorPW->__toString(),
-						'hasBeenForciblyEnded' => $m->hasBeenForciblyEnded->__toString(),
-						'running' => $m->running->__toString()
-					);
-				}
-				return $result;
-			}
-		}
-		else {
-			return null;
-		}
-
-	}
-
-	public function getMeetingInfoUrl($infoParams) {
-		/* USAGE:
+        $xml = $this->_processXmlResponse($this->getGetMeetingsUrl());
+        if($xml) {
+            // If we don't get a success code, stop processing and return just the returncode:
+            if ($xml->returncode != 'SUCCESS') {
+                $result = array(
+                    'returncode' => $xml->returncode->__toString()
+                );
+                return $result;
+            }
+            elseif ($xml->messageKey == 'noMeetings') {
+                /* No meetings on server, so return just this info: */
+                $result = array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'messageKey' => $xml->messageKey->__toString(),
+                    'message' => $xml->message->__toString()
+                );
+                return $result;
+            }
+            else {
+                // In this case, we have success and meetings. First return general response:
+                $result = array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'messageKey' => $xml->messageKey->__toString(),
+                    'message' => $xml->message->__toString()
+                );
+                // Then interate through meeting results and return them as part of the array:
+                foreach ($xml->meetings->meeting as $m) {
+                    $result[] = array(
+                        'meetingId' => $m->meetingID->__toString(),
+                        'meetingName' => $m->meetingName->__toString(),
+                        'createTime' => $m->createTime->__toString(),
+                        'attendeePw' => $m->attendeePW->__toString(),
+                        'moderatorPw' => $m->moderatorPW->__toString(),
+                        'hasBeenForciblyEnded' => $m->hasBeenForciblyEnded->__toString(),
+                        'running' => $m->running->__toString()
+                    );
+                }
+                return $result;
+            }
+        }
+        else {
+            return null;
+        }
+
+    }
+
+    public function getMeetingInfoUrl($infoParams) {
+        /* USAGE:
 		$infoParams = array(
 			'meetingId' => '1234',		-- REQUIRED - The unique id for the meeting
 			'password' => 'mp'			-- REQUIRED - The moderator password for the meeting
 		);
 		*/
-		$this->_meetingId = $this->_requiredParam($infoParams['meetingId']);
-		$this->_password = $this->_requiredParam($infoParams['password']);
-		$infoUrl = $this->_bbbServerBaseUrl."api/getMeetingInfo?";
-		$params =
-		'meetingID='.urlencode($this->_meetingId).
-		'&password='.urlencode($this->_password);
-		return ($infoUrl.$params.'&checksum='.sha1("getMeetingInfo".$params.$this->_securitySalt));
-	}
-
-	public function getMeetingInfoWithXmlResponseArray($infoParams) {
-		/* USAGE:
+        $this->_meetingId = $this->_requiredParam($infoParams['meetingId']);
+        $this->_password = $this->_requiredParam($infoParams['password']);
+        $infoUrl = $this->_bbbServerBaseUrl."api/getMeetingInfo?";
+        $params =
+        'meetingID='.urlencode($this->_meetingId).
+        '&password='.urlencode($this->_password);
+        return ($infoUrl.$params.'&checksum='.sha1("getMeetingInfo".$params.$this->_securitySalt));
+    }
+
+    public function getMeetingInfoWithXmlResponseArray($infoParams) {
+        /* USAGE:
 		$infoParams = array(
 			'meetingId' => '1234',		-- REQUIRED - The unique id for the meeting
 			'password' => 'mp'			-- REQUIRED - The moderator password for the meeting
 		);
 		*/
-		$xml = $this->_processXmlResponse($this->getMeetingInfoUrl($infoParams));
-		if($xml) {
-			// If we don't get a success code or messageKey, find out why:
-			if (($xml->returncode != 'SUCCESS') || ($xml->messageKey == null)) {
-				$result = array(
-					'returncode' => $xml->returncode->__toString(),
-					'messageKey' => $xml->messageKey->__toString(),
-					'message' => $xml->message->__toString()
-				);
-				return $result;
-			}
-			else {
-				// In this case, we have success and meeting info:
-				$result = array(
-					'returncode' => $xml->returncode->__toString(),
-					'meetingName' => $xml->meetingName->__toString(),
-					'meetingId' => $xml->meetingID->__toString(),
-					'createTime' => $xml->createTime->__toString(),
-					'voiceBridge' => $xml->voiceBridge->__toString(),
-					'attendeePw' => $xml->attendeePW->__toString(),
-					'moderatorPw' => $xml->moderatorPW->__toString(),
-					'running' => $xml->running->__toString(),
-					'recording' => $xml->recording->__toString(),
-					'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded->__toString(),
-					'startTime' => $xml->startTime->__toString(),
-					'endTime' => $xml->endTime->__toString(),
-					'participantCount' => $xml->participantCount->__toString(),
-					'maxUsers' => $xml->maxUsers->__toString(),
-					'moderatorCount' => $xml->moderatorCount->__toString(),
-				);
-				// Then interate through attendee results and return them as part of the array:
-				foreach ($xml->attendees->attendee as $a) {
-					$result[] = array(
-						'userId' => $a->userID->__toString(),
-						'fullName' => $a->fullName->__toString(),
-						'role' => $a->role->__toString()
-					);
-				}
-				return $result;
-			}
-		}
-		else {
-			return null;
-		}
-
-	}
-
-	/* __________________ BBB RECORDING METHODS _________________ */
-	/* The methods in the following section support the following categories of the BBB API:
+        $xml = $this->_processXmlResponse($this->getMeetingInfoUrl($infoParams));
+        if($xml) {
+            // If we don't get a success code or messageKey, find out why:
+            if (($xml->returncode != 'SUCCESS') || ($xml->messageKey == null)) {
+                $result = array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'messageKey' => $xml->messageKey->__toString(),
+                    'message' => $xml->message->__toString()
+                );
+                return $result;
+            }
+            else {
+                // In this case, we have success and meeting info:
+                $result = array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'meetingName' => $xml->meetingName->__toString(),
+                    'meetingId' => $xml->meetingID->__toString(),
+                    'createTime' => $xml->createTime->__toString(),
+                    'voiceBridge' => $xml->voiceBridge->__toString(),
+                    'attendeePw' => $xml->attendeePW->__toString(),
+                    'moderatorPw' => $xml->moderatorPW->__toString(),
+                    'running' => $xml->running->__toString(),
+                    'recording' => $xml->recording->__toString(),
+                    'hasBeenForciblyEnded' => $xml->hasBeenForciblyEnded->__toString(),
+                    'startTime' => $xml->startTime->__toString(),
+                    'endTime' => $xml->endTime->__toString(),
+                    'participantCount' => $xml->participantCount->__toString(),
+                    'maxUsers' => $xml->maxUsers->__toString(),
+                    'moderatorCount' => $xml->moderatorCount->__toString(),
+                );
+                // Then interate through attendee results and return them as part of the array:
+                foreach ($xml->attendees->attendee as $a) {
+                    $result[] = array(
+                        'userId' => $a->userID->__toString(),
+                        'fullName' => $a->fullName->__toString(),
+                        'role' => $a->role->__toString()
+                    );
+                }
+                return $result;
+            }
+        }
+        else {
+            return null;
+        }
+
+    }
+
+    /* __________________ BBB RECORDING METHODS _________________ */
+    /* The methods in the following section support the following categories of the BBB API:
 	-- getRecordings
 	-- publishRecordings
 	-- deleteRecordings
 	*/
 
-	public function getRecordingsUrl($recordingParams) {
-		/* USAGE:
+    public function getRecordingsUrl($recordingParams) {
+        /* USAGE:
 		$recordingParams = array(
 			'meetingId' => '1234',		-- OPTIONAL - comma separate if multiple ids
 		);
 		*/
-		$recordingsUrl = $this->_bbbServerBaseUrl."api/getRecordings?";
-		$params = 'meetingID='.urlencode($recordingParams['meetingId']);
-		return ($recordingsUrl.$params.'&checksum='.sha1("getRecordings".$params.$this->_securitySalt));
+        $recordingsUrl = $this->_bbbServerBaseUrl."api/getRecordings?";
+        $params = 'meetingID='.urlencode($recordingParams['meetingId']);
+        return ($recordingsUrl.$params.'&checksum='.sha1("getRecordings".$params.$this->_securitySalt));
 
-	}
+    }
 
-	public function getRecordingsWithXmlResponseArray($recordingParams) {
-		/* USAGE:
+    public function getRecordingsWithXmlResponseArray($recordingParams) {
+        /* USAGE:
 		$recordingParams = array(
 			'meetingId' => '1234',		-- OPTIONAL - comma separate if multiple ids
 		);
@@ -453,121 +453,121 @@ 
 		when creating a meeting, it will kick users out after the duration. Should
 		probably be required in user code when 'recording' is set to true.
 		*/
-		$xml = $this->_processXmlResponse($this->getRecordingsUrl($recordingParams));
-		if($xml) {
-			// If we don't get a success code or messageKey, find out why:
-			if (($xml->returncode != 'SUCCESS') || ($xml->messageKey == null)) {
-				$result = array(
-					'returncode' => $xml->returncode->__toString(),
-					'messageKey' => $xml->messageKey->__toString(),
-					'message' => $xml->message->__toString()
-				);
-				return $result;
-			}
-			else {
-				// In this case, we have success and recording info:
-				$result = array(
-					'returncode' => $xml->returncode->__toString(),
-					'messageKey' => $xml->messageKey->__toString(),
-					'message' => $xml->message->__toString()
-				);
-
-				foreach ($xml->recordings->recording as $r) {
-					$result[] = array(
-						'recordId' => $r->recordID->__toString(),
-						'meetingId' => $r->meetingID->__toString(),
-						'name' => $r->name->__toString(),
-						'published' => $r->published->__toString(),
-						'startTime' => $r->startTime->__toString(),
-						'endTime' => $r->endTime->__toString(),
-						'playbackFormatType' => $r->playback->format->type->__toString(),
-						'playbackFormatUrl' => $r->playback->format->url->__toString(),
-						'playbackFormatLength' => $r->playback->format->length->__toString(),
-						'metadataTitle' => $r->metadata->title->__toString(),
-						'metadataSubject' => $r->metadata->subject->__toString(),
-						'metadataDescription' => $r->metadata->description->__toString(),
-						'metadataCreator' => $r->metadata->creator->__toString(),
-						'metadataContributor' => $r->metadata->contributor->__toString(),
-						'metadataLanguage' => $r->metadata->language->__toString(),
-						// Add more here as needed for your app depending on your
-						// use of metadata when creating recordings.
-						);
-					}
-				return $result;
-			}
-		}
-		else {
-			return null;
-		}
-	}
-
-	public function getPublishRecordingsUrl($recordingParams) {
-		/* USAGE:
+        $xml = $this->_processXmlResponse($this->getRecordingsUrl($recordingParams));
+        if($xml) {
+            // If we don't get a success code or messageKey, find out why:
+            if (($xml->returncode != 'SUCCESS') || ($xml->messageKey == null)) {
+                $result = array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'messageKey' => $xml->messageKey->__toString(),
+                    'message' => $xml->message->__toString()
+                );
+                return $result;
+            }
+            else {
+                // In this case, we have success and recording info:
+                $result = array(
+                    'returncode' => $xml->returncode->__toString(),
+                    'messageKey' => $xml->messageKey->__toString(),
+                    'message' => $xml->message->__toString()
+                );
+
+                foreach ($xml->recordings->recording as $r) {
+                    $result[] = array(
+                        'recordId' => $r->recordID->__toString(),
+                        'meetingId' => $r->meetingID->__toString(),
+                        'name' => $r->name->__toString(),
+                        'published' => $r->published->__toString(),
+                        'startTime' => $r->startTime->__toString(),
+                        'endTime' => $r->endTime->__toString(),
+                        'playbackFormatType' => $r->playback->format->type->__toString(),
+                        'playbackFormatUrl' => $r->playback->format->url->__toString(),
+                        'playbackFormatLength' => $r->playback->format->length->__toString(),
+                        'metadataTitle' => $r->metadata->title->__toString(),
+                        'metadataSubject' => $r->metadata->subject->__toString(),
+                        'metadataDescription' => $r->metadata->description->__toString(),
+                        'metadataCreator' => $r->metadata->creator->__toString(),
+                        'metadataContributor' => $r->metadata->contributor->__toString(),
+                        'metadataLanguage' => $r->metadata->language->__toString(),
+                        // Add more here as needed for your app depending on your
+                        // use of metadata when creating recordings.
+                        );
+                    }
+                return $result;
+            }
+        }
+        else {
+            return null;
+        }
+    }
+
+    public function getPublishRecordingsUrl($recordingParams) {
+        /* USAGE:
 		$recordingParams = array(
 			'recordId' => '1234',		-- REQUIRED - comma separate if multiple ids
 			'publish' => 'true',		-- REQUIRED - boolean: true/false
 		);
 		*/
-		$recordingsUrl = $this->_bbbServerBaseUrl."api/publishRecordings?";
-		$params =
-		'recordID='.urlencode($recordingParams['recordId']).
-		'&publish='.urlencode($recordingParams['publish']);
-		return ($recordingsUrl.$params.'&checksum='.sha1("publishRecordings".$params.$this->_securitySalt));
+        $recordingsUrl = $this->_bbbServerBaseUrl."api/publishRecordings?";
+        $params =
+        'recordID='.urlencode($recordingParams['recordId']).
+        '&publish='.urlencode($recordingParams['publish']);
+        return ($recordingsUrl.$params.'&checksum='.sha1("publishRecordings".$params.$this->_securitySalt));
 
-	}
+    }
 
-	public function publishRecordingsWithXmlResponseArray($recordingParams) {
-		/* USAGE:
+    public function publishRecordingsWithXmlResponseArray($recordingParams) {
+        /* USAGE:
 		$recordingParams = array(
 			'recordId' => '1234',		-- REQUIRED - comma separate if multiple ids
 			'publish' => 'true',		-- REQUIRED - boolean: true/false
 		);
 		*/
-		$xml = $this->_processXmlResponse($this->getPublishRecordingsUrl($recordingParams));
-		if($xml) {
-			return array(
-				'returncode' => $xml->returncode->__toString(),
-				'published' => $xml->published->__toString() 	// -- Returns true/false.
-			);
-		}
-		else {
-			return null;
-		}
-
-
-	}
-
-	public function getDeleteRecordingsUrl($recordingParams) {
-		/* USAGE:
+        $xml = $this->_processXmlResponse($this->getPublishRecordingsUrl($recordingParams));
+        if($xml) {
+            return array(
+                'returncode' => $xml->returncode->__toString(),
+                'published' => $xml->published->__toString() 	// -- Returns true/false.
+            );
+        }
+        else {
+            return null;
+        }
+
+
+    }
+
+    public function getDeleteRecordingsUrl($recordingParams) {
+        /* USAGE:
 		$recordingParams = array(
 			'recordId' => '1234',		-- REQUIRED - comma separate if multiple ids
 		);
 		*/
-		$recordingsUrl = $this->_bbbServerBaseUrl."api/deleteRecordings?";
-		$params =
-		'recordID='.urlencode($recordingParams['recordId']);
-		return ($recordingsUrl.$params.'&checksum='.sha1("deleteRecordings".$params.$this->_securitySalt));
-	}
-
-	public function deleteRecordingsWithXmlResponseArray($recordingParams) {
-		/* USAGE:
+        $recordingsUrl = $this->_bbbServerBaseUrl."api/deleteRecordings?";
+        $params =
+        'recordID='.urlencode($recordingParams['recordId']);
+        return ($recordingsUrl.$params.'&checksum='.sha1("deleteRecordings".$params.$this->_securitySalt));
+    }
+
+    public function deleteRecordingsWithXmlResponseArray($recordingParams) {
+        /* USAGE:
 		$recordingParams = array(
 			'recordId' => '1234',		-- REQUIRED - comma separate if multiple ids
 		);
 		*/
 
-		$xml = $this->_processXmlResponse($this->getDeleteRecordingsUrl($recordingParams));
-		if($xml) {
-			return array(
-				'returncode' => $xml->returncode->__toString(),
-				'deleted' => $xml->deleted->__toString() 	// -- Returns true/false.
-			);
-		}
-		else {
-			return null;
-		}
-
-	}
+        $xml = $this->_processXmlResponse($this->getDeleteRecordingsUrl($recordingParams));
+        if($xml) {
+            return array(
+                'returncode' => $xml->returncode->__toString(),
+                'deleted' => $xml->deleted->__toString() 	// -- Returns true/false.
+            );
+        }
+        else {
+            return null;
+        }
+
+    }
 
 
 

plugin/buycourses/src/paypalfunctions.php

Doc Comments +2 added lines

@@ -324,6 +324,8 @@
  * @nvpStr is nvp string.

  * returns an associtive array containing the response from the server.

  *

+ * @param string $methodName
+ * @param string $nvpStr
  */

 function hash_call($methodName, $nvpStr)

 {

Indentation +4 added lines, -4 removed lines

@@ -111,8 +111,8 @@ 
  *		phoneNum:			the phoneNum  entered on the merchant's site

  */

 function CallMarkExpressCheckout($paymentAmount, $currencyCodeType, $paymentType, $returnURL,

-                                 $cancelURL, $shipToName, $shipToStreet, $shipToCity, $shipToState,

-                                 $shipToCountryCode, $shipToZip, $shipToStreet2, $phoneNum

+                                    $cancelURL, $shipToName, $shipToStreet, $shipToCity, $shipToState,

+                                    $shipToCountryCode, $shipToZip, $shipToStreet2, $phoneNum

 )

 {

     // Construct the parameter string that describes the SetExpressCheckout API call in the shortcut implementation

@@ -265,8 +265,8 @@ 
  */

 

 function DirectPayment($paymentType, $paymentAmount, $creditCardType, $creditCardNumber,

-                       $expDate, $cvv2, $firstName, $lastName, $street, $city, $state, $zip,

-                       $countryCode, $currencyCode)

+                        $expDate, $cvv2, $firstName, $lastName, $street, $city, $state, $zip,

+                        $countryCode, $currencyCode)

 {

     //Construct the parameter string that describes DoDirectPayment

     $nvpstr = "&AMT=" . $paymentAmount;

Spacing +44 added lines, -44 removed lines

@@ -67,12 +67,12 @@ 
 function CallShortcutExpressCheckout($paymentAmount, $currencyCodeType, $paymentType, $returnURL, $cancelURL, $extra)

 {

     // Construct the parameter string that describes the SetExpressCheckout API call in the shortcut implementation

-    $nvpstr = "&PAYMENTREQUEST_0_AMT=" . $paymentAmount;

-    $nvpstr .= "&PAYMENTREQUEST_0_ITEMAMT=" . $paymentAmount;

-    $nvpstr .= "&PAYMENTREQUEST_0_PAYMENTACTION=" . $paymentType;

-    $nvpstr .= "&RETURNURL=" . $returnURL;

-    $nvpstr .= "&CANCELURL=" . $cancelURL;

-    $nvpstr .= "&PAYMENTREQUEST_0_CURRENCYCODE=" . $currencyCodeType;

+    $nvpstr = "&PAYMENTREQUEST_0_AMT=".$paymentAmount;

+    $nvpstr .= "&PAYMENTREQUEST_0_ITEMAMT=".$paymentAmount;

+    $nvpstr .= "&PAYMENTREQUEST_0_PAYMENTACTION=".$paymentType;

+    $nvpstr .= "&RETURNURL=".$returnURL;

+    $nvpstr .= "&CANCELURL=".$cancelURL;

+    $nvpstr .= "&PAYMENTREQUEST_0_CURRENCYCODE=".$currencyCodeType;

     $nvpstr .= $extra;

 

     $_SESSION["currencyCodeType"] = $currencyCodeType;

@@ -116,20 +116,20 @@ 
 )

 {

     // Construct the parameter string that describes the SetExpressCheckout API call in the shortcut implementation

-    $nvpstr = "&PAYMENTREQUEST_0_AMT=" . $paymentAmount;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_PAYMENTACTION=" . $paymentType;

-    $nvpstr = $nvpstr . "&RETURNURL=" . $returnURL;

-    $nvpstr = $nvpstr . "&CANCELURL=" . $cancelURL;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_CURRENCYCODE=" . $currencyCodeType;

-    $nvpstr = $nvpstr . "&ADDROVERRIDE=1";

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTONAME=" . $shipToName;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOSTREET=" . $shipToStreet;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOSTREET2=" . $shipToStreet2;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOCITY=" . $shipToCity;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOSTATE=" . $shipToState;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOCOUNTRYCODE=" . $shipToCountryCode;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOZIP=" . $shipToZip;

-    $nvpstr = $nvpstr . "&PAYMENTREQUEST_0_SHIPTOPHONENUM=" . $phoneNum;

+    $nvpstr = "&PAYMENTREQUEST_0_AMT=".$paymentAmount;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_PAYMENTACTION=".$paymentType;

+    $nvpstr = $nvpstr."&RETURNURL=".$returnURL;

+    $nvpstr = $nvpstr."&CANCELURL=".$cancelURL;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_CURRENCYCODE=".$currencyCodeType;

+    $nvpstr = $nvpstr."&ADDROVERRIDE=1";

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTONAME=".$shipToName;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOSTREET=".$shipToStreet;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOSTREET2=".$shipToStreet2;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOCITY=".$shipToCity;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOSTATE=".$shipToState;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOCOUNTRYCODE=".$shipToCountryCode;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOZIP=".$shipToZip;

+    $nvpstr = $nvpstr."&PAYMENTREQUEST_0_SHIPTOPHONENUM=".$phoneNum;

 

     $_SESSION["currencyCodeType"] = $currencyCodeType;

     $_SESSION["PaymentType"] = $paymentType;

@@ -172,7 +172,7 @@ 
      * Build a second API request to PayPal, using the token as the

      * ID to get the details on the payment authorization

      */

-    $nvpstr = "&TOKEN=" . $token;

+    $nvpstr = "&TOKEN=".$token;

 

     /**

      * Make the API call and store the results in an array.

@@ -213,9 +213,9 @@ 
     $payerID = urlencode($_SESSION['payer_id']);

     $serverName = urlencode($_SERVER['SERVER_NAME']);

 

-    $nvpstr = '&TOKEN=' . $token . '&PAYERID=' . $payerID . '&PAYMENTREQUEST_0_PAYMENTACTION=' . $paymentType . '&PAYMENTREQUEST_0_AMT=' . $FinalPaymentAmt;

-    $nvpstr .= '&PAYMENTREQUEST_0_CURRENCYCODE=' . $currencyCodeType . '&IPADDRESS=' . $serverName;

-    $nvpstr = '&' . http_build_query([

+    $nvpstr = '&TOKEN='.$token.'&PAYERID='.$payerID.'&PAYMENTREQUEST_0_PAYMENTACTION='.$paymentType.'&PAYMENTREQUEST_0_AMT='.$FinalPaymentAmt;

+    $nvpstr .= '&PAYMENTREQUEST_0_CURRENCYCODE='.$currencyCodeType.'&IPADDRESS='.$serverName;

+    $nvpstr = '&'.http_build_query([

         'TOKEN' => $token,

         'PAYERID' => $payerID,

         'PAYMENTACTION' => $paymentType,

@@ -269,20 +269,20 @@ 
                        $countryCode, $currencyCode)

 {

     //Construct the parameter string that describes DoDirectPayment

-    $nvpstr = "&AMT=" . $paymentAmount;

-    $nvpstr = $nvpstr . "&CURRENCYCODE=" . $currencyCode;

-    $nvpstr = $nvpstr . "&PAYMENTACTION=" . $paymentType;

-    $nvpstr = $nvpstr . "&CREDITCARDTYPE=" . $creditCardType;

-    $nvpstr = $nvpstr . "&ACCT=" . $creditCardNumber;

-    $nvpstr = $nvpstr . "&EXPDATE=" . $expDate;

-    $nvpstr = $nvpstr . "&CVV2=" . $cvv2;

-    $nvpstr = $nvpstr . "&FIRSTNAME=" . $firstName;

-    $nvpstr = $nvpstr . "&LASTNAME=" . $lastName;

-    $nvpstr = $nvpstr . "&STREET=" . $street;

-    $nvpstr = $nvpstr . "&CITY=" . $city;

-    $nvpstr = $nvpstr . "&STATE=" . $state;

-    $nvpstr = $nvpstr . "&COUNTRYCODE=" . $countryCode;

-    $nvpstr = $nvpstr . "&IPADDRESS=" . $_SERVER['REMOTE_ADDR'];

+    $nvpstr = "&AMT=".$paymentAmount;

+    $nvpstr = $nvpstr."&CURRENCYCODE=".$currencyCode;

+    $nvpstr = $nvpstr."&PAYMENTACTION=".$paymentType;

+    $nvpstr = $nvpstr."&CREDITCARDTYPE=".$creditCardType;

+    $nvpstr = $nvpstr."&ACCT=".$creditCardNumber;

+    $nvpstr = $nvpstr."&EXPDATE=".$expDate;

+    $nvpstr = $nvpstr."&CVV2=".$cvv2;

+    $nvpstr = $nvpstr."&FIRSTNAME=".$firstName;

+    $nvpstr = $nvpstr."&LASTNAME=".$lastName;

+    $nvpstr = $nvpstr."&STREET=".$street;

+    $nvpstr = $nvpstr."&CITY=".$city;

+    $nvpstr = $nvpstr."&STATE=".$state;

+    $nvpstr = $nvpstr."&COUNTRYCODE=".$countryCode;

+    $nvpstr = $nvpstr."&IPADDRESS=".$_SERVER['REMOTE_ADDR'];

 

     $resArray = hash_call("DoDirectPayment", $nvpstr);

 

@@ -345,13 +345,13 @@ 
     //if USE_PROXY constant set to TRUE in Constants.php, then only proxy will be enabled.

     //Set proxy name to PROXY_HOST and port number to PROXY_PORT in constants.php

     if ($USE_PROXY) {

-        curl_setopt($ch, CURLOPT_PROXY, $PROXY_HOST . ":" . $PROXY_PORT);

+        curl_setopt($ch, CURLOPT_PROXY, $PROXY_HOST.":".$PROXY_PORT);

     }

 

     //NVPRequest for submitting to server

-    $nvpreq = "METHOD=" . urlencode($methodName) . "&VERSION=" . urlencode($version) .

-        "&PWD=" . urlencode($API_Password) . "&USER=" . urlencode($API_UserName) .

-        "&SIGNATURE=" . urlencode($API_Signature) . $nvpStr . "&BUTTONSOURCE=" . urlencode($sBNCode);

+    $nvpreq = "METHOD=".urlencode($methodName)."&VERSION=".urlencode($version).

+        "&PWD=".urlencode($API_Password)."&USER=".urlencode($API_UserName).

+        "&SIGNATURE=".urlencode($API_Signature).$nvpStr."&BUTTONSOURCE=".urlencode($sBNCode);

 

     //setting the nvpreq as POST FIELD to curl

     curl_setopt($ch, CURLOPT_POSTFIELDS, $nvpreq);

@@ -385,8 +385,8 @@ 
 {

     global $PAYPAL_URL;

     // Redirect to paypal.com here

-    $payPalURL = $PAYPAL_URL . $token;

-    header("Location: " . $payPalURL);

+    $payPalURL = $PAYPAL_URL.$token;

+    header("Location: ".$payPalURL);

     exit;

 }

 

plugin/clockworksms/lib/clockworksms_plugin.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     /**
      * create (a singleton function that ensures ClockworksmsPlugin instance is
      * created only once. If it is already created, it returns the instance)
-     * @return  object  ClockworksmsPlugin instance
+     * @return  ClockworksmsPlugin  ClockworksmsPlugin instance
      */
     public static function create()
     {

plugin/clockworksms/vendor/clockworksms_api.php

Doc Comments +4 added lines, -1 removed lines

@@ -471,6 +471,8 @@ 
   *
   * @param   string   method Clockwork method to call (sms/credit)
   * @param   string   data   Content of HTTP POST
+  * @param string $method
+  * @param string $data
   *
   * @return  string          Response from Clockwork
   * @author  Martin Steel
@@ -505,6 +507,7 @@ 
   *
   * @param   string url      URL to send to
   * @param   string data     Data to POST
+  * @param string $url
   * @return  string          Response returned by server
   * @author  Martin Steel
   */
@@ -631,7 +634,7 @@ 
    * Check if a number is a valid MSISDN
    *
    * @param string $val Value to check
-   * @return bool True if valid MSISDN
+   * @return integer True if valid MSISDN
    * @author James Inman
    * @since 1.3.0
    * @todo Take an optional country code and check that the number starts with it

Indentation +334 added lines, -334 removed lines

@@ -10,7 +10,7 @@ 
 */
 
 if ( !class_exists('ClockworkException') ) {
-  require_once('exception.php');
+    require_once('exception.php');
 }
 
 /**
@@ -21,130 +21,130 @@ 
 */
 class Clockwork {
 
-  /*
+    /*
   * Version of this class
   */
-  const VERSION           = '1.3.1';
-
-  /**
-  * All Clockwork API calls start with BASE_URL
-  * @author  Martin Steel
-  */
-  const API_BASE_URL      = 'api.clockworksms.com/xml/';
-
-  /**
-  * string to append to API_BASE_URL to check authentication
-  * @author  Martin Steel
-  */
-  const API_AUTH_METHOD   = 'authenticate';
-
-  /**
-  * string to append to API_BASE_URL for sending SMS
-  * @author  Martin Steel
-  */
-  const API_SMS_METHOD    = 'sms';
-
-  /**
-  * string to append to API_BASE_URL for checking message credit
-  * @author  Martin Steel
-  */
-  const API_CREDIT_METHOD = 'credit';
-
-  /**
-  * string to append to API_BASE_URL for checking account balance
-  * @author  Martin Steel
-  */
-  const API_BALANCE_METHOD = 'balance';
-
-  /** 
-  * Clockwork API Key
-  * 
-  * @var string
-  * @author  Martin Steel
-  */
-  public $key;
-
-  /**
-  * Use SSL when making HTTP requests
-  *
-  * If this is not set, SSL will be used where PHP supports it
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $ssl;
-
-  /**
-  * Proxy server hostname (Optional)
-  *
-  * @var string
-  * @author  Martin Steel
-  */
-  public $proxy_host;
-
-  /**
-  * Proxy server port (Optional)
-  *
-  * @var integer
-  * @author  Martin Steel
-  */
-  public $proxy_port;
-
-  /**
-  * From address used on text messages
-  *
-  * @var string (11 characters or 12 numbers)
-  * @author  Martin Steel
-  */
-  public $from;
-
-  /**
-  * Allow long SMS messages (Cost up to 3 credits)
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $long;
-
-  /**
-  * Truncate message text if it is too long
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $truncate;
-
-  /**
-  * Enables various logging of messages when true.
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $log;
-
-  /**
-  * What Clockwork should do if you send an invalid character
-  *
-  * Possible values:
-  *      'error'     - Return an error (Messasge is not sent)
-  *      'remove'    - Remove the invalid character(s)
-  *      'replace'   - Replace invalid characters where possible, remove others 
-  * @author  Martin Steel
-  */
-  public $invalid_char_action;
-
-  /**
-  * Create a new instance of the Clockwork wrapper
-  *
-  * @param   string  key         Your Clockwork API Key
-  * @param   array   options     Optional parameters for sending SMS
-  * @author  Martin Steel
-  */
-  public function __construct($key, array $options = array()) {
+    const VERSION           = '1.3.1';
+
+    /**
+     * All Clockwork API calls start with BASE_URL
+     * @author  Martin Steel
+     */
+    const API_BASE_URL      = 'api.clockworksms.com/xml/';
+
+    /**
+     * string to append to API_BASE_URL to check authentication
+     * @author  Martin Steel
+     */
+    const API_AUTH_METHOD   = 'authenticate';
+
+    /**
+     * string to append to API_BASE_URL for sending SMS
+     * @author  Martin Steel
+     */
+    const API_SMS_METHOD    = 'sms';
+
+    /**
+     * string to append to API_BASE_URL for checking message credit
+     * @author  Martin Steel
+     */
+    const API_CREDIT_METHOD = 'credit';
+
+    /**
+     * string to append to API_BASE_URL for checking account balance
+     * @author  Martin Steel
+     */
+    const API_BALANCE_METHOD = 'balance';
+
+    /** 
+     * Clockwork API Key
+     * 
+     * @var string
+     * @author  Martin Steel
+     */
+    public $key;
+
+    /**
+     * Use SSL when making HTTP requests
+     *
+     * If this is not set, SSL will be used where PHP supports it
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $ssl;
+
+    /**
+     * Proxy server hostname (Optional)
+     *
+     * @var string
+     * @author  Martin Steel
+     */
+    public $proxy_host;
+
+    /**
+     * Proxy server port (Optional)
+     *
+     * @var integer
+     * @author  Martin Steel
+     */
+    public $proxy_port;
+
+    /**
+     * From address used on text messages
+     *
+     * @var string (11 characters or 12 numbers)
+     * @author  Martin Steel
+     */
+    public $from;
+
+    /**
+     * Allow long SMS messages (Cost up to 3 credits)
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $long;
+
+    /**
+     * Truncate message text if it is too long
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $truncate;
+
+    /**
+     * Enables various logging of messages when true.
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $log;
+
+    /**
+     * What Clockwork should do if you send an invalid character
+     *
+     * Possible values:
+     *      'error'     - Return an error (Messasge is not sent)
+     *      'remove'    - Remove the invalid character(s)
+     *      'replace'   - Replace invalid characters where possible, remove others 
+     * @author  Martin Steel
+     */
+    public $invalid_char_action;
+
+    /**
+     * Create a new instance of the Clockwork wrapper
+     *
+     * @param   string  key         Your Clockwork API Key
+     * @param   array   options     Optional parameters for sending SMS
+     * @author  Martin Steel
+     */
+    public function __construct($key, array $options = array()) {
     if (empty($key)) {
-      throw new ClockworkException("Key can't be blank");      
+        throw new ClockworkException("Key can't be blank");      
     } else {
-      $this->key = $key;
+        $this->key = $key;
     }
         
     $this->ssl                  = (array_key_exists('ssl', $options)) ? $options['ssl'] : null;
@@ -155,22 +155,22 @@ 
     $this->truncate             = (array_key_exists('truncate', $options)) ? $options['truncate'] : null;
     $this->invalid_char_action  = (array_key_exists('invalid_char_action', $options)) ? $options['invalid_char_action'] : null;
     $this->log                  = (array_key_exists('log', $options)) ? $options['log'] : false;
-  }
+    }
 
-  /**
-  * Send some text messages
-  * 
-  *
-  * @author  Martin Steel
-  */
-  public function send(array $sms) {
+    /**
+     * Send some text messages
+     * 
+     *
+     * @author  Martin Steel
+     */
+    public function send(array $sms) {
     if (!is_array($sms)) {
-      throw new ClockworkException("sms parameter must be an array");
+        throw new ClockworkException("sms parameter must be an array");
     }
     $single_message = $this->is_assoc($sms);
 
     if ($single_message) {
-      $sms = array($sms);
+        $sms = array($sms);
     }
 
     $req_doc = new DOMDocument('1.0', 'UTF-8');
@@ -182,70 +182,70 @@ 
     $root->appendChild($user_node);
 
     for ($i = 0; $i < count($sms); $i++) {
-      $single = $sms[$i];
+        $single = $sms[$i];
 
-      $sms_node = $req_doc->createElement('SMS');
+        $sms_node = $req_doc->createElement('SMS');
            
-      // Phone number
-      $sms_node->appendChild($req_doc->createElement('To', $single['to'])); 
+        // Phone number
+        $sms_node->appendChild($req_doc->createElement('To', $single['to'])); 
             
-      // Message text
-      $content_node = $req_doc->createElement('Content');
-      $content_node->appendChild($req_doc->createTextNode($single['message']));
-      $sms_node->appendChild($content_node);
+        // Message text
+        $content_node = $req_doc->createElement('Content');
+        $content_node->appendChild($req_doc->createTextNode($single['message']));
+        $sms_node->appendChild($content_node);
 
-      // From
-      if (array_key_exists('from', $single) || isset($this->from)) {
+        // From
+        if (array_key_exists('from', $single) || isset($this->from)) {
         $from_node = $req_doc->createElement('From');
         $from_node->appendChild($req_doc->createTextNode(array_key_exists('from', $single) ? $single['from'] : $this->from));
         $sms_node->appendChild($from_node);
-      }
+        }
 
-      // Client ID
-      if (array_key_exists('client_id', $single)) {
+        // Client ID
+        if (array_key_exists('client_id', $single)) {
         $client_id_node = $req_doc->createElement('ClientID');
         $client_id_node->appendChild($req_doc->createTextNode($single['client_id']));
         $sms_node->appendChild($client_id_node);
-      }
+        }
 
-      // Long
-      if (array_key_exists('long', $single) || isset($this->long)) {
+        // Long
+        if (array_key_exists('long', $single) || isset($this->long)) {
         $long = array_key_exists('long', $single) ? $single['long'] : $this->long;
         $long_node = $req_doc->createElement('Long');
         $long_node->appendChild($req_doc->createTextNode($long ? 1 : 0));
         $sms_node->appendChild($long_node);
-      }
+        }
 
-      // Truncate
-      if (array_key_exists('truncate', $single) || isset($this->truncate)) {
+        // Truncate
+        if (array_key_exists('truncate', $single) || isset($this->truncate)) {
         $truncate = array_key_exists('truncate', $single) ? $single['truncate'] : $this->truncate;
         $trunc_node = $req_doc->createElement('Truncate');
         $trunc_node->appendChild($req_doc->createTextNode($truncate ? 1 : 0));
         $sms_node->appendChild($trunc_node);
-      }
+        }
 
-      // Invalid Char Action
-      if (array_key_exists('invalid_char_action', $single) || isset($this->invalid_char_action)) {
+        // Invalid Char Action
+        if (array_key_exists('invalid_char_action', $single) || isset($this->invalid_char_action)) {
         $action = array_key_exists('invalid_char_action', $single) ? $single['invalid_char_action'] : $this->invalid_char_action;
         switch (strtolower($action)) {
-          case 'error':
+            case 'error':
           $sms_node->appendChild($req_doc->createElement('InvalidCharAction', 1));
-          break;
-          case 'remove':
+            break;
+            case 'remove':
           $sms_node->appendChild($req_doc->createElement('InvalidCharAction', 2));
-          break;
-          case 'replace':
+            break;
+            case 'replace':
           $sms_node->appendChild($req_doc->createElement('InvalidCharAction', 3));
+            break;
+            default:
           break;
-          default:
-          break;
         }
-      }
+        }
 
-      // Wrapper ID
-      $sms_node->appendChild($req_doc->createElement('WrapperID', $i));
+        // Wrapper ID
+        $sms_node->appendChild($req_doc->createElement('WrapperID', $i));
 
-      $root->appendChild($sms_node);
+        $root->appendChild($sms_node);
     }
 
     $req_xml = $req_doc->saveXML();
@@ -259,12 +259,12 @@ 
     $err_desc = null;
 
     foreach($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch(strtolower($doc_child->nodeName)) {
+        switch(strtolower($doc_child->nodeName)) {
         case 'sms_resp':
         $resp = array();
         $wrapper_id = null;
         foreach($doc_child->childNodes AS $resp_node) {
-          switch(strtolower($resp_node->nodeName)) {
+            switch(strtolower($resp_node->nodeName)) {
             case 'messageid':
             $resp['id'] = $resp_node->nodeValue;
             break;
@@ -277,13 +277,13 @@ 
             case 'wrapperid':
             $wrapper_id = $resp_node->nodeValue;
             break;
-          }
+            }
         }
         if( array_key_exists('error_code', $resp ) ) 
         {
-          $resp['success'] = 0;
+            $resp['success'] = 0;
         } else {
-          $resp['success'] = 1;
+            $resp['success'] = 1;
         }
         $resp['sms'] = $sms[$wrapper_id];
         array_push($response, $resp);
@@ -294,28 +294,28 @@ 
         case 'errdesc':
         $err_desc = $doc_child->nodeValue;
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new ClockworkException($err_desc, $err_no);      
+        throw new ClockworkException($err_desc, $err_no);      
     }
         
     if ($single_message) {
-      return $response[0];
+        return $response[0];
     } else {
-      return $response;
+        return $response;
     }
-  }
-
-  /**
-  * Check how many SMS credits you have available
-  *
-  * @return  integer   SMS credits remaining
-  * @deprecated Use checkBalance() instead
-  * @author  Martin Steel
-  */
-  public function checkCredit() {
+    }
+
+    /**
+     * Check how many SMS credits you have available
+     *
+     * @return  integer   SMS credits remaining
+     * @deprecated Use checkBalance() instead
+     * @author  Martin Steel
+     */
+    public function checkCredit() {
     // Create XML doc for request
     $req_doc = new DOMDocument('1.0', 'UTF-8');
     $root = $req_doc->createElement('Credit');
@@ -336,7 +336,7 @@ 
     $err_desc = null;
         
     foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch ($doc_child->nodeName) {
+        switch ($doc_child->nodeName) {
         case "Credit":
         $credit = $doc_child->nodeValue;
         break;
@@ -348,22 +348,22 @@ 
         break;
         default:
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new ClockworkException($err_desc, $err_no);
+        throw new ClockworkException($err_desc, $err_no);
     }
     return $credit;
-  }
+    }
 
-  /**
-  * Check your account balance
-  *
-  * @return  array   Array of account balance: 
-  * @author  Martin Steel
-  */
-  public function checkBalance() {
+    /**
+     * Check your account balance
+     *
+     * @return  array   Array of account balance: 
+     * @author  Martin Steel
+     */
+    public function checkBalance() {
     // Create XML doc for request
     $req_doc = new DOMDocument('1.0', 'UTF-8');
     $root = $req_doc->createElement('Balance');
@@ -384,20 +384,20 @@ 
     $err_desc = null;
         
     foreach ($resp_doc->documentElement->childNodes as $doc_child) {
-      switch ($doc_child->nodeName) {
+        switch ($doc_child->nodeName) {
         case "Balance":
         $balance = number_format(floatval($doc_child->nodeValue), 2);
         break;
         case "Currency":
         foreach ($doc_child->childNodes as $resp_node) {
-          switch ($resp_node->tagName) {
+            switch ($resp_node->tagName) {
             case "Symbol":
             $symbol = $resp_node->nodeValue; 
             break;
             case "Code":
             $code = $resp_node->nodeValue; 
             break;
-          }
+            }
         }
         break;
         case "ErrNo":
@@ -408,23 +408,23 @@ 
         break;
         default:
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new ClockworkException($err_desc, $err_no);
+        throw new ClockworkException($err_desc, $err_no);
     }
         
     return array( 'symbol' => $symbol, 'balance' => $balance, 'code' => $code );
-  }
+    }
 
-  /**
-  * Check whether the API Key is valid
-  *
-  * @return  bool    True indicates a valid key
-  * @author  Martin Steel
-  */
-  public function checkKey() {
+    /**
+     * Check whether the API Key is valid
+     *
+     * @return  bool    True indicates a valid key
+     * @author  Martin Steel
+     */
+    public function checkKey() {
     // Create XML doc for request
     $req_doc = new DOMDocument('1.0', 'UTF-8');
     $root = $req_doc->createElement('Authenticate');
@@ -445,7 +445,7 @@ 
     $err_desc = null;
 
     foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch ($doc_child->nodeName) {
+        switch ($doc_child->nodeName) {
         case "CustID":
         $cust_id = $doc_child->nodeValue;
         break;
@@ -457,33 +457,33 @@ 
         break;
         default:
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new ClockworkException($err_desc, $err_no);
+        throw new ClockworkException($err_desc, $err_no);
     }
     return isset($cust_id);   
-  }
-
-  /**
-  * Make an HTTP POST to Clockwork
-  *
-  * @param   string   method Clockwork method to call (sms/credit)
-  * @param   string   data   Content of HTTP POST
-  *
-  * @return  string          Response from Clockwork
-  * @author  Martin Steel
-  */
-  protected function postToClockwork($method, $data) {
+    }
+
+    /**
+     * Make an HTTP POST to Clockwork
+     *
+     * @param   string   method Clockwork method to call (sms/credit)
+     * @param   string   data   Content of HTTP POST
+     *
+     * @return  string          Response from Clockwork
+     * @author  Martin Steel
+     */
+    protected function postToClockwork($method, $data) {
     if ($this->log) {
-      $this->logXML("API $method Request XML", $data);
+        $this->logXML("API $method Request XML", $data);
     }
     
     if( isset( $this->ssl ) ) {
-      $ssl = $this->ssl;
+        $ssl = $this->ssl;
     } else {
-      $ssl = $this->sslSupport();
+        $ssl = $this->sslSupport();
     }
 
     $url = $ssl ? 'https://' : 'http://';
@@ -492,152 +492,152 @@ 
     $response = $this->xmlPost($url, $data);
 
     if ($this->log) {
-      $this->logXML("API $method Response XML", $response);
+        $this->logXML("API $method Response XML", $response);
     }
 
     return $response;
-  }
-
-  /**
-  * Make a HTTP POST
-  *
-  * cURL will be used if available, otherwise tries the PHP stream functions
-  *
-  * @param   string url      URL to send to
-  * @param   string data     Data to POST
-  * @return  string          Response returned by server
-  * @author  Martin Steel
-  */
-  protected function xmlPost($url, $data) {
+    }
+
+    /**
+     * Make a HTTP POST
+     *
+     * cURL will be used if available, otherwise tries the PHP stream functions
+     *
+     * @param   string url      URL to send to
+     * @param   string data     Data to POST
+     * @return  string          Response returned by server
+     * @author  Martin Steel
+     */
+    protected function xmlPost($url, $data) {
     if(extension_loaded('curl')) {
-      $ch = curl_init($url);
-      curl_setopt($ch, CURLOPT_POST, 1);
-      curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
-      curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
-      curl_setopt($ch, CURLOPT_USERAGENT, 'Clockwork PHP Wrapper/1.0' . self::VERSION);
-      curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
-      if (isset($this->proxy_host) && isset($this->proxy_port)) {
+        $ch = curl_init($url);
+        curl_setopt($ch, CURLOPT_POST, 1);
+        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
+        curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
+        curl_setopt($ch, CURLOPT_USERAGENT, 'Clockwork PHP Wrapper/1.0' . self::VERSION);
+        curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
+        if (isset($this->proxy_host) && isset($this->proxy_port)) {
         curl_setopt($ch, CURLOPT_PROXY, $this->proxy_host);
         curl_setopt($ch, CURLOPT_PROXYPORT, $this->proxy_port);
-      }
+        }
 
-      $response = curl_exec($ch);
-      $info = curl_getinfo($ch);
+        $response = curl_exec($ch);
+        $info = curl_getinfo($ch);
 
-      if ($response === false || $info['http_code'] != 200) {
+        if ($response === false || $info['http_code'] != 200) {
         throw new Exception('HTTP Error calling Clockwork API - HTTP Status: ' . $info['http_code'] . ' - cURL Erorr: ' . curl_error($ch));
-      } elseif (curl_errno($ch) > 0) {
+        } elseif (curl_errno($ch) > 0) {
         throw new Exception('HTTP Error calling Clockwork API - cURL Error: ' . curl_error($ch));
-      }
+        }
 
-      curl_close($ch);
+        curl_close($ch);
 
-      return $response;
+        return $response;
     } elseif (function_exists('stream_get_contents')) {
-      // Enable error Track Errors
-      $track = ini_get('track_errors');
-      ini_set('track_errors',true);
+        // Enable error Track Errors
+        $track = ini_get('track_errors');
+        ini_set('track_errors',true);
 
-      $params = array('http' => array(
-      'method'  => 'POST',
-      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
-      'content' => $data
-      ));
+        $params = array('http' => array(
+        'method'  => 'POST',
+        'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
+        'content' => $data
+        ));
 
-      if (isset($this->proxy_host) && isset($this->proxy_port)) {
+        if (isset($this->proxy_host) && isset($this->proxy_port)) {
         $params['http']['proxy'] = 'tcp://'.$this->proxy_host . ':' . $this->proxy_port;
         $params['http']['request_fulluri'] = True;
-      }
+        }
 
-      $ctx = stream_context_create($params);
-      $fp = @fopen($url, 'rb', false, $ctx);
-      if (!$fp) {
+        $ctx = stream_context_create($params);
+        $fp = @fopen($url, 'rb', false, $ctx);
+        if (!$fp) {
         ini_set('track_errors',$track);
         throw new Exception("HTTP Error calling Clockwork API - fopen Error: $php_errormsg");
-      }
-      $response = @stream_get_contents($fp);
-      if ($response === false) {
+        }
+        $response = @stream_get_contents($fp);
+        if ($response === false) {
         ini_set('track_errors',$track);
         throw new Exception("HTTP Error calling Clockwork API - stream Error: $php_errormsg");
-      }
-      ini_set('track_errors',$track);
-      return $response;
+        }
+        ini_set('track_errors',$track);
+        return $response;
     } else {
-      throw new Exception("Clockwork requires PHP5 with cURL or HTTP stream support");
+        throw new Exception("Clockwork requires PHP5 with cURL or HTTP stream support");
     }
-  }
-
-  /**
-  * Does the server/HTTP wrapper support SSL
-  *
-  * This is a best guess effort, some servers have weird setups where even
-  * though cURL is compiled with SSL support is still fails to make
-  * any requests.
-  *
-  * @return bool     True if SSL is supported
-  * @author  Martin Steel
-  */
-  protected function sslSupport() {
+    }
+
+    /**
+     * Does the server/HTTP wrapper support SSL
+     *
+     * This is a best guess effort, some servers have weird setups where even
+     * though cURL is compiled with SSL support is still fails to make
+     * any requests.
+     *
+     * @return bool     True if SSL is supported
+     * @author  Martin Steel
+     */
+    protected function sslSupport() {
     $ssl = false;
     // See if PHP is compiled with cURL
     if (extension_loaded('curl')) {
-      $version = curl_version();
-      $ssl = ($version['features'] & CURL_VERSION_SSL) ? true : false;
+        $version = curl_version();
+        $ssl = ($version['features'] & CURL_VERSION_SSL) ? true : false;
     } elseif (extension_loaded('openssl')) {
-      $ssl = true;
+        $ssl = true;
     }
     return $ssl;
-  }
-
-  /**
-  * Log some XML, tidily if possible, in the PHP error log
-  *
-  * @param   string  log_msg The log message to prepend to the XML
-  * @param   string  xml     An XML formatted string
-  *
-  * @return  void
-  * @author  Martin Steel
-  */
-  protected function logXML($log_msg, $xml) {
+    }
+
+    /**
+     * Log some XML, tidily if possible, in the PHP error log
+     *
+     * @param   string  log_msg The log message to prepend to the XML
+     * @param   string  xml     An XML formatted string
+     *
+     * @return  void
+     * @author  Martin Steel
+     */
+    protected function logXML($log_msg, $xml) {
     // Tidy if possible
     if (class_exists('tidy')) {
-      $tidy = new tidy;
-      $config = array(
-      'indent'     => true,
-      'input-xml'  => true,
-      'output-xml' => true,
-      'wrap'       => 200
-      );
-      $tidy->parseString($xml, $config, 'utf8');
-      $tidy->cleanRepair();
-      $xml = $tidy;
+        $tidy = new tidy;
+        $config = array(
+        'indent'     => true,
+        'input-xml'  => true,
+        'output-xml' => true,
+        'wrap'       => 200
+        );
+        $tidy->parseString($xml, $config, 'utf8');
+        $tidy->cleanRepair();
+        $xml = $tidy;
     }
     // Output
     error_log("Clockwork $log_msg: $xml");
-  }
-
-  /**
-  * Check if an array is associative
-  *
-  * @param   array $array Array to check
-  * @return  bool
-  * @author  Martin Steel
-  */
-  protected function is_assoc($array) {
+    }
+
+    /**
+     * Check if an array is associative
+     *
+     * @param   array $array Array to check
+     * @return  bool
+     * @author  Martin Steel
+     */
+    protected function is_assoc($array) {
     return (bool)count(array_filter(array_keys($array), 'is_string'));
-  }
+    }
   
-  /**
-   * Check if a number is a valid MSISDN
-   *
-   * @param string $val Value to check
-   * @return bool True if valid MSISDN
-   * @author James Inman
-   * @since 1.3.0
-   * @todo Take an optional country code and check that the number starts with it
-   */
-  public static function is_valid_msisdn($val) {
+    /**
+     * Check if a number is a valid MSISDN
+     *
+     * @param string $val Value to check
+     * @return bool True if valid MSISDN
+     * @author James Inman
+     * @since 1.3.0
+     * @todo Take an optional country code and check that the number starts with it
+     */
+    public static function is_valid_msisdn($val) {
     return preg_match( '/^[1-9][0-9]{7,12}$/', $val );
-  }
+    }
 
 }

Spacing +25 added lines, -25 removed lines

@@ -9,7 +9,7 @@ 
 * @version     1.3.0
 */
 
-if ( !class_exists('ClockworkException') ) {
+if (!class_exists('ClockworkException')) {
   require_once('exception.php');
 }
 
@@ -24,25 +24,25 @@ 
   /*
   * Version of this class
   */
-  const VERSION           = '1.3.1';
+  const VERSION = '1.3.1';
 
   /**
   * All Clockwork API calls start with BASE_URL
   * @author  Martin Steel
   */
-  const API_BASE_URL      = 'api.clockworksms.com/xml/';
+  const API_BASE_URL = 'api.clockworksms.com/xml/';
 
   /**
   * string to append to API_BASE_URL to check authentication
   * @author  Martin Steel
   */
-  const API_AUTH_METHOD   = 'authenticate';
+  const API_AUTH_METHOD = 'authenticate';
 
   /**
   * string to append to API_BASE_URL for sending SMS
   * @author  Martin Steel
   */
-  const API_SMS_METHOD    = 'sms';
+  const API_SMS_METHOD = 'sms';
 
   /**
   * string to append to API_BASE_URL for checking message credit
@@ -258,13 +258,13 @@ 
     $err_no = null;
     $err_desc = null;
 
-    foreach($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch(strtolower($doc_child->nodeName)) {
+    foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
+      switch (strtolower($doc_child->nodeName)) {
         case 'sms_resp':
         $resp = array();
         $wrapper_id = null;
-        foreach($doc_child->childNodes AS $resp_node) {
-          switch(strtolower($resp_node->nodeName)) {
+        foreach ($doc_child->childNodes AS $resp_node) {
+          switch (strtolower($resp_node->nodeName)) {
             case 'messageid':
             $resp['id'] = $resp_node->nodeValue;
             break;
@@ -279,7 +279,7 @@ 
             break;
           }
         }
-        if( array_key_exists('error_code', $resp ) ) 
+        if (array_key_exists('error_code', $resp)) 
         {
           $resp['success'] = 0;
         } else {
@@ -415,7 +415,7 @@ 
       throw new ClockworkException($err_desc, $err_no);
     }
         
-    return array( 'symbol' => $symbol, 'balance' => $balance, 'code' => $code );
+    return array('symbol' => $symbol, 'balance' => $balance, 'code' => $code);
   }
 
   /**
@@ -480,14 +480,14 @@ 
       $this->logXML("API $method Request XML", $data);
     }
     
-    if( isset( $this->ssl ) ) {
+    if (isset($this->ssl)) {
       $ssl = $this->ssl;
     } else {
       $ssl = $this->sslSupport();
     }
 
     $url = $ssl ? 'https://' : 'http://';
-    $url .= self::API_BASE_URL . $method;
+    $url .= self::API_BASE_URL.$method;
 
     $response = $this->xmlPost($url, $data);
 
@@ -509,12 +509,12 @@ 
   * @author  Martin Steel
   */
   protected function xmlPost($url, $data) {
-    if(extension_loaded('curl')) {
+    if (extension_loaded('curl')) {
       $ch = curl_init($url);
       curl_setopt($ch, CURLOPT_POST, 1);
       curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
       curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
-      curl_setopt($ch, CURLOPT_USERAGENT, 'Clockwork PHP Wrapper/1.0' . self::VERSION);
+      curl_setopt($ch, CURLOPT_USERAGENT, 'Clockwork PHP Wrapper/1.0'.self::VERSION);
       curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
       if (isset($this->proxy_host) && isset($this->proxy_port)) {
         curl_setopt($ch, CURLOPT_PROXY, $this->proxy_host);
@@ -525,9 +525,9 @@ 
       $info = curl_getinfo($ch);
 
       if ($response === false || $info['http_code'] != 200) {
-        throw new Exception('HTTP Error calling Clockwork API - HTTP Status: ' . $info['http_code'] . ' - cURL Erorr: ' . curl_error($ch));
+        throw new Exception('HTTP Error calling Clockwork API - HTTP Status: '.$info['http_code'].' - cURL Erorr: '.curl_error($ch));
       } elseif (curl_errno($ch) > 0) {
-        throw new Exception('HTTP Error calling Clockwork API - cURL Error: ' . curl_error($ch));
+        throw new Exception('HTTP Error calling Clockwork API - cURL Error: '.curl_error($ch));
       }
 
       curl_close($ch);
@@ -536,31 +536,31 @@ 
     } elseif (function_exists('stream_get_contents')) {
       // Enable error Track Errors
       $track = ini_get('track_errors');
-      ini_set('track_errors',true);
+      ini_set('track_errors', true);
 
       $params = array('http' => array(
       'method'  => 'POST',
-      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
+      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/".self::VERSION."\r\n",
       'content' => $data
       ));
 
       if (isset($this->proxy_host) && isset($this->proxy_port)) {
-        $params['http']['proxy'] = 'tcp://'.$this->proxy_host . ':' . $this->proxy_port;
+        $params['http']['proxy'] = 'tcp://'.$this->proxy_host.':'.$this->proxy_port;
         $params['http']['request_fulluri'] = True;
       }
 
       $ctx = stream_context_create($params);
       $fp = @fopen($url, 'rb', false, $ctx);
       if (!$fp) {
-        ini_set('track_errors',$track);
+        ini_set('track_errors', $track);
         throw new Exception("HTTP Error calling Clockwork API - fopen Error: $php_errormsg");
       }
       $response = @stream_get_contents($fp);
       if ($response === false) {
-        ini_set('track_errors',$track);
+        ini_set('track_errors', $track);
         throw new Exception("HTTP Error calling Clockwork API - stream Error: $php_errormsg");
       }
-      ini_set('track_errors',$track);
+      ini_set('track_errors', $track);
       return $response;
     } else {
       throw new Exception("Clockwork requires PHP5 with cURL or HTTP stream support");
@@ -624,7 +624,7 @@ 
   * @author  Martin Steel
   */
   protected function is_assoc($array) {
-    return (bool)count(array_filter(array_keys($array), 'is_string'));
+    return (bool) count(array_filter(array_keys($array), 'is_string'));
   }
   
   /**
@@ -637,7 +637,7 @@ 
    * @todo Take an optional country code and check that the number starts with it
    */
   public static function is_valid_msisdn($val) {
-    return preg_match( '/^[1-9][0-9]{7,12}$/', $val );
+    return preg_match('/^[1-9][0-9]{7,12}$/', $val);
   }
 
 }

plugin/courselegal/CourseLegalPlugin.php

Doc Comments +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
      * @param int $courseId
      * @param int $sessionId
      *
-     * @return array
+     * @return string
      */
     public function getUserAcceptedLegal($userId, $courseId, $sessionId)
     {
@@ -112,7 +112,7 @@ 
     * @param int $courseCode
     * @param int $sessionId
     * @param boolean $sendEmail Optional. Indicate whether the mail must be sent. Default is true
-    * @return mixed
+    * @return false|string
     */
     public function saveUserLegal($userId, $courseCode, $sessionId, $sendEmail = true)
     {
@@ -218,7 +218,7 @@ 
      * @param int $courseId
      * @param int $sessionId
      *
-     * @return bool
+     * @return boolean|null
      */
     public function saveUserMailLegal($link, $userId, $courseId, $sessionId)
     {

Indentation +6 added lines, -6 removed lines

@@ -108,12 +108,12 @@
     }
 
     /**
-    * @param int $userId
-    * @param int $courseCode
-    * @param int $sessionId
-    * @param boolean $sendEmail Optional. Indicate whether the mail must be sent. Default is true
-    * @return mixed
-    */
+     * @param int $userId
+     * @param int $courseCode
+     * @param int $sessionId
+     * @param boolean $sendEmail Optional. Indicate whether the mail must be sent. Default is true
+     * @return mixed
+     */
     public function saveUserLegal($userId, $courseCode, $sessionId, $sendEmail = true)
     {
         $courseInfo = api_get_course_info($courseCode);

Spacing +1 added lines, -1 removed lines

@@ -421,7 +421,7 @@
                 array('id = ? ' => $id)
             );
             if (!empty($legalData['filename'])) {
-                $fileToDelete = $coursePath . '/' . $legalData['filename'];
+                $fileToDelete = $coursePath.'/'.$legalData['filename'];
                 if (file_exists($fileToDelete)) {
                     unlink($fileToDelete);
                 }

plugin/kannelsms/lib/kannelsms_plugin.class.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
     /**
      * create (a singleton function that ensures KannelsmsPlugin instance is
      * created only once. If it is already created, it returns the instance)
-     * @return  object  KannelsmsPlugin instance
+     * @return  KannelsmsPlugin  KannelsmsPlugin instance
      */
     public static function create()
     {

plugin/kannelsms/vendor/kannelsms_api.php

Doc Comments +5 added lines, -1 removed lines

@@ -138,6 +138,7 @@ 
   *
   * @param   string  key         Your Kannel API Key
   * @param   array   options     Optional parameters for sending SMS
+  * @param string $key
   * @author  Martin Steel
   */
   public function __construct($key, array $options = array()) {
@@ -471,6 +472,8 @@ 
   *
   * @param   string   method Kannel method to call (sms/credit)
   * @param   string   data   Content of HTTP POST
+  * @param string $method
+  * @param string $data
   *
   * @return  string          Response from Kannel
   * @author  Martin Steel
@@ -505,6 +508,7 @@ 
   *
   * @param   string url      URL to send to
   * @param   string data     Data to POST
+  * @param string $url
   * @return  string          Response returned by server
   * @author  Martin Steel
   */
@@ -631,7 +635,7 @@ 
    * Check if a number is a valid MSISDN
    *
    * @param string $val Value to check
-   * @return bool True if valid MSISDN
+   * @return integer True if valid MSISDN
    * @author James Inman
    * @since 1.3.0
    * @todo Take an optional country code and check that the number starts with it

Indentation +334 added lines, -334 removed lines

@@ -10,7 +10,7 @@ 
 */
 
 if ( !class_exists('KannelException') ) {
-  require_once('exception.php');
+    require_once('exception.php');
 }
 
 /**
@@ -21,130 +21,130 @@ 
 */
 class Kannel {
 
-  /*
+    /*
   * Version of this class
   */
-  const VERSION           = '1.3.1';
-
-  /**
-  * All Kannel API calls start with BASE_URL
-  * @author  Martin Steel
-  */
-  const API_BASE_URL      = 'api.kannelsms.com/xml/';
-
-  /**
-  * string to append to API_BASE_URL to check authentication
-  * @author  Martin Steel
-  */
-  const API_AUTH_METHOD   = 'authenticate';
-
-  /**
-  * string to append to API_BASE_URL for sending SMS
-  * @author  Martin Steel
-  */
-  const API_SMS_METHOD    = 'sms';
-
-  /**
-  * string to append to API_BASE_URL for checking message credit
-  * @author  Martin Steel
-  */
-  const API_CREDIT_METHOD = 'credit';
-
-  /**
-  * string to append to API_BASE_URL for checking account balance
-  * @author  Martin Steel
-  */
-  const API_BALANCE_METHOD = 'balance';
-
-  /** 
-  * Kannel API Key
-  * 
-  * @var string
-  * @author  Martin Steel
-  */
-  public $key;
-
-  /**
-  * Use SSL when making HTTP requests
-  *
-  * If this is not set, SSL will be used where PHP supports it
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $ssl;
-
-  /**
-  * Proxy server hostname (Optional)
-  *
-  * @var string
-  * @author  Martin Steel
-  */
-  public $proxy_host;
-
-  /**
-  * Proxy server port (Optional)
-  *
-  * @var integer
-  * @author  Martin Steel
-  */
-  public $proxy_port;
-
-  /**
-  * From address used on text messages
-  *
-  * @var string (11 characters or 12 numbers)
-  * @author  Martin Steel
-  */
-  public $from;
-
-  /**
-  * Allow long SMS messages (Cost up to 3 credits)
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $long;
-
-  /**
-  * Truncate message text if it is too long
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $truncate;
-
-  /**
-  * Enables various logging of messages when true.
-  *
-  * @var bool
-  * @author  Martin Steel
-  */
-  public $log;
-
-  /**
-  * What Kannel should do if you send an invalid character
-  *
-  * Possible values:
-  *      'error'     - Return an error (Messasge is not sent)
-  *      'remove'    - Remove the invalid character(s)
-  *      'replace'   - Replace invalid characters where possible, remove others 
-  * @author  Martin Steel
-  */
-  public $invalid_char_action;
-
-  /**
-  * Create a new instance of the Kannel wrapper
-  *
-  * @param   string  key         Your Kannel API Key
-  * @param   array   options     Optional parameters for sending SMS
-  * @author  Martin Steel
-  */
-  public function __construct($key, array $options = array()) {
+    const VERSION           = '1.3.1';
+
+    /**
+     * All Kannel API calls start with BASE_URL
+     * @author  Martin Steel
+     */
+    const API_BASE_URL      = 'api.kannelsms.com/xml/';
+
+    /**
+     * string to append to API_BASE_URL to check authentication
+     * @author  Martin Steel
+     */
+    const API_AUTH_METHOD   = 'authenticate';
+
+    /**
+     * string to append to API_BASE_URL for sending SMS
+     * @author  Martin Steel
+     */
+    const API_SMS_METHOD    = 'sms';
+
+    /**
+     * string to append to API_BASE_URL for checking message credit
+     * @author  Martin Steel
+     */
+    const API_CREDIT_METHOD = 'credit';
+
+    /**
+     * string to append to API_BASE_URL for checking account balance
+     * @author  Martin Steel
+     */
+    const API_BALANCE_METHOD = 'balance';
+
+    /** 
+     * Kannel API Key
+     * 
+     * @var string
+     * @author  Martin Steel
+     */
+    public $key;
+
+    /**
+     * Use SSL when making HTTP requests
+     *
+     * If this is not set, SSL will be used where PHP supports it
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $ssl;
+
+    /**
+     * Proxy server hostname (Optional)
+     *
+     * @var string
+     * @author  Martin Steel
+     */
+    public $proxy_host;
+
+    /**
+     * Proxy server port (Optional)
+     *
+     * @var integer
+     * @author  Martin Steel
+     */
+    public $proxy_port;
+
+    /**
+     * From address used on text messages
+     *
+     * @var string (11 characters or 12 numbers)
+     * @author  Martin Steel
+     */
+    public $from;
+
+    /**
+     * Allow long SMS messages (Cost up to 3 credits)
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $long;
+
+    /**
+     * Truncate message text if it is too long
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $truncate;
+
+    /**
+     * Enables various logging of messages when true.
+     *
+     * @var bool
+     * @author  Martin Steel
+     */
+    public $log;
+
+    /**
+     * What Kannel should do if you send an invalid character
+     *
+     * Possible values:
+     *      'error'     - Return an error (Messasge is not sent)
+     *      'remove'    - Remove the invalid character(s)
+     *      'replace'   - Replace invalid characters where possible, remove others 
+     * @author  Martin Steel
+     */
+    public $invalid_char_action;
+
+    /**
+     * Create a new instance of the Kannel wrapper
+     *
+     * @param   string  key         Your Kannel API Key
+     * @param   array   options     Optional parameters for sending SMS
+     * @author  Martin Steel
+     */
+    public function __construct($key, array $options = array()) {
     if (empty($key)) {
-      throw new KannelException("Key can't be blank");
+        throw new KannelException("Key can't be blank");
     } else {
-      $this->key = $key;
+        $this->key = $key;
     }
         
     $this->ssl                  = (array_key_exists('ssl', $options)) ? $options['ssl'] : null;
@@ -155,22 +155,22 @@ 
     $this->truncate             = (array_key_exists('truncate', $options)) ? $options['truncate'] : null;
     $this->invalid_char_action  = (array_key_exists('invalid_char_action', $options)) ? $options['invalid_char_action'] : null;
     $this->log                  = (array_key_exists('log', $options)) ? $options['log'] : false;
-  }
+    }
 
-  /**
-  * Send some text messages
-  * 
-  *
-  * @author  Martin Steel
-  */
-  public function send(array $sms) {
+    /**
+     * Send some text messages
+     * 
+     *
+     * @author  Martin Steel
+     */
+    public function send(array $sms) {
     if (!is_array($sms)) {
-      throw new KannelException("sms parameter must be an array");
+        throw new KannelException("sms parameter must be an array");
     }
     $single_message = $this->is_assoc($sms);
 
     if ($single_message) {
-      $sms = array($sms);
+        $sms = array($sms);
     }
 
     $req_doc = new \DOMDocument('1.0', 'UTF-8');
@@ -182,70 +182,70 @@ 
     $root->appendChild($user_node);
 
     for ($i = 0; $i < count($sms); $i++) {
-      $single = $sms[$i];
+        $single = $sms[$i];
 
-      $sms_node = $req_doc->createElement('SMS');
+        $sms_node = $req_doc->createElement('SMS');
            
-      // Phone number
-      $sms_node->appendChild($req_doc->createElement('To', $single['to'])); 
+        // Phone number
+        $sms_node->appendChild($req_doc->createElement('To', $single['to'])); 
             
-      // Message text
-      $content_node = $req_doc->createElement('Content');
-      $content_node->appendChild($req_doc->createTextNode($single['message']));
-      $sms_node->appendChild($content_node);
+        // Message text
+        $content_node = $req_doc->createElement('Content');
+        $content_node->appendChild($req_doc->createTextNode($single['message']));
+        $sms_node->appendChild($content_node);
 
-      // From
-      if (array_key_exists('from', $single) || isset($this->from)) {
+        // From
+        if (array_key_exists('from', $single) || isset($this->from)) {
         $from_node = $req_doc->createElement('From');
         $from_node->appendChild($req_doc->createTextNode(array_key_exists('from', $single) ? $single['from'] : $this->from));
         $sms_node->appendChild($from_node);
-      }
+        }
 
-      // Client ID
-      if (array_key_exists('client_id', $single)) {
+        // Client ID
+        if (array_key_exists('client_id', $single)) {
         $client_id_node = $req_doc->createElement('ClientID');
         $client_id_node->appendChild($req_doc->createTextNode($single['client_id']));
         $sms_node->appendChild($client_id_node);
-      }
+        }
 
-      // Long
-      if (array_key_exists('long', $single) || isset($this->long)) {
+        // Long
+        if (array_key_exists('long', $single) || isset($this->long)) {
         $long = array_key_exists('long', $single) ? $single['long'] : $this->long;
         $long_node = $req_doc->createElement('Long');
         $long_node->appendChild($req_doc->createTextNode($long ? 1 : 0));
         $sms_node->appendChild($long_node);
-      }
+        }
 
-      // Truncate
-      if (array_key_exists('truncate', $single) || isset($this->truncate)) {
+        // Truncate
+        if (array_key_exists('truncate', $single) || isset($this->truncate)) {
         $truncate = array_key_exists('truncate', $single) ? $single['truncate'] : $this->truncate;
         $trunc_node = $req_doc->createElement('Truncate');
         $trunc_node->appendChild($req_doc->createTextNode($truncate ? 1 : 0));
         $sms_node->appendChild($trunc_node);
-      }
+        }
 
-      // Invalid Char Action
-      if (array_key_exists('invalid_char_action', $single) || isset($this->invalid_char_action)) {
+        // Invalid Char Action
+        if (array_key_exists('invalid_char_action', $single) || isset($this->invalid_char_action)) {
         $action = array_key_exists('invalid_char_action', $single) ? $single['invalid_char_action'] : $this->invalid_char_action;
         switch (strtolower($action)) {
-          case 'error':
+            case 'error':
           $sms_node->appendChild($req_doc->createElement('InvalidCharAction', 1));
-          break;
-          case 'remove':
+            break;
+            case 'remove':
           $sms_node->appendChild($req_doc->createElement('InvalidCharAction', 2));
-          break;
-          case 'replace':
+            break;
+            case 'replace':
           $sms_node->appendChild($req_doc->createElement('InvalidCharAction', 3));
+            break;
+            default:
           break;
-          default:
-          break;
         }
-      }
+        }
 
-      // Wrapper ID
-      $sms_node->appendChild($req_doc->createElement('WrapperID', $i));
+        // Wrapper ID
+        $sms_node->appendChild($req_doc->createElement('WrapperID', $i));
 
-      $root->appendChild($sms_node);
+        $root->appendChild($sms_node);
     }
 
     $req_xml = $req_doc->saveXML();
@@ -259,12 +259,12 @@ 
     $err_desc = null;
 
     foreach($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch(strtolower($doc_child->nodeName)) {
+        switch(strtolower($doc_child->nodeName)) {
         case 'sms_resp':
         $resp = array();
         $wrapper_id = null;
         foreach($doc_child->childNodes AS $resp_node) {
-          switch(strtolower($resp_node->nodeName)) {
+            switch(strtolower($resp_node->nodeName)) {
             case 'messageid':
             $resp['id'] = $resp_node->nodeValue;
             break;
@@ -277,13 +277,13 @@ 
             case 'wrapperid':
             $wrapper_id = $resp_node->nodeValue;
             break;
-          }
+            }
         }
         if( array_key_exists('error_code', $resp ) ) 
         {
-          $resp['success'] = 0;
+            $resp['success'] = 0;
         } else {
-          $resp['success'] = 1;
+            $resp['success'] = 1;
         }
         $resp['sms'] = $sms[$wrapper_id];
         array_push($response, $resp);
@@ -294,28 +294,28 @@ 
         case 'errdesc':
         $err_desc = $doc_child->nodeValue;
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new KannelException($err_desc, $err_no);
+        throw new KannelException($err_desc, $err_no);
     }
         
     if ($single_message) {
-      return $response[0];
+        return $response[0];
     } else {
-      return $response;
+        return $response;
     }
-  }
-
-  /**
-  * Check how many SMS credits you have available
-  *
-  * @return  integer   SMS credits remaining
-  * @deprecated Use checkBalance() instead
-  * @author  Martin Steel
-  */
-  public function checkCredit() {
+    }
+
+    /**
+     * Check how many SMS credits you have available
+     *
+     * @return  integer   SMS credits remaining
+     * @deprecated Use checkBalance() instead
+     * @author  Martin Steel
+     */
+    public function checkCredit() {
     // Create XML doc for request
     $req_doc = new \DOMDocument('1.0', 'UTF-8');
     $root = $req_doc->createElement('Credit');
@@ -336,7 +336,7 @@ 
     $err_desc = null;
         
     foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch ($doc_child->nodeName) {
+        switch ($doc_child->nodeName) {
         case "Credit":
         $credit = $doc_child->nodeValue;
         break;
@@ -348,22 +348,22 @@ 
         break;
         default:
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new KannelException($err_desc, $err_no);
+        throw new KannelException($err_desc, $err_no);
     }
     return $credit;
-  }
+    }
 
-  /**
-  * Check your account balance
-  *
-  * @return  array   Array of account balance: 
-  * @author  Martin Steel
-  */
-  public function checkBalance() {
+    /**
+     * Check your account balance
+     *
+     * @return  array   Array of account balance: 
+     * @author  Martin Steel
+     */
+    public function checkBalance() {
     // Create XML doc for request
     $req_doc = new \DOMDocument('1.0', 'UTF-8');
     $root = $req_doc->createElement('Balance');
@@ -384,20 +384,20 @@ 
     $err_desc = null;
         
     foreach ($resp_doc->documentElement->childNodes as $doc_child) {
-      switch ($doc_child->nodeName) {
+        switch ($doc_child->nodeName) {
         case "Balance":
         $balance = number_format(floatval($doc_child->nodeValue), 2);
         break;
         case "Currency":
         foreach ($doc_child->childNodes as $resp_node) {
-          switch ($resp_node->tagName) {
+            switch ($resp_node->tagName) {
             case "Symbol":
             $symbol = $resp_node->nodeValue; 
             break;
             case "Code":
             $code = $resp_node->nodeValue; 
             break;
-          }
+            }
         }
         break;
         case "ErrNo":
@@ -408,23 +408,23 @@ 
         break;
         default:
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new KannelException($err_desc, $err_no);
+        throw new KannelException($err_desc, $err_no);
     }
         
     return array( 'symbol' => $symbol, 'balance' => $balance, 'code' => $code );
-  }
+    }
 
-  /**
-  * Check whether the API Key is valid
-  *
-  * @return  bool    True indicates a valid key
-  * @author  Martin Steel
-  */
-  public function checkKey() {
+    /**
+     * Check whether the API Key is valid
+     *
+     * @return  bool    True indicates a valid key
+     * @author  Martin Steel
+     */
+    public function checkKey() {
     // Create XML doc for request
     $req_doc = new \DOMDocument('1.0', 'UTF-8');
     $root = $req_doc->createElement('Authenticate');
@@ -445,7 +445,7 @@ 
     $err_desc = null;
 
     foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch ($doc_child->nodeName) {
+        switch ($doc_child->nodeName) {
         case "CustID":
         $cust_id = $doc_child->nodeValue;
         break;
@@ -457,33 +457,33 @@ 
         break;
         default:
         break;
-      }
+        }
     }
 
     if (isset($err_no)) {
-      throw new KannelException($err_desc, $err_no);
+        throw new KannelException($err_desc, $err_no);
     }
     return isset($cust_id);   
-  }
-
-  /**
-  * Make an HTTP POST to Kannel
-  *
-  * @param   string   method Kannel method to call (sms/credit)
-  * @param   string   data   Content of HTTP POST
-  *
-  * @return  string          Response from Kannel
-  * @author  Martin Steel
-  */
-  protected function postToKannel($method, $data) {
+    }
+
+    /**
+     * Make an HTTP POST to Kannel
+     *
+     * @param   string   method Kannel method to call (sms/credit)
+     * @param   string   data   Content of HTTP POST
+     *
+     * @return  string          Response from Kannel
+     * @author  Martin Steel
+     */
+    protected function postToKannel($method, $data) {
     if ($this->log) {
-      $this->logXML("API $method Request XML", $data);
+        $this->logXML("API $method Request XML", $data);
     }
     
     if( isset( $this->ssl ) ) {
-      $ssl = $this->ssl;
+        $ssl = $this->ssl;
     } else {
-      $ssl = $this->sslSupport();
+        $ssl = $this->sslSupport();
     }
 
     $url = $ssl ? 'https://' : 'http://';
@@ -492,152 +492,152 @@ 
     $response = $this->xmlPost($url, $data);
 
     if ($this->log) {
-      $this->logXML("API $method Response XML", $response);
+        $this->logXML("API $method Response XML", $response);
     }
 
     return $response;
-  }
-
-  /**
-  * Make a HTTP POST
-  *
-  * cURL will be used if available, otherwise tries the PHP stream functions
-  *
-  * @param   string url      URL to send to
-  * @param   string data     Data to POST
-  * @return  string          Response returned by server
-  * @author  Martin Steel
-  */
-  protected function xmlPost($url, $data) {
+    }
+
+    /**
+     * Make a HTTP POST
+     *
+     * cURL will be used if available, otherwise tries the PHP stream functions
+     *
+     * @param   string url      URL to send to
+     * @param   string data     Data to POST
+     * @return  string          Response returned by server
+     * @author  Martin Steel
+     */
+    protected function xmlPost($url, $data) {
     if(extension_loaded('curl')) {
-      $ch = curl_init($url);
-      curl_setopt($ch, CURLOPT_POST, 1);
-      curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
-      curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
-      curl_setopt($ch, CURLOPT_USERAGENT, 'Kannel PHP Wrapper/1.0' . self::VERSION);
-      curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
-      if (isset($this->proxy_host) && isset($this->proxy_port)) {
+        $ch = curl_init($url);
+        curl_setopt($ch, CURLOPT_POST, 1);
+        curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
+        curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
+        curl_setopt($ch, CURLOPT_USERAGENT, 'Kannel PHP Wrapper/1.0' . self::VERSION);
+        curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
+        if (isset($this->proxy_host) && isset($this->proxy_port)) {
         curl_setopt($ch, CURLOPT_PROXY, $this->proxy_host);
         curl_setopt($ch, CURLOPT_PROXYPORT, $this->proxy_port);
-      }
+        }
 
-      $response = curl_exec($ch);
-      $info = curl_getinfo($ch);
+        $response = curl_exec($ch);
+        $info = curl_getinfo($ch);
 
-      if ($response === false || $info['http_code'] != 200) {
+        if ($response === false || $info['http_code'] != 200) {
         throw new \Exception('HTTP Error calling Kannel API - HTTP Status: ' . $info['http_code'] . ' - cURL Erorr: ' . curl_error($ch));
-      } elseif (curl_errno($ch) > 0) {
+        } elseif (curl_errno($ch) > 0) {
         throw new \Exception('HTTP Error calling Kannel API - cURL Error: ' . curl_error($ch));
-      }
+        }
 
-      curl_close($ch);
+        curl_close($ch);
 
-      return $response;
+        return $response;
     } elseif (function_exists('stream_get_contents')) {
-      // Enable error Track Errors
-      $track = ini_get('track_errors');
-      ini_set('track_errors',true);
+        // Enable error Track Errors
+        $track = ini_get('track_errors');
+        ini_set('track_errors',true);
 
-      $params = array('http' => array(
-      'method'  => 'POST',
-      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
-      'content' => $data
-      ));
+        $params = array('http' => array(
+        'method'  => 'POST',
+        'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
+        'content' => $data
+        ));
 
-      if (isset($this->proxy_host) && isset($this->proxy_port)) {
+        if (isset($this->proxy_host) && isset($this->proxy_port)) {
         $params['http']['proxy'] = 'tcp://'.$this->proxy_host . ':' . $this->proxy_port;
         $params['http']['request_fulluri'] = True;
-      }
+        }
 
-      $ctx = stream_context_create($params);
-      $fp = @fopen($url, 'rb', false, $ctx);
-      if (!$fp) {
+        $ctx = stream_context_create($params);
+        $fp = @fopen($url, 'rb', false, $ctx);
+        if (!$fp) {
         ini_set('track_errors',$track);
         throw new \Exception("HTTP Error calling Kannel API - fopen Error: $php_errormsg");
-      }
-      $response = @stream_get_contents($fp);
-      if ($response === false) {
+        }
+        $response = @stream_get_contents($fp);
+        if ($response === false) {
         ini_set('track_errors',$track);
         throw new \Exception("HTTP Error calling Kannel API - stream Error: $php_errormsg");
-      }
-      ini_set('track_errors',$track);
-      return $response;
+        }
+        ini_set('track_errors',$track);
+        return $response;
     } else {
-      throw new \Exception("Kannel requires PHP5 with cURL or HTTP stream support");
+        throw new \Exception("Kannel requires PHP5 with cURL or HTTP stream support");
     }
-  }
-
-  /**
-  * Does the server/HTTP wrapper support SSL
-  *
-  * This is a best guess effort, some servers have weird setups where even
-  * though cURL is compiled with SSL support is still fails to make
-  * any requests.
-  *
-  * @return bool     True if SSL is supported
-  * @author  Martin Steel
-  */
-  protected function sslSupport() {
+    }
+
+    /**
+     * Does the server/HTTP wrapper support SSL
+     *
+     * This is a best guess effort, some servers have weird setups where even
+     * though cURL is compiled with SSL support is still fails to make
+     * any requests.
+     *
+     * @return bool     True if SSL is supported
+     * @author  Martin Steel
+     */
+    protected function sslSupport() {
     $ssl = false;
     // See if PHP is compiled with cURL
     if (extension_loaded('curl')) {
-      $version = curl_version();
-      $ssl = ($version['features'] & CURL_VERSION_SSL) ? true : false;
+        $version = curl_version();
+        $ssl = ($version['features'] & CURL_VERSION_SSL) ? true : false;
     } elseif (extension_loaded('openssl')) {
-      $ssl = true;
+        $ssl = true;
     }
     return $ssl;
-  }
-
-  /**
-  * Log some XML, tidily if possible, in the PHP error log
-  *
-  * @param   string  log_msg The log message to prepend to the XML
-  * @param   string  xml     An XML formatted string
-  *
-  * @return  void
-  * @author  Martin Steel
-  */
-  protected function logXML($log_msg, $xml) {
+    }
+
+    /**
+     * Log some XML, tidily if possible, in the PHP error log
+     *
+     * @param   string  log_msg The log message to prepend to the XML
+     * @param   string  xml     An XML formatted string
+     *
+     * @return  void
+     * @author  Martin Steel
+     */
+    protected function logXML($log_msg, $xml) {
     // Tidy if possible
     if (class_exists('tidy')) {
-      $tidy = new \tidy;
-      $config = array(
-      'indent'     => true,
-      'input-xml'  => true,
-      'output-xml' => true,
-      'wrap'       => 200
-      );
-      $tidy->parseString($xml, $config, 'utf8');
-      $tidy->cleanRepair();
-      $xml = $tidy;
+        $tidy = new \tidy;
+        $config = array(
+        'indent'     => true,
+        'input-xml'  => true,
+        'output-xml' => true,
+        'wrap'       => 200
+        );
+        $tidy->parseString($xml, $config, 'utf8');
+        $tidy->cleanRepair();
+        $xml = $tidy;
     }
     // Output
     error_log("Kannel $log_msg: $xml");
-  }
-
-  /**
-  * Check if an array is associative
-  *
-  * @param   array $array Array to check
-  * @return  bool
-  * @author  Martin Steel
-  */
-  protected function is_assoc($array) {
+    }
+
+    /**
+     * Check if an array is associative
+     *
+     * @param   array $array Array to check
+     * @return  bool
+     * @author  Martin Steel
+     */
+    protected function is_assoc($array) {
     return (bool)count(array_filter(array_keys($array), 'is_string'));
-  }
+    }
   
-  /**
-   * Check if a number is a valid MSISDN
-   *
-   * @param string $val Value to check
-   * @return bool True if valid MSISDN
-   * @author James Inman
-   * @since 1.3.0
-   * @todo Take an optional country code and check that the number starts with it
-   */
-  public static function is_valid_msisdn($val) {
+    /**
+     * Check if a number is a valid MSISDN
+     *
+     * @param string $val Value to check
+     * @return bool True if valid MSISDN
+     * @author James Inman
+     * @since 1.3.0
+     * @todo Take an optional country code and check that the number starts with it
+     */
+    public static function is_valid_msisdn($val) {
     return preg_match( '/^[1-9][0-9]{7,12}$/', $val );
-  }
+    }
 
 }

Spacing +25 added lines, -25 removed lines

@@ -9,7 +9,7 @@ 
 * @version     1.3.0
 */
 
-if ( !class_exists('KannelException') ) {
+if (!class_exists('KannelException')) {
   require_once('exception.php');
 }
 
@@ -24,25 +24,25 @@ 
   /*
   * Version of this class
   */
-  const VERSION           = '1.3.1';
+  const VERSION = '1.3.1';
 
   /**
   * All Kannel API calls start with BASE_URL
   * @author  Martin Steel
   */
-  const API_BASE_URL      = 'api.kannelsms.com/xml/';
+  const API_BASE_URL = 'api.kannelsms.com/xml/';
 
   /**
   * string to append to API_BASE_URL to check authentication
   * @author  Martin Steel
   */
-  const API_AUTH_METHOD   = 'authenticate';
+  const API_AUTH_METHOD = 'authenticate';
 
   /**
   * string to append to API_BASE_URL for sending SMS
   * @author  Martin Steel
   */
-  const API_SMS_METHOD    = 'sms';
+  const API_SMS_METHOD = 'sms';
 
   /**
   * string to append to API_BASE_URL for checking message credit
@@ -258,13 +258,13 @@ 
     $err_no = null;
     $err_desc = null;
 
-    foreach($resp_doc->documentElement->childNodes AS $doc_child) {
-      switch(strtolower($doc_child->nodeName)) {
+    foreach ($resp_doc->documentElement->childNodes AS $doc_child) {
+      switch (strtolower($doc_child->nodeName)) {
         case 'sms_resp':
         $resp = array();
         $wrapper_id = null;
-        foreach($doc_child->childNodes AS $resp_node) {
-          switch(strtolower($resp_node->nodeName)) {
+        foreach ($doc_child->childNodes AS $resp_node) {
+          switch (strtolower($resp_node->nodeName)) {
             case 'messageid':
             $resp['id'] = $resp_node->nodeValue;
             break;
@@ -279,7 +279,7 @@ 
             break;
           }
         }
-        if( array_key_exists('error_code', $resp ) ) 
+        if (array_key_exists('error_code', $resp)) 
         {
           $resp['success'] = 0;
         } else {
@@ -415,7 +415,7 @@ 
       throw new KannelException($err_desc, $err_no);
     }
         
-    return array( 'symbol' => $symbol, 'balance' => $balance, 'code' => $code );
+    return array('symbol' => $symbol, 'balance' => $balance, 'code' => $code);
   }
 
   /**
@@ -480,14 +480,14 @@ 
       $this->logXML("API $method Request XML", $data);
     }
     
-    if( isset( $this->ssl ) ) {
+    if (isset($this->ssl)) {
       $ssl = $this->ssl;
     } else {
       $ssl = $this->sslSupport();
     }
 
     $url = $ssl ? 'https://' : 'http://';
-    $url .= self::API_BASE_URL . $method;
+    $url .= self::API_BASE_URL.$method;
 
     $response = $this->xmlPost($url, $data);
 
@@ -509,12 +509,12 @@ 
   * @author  Martin Steel
   */
   protected function xmlPost($url, $data) {
-    if(extension_loaded('curl')) {
+    if (extension_loaded('curl')) {
       $ch = curl_init($url);
       curl_setopt($ch, CURLOPT_POST, 1);
       curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
       curl_setopt($ch, CURLOPT_HTTPHEADER, Array("Content-Type: text/xml"));
-      curl_setopt($ch, CURLOPT_USERAGENT, 'Kannel PHP Wrapper/1.0' . self::VERSION);
+      curl_setopt($ch, CURLOPT_USERAGENT, 'Kannel PHP Wrapper/1.0'.self::VERSION);
       curl_setopt($ch, CURLOPT_POSTFIELDS, $data);
       if (isset($this->proxy_host) && isset($this->proxy_port)) {
         curl_setopt($ch, CURLOPT_PROXY, $this->proxy_host);
@@ -525,9 +525,9 @@ 
       $info = curl_getinfo($ch);
 
       if ($response === false || $info['http_code'] != 200) {
-        throw new \Exception('HTTP Error calling Kannel API - HTTP Status: ' . $info['http_code'] . ' - cURL Erorr: ' . curl_error($ch));
+        throw new \Exception('HTTP Error calling Kannel API - HTTP Status: '.$info['http_code'].' - cURL Erorr: '.curl_error($ch));
       } elseif (curl_errno($ch) > 0) {
-        throw new \Exception('HTTP Error calling Kannel API - cURL Error: ' . curl_error($ch));
+        throw new \Exception('HTTP Error calling Kannel API - cURL Error: '.curl_error($ch));
       }
 
       curl_close($ch);
@@ -536,31 +536,31 @@ 
     } elseif (function_exists('stream_get_contents')) {
       // Enable error Track Errors
       $track = ini_get('track_errors');
-      ini_set('track_errors',true);
+      ini_set('track_errors', true);
 
       $params = array('http' => array(
       'method'  => 'POST',
-      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/" . self::VERSION . "\r\n",
+      'header'  => "Content-Type: text/xml\r\nUser-Agent: mediaburst PHP Wrapper/".self::VERSION."\r\n",
       'content' => $data
       ));
 
       if (isset($this->proxy_host) && isset($this->proxy_port)) {
-        $params['http']['proxy'] = 'tcp://'.$this->proxy_host . ':' . $this->proxy_port;
+        $params['http']['proxy'] = 'tcp://'.$this->proxy_host.':'.$this->proxy_port;
         $params['http']['request_fulluri'] = True;
       }
 
       $ctx = stream_context_create($params);
       $fp = @fopen($url, 'rb', false, $ctx);
       if (!$fp) {
-        ini_set('track_errors',$track);
+        ini_set('track_errors', $track);
         throw new \Exception("HTTP Error calling Kannel API - fopen Error: $php_errormsg");
       }
       $response = @stream_get_contents($fp);
       if ($response === false) {
-        ini_set('track_errors',$track);
+        ini_set('track_errors', $track);
         throw new \Exception("HTTP Error calling Kannel API - stream Error: $php_errormsg");
       }
-      ini_set('track_errors',$track);
+      ini_set('track_errors', $track);
       return $response;
     } else {
       throw new \Exception("Kannel requires PHP5 with cURL or HTTP stream support");
@@ -624,7 +624,7 @@ 
   * @author  Martin Steel
   */
   protected function is_assoc($array) {
-    return (bool)count(array_filter(array_keys($array), 'is_string'));
+    return (bool) count(array_filter(array_keys($array), 'is_string'));
   }
   
   /**
@@ -637,7 +637,7 @@ 
    * @todo Take an optional country code and check that the number starts with it
    */
   public static function is_valid_msisdn($val) {
-    return preg_match( '/^[1-9][0-9]{7,12}$/', $val );
+    return preg_match('/^[1-9][0-9]{7,12}$/', $val);
   }
 
 }

plugin/openmeetings/lib/openmeetings.class.php

Doc Comments +2 added lines, -1 removed lines

@@ -213,7 +213,8 @@
     /**
      * Returns a meeting "join" URL
      * @param string The name of the meeting (usually the course code)
-     * @return mixed The URL to join the meeting, or false on error
+     * @param false|string $meetingId
+     * @return false|null The URL to join the meeting, or false on error
      * @todo implement moderator pass
      * @assert ('') === false
      * @assert ('abcdefghijklmnopqrstuvwxyzabcdefghijklmno') === false

Spacing +10 added lines, -10 removed lines

@@ -49,9 +49,9 @@ 
         $om_user   = $plugin->get('user');
         $om_pass   = $plugin->get('pass');
         $accessUrl = api_get_access_url($_configuration['access_url']);
-        $this->externalType = substr($accessUrl['url'], strpos($accessUrl['url'], '://')+3, -1);
+        $this->externalType = substr($accessUrl['url'], strpos($accessUrl['url'], '://') + 3, -1);
         if (strcmp($this->externalType, 'localhost') == 0) {
-            $this->externalType = substr(api_get_path(WEB_PATH), strpos(api_get_path(WEB_PATH), '://')+3, -1);
+            $this->externalType = substr(api_get_path(WEB_PATH), strpos(api_get_path(WEB_PATH), '://') + 3, -1);
         }
         $this->externalType = 'chamilolms.'.$this->externalType;
         $this->table = \Database::get_main_table('plugin_openmeetings');
@@ -161,7 +161,7 @@ 
             // This fills the following attributes: status, name, comment, chamiloCourseId, chamiloSessionId
             $room = new Room();
             $room->loadRoomId($meetingData['room_id']);
-            $roomArray = (array)$room;
+            $roomArray = (array) $room;
             $roomArray['SID'] = $this->sessionId;
             $roomId = $this->gateway->updateRoomWithModeration($room);
             if ($roomId != $meetingData['room_id']) {
@@ -176,7 +176,7 @@ 
             $room->SID = $this->sessionId;
             $room->name = $this->roomName;
             //$room->roomtypes_id = $room->roomtypes_id;
-            $room->comment = urlencode(get_lang('Course').': ' . $params['meeting_name'] . ' - '.$_configuration['software_name']);
+            $room->comment = urlencode(get_lang('Course').': '.$params['meeting_name'].' - '.$_configuration['software_name']);
             //$room->numberOfPartizipants = $room->numberOfPartizipants;
             $room->ispublic = $room->getString('isPublic', 'false');
             //$room->appointment = $room->getString('appointment');
@@ -200,7 +200,7 @@ 
             $params['room_id'] = $roomId;
             $params['c_id'] = api_get_course_int_id();
             $params['session_id'] = api_get_session_id();
-            $params['record'] = ($room->allowRecording?1:0);
+            $params['record'] = ($room->allowRecording ? 1 : 0);
 
             $id = \Database::insert($this->table, $params);
 
@@ -238,7 +238,7 @@ 
         }
         $params = array('room_id' => $meetingData['room_id']);
         $returnVal = $this->setUserObjectAndGenerateRoomHashByURLAndRecFlag($params);
-        $iframe = $this->url . "/?" ."secureHash=" . $returnVal;
+        $iframe = $this->url."/?"."secureHash=".$returnVal;
         printf("<iframe src='%s' width='%s' height = '%s' />", $iframe, "100%", 640);
     }
 
@@ -346,7 +346,7 @@ 
         $userId = $_SESSION['_user']['user_id'];
         $systemType = 'Chamilo';
         $room_id = $params['room_id'];
-        $becomeModerator = ( $this->isTeacher() ? 1 : 0 );
+        $becomeModerator = ($this->isTeacher() ? 1 : 0);
         $allowRecording = 1; //Provisional
 
         $urlWsdl = $this->url."/services/UserService?wsdl";
@@ -420,7 +420,7 @@ 
                 error_log(__FILE__.'+'.__LINE__.' Meetings found: '.print_r($meetingDb, 1));
                 $remoteMeeting = array();
                 $meetingDb['created_at'] = api_get_local_time($meetingDb['created_at']);
-                $meetingDb['closed_at'] = (!empty($meetingDb['closed_at']) ? api_get_local_time($meetingDb['closed_at']):'');
+                $meetingDb['closed_at'] = (!empty($meetingDb['closed_at']) ? api_get_local_time($meetingDb['closed_at']) : '');
                 // Fixed value for now
                 $meetingDb['participantCount'] = 40;
                 $rec = $this->gateway->getFlvRecordingByRoomId($meetingDb['room_id']);
@@ -436,7 +436,7 @@ 
                         \Display::url('[.avi]', $link2, array('target' => '_blank'));
 
                 }
-                $item['show_links']  = implode('<br />', $links);
+                $item['show_links'] = implode('<br />', $links);
 
                 // The following code is currently commented because the web service
                 // says this is not allowed by the SOAP user.
@@ -486,7 +486,7 @@ 
                 $remoteMeeting = $current_room;
                 */
 
-                if (empty( $remoteMeeting )) {
+                if (empty($remoteMeeting)) {
                 /*
                     error_log(__FILE__.'+'.__LINE__.' Empty remote Meeting for now');
                     if ($meetingDb['status'] == 1 && $this->isTeacher()) {

Indentation +6 added lines, -6 removed lines

@@ -126,7 +126,7 @@ 
             return false;
         }
     }
-    */
+     */
 
     /*
      * Creating a Room for the meeting
@@ -256,10 +256,10 @@ 
         return true;
     }
 
-     /**
-     * Gets the password for a specific meeting for the current user
-     * @return string A moderator password if user is teacher, or the course code otherwise
-     */
+        /**
+         * Gets the password for a specific meeting for the current user
+         * @return string A moderator password if user is teacher, or the course code otherwise
+         */
     public function getMeetingUserPassword()
     {
         if ($this->isTeacher()) {
@@ -538,7 +538,7 @@ 
     //
     //            }
     //
-                 //$item['created_at'] = api_convert_and_format_date($meetingDb['created_at']);
+                    //$item['created_at'] = api_convert_and_format_date($meetingDb['created_at']);
     //            //created_at
     //
     //            $item['publish_url'] = api_get_self().'?action=publish&id='.$meetingDb['id'];