diff --git a/plugin/bbb/lib/bbb.lib.php b/plugin/bbb/lib/bbb.lib.php
index f112671502..b064e56169 100755
--- a/plugin/bbb/lib/bbb.lib.php
+++ b/plugin/bbb/lib/bbb.lib.php
@@ -579,7 +579,7 @@ class bbb
             }
         }
 
-        if (strval($meetingIsRunningInfo['returncode']) == 'SUCCESS' &&
+        if (strval($meetingIsRunningInfo['returncode']) === 'SUCCESS' &&
             isset($meetingIsRunningInfo['meetingName']) &&
             !empty($meetingIsRunningInfo['meetingName'])
         ) {
@@ -1140,7 +1140,7 @@ class bbb
 
         // Check if there are recordings for this meeting
         $recordings = $this->api->getRecordings(['meetingId' => $meetingData['remote_id']]);
-        if (!empty($recordings) && isset($recordings['messageKey']) && $recordings['messageKey'] == 'noRecordings') {
+        if (!empty($recordings) && isset($recordings['messageKey']) && $recordings['messageKey'] === 'noRecordings') {
             return false;
         } else {
             if (!empty($recordings['records'])) {
diff --git a/plugin/bbb/lib/bbb_api.php b/plugin/bbb/lib/bbb_api.php
index 82decef55f..a8f10856a5 100755
--- a/plugin/bbb/lib/bbb_api.php
+++ b/plugin/bbb/lib/bbb_api.php
@@ -647,6 +647,10 @@ class BigBlueButtonBN {
 	 */
 	public function generateRecording($recordingParams)
 	{
+	    if (empty($recordingParams)) {
+	        return false;
+        }
+
 		$recordingsUrl = $this->_bbbServerBaseUrl.'../demo/regenerateRecord.jsp?';
 		$params = 'recordID='.urlencode($recordingParams['recordId']);
 		$url = $recordingsUrl.$params.'&checksum='.sha1('regenerateRecord'.$params.$this->_securitySalt);
@@ -662,4 +666,4 @@ class BigBlueButtonBN {
 
         return true;
 	}
-}
\ No newline at end of file
+}
diff --git a/plugin/bbb/listing.php b/plugin/bbb/listing.php
index efc1324099..7241b2bc7b 100755
--- a/plugin/bbb/listing.php
+++ b/plugin/bbb/listing.php
@@ -67,13 +67,16 @@ if ($conferenceManager) {
                 api_not_allowed();
             }
             $result = $bbb->regenerateRecording($_GET['id'], $_GET['record_id']);
+
             if ($result) {
                 $message = Display::return_message(get_lang('Success'), 'success');
             } else {
                 $message = Display::return_message(get_lang('Error'), 'error');
             }
+
             Display::addFlash($message);
             header('Location: '.$bbb->getListingUrl());
+            exit;
             break;
         case 'delete_record':
             $result = $bbb->deleteRecording($_GET['id']);
@@ -150,7 +153,6 @@ if (($meetingExists || $userCanSeeJoinButton) && ($maxUsers == 0 || $maxUsers >
     $showJoinButton = true;
 }
 $conferenceUrl = $bbb->getConferenceUrl();
-
 $courseInfo = api_get_course_info();
 $formToString = '';
 
@@ -161,7 +163,7 @@ if ($bbb->isGlobalConference() === false &&
 ) {
     $url = api_get_self().'?'.api_get_cidreq(true, false).'&gidReq=';
     $htmlHeadXtra[] = '<script>        
-        $(document).ready(function(){
+        $(document).ready(function() {
             $("#group_select").on("change", function() {
                 var groupId = $(this).find("option:selected").val();
                 var url = "'.$url.'";                
@@ -193,18 +195,6 @@ if ($bbb->isGlobalConference() === false &&
     }
 }
 
-$tpl = new Template($tool_name);
-$tpl->assign('allow_to_edit', $conferenceManager);
-$tpl->assign('meetings', $meetings);
-$tpl->assign('conference_url', $conferenceUrl);
-$tpl->assign('users_online', $usersOnline);
-$tpl->assign('conference_manager', $conferenceManager);
-$tpl->assign('max_users_limit', $maxUsers);
-$tpl->assign('bbb_status', $status);
-$tpl->assign('show_join_button', $showJoinButton);
-$tpl->assign('message', $message);
-$tpl->assign('form', $formToString);
-
 // Default URL
 $urlList[] = Display::url(
     $plugin->get_lang('EnterConference'),
@@ -213,7 +203,7 @@ $urlList[] = Display::url(
 );
 
 $type = $plugin->get('launch_type');
-$warningIntefaceMessage = '';
+$warningInterfaceMessage = '';
 $showClientOptions = false;
 
 switch ($type) {
@@ -228,7 +218,7 @@ switch ($type) {
     case BBBPlugin::LAUNCH_TYPE_SET_BY_TEACHER:
         if ($conferenceManager) {
             $urlList = $plugin->getUrlInterfaceLinks($conferenceUrl);
-            $warningIntefaceMessage = Display::return_message($plugin->get_lang('ParticipantsWillUseSameInterface'));
+            $warningInterfaceMessage = Display::return_message($plugin->get_lang('ParticipantsWillUseSameInterface'));
             $showClientOptions = true;
         } else {
             $meetingInfo = $bbb->getMeetingByName($videoConferenceName);
@@ -248,8 +238,8 @@ switch ($type) {
             $showClientOptions = true;
         } else {
             if ($meetingExists) {
-                $meetingInfo = $bbb->getMeetingByName($videoConferenceName);
-                $meetinUserInfo = $bbb->getMeetingParticipantInfo($meetingInfo['id'], api_get_user_id());
+                //$meetingInfo = $bbb->getMeetingByName($videoConferenceName);
+                //$meetinUserInfo = $bbb->getMeetingParticipantInfo($meetingInfo['id'], api_get_user_id());
                 $urlList = $plugin->getUrlInterfaceLinks($conferenceUrl);
                 $showClientOptions = true;
             }
@@ -257,13 +247,24 @@ switch ($type) {
 
         break;
 }
-
+$tpl = new Template($tool_name);
+$tpl->assign('allow_to_edit', $conferenceManager);
+$tpl->assign('meetings', $meetings);
+$tpl->assign('conference_url', $conferenceUrl);
+$tpl->assign('users_online', $usersOnline);
+$tpl->assign('conference_manager', $conferenceManager);
+$tpl->assign('max_users_limit', $maxUsers);
+$tpl->assign('bbb_status', $status);
+$tpl->assign('show_join_button', $showJoinButton);
+$tpl->assign('message', $message);
+$tpl->assign('form', $formToString);
 $tpl->assign('enter_conference_links', $urlList);
-$tpl->assign('warning_inteface_msg', $warningIntefaceMessage);
+$tpl->assign('warning_inteface_msg', $warningInterfaceMessage);
 $tpl->assign('show_client_options', $showClientOptions);
 
 $listing_tpl = 'bbb/view/listing.tpl';
 $content = $tpl->fetch($listing_tpl);
+
 $actionLinks = '';
 if (api_is_platform_admin()) {
     $actionLinks .= Display::toolbarButton(
diff --git a/plugin/bbb/start.php b/plugin/bbb/start.php
index 2eb34b5149..30a47ae65d 100755
--- a/plugin/bbb/start.php
+++ b/plugin/bbb/start.php
@@ -13,10 +13,6 @@ require_once __DIR__.'/config.php';
 
 $logInfo = [
     'tool' => 'Videoconference',
-    'tool_id' => 0,
-    'tool_id_detail' => 0,
-    'action' => '',
-    'action_details' => '',
 ];
 Event::registerLog($logInfo);
 
@@ -68,7 +64,6 @@ if ($bbb->pluginEnabled) {
             $meetingParams = [];
             $meetingParams['meeting_name'] = $bbb->getCurrentVideoConferenceName();
             $meetingParams['interface'] = $interface;
-
             if ($bbb->meetingExists($meetingParams['meeting_name'])) {
                 $joinUrl = $bbb->joinMeeting($meetingParams['meeting_name']);
                 if ($joinUrl) {