chamilo / chamilo-lms

main/inc/lib/CourseChatUtils.php

Doc Comments +2 added lines, -2 removed lines

@@ -1415,7 +1415,7 @@ 
 
     /**
      * Get the emoji list to include in chat
-     * @return array
+     * @return string[]
      */
     public static function getEmojisToInclude()
     {
@@ -1738,7 +1738,7 @@ 
 
     /**
      * Get the number of users connected in chat
-     * @return mixed
+     * @return integer
      */
     public function countUsersOnline()
     {

Spacing +65 added lines, -65 removed lines

@@ -62,7 +62,7 @@ 
             return '';
         }
 
-        Emojione\Emojione::$imagePathPNG = api_get_path(WEB_LIBRARY_PATH) . 'javascript/emojione/png/';
+        Emojione\Emojione::$imagePathPNG = api_get_path(WEB_LIBRARY_PATH).'javascript/emojione/png/';
         Emojione\Emojione::$ascii = true;
 
         $message = trim($message);
@@ -95,15 +95,15 @@ 
         $user = $em->find('ChamiloUserBundle:User', $this->userId);
         $courseInfo = api_get_course_info_by_id($this->courseId);
         $isMaster = (bool) api_is_course_admin();
-        $document_path = api_get_path(SYS_COURSE_PATH) . $courseInfo['path'] . '/document';
+        $document_path = api_get_path(SYS_COURSE_PATH).$courseInfo['path'].'/document';
         $basepath_chat = '/chat_files';
 
         if (!$this->groupId) {
             $group_info = GroupManager::get_group_properties($this->groupId);
-            $basepath_chat = $group_info['directory'] . '/chat_files';
+            $basepath_chat = $group_info['directory'].'/chat_files';
         }
 
-        $chat_path = $document_path . $basepath_chat . '/';
+        $chat_path = $document_path.$basepath_chat.'/';
 
         if (!is_dir($chat_path)) {
             if (is_file($chat_path)) {
@@ -113,25 +113,25 @@ 
 
         $date_now = date('Y-m-d');
         $timeNow = date('d/m/y H:i:s');
-        $basename_chat = 'messages-' . $date_now;
+        $basename_chat = 'messages-'.$date_now;
 
         if ($this->groupId && !$friendId) {
-            $basename_chat = 'messages-' . $date_now . '_gid-' . $this->groupId;
+            $basename_chat = 'messages-'.$date_now.'_gid-'.$this->groupId;
         } elseif ($this->sessionId && !$friendId) {
-            $basename_chat = 'messages-' . $date_now . '_sid-' . $this->sessionId;
+            $basename_chat = 'messages-'.$date_now.'_sid-'.$this->sessionId;
         } elseif ($friendId) {
             if ($this->userId < $friendId) {
-                $basename_chat = 'messages-' . $date_now . '_uid-' . $this->userId . '-' . $friendId;
+                $basename_chat = 'messages-'.$date_now.'_uid-'.$this->userId.'-'.$friendId;
             } else {
-                $basename_chat = 'messages-' . $date_now . '_uid-' . $friendId . '-' . $this->userId;
+                $basename_chat = 'messages-'.$date_now.'_uid-'.$friendId.'-'.$this->userId;
             }
         }
 
         $message = self::prepareMessage($message);
 
-        $fileTitle = $basename_chat . '.log.html';
-        $filePath = $basepath_chat . '/' . $fileTitle;
-        $absoluteFilePath = $chat_path . $fileTitle;
+        $fileTitle = $basename_chat.'.log.html';
+        $filePath = $basepath_chat.'/'.$fileTitle;
+        $absoluteFilePath = $chat_path.$fileTitle;
 
         if (!file_exists($absoluteFilePath)) {
             $doc_id = add_document($courseInfo, $filePath, 'file', 0, $fileTitle);
@@ -164,23 +164,23 @@ 
             $fileContent = '
                 <div class="message-teacher">
                     <div class="content-message">
-                        <div class="chat-message-block-name">' . $user->getCompleteName() . '</div>
-                        <div class="chat-message-block-content">' . $message . '</div>
-                        <div class="message-date">' . $timeNow . '</div>
+                        <div class="chat-message-block-name">' . $user->getCompleteName().'</div>
+                        <div class="chat-message-block-content">' . $message.'</div>
+                        <div class="message-date">' . $timeNow.'</div>
                     </div>
                     <div class="icon-message"></div>
-                    <img class="chat-image" src="' . $userPhoto . '">
+                    <img class="chat-image" src="' . $userPhoto.'">
                 </div>
             ';
         } else {
             $fileContent = '
                 <div class="message-student">
-                    <img class="chat-image" src="' . $userPhoto . '">
+                    <img class="chat-image" src="' . $userPhoto.'">
                     <div class="icon-message"></div>
                     <div class="content-message">
-                        <div class="chat-message-block-name">' . $user->getCompleteName() . '</div>
-                        <div class="chat-message-block-content">' . $message . '</div>
-                        <div class="message-date">' . $timeNow . '</div>
+                        <div class="chat-message-block-name">' . $user->getCompleteName().'</div>
+                        <div class="chat-message-block-content">' . $message.'</div>
+                        <div class="message-date">' . $timeNow.'</div>
                     </div>
                 </div>
             ';
@@ -275,9 +275,9 @@ 
         $extraCondition = null;
 
         if ($this->groupId) {
-            $extraCondition = 'AND ccc.toGroupId = ' . intval($this->groupId);
+            $extraCondition = 'AND ccc.toGroupId = '.intval($this->groupId);
         } else {
-            $extraCondition = 'AND ccc.sessionId = ' . intval($this->sessionId);
+            $extraCondition = 'AND ccc.sessionId = '.intval($this->sessionId);
         }
 
         $currentTime = new DateTime(api_get_utc_datetime(), new DateTimeZone('UTC'));
@@ -1413,7 +1413,7 @@ 
     {
         return [
             ':bowtie:',
-            ':smile:'|
+            ':smile:' |
             ':laughing:',
             ':blush:',
             ':smiley:',
@@ -1487,17 +1487,17 @@ 
     {
         $date = date('Y-m-d');
 
-        $base = 'messages-' . $date . '.log.html';
+        $base = 'messages-'.$date.'.log.html';
 
         if ($this->groupId && !$friendId) {
-            $base = 'messages-' . $date . '_gid-' . $this->groupId . '.log.html';
+            $base = 'messages-'.$date.'_gid-'.$this->groupId.'.log.html';
         } elseif ($this->sessionId && !$friendId) {
-            $base = 'messages-' . $date . '_sid-' . $this->sessionId . '.log.html';
+            $base = 'messages-'.$date.'_sid-'.$this->sessionId.'.log.html';
         } elseif ($friendId) {
             if ($this->userId < $friendId) {
-                $base = 'messages-' . $date . '_uid-' . $this->userId . '-' . $friendId . '.log.html';
+                $base = 'messages-'.$date.'_uid-'.$this->userId.'-'.$friendId.'.log.html';
             } else {
-                $base = 'messages-' . $date . '_uid-' . $friendId . '-' . $this->userId . '.log.html';
+                $base = 'messages-'.$date.'_uid-'.$friendId.'-'.$this->userId.'.log.html';
             }
         }
 
@@ -1506,16 +1506,16 @@ 
         }
 
         $courseInfo = api_get_course_info_by_id($this->courseId);
-        $document_path = api_get_path(SYS_COURSE_PATH) . $courseInfo['path'] . '/document';
+        $document_path = api_get_path(SYS_COURSE_PATH).$courseInfo['path'].'/document';
 
-        $chatPath = $document_path . '/chat_files/';
+        $chatPath = $document_path.'/chat_files/';
 
         if ($this->groupId) {
             $group_info = GroupManager::get_group_properties($this->groupId);
-            $chatPath = $document_path . $group_info['directory'] . '/chat_files/';
+            $chatPath = $document_path.$group_info['directory'].'/chat_files/';
         }
 
-        return $chatPath . $base;
+        return $chatPath.$base;
     }
 
     /**
@@ -1528,16 +1528,16 @@ 
     {
         $courseInfo = api_get_course_info_by_id($this->courseId);
         $date_now = date('Y-m-d');
-        $isMaster = (bool)api_is_course_admin();
+        $isMaster = (bool) api_is_course_admin();
         $basepath_chat = '/chat_files';
-        $document_path = api_get_path(SYS_COURSE_PATH) . $courseInfo['path'] . '/document';
+        $document_path = api_get_path(SYS_COURSE_PATH).$courseInfo['path'].'/document';
 
         if ($this->groupId) {
             $group_info = GroupManager:: get_group_properties($this->groupId);
-            $basepath_chat = $group_info['directory'] . '/chat_files';
+            $basepath_chat = $group_info['directory'].'/chat_files';
         }
 
-        $chat_path = $document_path . $basepath_chat . '/';
+        $chat_path = $document_path.$basepath_chat.'/';
 
         if (!is_dir($chat_path)) {
             if (is_file($chat_path)) {
@@ -1564,24 +1564,24 @@ 
             }
         }
 
-        $filename_chat = 'messages-' . $date_now . '.log.html';
+        $filename_chat = 'messages-'.$date_now.'.log.html';
 
         if ($this->groupId && !$friendId) {
-            $filename_chat = 'messages-' . $date_now . '_gid-' . $this->groupId . '.log.html';
+            $filename_chat = 'messages-'.$date_now.'_gid-'.$this->groupId.'.log.html';
         } else if ($this->sessionId && !$friendId) {
-            $filename_chat = 'messages-' . $date_now . '_sid-' . $this->sessionId . '.log.html';
+            $filename_chat = 'messages-'.$date_now.'_sid-'.$this->sessionId.'.log.html';
         } elseif ($friendId) {
             if ($this->userId < $friendId) {
-                $filename_chat = 'messages-' . $date_now . '_uid-' . $this->userId . '-' . $friendId . '.log.html';
+                $filename_chat = 'messages-'.$date_now.'_uid-'.$this->userId.'-'.$friendId.'.log.html';
             } else {
-                $filename_chat = 'messages-' . $date_now . '_uid-' . $friendId . '-' . $this->userId . '.log.html';
+                $filename_chat = 'messages-'.$date_now.'_uid-'.$friendId.'-'.$this->userId.'.log.html';
             }
         }
 
-        if (!file_exists($chat_path . $filename_chat)) {
-            @fclose(fopen($chat_path . $filename_chat, 'w'));
+        if (!file_exists($chat_path.$filename_chat)) {
+            @fclose(fopen($chat_path.$filename_chat, 'w'));
             if (!api_is_anonymous()) {
-                $doc_id = add_document($courseInfo, $basepath_chat . '/' . $filename_chat, 'file', 0, $filename_chat);
+                $doc_id = add_document($courseInfo, $basepath_chat.'/'.$filename_chat, 'file', 0, $filename_chat);
                 api_item_property_update(
                     $courseInfo,
                     TOOL_DOCUMENT,
@@ -1610,17 +1610,17 @@ 
             }
         }
 
-        $basename_chat = 'messages-' . $date_now;
+        $basename_chat = 'messages-'.$date_now;
 
         if ($this->groupId && !$friendId) {
-            $basename_chat = 'messages-' . $date_now . '_gid-' . $this->groupId;
+            $basename_chat = 'messages-'.$date_now.'_gid-'.$this->groupId;
         } else if ($this->sessionId && !$friendId) {
-            $basename_chat = 'messages-' . $date_now . '_sid-' . $this->sessionId;
+            $basename_chat = 'messages-'.$date_now.'_sid-'.$this->sessionId;
         } elseif ($friendId) {
             if ($this->userId < $friendId) {
-                $basename_chat = 'messages-' . $date_now . '_uid-' . $this->userId . '-' . $friendId;
+                $basename_chat = 'messages-'.$date_now.'_uid-'.$this->userId.'-'.$friendId;
             } else {
-                $basename_chat = 'messages-' . $date_now . '_uid-' . $friendId . '-' . $this->userId;
+                $basename_chat = 'messages-'.$date_now.'_uid-'.$friendId.'-'.$this->userId;
             }
         }
 
@@ -1628,19 +1628,19 @@ 
 
             $i = 1;
 
-            while (file_exists($chat_path . $basename_chat . '-' . $i . '.log.html')) {
+            while (file_exists($chat_path.$basename_chat.'-'.$i.'.log.html')) {
                 $i++;
             }
 
-            @rename($chat_path . $basename_chat . '.log.html', $chat_path . $basename_chat . '-' . $i . '.log.html');
-            @fclose(fopen($chat_path . $basename_chat . '.log.html', 'w'));
+            @rename($chat_path.$basename_chat.'.log.html', $chat_path.$basename_chat.'-'.$i.'.log.html');
+            @fclose(fopen($chat_path.$basename_chat.'.log.html', 'w'));
 
             $doc_id = add_document(
                 $courseInfo,
-                $basepath_chat . '/' . $basename_chat . '-' . $i . '.log.html',
+                $basepath_chat.'/'.$basename_chat.'-'.$i.'.log.html',
                 'file',
-                filesize($chat_path . $basename_chat . '-' . $i . '.log.html'),
-                $basename_chat . '-' . $i . '.log.html'
+                filesize($chat_path.$basename_chat.'-'.$i.'.log.html'),
+                $basename_chat.'-'.$i.'.log.html'
             );
 
             api_item_property_update(
@@ -1671,7 +1671,7 @@ 
 
             $doc_id = DocumentManager::get_document_id(
                 $courseInfo,
-                $basepath_chat . '/' . $basename_chat . '.log.html'
+                $basepath_chat.'/'.$basename_chat.'.log.html'
             );
 
             update_existing_document($courseInfo, $doc_id, 0);
@@ -1680,8 +1680,8 @@ 
         $remove = 0;
         $content = array();
 
-        if (file_exists($chat_path . $basename_chat . '.log.html')) {
-            $content = file($chat_path . $basename_chat . '.log.html');
+        if (file_exists($chat_path.$basename_chat.'.log.html')) {
+            $content = file($chat_path.$basename_chat.'.log.html');
             $nbr_lines = sizeof($content);
             $remove = $nbr_lines - 100;
         }
@@ -1694,7 +1694,7 @@ 
 
         if (isset($_GET['origin']) && $_GET['origin'] == 'whoisonline') {
             //the caller
-            $content[0] = get_lang('CallSent') . '<br />' . $content[0];
+            $content[0] = get_lang('CallSent').'<br />'.$content[0];
         }
         if (isset($_GET['origin']) && $_GET['origin'] == 'whoisonlinejoin') {
             //the joiner (we have to delete the chat request to him when he joins the chat)
@@ -1719,7 +1719,7 @@ 
             $history .= '
                 <div id="clear-chat">
                     <button type="button" id="chat-reset" class="btn btn-danger btn-sm">
-                        ' . get_lang('ClearList') . '
+                        ' . get_lang('ClearList').'
                     </button>
                 </div>
             ';
@@ -1740,9 +1740,9 @@ 
         $extraCondition = null;
 
         if ($this->groupId) {
-            $extraCondition = 'AND ccc.toGroupId = ' . intval($this->groupId);
+            $extraCondition = 'AND ccc.toGroupId = '.intval($this->groupId);
         } else {
-            $extraCondition = 'AND ccc.sessionId = ' . intval($this->sessionId);
+            $extraCondition = 'AND ccc.sessionId = '.intval($this->sessionId);
         }
 
         $number = Database::getManager()
@@ -1772,9 +1772,9 @@ 
         $extraCondition = null;
 
         if ($this->groupId) {
-            $extraCondition = 'AND ccc.toGroupId = ' . intval($this->groupId);
+            $extraCondition = 'AND ccc.toGroupId = '.intval($this->groupId);
         } else {
-            $extraCondition = 'AND ccc.sessionId = ' . intval($this->sessionId);
+            $extraCondition = 'AND ccc.sessionId = '.intval($this->sessionId);
         }
 
         $number = Database::getManager()
@@ -1810,7 +1810,7 @@ 
                 'lastname' => $user->getLastname(),
                 'status' => !$this->sessionId ? $subscription->getStatus() : $user->getStatus(),
                 'image_url' => UserManager::getUserPicture($user->getId(), USER_IMAGE_SIZE_MEDIUM),
-                'profile_url' => api_get_path(WEB_CODE_PATH) . 'social/profile.php?u=' . $user->getId(),
+                'profile_url' => api_get_path(WEB_CODE_PATH).'social/profile.php?u='.$user->getId(),
                 'complete_name' => $user->getCompleteName(),
                 'username' => $user->getUsername(),
                 'email' => $user->getEmail(),

main/chat/chat.php

Spacing +4 added lines, -4 removed lines

@@ -17,11 +17,11 @@ 
 ];
 
 foreach ($externalCSS as $css) {
-    $htmlHeadXtra[] = api_get_css(api_get_path(WEB_LIBRARY_JS_PATH) . $css);
+    $htmlHeadXtra[] = api_get_css(api_get_path(WEB_LIBRARY_JS_PATH).$css);
 }
 
-$htmlHeadXtra[] = api_get_css(api_get_path(WEB_CSS_PATH) . 'chat.css');
-$htmlHeadXtra[] = api_get_css(api_get_path(WEB_CSS_PATH) . 'markdown.css');
+$htmlHeadXtra[] = api_get_css(api_get_path(WEB_CSS_PATH).'chat.css');
+$htmlHeadXtra[] = api_get_css(api_get_path(WEB_CSS_PATH).'markdown.css');
 
 $externalJS = [
     'highlight/highlight.pack.js',
@@ -41,7 +41,7 @@ 
         continue;
     }
 
-    $iconList[$key] = strtoupper($icon) . '.png';
+    $iconList[$key] = strtoupper($icon).'.png';
 }
 
 $view = new Template(get_lang('Chat'), false, false, false, true, false);

main/inc/lib/online.inc.php

Spacing +37 added lines, -37 removed lines

@@ -32,17 +32,17 @@ 
 
         $login_date = api_get_utc_datetime();
         $access_url_id = 1;
-        if (api_get_multiple_access_url() && api_get_current_access_url_id()!=-1) {
+        if (api_get_multiple_access_url() && api_get_current_access_url_id() != -1) {
             $access_url_id = api_get_current_access_url_id();
         }
         $session_id = api_get_session_id();
         // if the $_course array exists this means we are in a course and we have to store this in the who's online table also
         // to have the x users in this course feature working
-        if (is_array($_course) && count($_course)>0 && !empty($_course['id'])) {
-            $query = "REPLACE INTO ".$online_table ." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
+        if (is_array($_course) && count($_course) > 0 && !empty($_course['id'])) {
+            $query = "REPLACE INTO ".$online_table." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
                       VALUES ($uid,$uid,'$login_date','$user_ip', '".$_course['real_id']."' , '$session_id' , '$access_url_id' )";
         } else {
-            $query = "REPLACE INTO ".$online_table ." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
+            $query = "REPLACE INTO ".$online_table." (login_id,login_user_id,login_date,user_ip, c_id, session_id, access_url_id)
                       VALUES ($uid,$uid,'$login_date','$user_ip', 0, '$session_id', '$access_url_id')";
         }
         Database::query($query);
@@ -63,7 +63,7 @@ 
             $isFirstLogin = Session::read('first_user_login');
             if (empty($isFirstLogin)) {
                 $sql = "SELECT login_id FROM $table
-                        WHERE login_user_id = " . $userId . " LIMIT 1";
+                        WHERE login_user_id = ".$userId." LIMIT 1";
 
                 $result = Database::query($sql);
                 $loginData = array();
@@ -117,8 +117,8 @@ 
     		ORDER BY login_date DESC
     		LIMIT 0,1";
     $q_last_connection = Database::query($sql);
-    if (Database::num_rows($q_last_connection)>0) {
-        $i_id_last_connection = Database::result($q_last_connection,0,"login_id");
+    if (Database::num_rows($q_last_connection) > 0) {
+        $i_id_last_connection = Database::result($q_last_connection, 0, "login_id");
     }
 
     if (!isset($_SESSION['login_as'])) {
@@ -167,7 +167,7 @@ 
 {
 	$online_table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
     $user_id = intval($user_id);
-	$query = "DELETE FROM " . $online_table . " WHERE login_user_id = $user_id";
+	$query = "DELETE FROM ".$online_table." WHERE login_user_id = $user_id";
 	Database::query($query);
 }
 
@@ -183,7 +183,7 @@ 
 	$access_url_id = api_get_current_access_url_id();
 	$time_limit = api_get_setting('time_limit_whosonline');
 
-    $online_time = time() - $time_limit*60;
+    $online_time = time() - $time_limit * 60;
     $limit_date = api_get_utc_datetime($online_time);
     $user_id = intval($user_id);
 
@@ -240,7 +240,7 @@ 
     $current_date = api_get_utc_datetime($online_time);
 	$track_online_table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
 	$friend_user_table = Database::get_main_table(TABLE_MAIN_USER_REL_USER);
-	$table_user	= Database::get_main_table(TABLE_MAIN_USER);
+	$table_user = Database::get_main_table(TABLE_MAIN_USER);
 
 	if ($friends) {
 		// 	who friends from social network is online
@@ -256,8 +256,8 @@ 
                   LIMIT $from, $number_of_items";
 	} else {
 		$query = "SELECT DISTINCT login_user_id, login_date
-                    FROM ".$track_online_table ." e
-		            INNER JOIN ".$table_user ." u ON (u.id = e.login_user_id)
+                    FROM ".$track_online_table." e
+		            INNER JOIN ".$table_user." u ON (u.id = e.login_user_id)
                   WHERE u.status != ".ANONYMOUS." AND login_date >= '".$current_date."'
                   ORDER BY $column $direction
                   LIMIT $from, $number_of_items";
@@ -280,8 +280,8 @@ 
 			} else {
 				// all users online
 				$query = "SELECT login_user_id, login_date
-						  FROM ".$track_online_table ." track
-                          INNER JOIN ".$table_user ." u
+						  FROM ".$track_online_table." track
+                          INNER JOIN ".$table_user." u
                           ON (u.id=track.login_user_id)
 						  WHERE u.status != ".ANONYMOUS." AND track.access_url_id =  $access_url_id AND
                                 login_date >= '".$current_date."'
@@ -396,10 +396,10 @@ 
         $time_limit = intval($time_limit);
     }
 
-    $online_time 		= time() - $time_limit*60;
+    $online_time 		= time() - $time_limit * 60;
     $current_date		= api_get_utc_datetime($online_time);
     $track_online_table = Database::get_main_table(TABLE_STATISTIC_TRACK_E_ONLINE);
-    $course_code         = Database::escape_string($course_code);
+    $course_code = Database::escape_string($course_code);
 	$courseInfo = api_get_course_info($course_code);
 	$courseId = $courseInfo['real_id'];
 
@@ -414,7 +414,7 @@ 
 	if ($result) {
 		$users_online = array();
 
-		while(list($login_user_id, $login_date) = Database::fetch_row($result)) {
+		while (list($login_user_id, $login_date) = Database::fetch_row($result)) {
             $users_online[] = $login_user_id;
 		}
 		return $users_online;
@@ -423,7 +423,7 @@ 
 	}
 }
 
-function who_is_online_in_this_course_count($uid, $time_limit, $coursecode=null)
+function who_is_online_in_this_course_count($uid, $time_limit, $coursecode = null)
 {
 	if (empty($coursecode)) {
 		return false;
@@ -464,8 +464,8 @@ 
 	$user_table = Database::get_main_table(TABLE_MAIN_USER);
 	$query = "SELECT firstname, lastname FROM ".$user_table." WHERE id=$uid";
 	$result = @Database::query($query);
-	if (count($result)>0) {
-		while(list($firstname,$lastname)= Database::fetch_array($result)) {
+	if (count($result) > 0) {
+		while (list($firstname, $lastname) = Database::fetch_array($result)) {
 			$str = str_replace(' ', ' ', api_get_person_name($firstname, $lastname));
 			return $str;
 		}
@@ -486,27 +486,27 @@ 
 	}
     $userId = intval($_user['user_id']);
 	$track_user_table = Database::get_main_table(TABLE_MAIN_USER);
-	$sql="SELECT chatcall_user_id, chatcall_date FROM $track_user_table
+	$sql = "SELECT chatcall_user_id, chatcall_date FROM $track_user_table
 	      WHERE ( id = $userId )";
-	$result=Database::query($sql);
-	$row=Database::fetch_array($result);
-
-	$login_date=$row['chatcall_date'];
-	$hour = substr($login_date,11,2);
-	$minute = substr($login_date,14,2);
-	$second = substr($login_date,17,2);
-	$month = substr($login_date,5,2);
-	$day = substr($login_date,8,2);
-	$year = substr($login_date,0,4);
-	$calltime = mktime($hour,$minute,$second,$month,$day,$year);
+	$result = Database::query($sql);
+	$row = Database::fetch_array($result);
+
+	$login_date = $row['chatcall_date'];
+	$hour = substr($login_date, 11, 2);
+	$minute = substr($login_date, 14, 2);
+	$second = substr($login_date, 17, 2);
+	$month = substr($login_date, 5, 2);
+	$day = substr($login_date, 8, 2);
+	$year = substr($login_date, 0, 4);
+	$calltime = mktime($hour, $minute, $second, $month, $day, $year);
 
 	$time = api_get_utc_datetime();
-	$minute_passed=5;  //within this limit, the chat call request is valid
-	$limittime = mktime(date("H"),date("i")-$minute_passed,date("s"),date("m"),date("d"),date("Y"));
+	$minute_passed = 5; //within this limit, the chat call request is valid
+	$limittime = mktime(date("H"), date("i") - $minute_passed, date("s"), date("m"), date("d"), date("Y"));
 
-	if (($row['chatcall_user_id']) and ($calltime>$limittime)) {
-		$webpath=api_get_path(WEB_CODE_PATH);
-		$message=get_lang('YouWereCalled').' : '.GetFullUserName($row['chatcall_user_id'],'').'<br>'.get_lang('DoYouAccept')
+	if (($row['chatcall_user_id']) and ($calltime > $limittime)) {
+		$webpath = api_get_path(WEB_CODE_PATH);
+		$message = get_lang('YouWereCalled').' : '.GetFullUserName($row['chatcall_user_id'], '').'<br>'.get_lang('DoYouAccept')
 							."<p>"
 				."<a href=\"".$webpath."chat/chat.php?cidReq=".$_cid."&origin=whoisonlinejoin\">"
 				. get_lang("Yes")