Minor - Removed notices

main/webservices/cm_webservice_course.php

@@ -62,7 +62,7 @@ class WSCMCourse extends WSCM {
 			$this->handleError($verifKey);
 		} else {
 			$results = array();
-			foreach($users as $user) {
+			foreach($courses as $course) {
 				$result_tmp = array();
 				$result_op = $this->deleteCourseHelper($course['course_id_field_name'], $course['course_id_value']);
 				$result_tmp['course_id_value'] = $course['course_id_value'];
@@ -170,6 +170,8 @@ class WSCMCourse extends WSCM {
 			$results = array();
 			foreach($courses as $course) {
 				$result_tmp = array();
+                //reinitialize variables just in case
+                $title = $category_code = $wanted_code = $tutor_name = $course_admin_user_id_field_name = $course_admin_user_id_value = $language = $course_id_field_name = $course_id_value = $extras = null;
 				extract($course);
 				$result = $this->createCourseHelper($title, $category_code, $wanted_code, $tutor_name, $course_admin_user_id_field_name, $course_admin_user_id_value, $language, $course_id_field_name, $course_id_value, $extras);
 				if($result instanceof WSCMError) {
@@ -322,7 +324,7 @@ class WSCMCourse extends WSCM {
 				$course_tmp['number_students'] = count($user_list);
 
 				// Determining external course id
-				$course_tmp['external_course_id'] = CourseManager::get_course_extra_field_value($course_field_name, $course['code']);
+				$course_tmp['external_course_id'] = CourseManager::get_course_extra_field_value($course_id_field_name, $course['code']);
 
 
 				$courses_result[] = $course_tmp;

main/webservices/cm_webservice_user.php

@@ -94,7 +94,8 @@ class WSCMUser extends WSCM {
     }
 
     public function send_invitation($username, $password, $userfriend_id, $content_message = '')
-    { 
+    {
+        global $charset;
         if($this->verifyUserPass($username, $password) == "valid")
         { 
 		$user_id = UserManager::get_user_id_from_username($username); 

main/webservices/webservice_course.php

@@ -173,6 +173,8 @@ class WSCourse extends WS {
 			$results = array();
 			foreach($courses as $course) {
 				$result_tmp = array();
+                // re-initialize variables just in case
+                $title = $category_code = $wanted_code = $tutor_name = $course_admin_user_id_field_name = $course_admin_user_id_value = $language = $course_id_field_name = $course_id_value = $extras = 0;
 				extract($course);
 				$result = $this->createCourseHelper($title, $category_code, $wanted_code, $tutor_name, $course_admin_user_id_field_name, $course_admin_user_id_value, $language, $course_id_field_name, $course_id_value, $extras);
 				if($result instanceof WSError) {

main/webservices/webservice_user.php

@@ -297,6 +297,8 @@ class WSUser extends WS {
 			$results = array();
 			foreach($users as $user) {
 				$result_tmp = array();
+                // re-initialize variables just in case
+                $firstname = $lastname = $status = $login = $password = $encrypt_method = $user_id_field_name = $user_id_value = $visibility = $email = $language = $phone = $expiration_date = $extras = null;
 				extract($user);
 				$result = $this->createUserHelper($firstname, $lastname, $status, $login, $password, $encrypt_method, $user_id_field_name, $user_id_value, $visibility, $email, $language, $phone, $expiration_date, $extras);
 				if($result instanceof WSError) {
@@ -406,6 +408,8 @@ class WSUser extends WS {
 			$results = array();
 			foreach($users as $user) {
 				$result_tmp = array();
+                // re-initialize variables just in case
+                $user_id_field_name = $user_id_value = $firstname = $lastname = $status = $loginname = $password = $encrypt_method = $email = $language = $phone = $expiration_date = $extras = null;
 				extract($user);
 				$result_op = $this->editUserHelper($user_id_field_name, $user_id_value, $firstname, $lastname, $status, $loginname, $password, $encrypt_method, $email, $language, $phone, $expiration_date, $extras);
 				$result_tmp['user_id_value'] = $user['user_id_value'];