chamilo / chamilo-lms

main/inc/lib/internationalization_database/transliteration/x02.php

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 // $Id: x02.php,v 1.1.8.1 2008/06/12 20:34:31 smk Exp $
 
 return array(
-  'en' => array('A', 'a', 'A', 'a', 'E', 'e', 'E', 'e', 'I', 'i', 'I', 'i', 'O', 'o', 'O', 'o',
+    'en' => array('A', 'a', 'A', 'a', 'E', 'e', 'E', 'e', 'I', 'i', 'I', 'i', 'O', 'o', 'O', 'o',
     'R', 'r', 'R', 'r', 'U', 'u', 'U', 'u', 'S', 's', 'T', 't', 'Y', 'y', 'H', 'h',
     'N', 'd', 'OU', 'ou', 'Z', 'z', 'A', 'a', 'E', 'e', 'O', 'o', 'O', 'o', 'O', 'o',
     'O', 'o', 'Y', 'y', 'l', 'n', 't', 'j', 'db', 'qp', 'A', 'C', 'c', 'L', 'T', 's',

main/inc/lib/internationalization_database/transliteration/x16.php

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 // $Id: x16.php,v 1.1.8.1 2008/06/12 20:34:31 smk Exp $
 
 return array(
-  'en' => array('kka', 'kk', 'nu', 'no', 'ne', 'nee', 'ni', 'na', 'mu', 'mo', 'me', 'mee', 'mi', 'ma', 'yu', 'yo',
+    'en' => array('kka', 'kk', 'nu', 'no', 'ne', 'nee', 'ni', 'na', 'mu', 'mo', 'me', 'mee', 'mi', 'ma', 'yu', 'yo',
     'ye', 'yee', 'yi', 'ya', 'ju', 'ju', 'jo', 'je', 'jee', 'ji', 'ji', 'ja', 'jju', 'jjo', 'jje', 'jjee',
     'jji', 'jja', 'lu', 'lo', 'le', 'lee', 'li', 'la', 'dlu', 'dlo', 'dle', 'dlee', 'dli', 'dla', 'lhu', 'lho',
     'lhe', 'lhee', 'lhi', 'lha', 'tlhu', 'tlho', 'tlhe', 'tlhee', 'tlhi', 'tlha', 'tlu', 'tlo', 'tle', 'tlee', 'tli', 'tla',

main/inc/lib/internationalization_database/transliteration/x6c.php

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 // $Id: x6c.php,v 1.1.8.1 2008/06/12 20:34:31 smk Exp $
 
 return array(
-  'en' => array('Lu ', 'Mu ', 'Li ', 'Tong ', 'Rong ', 'Chang ', 'Pu ', 'Luo ', 'Zhan ', 'Sao ', 'Zhan ', 'Meng ', 'Luo ', 'Qu ', 'Die ', 'Shi ',
+    'en' => array('Lu ', 'Mu ', 'Li ', 'Tong ', 'Rong ', 'Chang ', 'Pu ', 'Luo ', 'Zhan ', 'Sao ', 'Zhan ', 'Meng ', 'Luo ', 'Qu ', 'Die ', 'Shi ',
     'Di ', 'Min ', 'Jue ', 'Mang ', 'Qi ', 'Pie ', 'Nai ', 'Qi ', 'Dao ', 'Xian ', 'Chuan ', 'Fen ', 'Ri ', 'Nei ', NULL, 'Fu ',
     'Shen ', 'Dong ', 'Qing ', 'Qi ', 'Yin ', 'Xi ', 'Hai ', 'Yang ', 'An ', 'Ya ', 'Ke ', 'Qing ', 'Ya ', 'Dong ', 'Dan ', 'Lu ',
     'Qing ', 'Yang ', 'Yun ', 'Yun ', 'Shui ', 'San ', 'Zheng ', 'Bing ', 'Yong ', 'Dang ', 'Shitamizu ', 'Le ', 'Ni ', 'Tun ', 'Fan ', 'Gui ',

main/inc/lib/internationalization_database/transliteration/xb8.php

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 // $Id: xb8.php,v 1.1.8.1 2008/06/12 20:34:32 smk Exp $
 
 return array(
-  'en' => array('reoss', 'reong', 'reoj', 'reoc', 'reok', 'reot', 'reop', 'reoh', 're', 'reg', 'regg', 'regs', 'ren', 'renj', 'renh', 'red',
+    'en' => array('reoss', 'reong', 'reoj', 'reoc', 'reok', 'reot', 'reop', 'reoh', 're', 'reg', 'regg', 'regs', 'ren', 'renj', 'renh', 'red',
     'rel', 'relg', 'relm', 'relb', 'rels', 'relt', 'relp', 'relh', 'rem', 'reb', 'rebs', 'res', 'ress', 'reng', 'rej', 'rec',
     'rek', 'ret', 'rep', 'reh', 'ryeo', 'ryeog', 'ryeogg', 'ryeogs', 'ryeon', 'ryeonj', 'ryeonh', 'ryeod', 'ryeol', 'ryeolg', 'ryeolm', 'ryeolb',
     'ryeols', 'ryeolt', 'ryeolp', 'ryeolh', 'ryeom', 'ryeob', 'ryeobs', 'ryeos', 'ryeoss', 'ryeong', 'ryeoj', 'ryeoc', 'ryeok', 'ryeot', 'ryeop', 'ryeoh',

main/inc/lib/internationalization_database/transliteration/xbd.php

Indentation +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 // $Id: xbd.php,v 1.1.8.1 2008/06/12 20:34:32 smk Exp $
 
 return array(
-  'en' => array('bols', 'bolt', 'bolp', 'bolh', 'bom', 'bob', 'bobs', 'bos', 'boss', 'bong', 'boj', 'boc', 'bok', 'bot', 'bop', 'boh',
+    'en' => array('bols', 'bolt', 'bolp', 'bolh', 'bom', 'bob', 'bobs', 'bos', 'boss', 'bong', 'boj', 'boc', 'bok', 'bot', 'bop', 'boh',
     'bwa', 'bwag', 'bwagg', 'bwags', 'bwan', 'bwanj', 'bwanh', 'bwad', 'bwal', 'bwalg', 'bwalm', 'bwalb', 'bwals', 'bwalt', 'bwalp', 'bwalh',
     'bwam', 'bwab', 'bwabs', 'bwas', 'bwass', 'bwang', 'bwaj', 'bwac', 'bwak', 'bwat', 'bwap', 'bwah', 'bwae', 'bwaeg', 'bwaegg', 'bwaegs',
     'bwaen', 'bwaenj', 'bwaenh', 'bwaed', 'bwael', 'bwaelg', 'bwaelm', 'bwaelb', 'bwaels', 'bwaelt', 'bwaelp', 'bwaelh', 'bwaem', 'bwaeb', 'bwaebs', 'bwaes',

main/inc/lib/login_redirection.class.php

Spacing +6 added lines, -6 removed lines

@@ -18,30 +18,30 @@ 
         if (api_is_student() && !api_get_setting('student_page_after_login') == '') {
             $redirect_url = html_entity_decode(api_get_setting('student_page_after_login'));
             if ($redirect_url[0] == "/") {
-                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1) . $redirect_url;
+                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1).$redirect_url;
             }
         }
         if (api_is_teacher() && !api_get_setting('teacher_page_after_login') == '') {
             $redirect_url = html_entity_decode(api_get_setting('teacher_page_after_login'));
             if ($redirect_url[0] == "/") {
-                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1) . $redirect_url;
+                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1).$redirect_url;
             }
         }
         if (api_is_drh() && !api_get_setting('drh_page_after_login') == '') {
             $redirect_url = html_entity_decode(api_get_setting('drh_page_after_login'));
             if ($redirect_url[0] == "/") {
-                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1) . $redirect_url;
+                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1).$redirect_url;
             }
         }
         if (api_is_session_admin() && !api_get_setting('sessionadmin_page_after_login') == '') {
             $redirect_url = html_entity_decode(api_get_setting('sessionadmin_page_after_login'));
             if ($redirect_url[0] == "/") {
-                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1) . $redirect_url;
+                $redirect_url = substr(api_get_path(WEB_PATH), 0, -1).$redirect_url;
             }
         }
 
         if (!empty($redirect_url)) {
-            header('Location: ' . $redirect_url . $param);
+            header('Location: '.$redirect_url.$param);
             exit();
         }
 
@@ -49,7 +49,7 @@ 
         if (CustomPages::enabled()) {
             CustomPages::display(CustomPages::INDEX_LOGGED);
         }
-        header('location: ' . api_get_path(WEB_PATH) . api_get_setting('page_after_login') . $param);
+        header('location: '.api_get_path(WEB_PATH).api_get_setting('page_after_login').$param);
         exit();
     }
 }

main/inc/lib/sessionmanager.lib.php

Indentation +1 added lines, -1 removed lines

@@ -6022,7 +6022,7 @@
         ]);
     }
 
-	/**
+    /**
      * Get the count of user courses in session
      * @param int $sessionId The session id
      * @return array

Braces +14 added lines, -10 removed lines

@@ -1588,8 +1588,10 @@ 
         if (empty($session_visibility)) {
             $session_visibility = $session->getVisibility();
             //default status loaded if empty
-            if (empty($session_visibility))
-                $session_visibility = SESSION_VISIBLE_READ_ONLY; // by default readonly 1
+            if (empty($session_visibility)) {
+                            $session_visibility = SESSION_VISIBLE_READ_ONLY;
+            }
+            // by default readonly 1
         } else {
             if (!in_array($session_visibility, array(SESSION_VISIBLE_READ_ONLY, SESSION_VISIBLE, SESSION_INVISIBLE))) {
                 $session_visibility = SESSION_VISIBLE_READ_ONLY;
@@ -2792,10 +2794,11 @@ 
                                 c_id = $courseId AND
                                 user_id = $user_id ";
                     $result = Database::query($sql);
-                    if (Database::affected_rows($result) > 0)
-                        return true;
-                    else
-                        return false;
+                    if (Database::affected_rows($result) > 0) {
+                                            return true;
+                    } else {
+                                            return false;
+                    }
                 } else {
                     // The user is not subscribed to the session, so make sure
                     // he isn't subscribed to a course in this session either
@@ -2806,10 +2809,11 @@ 
                                 c_id = $courseId AND
                                 user_id = $user_id ";
                     $result = Database::query($sql);
-                    if (Database::affected_rows($result) > 0)
-                        return true;
-                    else
-                        return false;
+                    if (Database::affected_rows($result) > 0) {
+                                            return true;
+                    } else {
+                                            return false;
+                    }
                 }
             } else {
                 // Assign user as a coach to course

Spacing +195 added lines, -195 removed lines

@@ -143,7 +143,7 @@ 
                     return $msg;
                 }
             } else {
-                $rs = Database::query("SELECT 1 FROM $tbl_session WHERE name='" . $name . "'");
+                $rs = Database::query("SELECT 1 FROM $tbl_session WHERE name='".$name."'");
                 if (Database::num_rows($rs)) {
                     $msg = get_lang('SessionNameAlreadyExists');
                     return $msg;
@@ -256,7 +256,7 @@ 
     public static function session_name_exists($name)
     {
         $name = Database::escape_string($name);
-        $sql = "SELECT COUNT(*) as count FROM " . Database::get_main_table(TABLE_MAIN_SESSION) . "
+        $sql = "SELECT COUNT(*) as count FROM ".Database::get_main_table(TABLE_MAIN_SESSION)."
                 WHERE name = '$name'";
         $result = Database::fetch_array(Database::query($sql));
 
@@ -287,7 +287,7 @@ 
             $where .= " AND (
                             s.session_admin_id = $user_id  OR
                             sru.user_id = '$user_id' AND
-                            sru.relation_type = '" . SESSION_RELATION_TYPE_RRHH . "'
+                            sru.relation_type = '".SESSION_RELATION_TYPE_RRHH."'
                             )
                       ";
 
@@ -305,7 +305,7 @@ 
             $where_condition = str_replace('category_name', 'sc.name', $where_condition);
             $where_condition = str_replace(
                 array("AND session_active = '1'  )", " AND (  session_active = '1'  )"),
-                array(') GROUP BY s.name HAVING session_active = 1 ', " GROUP BY s.name HAVING session_active = 1 " )
+                array(') GROUP BY s.name HAVING session_active = 1 ', " GROUP BY s.name HAVING session_active = 1 ")
                 , $where_condition
             );
             $where_condition = str_replace(
@@ -350,7 +350,7 @@ 
 
             $access_url_id = api_get_current_access_url_id();
             if ($access_url_id != -1) {
-                $where.= " AND ar.access_url_id = $access_url_id ";
+                $where .= " AND ar.access_url_id = $access_url_id ";
 
                 $sql = "SELECT count(id) as total_rows FROM (
                 SELECT DISTINCT
@@ -400,14 +400,14 @@ 
             if (api_is_session_admin() &&
                 api_get_setting('allow_session_admins_to_manage_all_sessions') == 'false'
             ) {
-                $where .=" AND s.session_admin_id = $user_id ";
+                $where .= " AND s.session_admin_id = $user_id ";
             }
         }
 
         if (!api_is_platform_admin() && api_is_teacher() &&
             api_get_setting('allow_teachers_to_create_sessions') == 'true'
         ) {
-            $where .=" AND s.id_coach = $user_id ";
+            $where .= " AND s.id_coach = $user_id ";
         }
 
         $extra_field = new ExtraField('session');
@@ -457,10 +457,10 @@ 
         $query = "$select FROM $tbl_session s $inject_joins $where $inject_where";
 
         if (api_is_multiple_url_enabled()) {
-            $table_access_url_rel_session= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
+            $table_access_url_rel_session = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
             $access_url_id = api_get_current_access_url_id();
             if ($access_url_id != -1) {
-                $where.= " AND ar.access_url_id = $access_url_id ";
+                $where .= " AND ar.access_url_id = $access_url_id ";
                 $query = "$select
                         FROM $tbl_session s $inject_joins
                         INNER JOIN $table_access_url_rel_session ar
@@ -564,7 +564,7 @@ 
                 INNER JOIN $tbl_lp l ON l.id = v.lp_id
                 INNER JOIN $tbl_user u ON u.user_id = v.user_id
                 INNER JOIN $tbl_course c
-                WHERE v.session_id = " . $sessionId;
+                WHERE v.session_id = ".$sessionId;
         $result_rows = Database::query($sql);
         $row = Database::fetch_array($result_rows);
         $num = $row['total_rows'];
@@ -616,7 +616,7 @@ 
 
         $limit = null;
         if (!empty($options['limit'])) {
-            $limit = " LIMIT " . $options['limit'];
+            $limit = " LIMIT ".$options['limit'];
         }
 
         if (!empty($options['where'])) {
@@ -625,7 +625,7 @@ 
 
         $order = null;
         if (!empty($options['order'])) {
-            $order = " ORDER BY " . $options['order'];
+            $order = " ORDER BY ".$options['order'];
         }
 
         $sql = "SELECT u.user_id, u.lastname, u.firstname, u.username, u.email, s.c_id
@@ -684,13 +684,13 @@ 
             foreach ($lessons as $lesson) {
                 $data[$lesson['id']] = (!empty($user_lessons[$lesson['id']]['progress'])) ? $user_lessons[$lesson['id']]['progress'] : 0;
                 $progress += $data[$lesson['id']];
-                $data[$lesson['id']] = $data[$lesson['id']] . '%';
+                $data[$lesson['id']] = $data[$lesson['id']].'%';
                 $count++;
             }
             if ($count == 0) {
                 $data['total'] = 0;
             } else {
-                $data['total'] = round($progress / $count, 2) . '%';
+                $data['total'] = round($progress / $count, 2).'%';
             }
             $table[] = $data;
         }
@@ -731,7 +731,7 @@ 
 
         $limit = null;
         if (!empty($options['limit'])) {
-            $limit = " LIMIT " . $options['limit'];
+            $limit = " LIMIT ".$options['limit'];
         }
 
         if (!empty($options['where'])) {
@@ -740,7 +740,7 @@ 
 
         $order = null;
         if (!empty($options['order'])) {
-            $order = " ORDER BY " . $options['order'];
+            $order = " ORDER BY ".$options['order'];
         }
 
         $sql = "SELECT u.user_id, u.lastname, u.firstname, u.username, u.email, s.c_id
@@ -842,7 +842,7 @@ 
 
         $limit = null;
         if (!empty($options['limit'])) {
-            $limit = " LIMIT " . $options['limit'];
+            $limit = " LIMIT ".$options['limit'];
         }
 
         if (!empty($options['where'])) {
@@ -851,7 +851,7 @@ 
 
         $order = null;
         if (!empty($options['order'])) {
-            $order = " ORDER BY " . $options['order'];
+            $order = " ORDER BY ".$options['order'];
         }
 
         //TODO, fix create report without session
@@ -884,7 +884,7 @@ 
         /**
          *  Lessons
          */
-        $sql = "SELECT * FROM $tbl_course_lp WHERE c_id = %s ";  //AND session_id = %s
+        $sql = "SELECT * FROM $tbl_course_lp WHERE c_id = %s "; //AND session_id = %s
         $sql_query = sprintf($sql, $course['real_id']);
         $result = Database::query($sql_query);
         $arrLesson = array(array());
@@ -892,7 +892,7 @@ 
             if (empty($arrLesson[$row['session_id']]['lessons_total'])) {
                 $arrLesson[$row['session_id']]['lessons_total'] = 1;
             } else {
-                $arrLesson[$row['session_id']]['lessons_total'] ++;
+                $arrLesson[$row['session_id']]['lessons_total']++;
             }
         }
 
@@ -959,7 +959,7 @@ 
                 $course['real_id']
             );
             foreach ($user_list as $user_id) {
-                isset($survey_user_list[$user_id]) ? $survey_user_list[$user_id] ++ : $survey_user_list[$user_id] = 1;
+                isset($survey_user_list[$user_id]) ? $survey_user_list[$user_id]++ : $survey_user_list[$user_id] = 1;
             }
         }
 
@@ -1013,7 +1013,7 @@ 
             $assignments_done = Tracking::count_student_assignments($user['user_id'], $course['code'], $user['id_session']);
             $assignments_left = $assignments_total - $assignments_done;
             if (!empty($assignments_total)) {
-                $assignments_progress = round((( $assignments_done * 100 ) / $assignments_total), 2);
+                $assignments_progress = round((($assignments_done * 100) / $assignments_total), 2);
             } else {
                 $assignments_progress = 0;
             }
@@ -1043,7 +1043,7 @@ 
             $wiki_read = $row['count'];
             $wiki_unread = $wiki_total - $wiki_read;
             if (!empty($wiki_total)) {
-                $wiki_progress = round((( $wiki_read * 100 ) / $wiki_total), 2);
+                $wiki_progress = round((($wiki_read * 100) / $wiki_total), 2);
             } else {
                 $wiki_progress = 0;
             }
@@ -1052,7 +1052,7 @@ 
             $surveys_done = (isset($survey_user_list[$user['user_id']]) ? $survey_user_list[$user['user_id']] : 0);
             $surveys_left = $surveys_total - $surveys_done;
             if (!empty($surveys_total)) {
-                $surveys_progress = round((( $surveys_done * 100 ) / $surveys_total), 2);
+                $surveys_progress = round((($surveys_done * 100) / $surveys_total), 2);
             } else {
                 $surveys_progress = 0;
             }
@@ -1065,7 +1065,7 @@ 
             );
             $forums_left = $forums_total - $forums_done;
             if (!empty($forums_total)) {
-                $forums_progress = round((( $forums_done * 100 ) / $forums_total), 2);
+                $forums_progress = round((($forums_done * 100) / $forums_total), 2);
             } else {
                 $forums_progress = 0;
             }
@@ -1073,58 +1073,58 @@ 
             //Overall Total
             $overall_total = ($course_description_progress + $exercises_progress + $forums_progress + $assignments_progress + $wiki_progress + $surveys_progress) / 6;
 
-            $link = '<a href="' . api_get_path(WEB_CODE_PATH) . 'mySpace/myStudents.php?student=' . $user[0] . '&details=true&course=' . $course['code'] . '&id_session=' . $user['id_session'] . '"> %s </a>';
-            $linkForum = '<a href="' . api_get_path(WEB_CODE_PATH) . 'forum/index.php?cidReq=' . $course['code'] . '&id_session=' . $user['id_session'] . '"> %s </a>';
-            $linkWork = '<a href="' . api_get_path(WEB_CODE_PATH) . 'work/work.php?cidReq=' . $course['code'] . '&id_session=' . $user['id_session'] . '"> %s </a>';
-            $linkWiki = '<a href="' . api_get_path(WEB_CODE_PATH) . 'wiki/index.php?cidReq=' . $course['code'] . '&session_id=' . $user['id_session'] . '&action=statistics"> %s </a>';
-            $linkSurvey = '<a href="' . api_get_path(WEB_CODE_PATH) . 'survey/survey_list.php?cidReq=' . $course['code'] . '&id_session=' . $user['id_session'] . '"> %s </a>';
+            $link = '<a href="'.api_get_path(WEB_CODE_PATH).'mySpace/myStudents.php?student='.$user[0].'&details=true&course='.$course['code'].'&id_session='.$user['id_session'].'"> %s </a>';
+            $linkForum = '<a href="'.api_get_path(WEB_CODE_PATH).'forum/index.php?cidReq='.$course['code'].'&id_session='.$user['id_session'].'"> %s </a>';
+            $linkWork = '<a href="'.api_get_path(WEB_CODE_PATH).'work/work.php?cidReq='.$course['code'].'&id_session='.$user['id_session'].'"> %s </a>';
+            $linkWiki = '<a href="'.api_get_path(WEB_CODE_PATH).'wiki/index.php?cidReq='.$course['code'].'&session_id='.$user['id_session'].'&action=statistics"> %s </a>';
+            $linkSurvey = '<a href="'.api_get_path(WEB_CODE_PATH).'survey/survey_list.php?cidReq='.$course['code'].'&id_session='.$user['id_session'].'"> %s </a>';
 
             $table[] = array(
                 'lastname' => $user[1],
                 'firstname' => $user[2],
                 'username' => $user[3],
                 #'profile'   => '',
-                'total' => round($overall_total, 2) . '%',
-                'courses' => sprintf($link, $course_description_progress . '%'),
-                'lessons' => sprintf($link, $lessons_progress . '%'),
-                'exercises' => sprintf($link, $exercises_progress . '%'),
-                'forums' => sprintf($link, $forums_progress . '%'),
-                'homeworks' => sprintf($link, $assignments_progress . '%'),
-                'wikis' => sprintf($link, $wiki_progress . '%'),
-                'surveys' => sprintf($link, $surveys_progress . '%'),
+                'total' => round($overall_total, 2).'%',
+                'courses' => sprintf($link, $course_description_progress.'%'),
+                'lessons' => sprintf($link, $lessons_progress.'%'),
+                'exercises' => sprintf($link, $exercises_progress.'%'),
+                'forums' => sprintf($link, $forums_progress.'%'),
+                'homeworks' => sprintf($link, $assignments_progress.'%'),
+                'wikis' => sprintf($link, $wiki_progress.'%'),
+                'surveys' => sprintf($link, $surveys_progress.'%'),
                 //course description
-                'course_description_progress' => $course_description_progress . '%',
+                'course_description_progress' => $course_description_progress.'%',
                 //lessons
                 'lessons_total' => sprintf($link, $lessons_total),
                 'lessons_done' => sprintf($link, $lessons_done),
                 'lessons_left' => sprintf($link, $lessons_left),
-                'lessons_progress' => sprintf($link, $lessons_progress . '%'),
+                'lessons_progress' => sprintf($link, $lessons_progress.'%'),
                 //exercises
                 'exercises_total' => sprintf($link, $exercises_total),
                 'exercises_done' => sprintf($link, $exercises_done),
                 'exercises_left' => sprintf($link, $exercises_left),
-                'exercises_progress' => sprintf($link, $exercises_progress . '%'),
+                'exercises_progress' => sprintf($link, $exercises_progress.'%'),
                 //forums
                 'forums_total' => sprintf($linkForum, $forums_total),
                 'forums_done' => sprintf($linkForum, $forums_done),
                 'forums_left' => sprintf($linkForum, $forums_left),
-                'forums_progress' => sprintf($linkForum, $forums_progress . '%'),
+                'forums_progress' => sprintf($linkForum, $forums_progress.'%'),
                 //assignments
                 'assignments_total' => sprintf($linkWork, $assignments_total),
                 'assignments_done' => sprintf($linkWork, $assignments_done),
                 'assignments_left' => sprintf($linkWork, $assignments_left),
-                'assignments_progress' => sprintf($linkWork, $assignments_progress . '%'),
+                'assignments_progress' => sprintf($linkWork, $assignments_progress.'%'),
                 //wiki
                 'wiki_total' => sprintf($linkWiki, $wiki_total),
                 'wiki_revisions' => sprintf($linkWiki, $wiki_revisions),
                 'wiki_read' => sprintf($linkWiki, $wiki_read),
                 'wiki_unread' => sprintf($linkWiki, $wiki_unread),
-                'wiki_progress' => sprintf($linkWiki, $wiki_progress . '%'),
+                'wiki_progress' => sprintf($linkWiki, $wiki_progress.'%'),
                 //survey
                 'surveys_total' => sprintf($linkSurvey, $surveys_total),
                 'surveys_done' => sprintf($linkSurvey, $surveys_done),
                 'surveys_left' => sprintf($linkSurvey, $surveys_left),
-                'surveys_progress' => sprintf($linkSurvey, $surveys_progress . '%'),
+                'surveys_progress' => sprintf($linkSurvey, $surveys_progress.'%'),
             );
         }
 
@@ -1204,7 +1204,7 @@ 
 
         $limit = null;
         if (!empty($options['limit'])) {
-            $limit = " LIMIT " . $options['limit'];
+            $limit = " LIMIT ".$options['limit'];
         }
 
         if (!empty($options['where'])) {
@@ -1213,7 +1213,7 @@ 
 
         $order = null;
         if (!empty($options['order'])) {
-            $order = " ORDER BY " . $options['order'];
+            $order = " ORDER BY ".$options['order'];
         }
 
         //TODO add course name
@@ -1226,7 +1226,7 @@ 
                     " : "
                     u.lastname,
                     u.firstname,
-                ") . "
+                ")."
                 a.logout_course_date,
                 a.counter,
                 c.title,
@@ -1308,7 +1308,7 @@ 
             if (Database::num_rows($result) > 0) {
                 $row = Database::fetch_array($result);
                 $count = $row['count'] + 1;
-                $session_name = $session_name . '_' . $count;
+                $session_name = $session_name.'_'.$count;
                 $result = self::session_name_exists($session_name);
                 if (!$result) {
                     return $session_name;
@@ -1388,7 +1388,7 @@ 
 
             return false;
         } else {
-            $sql = "SELECT id FROM $tbl_session WHERE name='" . Database::escape_string($name) . "'";
+            $sql = "SELECT id FROM $tbl_session WHERE name='".Database::escape_string($name)."'";
             $rs = Database::query($sql);
             $exists = false;
             while ($row = Database::fetch_array($rs)) {
@@ -1754,7 +1754,7 @@ 
         // Delete users from the session
         if ($empty_users === true) {
             $sql = "DELETE FROM $tbl_session_rel_user
-                    WHERE session_id = $id_session AND relation_type<>" . SESSION_RELATION_TYPE_RRHH . "";
+                    WHERE session_id = $id_session AND relation_type<>".SESSION_RELATION_TYPE_RRHH."";
             Database::query($sql);
         }
 
@@ -1765,7 +1765,7 @@ 
             $enreg_user = Database::escape_string($enreg_user);
             $nbr_users++;
             $sql = "INSERT IGNORE INTO $tbl_session_rel_user (relation_type, session_id, user_id, registered_at)
-                    VALUES (0, $id_session, $enreg_user, '" . api_get_utc_datetime() . "')";
+                    VALUES (0, $id_session, $enreg_user, '".api_get_utc_datetime()."')";
             Database::query($sql);
         }
 
@@ -1855,9 +1855,9 @@ 
         is_array($courseInfo) ? $courseId = $courseInfo['real_id'] : $courseId = $courseInfo;
 
         $statusCondition = null;
-        if (isset($status) && !is_null($status))  {
+        if (isset($status) && !is_null($status)) {
             $status = intval($status);
-            $statusCondition  = " AND status = $status";
+            $statusCondition = " AND status = $status";
         }
 
         foreach ($userList as $userId) {
@@ -1989,7 +1989,7 @@ 
             if (empty($count)) {
                 // If user is not registered to a session then add it.
                 $sql = "INSERT IGNORE INTO $tbl_session_rel_user (session_id, user_id, registered_at)
-                        VALUES ($session_id, $enreg_user, '" . api_get_utc_datetime() . "')";
+                        VALUES ($session_id, $enreg_user, '".api_get_utc_datetime()."')";
                 Database::query($sql);
 
                 $sql = "UPDATE $tbl_session SET nbr_users = nbr_users + 1
@@ -2032,7 +2032,7 @@ 
 		               WHERE
                             session_id = $session_id AND
 		                    user_id = $user_id AND
-		                    relation_type <> " . SESSION_RELATION_TYPE_RRHH . "";
+		                    relation_type <> ".SESSION_RELATION_TYPE_RRHH."";
         $result = Database::query($delete_sql);
         $return = Database::affected_rows($result);
 
@@ -2116,7 +2116,7 @@ 
                 FROM $tbl_session_rel_user
                 WHERE
                     session_id = $sessionId AND
-                    relation_type<>" . SESSION_RELATION_TYPE_RRHH;
+                    relation_type<>".SESSION_RELATION_TYPE_RRHH;
         $result = Database::query($sql);
         $user_list = Database::store_result($result);
 
@@ -2128,7 +2128,7 @@ 
 
                     $sql = "DELETE FROM $tbl_session_rel_course
                             WHERE
-                                c_id = " . $existingCourse['c_id'] . " AND
+                                c_id = ".$existingCourse['c_id']." AND
                                 session_id = $sessionId";
                     Database::query($sql);
 
@@ -2191,7 +2191,7 @@ 
                             $links = $cat->get_links(null, false, $courseInfo['code'], 0);
 
                             $cat->set_session_id($sessionId);
-                            $oldCategoryId= $cat->get_id();
+                            $oldCategoryId = $cat->get_id();
                             $newId = $cat->add();
                             $newCategoryIdList[$oldCategoryId] = $newId;
 
@@ -2379,8 +2379,8 @@ 
         $return_value = false;
         $sql = "SELECT c_id FROM $tbl_session_course
                 WHERE
-                  session_id = " . intval($session_id) . " AND
-                  c_id = " . intval($courseId) . "";
+                  session_id = ".intval($session_id)." AND
+                  c_id = " . intval($courseId)."";
         $result = Database::query($sql);
         $num = Database::num_rows($result);
         if ($num > 0) {
@@ -2403,8 +2403,8 @@ 
         }
 
         $sql = 'SELECT *
-		        FROM ' . $tbl_session . '
-		        WHERE name = "' . Database::escape_string($session_name) . '"';
+		        FROM ' . $tbl_session.'
+		        WHERE name = "' . Database::escape_string($session_name).'"';
         $result = Database::query($sql);
         $num = Database::num_rows($result);
         if ($num > 0) {
@@ -2444,8 +2444,8 @@ 
         $month_end = intval($smonth_end);
         $day_end = intval($sday_end);
 
-        $date_start = "$year_start-" . (($month_start < 10) ? "0$month_start" : $month_start) . "-" . (($day_start < 10) ? "0$day_start" : $day_start);
-        $date_end = "$year_end-" . (($month_end < 10) ? "0$month_end" : $month_end) . "-" . (($day_end < 10) ? "0$day_end" : $day_end);
+        $date_start = "$year_start-".(($month_start < 10) ? "0$month_start" : $month_start)."-".(($day_start < 10) ? "0$day_start" : $day_start);
+        $date_end = "$year_end-".(($month_end < 10) ? "0$month_end" : $month_end)."-".(($day_end < 10) ? "0$day_end" : $day_end);
 
         if (empty($name)) {
             $msg = get_lang('SessionCategoryNameIsRequired');
@@ -2517,8 +2517,8 @@ 
         $month_end = intval($smonth_end);
         $day_end = intval($sday_end);
         $id = intval($id);
-        $date_start = "$year_start-" . (($month_start < 10) ? "0$month_start" : $month_start) . "-" . (($day_start < 10) ? "0$day_start" : $day_start);
-        $date_end = "$year_end-" . (($month_end < 10) ? "0$month_end" : $month_end) . "-" . (($day_end < 10) ? "0$day_end" : $day_end);
+        $date_start = "$year_start-".(($month_start < 10) ? "0$month_start" : $month_start)."-".(($day_start < 10) ? "0$day_start" : $day_start);
+        $date_end = "$year_end-".(($month_end < 10) ? "0$month_end" : $month_end)."-".(($day_end < 10) ? "0$day_end" : $day_end);
 
         if (empty($name)) {
             $msg = get_lang('SessionCategoryNameIsRequired');
@@ -2538,13 +2538,13 @@ 
         if ($date_end <> null) {
             $sql = "UPDATE $tbl_session_category
                     SET
-                        name = '" . Database::escape_string($name) . "',
+                        name = '".Database::escape_string($name)."',
                         date_start = '$date_start' ,
                         date_end = '$date_end'
                     WHERE id= $id";
         } else {
             $sql = "UPDATE $tbl_session_category SET
-                        name = '" . Database::escape_string($name) . "',
+                        name = '".Database::escape_string($name)."',
                         date_start = '$date_start',
                         date_end = NULL
                     WHERE id= $id";
@@ -2574,10 +2574,10 @@ 
 
         //Setting session_category_id to 0
         $sql = "UPDATE $tbl_session SET session_category_id = 0
-                WHERE session_category_id IN (" . $id_checked . ")";
+                WHERE session_category_id IN (".$id_checked.")";
         Database::query($sql);
 
-        $sql = "SELECT id FROM $tbl_session WHERE session_category_id IN (" . $id_checked . ")";
+        $sql = "SELECT id FROM $tbl_session WHERE session_category_id IN (".$id_checked.")";
         $result = Database::query($sql);
         while ($rows = Database::fetch_array($result)) {
             $session_id = $rows['id'];
@@ -2589,7 +2589,7 @@ 
                 }
             }
         }
-        $sql = "DELETE FROM $tbl_session_category WHERE id IN (" . $id_checked . ")";
+        $sql = "DELETE FROM $tbl_session_category WHERE id IN (".$id_checked.")";
         Database::query($sql);
 
         // Add event to system log
@@ -2662,7 +2662,7 @@ 
                 $value = Database::escape_string($options['value']);
                 $sql_query .= ' AND ';
                 if (in_array($field, $availableFields) && in_array($operator, $availableOperator)) {
-                    $sql_query .= $field . " $operator '" . $value . "'";
+                    $sql_query .= $field." $operator '".$value."'";
                 }
             }
         }
@@ -2729,8 +2729,8 @@ 
     {
         $tbl_session_category = Database::get_main_table(TABLE_MAIN_SESSION_CATEGORY);
         $id = api_get_current_access_url_id();
-        $sql = 'SELECT * FROM ' . $tbl_session_category . '
-                WHERE access_url_id = ' . $id . '
+        $sql = 'SELECT * FROM '.$tbl_session_category.'
+                WHERE access_url_id = ' . $id.'
                 ORDER BY name ASC';
         $result = Database::query($sql);
         if (Database::num_rows($result) > 0) {
@@ -2909,11 +2909,11 @@ 
                         INNER JOIN $tbl_session_rel_access_url a ON (a.session_id = s.session_id)
                         WHERE
                             s.user_id = $userId AND
-                            relation_type=" . SESSION_RELATION_TYPE_RRHH . " AND
-                            access_url_id = " . api_get_current_access_url_id() . "";
+                            relation_type=".SESSION_RELATION_TYPE_RRHH." AND
+                            access_url_id = " . api_get_current_access_url_id()."";
             } else {
                 $sql = "SELECT session_id FROM $tbl_session_rel_user s
-                        WHERE user_id = $userId AND relation_type=" . SESSION_RELATION_TYPE_RRHH . "";
+                        WHERE user_id = $userId AND relation_type=".SESSION_RELATION_TYPE_RRHH."";
             }
             $result = Database::query($sql);
 
@@ -2923,7 +2923,7 @@ 
                             WHERE
                                 session_id = {$row['session_id']} AND
                                 user_id = $userId AND
-                                relation_type=" . SESSION_RELATION_TYPE_RRHH . " ";
+                                relation_type=".SESSION_RELATION_TYPE_RRHH." ";
                     Database::query($sql);
                 }
             }
@@ -2937,8 +2937,8 @@ 
                         VALUES (
                             $session_id,
                             $userId,
-                            '" . SESSION_RELATION_TYPE_RRHH . "',
-                            '" . api_get_utc_datetime() . "'
+                            '".SESSION_RELATION_TYPE_RRHH."',
+                            '" . api_get_utc_datetime()."'
                         )";
 
                 Database::query($sql);
@@ -2980,8 +2980,8 @@ 
                     WHERE
                         sru.user_id = '$userId' AND
                         sru.session_id = '$sessionId' AND
-                        sru.relation_type = '" . SESSION_RELATION_TYPE_RRHH . "' AND
-                        access_url_id = " . api_get_current_access_url_id() . "
+                        sru.relation_type = '".SESSION_RELATION_TYPE_RRHH."' AND
+                        access_url_id = " . api_get_current_access_url_id()."
                         ";
         } else {
             $sql = "$select FROM $tbl_session s
@@ -2990,7 +2990,7 @@ 
                         sru.session_id = s.id AND
                         sru.user_id = '$userId' AND
                         sru.session_id = '$sessionId' AND
-                        sru.relation_type = '" . SESSION_RELATION_TYPE_RRHH . "'
+                        sru.relation_type = '".SESSION_RELATION_TYPE_RRHH."'
                     ";
         }
 
@@ -3088,7 +3088,7 @@ 
 
         $limitCondition = null;
         if (!empty($start) && !empty($limit)) {
-            $limitCondition = " LIMIT " . intval($start) . ", " . intval($limit);
+            $limitCondition = " LIMIT ".intval($start).", ".intval($limit);
         }
 
         if (empty($orderCondition)) {
@@ -3166,26 +3166,26 @@ 
 
         $sessions = array();
         if (Database::num_rows($result) > 0) {
-            $sysUploadPath = api_get_path(SYS_UPLOAD_PATH). 'sessions/';
-            $webUploadPath = api_get_path(WEB_UPLOAD_PATH). 'sessions/';
+            $sysUploadPath = api_get_path(SYS_UPLOAD_PATH).'sessions/';
+            $webUploadPath = api_get_path(WEB_UPLOAD_PATH).'sessions/';
             $imgPath = Display::returnIconPath('session_default_small.png');
 
             $tableExtraFields = Database::get_main_table(TABLE_EXTRA_FIELD);
-            $sql = "SELECT id FROM " . $tableExtraFields . "
+            $sql = "SELECT id FROM ".$tableExtraFields."
                     WHERE extra_field_type = 3 AND variable='image'";
             $resultField = Database::query($sql);
             $imageFieldId = Database::fetch_assoc($resultField);
 
             while ($row = Database::fetch_array($result)) {
 
-                $row['image'] =  null;
-                $sessionImage = $sysUploadPath . $imageFieldId['id'] . '_' . $row['id'] . '.png';
+                $row['image'] = null;
+                $sessionImage = $sysUploadPath.$imageFieldId['id'].'_'.$row['id'].'.png';
 
                 if (is_file($sessionImage)) {
-                    $sessionImage = $webUploadPath . $imageFieldId['id'] . '_' . $row['id'] . '.png';
+                    $sessionImage = $webUploadPath.$imageFieldId['id'].'_'.$row['id'].'.png';
                     $row['image'] = $sessionImage;
                 } else {
-                    $row['image'] =  $imgPath;
+                    $row['image'] = $imgPath;
                 }
 
                 $sessions[$row['id']] = $row;
@@ -3255,7 +3255,7 @@ 
                 return intval($count[0]);
             }
 
-            while ($row = Database::fetch_array($result,'ASSOC'))	{
+            while ($row = Database::fetch_array($result, 'ASSOC')) {
                 $courses[$row['real_id']] = $row;
             }
         }
@@ -3299,7 +3299,7 @@ 
         }
 
         $tbl_course = Database::get_main_table(TABLE_MAIN_COURSE);
-        $tbl_session_rel_course	= Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
+        $tbl_session_rel_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
 
         if ($getCount) {
             $select = "SELECT COUNT(DISTINCT(c.code)) as count ";
@@ -3324,7 +3324,7 @@ 
 		        ";
         if ($getCount) {
             $result = Database::query($sql);
-            $row = Database::fetch_array($result,'ASSOC');
+            $row = Database::fetch_array($result, 'ASSOC');
             return $row['count'];
         }
 
@@ -3339,7 +3339,7 @@ 
         $courses = array();
 
         if ($num_rows > 0) {
-            while ($row = Database::fetch_array($result,'ASSOC'))	{
+            while ($row = Database::fetch_array($result, 'ASSOC')) {
                 $courses[$row['id']] = $row;
             }
         }
@@ -3417,7 +3417,7 @@ 
         $result = Database::query($sql);
         $num_rows = Database::num_rows($result);
         if ($num_rows > 0) {
-            $row = Database::fetch_array($result,'ASSOC');
+            $row = Database::fetch_array($result, 'ASSOC');
             return $row['count'];
         }
 
@@ -3529,9 +3529,9 @@ 
 
             if ($access_url_id != -1) {
                 $sql = 'SELECT DISTINCT session.*
-                    FROM ' . $session_table . ' session INNER JOIN ' . $tbl_session_rel_access_url . ' session_rel_url
+                    FROM ' . $session_table.' session INNER JOIN '.$tbl_session_rel_access_url.' session_rel_url
                     ON (session.id = session_rel_url.session_id)
-                    WHERE '.$sqlCoach.' access_url_id = ' . $access_url_id;
+                    WHERE '.$sqlCoach.' access_url_id = '.$access_url_id;
             }
         }
         $sql .= ' ORDER by name';
@@ -3565,8 +3565,8 @@ 
                 FROM $tbl_session_rel_course_rel_user session_rcru, $tbl_user user
                 WHERE
                     session_rcru.user_id = user.user_id AND
-                    session_rcru.session_id = '" . intval($session_id) . "' AND
-                    session_rcru.c_id ='" . intval($courseId) . "' AND
+                    session_rcru.session_id = '".intval($session_id)."' AND
+                    session_rcru.c_id ='" . intval($courseId)."' AND
                     user.user_id = " . intval($user_id);
 
         $result = Database::query($sql);
@@ -3597,8 +3597,8 @@ 
         $sql = "SELECT session_rcru.status
                 FROM $tbl_session_rel_course_rel_user session_rcru, $tbl_user user
                 WHERE session_rcru.user_id = user.user_id AND
-                    session_rcru.session_id = '" . intval($session_id) . "' AND
-                    session_rcru.c_id ='" . intval($courseId) . "' AND
+                    session_rcru.session_id = '".intval($session_id)."' AND
+                    session_rcru.c_id ='" . intval($courseId)."' AND
                     user.user_id = " . intval($user_id);
         $result = Database::query($sql);
         $status = false;
@@ -3675,7 +3675,7 @@ 
         // Get timestamp for now in UTC - see http://php.net/manual/es/function.time.php#117251
         $now = time() - date('Z');
         // Timestamp in one month
-        $inOneMonth = $now + (30*24*3600);
+        $inOneMonth = $now + (30 * 24 * 3600);
         $inOneMonth = api_get_local_time($inOneMonth);
         if (api_strtotime($s['access_start_date']) < $now) {
             $s['access_start_date'] = api_get_local_time($now);
@@ -3697,16 +3697,16 @@ 
         }
         // Now try to create the session
         $sid = self::create_session(
-            $s['name'] . ' ' . get_lang('CopyLabelSuffix'),
+            $s['name'].' '.get_lang('CopyLabelSuffix'),
             $s['access_start_date'],
             $s['access_end_date'],
             $s['display_start_date'],
             $s['display_end_date'],
             $s['coach_access_start_date'],
             $s['coach_access_end_date'],
-            (int)$s['id_coach'],
+            (int) $s['id_coach'],
             $s['session_category_id'],
-            (int)$s['visibility'],
+            (int) $s['visibility'],
             true
         );
 
@@ -3820,7 +3820,7 @@ 
         $session_table = Database::get_main_table(TABLE_MAIN_SESSION);
         $sql = "SELECT DISTINCT id
 	         	FROM $session_table
-	         	WHERE session.id_coach =  '" . $user_id . "' AND id = '$session_id'";
+	         	WHERE session.id_coach =  '".$user_id."' AND id = '$session_id'";
         $result = Database::query($sql);
         if ($result && Database::num_rows($result)) {
             return true;
@@ -3839,7 +3839,7 @@ 
         $access_url_rel_session_table = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
         $sql = "SELECT count(id) FROM $session_table s";
         if (!empty($access_url_id) && $access_url_id == intval($access_url_id)) {
-            $sql .= ", $access_url_rel_session_table u " .
+            $sql .= ", $access_url_rel_session_table u ".
                 " WHERE s.id = u.session_id AND u.access_url_id = $access_url_id";
         }
         $res = Database::query($sql);
@@ -4090,7 +4090,7 @@ 
 
         $tbl_session = Database::get_main_table(TABLE_MAIN_SESSION);
         $tbl_session_user = Database::get_main_table(TABLE_MAIN_SESSION_USER);
-        $tbl_session_course  = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
+        $tbl_session_course = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
         $tbl_session_course_user = Database::get_main_table(TABLE_MAIN_SESSION_COURSE_USER);
 
         $sessions = array();
@@ -4184,10 +4184,10 @@ 
                     $suffix = null;
                     while (!$unique_name) {
                         if ($i > 1) {
-                            $suffix = ' - ' . $i;
+                            $suffix = ' - '.$i;
                         }
-                        $sql = 'SELECT 1 FROM ' . $tbl_session . '
-                                WHERE name="' . $session_name . $suffix . '"';
+                        $sql = 'SELECT 1 FROM '.$tbl_session.'
+                                WHERE name="' . $session_name.$suffix.'"';
                         $rs = Database::query($sql);
 
                         if (Database::result($rs, 0, 0)) {
@@ -4205,13 +4205,13 @@ 
 
                     // Creating the session.
                     $sql = "INSERT IGNORE INTO $tbl_session SET
-                            name = '" . $session_name . "',
+                            name = '".$session_name."',
                             id_coach = '$coach_id',
                             access_start_date = '$date_start',
                             access_end_date = '$date_end',
                             visibility = '$visibilityAfterExpirationPerSession',
                             $sessionCondition
-                            session_admin_id = " . intval($defaultUserId) . $extraParameters . $extraSessionParameters;
+                            session_admin_id = ".intval($defaultUserId).$extraParameters.$extraSessionParameters;
                     Database::query($sql);
 
                     $session_id = Database::insert_id();
@@ -4251,7 +4251,7 @@ 
                                 access_start_date = '$date_start',
                                 access_end_date = '$date_end',
                                 visibility = '$visibilityAfterExpirationPerSession',
-                                session_category_id = '$session_category_id' " . $extraParameters . $extraSessionParameters;
+                                session_category_id = '$session_category_id' ".$extraParameters.$extraSessionParameters;
 
                         Database::query($sql);
 
@@ -4271,7 +4271,7 @@ 
 
                             // Delete session-user relation only for students
                             $sql = "DELETE FROM $tbl_session_user
-                                    WHERE session_id = '$session_id' AND relation_type <> " . SESSION_RELATION_TYPE_RRHH;
+                                    WHERE session_id = '$session_id' AND relation_type <> ".SESSION_RELATION_TYPE_RRHH;
                             Database::query($sql);
 
                             $sql = "DELETE FROM $tbl_session_course WHERE session_id = '$session_id'";
@@ -4359,7 +4359,7 @@ 
 
                             // Delete session-user relation only for students
                             $sql = "DELETE FROM $tbl_session_user
-                                    WHERE session_id = '$session_id' AND relation_type <> " . SESSION_RELATION_TYPE_RRHH;
+                                    WHERE session_id = '$session_id' AND relation_type <> ".SESSION_RELATION_TYPE_RRHH;
                             Database::query($sql);
 
                             $sql = "DELETE FROM $tbl_session_course WHERE session_id = '$session_id'";
@@ -4402,7 +4402,7 @@ 
                             $sql = "INSERT IGNORE INTO $tbl_session_user SET
                                     user_id = '$user_id',
                                     session_id = '$session_id',
-                                    registered_at = '" . api_get_utc_datetime() . "'";
+                                    registered_at = '".api_get_utc_datetime()."'";
                             Database::query($sql);
                             if ($debug) {
                                 $logger->addInfo("Sessions - Adding User #$user_id ($user) to session #$session_id");
@@ -5040,7 +5040,7 @@ 
 
         if (!empty($lastConnectionDate)) {
             $lastConnectionDate = Database::escape_string($lastConnectionDate);
-            $userConditions .=  " AND u.last_login <= '$lastConnectionDate' ";
+            $userConditions .= " AND u.last_login <= '$lastConnectionDate' ";
         }
 
         if (!empty($keyword)) {
@@ -5099,7 +5099,7 @@ 
         $result = Database::query($sql);
         $result = Database::store_result($result);
 
-        return $result ;
+        return $result;
     }
 
     /**
@@ -5159,7 +5159,7 @@ 
                     $userToString = null;
                     foreach ($userList as $userInfo) {
                         $newUserList[] = $userInfo['user_id'];
-                        $userToString .= $userInfo['firstname'] . ' ' . $userInfo['lastname'] . '<br />';
+                        $userToString .= $userInfo['firstname'].' '.$userInfo['lastname'].'<br />';
                     }
 
                     if (!empty($sessionsDestination)) {
@@ -5172,7 +5172,7 @@ 
                                 $messages[] = Display::return_message(sprintf(get_lang('SessionXSkipped'), $sessionDestinationId), 'warning', false);
                                 continue;
                             }
-                            $messages[] = Display::return_message(get_lang('StudentList') . '<br />' . $userToString, 'info', false);
+                            $messages[] = Display::return_message(get_lang('StudentList').'<br />'.$userToString, 'info', false);
                             SessionManager::suscribe_users_to_session(
                                 $sessionDestinationId,
                                 $newUserList,
@@ -5223,7 +5223,7 @@ 
                 }
             }
         }
-        $sessionUrl = api_get_path(WEB_CODE_PATH) . 'admin/resume_session.php?id_session=';
+        $sessionUrl = api_get_path(WEB_CODE_PATH).'admin/resume_session.php?id_session=';
 
         $htmlResult = null;
 
@@ -5238,7 +5238,7 @@ 
                     $sessionInfo = self::fetch($sessionId);
                     $htmlResult .= '<br />';
                     $htmlResult .= Display::url(
-                        get_lang('Session') . ': ' . $sessionInfo['name'] . ' <br />', $sessionUrl . $sessionId, array('target' => '_blank')
+                        get_lang('Session').': '.$sessionInfo['name'].' <br />', $sessionUrl.$sessionId, array('target' => '_blank')
                     );
                     $teacherList = array();
                     foreach ($coachList as $coachId) {
@@ -5351,7 +5351,7 @@ 
                 }
 
                 $sessionIdList = array_map('intval', $sessionIdList);
-                $sessionToString = implode("', '",  $sessionIdList);
+                $sessionToString = implode("', '", $sessionIdList);
 
                 $course = Database::get_main_table(TABLE_MAIN_COURSE);
                 $sessionCourse = Database::get_main_table(TABLE_MAIN_SESSION_COURSE);
@@ -5363,7 +5363,7 @@ 
                         INNER JOIN $courseUser cu ON (cu.c_id = c.id)
 		                WHERE src.session_id IN ('$sessionToString') AND cu.status = 1";
                 $result = Database::query($sql);
-                while($row = Database::fetch_array($result, 'ASSOC')) {
+                while ($row = Database::fetch_array($result, 'ASSOC')) {
                     $teacherListId[$row['user_id']] = $row['user_id'];
                 }
             } else {
@@ -5438,7 +5438,7 @@ 
         $toolList = self::getCourseToolToBeManaged();
 
         foreach ($toolList as $tool) {
-            $method = 'add' . $tool;
+            $method = 'add'.$tool;
             if (method_exists(get_class(), $method)) {
                 self::$method($sessionId, $courseId);
             }
@@ -5457,7 +5457,7 @@ 
         $toolList = self::getCourseToolToBeManaged();
 
         foreach ($toolList as $tool) {
-            $method = 'remove' . $tool;
+            $method = 'remove'.$tool;
             if (method_exists(get_class(), $method)) {
                 self::$method($sessionId, $courseId);
             }
@@ -5575,18 +5575,18 @@ 
                         }
                     }
 
-                    $message .= '<strong>' . get_lang('User') . '</strong> ' . $userInfo['complete_name'] . ' <br />';
+                    $message .= '<strong>'.get_lang('User').'</strong> '.$userInfo['complete_name'].' <br />';
 
                     if (!in_array($userInfo['status'], array(DRH)) && !api_is_platform_admin_by_id($userInfo['user_id'])) {
-                        $message .= get_lang('UserMustHaveTheDrhRole') . '<br />';
+                        $message .= get_lang('UserMustHaveTheDrhRole').'<br />';
                         continue;
                     }
 
                     if (!empty($sessionList)) {
-                        $message .= '<strong>' . get_lang('Sessions') . ':</strong> <br />';
-                        $message .= implode(', ', $sessionList) . '<br /><br />';
+                        $message .= '<strong>'.get_lang('Sessions').':</strong> <br />';
+                        $message .= implode(', ', $sessionList).'<br /><br />';
                     } else {
-                        $message .= get_lang('NoSessionProvided') . ' <br /><br />';
+                        $message .= get_lang('NoSessionProvided').' <br /><br />';
                     }
                 }
             }
@@ -5766,8 +5766,8 @@ 
 
         $firstAccess = api_strtotime($courseAccess['login_course_date'], 'UTC');
 
-        $endDateInSeconds = $firstAccess + $duration*24*60*60;
-        $leftDays = round(($endDateInSeconds- $currentTime) / 60 / 60 / 24);
+        $endDateInSeconds = $firstAccess + $duration * 24 * 60 * 60;
+        $leftDays = round(($endDateInSeconds - $currentTime) / 60 / 60 / 24);
 
         return $leftDays;
     }
@@ -6049,7 +6049,7 @@ 
         $sql = "SELECT COUNT(1) as count, u.id, scu.status status_in_session, u.status user_status
                 FROM $tableSessionRelCourseRelUser scu
                 INNER JOIN $tableUser u ON scu.user_id = u.id
-                WHERE scu.session_id = " . intval($sessionId) ."
+                WHERE scu.session_id = ".intval($sessionId)."
                 GROUP BY u.id";
 
         $result = Database::query($sql);
@@ -6084,7 +6084,7 @@ 
             $sfTable = Database::get_main_table(TABLE_EXTRA_FIELD);
             $sfvTable = Database::get_main_table(TABLE_EXTRA_FIELD_VALUES);
             // Join session field and session field values tables
-            $joinTable = $sfTable . ' sf INNER JOIN ' . $sfvTable . ' sfv ON sf.id = sfv.field_id';
+            $joinTable = $sfTable.' sf INNER JOIN '.$sfvTable.' sfv ON sf.id = sfv.field_id';
             $fieldsArray = array();
             foreach ($extraFields as $field) {
                 $fieldsArray[] = Database::escape_string($field);
@@ -6127,14 +6127,14 @@ 
                 for ($i = 1; $i < count($fieldsArray); $i++) {
                     $whereParams .= ', ?';
                 }
-                $whereFieldVariables = ' variable IN ( ' . $whereParams .' )';
-                $whereFieldIds = 'field_id IN ( ' . $whereParams .  ' )';
+                $whereFieldVariables = ' variable IN ( '.$whereParams.' )';
+                $whereFieldIds = 'field_id IN ( '.$whereParams.' )';
             }
             // Get session fields
             $extraField = new ExtraField('session');
             $questionMarks = substr(str_repeat('?, ', count($fieldsArray)), 0, -2);
             $fieldsList = $extraField->get_all(array(
-                ' variable IN ( ' . $questionMarks . ' )' => $fieldsArray,
+                ' variable IN ( '.$questionMarks.' )' => $fieldsArray,
             ));
             // Index session fields
             foreach ($fieldsList as $field) {
@@ -6143,7 +6143,7 @@ 
             // Get session field values
             $extra = new ExtraFieldValue('session');
             $questionMarksFields = substr(str_repeat('?, ', count($fields)), 0, -2);
-            $sessionFieldValueList = $extra->get_all(array ('where' => array('field_id IN ( ' . $questionMarksFields . ' )' => array_keys($fields))));
+            $sessionFieldValueList = $extra->get_all(array('where' => array('field_id IN ( '.$questionMarksFields.' )' => array_keys($fields))));
             // Add session fields values to session list
             foreach ($sessionList as $id => &$session) {
                 foreach ($sessionFieldValueList as $sessionFieldValue) {
@@ -6196,7 +6196,7 @@ 
             // Check the result
             if ($result < 1) {
                 // If not found any result, update error message
-                $errorResult['errorMessage'] = 'Not found any session category name ' . $categoryName;
+                $errorResult['errorMessage'] = 'Not found any session category name '.$categoryName;
             } elseif (count($result) > 1 && !$force) {
                 // If found more than one result and force is disabled, update error message
                 $errorResult['errorMessage'] = 'Found many session categories';
@@ -6255,7 +6255,7 @@ 
             // Check if session list query had result
             if (!empty($sessionList)) {
                 // implode all session id
-                $sessionIdsString = '(' . implode(', ', array_keys($sessionList)) . ')';
+                $sessionIdsString = '('.implode(', ', array_keys($sessionList)).')';
                 // Get all field variables
                 $sessionFieldList = Database::select(
                     'id, variable',
@@ -6342,7 +6342,7 @@ 
                 return $sessionList;
             } else {
                 // Not found result, update error message
-                $errorResult['errorMessage'] = 'Not found any session for session category id ' . $sessionCategoryId;
+                $errorResult['errorMessage'] = 'Not found any session for session category id '.$sessionCategoryId;
             }
         }
 
@@ -6446,7 +6446,7 @@ 
 
         $sessionExtraField = new ExtraField('session');
         $fieldList = $sessionExtraField->get_all(array(
-            "variable IN ( " . implode(", ", $variablePlaceHolders) . " ) " => $variables,
+            "variable IN ( ".implode(", ", $variablePlaceHolders)." ) " => $variables,
         ));
 
         $fields = array();
@@ -6460,7 +6460,7 @@ 
         $extra = new ExtraFieldValue('session');
         $sessionFieldValueList = $extra->get_all(
             array(
-                "field_id IN ( " . implode(", ", $variablePlaceHolders) . " )" => array_keys($fields),
+                "field_id IN ( ".implode(", ", $variablePlaceHolders)." )" => array_keys($fields),
             )
         );
 
@@ -6531,7 +6531,7 @@ 
                 FROM $sessionTable s
                 INNER JOIN $sessionUserTable sru ON s.id = sru.id_session
                 WHERE
-                    (sru.id_user IN (" . implode(', ', $userIdList) . ")
+                    (sru.id_user IN (".implode(', ', $userIdList).")
                     AND sru.relation_type = 0
                 )";
 
@@ -6547,7 +6547,7 @@ 
                         WHERE
                             srau.access_url_id = $accessUrlId
                             AND (
-                                sru.id_user IN (" . implode(', ', $userIdList) . ")
+                                sru.id_user IN (".implode(', ', $userIdList).")
                                 AND sru.relation_type = 0
                             )";
             }
@@ -6568,7 +6568,7 @@ 
      */
     public static function getSessionVisibility($sessionInfo)
     {
-        switch($sessionInfo['visibility']) {
+        switch ($sessionInfo['visibility']) {
             case 1:
                 return get_lang('ReadOnly');
             case 2:
@@ -6599,7 +6599,7 @@ 
 
         $result = '';
         if (!empty($startDateToLocal) && !empty($endDateToLocal)) {
-            $result =  sprintf(get_lang('FromDateXToDateY'), $startDateToLocal, $endDateToLocal);
+            $result = sprintf(get_lang('FromDateXToDateY'), $startDateToLocal, $endDateToLocal);
         } else {
             if (!empty($startDateToLocal)) {
                 $result = get_lang('From').' '.$startDateToLocal;
@@ -6742,7 +6742,7 @@ 
                     get_lang('CoachName'),
                     null,
                     [
-                        'url' => api_get_path(WEB_AJAX_PATH) . 'session.ajax.php?a=search_general_coach',
+                        'url' => api_get_path(WEB_AJAX_PATH).'session.ajax.php?a=search_general_coach',
                         'width' => '100%',
                     ]
                 );
@@ -6753,7 +6753,7 @@ 
         $form->addHtml('<div id="ajax_list_coachs"></div>');
 
         $form->addButtonAdvancedSettings('advanced_params');
-        $form->addElement('html','<div id="advanced_params_options" style="display:none">');
+        $form->addElement('html', '<div id="advanced_params_options" style="display:none">');
 
         $form->addSelect('session_category', get_lang('SessionCategory'), $categoriesOptions, array(
             'id' => 'session_category'
@@ -6891,7 +6891,7 @@ 
         $extra_field = new ExtraField('session');
         $extra = $extra_field->addElements($form, $sessionId);
 
-        $form->addElement('html','</div>');
+        $form->addElement('html', '</div>');
 
         $js = $extra['jquery_ready_content'];
 
@@ -6922,16 +6922,16 @@ 
         if (api_is_session_admin() &&
             api_get_setting('allow_session_admins_to_see_all_sessions') == 'false'
         ) {
-            $where.=" WHERE s.session_admin_id = $user_id ";
+            $where .= " WHERE s.session_admin_id = $user_id ";
         }
 
         if (!empty($options['where'])) {
             $options['where'] = str_replace('course_title', 'c.title', $options['where']);
-            $options['where'] = str_replace("( session_active = '0' )", '1=1',  $options['where']);
+            $options['where'] = str_replace("( session_active = '0' )", '1=1', $options['where']);
 
             $options['where'] = str_replace(
                 array("AND session_active = '1'  )", " AND (  session_active = '1'  )"),
-                array(') GROUP BY s.name HAVING session_active = 1 ', " GROUP BY s.name HAVING session_active = 1 " )
+                array(') GROUP BY s.name HAVING session_active = 1 ', " GROUP BY s.name HAVING session_active = 1 ")
                 , $options['where']
             );
 
@@ -6973,10 +6973,10 @@ 
                        $where ";
 
         if (api_is_multiple_url_enabled()) {
-            $table_access_url_rel_session= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
+            $table_access_url_rel_session = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
             $access_url_id = api_get_current_access_url_id();
             if ($access_url_id != -1) {
-                $where.= " AND ar.access_url_id = $access_url_id ";
+                $where .= " AND ar.access_url_id = $access_url_id ";
 
                 $query_rows = "SELECT count(*) as total_rows
                                FROM $tbl_session s
@@ -7021,12 +7021,12 @@ 
                     //get_lang('CourseTitle'),
                     get_lang('Visibility'),
                 );
-                $column_model = array (
-                    array('name'=>'name', 'index'=>'s.name', 'width'=>'160',  'align'=>'left', 'search' => 'true', 'searchoptions' => array('sopt' => $operators)),
-                    array('name'=>'category_name', 'index'=>'category_name', 'width'=>'40',  'align'=>'left', 'search' => 'true', 'searchoptions' => array('sopt' => $operators)),
-                    array('name'=>'display_start_date', 'index'=>'display_start_date', 'width'=>'50',   'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_today', 'sopt' => $date_operators)),
-                    array('name'=>'display_end_date', 'index'=>'display_end_date', 'width'=>'50',   'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_one_month', 'sopt' => $date_operators)),
-                    array('name'=>'visibility', 'index'=>'visibility',      'width'=>'40',   'align'=>'left', 'search' => 'false'),
+                $column_model = array(
+                    array('name'=>'name', 'index'=>'s.name', 'width'=>'160', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('sopt' => $operators)),
+                    array('name'=>'category_name', 'index'=>'category_name', 'width'=>'40', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('sopt' => $operators)),
+                    array('name'=>'display_start_date', 'index'=>'display_start_date', 'width'=>'50', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_today', 'sopt' => $date_operators)),
+                    array('name'=>'display_end_date', 'index'=>'display_end_date', 'width'=>'50', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_one_month', 'sopt' => $date_operators)),
+                    array('name'=>'visibility', 'index'=>'visibility', 'width'=>'40', 'align'=>'left', 'search' => 'false'),
                 );
                 break;
             case 'complete':
@@ -7039,12 +7039,12 @@ 
                     get_lang('Visibility'),
                     get_lang('CourseTitle'),
                 );
-                $column_model = array (
-                    array('name'=>'name', 'index'=>'s.name', 'width'=>'200',  'align'=>'left', 'search' => 'true', 'searchoptions' => array('sopt' => $operators)),
-                    array('name'=>'display_start_date', 'index'=>'display_start_date', 'width'=>'70',   'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_today', 'sopt' => $date_operators)),
-                    array('name'=>'display_end_date', 'index'=>'display_end_date', 'width'=>'70',   'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_one_month', 'sopt' => $date_operators)),
-                    array('name'=>'coach_name', 'index'=>'coach_name',     'width'=>'70',   'align'=>'left', 'search' => 'false', 'searchoptions' => array('sopt' => $operators)),
-                    array('name'=>'session_active', 'index'=>'session_active', 'width'=>'25',   'align'=>'left', 'search' => 'true', 'stype'=>'select',
+                $column_model = array(
+                    array('name'=>'name', 'index'=>'s.name', 'width'=>'200', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('sopt' => $operators)),
+                    array('name'=>'display_start_date', 'index'=>'display_start_date', 'width'=>'70', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_today', 'sopt' => $date_operators)),
+                    array('name'=>'display_end_date', 'index'=>'display_end_date', 'width'=>'70', 'align'=>'left', 'search' => 'true', 'searchoptions' => array('dataInit' => 'date_pick_one_month', 'sopt' => $date_operators)),
+                    array('name'=>'coach_name', 'index'=>'coach_name', 'width'=>'70', 'align'=>'left', 'search' => 'false', 'searchoptions' => array('sopt' => $operators)),
+                    array('name'=>'session_active', 'index'=>'session_active', 'width'=>'25', 'align'=>'left', 'search' => 'true', 'stype'=>'select',
                         // for the bottom bar
                         'searchoptions' => array(
                             'defaultValue'  => '1',
@@ -7052,8 +7052,8 @@ 
                         // for the top bar
                         'editoptions' => array('value' => '" ":'.get_lang('All').';1:'.get_lang('Active').';0:'.get_lang('Inactive')),
                     ),
-                    array('name'=>'visibility',     'index'=>'visibility',      'width'=>'40',   'align'=>'left', 'search' => 'false'),
-                    array('name'=>'course_title',    'index'=>'course_title',   'width'=>'50',   'hidden' => 'true', 'search' => 'true', 'searchoptions' => array('searchhidden' =>'true','sopt' => $operators)),
+                    array('name'=>'visibility', 'index'=>'visibility', 'width'=>'40', 'align'=>'left', 'search' => 'false'),
+                    array('name'=>'course_title', 'index'=>'course_title', 'width'=>'50', 'hidden' => 'true', 'search' => 'true', 'searchoptions' => array('searchhidden' =>'true', 'sopt' => $operators)),
                 );
                 break;
         }
@@ -7062,14 +7062,14 @@ 
         $session_field = new ExtraField('session');
         $rules = $session_field->getRules($columns, $column_model);
 
-        $column_model[] = array('name'=>'actions', 'index'=>'actions', 'width'=>'80',  'align'=>'left','formatter'=>'action_formatter','sortable'=>'false', 'search' => 'false');
+        $column_model[] = array('name'=>'actions', 'index'=>'actions', 'width'=>'80', 'align'=>'left', 'formatter'=>'action_formatter', 'sortable'=>'false', 'search' => 'false');
         $columns[] = get_lang('Actions');
 
         foreach ($column_model as $col_model) {
             $simple_column_name[] = $col_model['name'];
         }
 
-        $return_array =  array(
+        $return_array = array(
             'columns' => $columns,
             'column_model' => $column_model,
             'rules' => $rules,
@@ -7158,7 +7158,7 @@ 
             if (api_is_session_admin() &&
                 api_get_setting('allow_session_admins_to_manage_all_sessions') == 'false'
             ) {
-                $where.=" AND s.session_admin_id = $user_id ";
+                $where .= " AND s.session_admin_id = $user_id ";
             }
         }
 
@@ -7241,11 +7241,11 @@ 
             }
             $options['where'] = str_replace('course_title', 'c.title', $options['where']);
 
-            $options['where'] = str_replace("( session_active = '0' )", '1=1',  $options['where']);
+            $options['where'] = str_replace("( session_active = '0' )", '1=1', $options['where']);
 
             $options['where'] = str_replace(
                 array("AND session_active = '1'  )", " AND (  session_active = '1'  )"),
-                array(') GROUP BY s.name HAVING session_active = 1 ', " GROUP BY s.name HAVING session_active = 1 " )
+                array(') GROUP BY s.name HAVING session_active = 1 ', " GROUP BY s.name HAVING session_active = 1 ")
                 , $options['where']
             );
 
@@ -7280,10 +7280,10 @@ 
             $where;
 
         if (api_is_multiple_url_enabled()) {
-            $table_access_url_rel_session= Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
+            $table_access_url_rel_session = Database::get_main_table(TABLE_MAIN_ACCESS_URL_REL_SESSION);
             $access_url_id = api_get_current_access_url_id();
             if ($access_url_id != -1) {
-                $where.= " AND ar.access_url_id = $access_url_id ";
+                $where .= " AND ar.access_url_id = $access_url_id ";
                 $query = "$select
                     FROM $tbl_session s
                     LEFT JOIN $tbl_session_field_values fv ON (fv.session_id = s.id)
@@ -7310,7 +7310,7 @@ 
         $formatted_sessions = array();
 
         if (Database::num_rows($result)) {
-            $sessions   = Database::store_result($result, 'ASSOC');
+            $sessions = Database::store_result($result, 'ASSOC');
             foreach ($sessions as $session) {
                 $session_id = $session['id'];
                 $session['name'] = Display::url($session['name'], "resume_session.php?id_session=".$session['id']);
@@ -7325,14 +7325,14 @@ 
 
                 switch ($session['visibility']) {
                     case SESSION_VISIBLE_READ_ONLY: //1
-                        $session['visibility'] =  get_lang('ReadOnly');
+                        $session['visibility'] = get_lang('ReadOnly');
                         break;
                     case SESSION_VISIBLE:           //2
                     case SESSION_AVAILABLE:         //4
-                        $session['visibility'] =  get_lang('Visible');
+                        $session['visibility'] = get_lang('Visible');
                         break;
                     case SESSION_INVISIBLE:         //3
-                        $session['visibility'] =  api_ucfirst(get_lang('Invisible'));
+                        $session['visibility'] = api_ucfirst(get_lang('Invisible'));
                         break;
                 }
 
@@ -7408,7 +7408,7 @@ 
         if (empty($session)) {
             return false;
         }
-        return api_get_path(WEB_CODE_PATH) . 'session/resume_session.php?id_session=' . $id;
+        return api_get_path(WEB_CODE_PATH).'session/resume_session.php?id_session='.$id;
     }
 
     /**
@@ -7426,7 +7426,7 @@ 
             return false;
         }
         if (empty($courseId)) {
-            return api_get_path(WEB_CODE_PATH) . 'session/index.php?session_id=' . $id;
+            return api_get_path(WEB_CODE_PATH).'session/index.php?session_id='.$id;
         } else {
             $courseInfo = api_get_course_info_by_id($courseId);
             if ($courseInfo) {
@@ -7672,7 +7672,7 @@ 
             // sort $listCat by catSessionName
             usort($listCat, 'self::compareBySessionName');
             // in each catSession sort sessionList by sessionName
-            foreach($listCat as $i => $listCatSessionInfo) {
+            foreach ($listCat as $i => $listCatSessionInfo) {
                 $listSessionList = $listCatSessionInfo['sessionList'];
                 usort($listSessionList, 'self::compareCatSessionInfo');
                 $listCat[$i]['sessionList'] = $listSessionList;
@@ -7687,8 +7687,8 @@ 
             );
 
             $userCatId = intval($userCatId);
-            $listResults[$userCatId]['courseInUserCategoryId'] =  $userCatId;
-            $listResults[$userCatId]['courseInUserCategoryTitle'] =  $userCatTitle;
+            $listResults[$userCatId]['courseInUserCategoryId'] = $userCatId;
+            $listResults[$userCatId]['courseInUserCategoryTitle'] = $userCatTitle;
             $listResults[$userCatId]['courseInUserCatList'][] = $listOneCourse;
         }
 
@@ -7714,7 +7714,7 @@ 
     {
         if ($listA['sessionName'] == $listB['sessionName']) {
             return 0;
-        } else if($listA['sessionName'] > $listB['sessionName']) {
+        } else if ($listA['sessionName'] > $listB['sessionName']) {
             return 1;
         } else {
             return -1;
@@ -7734,7 +7734,7 @@ 
             return 1;
         } else if ($listA['catSessionName'] == $listB['catSessionName']) {
             return 0;
-        } else if($listA['catSessionName'] > $listB['catSessionName']) {
+        } else if ($listA['catSessionName'] > $listB['catSessionName']) {
             return 1;
         } else {
             return -1;
@@ -7750,7 +7750,7 @@ 
     {
         if ($listA['courseInUserCategoryTitle'] == $listB['courseInUserCategoryTitle']) {
             return 0;
-        } else if($listA['courseInUserCategoryTitle'] > $listB['courseInUserCategoryTitle']) {
+        } else if ($listA['courseInUserCategoryTitle'] > $listB['courseInUserCategoryTitle']) {
             return 1;
         } else {
             return -1;
@@ -7766,7 +7766,7 @@ 
     {
         if ($listA['title'] == $listB['title']) {
             return 0;
-        } else if($listA['title'] > $listB['title']) {
+        } else if ($listA['title'] > $listB['title']) {
             return 1;
         } else {
             return -1;
@@ -7812,8 +7812,8 @@ 
 
                 $marginShift = 20;
                 if ($catSessionName != '') {
-                    $htmlCatSessions .= '<div style="margin-left:'.$marginShift.'px;">' .
-                        CourseManager::course_item_html($listParamsCatSession, true) . '</div>';
+                    $htmlCatSessions .= '<div style="margin-left:'.$marginShift.'px;">'.
+                        CourseManager::course_item_html($listParamsCatSession, true).'</div>';
                     $marginShift = 40;
                 }
 

Doc Comments +25 added lines, -20 removed lines

@@ -1334,9 +1334,9 @@ 
      * @param integer   $sessionCategoryId
      * @param int       $visibility
      * @param string    $description
-     * @param bool      $showDescription
+     * @param integer      $showDescription
      * @param int       $duration
-     * @param array     $extraFields
+     * @param boolean|null     $extraFields
      * @param int       $sessionAdminId
      * @param boolean $sendSubscriptionNotification Optional.
      *          Whether send a mail notification to users being subscribed
@@ -1565,7 +1565,7 @@ 
      * @param array $user_list
      * @param int $session_visibility
      * @param bool $empty_users
-     * @return bool
+     * @return false|null
      */
     public static function suscribe_users_to_session(
         $id_session,
@@ -1837,7 +1837,7 @@ 
      * @param array $courseInfo
      * @param int $status
      * @param bool $updateTotal
-     * @return bool
+     * @return false|null
      */
     public static function removeUsersFromCourseSession(
         $userList,
@@ -1903,7 +1903,7 @@ 
      * @param string $course_code
      * @param int $session_visibility
      * @param bool $removeUsersNotInList
-     * @return bool
+     * @return false|null
      */
     public static function subscribe_users_to_session_course(
         $user_list,
@@ -2086,7 +2086,7 @@ 
      * @param	bool	$removeExistingCoursesWithUsers Whether to unsubscribe
      * existing courses and users (true, default) or not (false)
      * @param $copyEvaluation from base course to session course
-     * @return	void	Nothing, or false on error
+     * @return	false|null	Nothing, or false on error
      * */
     public static function add_courses_to_session(
         $sessionId,
@@ -2282,6 +2282,8 @@ 
      *
      * @param int Session id
      * @param int Course id
+     * @param integer $session_id
+     * @param integer $course_id
      * @return bool True in case of success, false otherwise
      */
     public static function unsubscribe_course_from_session($session_id, $course_id)
@@ -2337,7 +2339,7 @@ 
      * @param	string	$variable Field's internal variable name
      * @param	int		$fieldType Field's type
      * @param	string	$displayText Field's language var name
-     * @return int     new extra field id
+     * @return boolean     new extra field id
      */
     public static function create_session_extra_field($variable, $fieldType, $displayText)
     {
@@ -2356,7 +2358,7 @@ 
      * @param	integer	Course ID
      * @param	string	Field variable name
      * @param	string	Field value
-     * @return	boolean	true if field updated, false otherwise
+     * @return	boolean|null	true if field updated, false otherwise
      */
     public static function update_session_extra_field_value($sessionId, $variable, $value = '')
     {
@@ -2561,7 +2563,7 @@ 
      * @param	array	id_checked
      * @param	bool	include delete session
      * @param	bool	optional, true if the function is called by a webservice, false otherwise.
-     * @return	void	Nothing, or false on error
+     * @return	boolean	Nothing, or false on error
      * The parameters is a array to delete sessions
      * */
     public static function delete_session_category($id_checked, $delete_session = false, $from_ws = false)
@@ -3056,6 +3058,7 @@ 
      * @param string $orderCondition
      * @param string $keyword
      * @param string $description
+     * @param integer $status
      * @return array sessions
      */
     public static function getSessionsFollowedByUser(
@@ -3268,8 +3271,8 @@ 
     /**
      * Gets the list of courses by session filtered by access_url
      *
-     * @param $userId
-     * @param $sessionId
+     * @param integer $userId
+     * @param null|integer $sessionId
      * @param null $from
      * @param null $limit
      * @param null $column
@@ -3394,6 +3397,7 @@ 
     /**
      * Gets the count of courses by session filtered by access_url
      * @param int session id
+     * @param integer $session_id
      * @return array list of courses
      */
     public static function getCourseCountBySessionId($session_id, $keyword = null)
@@ -3645,6 +3649,7 @@ 
      * Updates a session status
      * @param	int 	session id
      * @param	int 	status
+     * @param integer $status
      */
     public static function set_session_status($session_id, $status)
     {
@@ -3853,7 +3858,7 @@ 
      * Protect a session to be edited.
      * @param int $id
      * @param bool $checkSession
-     * @return mixed | bool true if pass the check, api_not_allowed otherwise
+     * @return boolean|null | bool true if pass the check, api_not_allowed otherwise
      */
     public static function protectSession($id, $checkSession = true)
     {
@@ -3961,7 +3966,7 @@ 
 
     /**
      * @param $id
-     * @return bool
+     * @return null|boolean
      */
     public static function protect_teacher_session_edit($id)
     {
@@ -4025,7 +4030,7 @@ 
      *  true: if the session exists it will be updated.
      *  false: if session exists a new session will be created adding a counter session1, session2, etc
      * @param int $defaultUserId
-     * @param mixed $logger
+     * @param Logger $logger
      * @param array $extraFields convert a file row to an extra field. Example in CSV file there's a SessionID then it will
      * converted to extra_external_session_id if you set this: array('SessionId' => 'extra_external_session_id')
      * @param string $extraFieldId
@@ -5267,7 +5272,6 @@ 
      * @param string $lastConnectionDate
      * @param array $sessionIdList
      * @param array $studentIdList
-     * @param int $userStatus STUDENT|COURSEMANAGER constants
      *
      * @return array|int
      */
@@ -5419,7 +5423,7 @@ 
     /**
      * Get the list of course tools that have to be dealt with in case of
      * registering any course to a session
-     * @return array The list of tools to be dealt with (literal names)
+     * @return string[] The list of tools to be dealt with (literal names)
      */
     public static function getCourseToolToBeManaged()
     {
@@ -5433,7 +5437,7 @@ 
      * Calls the methods bound to each tool when a course is registered into a session
      * @param int $sessionId
      * @param int $courseId
-     * @return void
+     * @return boolean|null
      */
     public static function installCourse($sessionId, $courseId)
     {
@@ -6073,6 +6077,7 @@ 
      * @param int $categoryId The internal ID of the session category
      * @param string $target Value to search for in the session field values
      * @param array $extraFields A list of fields to be scanned and returned
+     * @param DateTime $publicationDate
      * @return mixed
      */
     public static function getShortSessionListAndExtraByCategory($categoryId, $target, $extraFields = null, $publicationDate = null)
@@ -7402,7 +7407,7 @@ 
     /**
      * Get link to the admin page for this session
      * @param   int $id Session ID
-     * @return mixed    URL to the admin page to manage the session, or false on error
+     * @return false|string    URL to the admin page to manage the session, or false on error
      */
     public static function getAdminPath($id)
     {
@@ -7419,7 +7424,7 @@ 
      * If a course is provided, build the link to the course
      * @param   int $id Session ID
      * @param   int $courseId Course ID (optional) in case the link has to send straight to the course
-     * @return mixed    URL to the page to use the session, or false on error
+     * @return false|string    URL to the page to use the session, or false on error
      */
     public static function getPath($id, $courseId = 0)
     {
@@ -7517,7 +7522,7 @@ 
     /**
      * Return true if coach is allowed to access this session
      * @param int $sessionId
-     * @return bool
+     * @return integer
      */
     public static function isSessionDateOkForCoach($sessionId)
     {

main/inc/lib/attendance.lib.php

Spacing +63 added lines, -63 removed lines

@@ -73,7 +73,7 @@ 
             $course_id = intval($course_id);
         }
 
-        $session_id = isset($session_id)?intval($session_id):api_get_session_id();
+        $session_id = isset($session_id) ? intval($session_id) : api_get_session_id();
         $condition_session = api_get_session_condition($session_id);
 
         // Get attendance data
@@ -82,7 +82,7 @@ 
                 WHERE c_id = $course_id AND active = 1 $condition_session ";
         $rs  = Database::query($sql);
         if (Database::num_rows($rs) > 0) {
-            while ($row = Database::fetch_array($rs,'ASSOC')) {
+            while ($row = Database::fetch_array($rs, 'ASSOC')) {
                 $data[$row['id']] = $row;
             }
         }
@@ -108,7 +108,7 @@ 
 		$from = intval($from);
 		$number_of_items = intval($number_of_items);
 
-		if (!in_array($direction, array('ASC','DESC'))) {
+		if (!in_array($direction, array('ASC', 'DESC'))) {
 			$direction = 'ASC';
 		}
 
@@ -135,7 +135,7 @@ 
 				LIMIT $from,$number_of_items ";
 
 		$res = Database::query($sql);
-		$attendances = array ();
+		$attendances = array();
 		$user_info = api_get_user_info();
 		$allowDelete = api_get_setting('allow_delete_attendance');
 
@@ -194,8 +194,8 @@ 
 						$attendance[2] = '<span class="muted">'.$attendance[2].'</span>';
 					}
 					if ($allowDelete === 'true') {
-						$actions .= '<a href="index.php?' . api_get_cidreq() . '&action=attendance_delete&attendance_id=' . $attendance[0]. '">' .
-							Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL) . '</a>';
+						$actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_delete&attendance_id='.$attendance[0].'">'.
+							Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL).'</a>';
 					}
 				} else {
 					$is_locked_attendance = self::is_locked_attendance($attendance[0]);
@@ -215,8 +215,8 @@ 
 							$attendance[2] = '<span class="muted">'.$attendance[2].'</span>';
 						}
 						if ($allowDelete === 'true') {
-							$actions .= ' <a href="index.php?' . api_get_cidreq() . '&action=attendance_delete&attendance_id=' . $attendance[0].'">' .
-								Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL) . '</a>';
+							$actions .= ' <a href="index.php?'.api_get_cidreq().'&action=attendance_delete&attendance_id='.$attendance[0].'">'.
+								Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL).'</a>';
 						}
 					}
 				}
@@ -225,7 +225,7 @@ 
 				$is_done_all_calendar = self::is_all_attendance_calendar_done($attendance[0]);
 
 				if ($is_done_all_calendar) {
-					$locked   = $attendance[4];
+					$locked = $attendance[4];
 					if ($locked == 0) {
 						if (api_is_platform_admin()) {
 							$message_alert = get_lang('AreYouSureToLockTheAttendance');
@@ -245,7 +245,7 @@ 
 				}
 				$actions .= '</center>';
 
-				$attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3],$actions);
+				$attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3], $actions);
 			} else {
 				$attendance[0] = '&nbsp;';
 				$attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3]);
@@ -285,13 +285,13 @@ 
 	public function attendance_add($link_to_gradebook = false)
 	{
 		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+		$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
 		$table_link = Database:: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
 		$session_id = api_get_session_id();
 		$user_id = api_get_user_id();
 		$course_code = $_course['code'];
 		$course_id = $_course['real_id'];
-		$title_gradebook= $this->attendance_qualify_title;
+		$title_gradebook = $this->attendance_qualify_title;
 		$value_calification = 0;
 		$weight_calification = floatval($this->attendance_weight);
 
@@ -434,12 +434,12 @@ 
 	public function attendance_restore($attendance_id)
 	{
 		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+		$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
 		$user_id = api_get_user_id();
 		$course_id = $_course['real_id'];
 		if (is_array($attendance_id)) {
 			foreach ($attendance_id as $id) {
-				$id	= intval($id);
+				$id = intval($id);
 				$sql = "UPDATE $tbl_attendance SET active = 1
 						WHERE c_id = $course_id AND id = '$id'";
 				$result = Database::query($sql);
@@ -455,8 +455,8 @@ 
                     );
 				}
 			}
-		} else  {
-			$attendance_id	= intval($attendance_id);
+		} else {
+			$attendance_id = intval($attendance_id);
 			$sql = "UPDATE $tbl_attendance SET active = 1
 					WHERE c_id = $course_id AND id = '$attendance_id'";
 			$result = Database::query($sql);
@@ -484,13 +484,13 @@ 
 	public function attendance_delete($attendance_id)
 	{
 		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+		$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
         $user_id = api_get_user_id();
         $course_id = $_course['real_id'];
 
 		if (is_array($attendance_id)) {
 			foreach ($attendance_id as $id) {
-				$id	= intval($id);
+				$id = intval($id);
 				$sql = "UPDATE $tbl_attendance SET active = 2
 						WHERE c_id = $course_id AND id = '$id'";
 				$result = Database::query($sql);
@@ -506,8 +506,8 @@ 
                     );
 				}
 			}
-		} else  {
-			$attendance_id= intval($attendance_id);
+		} else {
+			$attendance_id = intval($attendance_id);
 			$sql = "UPDATE $tbl_attendance SET active = 2
 					WHERE c_id = $course_id AND id = '$attendance_id'";
 
@@ -538,7 +538,7 @@ 
 	public function changeVisibility($attendanceId, $status = 1)
 	{
 		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+		$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
 		$user_id = api_get_user_id();
 		$course_id = $_course['real_id'];
 		$status = intval($status);
@@ -550,7 +550,7 @@ 
 
 		if (is_array($attendanceId)) {
 			foreach ($attendanceId as $id) {
-				$id	= intval($id);
+				$id = intval($id);
 				$sql = "UPDATE $tbl_attendance SET active = $status
 						WHERE c_id = $course_id AND id = '$id'";
 				$result = Database::query($sql);
@@ -560,8 +560,8 @@ 
 					api_item_property_update($_course, TOOL_ATTENDANCE, $id, $action, $user_id);
 				}
 			}
-		} else  {
-			$attendanceId	= intval($attendanceId);
+		} else {
+			$attendanceId = intval($attendanceId);
 			$sql = "UPDATE $tbl_attendance SET active = $status
 					WHERE c_id = $course_id AND id = '$attendanceId'";
 			$result = Database::query($sql);
@@ -591,7 +591,7 @@ 
 		$tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
 		$course_id = api_get_course_int_id();
 		$attendance_id = intval($attendance_id);
-		$locked = ($lock)?1:0;
+		$locked = ($lock) ? 1 : 0;
 		$upd = "UPDATE $tbl_attendance SET locked = $locked
 				WHERE c_id = $course_id AND id = $attendance_id";
 		$result = Database::query($upd);
@@ -720,8 +720,8 @@ 
 	 */
 	public function attendance_sheet_add($calendar_id, $users_present, $attendance_id)
 	{
-		$tbl_attendance_sheet 	= Database::get_course_table(TABLE_ATTENDANCE_SHEET);
-		$tbl_attendance_calendar= Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+		$tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
+		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
 
 		$calendar_id = intval($calendar_id);
 		$attendance_id = intval($attendance_id);
@@ -729,7 +729,7 @@ 
 		$course_id = api_get_course_int_id();
 
 		$user_ids = array_keys($users);
-		$users_absent = array_diff($user_ids,$users_present);
+		$users_absent = array_diff($user_ids, $users_present);
 		$affected_rows = 0;
 
 		// get last edit type
@@ -993,13 +993,13 @@ 
 	public function get_faults_of_user($user_id, $attendance_id, $groupId = null)
 	{
 		// initializing database table and variables
-		$user_id 		= intval($user_id);
+		$user_id = intval($user_id);
 		$attendance_id 	= intval($attendance_id);
 		$results = array();
 		$calendar_count = self::get_number_of_attendance_calendar($attendance_id, $groupId, NULL, $user_id);
 		// $total_done_attendance 	= $attendance_data['attendance_qualify_max'];
-        $total_done_attendance 	= self::get_number_of_attendance_calendar($attendance_id, $groupId, true, $user_id);
-		$attendance_user_score  = $this->get_user_score($user_id, $attendance_id, $groupId);
+        $total_done_attendance = self::get_number_of_attendance_calendar($attendance_id, $groupId, true, $user_id);
+		$attendance_user_score = $this->get_user_score($user_id, $attendance_id, $groupId);
 
 		//This is the main change of the BT#1381
 		//$total_done_attendance = $calendar_count;
@@ -1012,7 +1012,7 @@ 
         }
 
         $faults = $faults > 0 ? $faults : 0;
-		$faults_porcent = $calendar_count > 0 ?round(($faults*100)/$calendar_count,0):0;
+		$faults_porcent = $calendar_count > 0 ? round(($faults * 100) / $calendar_count, 0) : 0;
         $results['faults'] = $faults;
         $results['total'] = $calendar_count;
         $results['faults_porcent'] = $faults_porcent;
@@ -1046,13 +1046,13 @@ 
 			//$course_code = $course['code'];
 			//$course_info = api_get_course_info($course_code);
 			$course_id = $course['real_id'];
-			$tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+			$tbl_attendance_result = Database::get_course_table(TABLE_ATTENDANCE_RESULT);
 
 			$attendances_by_course = $this->get_attendances_list($course_id);
 
 			foreach ($attendances_by_course as $attendance) {
 				// get total faults and total weight
-				$total_done_attendance 	= $attendance['attendance_qualify_max'];
+				$total_done_attendance = $attendance['attendance_qualify_max'];
 				$sql = "SELECT score
 						FROM $tbl_attendance_result
                         WHERE
@@ -1065,16 +1065,16 @@ 
 					$row = Database::fetch_array($rs);
 					$score = $row['score'];
 				}
-				$faults = $total_done_attendance-$score;
-				$faults = $faults > 0 ? $faults:0;
+				$faults = $total_done_attendance - $score;
+				$faults = $faults > 0 ? $faults : 0;
 				$total_faults += $faults;
 				$total_weight += $total_done_attendance;
 			}
 		}
 
-		$porcent = $total_weight > 0 ?round(($total_faults*100)/$total_weight,0):0;
+		$porcent = $total_weight > 0 ? round(($total_faults * 100) / $total_weight, 0) : 0;
 		$results['faults'] 	= $total_faults;
-		$results['total']	= $total_weight;
+		$results['total'] = $total_weight;
 		$results['porcent'] = $porcent;
 
 		return $results;
@@ -1092,7 +1092,7 @@ 
 	{
 		// Database tables and variables
 		$course_info = api_get_course_info($course_code);
-		$tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+		$tbl_attendance_result = Database::get_course_table(TABLE_ATTENDANCE_RESULT);
 		$user_id = intval($user_id);
 		$results = array();
 		$total_faults = $total_weight = $porcent = 0;
@@ -1100,7 +1100,7 @@ 
 
 		foreach ($attendances_by_course as $attendance) {
 			// Get total faults and total weight
-			$total_done_attendance 	= $attendance['attendance_qualify_max'];
+			$total_done_attendance = $attendance['attendance_qualify_max'];
 			$sql = "SELECT score FROM $tbl_attendance_result
 					WHERE
 						c_id = {$course_info['real_id']} AND
@@ -1112,15 +1112,15 @@ 
 				$row = Database::fetch_array($rs);
 				$score = $row['score'];
 			}
-			$faults = $total_done_attendance-$score;
-			$faults = $faults > 0 ? $faults:0;
+			$faults = $total_done_attendance - $score;
+			$faults = $faults > 0 ? $faults : 0;
 			$total_faults += $faults;
 			$total_weight += $total_done_attendance;
 		}
 
-		$porcent = $total_weight > 0 ?round(($total_faults*100)/$total_weight,0):0;
+		$porcent = $total_weight > 0 ? round(($total_faults * 100) / $total_weight, 0) : 0;
 		$results['faults'] 	= $total_faults;
-		$results['total']	= $total_weight;
+		$results['total'] = $total_weight;
 		$results['porcent'] = $porcent;
 
 		return $results;
@@ -1134,8 +1134,8 @@ 
 	 */
 	public function get_users_attendance_sheet($attendance_id, $user_id = 0, $groupId = null)
 	{
-		$tbl_attendance_sheet 	= Database::get_course_table(TABLE_ATTENDANCE_SHEET);
-		$tbl_attendance_calendar= Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+		$tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
+		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
 
 		$attendance_calendar = $this->get_attendance_calendar($attendance_id, 'all', null, $groupId);
 		$calendar_ids = array();
@@ -1157,7 +1157,7 @@ 
 					        WHERE
 					        	c_id = $course_id AND
 					        	user_id = '$uid' AND
-					        	attendance_calendar_id IN(".implode(',',$calendar_ids).")
+					        	attendance_calendar_id IN(".implode(',', $calendar_ids).")
 							";
 					$res = Database::query($sql);
 					if (Database::num_rows($res) > 0) {
@@ -1179,7 +1179,7 @@ 
 							att.c_id = $course_id AND
 							cal.c_id =  $course_id AND
 							att.user_id = '$user_id' AND
-							att.attendance_calendar_id IN (".implode(',',$calendar_ids).")
+							att.attendance_calendar_id IN (".implode(',', $calendar_ids).")
                         ORDER BY date_time";
 				$res = Database::query($sql);
 				if (Database::num_rows($res) > 0) {
@@ -1255,7 +1255,7 @@ 
 	 */
 	public function get_user_score($user_id, $attendance_id, $groupId = null)
 	{
-		$tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+		$tbl_attendance_result = Database::get_course_table(TABLE_ATTENDANCE_RESULT);
         $tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
         $tbl_attendance_cal_rel_group = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
         $tbl_attendance_cal = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
@@ -1369,7 +1369,7 @@ 
                    ";
 		}
 
-		if (!in_array($type, array('today', 'all', 'all_done', 'all_not_done','calendar_id'))) {
+		if (!in_array($type, array('today', 'all', 'all_done', 'all_not_done', 'calendar_id'))) {
 			$type = 'all';
 		}
 
@@ -1377,7 +1377,7 @@ 
 			case 'calendar_id':
 				$calendar_id = intval($calendar_id);
 				if (!empty($calendar_id)) {
-					$sql.= " AND c.id = $calendar_id";
+					$sql .= " AND c.id = $calendar_id";
 				}
 				break;
 			case 'today':
@@ -1398,7 +1398,7 @@ 
 		$rs = Database::query($sql);
 		$data = array();
 		if (Database::num_rows($rs) > 0) {
-			while ($row = Database::fetch_array($rs,'ASSOC')) {
+			while ($row = Database::fetch_array($rs, 'ASSOC')) {
 				$row['db_date_time'] = $row['date_time'];
 				$row['date_time'] = api_get_local_time($row['date_time']);
 				$row['date'] = api_format_date($row['date_time'], DATE_FORMAT_SHORT);
@@ -1762,14 +1762,14 @@ 
 	 * @param   int     The number of years to add
 	 * @return  int     The new timestamp
 	 */
-	private function add_month($timestamp, $num=1)
+	private function add_month($timestamp, $num = 1)
 	{
 		$values = api_get_utc_datetime($timestamp);
-		$values = str_replace(array(':','-',' '), '/', $values);
-		list($y, $m, $d, $h, $n, $s) = split('/',$values);
-		if($m+$num>12) {
-			$y += floor($num/12);
-			$m += $num%12;
+		$values = str_replace(array(':', '-', ' '), '/', $values);
+		list($y, $m, $d, $h, $n, $s) = split('/', $values);
+		if ($m + $num > 12) {
+			$y += floor($num / 12);
+			$m += $num % 12;
 		} else {
 			$m += $num;
 		}
@@ -1827,7 +1827,7 @@ 
 	 * @param	bool	true for removing all calendar inside current attendance, false for removing by calendar id
 	 * @return	int affected rows
 	 */
-	public function attendance_calendar_delete($calendar_id, $attendance_id , $all_delete = false)
+	public function attendance_calendar_delete($calendar_id, $attendance_id, $all_delete = false)
 	{
 		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
 		$tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
@@ -1963,7 +1963,7 @@ 
 		}
 
 		$sessionId = api_get_session_id();
-		$courseCode  = api_get_course_id();
+		$courseCode = api_get_course_id();
 		if (!empty($sessionId)) {
 			$users = CourseManager:: get_user_list_from_course_code(
 				$courseCode,
@@ -1984,7 +1984,7 @@ 
 
 		$dateTimeStartOriginal = new DateTime($startDate);
 		$dateTimeStart = new DateTime($startDate);
-		$dateTimeEnd= new DateTime($endDate);
+		$dateTimeEnd = new DateTime($endDate);
 		$interval = $dateTimeStart->diff($dateTimeEnd);
 		$days = intval($interval->format('%a'));
 
@@ -2057,7 +2057,7 @@ 
 				0,
 				$user['lastname'].' '.$user['firstname'].' ('.$user['username'].')'
 			);
-			$row ++;
+			$row++;
 		}
 
 		$column = 1;
@@ -2134,7 +2134,7 @@ 
 		$tableToString = $table->toHtml();
 
 		$params = array(
-			'filename' => get_lang('Attendance') . '_' . api_get_utc_datetime(),
+			'filename' => get_lang('Attendance').'_'.api_get_utc_datetime(),
 			'pdf_title' => get_lang('Attendance'),
 			'course_code' => api_get_course_id(),
 			'show_real_course_teachers' => true

Indentation +1704 added lines, -1704 removed lines

@@ -11,50 +11,50 @@ 
  */
 class Attendance
 {
-	private $session_id;
-	private $course_id;
-	private $date_time;
-	private $name;
-	private $description;
-	private $attendance_qualify_title;
-	private $attendance_weight;
-	private $course_int_id;
-	public $category_id;
-
-	// constants
-	const DONE_ATTENDANCE_LOG_TYPE = 'done_attendance_sheet';
-	const UPDATED_ATTENDANCE_LOG_TYPE = 'updated_attendance_sheet';
-	const LOCKED_ATTENDANCE_LOG_TYPE = 'locked_attendance_sheet';
-
-	/**
-	 * Constructor
-	 */
-	public function __construct()
-	{
-		//$this->course_int_id = api_get_course_int_id();
-	}
-
-	/**
-	 * Get the total number of attendance inside current course and current session
-	 * @see SortableTable#get_total_number_of_items()
-	 */
-	public static function get_number_of_attendances($active = -1)
-	{
-		$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
-		$session_id = api_get_session_id();
-		$condition_session = api_get_session_condition($session_id);
-		$course_id = api_get_course_int_id();
-		$sql = "SELECT COUNT(att.id) AS total_number_of_items
+    private $session_id;
+    private $course_id;
+    private $date_time;
+    private $name;
+    private $description;
+    private $attendance_qualify_title;
+    private $attendance_weight;
+    private $course_int_id;
+    public $category_id;
+
+    // constants
+    const DONE_ATTENDANCE_LOG_TYPE = 'done_attendance_sheet';
+    const UPDATED_ATTENDANCE_LOG_TYPE = 'updated_attendance_sheet';
+    const LOCKED_ATTENDANCE_LOG_TYPE = 'locked_attendance_sheet';
+
+    /**
+     * Constructor
+     */
+    public function __construct()
+    {
+        //$this->course_int_id = api_get_course_int_id();
+    }
+
+    /**
+     * Get the total number of attendance inside current course and current session
+     * @see SortableTable#get_total_number_of_items()
+     */
+    public static function get_number_of_attendances($active = -1)
+    {
+        $tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
+        $session_id = api_get_session_id();
+        $condition_session = api_get_session_condition($session_id);
+        $course_id = api_get_course_int_id();
+        $sql = "SELECT COUNT(att.id) AS total_number_of_items
 				FROM $tbl_attendance att
 		        WHERE c_id = $course_id $condition_session ";
-		if ($active == 1 || $active == 0) {
-			$sql .= "AND att.active = $active";
-		}
-		$res = Database::query($sql);
-		$obj = Database::fetch_object($res);
+        if ($active == 1 || $active == 0) {
+            $sql .= "AND att.active = $active";
+        }
+        $res = Database::query($sql);
+        $obj = Database::fetch_object($res);
 
-		return $obj->total_number_of_items;
-	}
+        return $obj->total_number_of_items;
+    }
 
     /**
      * Get attendance list only the id, name and attendance_qualify_max fields
@@ -90,36 +90,36 @@ 
         return $data;
     }
 
-	/**
-	 * Get the attendaces to display on the current page (fill the sortable-table)
-	 * @param   int     offset of first user to recover
-	 * @param   int     Number of users to get
-	 * @param   int     Column to sort on
-	 * @param   string  Order (ASC,DESC)
-	 * @see SortableTable#get_table_data($from)
-	 */
-	public static function get_attendance_data($from, $number_of_items, $column, $direction)
-	{
-		$tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
-		$course_id = api_get_course_int_id();
-		$session_id = api_get_session_id();
-		$condition_session = api_get_session_condition($session_id);
-		$column = intval($column);
-		$from = intval($from);
-		$number_of_items = intval($number_of_items);
-
-		if (!in_array($direction, array('ASC','DESC'))) {
-			$direction = 'ASC';
-		}
-
-		$active_plus = '';
-		if ((isset($_GET['isStudentView']) && $_GET['isStudentView'] == 'true') ||
-			!api_is_allowed_to_edit(null, true)
-		) {
-			$active_plus = ' AND att.active = 1';
-		}
-
-		$sql = "SELECT
+    /**
+     * Get the attendaces to display on the current page (fill the sortable-table)
+     * @param   int     offset of first user to recover
+     * @param   int     Number of users to get
+     * @param   int     Column to sort on
+     * @param   string  Order (ASC,DESC)
+     * @see SortableTable#get_table_data($from)
+     */
+    public static function get_attendance_data($from, $number_of_items, $column, $direction)
+    {
+        $tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
+        $course_id = api_get_course_int_id();
+        $session_id = api_get_session_id();
+        $condition_session = api_get_session_condition($session_id);
+        $column = intval($column);
+        $from = intval($from);
+        $number_of_items = intval($number_of_items);
+
+        if (!in_array($direction, array('ASC','DESC'))) {
+            $direction = 'ASC';
+        }
+
+        $active_plus = '';
+        if ((isset($_GET['isStudentView']) && $_GET['isStudentView'] == 'true') ||
+            !api_is_allowed_to_edit(null, true)
+        ) {
+            $active_plus = ' AND att.active = 1';
+        }
+
+        $sql = "SELECT
                     att.id AS col0,
                     att.name AS col1,
                     att.description AS col2,
@@ -134,318 +134,318 @@ 
 				ORDER BY col$column $direction
 				LIMIT $from,$number_of_items ";
 
-		$res = Database::query($sql);
-		$attendances = array ();
-		$user_info = api_get_user_info();
-		$allowDelete = api_get_setting('allow_delete_attendance');
-
-		while ($attendance = Database::fetch_row($res)) {
-
-			$student_param = '';
-			if (api_is_drh() && $_GET['student_id']) {
-				$student_param = '&student_id='.intval($_GET['student_id']);
-			}
-
-			$session_star = '';
-
-			if (api_get_session_id() == $attendance[6]) {
-				$session_star = api_get_session_image(api_get_session_id(), $user_info['status']);
-			}
-			if ($attendance[5] == 1) {
-
-				$isDrhOfCourse = CourseManager::isUserSubscribedInCourseAsDrh(
-					api_get_user_id(),
-					api_get_course_info()
-				);
-
-				if (api_is_allowed_to_edit(null, true) || $isDrhOfCourse) {
-					// Link to edit
-					$attendance[1] = '<a href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list&attendance_id='.$attendance[0].$student_param.'">'.$attendance[1].'</a>'.$session_star;
-				} else {
-					// Link to view
-					$attendance[1] = '<a href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list_no_edit&attendance_id='.$attendance[0].$student_param.'">'.$attendance[1].'</a>'.$session_star;
-				}
-
-			} else {
-				$attendance[1] = '<a class="muted" href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list&attendance_id='.$attendance[0].$student_param.'">'.$attendance[1].'</a>'.$session_star;
-			}
-
-			if ($attendance[5] == 1) {
-				$attendance[3] = '<center>'.$attendance[3].'</center>';
-			} else {
-				$attendance[3] = '<center><span class="muted">'.$attendance[3].'</span></center>';
-			}
-
-			$attendance[3] = '<center>'.$attendance[3].'</center>';
-			if (api_is_allowed_to_edit(null, true)) {
-				$actions  = '';
-				$actions .= '<center>';
-
-				if (api_is_platform_admin()) {
-					$actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_edit&attendance_id='.$attendance[0].'">'.
-						Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL).'</a>&nbsp;';
-					// Visible
-					if ($attendance[5] == 1) {
-						$actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_set_invisible&attendance_id='.$attendance[0].'">'.
-							Display::return_icon('visible.png', get_lang('Hide'), array(), ICON_SIZE_SMALL).'</a>';
-					} else {
-						$actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_set_visible&attendance_id='.$attendance[0].'">'.
-							Display::return_icon('invisible.png', get_lang('Show'), array(), ICON_SIZE_SMALL).'</a>';
-						$attendance[2] = '<span class="muted">'.$attendance[2].'</span>';
-					}
-					if ($allowDelete === 'true') {
-						$actions .= '<a href="index.php?' . api_get_cidreq() . '&action=attendance_delete&attendance_id=' . $attendance[0]. '">' .
-							Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL) . '</a>';
-					}
-				} else {
-					$is_locked_attendance = self::is_locked_attendance($attendance[0]);
-					if ($is_locked_attendance) {
-						$actions .= Display::return_icon('edit_na.png', get_lang('Edit')).'&nbsp;';
-						$actions .= Display::return_icon('visible.png', get_lang('Hide'));
-					} else {
-						$actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_edit&attendance_id='.$attendance[0].'">'.
-							Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL).'</a>&nbsp;';
-
-						if ($attendance[5] == 1) {
-							$actions .= ' <a href="index.php?'.api_get_cidreq().'&action=attendance_set_invisible&attendance_id='.$attendance[0].'">'.
-								Display::return_icon('visible.png', get_lang('Hide'), array(), ICON_SIZE_SMALL).'</a>';
-						} else {
-							$actions .= ' <a href="index.php?'.api_get_cidreq().'&action=attendance_set_visible&attendance_id='.$attendance[0].'">'.
-								Display::return_icon('invisible.png', get_lang('Show'), array(), ICON_SIZE_SMALL).'</a>';
-							$attendance[2] = '<span class="muted">'.$attendance[2].'</span>';
-						}
-						if ($allowDelete === 'true') {
-							$actions .= ' <a href="index.php?' . api_get_cidreq() . '&action=attendance_delete&attendance_id=' . $attendance[0].'">' .
-								Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL) . '</a>';
-						}
-					}
-				}
-
-				// display lock/unlock icon
-				$is_done_all_calendar = self::is_all_attendance_calendar_done($attendance[0]);
-
-				if ($is_done_all_calendar) {
-					$locked   = $attendance[4];
-					if ($locked == 0) {
-						if (api_is_platform_admin()) {
-							$message_alert = get_lang('AreYouSureToLockTheAttendance');
-						} else {
-							$message_alert = get_lang('UnlockMessageInformation');
-						}
-						$actions .= '&nbsp;<a onclick="javascript:if(!confirm(\''.$message_alert.'\')) return false;" href="index.php?'.api_get_cidreq().'&action=lock_attendance&attendance_id='.$attendance[0].'">'.
+        $res = Database::query($sql);
+        $attendances = array ();
+        $user_info = api_get_user_info();
+        $allowDelete = api_get_setting('allow_delete_attendance');
+
+        while ($attendance = Database::fetch_row($res)) {
+
+            $student_param = '';
+            if (api_is_drh() && $_GET['student_id']) {
+                $student_param = '&student_id='.intval($_GET['student_id']);
+            }
+
+            $session_star = '';
+
+            if (api_get_session_id() == $attendance[6]) {
+                $session_star = api_get_session_image(api_get_session_id(), $user_info['status']);
+            }
+            if ($attendance[5] == 1) {
+
+                $isDrhOfCourse = CourseManager::isUserSubscribedInCourseAsDrh(
+                    api_get_user_id(),
+                    api_get_course_info()
+                );
+
+                if (api_is_allowed_to_edit(null, true) || $isDrhOfCourse) {
+                    // Link to edit
+                    $attendance[1] = '<a href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list&attendance_id='.$attendance[0].$student_param.'">'.$attendance[1].'</a>'.$session_star;
+                } else {
+                    // Link to view
+                    $attendance[1] = '<a href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list_no_edit&attendance_id='.$attendance[0].$student_param.'">'.$attendance[1].'</a>'.$session_star;
+                }
+
+            } else {
+                $attendance[1] = '<a class="muted" href="index.php?'.api_get_cidreq().'&action=attendance_sheet_list&attendance_id='.$attendance[0].$student_param.'">'.$attendance[1].'</a>'.$session_star;
+            }
+
+            if ($attendance[5] == 1) {
+                $attendance[3] = '<center>'.$attendance[3].'</center>';
+            } else {
+                $attendance[3] = '<center><span class="muted">'.$attendance[3].'</span></center>';
+            }
+
+            $attendance[3] = '<center>'.$attendance[3].'</center>';
+            if (api_is_allowed_to_edit(null, true)) {
+                $actions  = '';
+                $actions .= '<center>';
+
+                if (api_is_platform_admin()) {
+                    $actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_edit&attendance_id='.$attendance[0].'">'.
+                        Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL).'</a>&nbsp;';
+                    // Visible
+                    if ($attendance[5] == 1) {
+                        $actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_set_invisible&attendance_id='.$attendance[0].'">'.
+                            Display::return_icon('visible.png', get_lang('Hide'), array(), ICON_SIZE_SMALL).'</a>';
+                    } else {
+                        $actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_set_visible&attendance_id='.$attendance[0].'">'.
+                            Display::return_icon('invisible.png', get_lang('Show'), array(), ICON_SIZE_SMALL).'</a>';
+                        $attendance[2] = '<span class="muted">'.$attendance[2].'</span>';
+                    }
+                    if ($allowDelete === 'true') {
+                        $actions .= '<a href="index.php?' . api_get_cidreq() . '&action=attendance_delete&attendance_id=' . $attendance[0]. '">' .
+                            Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL) . '</a>';
+                    }
+                } else {
+                    $is_locked_attendance = self::is_locked_attendance($attendance[0]);
+                    if ($is_locked_attendance) {
+                        $actions .= Display::return_icon('edit_na.png', get_lang('Edit')).'&nbsp;';
+                        $actions .= Display::return_icon('visible.png', get_lang('Hide'));
+                    } else {
+                        $actions .= '<a href="index.php?'.api_get_cidreq().'&action=attendance_edit&attendance_id='.$attendance[0].'">'.
+                            Display::return_icon('edit.png', get_lang('Edit'), array(), ICON_SIZE_SMALL).'</a>&nbsp;';
+
+                        if ($attendance[5] == 1) {
+                            $actions .= ' <a href="index.php?'.api_get_cidreq().'&action=attendance_set_invisible&attendance_id='.$attendance[0].'">'.
+                                Display::return_icon('visible.png', get_lang('Hide'), array(), ICON_SIZE_SMALL).'</a>';
+                        } else {
+                            $actions .= ' <a href="index.php?'.api_get_cidreq().'&action=attendance_set_visible&attendance_id='.$attendance[0].'">'.
+                                Display::return_icon('invisible.png', get_lang('Show'), array(), ICON_SIZE_SMALL).'</a>';
+                            $attendance[2] = '<span class="muted">'.$attendance[2].'</span>';
+                        }
+                        if ($allowDelete === 'true') {
+                            $actions .= ' <a href="index.php?' . api_get_cidreq() . '&action=attendance_delete&attendance_id=' . $attendance[0].'">' .
+                                Display::return_icon('delete.png', get_lang('Delete'), array(), ICON_SIZE_SMALL) . '</a>';
+                        }
+                    }
+                }
+
+                // display lock/unlock icon
+                $is_done_all_calendar = self::is_all_attendance_calendar_done($attendance[0]);
+
+                if ($is_done_all_calendar) {
+                    $locked   = $attendance[4];
+                    if ($locked == 0) {
+                        if (api_is_platform_admin()) {
+                            $message_alert = get_lang('AreYouSureToLockTheAttendance');
+                        } else {
+                            $message_alert = get_lang('UnlockMessageInformation');
+                        }
+                        $actions .= '&nbsp;<a onclick="javascript:if(!confirm(\''.$message_alert.'\')) return false;" href="index.php?'.api_get_cidreq().'&action=lock_attendance&attendance_id='.$attendance[0].'">'.
                             Display::return_icon('unlock.png', get_lang('LockAttendance')).'</a>';
-					} else {
-						if (api_is_platform_admin()) {
-							$actions .= '&nbsp;<a onclick="javascript:if(!confirm(\''.get_lang('AreYouSureToUnlockTheAttendance').'\')) return false;" href="index.php?'.api_get_cidreq().'&action=unlock_attendance&attendance_id='.$attendance[0].'">'.
-									Display::return_icon('locked.png', get_lang('UnlockAttendance')).'</a>';
-						} else {
-							$actions .= '&nbsp;'.Display::return_icon('locked_na.png', get_lang('LockedAttendance'));
-						}
-					}
-				}
-				$actions .= '</center>';
-
-				$attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3],$actions);
-			} else {
-				$attendance[0] = '&nbsp;';
-				$attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3]);
-			}
-		}
-		return $attendances;
-	}
-
-	/**
-	 * Get the attendances by id to display on the current page
-	 * @param  int     $attendance_id
-	 * @return array   attendance data
-	 */
-	public function get_attendance_by_id($attendance_id)
-	{
-		$tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-		$attendance_data = array();
-		$sql = "SELECT * FROM $tbl_attendance
+                    } else {
+                        if (api_is_platform_admin()) {
+                            $actions .= '&nbsp;<a onclick="javascript:if(!confirm(\''.get_lang('AreYouSureToUnlockTheAttendance').'\')) return false;" href="index.php?'.api_get_cidreq().'&action=unlock_attendance&attendance_id='.$attendance[0].'">'.
+                                    Display::return_icon('locked.png', get_lang('UnlockAttendance')).'</a>';
+                        } else {
+                            $actions .= '&nbsp;'.Display::return_icon('locked_na.png', get_lang('LockedAttendance'));
+                        }
+                    }
+                }
+                $actions .= '</center>';
+
+                $attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3],$actions);
+            } else {
+                $attendance[0] = '&nbsp;';
+                $attendances[] = array($attendance[0], $attendance[1], $attendance[2], $attendance[3]);
+            }
+        }
+        return $attendances;
+    }
+
+    /**
+     * Get the attendances by id to display on the current page
+     * @param  int     $attendance_id
+     * @return array   attendance data
+     */
+    public function get_attendance_by_id($attendance_id)
+    {
+        $tbl_attendance = Database :: get_course_table(TABLE_ATTENDANCE);
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+        $attendance_data = array();
+        $sql = "SELECT * FROM $tbl_attendance
 				WHERE c_id = $course_id AND id = '$attendance_id'";
-		$res = Database::query($sql);
-		if (Database::num_rows($res) > 0) {
-			while ($row = Database::fetch_array($res)) {
-				$attendance_data = $row;
-			}
-		}
-		return $attendance_data;
-	}
-
-	/**
-	 * Add attendances sheet inside table. This is the *list of* dates, not
-	 * a specific date in itself.
-	 * @param  bool   true for adding link in gradebook or false otherwise (optional)
-	 * @return int    last attendance id
-	 */
-	public function attendance_add($link_to_gradebook = false)
-	{
-		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
-		$table_link = Database:: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
-		$session_id = api_get_session_id();
-		$user_id = api_get_user_id();
-		$course_code = $_course['code'];
-		$course_id = $_course['real_id'];
-		$title_gradebook= $this->attendance_qualify_title;
-		$value_calification = 0;
-		$weight_calification = floatval($this->attendance_weight);
-
-		$params = [
-			'c_id' => $course_id,
-			'name' => $this->name,
-			'description' => $this->description,
-			'attendance_qualify_title' => $title_gradebook,
-			'attendance_weight' => $weight_calification,
-			'session_id' => $session_id,
-			'active' => 1,
-			'attendance_qualify_max' => 0,
-			'locked' => 0
-		];
-		$last_id = Database::insert($tbl_attendance, $params);
-
-		if (!empty($last_id)) {
-
-			$sql = "UPDATE $tbl_attendance SET id = iid WHERE iid = $last_id";
-			Database::query($sql);
-
-			api_item_property_update(
-				$_course,
-				TOOL_ATTENDANCE,
-				$last_id,
-				"AttendanceAdded",
-				$user_id
-			);
-
-		}
-		// add link to gradebook
-		if ($link_to_gradebook && !empty($this->category_id)) {
-			$description = '';
+        $res = Database::query($sql);
+        if (Database::num_rows($res) > 0) {
+            while ($row = Database::fetch_array($res)) {
+                $attendance_data = $row;
+            }
+        }
+        return $attendance_data;
+    }
+
+    /**
+     * Add attendances sheet inside table. This is the *list of* dates, not
+     * a specific date in itself.
+     * @param  bool   true for adding link in gradebook or false otherwise (optional)
+     * @return int    last attendance id
+     */
+    public function attendance_add($link_to_gradebook = false)
+    {
+        $_course = api_get_course_info();
+        $tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+        $table_link = Database:: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
+        $session_id = api_get_session_id();
+        $user_id = api_get_user_id();
+        $course_code = $_course['code'];
+        $course_id = $_course['real_id'];
+        $title_gradebook= $this->attendance_qualify_title;
+        $value_calification = 0;
+        $weight_calification = floatval($this->attendance_weight);
+
+        $params = [
+            'c_id' => $course_id,
+            'name' => $this->name,
+            'description' => $this->description,
+            'attendance_qualify_title' => $title_gradebook,
+            'attendance_weight' => $weight_calification,
+            'session_id' => $session_id,
+            'active' => 1,
+            'attendance_qualify_max' => 0,
+            'locked' => 0
+        ];
+        $last_id = Database::insert($tbl_attendance, $params);
+
+        if (!empty($last_id)) {
+
+            $sql = "UPDATE $tbl_attendance SET id = iid WHERE iid = $last_id";
+            Database::query($sql);
+
+            api_item_property_update(
+                $_course,
+                TOOL_ATTENDANCE,
+                $last_id,
+                "AttendanceAdded",
+                $user_id
+            );
+
+        }
+        // add link to gradebook
+        if ($link_to_gradebook && !empty($this->category_id)) {
+            $description = '';
             $link_info = GradebookUtils::is_resource_in_course_gradebook(
                 $course_code,
                 7,
                 $last_id,
                 $session_id
             );
-			$link_id = $link_info['id'];
-			if (!$link_info) {
-				GradebookUtils::add_resource_to_course_gradebook(
-					$this->category_id,
-					$course_code,
-					7,
-					$last_id,
-					$title_gradebook,
-					$weight_calification,
-					$value_calification,
-					$description,
-					1,
-					$session_id
-				);
-			} else {
-				Database::query('UPDATE '.$table_link.' SET weight='.$weight_calification.' WHERE id='.$link_id.'');
-			}
-		}
-		return $last_id;
-	}
-
-	/**
-	 * edit attendances inside table
-	 * @param 	int	   attendance id
-	 * @param  	bool   true for adding link in gradebook or false otherwise (optional)
-	 * @return 	int    last id
-	 */
-	public function attendance_edit($attendance_id, $link_to_gradebook = false)
-	{
-		$_course = api_get_course_info();
-		$tbl_attendance = Database:: get_course_table(TABLE_ATTENDANCE);
-		$table_link = Database:: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
-
-		$session_id = api_get_session_id();
-		$user_id = api_get_user_id();
-		$attendance_id = intval($attendance_id);
-		$course_code = $_course['code'];
-		$course_id = $_course['real_id'];
-		$title_gradebook = $this->attendance_qualify_title;
-		$value_calification = 0;
-		$weight_calification = floatval($this->attendance_weight);
-
-		if (!empty($attendance_id)) {
-
-			$params = [
-				'name' => $this->name,
-				'description' => $this->description,
-				'attendance_qualify_title' => $title_gradebook,
-				'attendance_weight' => $weight_calification
-			];
-			Database::update(
-				$tbl_attendance,
-				$params,
-				['c_id = ? AND id = ?' => [$course_id, $attendance_id]]
-			);
-
-			api_item_property_update(
-				$_course,
-				TOOL_ATTENDANCE,
-				$attendance_id,
-				"AttendanceUpdated",
-				$user_id
-			);
-
-			// add link to gradebook
-			if ($link_to_gradebook && !empty($this->category_id)) {
-				$description = '';
-				$link_info = GradebookUtils::is_resource_in_course_gradebook(
-					$course_code,
-					7,
-					$attendance_id,
-					$session_id
-				);
-				if (!$link_info) {
-					GradebookUtils::add_resource_to_course_gradebook(
-						$this->category_id,
-						$course_code,
-						7,
-						$attendance_id,
-						$title_gradebook,
-						$weight_calification,
-						$value_calification,
-						$description,
-						1,
-						$session_id
-					);
-				} else {
-					Database::query('UPDATE '.$table_link.' SET weight='.$weight_calification.' WHERE id='.$link_info['id'].'');
-				}
-			}
-			return $attendance_id;
-		}
-		return null;
-	}
-
-	/**
-	 * Restore attendance
-	 * @param 	int|array	   one or many attendances id
-	 * @return 	int    		   affected rows
-	 */
-	public function attendance_restore($attendance_id)
-	{
-		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
-		$user_id = api_get_user_id();
-		$course_id = $_course['real_id'];
-		if (is_array($attendance_id)) {
-			foreach ($attendance_id as $id) {
-				$id	= intval($id);
-				$sql = "UPDATE $tbl_attendance SET active = 1
+            $link_id = $link_info['id'];
+            if (!$link_info) {
+                GradebookUtils::add_resource_to_course_gradebook(
+                    $this->category_id,
+                    $course_code,
+                    7,
+                    $last_id,
+                    $title_gradebook,
+                    $weight_calification,
+                    $value_calification,
+                    $description,
+                    1,
+                    $session_id
+                );
+            } else {
+                Database::query('UPDATE '.$table_link.' SET weight='.$weight_calification.' WHERE id='.$link_id.'');
+            }
+        }
+        return $last_id;
+    }
+
+    /**
+     * edit attendances inside table
+     * @param 	int	   attendance id
+     * @param  	bool   true for adding link in gradebook or false otherwise (optional)
+     * @return 	int    last id
+     */
+    public function attendance_edit($attendance_id, $link_to_gradebook = false)
+    {
+        $_course = api_get_course_info();
+        $tbl_attendance = Database:: get_course_table(TABLE_ATTENDANCE);
+        $table_link = Database:: get_main_table(TABLE_MAIN_GRADEBOOK_LINK);
+
+        $session_id = api_get_session_id();
+        $user_id = api_get_user_id();
+        $attendance_id = intval($attendance_id);
+        $course_code = $_course['code'];
+        $course_id = $_course['real_id'];
+        $title_gradebook = $this->attendance_qualify_title;
+        $value_calification = 0;
+        $weight_calification = floatval($this->attendance_weight);
+
+        if (!empty($attendance_id)) {
+
+            $params = [
+                'name' => $this->name,
+                'description' => $this->description,
+                'attendance_qualify_title' => $title_gradebook,
+                'attendance_weight' => $weight_calification
+            ];
+            Database::update(
+                $tbl_attendance,
+                $params,
+                ['c_id = ? AND id = ?' => [$course_id, $attendance_id]]
+            );
+
+            api_item_property_update(
+                $_course,
+                TOOL_ATTENDANCE,
+                $attendance_id,
+                "AttendanceUpdated",
+                $user_id
+            );
+
+            // add link to gradebook
+            if ($link_to_gradebook && !empty($this->category_id)) {
+                $description = '';
+                $link_info = GradebookUtils::is_resource_in_course_gradebook(
+                    $course_code,
+                    7,
+                    $attendance_id,
+                    $session_id
+                );
+                if (!$link_info) {
+                    GradebookUtils::add_resource_to_course_gradebook(
+                        $this->category_id,
+                        $course_code,
+                        7,
+                        $attendance_id,
+                        $title_gradebook,
+                        $weight_calification,
+                        $value_calification,
+                        $description,
+                        1,
+                        $session_id
+                    );
+                } else {
+                    Database::query('UPDATE '.$table_link.' SET weight='.$weight_calification.' WHERE id='.$link_info['id'].'');
+                }
+            }
+            return $attendance_id;
+        }
+        return null;
+    }
+
+    /**
+     * Restore attendance
+     * @param 	int|array	   one or many attendances id
+     * @return 	int    		   affected rows
+     */
+    public function attendance_restore($attendance_id)
+    {
+        $_course = api_get_course_info();
+        $tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+        $user_id = api_get_user_id();
+        $course_id = $_course['real_id'];
+        if (is_array($attendance_id)) {
+            foreach ($attendance_id as $id) {
+                $id	= intval($id);
+                $sql = "UPDATE $tbl_attendance SET active = 1
 						WHERE c_id = $course_id AND id = '$id'";
-				$result = Database::query($sql);
-				$affected_rows = Database::affected_rows($result);
-				if (!empty($affected_rows)) {
-					// update row item property table
+                $result = Database::query($sql);
+                $affected_rows = Database::affected_rows($result);
+                if (!empty($affected_rows)) {
+                    // update row item property table
                     api_item_property_update(
                         $_course,
                         TOOL_ATTENDANCE,
@@ -453,16 +453,16 @@ 
                         "restore",
                         $user_id
                     );
-				}
-			}
-		} else  {
-			$attendance_id	= intval($attendance_id);
-			$sql = "UPDATE $tbl_attendance SET active = 1
+                }
+            }
+        } else  {
+            $attendance_id	= intval($attendance_id);
+            $sql = "UPDATE $tbl_attendance SET active = 1
 					WHERE c_id = $course_id AND id = '$attendance_id'";
-			$result = Database::query($sql);
-			$affected_rows = Database::affected_rows($result);
-			if (!empty($affected_rows)) {
-				// update row item property table
+            $result = Database::query($sql);
+            $affected_rows = Database::affected_rows($result);
+            if (!empty($affected_rows)) {
+                // update row item property table
                 api_item_property_update(
                     $_course,
                     TOOL_ATTENDANCE,
@@ -470,33 +470,33 @@ 
                     "restore",
                     $user_id
                 );
-			}
-		}
-
-		return $affected_rows;
-	}
-
-	/**
-	 * Delete attendances
-	 * @param 	int|array	$attendance_id   one or many attendances id
-	 * @return 	int    		   affected rows
-	 */
-	public function attendance_delete($attendance_id)
-	{
-		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+            }
+        }
+
+        return $affected_rows;
+    }
+
+    /**
+     * Delete attendances
+     * @param 	int|array	$attendance_id   one or many attendances id
+     * @return 	int    		   affected rows
+     */
+    public function attendance_delete($attendance_id)
+    {
+        $_course = api_get_course_info();
+        $tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
         $user_id = api_get_user_id();
         $course_id = $_course['real_id'];
 
-		if (is_array($attendance_id)) {
-			foreach ($attendance_id as $id) {
-				$id	= intval($id);
-				$sql = "UPDATE $tbl_attendance SET active = 2
+        if (is_array($attendance_id)) {
+            foreach ($attendance_id as $id) {
+                $id	= intval($id);
+                $sql = "UPDATE $tbl_attendance SET active = 2
 						WHERE c_id = $course_id AND id = '$id'";
-				$result = Database::query($sql);
-				$affected_rows = Database::affected_rows($result);
-				if (!empty($affected_rows)) {
-					// update row item property table
+                $result = Database::query($sql);
+                $affected_rows = Database::affected_rows($result);
+                if (!empty($affected_rows)) {
+                    // update row item property table
                     api_item_property_update(
                         $_course,
                         TOOL_ATTENDANCE,
@@ -504,142 +504,142 @@ 
                         "delete",
                         $user_id
                     );
-				}
-			}
-		} else  {
-			$attendance_id= intval($attendance_id);
-			$sql = "UPDATE $tbl_attendance SET active = 2
+                }
+            }
+        } else  {
+            $attendance_id= intval($attendance_id);
+            $sql = "UPDATE $tbl_attendance SET active = 2
 					WHERE c_id = $course_id AND id = '$attendance_id'";
 
-			$result = Database::query($sql);
-			$affected_rows = Database::affected_rows($result);
-			if (!empty($affected_rows)) {
-				// update row item property table
-				api_item_property_update(
-					$_course,
-					TOOL_ATTENDANCE,
-					$attendance_id,
-					"delete",
-					$user_id
-				);
-			}
-		}
-
-		return $affected_rows;
-	}
-
-	/**
-	 * Changes visibility
-	 * @param int|array	$attendanceId   one or many attendances id
-	 * @param int status
+            $result = Database::query($sql);
+            $affected_rows = Database::affected_rows($result);
+            if (!empty($affected_rows)) {
+                // update row item property table
+                api_item_property_update(
+                    $_course,
+                    TOOL_ATTENDANCE,
+                    $attendance_id,
+                    "delete",
+                    $user_id
+                );
+            }
+        }
+
+        return $affected_rows;
+    }
+
+    /**
+     * Changes visibility
+     * @param int|array	$attendanceId   one or many attendances id
+     * @param int status
      *
-	 * @return 	int affected rows
-	 */
-	public function changeVisibility($attendanceId, $status = 1)
-	{
-		$_course = api_get_course_info();
-		$tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
-		$user_id = api_get_user_id();
-		$course_id = $_course['real_id'];
-		$status = intval($status);
-
-		$action = 'visible';
-		if ($status == 0) {
-			$action = 'invisible';
-		}
-
-		if (is_array($attendanceId)) {
-			foreach ($attendanceId as $id) {
-				$id	= intval($id);
-				$sql = "UPDATE $tbl_attendance SET active = $status
+     * @return 	int affected rows
+     */
+    public function changeVisibility($attendanceId, $status = 1)
+    {
+        $_course = api_get_course_info();
+        $tbl_attendance	= Database :: get_course_table(TABLE_ATTENDANCE);
+        $user_id = api_get_user_id();
+        $course_id = $_course['real_id'];
+        $status = intval($status);
+
+        $action = 'visible';
+        if ($status == 0) {
+            $action = 'invisible';
+        }
+
+        if (is_array($attendanceId)) {
+            foreach ($attendanceId as $id) {
+                $id	= intval($id);
+                $sql = "UPDATE $tbl_attendance SET active = $status
 						WHERE c_id = $course_id AND id = '$id'";
-				$result = Database::query($sql);
-				$affected_rows = Database::affected_rows($result);
-				if (!empty($affected_rows)) {
-					// update row item property table
-					api_item_property_update($_course, TOOL_ATTENDANCE, $id, $action, $user_id);
-				}
-			}
-		} else  {
-			$attendanceId	= intval($attendanceId);
-			$sql = "UPDATE $tbl_attendance SET active = $status
+                $result = Database::query($sql);
+                $affected_rows = Database::affected_rows($result);
+                if (!empty($affected_rows)) {
+                    // update row item property table
+                    api_item_property_update($_course, TOOL_ATTENDANCE, $id, $action, $user_id);
+                }
+            }
+        } else  {
+            $attendanceId	= intval($attendanceId);
+            $sql = "UPDATE $tbl_attendance SET active = $status
 					WHERE c_id = $course_id AND id = '$attendanceId'";
-			$result = Database::query($sql);
-			$affected_rows = Database::affected_rows($result);
-			if (!empty($affected_rows)) {
-				// update row item property table
-				api_item_property_update(
-					$_course,
-					TOOL_ATTENDANCE,
-					$attendanceId,
-					$action,
-					$user_id
-				);
-			}
-		}
-
-		return $affected_rows;
-	}
-
-	/**
-	 * Lock or unlock an attendance
-	 * @param   int     attendance id
-	 * @param   bool    True to lock or false otherwise
-	 */
-	public function lock_attendance($attendance_id, $lock = true)
-	{
-		$tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
-		$course_id = api_get_course_int_id();
-		$attendance_id = intval($attendance_id);
-		$locked = ($lock)?1:0;
-		$upd = "UPDATE $tbl_attendance SET locked = $locked
+            $result = Database::query($sql);
+            $affected_rows = Database::affected_rows($result);
+            if (!empty($affected_rows)) {
+                // update row item property table
+                api_item_property_update(
+                    $_course,
+                    TOOL_ATTENDANCE,
+                    $attendanceId,
+                    $action,
+                    $user_id
+                );
+            }
+        }
+
+        return $affected_rows;
+    }
+
+    /**
+     * Lock or unlock an attendance
+     * @param   int     attendance id
+     * @param   bool    True to lock or false otherwise
+     */
+    public function lock_attendance($attendance_id, $lock = true)
+    {
+        $tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
+        $course_id = api_get_course_int_id();
+        $attendance_id = intval($attendance_id);
+        $locked = ($lock)?1:0;
+        $upd = "UPDATE $tbl_attendance SET locked = $locked
 				WHERE c_id = $course_id AND id = $attendance_id";
-		$result = Database::query($upd);
-		$affected_rows = Database::affected_rows($result);
-		if ($affected_rows && $lock) {
-			// Save attendance sheet log
-			$lastedit_date = api_get_utc_datetime();
-			$lastedit_type = self::LOCKED_ATTENDANCE_LOG_TYPE;
-			$lastedit_user_id = api_get_user_id();
-			$this->save_attendance_sheet_log(
-				$attendance_id,
-				$lastedit_date,
-				$lastedit_type,
-				$lastedit_user_id
-			);
-		}
-		return $affected_rows;
-	}
-
-	/**
-	 * Get registered users inside current course
-	 * @param 	int	   $attendance_id attendance id for showing attendance result field (optional)
-	 * @param int $groupId
-	 * @return 	array  users data
-	 */
-	public function get_users_rel_course($attendance_id = 0, $groupId = null)
-	{
-		$current_session_id = api_get_session_id();
-		$current_course_id  = api_get_course_id();
-		$currentCourseIntId = api_get_course_int_id();
-
-		$studentInGroup = array();
-
-		if (!empty($current_session_id)) {
-			$a_course_users = CourseManager:: get_user_list_from_course_code(
-				$current_course_id,
-				$current_session_id,
-				'',
-				'lastname'
-			);
-		} else {
-			$a_course_users = CourseManager:: get_user_list_from_course_code(
-				$current_course_id,
-				0,
-				'',
-				'lastname'
-			);
-		}
+        $result = Database::query($upd);
+        $affected_rows = Database::affected_rows($result);
+        if ($affected_rows && $lock) {
+            // Save attendance sheet log
+            $lastedit_date = api_get_utc_datetime();
+            $lastedit_type = self::LOCKED_ATTENDANCE_LOG_TYPE;
+            $lastedit_user_id = api_get_user_id();
+            $this->save_attendance_sheet_log(
+                $attendance_id,
+                $lastedit_date,
+                $lastedit_type,
+                $lastedit_user_id
+            );
+        }
+        return $affected_rows;
+    }
+
+    /**
+     * Get registered users inside current course
+     * @param 	int	   $attendance_id attendance id for showing attendance result field (optional)
+     * @param int $groupId
+     * @return 	array  users data
+     */
+    public function get_users_rel_course($attendance_id = 0, $groupId = null)
+    {
+        $current_session_id = api_get_session_id();
+        $current_course_id  = api_get_course_id();
+        $currentCourseIntId = api_get_course_int_id();
+
+        $studentInGroup = array();
+
+        if (!empty($current_session_id)) {
+            $a_course_users = CourseManager:: get_user_list_from_course_code(
+                $current_course_id,
+                $current_session_id,
+                '',
+                'lastname'
+            );
+        } else {
+            $a_course_users = CourseManager:: get_user_list_from_course_code(
+                $current_course_id,
+                0,
+                '',
+                'lastname'
+            );
+        }
 
         if (!empty($groupId)) {
             $students = GroupManager::getStudents($groupId);
@@ -650,192 +650,192 @@ 
             }
         }
 
-		// get registered users inside current course
-		$a_users = array();
-		foreach ($a_course_users as $key => $user_data) {
-			$value = array();
-			$uid = $user_data['user_id'];
-			$userInfo = api_get_user_info($uid);
-
-			$status = $user_data['status'];
-
-			if (!empty($groupId)) {
-				if (!isset($studentInGroup[$uid])) {
-					continue;
-				}
-			}
-
-			$user_status_in_session = null;
-			$user_status_in_course  = null;
-
-			if (api_get_session_id()) {
-				$user_status_in_session = SessionManager::get_user_status_in_course_session(
-					$uid,
-					$currentCourseIntId,
-					$current_session_id
-				);
-			} else {
-				$user_status_in_course = CourseManager::get_user_in_course_status(
-					$uid,
-					$current_course_id
-				);
-			}
-
-			// Not taking into account DRH or COURSEMANAGER
-			if ($uid <= 1 ||
-				$status == DRH ||
-				$user_status_in_course == COURSEMANAGER ||
-				$user_status_in_session == 2
-			) {
-				continue;
-			}
-
-			if (!empty($attendance_id)) {
-				$user_faults = $this->get_faults_of_user($uid, $attendance_id, $groupId);
-				$value['attendance_result'] = $user_faults['faults'].'/'.$user_faults['total'].' ('.$user_faults['faults_porcent'].'%)';
-				$value['result_color_bar'] 	= $user_faults['color_bar'];
-			}
+        // get registered users inside current course
+        $a_users = array();
+        foreach ($a_course_users as $key => $user_data) {
+            $value = array();
+            $uid = $user_data['user_id'];
+            $userInfo = api_get_user_info($uid);
+
+            $status = $user_data['status'];
+
+            if (!empty($groupId)) {
+                if (!isset($studentInGroup[$uid])) {
+                    continue;
+                }
+            }
+
+            $user_status_in_session = null;
+            $user_status_in_course  = null;
+
+            if (api_get_session_id()) {
+                $user_status_in_session = SessionManager::get_user_status_in_course_session(
+                    $uid,
+                    $currentCourseIntId,
+                    $current_session_id
+                );
+            } else {
+                $user_status_in_course = CourseManager::get_user_in_course_status(
+                    $uid,
+                    $current_course_id
+                );
+            }
+
+            // Not taking into account DRH or COURSEMANAGER
+            if ($uid <= 1 ||
+                $status == DRH ||
+                $user_status_in_course == COURSEMANAGER ||
+                $user_status_in_session == 2
+            ) {
+                continue;
+            }
+
+            if (!empty($attendance_id)) {
+                $user_faults = $this->get_faults_of_user($uid, $attendance_id, $groupId);
+                $value['attendance_result'] = $user_faults['faults'].'/'.$user_faults['total'].' ('.$user_faults['faults_porcent'].'%)';
+                $value['result_color_bar'] 	= $user_faults['color_bar'];
+            }
 
             $photo = Display::img($userInfo['avatar_small'], $userInfo['complete_name'], [], false);
 
-			$value['photo'] = $photo;
-			$value['firstname'] = $user_data['firstname'];
-			$value['lastname'] = $user_data['lastname'];
-			$value['username'] = $user_data['username'];
-			$value['user_id'] = $uid;
-
-			//Sending only 5 items in the array instead of 60
-			$a_users[$key] = $value;
-		}
-
-		return $a_users;
-	}
-
-	/**
-	 * add attendances sheet inside table
-	 * @param 	int	   $calendar_id attendance calendar id
-	 * @param  	array  $users_present present users during current class
-	 * @param	int	   $attendance_id
-	 * @return 	int    affected rows
-	 */
-	public function attendance_sheet_add($calendar_id, $users_present, $attendance_id)
-	{
-		$tbl_attendance_sheet 	= Database::get_course_table(TABLE_ATTENDANCE_SHEET);
-		$tbl_attendance_calendar= Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-
-		$calendar_id = intval($calendar_id);
-		$attendance_id = intval($attendance_id);
-		$users = $this->get_users_rel_course();
-		$course_id = api_get_course_int_id();
-
-		$user_ids = array_keys($users);
-		$users_absent = array_diff($user_ids,$users_present);
-		$affected_rows = 0;
-
-		// get last edit type
-		$calendar_data = $this->get_attendance_calendar_by_id($calendar_id);
-		$lastedit_type = self::DONE_ATTENDANCE_LOG_TYPE;
-		if ($calendar_data['done_attendance']) {
-			$lastedit_type = self::UPDATED_ATTENDANCE_LOG_TYPE;
-		}
-
-		// save users present in class
-		foreach ($users_present as $user_present) {
-			$uid = intval($user_present);
-			// check if user already was registered with the $calendar_id
-			$sql = "SELECT user_id FROM $tbl_attendance_sheet
+            $value['photo'] = $photo;
+            $value['firstname'] = $user_data['firstname'];
+            $value['lastname'] = $user_data['lastname'];
+            $value['username'] = $user_data['username'];
+            $value['user_id'] = $uid;
+
+            //Sending only 5 items in the array instead of 60
+            $a_users[$key] = $value;
+        }
+
+        return $a_users;
+    }
+
+    /**
+     * add attendances sheet inside table
+     * @param 	int	   $calendar_id attendance calendar id
+     * @param  	array  $users_present present users during current class
+     * @param	int	   $attendance_id
+     * @return 	int    affected rows
+     */
+    public function attendance_sheet_add($calendar_id, $users_present, $attendance_id)
+    {
+        $tbl_attendance_sheet 	= Database::get_course_table(TABLE_ATTENDANCE_SHEET);
+        $tbl_attendance_calendar= Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+
+        $calendar_id = intval($calendar_id);
+        $attendance_id = intval($attendance_id);
+        $users = $this->get_users_rel_course();
+        $course_id = api_get_course_int_id();
+
+        $user_ids = array_keys($users);
+        $users_absent = array_diff($user_ids,$users_present);
+        $affected_rows = 0;
+
+        // get last edit type
+        $calendar_data = $this->get_attendance_calendar_by_id($calendar_id);
+        $lastedit_type = self::DONE_ATTENDANCE_LOG_TYPE;
+        if ($calendar_data['done_attendance']) {
+            $lastedit_type = self::UPDATED_ATTENDANCE_LOG_TYPE;
+        }
+
+        // save users present in class
+        foreach ($users_present as $user_present) {
+            $uid = intval($user_present);
+            // check if user already was registered with the $calendar_id
+            $sql = "SELECT user_id FROM $tbl_attendance_sheet
 					WHERE c_id = $course_id AND user_id='$uid' AND attendance_calendar_id = '$calendar_id'";
-			$rs  = Database::query($sql);
-			if (Database::num_rows($rs) == 0) {
-				$sql = "INSERT INTO $tbl_attendance_sheet SET
+            $rs  = Database::query($sql);
+            if (Database::num_rows($rs) == 0) {
+                $sql = "INSERT INTO $tbl_attendance_sheet SET
 						c_id					= $course_id,
 						user_id 				= '$uid',
 						attendance_calendar_id 	= '$calendar_id',
 						presence 				= 1";
-				$result = Database::query($sql);
+                $result = Database::query($sql);
 
-				$affected_rows += Database::affected_rows($result);
-			} else {
-				$sql = "UPDATE $tbl_attendance_sheet SET presence = 1
+                $affected_rows += Database::affected_rows($result);
+            } else {
+                $sql = "UPDATE $tbl_attendance_sheet SET presence = 1
 						WHERE
 							c_id = $course_id AND
 							user_id ='$uid' AND
 							attendance_calendar_id = '$calendar_id'
 						";
-				$result = Database::query($sql);
-				$affected_rows += Database::affected_rows($result);
-			}
-		}
-
-		// save users absent in class
-		foreach ($users_absent as $user_absent) {
-			$uid = intval($user_absent);
-			// check if user already was registered with the $calendar_id
-			$sql = "SELECT user_id FROM $tbl_attendance_sheet
+                $result = Database::query($sql);
+                $affected_rows += Database::affected_rows($result);
+            }
+        }
+
+        // save users absent in class
+        foreach ($users_absent as $user_absent) {
+            $uid = intval($user_absent);
+            // check if user already was registered with the $calendar_id
+            $sql = "SELECT user_id FROM $tbl_attendance_sheet
 					WHERE c_id = $course_id AND user_id='$uid' AND attendance_calendar_id = '$calendar_id'";
-			$rs  = Database::query($sql);
-			if (Database::num_rows($rs) == 0) {
-				$sql = "INSERT INTO $tbl_attendance_sheet SET
+            $rs  = Database::query($sql);
+            if (Database::num_rows($rs) == 0) {
+                $sql = "INSERT INTO $tbl_attendance_sheet SET
 						c_id = $course_id,
 						user_id ='$uid',
 						attendance_calendar_id = '$calendar_id',
 						presence = 0";
-				$result = Database::query($sql);
+                $result = Database::query($sql);
 
-				Database::insert_id();
+                Database::insert_id();
 
-				$affected_rows += Database::affected_rows($result);
-			} else {
-				$sql = "UPDATE $tbl_attendance_sheet SET presence = 0
+                $affected_rows += Database::affected_rows($result);
+            } else {
+                $sql = "UPDATE $tbl_attendance_sheet SET presence = 0
 						WHERE
 							c_id = $course_id AND
 							user_id ='$uid' AND
 							attendance_calendar_id = '$calendar_id'";
-				$result = Database::query($sql);
-				$affected_rows += Database::affected_rows($result);
-			}
-		}
+                $result = Database::query($sql);
+                $affected_rows += Database::affected_rows($result);
+            }
+        }
 
-		// update done_attendance inside attendance calendar table
-		$sql = "UPDATE $tbl_attendance_calendar SET done_attendance = 1
+        // update done_attendance inside attendance calendar table
+        $sql = "UPDATE $tbl_attendance_calendar SET done_attendance = 1
 				WHERE  c_id = $course_id AND id = '$calendar_id'";
-		Database::query($sql);
-
-		// save users' results
-		$this->update_users_results($user_ids, $attendance_id);
-
-		if ($affected_rows) {
-			//save attendance sheet log
-			$lastedit_date = api_get_utc_datetime();
-			$lastedit_user_id = api_get_user_id();
-			$calendar_date_value = $calendar_data['date_time'];
-			$this->save_attendance_sheet_log(
-				$attendance_id,
-				$lastedit_date,
-				$lastedit_type,
-				$lastedit_user_id,
-				$calendar_date_value
-			);
-		}
-
-		return $affected_rows;
-	}
-
-	/**
-	 * update users' attendance results
-	 * @param 	array  $user_ids registered users inside current course
-	 * @param	int	   $attendance_id
-	 * @return 	void
-	 */
-	public function update_users_results($user_ids, $attendance_id)
-	{
-		$tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
-		$tbl_attendance_result = Database::get_course_table(TABLE_ATTENDANCE_RESULT);
-		$tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
-		$course_id = api_get_course_int_id();
-
-		$attendance_id = intval($attendance_id);
-		// fill results about presence of students
+        Database::query($sql);
+
+        // save users' results
+        $this->update_users_results($user_ids, $attendance_id);
+
+        if ($affected_rows) {
+            //save attendance sheet log
+            $lastedit_date = api_get_utc_datetime();
+            $lastedit_user_id = api_get_user_id();
+            $calendar_date_value = $calendar_data['date_time'];
+            $this->save_attendance_sheet_log(
+                $attendance_id,
+                $lastedit_date,
+                $lastedit_type,
+                $lastedit_user_id,
+                $calendar_date_value
+            );
+        }
+
+        return $affected_rows;
+    }
+
+    /**
+     * update users' attendance results
+     * @param 	array  $user_ids registered users inside current course
+     * @param	int	   $attendance_id
+     * @return 	void
+     */
+    public function update_users_results($user_ids, $attendance_id)
+    {
+        $tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
+        $tbl_attendance_result = Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+        $tbl_attendance = Database::get_course_table(TABLE_ATTENDANCE);
+        $course_id = api_get_course_int_id();
+
+        $attendance_id = intval($attendance_id);
+        // fill results about presence of students
         $attendance_calendar = $this->get_attendance_calendar(
             $attendance_id,
             'all',
@@ -843,82 +843,82 @@ 
             null,
             true
         );
-		$calendar_ids = array();
-		// get all dates from calendar by current attendance
-		foreach ($attendance_calendar as $cal) {
-			$calendar_ids[] = $cal['id'];
-		}
-
-		// get count of presences by users inside current attendance and save like results
-		if (count($user_ids) > 0) {
-			foreach ($user_ids as $uid) {
-				$count_presences = 0;
-				if (count($calendar_ids) > 0) {
-					$sql = "SELECT count(presence) as count_presences
+        $calendar_ids = array();
+        // get all dates from calendar by current attendance
+        foreach ($attendance_calendar as $cal) {
+            $calendar_ids[] = $cal['id'];
+        }
+
+        // get count of presences by users inside current attendance and save like results
+        if (count($user_ids) > 0) {
+            foreach ($user_ids as $uid) {
+                $count_presences = 0;
+                if (count($calendar_ids) > 0) {
+                    $sql = "SELECT count(presence) as count_presences
 							FROM $tbl_attendance_sheet
 					        WHERE
 					        	c_id = $course_id AND
 					        	user_id = '$uid' AND
 					        	attendance_calendar_id IN (".implode(',', $calendar_ids).") AND
 					        	presence = 1";
-					$rs_count  = Database::query($sql);
-					$row_count = Database::fetch_array($rs_count);
-					$count_presences = $row_count['count_presences'];
-				}
+                    $rs_count  = Database::query($sql);
+                    $row_count = Database::fetch_array($rs_count);
+                    $count_presences = $row_count['count_presences'];
+                }
 
-				// save results
-				$sql = "SELECT id FROM $tbl_attendance_result
+                // save results
+                $sql = "SELECT id FROM $tbl_attendance_result
 						WHERE
 							c_id = $course_id AND
 							user_id = '$uid' AND
 							attendance_id = '$attendance_id' ";
-				$rs_check_result = Database::query($sql);
+                $rs_check_result = Database::query($sql);
 
-				if (Database::num_rows($rs_check_result) > 0) {
-					// update result
-					$sql = "UPDATE $tbl_attendance_result SET
+                if (Database::num_rows($rs_check_result) > 0) {
+                    // update result
+                    $sql = "UPDATE $tbl_attendance_result SET
 							score = '$count_presences'
 							WHERE
 								c_id = $course_id AND
 								user_id='$uid' AND
 								attendance_id='$attendance_id'";
-					Database::query($sql);
-				} else {
-					// insert new result
-					$sql = "INSERT INTO $tbl_attendance_result SET
+                    Database::query($sql);
+                } else {
+                    // insert new result
+                    $sql = "INSERT INTO $tbl_attendance_result SET
 							c_id = $course_id ,
 							user_id			= '$uid',
 							attendance_id 	= '$attendance_id',
 							score			= '$count_presences'";
-					Database::query($sql);
+                    Database::query($sql);
 
-					$insertId = Database::insert_id();
+                    $insertId = Database::insert_id();
                     if ($insertId) {
                         $sql = "UPDATE $tbl_attendance_result SET id = iid WHERE iid = $insertId";
                         Database::query($sql);
                     }
-				}
-			}
-		}
+                }
+            }
+        }
 
-		// update attendance qualify max
-		$count_done_calendar = self::get_done_attendance_calendar($attendance_id);
+        // update attendance qualify max
+        $count_done_calendar = self::get_done_attendance_calendar($attendance_id);
 
-		$sql = "UPDATE $tbl_attendance SET
+        $sql = "UPDATE $tbl_attendance SET
 				attendance_qualify_max = '$count_done_calendar'
 				WHERE c_id = $course_id AND id = '$attendance_id'";
-		Database::query($sql);
-	}
-
-	/**
-	 * update attendance_sheet_log table, is used as history of an attendance sheet
-	 * @param   int     Attendance id
-	 * @param   string  Last edit datetime
-	 * @param   string  Event type ('locked_attendance', 'done_attendance_sheet' ...)
-	 * @param   int     Last edit user id
-	 * @param   string  Calendar datetime value (optional, when event type is 'done_attendance_sheet')
-	 * @return  int     Affected rows
-	 */
+        Database::query($sql);
+    }
+
+    /**
+     * update attendance_sheet_log table, is used as history of an attendance sheet
+     * @param   int     Attendance id
+     * @param   string  Last edit datetime
+     * @param   string  Event type ('locked_attendance', 'done_attendance_sheet' ...)
+     * @param   int     Last edit user id
+     * @param   string  Calendar datetime value (optional, when event type is 'done_attendance_sheet')
+     * @return  int     Affected rows
+     */
     public function save_attendance_sheet_log(
         $attendance_id,
         $lastedit_date,
@@ -926,22 +926,22 @@ 
         $lastedit_user_id,
         $calendar_date_value = null
     ) {
-		$course_id = api_get_course_int_id();
+        $course_id = api_get_course_int_id();
 
-		// define table
-		$tbl_attendance_sheet_log = Database::get_course_table(TABLE_ATTENDANCE_SHEET_LOG);
+        // define table
+        $tbl_attendance_sheet_log = Database::get_course_table(TABLE_ATTENDANCE_SHEET_LOG);
 
-		// protect data
-		$attendance_id = intval($attendance_id);
-		$lastedit_user_id = intval($lastedit_user_id);
+        // protect data
+        $attendance_id = intval($attendance_id);
+        $lastedit_user_id = intval($lastedit_user_id);
 
-		if (isset($calendar_date_value)) {
-			$calendar_date_value = $calendar_date_value;
-		} else {
-			$calendar_date_value = '';
-		}
+        if (isset($calendar_date_value)) {
+            $calendar_date_value = $calendar_date_value;
+        } else {
+            $calendar_date_value = '';
+        }
 
-		// save data
+        // save data
         $params = [
             'c_id' => $course_id,
             'attendance_id' => $attendance_id,
@@ -950,228 +950,228 @@ 
             'lastedit_user_id' => $lastedit_user_id,
             'calendar_date_value' => $calendar_date_value,
         ];
-		$insertId = Database::insert($tbl_attendance_sheet_log, $params);
+        $insertId = Database::insert($tbl_attendance_sheet_log, $params);
         if ($insertId) {
             $sql = "UPDATE $tbl_attendance_sheet_log SET id = iid WHERE iid = $insertId";
             Database::query($sql);
         }
 
-		return $insertId;
-	}
-
-	/**
-	 * Get number of done attendances inside current sheet
-	 * @param	int	   attendance id
-	 * @return 	int	   number of done attendances
-	 */
-	public static function get_done_attendance_calendar($attendance_id)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-		$sql = "SELECT count(done_attendance) as count
+        return $insertId;
+    }
+
+    /**
+     * Get number of done attendances inside current sheet
+     * @param	int	   attendance id
+     * @return 	int	   number of done attendances
+     */
+    public static function get_done_attendance_calendar($attendance_id)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+        $sql = "SELECT count(done_attendance) as count
                 FROM $tbl_attendance_calendar
                 WHERE
                     c_id = $course_id AND
                     attendance_id = '$attendance_id' AND
                     done_attendance = 1
                 ";
-		$rs  = Database::query($sql);
-		$row = Database::fetch_array($rs);
-		$count = $row['count'];
-		return $count;
-	}
-
-	/**
-	 * Get results of faults (absents) by user
-	 * @param int $user_id
-	 * @param int $attendance_id
-	 * @param int $groupId
-	 * @return 	array  results containing number of faults, total done attendance,
-	 * percent of faults and color depend on result (red, orange)
-	 */
-	public function get_faults_of_user($user_id, $attendance_id, $groupId = null)
-	{
-		// initializing database table and variables
-		$user_id 		= intval($user_id);
-		$attendance_id 	= intval($attendance_id);
-		$results = array();
-		$calendar_count = self::get_number_of_attendance_calendar($attendance_id, $groupId, NULL, $user_id);
-		// $total_done_attendance 	= $attendance_data['attendance_qualify_max'];
+        $rs  = Database::query($sql);
+        $row = Database::fetch_array($rs);
+        $count = $row['count'];
+        return $count;
+    }
+
+    /**
+     * Get results of faults (absents) by user
+     * @param int $user_id
+     * @param int $attendance_id
+     * @param int $groupId
+     * @return 	array  results containing number of faults, total done attendance,
+     * percent of faults and color depend on result (red, orange)
+     */
+    public function get_faults_of_user($user_id, $attendance_id, $groupId = null)
+    {
+        // initializing database table and variables
+        $user_id 		= intval($user_id);
+        $attendance_id 	= intval($attendance_id);
+        $results = array();
+        $calendar_count = self::get_number_of_attendance_calendar($attendance_id, $groupId, NULL, $user_id);
+        // $total_done_attendance 	= $attendance_data['attendance_qualify_max'];
         $total_done_attendance 	= self::get_number_of_attendance_calendar($attendance_id, $groupId, true, $user_id);
-		$attendance_user_score  = $this->get_user_score($user_id, $attendance_id, $groupId);
+        $attendance_user_score  = $this->get_user_score($user_id, $attendance_id, $groupId);
 
-		//This is the main change of the BT#1381
-		//$total_done_attendance = $calendar_count;
+        //This is the main change of the BT#1381
+        //$total_done_attendance = $calendar_count;
 
-		// calculate results
-		$faults = $total_done_attendance - $attendance_user_score;
+        // calculate results
+        $faults = $total_done_attendance - $attendance_user_score;
 
         if (empty($calendar_count)) {
             $faults = 0;
         }
 
         $faults = $faults > 0 ? $faults : 0;
-		$faults_porcent = $calendar_count > 0 ?round(($faults*100)/$calendar_count,0):0;
+        $faults_porcent = $calendar_count > 0 ?round(($faults*100)/$calendar_count,0):0;
         $results['faults'] = $faults;
         $results['total'] = $calendar_count;
         $results['faults_porcent'] = $faults_porcent;
-		$color_bar = '';
+        $color_bar = '';
 
         if ($faults_porcent > 25) {
             $color_bar = '#f28989';
-		} else if ($faults_porcent > 10) {
-			$color_bar = '#F90';
-		}
-		$results['color_bar'] = $color_bar;
-
-		return $results;
-	}
-
-	/**
-	 * Get results of faults average for all courses by user
-	 * @param	int	   $user_id
-	 * @return 	array  results containing number of faults, total done attendance,
-	 * percentage of faults and color depend on result (red, orange)
-	 */
-	public function get_faults_average_inside_courses($user_id)
-	{
-		// get all courses of current user
-		$courses = CourseManager::get_courses_list_by_user_id($user_id, true);
-
-		$user_id = intval($user_id);
-		$results = array();
-		$total_faults = $total_weight = $porcent = 0;
-		foreach ($courses as $course) {
-			//$course_code = $course['code'];
-			//$course_info = api_get_course_info($course_code);
-			$course_id = $course['real_id'];
-			$tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
-
-			$attendances_by_course = $this->get_attendances_list($course_id);
-
-			foreach ($attendances_by_course as $attendance) {
-				// get total faults and total weight
-				$total_done_attendance 	= $attendance['attendance_qualify_max'];
-				$sql = "SELECT score
+        } else if ($faults_porcent > 10) {
+            $color_bar = '#F90';
+        }
+        $results['color_bar'] = $color_bar;
+
+        return $results;
+    }
+
+    /**
+     * Get results of faults average for all courses by user
+     * @param	int	   $user_id
+     * @return 	array  results containing number of faults, total done attendance,
+     * percentage of faults and color depend on result (red, orange)
+     */
+    public function get_faults_average_inside_courses($user_id)
+    {
+        // get all courses of current user
+        $courses = CourseManager::get_courses_list_by_user_id($user_id, true);
+
+        $user_id = intval($user_id);
+        $results = array();
+        $total_faults = $total_weight = $porcent = 0;
+        foreach ($courses as $course) {
+            //$course_code = $course['code'];
+            //$course_info = api_get_course_info($course_code);
+            $course_id = $course['real_id'];
+            $tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+
+            $attendances_by_course = $this->get_attendances_list($course_id);
+
+            foreach ($attendances_by_course as $attendance) {
+                // get total faults and total weight
+                $total_done_attendance 	= $attendance['attendance_qualify_max'];
+                $sql = "SELECT score
 						FROM $tbl_attendance_result
                         WHERE
                         	c_id = $course_id AND
                         	user_id = $user_id AND
                         	attendance_id = ".$attendance['id'];
-				$rs = Database::query($sql);
-				$score = 0;
-				if (Database::num_rows($rs) > 0) {
-					$row = Database::fetch_array($rs);
-					$score = $row['score'];
-				}
-				$faults = $total_done_attendance-$score;
-				$faults = $faults > 0 ? $faults:0;
-				$total_faults += $faults;
-				$total_weight += $total_done_attendance;
-			}
-		}
-
-		$porcent = $total_weight > 0 ?round(($total_faults*100)/$total_weight,0):0;
-		$results['faults'] 	= $total_faults;
-		$results['total']	= $total_weight;
-		$results['porcent'] = $porcent;
-
-		return $results;
-	}
-
-	/**
-	 * Get results of faults average by course
-	 * @param	int	   $user_id
-	 * @param 	string	$course_code
-	 * @param	int	   Session id (optional)
-	 * @return 	array  results containing number of faults,
-	 * total done attendance, porcent of faults and color depend on result (red, orange)
-	 */
-	public function get_faults_average_by_course($user_id, $course_code, $session_id = null)
-	{
-		// Database tables and variables
-		$course_info = api_get_course_info($course_code);
-		$tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
-		$user_id = intval($user_id);
-		$results = array();
-		$total_faults = $total_weight = $porcent = 0;
-		$attendances_by_course = $this->get_attendances_list($course_info['real_id'], $session_id);
-
-		foreach ($attendances_by_course as $attendance) {
-			// Get total faults and total weight
-			$total_done_attendance 	= $attendance['attendance_qualify_max'];
-			$sql = "SELECT score FROM $tbl_attendance_result
-					WHERE
+                $rs = Database::query($sql);
+                $score = 0;
+                if (Database::num_rows($rs) > 0) {
+                    $row = Database::fetch_array($rs);
+                    $score = $row['score'];
+                }
+                $faults = $total_done_attendance-$score;
+                $faults = $faults > 0 ? $faults:0;
+                $total_faults += $faults;
+                $total_weight += $total_done_attendance;
+            }
+        }
+
+        $porcent = $total_weight > 0 ?round(($total_faults*100)/$total_weight,0):0;
+        $results['faults'] 	= $total_faults;
+        $results['total']	= $total_weight;
+        $results['porcent'] = $porcent;
+
+        return $results;
+    }
+
+    /**
+     * Get results of faults average by course
+     * @param	int	   $user_id
+     * @param 	string	$course_code
+     * @param	int	   Session id (optional)
+     * @return 	array  results containing number of faults,
+     * total done attendance, porcent of faults and color depend on result (red, orange)
+     */
+    public function get_faults_average_by_course($user_id, $course_code, $session_id = null)
+    {
+        // Database tables and variables
+        $course_info = api_get_course_info($course_code);
+        $tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+        $user_id = intval($user_id);
+        $results = array();
+        $total_faults = $total_weight = $porcent = 0;
+        $attendances_by_course = $this->get_attendances_list($course_info['real_id'], $session_id);
+
+        foreach ($attendances_by_course as $attendance) {
+            // Get total faults and total weight
+            $total_done_attendance 	= $attendance['attendance_qualify_max'];
+            $sql = "SELECT score FROM $tbl_attendance_result
+					WHERE
 						c_id = {$course_info['real_id']} AND
 						user_id = $user_id AND
 						attendance_id=".$attendance['id'];
-			$rs = Database::query($sql);
-			$score = 0;
-			if (Database::num_rows($rs) > 0) {
-				$row = Database::fetch_array($rs);
-				$score = $row['score'];
-			}
-			$faults = $total_done_attendance-$score;
-			$faults = $faults > 0 ? $faults:0;
-			$total_faults += $faults;
-			$total_weight += $total_done_attendance;
-		}
-
-		$porcent = $total_weight > 0 ?round(($total_faults*100)/$total_weight,0):0;
-		$results['faults'] 	= $total_faults;
-		$results['total']	= $total_weight;
-		$results['porcent'] = $porcent;
-
-		return $results;
-	}
-
-	/**
-	 * Get registered users' attendance sheet inside current course
-	 * @param	int	   $attendance_id
-	 * @param	int	   $user_id for showing data for only one user (optional)
-	 * @return 	array  users attendance sheet data
-	 */
-	public function get_users_attendance_sheet($attendance_id, $user_id = 0, $groupId = null)
-	{
-		$tbl_attendance_sheet 	= Database::get_course_table(TABLE_ATTENDANCE_SHEET);
-		$tbl_attendance_calendar= Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-
-		$attendance_calendar = $this->get_attendance_calendar($attendance_id, 'all', null, $groupId);
-		$calendar_ids = array();
-		// get all dates from calendar by current attendance
-		foreach ($attendance_calendar as $cal) {
-			$calendar_ids[] = $cal['id'];
-		}
-
-		$course_id = api_get_course_int_id();
-
-		$data = array();
-		if (empty($user_id)) {
-			// get all registered users inside current course
-			$users = $this->get_users_rel_course();
-			$user_ids = array_keys($users);
-			if (count($calendar_ids) > 0 && count($user_ids) > 0) {
-				foreach ($user_ids as $uid) {
-					$sql = "SELECT * FROM $tbl_attendance_sheet
+            $rs = Database::query($sql);
+            $score = 0;
+            if (Database::num_rows($rs) > 0) {
+                $row = Database::fetch_array($rs);
+                $score = $row['score'];
+            }
+            $faults = $total_done_attendance-$score;
+            $faults = $faults > 0 ? $faults:0;
+            $total_faults += $faults;
+            $total_weight += $total_done_attendance;
+        }
+
+        $porcent = $total_weight > 0 ?round(($total_faults*100)/$total_weight,0):0;
+        $results['faults'] 	= $total_faults;
+        $results['total']	= $total_weight;
+        $results['porcent'] = $porcent;
+
+        return $results;
+    }
+
+    /**
+     * Get registered users' attendance sheet inside current course
+     * @param	int	   $attendance_id
+     * @param	int	   $user_id for showing data for only one user (optional)
+     * @return 	array  users attendance sheet data
+     */
+    public function get_users_attendance_sheet($attendance_id, $user_id = 0, $groupId = null)
+    {
+        $tbl_attendance_sheet 	= Database::get_course_table(TABLE_ATTENDANCE_SHEET);
+        $tbl_attendance_calendar= Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+
+        $attendance_calendar = $this->get_attendance_calendar($attendance_id, 'all', null, $groupId);
+        $calendar_ids = array();
+        // get all dates from calendar by current attendance
+        foreach ($attendance_calendar as $cal) {
+            $calendar_ids[] = $cal['id'];
+        }
+
+        $course_id = api_get_course_int_id();
+
+        $data = array();
+        if (empty($user_id)) {
+            // get all registered users inside current course
+            $users = $this->get_users_rel_course();
+            $user_ids = array_keys($users);
+            if (count($calendar_ids) > 0 && count($user_ids) > 0) {
+                foreach ($user_ids as $uid) {
+                    $sql = "SELECT * FROM $tbl_attendance_sheet
 					        WHERE
 					        	c_id = $course_id AND
 					        	user_id = '$uid' AND
 					        	attendance_calendar_id IN(".implode(',',$calendar_ids).")
 							";
-					$res = Database::query($sql);
-					if (Database::num_rows($res) > 0) {
-						while ($row = Database::fetch_array($res)) {
-							$data[$uid][$row['attendance_calendar_id']]['presence'] = $row['presence'];
-						}
-					}
-				}
-			}
-		} else {
-			// Get attendance for current user
-			$user_id = intval($user_id);
-			if (count($calendar_ids) > 0) {
-				$sql = "SELECT cal.date_time, att.presence
+                    $res = Database::query($sql);
+                    if (Database::num_rows($res) > 0) {
+                        while ($row = Database::fetch_array($res)) {
+                            $data[$uid][$row['attendance_calendar_id']]['presence'] = $row['presence'];
+                        }
+                    }
+                }
+            }
+        } else {
+            // Get attendance for current user
+            $user_id = intval($user_id);
+            if (count($calendar_ids) > 0) {
+                $sql = "SELECT cal.date_time, att.presence
                         FROM $tbl_attendance_sheet att
 						INNER JOIN  $tbl_attendance_calendar cal
 						ON cal.id = att.attendance_calendar_id
@@ -1181,87 +1181,87 @@ 
 							att.user_id = '$user_id' AND
 							att.attendance_calendar_id IN (".implode(',',$calendar_ids).")
                         ORDER BY date_time";
-				$res = Database::query($sql);
-				if (Database::num_rows($res) > 0) {
-					while ($row = Database::fetch_array($res)) {
-						$row['date_time'] = api_convert_and_format_date($row['date_time'], null, date_default_timezone_get());
-						$data[$user_id][] = $row;
-					}
-				}
-			}
-		}
-		return $data;
-	}
-
-	/**
-	 * Get next attendance calendar without presences (done attendances)
-	 * @param	int	attendance id
-	 * @return 	int attendance calendar id
-	 */
-	public function get_next_attendance_calendar_id($attendance_id)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-
-		$sql = "SELECT id FROM $tbl_attendance_calendar
+                $res = Database::query($sql);
+                if (Database::num_rows($res) > 0) {
+                    while ($row = Database::fetch_array($res)) {
+                        $row['date_time'] = api_convert_and_format_date($row['date_time'], null, date_default_timezone_get());
+                        $data[$user_id][] = $row;
+                    }
+                }
+            }
+        }
+        return $data;
+    }
+
+    /**
+     * Get next attendance calendar without presences (done attendances)
+     * @param	int	attendance id
+     * @return 	int attendance calendar id
+     */
+    public function get_next_attendance_calendar_id($attendance_id)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+
+        $sql = "SELECT id FROM $tbl_attendance_calendar
 		        WHERE
 		        	c_id = $course_id AND
 		        	attendance_id = '$attendance_id' AND
 		        	done_attendance = 0
 		        ORDER BY date_time
 		        LIMIT 1";
-		$rs = Database::query($sql);
-		$next_calendar_id = 0;
-		if (Database::num_rows($rs) > 0) {
-			$row = Database::fetch_array($rs);
-			$next_calendar_id = $row['id'];
-		}
-		return $next_calendar_id;
-	}
-
-	/**
-	 * Get next attendance calendar datetime without presences (done attendances)
-	 * @param	int	attendance id
-	 * @return 	int UNIX time format datetime
-	 */
-	public function get_next_attendance_calendar_datetime($attendance_id)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$course_id = api_get_course_int_id();
-		$attendance_id = intval($attendance_id);
-		$sql = "SELECT id, date_time FROM $tbl_attendance_calendar
+        $rs = Database::query($sql);
+        $next_calendar_id = 0;
+        if (Database::num_rows($rs) > 0) {
+            $row = Database::fetch_array($rs);
+            $next_calendar_id = $row['id'];
+        }
+        return $next_calendar_id;
+    }
+
+    /**
+     * Get next attendance calendar datetime without presences (done attendances)
+     * @param	int	attendance id
+     * @return 	int UNIX time format datetime
+     */
+    public function get_next_attendance_calendar_datetime($attendance_id)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $course_id = api_get_course_int_id();
+        $attendance_id = intval($attendance_id);
+        $sql = "SELECT id, date_time FROM $tbl_attendance_calendar
 				WHERE
 					c_id = $course_id AND
 					attendance_id = '$attendance_id' AND
 					done_attendance = 0
 				ORDER BY date_time
 				LIMIT 1";
-		$rs = Database::query($sql);
-		$next_calendar_datetime = 0;
-		if (Database::num_rows($rs) > 0) {
-			$row = Database::fetch_array($rs);
-			$next_calendar_datetime = api_get_local_time($row['date_time']);
-		}
-
-		return $next_calendar_datetime;
-	}
-
-	/**
-	 * Get user' score from current attendance
-	 * @param	int	$user_id
-	 * @param	int $attendance_id
-	 * @return	int score
-	 */
-	public function get_user_score($user_id, $attendance_id, $groupId = null)
-	{
-		$tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
+        $rs = Database::query($sql);
+        $next_calendar_datetime = 0;
+        if (Database::num_rows($rs) > 0) {
+            $row = Database::fetch_array($rs);
+            $next_calendar_datetime = api_get_local_time($row['date_time']);
+        }
+
+        return $next_calendar_datetime;
+    }
+
+    /**
+     * Get user' score from current attendance
+     * @param	int	$user_id
+     * @param	int $attendance_id
+     * @return	int score
+     */
+    public function get_user_score($user_id, $attendance_id, $groupId = null)
+    {
+        $tbl_attendance_result 	= Database::get_course_table(TABLE_ATTENDANCE_RESULT);
         $tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
         $tbl_attendance_cal_rel_group = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
         $tbl_attendance_cal = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$user_id = intval($user_id);
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
+        $user_id = intval($user_id);
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
         if (empty($groupId)) {
             $sql = "SELECT score FROM $tbl_attendance_result
                     WHERE
@@ -1285,68 +1285,68 @@ 
                         )
                     ";
         }
-		$rs = Database::query($sql);
-		$score = 0;
-		if (Database::num_rows($rs) > 0) {
-			$row = Database::fetch_array($rs);
-			$score = $row['score'];
-		}
-
-		return $score;
-	}
-
-	/**
-	 * Get attendance calendar data by id
-	 * @param	int	attendance calendar id
-	 * @return	array attendance calendar data
-	 */
-	public function get_attendance_calendar_by_id($calendar_id)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$calendar_id = intval($calendar_id);
-		$course_id = api_get_course_int_id();
-		$sql = "SELECT * FROM $tbl_attendance_calendar
+        $rs = Database::query($sql);
+        $score = 0;
+        if (Database::num_rows($rs) > 0) {
+            $row = Database::fetch_array($rs);
+            $score = $row['score'];
+        }
+
+        return $score;
+    }
+
+    /**
+     * Get attendance calendar data by id
+     * @param	int	attendance calendar id
+     * @return	array attendance calendar data
+     */
+    public function get_attendance_calendar_by_id($calendar_id)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $calendar_id = intval($calendar_id);
+        $course_id = api_get_course_int_id();
+        $sql = "SELECT * FROM $tbl_attendance_calendar
 				WHERE c_id = $course_id AND id = '$calendar_id' ";
-		$rs = Database::query($sql);
-		$data = array();
-		if (Database::num_rows($rs) > 0) {
-			while ($row = Database::fetch_array($rs)) {
-				$row['date_time'] = api_get_local_time($row['date_time']);
-				$data = $row;
-			}
-		}
-
-		return $data;
-	}
-
-	/**
-	 * Get all attendance calendar data inside current attendance
-	 * @param int	$attendance_id
-	 * @param string $type
-	 * @param int $calendar_id
-	 * @param int $groupId
-	 * @param bool $showAll = false show group calendar items or not
-	 *
-	 * @return	array attendance calendar data
-	 */
-	public function get_attendance_calendar(
-		$attendance_id,
-		$type = 'all',
-		$calendar_id = null,
-		$groupId = null,
-		$showAll = false
-	) {
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $rs = Database::query($sql);
+        $data = array();
+        if (Database::num_rows($rs) > 0) {
+            while ($row = Database::fetch_array($rs)) {
+                $row['date_time'] = api_get_local_time($row['date_time']);
+                $data = $row;
+            }
+        }
+
+        return $data;
+    }
+
+    /**
+     * Get all attendance calendar data inside current attendance
+     * @param int	$attendance_id
+     * @param string $type
+     * @param int $calendar_id
+     * @param int $groupId
+     * @param bool $showAll = false show group calendar items or not
+     *
+     * @return	array attendance calendar data
+     */
+    public function get_attendance_calendar(
+        $attendance_id,
+        $type = 'all',
+        $calendar_id = null,
+        $groupId = null,
+        $showAll = false
+    ) {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
         $tbl_acrg = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-		$groupCondition = null;
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+        $groupCondition = null;
 
-		if ($showAll) {
-			$sql = "SELECT * FROM $tbl_attendance_calendar c
+        if ($showAll) {
+            $sql = "SELECT * FROM $tbl_attendance_calendar c
 					WHERE c_id = $course_id AND attendance_id = '$attendance_id'";
-		} else {
-			$sql = "SELECT * FROM $tbl_attendance_calendar c
+        } else {
+            $sql = "SELECT * FROM $tbl_attendance_calendar c
 					WHERE
 						c_id = $course_id AND
 						attendance_id = '$attendance_id' AND
@@ -1355,11 +1355,11 @@ 
 							WHERE c_id = $course_id AND group_id != 0 AND group_id IS NOT NULL
 						)
 					";
-		}
+        }
 
-		if (!empty($groupId)) {
-			$groupId = intval($groupId);
-			$sql = "SELECT c.* FROM $tbl_attendance_calendar c
+        if (!empty($groupId)) {
+            $groupId = intval($groupId);
+            $sql = "SELECT c.* FROM $tbl_attendance_calendar c
 					INNER JOIN $tbl_acrg g
 					ON c.c_id = g.c_id AND c.id = g.calendar_id
 					WHERE
@@ -1367,62 +1367,62 @@ 
 					    g.group_id = '$groupId' AND
 					    c.attendance_id = '$attendance_id'
                    ";
-		}
-
-		if (!in_array($type, array('today', 'all', 'all_done', 'all_not_done','calendar_id'))) {
-			$type = 'all';
-		}
-
-		switch ($type) {
-			case 'calendar_id':
-				$calendar_id = intval($calendar_id);
-				if (!empty($calendar_id)) {
-					$sql.= " AND c.id = $calendar_id";
-				}
-				break;
-			case 'today':
-				//$sql .= ' AND DATE_FORMAT(date_time,"%d-%m-%Y") = DATE_FORMAT("'.api_get_utc_datetime().'", "%d-%m-%Y" )';
-				break;
-			case 'all_done':
-				$sql .= " AND done_attendance = 1 ";
-				break;
-			case 'all_not_done':
-				$sql .= " AND done_attendance = 0 ";
-				break;
-			case 'all':
-			default:
-				break;
-		}
-		$sql .= " ORDER BY date_time ";
+        }
 
-		$rs = Database::query($sql);
-		$data = array();
-		if (Database::num_rows($rs) > 0) {
-			while ($row = Database::fetch_array($rs,'ASSOC')) {
-				$row['db_date_time'] = $row['date_time'];
-				$row['date_time'] = api_get_local_time($row['date_time']);
-				$row['date'] = api_format_date($row['date_time'], DATE_FORMAT_SHORT);
-				$row['time'] = api_format_date($row['date_time'], TIME_NO_SEC_FORMAT);
-				$row['groups'] = $this->getGroupListByAttendanceCalendar($row['id'], $course_id);
-				if ($type == 'today') {
-					if (date('d-m-Y', api_strtotime($row['date_time'], 'UTC')) == date('d-m-Y', time())) {
-						$data[] = $row;
-					}
-				} else {
-					$data[] = $row;
-				}
-			}
-		}
-
-		return $data;
-	}
-
-	/**
-	 * Get number of attendance calendar inside current attendance
-	 * @param	int	$attendance_id
-	 * @param	int	$groupId
-	 * @return	int number of dates in attendance calendar
-	 */
+        if (!in_array($type, array('today', 'all', 'all_done', 'all_not_done','calendar_id'))) {
+            $type = 'all';
+        }
+
+        switch ($type) {
+            case 'calendar_id':
+                $calendar_id = intval($calendar_id);
+                if (!empty($calendar_id)) {
+                    $sql.= " AND c.id = $calendar_id";
+                }
+                break;
+            case 'today':
+                //$sql .= ' AND DATE_FORMAT(date_time,"%d-%m-%Y") = DATE_FORMAT("'.api_get_utc_datetime().'", "%d-%m-%Y" )';
+                break;
+            case 'all_done':
+                $sql .= " AND done_attendance = 1 ";
+                break;
+            case 'all_not_done':
+                $sql .= " AND done_attendance = 0 ";
+                break;
+            case 'all':
+            default:
+                break;
+        }
+        $sql .= " ORDER BY date_time ";
+
+        $rs = Database::query($sql);
+        $data = array();
+        if (Database::num_rows($rs) > 0) {
+            while ($row = Database::fetch_array($rs,'ASSOC')) {
+                $row['db_date_time'] = $row['date_time'];
+                $row['date_time'] = api_get_local_time($row['date_time']);
+                $row['date'] = api_format_date($row['date_time'], DATE_FORMAT_SHORT);
+                $row['time'] = api_format_date($row['date_time'], TIME_NO_SEC_FORMAT);
+                $row['groups'] = $this->getGroupListByAttendanceCalendar($row['id'], $course_id);
+                if ($type == 'today') {
+                    if (date('d-m-Y', api_strtotime($row['date_time'], 'UTC')) == date('d-m-Y', time())) {
+                        $data[] = $row;
+                    }
+                } else {
+                    $data[] = $row;
+                }
+            }
+        }
+
+        return $data;
+    }
+
+    /**
+     * Get number of attendance calendar inside current attendance
+     * @param	int	$attendance_id
+     * @param	int	$groupId
+     * @return	int number of dates in attendance calendar
+     */
     public static function get_number_of_attendance_calendar($attendance_id, $groupId = 0, $done_attendance = NULL, $userId = 0)
     {
         $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
@@ -1508,112 +1508,112 @@ 
 
 
     /**
-	 * Get count dates inside attendance calendar by attendance id
-	 * @param	int	$attendance_id
-	 * @return	int     count of dates
-	 */
-	public static function get_count_dates_inside_attendance_calendar($attendance_id)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-		$sql = "SELECT count(id) FROM $tbl_attendance_calendar
+     * Get count dates inside attendance calendar by attendance id
+     * @param	int	$attendance_id
+     * @return	int     count of dates
+     */
+    public static function get_count_dates_inside_attendance_calendar($attendance_id)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+        $sql = "SELECT count(id) FROM $tbl_attendance_calendar
                 WHERE
                 	c_id = $course_id AND
                 	attendance_id = '$attendance_id'";
-		$rs = Database::query($sql);
-		$count = 0;
-		if (Database::num_rows($rs) > 0) {
-			$row = Database::fetch_row($rs);
-			$count = $row[0];
-		}
-		return $count;
-	}
-
-	/**
-	 * check if all calendar of an attendance is done
-	 * @param   int     $attendance_id
-	 * @return  bool    True if all calendar is done, otherwise false
-	 */
-	public static function is_all_attendance_calendar_done($attendance_id)
-	{
-		$attendance_id = intval($attendance_id);
-		$done_calendar = self::get_done_attendance_calendar($attendance_id);
-		$count_dates_in_calendar = self::get_count_dates_inside_attendance_calendar($attendance_id);
-		$number_of_dates = self::get_number_of_attendance_calendar($attendance_id);
-
-		$result = false;
-		if ($number_of_dates && (intval($count_dates_in_calendar) == intval($done_calendar))) {
-			$result = true;
-		}
-		return $result;
-	}
-
-	/**
-	 * check if an attendance is locked
-	 * @param   int $attendance_id
-	 * @param   bool
-	 */
-	public static function is_locked_attendance($attendance_id)
-	{
-		//use gradebook lock
-		$result = api_resource_is_locked_by_gradebook($attendance_id, LINK_ATTENDANCE);
-		return $result;
-	}
-
-	/**
-	 * Add new datetime inside attendance calendar table
-	 * @param	int		$attendance_id
-	 * @param 	array 	$groupList
-	 * @return	int affected rows
-	 */
-	public function attendance_calendar_add($attendance_id, $groupList = array())
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$affected_rows = 0;
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-		// check if datetime already exists inside the table
-		/*$sql = "SELECT id FROM $tbl_attendance_calendar
+        $rs = Database::query($sql);
+        $count = 0;
+        if (Database::num_rows($rs) > 0) {
+            $row = Database::fetch_row($rs);
+            $count = $row[0];
+        }
+        return $count;
+    }
+
+    /**
+     * check if all calendar of an attendance is done
+     * @param   int     $attendance_id
+     * @return  bool    True if all calendar is done, otherwise false
+     */
+    public static function is_all_attendance_calendar_done($attendance_id)
+    {
+        $attendance_id = intval($attendance_id);
+        $done_calendar = self::get_done_attendance_calendar($attendance_id);
+        $count_dates_in_calendar = self::get_count_dates_inside_attendance_calendar($attendance_id);
+        $number_of_dates = self::get_number_of_attendance_calendar($attendance_id);
+
+        $result = false;
+        if ($number_of_dates && (intval($count_dates_in_calendar) == intval($done_calendar))) {
+            $result = true;
+        }
+        return $result;
+    }
+
+    /**
+     * check if an attendance is locked
+     * @param   int $attendance_id
+     * @param   bool
+     */
+    public static function is_locked_attendance($attendance_id)
+    {
+        //use gradebook lock
+        $result = api_resource_is_locked_by_gradebook($attendance_id, LINK_ATTENDANCE);
+        return $result;
+    }
+
+    /**
+     * Add new datetime inside attendance calendar table
+     * @param	int		$attendance_id
+     * @param 	array 	$groupList
+     * @return	int affected rows
+     */
+    public function attendance_calendar_add($attendance_id, $groupList = array())
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $affected_rows = 0;
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+        // check if datetime already exists inside the table
+        /*$sql = "SELECT id FROM $tbl_attendance_calendar
 		        WHERE
 		        	c_id = $course_id AND
 		        	date_time='".Database::escape_string($this->date_time)."' AND
 		        	attendance_id = '$attendance_id'";
 		$rs = Database::query($sql);
 		if (Database::num_rows($rs) == 0) {*/
-			$params = array(
-				'c_id' =>  $course_id,
-				'date_time' => $this->date_time,
-				'attendance_id' => $attendance_id,
+            $params = array(
+                'c_id' =>  $course_id,
+                'date_time' => $this->date_time,
+                'attendance_id' => $attendance_id,
                 'done_attendance' => 0
-			);
-			$id = Database::insert($tbl_attendance_calendar, $params);
-
-			if ($id) {
-				$sql = "UPDATE $tbl_attendance_calendar SET id = iid WHERE iid = $id";
-				Database::query($sql);
-				$affected_rows++;
-			}
-			$this->addAttendanceCalendarToGroup($id, $course_id, $groupList);
-		//}
-
-		// update locked attendance
-		$is_all_calendar_done = self::is_all_attendance_calendar_done($attendance_id);
-		if (!$is_all_calendar_done) {
-			self::lock_attendance($attendance_id, false);
-		} else {
-			self::lock_attendance($attendance_id);
-		}
-		return $affected_rows;
-	}
-
-	/**
-	 * @param int   $calendarId
-	 * @param int 	$courseId
-	 * @param array $groupList
-	 */
-	public function addAttendanceCalendarToGroup($calendarId, $courseId, $groupList)
-	{
+            );
+            $id = Database::insert($tbl_attendance_calendar, $params);
+
+            if ($id) {
+                $sql = "UPDATE $tbl_attendance_calendar SET id = iid WHERE iid = $id";
+                Database::query($sql);
+                $affected_rows++;
+            }
+            $this->addAttendanceCalendarToGroup($id, $course_id, $groupList);
+        //}
+
+        // update locked attendance
+        $is_all_calendar_done = self::is_all_attendance_calendar_done($attendance_id);
+        if (!$is_all_calendar_done) {
+            self::lock_attendance($attendance_id, false);
+        } else {
+            self::lock_attendance($attendance_id);
+        }
+        return $affected_rows;
+    }
+
+    /**
+     * @param int   $calendarId
+     * @param int 	$courseId
+     * @param array $groupList
+     */
+    public function addAttendanceCalendarToGroup($calendarId, $courseId, $groupList)
+    {
         if (empty($groupList)) {
             return false;
         }
@@ -1641,506 +1641,506 @@ 
                 Database::insert($table, $params);
             }
         }
-	}
-
-	/**
-	 * @param int $calendarId
-	 * @param int $courseId
-	 * @param int $groupId
-	 * @return array
-	 */
-	public function getGroupListByAttendanceCalendar($calendarId, $courseId)
-	{
-		$table = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
-		return Database::select(
-			'*',
-			$table,
-			array('where'=>
-				array(
-					'calendar_id = ? AND c_id = ?' => array($calendarId, $courseId)
-				)
-			)
-		);
-	}
-
-	/**
-	 * @param int $calendarId
-	 * @param int $courseId
-	 * @param int $groupId
-	 * @return array
-	 */
-	public function getAttendanceCalendarGroup($calendarId, $courseId, $groupId)
-	{
-		$table = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
-		return Database::select(
-			'*',
-			$table,
-			array('where'=>
-				array(
-					'calendar_id = ? AND c_id = ? AND group_id = ?' => array($calendarId, $courseId, $groupId)
-				)
-			)
-		);
-	}
-
-	/**
-	 * @param int $calendarId
-	 * @param int $courseId
-	 * @param int $groupId
+    }
+
+    /**
+     * @param int $calendarId
+     * @param int $courseId
+     * @param int $groupId
+     * @return array
+     */
+    public function getGroupListByAttendanceCalendar($calendarId, $courseId)
+    {
+        $table = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
+        return Database::select(
+            '*',
+            $table,
+            array('where'=>
+                array(
+                    'calendar_id = ? AND c_id = ?' => array($calendarId, $courseId)
+                )
+            )
+        );
+    }
+
+    /**
+     * @param int $calendarId
+     * @param int $courseId
+     * @param int $groupId
+     * @return array
+     */
+    public function getAttendanceCalendarGroup($calendarId, $courseId, $groupId)
+    {
+        $table = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
+        return Database::select(
+            '*',
+            $table,
+            array('where'=>
+                array(
+                    'calendar_id = ? AND c_id = ? AND group_id = ?' => array($calendarId, $courseId, $groupId)
+                )
+            )
+        );
+    }
+
+    /**
+     * @param int $calendarId
+     * @param int $courseId
+     * @param int $groupId
      *
-	 * @return array
-	 */
-	public function deleteAttendanceCalendarGroup($calendarId, $courseId)
-	{
-		$table = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
-		Database::delete(
-			$table,
-			array(
-				'calendar_id = ? AND c_id = ?' => array($calendarId, $courseId)
-			)
-		);
-	}
-
-	/**
-	 * save repeated date inside attendance calendar table
-	 * @param int $attendance_id
-	 * @param int $start_date start date in tms
-	 * @param int $end_date end date in tms
-	 * @param string $repeat_type daily, weekly, monthlyByDate
-	 * @param array $groupList
-	 */
-	public function attendance_repeat_calendar_add(
-		$attendance_id,
-		$start_date,
-		$end_date,
-		$repeat_type,
-		$groupList = array()
-	) {
-		$attendance_id = intval($attendance_id);
-		// save start date
-		$datetimezone = api_get_utc_datetime($start_date);
-		$this->set_date_time($datetimezone);
-		$this->attendance_calendar_add($attendance_id, $groupList);
-
-		// 86400 = 24 hours in seconds
-		// 604800 = 1 week in seconds
-		// Saves repeated dates
-		switch ($repeat_type) {
-			case 'daily':
-				$j = 1;
-				for ($i = $start_date + 86400; ($i <= $end_date); $i += 86400) {
-					$datetimezone = api_get_utc_datetime($i);
-					$this->set_date_time($datetimezone);
-					$this->attendance_calendar_add($attendance_id, $groupList);
-					$j++;
-				}
-				break;
-			case 'weekly':
-				$j = 1;
-				for ($i = $start_date + 604800; ($i <= $end_date); $i += 604800) {
-					$datetimezone = api_get_utc_datetime($i);
-					$this->set_date_time($datetimezone);
-					$this->attendance_calendar_add($attendance_id, $groupList);
-					$j++;
-				}
-				break;
-			case 'monthlyByDate':
-				$j = 1;
-				//@todo fix bug with february
-				for ($i = $start_date + 2419200; ($i <= $end_date); $i += 2419200) {
-					$datetimezone = api_get_utc_datetime($i);
-					$this->set_date_time($datetimezone);
-					$this->attendance_calendar_add($attendance_id, $groupList);
-					$j++;
-				}
-				break;
-		}
-	}
-
-	/**
-	 * Adds x months to a UNIX timestamp
-	 * @param   int     The timestamp
-	 * @param   int     The number of years to add
-	 * @return  int     The new timestamp
-	 */
-	private function add_month($timestamp, $num=1)
-	{
-		$values = api_get_utc_datetime($timestamp);
-		$values = str_replace(array(':','-',' '), '/', $values);
-		list($y, $m, $d, $h, $n, $s) = split('/',$values);
-		if($m+$num>12) {
-			$y += floor($num/12);
-			$m += $num%12;
-		} else {
-			$m += $num;
-		}
-		//date_default_timezone_set('UTC');
-		// return mktime($h, $n, $s, $m, $d, $y);
-		$result = api_strtotime($y.'-'.$m.'-'.$d.' '.$h.':'.$n.':'.$s, 'UTC');
-		if (!empty($result)) {
-			return $result;
-		}
-		return false;
-	}
-
-	/**
-	 * edit a datetime inside attendance calendar table
-	 * @param	int	attendance calendar id
-	 * @param	int	attendance id
-	 * @return	int affected rows
-	 */
-	public function attendance_calendar_edit($calendar_id, $attendance_id)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$affected_rows = 0;
-		$attendance_id = intval($attendance_id);
-		$course_id = api_get_course_int_id();
-		// check if datetime already exists inside the table
-		$sql = "SELECT id FROM $tbl_attendance_calendar
+     * @return array
+     */
+    public function deleteAttendanceCalendarGroup($calendarId, $courseId)
+    {
+        $table = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR_REL_GROUP);
+        Database::delete(
+            $table,
+            array(
+                'calendar_id = ? AND c_id = ?' => array($calendarId, $courseId)
+            )
+        );
+    }
+
+    /**
+     * save repeated date inside attendance calendar table
+     * @param int $attendance_id
+     * @param int $start_date start date in tms
+     * @param int $end_date end date in tms
+     * @param string $repeat_type daily, weekly, monthlyByDate
+     * @param array $groupList
+     */
+    public function attendance_repeat_calendar_add(
+        $attendance_id,
+        $start_date,
+        $end_date,
+        $repeat_type,
+        $groupList = array()
+    ) {
+        $attendance_id = intval($attendance_id);
+        // save start date
+        $datetimezone = api_get_utc_datetime($start_date);
+        $this->set_date_time($datetimezone);
+        $this->attendance_calendar_add($attendance_id, $groupList);
+
+        // 86400 = 24 hours in seconds
+        // 604800 = 1 week in seconds
+        // Saves repeated dates
+        switch ($repeat_type) {
+            case 'daily':
+                $j = 1;
+                for ($i = $start_date + 86400; ($i <= $end_date); $i += 86400) {
+                    $datetimezone = api_get_utc_datetime($i);
+                    $this->set_date_time($datetimezone);
+                    $this->attendance_calendar_add($attendance_id, $groupList);
+                    $j++;
+                }
+                break;
+            case 'weekly':
+                $j = 1;
+                for ($i = $start_date + 604800; ($i <= $end_date); $i += 604800) {
+                    $datetimezone = api_get_utc_datetime($i);
+                    $this->set_date_time($datetimezone);
+                    $this->attendance_calendar_add($attendance_id, $groupList);
+                    $j++;
+                }
+                break;
+            case 'monthlyByDate':
+                $j = 1;
+                //@todo fix bug with february
+                for ($i = $start_date + 2419200; ($i <= $end_date); $i += 2419200) {
+                    $datetimezone = api_get_utc_datetime($i);
+                    $this->set_date_time($datetimezone);
+                    $this->attendance_calendar_add($attendance_id, $groupList);
+                    $j++;
+                }
+                break;
+        }
+    }
+
+    /**
+     * Adds x months to a UNIX timestamp
+     * @param   int     The timestamp
+     * @param   int     The number of years to add
+     * @return  int     The new timestamp
+     */
+    private function add_month($timestamp, $num=1)
+    {
+        $values = api_get_utc_datetime($timestamp);
+        $values = str_replace(array(':','-',' '), '/', $values);
+        list($y, $m, $d, $h, $n, $s) = split('/',$values);
+        if($m+$num>12) {
+            $y += floor($num/12);
+            $m += $num%12;
+        } else {
+            $m += $num;
+        }
+        //date_default_timezone_set('UTC');
+        // return mktime($h, $n, $s, $m, $d, $y);
+        $result = api_strtotime($y.'-'.$m.'-'.$d.' '.$h.':'.$n.':'.$s, 'UTC');
+        if (!empty($result)) {
+            return $result;
+        }
+        return false;
+    }
+
+    /**
+     * edit a datetime inside attendance calendar table
+     * @param	int	attendance calendar id
+     * @param	int	attendance id
+     * @return	int affected rows
+     */
+    public function attendance_calendar_edit($calendar_id, $attendance_id)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $affected_rows = 0;
+        $attendance_id = intval($attendance_id);
+        $course_id = api_get_course_int_id();
+        // check if datetime already exists inside the table
+        $sql = "SELECT id FROM $tbl_attendance_calendar
 		        WHERE
 		        	c_id = $course_id AND
 		        	date_time = '".Database::escape_string($this->date_time)."' AND
 		        	attendance_id = '$attendance_id'";
-		$rs = Database::query($sql);
+        $rs = Database::query($sql);
 
-		if (Database::num_rows($rs) == 0) {
-			$sql = "UPDATE $tbl_attendance_calendar
+        if (Database::num_rows($rs) == 0) {
+            $sql = "UPDATE $tbl_attendance_calendar
 					SET date_time='".Database::escape_string($this->date_time)."'
 					WHERE c_id = $course_id AND id = '".intval($calendar_id)."'";
-			Database::query($sql);
-		}
-
-		// update locked attendance
-		$is_all_calendar_done = self::is_all_attendance_calendar_done($attendance_id);
-		if (!$is_all_calendar_done) {
-			self::lock_attendance($attendance_id, false);
-		} else {
-			self::lock_attendance($attendance_id);
-		}
-
-		return $affected_rows;
-	}
-
-	/**
-	 * delete a datetime from attendance calendar table
-	 * @param	int		attendance calendar id
-	 * @param	int		attendance id
-	 * @param	bool	true for removing all calendar inside current attendance, false for removing by calendar id
-	 * @return	int affected rows
-	 */
-	public function attendance_calendar_delete($calendar_id, $attendance_id , $all_delete = false)
-	{
-		$tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
-		$tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
-
-		$attendance_id = intval($attendance_id);
-		// get all registered users inside current course
-		$users = $this->get_users_rel_course();
-		$user_ids = array_keys($users);
-		$course_id = api_get_course_int_id();
-		$affected_rows = 0;
-		if ($all_delete) {
-			$attendance_calendar = $this->get_attendance_calendar($attendance_id);
-			// get all dates from calendar by current attendance
-			if (!empty($attendance_calendar)) {
-				foreach ($attendance_calendar as $cal) {
-					// delete all data from attendance sheet
-					$sql = "DELETE FROM $tbl_attendance_sheet
+            Database::query($sql);
+        }
+
+        // update locked attendance
+        $is_all_calendar_done = self::is_all_attendance_calendar_done($attendance_id);
+        if (!$is_all_calendar_done) {
+            self::lock_attendance($attendance_id, false);
+        } else {
+            self::lock_attendance($attendance_id);
+        }
+
+        return $affected_rows;
+    }
+
+    /**
+     * delete a datetime from attendance calendar table
+     * @param	int		attendance calendar id
+     * @param	int		attendance id
+     * @param	bool	true for removing all calendar inside current attendance, false for removing by calendar id
+     * @return	int affected rows
+     */
+    public function attendance_calendar_delete($calendar_id, $attendance_id , $all_delete = false)
+    {
+        $tbl_attendance_calendar = Database::get_course_table(TABLE_ATTENDANCE_CALENDAR);
+        $tbl_attendance_sheet = Database::get_course_table(TABLE_ATTENDANCE_SHEET);
+
+        $attendance_id = intval($attendance_id);
+        // get all registered users inside current course
+        $users = $this->get_users_rel_course();
+        $user_ids = array_keys($users);
+        $course_id = api_get_course_int_id();
+        $affected_rows = 0;
+        if ($all_delete) {
+            $attendance_calendar = $this->get_attendance_calendar($attendance_id);
+            // get all dates from calendar by current attendance
+            if (!empty($attendance_calendar)) {
+                foreach ($attendance_calendar as $cal) {
+                    // delete all data from attendance sheet
+                    $sql = "DELETE FROM $tbl_attendance_sheet
 							WHERE c_id = $course_id AND attendance_calendar_id = '".intval($cal['id'])."'";
-					Database::query($sql);
-					// delete data from attendance calendar
-					$sql = "DELETE FROM $tbl_attendance_calendar
+                    Database::query($sql);
+                    // delete data from attendance calendar
+                    $sql = "DELETE FROM $tbl_attendance_calendar
 							WHERE c_id = $course_id AND id = '".intval($cal['id'])."'";
-					Database::query($sql);
-
-					$this->deleteAttendanceCalendarGroup($cal['id'], $course_id);
-					$affected_rows++;
-				}
-			}
-		} else {
-			// delete just one row from attendance sheet by the calendar id
-			$sql = "DELETE FROM $tbl_attendance_sheet
+                    Database::query($sql);
+
+                    $this->deleteAttendanceCalendarGroup($cal['id'], $course_id);
+                    $affected_rows++;
+                }
+            }
+        } else {
+            // delete just one row from attendance sheet by the calendar id
+            $sql = "DELETE FROM $tbl_attendance_sheet
 					WHERE c_id = $course_id AND attendance_calendar_id = '".intval($calendar_id)."'";
-			Database::query($sql);
-			// delete data from attendance calendar
-			$sql = "DELETE FROM $tbl_attendance_calendar
+            Database::query($sql);
+            // delete data from attendance calendar
+            $sql = "DELETE FROM $tbl_attendance_calendar
 					WHERE c_id = $course_id AND id = '".intval($calendar_id)."'";
-			Database::query($sql);
-
-			$this->deleteAttendanceCalendarGroup($calendar_id, $course_id);
-			$affected_rows++;
-		}
-
-		// update users' results
-		$this->update_users_results($user_ids, $attendance_id);
-
-		return $affected_rows;
-	}
-
-	/** Setters for fields of attendances tables **/
-	public function set_session_id($session_id)
-	{
-		$this->session_id = $session_id;
-	}
-
-	public function set_course_id($course_id)
-	{
-		$this->course_id = $course_id;
-	}
-
-	public function set_date_time($datetime)
-	{
-		$this->date_time = $datetime;
-	}
-
-	public function set_name($name)
-	{
-		$this->name = $name;
-	}
-
-	public function set_description($description)
-	{
-		$this->description = $description;
-	}
-
-	public function set_attendance_qualify_title($attendance_qualify_title)
-	{
-		$this->attendance_qualify_title = $attendance_qualify_title;
-	}
-
-	public function set_attendance_weight($attendance_weight)
-	{
-		$this->attendance_weight = $attendance_weight;
-	}
-
-	/** Getters for fields of attendances tables **/
-	public function get_session_id()
-	{
-		return $this->session_id;
-	}
-
-	public function get_course_id()
-	{
-		return $this->course_id;
-	}
-
-	public function get_date_time()
-	{
-		return $this->date_time;
-	}
-
-	public function get_name()
-	{
-		return $this->name;
-	}
-
-	public function get_description()
-	{
-		return $this->description;
-	}
-
-	public function get_attendance_qualify_title()
-	{
-		return $this->attendance_qualify_title;
-	}
-
-	public function get_attendance_weight()
-	{
-		return $this->attendance_weight;
-	}
-
-	/**
-	 * @param string $startDate in UTC time
-	 * @param string $endDate in UTC time
-	 *
-	 * @return array
-	 */
-	public function getAttendanceLogin($startDate, $endDate)
-	{
-		if (empty($startDate) || $startDate == '0000-00-00' ||
-			empty($endDate) || $endDate == '0000-00-00'
-		) {
-			return false;
-		}
-
-		$sessionId = api_get_session_id();
-		$courseCode  = api_get_course_id();
-		if (!empty($sessionId)) {
-			$users = CourseManager:: get_user_list_from_course_code(
-				$courseCode,
-				$sessionId,
-				'',
-				'lastname',
-				0
-			);
-		} else {
-			$users = CourseManager:: get_user_list_from_course_code(
-				$courseCode,
-				0,
-				'',
-				'lastname',
-				STUDENT
-			);
-		}
-
-		$dateTimeStartOriginal = new DateTime($startDate);
-		$dateTimeStart = new DateTime($startDate);
-		$dateTimeEnd= new DateTime($endDate);
-		$interval = $dateTimeStart->diff($dateTimeEnd);
-		$days = intval($interval->format('%a'));
-
-		$dateList = array($dateTimeStart->format('Y-m-d'));
-		$headers = array(
-			get_lang('User'),
-			$dateTimeStart->format('Y-m-d')
-		);
-
-		for ($i = 0; $i < $days; $i++) {
-			$dateTimeStart = $dateTimeStart->add(new DateInterval('P1D'));
-			$date = $dateTimeStart->format('Y-m-d');
-			$dateList[] = $date;
-			$headers[] = $date;
-		}
-
-		$accessData = CourseManager::getCourseAccessPerCourseAndSession(
-			$courseCode,
-			$sessionId,
-			$dateTimeStartOriginal->format('Y-m-d H:i:s'),
-			$dateTimeEnd->format('Y-m-d H:i:s')
-		);
-
-		$results = array();
-		if (!empty($accessData)) {
-			foreach ($accessData as $data) {
-				$onlyDate = substr($data['login_course_date'], 0, 10);
-				$results[$data['user_id']][$onlyDate] = true;
-			}
-		}
-
-		return array(
-			'users' => $users,
-			'dateList' => $dateList,
-			'headers' => $headers,
-			'results' => $results
-		);
-	}
-
-	/**
-	 * @param string $startDate in UTC time
-	 * @param string $endDate in UTC time
-	 *
-	 * @return string
-	 */
-	public function getAttendanceLoginTable($startDate, $endDate)
-	{
-		$data = $this->getAttendanceLogin($startDate, $endDate);
-
-		if (!$data) {
-			return null;
-		}
-
-		$headers = $data['headers'];
-		$dateList = $data['dateList'];
-		$users = $data['users'];
-		$results = $data['results'];
-
-		$table = new HTML_Table(array('class' => 'data_table'));
-		$row = 0;
-		$column = 0;
-		foreach ($headers as $header) {
-			$table->setHeaderContents($row, $column, $header);
-			$column++;
-		}
-		$row = 1;
-		foreach ($users as $user) {
-			$table->setCellContents(
-				$row,
-				0,
-				$user['lastname'].' '.$user['firstname'].' ('.$user['username'].')'
-			);
-			$row ++;
-		}
-
-		$column = 1;
-		$row = 1;
-		foreach ($users as $user) {
-			foreach ($dateList as $date) {
-				$status = null;
-				if (isset($results[$user['user_id']]) &&
-					isset($results[$user['user_id']][$date])
-				) {
-					$status = 'X';
-				}
-				$table->setCellContents($row, $column, $status);
-				$column++;
-			}
-			$row++;
-			$column = 1;
-		}
-
-		return $table->toHtml();
-	}
-
-	/**
-	 * @param string $startDate in UTC time
-	 * @param string $endDate in UTC time
-	 *
-	 * @return string
-	 */
-	public function exportAttendanceLogin($startDate, $endDate)
-	{
-		$data = $this->getAttendanceLogin($startDate, $endDate);
-
-		if (!$data) {
-			return null;
-		}
-		$users = $data['users'];
-		$results = $data['results'];
-
-		$table = new HTML_Table(array('class' => 'data_table'));
-
-		$table->setHeaderContents(0, 0, get_lang('User'));
-		$table->setHeaderContents(0, 1, get_lang('Date'));
-
-		$row = 1;
-		foreach ($users as $user) {
-			$table->setCellContents(
-				$row,
-				0,
-				$user['lastname'].' '.$user['firstname'].' ('.$user['username'].')'
-			);
-			$row++;
-		}
-		$table->setColAttributes(0, array('style' => 'width:28%'));
-
-		$row = 1;
-		foreach ($users as $user) {
-			if (isset($results[$user['user_id']]) &&
-				!empty($results[$user['user_id']])
-			) {
-				$dates = implode(', ', array_keys($results[$user['user_id']]));
-				$table->setCellContents($row, 1, $dates);
-			}
-			$row++;
-		}
-
-		//$tableToString = null;
-
-		//$sessionInfo = api_get_session_info(api_get_session_id());
-		//if (!empty($sessionInfo)) {
-		/*$tableToString .= '<strong>'.get_lang('PeriodToDisplay').'</strong>: '.
+            Database::query($sql);
+
+            $this->deleteAttendanceCalendarGroup($calendar_id, $course_id);
+            $affected_rows++;
+        }
+
+        // update users' results
+        $this->update_users_results($user_ids, $attendance_id);
+
+        return $affected_rows;
+    }
+
+    /** Setters for fields of attendances tables **/
+    public function set_session_id($session_id)
+    {
+        $this->session_id = $session_id;
+    }
+
+    public function set_course_id($course_id)
+    {
+        $this->course_id = $course_id;
+    }
+
+    public function set_date_time($datetime)
+    {
+        $this->date_time = $datetime;
+    }
+
+    public function set_name($name)
+    {
+        $this->name = $name;
+    }
+
+    public function set_description($description)
+    {
+        $this->description = $description;
+    }
+
+    public function set_attendance_qualify_title($attendance_qualify_title)
+    {
+        $this->attendance_qualify_title = $attendance_qualify_title;
+    }
+
+    public function set_attendance_weight($attendance_weight)
+    {
+        $this->attendance_weight = $attendance_weight;
+    }
+
+    /** Getters for fields of attendances tables **/
+    public function get_session_id()
+    {
+        return $this->session_id;
+    }
+
+    public function get_course_id()
+    {
+        return $this->course_id;
+    }
+
+    public function get_date_time()
+    {
+        return $this->date_time;
+    }
+
+    public function get_name()
+    {
+        return $this->name;
+    }
+
+    public function get_description()
+    {
+        return $this->description;
+    }
+
+    public function get_attendance_qualify_title()
+    {
+        return $this->attendance_qualify_title;
+    }
+
+    public function get_attendance_weight()
+    {
+        return $this->attendance_weight;
+    }
+
+    /**
+     * @param string $startDate in UTC time
+     * @param string $endDate in UTC time
+     *
+     * @return array
+     */
+    public function getAttendanceLogin($startDate, $endDate)
+    {
+        if (empty($startDate) || $startDate == '0000-00-00' ||
+            empty($endDate) || $endDate == '0000-00-00'
+        ) {
+            return false;
+        }
+
+        $sessionId = api_get_session_id();
+        $courseCode  = api_get_course_id();
+        if (!empty($sessionId)) {
+            $users = CourseManager:: get_user_list_from_course_code(
+                $courseCode,
+                $sessionId,
+                '',
+                'lastname',
+                0
+            );
+        } else {
+            $users = CourseManager:: get_user_list_from_course_code(
+                $courseCode,
+                0,
+                '',
+                'lastname',
+                STUDENT
+            );
+        }
+
+        $dateTimeStartOriginal = new DateTime($startDate);
+        $dateTimeStart = new DateTime($startDate);
+        $dateTimeEnd= new DateTime($endDate);
+        $interval = $dateTimeStart->diff($dateTimeEnd);
+        $days = intval($interval->format('%a'));
+
+        $dateList = array($dateTimeStart->format('Y-m-d'));
+        $headers = array(
+            get_lang('User'),
+            $dateTimeStart->format('Y-m-d')
+        );
+
+        for ($i = 0; $i < $days; $i++) {
+            $dateTimeStart = $dateTimeStart->add(new DateInterval('P1D'));
+            $date = $dateTimeStart->format('Y-m-d');
+            $dateList[] = $date;
+            $headers[] = $date;
+        }
+
+        $accessData = CourseManager::getCourseAccessPerCourseAndSession(
+            $courseCode,
+            $sessionId,
+            $dateTimeStartOriginal->format('Y-m-d H:i:s'),
+            $dateTimeEnd->format('Y-m-d H:i:s')
+        );
+
+        $results = array();
+        if (!empty($accessData)) {
+            foreach ($accessData as $data) {
+                $onlyDate = substr($data['login_course_date'], 0, 10);
+                $results[$data['user_id']][$onlyDate] = true;
+            }
+        }
+
+        return array(
+            'users' => $users,
+            'dateList' => $dateList,
+            'headers' => $headers,
+            'results' => $results
+        );
+    }
+
+    /**
+     * @param string $startDate in UTC time
+     * @param string $endDate in UTC time
+     *
+     * @return string
+     */
+    public function getAttendanceLoginTable($startDate, $endDate)
+    {
+        $data = $this->getAttendanceLogin($startDate, $endDate);
+
+        if (!$data) {
+            return null;
+        }
+
+        $headers = $data['headers'];
+        $dateList = $data['dateList'];
+        $users = $data['users'];
+        $results = $data['results'];
+
+        $table = new HTML_Table(array('class' => 'data_table'));
+        $row = 0;
+        $column = 0;
+        foreach ($headers as $header) {
+            $table->setHeaderContents($row, $column, $header);
+            $column++;
+        }
+        $row = 1;
+        foreach ($users as $user) {
+            $table->setCellContents(
+                $row,
+                0,
+                $user['lastname'].' '.$user['firstname'].' ('.$user['username'].')'
+            );
+            $row ++;
+        }
+
+        $column = 1;
+        $row = 1;
+        foreach ($users as $user) {
+            foreach ($dateList as $date) {
+                $status = null;
+                if (isset($results[$user['user_id']]) &&
+                    isset($results[$user['user_id']][$date])
+                ) {
+                    $status = 'X';
+                }
+                $table->setCellContents($row, $column, $status);
+                $column++;
+            }
+            $row++;
+            $column = 1;
+        }
+
+        return $table->toHtml();
+    }
+
+    /**
+     * @param string $startDate in UTC time
+     * @param string $endDate in UTC time
+     *
+     * @return string
+     */
+    public function exportAttendanceLogin($startDate, $endDate)
+    {
+        $data = $this->getAttendanceLogin($startDate, $endDate);
+
+        if (!$data) {
+            return null;
+        }
+        $users = $data['users'];
+        $results = $data['results'];
+
+        $table = new HTML_Table(array('class' => 'data_table'));
+
+        $table->setHeaderContents(0, 0, get_lang('User'));
+        $table->setHeaderContents(0, 1, get_lang('Date'));
+
+        $row = 1;
+        foreach ($users as $user) {
+            $table->setCellContents(
+                $row,
+                0,
+                $user['lastname'].' '.$user['firstname'].' ('.$user['username'].')'
+            );
+            $row++;
+        }
+        $table->setColAttributes(0, array('style' => 'width:28%'));
+
+        $row = 1;
+        foreach ($users as $user) {
+            if (isset($results[$user['user_id']]) &&
+                !empty($results[$user['user_id']])
+            ) {
+                $dates = implode(', ', array_keys($results[$user['user_id']]));
+                $table->setCellContents($row, 1, $dates);
+            }
+            $row++;
+        }
+
+        //$tableToString = null;
+
+        //$sessionInfo = api_get_session_info(api_get_session_id());
+        //if (!empty($sessionInfo)) {
+        /*$tableToString .= '<strong>'.get_lang('PeriodToDisplay').'</strong>: '.
 			sprintf(get_lang('FromDateXToDateY'), $startDate, $endDate);*/
-		//}
-
-		$tableToString = $table->toHtml();
-
-		$params = array(
-			'filename' => get_lang('Attendance') . '_' . api_get_utc_datetime(),
-			'pdf_title' => get_lang('Attendance'),
-			'course_code' => api_get_course_id(),
-			'show_real_course_teachers' => true
-		);
-		$pdf = new PDF('A4', null, $params);
-		$pdf->html_to_pdf_with_template($tableToString);
-	}
+        //}
+
+        $tableToString = $table->toHtml();
+
+        $params = array(
+            'filename' => get_lang('Attendance') . '_' . api_get_utc_datetime(),
+            'pdf_title' => get_lang('Attendance'),
+            'course_code' => api_get_course_id(),
+            'show_real_course_teachers' => true
+        );
+        $pdf = new PDF('A4', null, $params);
+        $pdf->html_to_pdf_with_template($tableToString);
+    }
 }

main/inc/lib/magpierss/rss_fetch.inc

Spacing +77 added lines, -77 removed lines

@@ -26,15 +26,15 @@ 
 }
 
 if (!defined('MAGPIE_DIR')) {
-    define('MAGPIE_DIR', dirname(__FILE__) . DIR_SEP);
+    define('MAGPIE_DIR', dirname(__FILE__).DIR_SEP);
 }
 
-require_once( MAGPIE_DIR . 'rss_parse.inc' );
-require_once( MAGPIE_DIR . 'rss_cache.inc' );
+require_once(MAGPIE_DIR.'rss_parse.inc');
+require_once(MAGPIE_DIR.'rss_cache.inc');
 
 // for including 3rd party libraries
-define('MAGPIE_EXTLIB', MAGPIE_DIR . 'extlib' . DIR_SEP);
-require_once( MAGPIE_EXTLIB . 'Snoopy.class.inc');
+define('MAGPIE_EXTLIB', MAGPIE_DIR.'extlib'.DIR_SEP);
+require_once(MAGPIE_EXTLIB.'Snoopy.class.inc');
 define('MAGPIE_CACHE_DIR', api_get_path(SYS_ARCHIVE_PATH));
 
 /* 
@@ -87,21 +87,21 @@ 
 
 $MAGPIE_ERROR = "";
 
-function fetch_rss ($url) {
+function fetch_rss($url) {
     // initialize constants
     init();
     
-    if ( !isset($url) ) {
+    if (!isset($url)) {
         error("fetch_rss called without a url");
         return false;
     }
     
     // if cache is disabled
-    if ( !MAGPIE_CACHE_ON ) {
+    if (!MAGPIE_CACHE_ON) {
         // fetch file, and parse it
-        $resp = _fetch_remote_file( $url );
-        if ( is_success( $resp->status ) ) {
-            return _response_to_rss( $resp );
+        $resp = _fetch_remote_file($url);
+        if (is_success($resp->status)) {
+            return _response_to_rss($resp);
         }
         else {
             error("Failed to fetch $url and cache is off");
@@ -116,34 +116,34 @@ 
         // 3. if cached obj fails freshness check, fetch remote
         // 4. if remote fails, return stale object, or error
         
-        $cache = new RSSCache( MAGPIE_CACHE_DIR, MAGPIE_CACHE_AGE );
+        $cache = new RSSCache(MAGPIE_CACHE_DIR, MAGPIE_CACHE_AGE);
         
         if (MAGPIE_DEBUG and $cache->ERROR) {
             debug($cache->ERROR, E_USER_WARNING);
         }
         
         
-        $cache_status    = 0;       // response of check_cache
+        $cache_status    = 0; // response of check_cache
         $request_headers = array(); // HTTP headers to send with fetch
-        $rss             = 0;       // parsed RSS object
-        $errormsg        = 0;       // errors, if any
+        $rss             = 0; // parsed RSS object
+        $errormsg        = 0; // errors, if any
         
         // store parsed XML by desired output encoding
         // as character munging happens at parse time
-        $cache_key       = $url . MAGPIE_OUTPUT_ENCODING;
+        $cache_key = $url.MAGPIE_OUTPUT_ENCODING;
         
         if (!$cache->ERROR) {
             // return cache HIT, MISS, or STALE
-            $cache_status = $cache->check_cache( $cache_key);
+            $cache_status = $cache->check_cache($cache_key);
         }
                 
         // if object cached, and cache is fresh, return cached obj
-        if ( $cache_status == 'HIT' ) {
-            $rss = $cache->get( $cache_key );
-            if ( isset($rss) and $rss ) {
+        if ($cache_status == 'HIT') {
+            $rss = $cache->get($cache_key);
+            if (isset($rss) and $rss) {
                 // should be cache age
                 $rss->from_cache = 1;
-                if ( MAGPIE_DEBUG > 1) {
+                if (MAGPIE_DEBUG > 1) {
                     debug("MagpieRSS: Cache HIT", E_USER_NOTICE);
                 }
                 return $rss;
@@ -153,50 +153,50 @@ 
         // else attempt a conditional get
         
         // setup headers
-        if ( $cache_status == 'STALE' ) {
-            $rss = $cache->get( $cache_key );
-            if ( $rss and $rss->etag and $rss->last_modified ) {
+        if ($cache_status == 'STALE') {
+            $rss = $cache->get($cache_key);
+            if ($rss and $rss->etag and $rss->last_modified) {
                 $request_headers['If-None-Match'] = $rss->etag;
                 $request_headers['If-Last-Modified'] = $rss->last_modified;
             }
         }
         
-        $resp = _fetch_remote_file( $url, $request_headers );
+        $resp = _fetch_remote_file($url, $request_headers);
         
         if (isset($resp) and $resp) {
-          if ($resp->status == '304' ) {
+          if ($resp->status == '304') {
                 // we have the most current copy
-                if ( MAGPIE_DEBUG > 1) {
+                if (MAGPIE_DEBUG > 1) {
                     debug("Got 304 for $url");
                 }
                 // reset cache on 304 (at minutillo insistent prodding)
                 $cache->set($cache_key, $rss);
                 return $rss;
             }
-            elseif ( is_success( $resp->status ) ) {
-                $rss = _response_to_rss( $resp );
-                if ( $rss ) {
+            elseif (is_success($resp->status)) {
+                $rss = _response_to_rss($resp);
+                if ($rss) {
                     if (MAGPIE_DEBUG > 1) {
                         debug("Fetch successful");
                     }
                     // add object to cache
-                    $cache->set( $cache_key, $rss );
+                    $cache->set($cache_key, $rss);
                     return $rss;
                 }
             }
             else {
                 $errormsg = "Failed to fetch $url ";
-                if ( $resp->status == '-100' ) {
-                    $errormsg .= "(Request timed out after " . MAGPIE_FETCH_TIME_OUT . " seconds)";
+                if ($resp->status == '-100') {
+                    $errormsg .= "(Request timed out after ".MAGPIE_FETCH_TIME_OUT." seconds)";
                 }
-                elseif ( $resp->error ) {
+                elseif ($resp->error) {
                     # compensate for Snoopy's annoying habbit to tacking
                     # on '\n'
                     $http_error = substr($resp->error, 0, -2); 
                     $errormsg .= "(HTTP Error: $http_error)";
                 }
                 else {
-                    $errormsg .=  "(HTTP Response: " . $resp->response_code .')';
+                    $errormsg .= "(HTTP Response: ".$resp->response_code.')';
                 }
             }
         }
@@ -208,7 +208,7 @@ 
         
         // attempt to return cached object
         if ($rss) {
-            if ( MAGPIE_DEBUG ) {
+            if (MAGPIE_DEBUG) {
                 debug("Returning STALE object for $url");
             }
             return $rss;
@@ -228,21 +228,21 @@ 
     Purpose:    set MAGPIE_ERROR, and trigger error
 \*=======================================================================*/
 
-function error ($errormsg, $lvl=E_USER_WARNING) {
+function error($errormsg, $lvl = E_USER_WARNING) {
         global $MAGPIE_ERROR;
         
         // append PHP's error message if track_errors enabled
-        if ( isset($php_errormsg) ) { 
+        if (isset($php_errormsg)) { 
             $errormsg .= " ($php_errormsg)";
         }
-        if ( $errormsg ) {
+        if ($errormsg) {
             $errormsg = "MagpieRSS: $errormsg";
             $MAGPIE_ERROR = $errormsg;
-            trigger_error( $errormsg, $lvl);                
+            trigger_error($errormsg, $lvl);                
         }
 }
 
-function debug ($debugmsg, $lvl=E_USER_NOTICE) {
+function debug($debugmsg, $lvl = E_USER_NOTICE) {
     trigger_error("MagpieRSS [debug] $debugmsg", $lvl);
 }
             
@@ -250,10 +250,10 @@ 
     Function:   magpie_error
     Purpose:    accessor for the magpie error variable
 \*=======================================================================*/
-function magpie_error ($errormsg="") {
+function magpie_error($errormsg = "") {
     global $MAGPIE_ERROR;
     
-    if ( isset($errormsg) and $errormsg ) { 
+    if (isset($errormsg) and $errormsg) { 
         $MAGPIE_ERROR = $errormsg;
     }
     
@@ -267,13 +267,13 @@ 
                 headers to send along with the request (optional)
     Output:     an HTTP response object (see Snoopy.class.inc)  
 \*=======================================================================*/
-function _fetch_remote_file ($url, $headers = "" ) {
+function _fetch_remote_file($url, $headers = "") {
     // Snoopy is an HTTP client in PHP
     $client = new Snoopy();
     $client->agent = MAGPIE_USER_AGENT;
     $client->read_timeout = MAGPIE_FETCH_TIME_OUT;
     $client->use_gzip = MAGPIE_USE_GZIP;
-    if (is_array($headers) ) {
+    if (is_array($headers)) {
         $client->rawheaders = $headers;
     }
     
@@ -288,14 +288,14 @@ 
     Input:      an HTTP response object (see Snoopy)
     Output:     parsed RSS object (see rss_parse)
 \*=======================================================================*/
-function _response_to_rss ($resp) {
-    $rss = new MagpieRSS( $resp->results, MAGPIE_OUTPUT_ENCODING, MAGPIE_INPUT_ENCODING, MAGPIE_DETECT_ENCODING );
+function _response_to_rss($resp) {
+    $rss = new MagpieRSS($resp->results, MAGPIE_OUTPUT_ENCODING, MAGPIE_INPUT_ENCODING, MAGPIE_DETECT_ENCODING);
     
     // if RSS parsed successfully       
-    if ( $rss and !$rss->ERROR) {
+    if ($rss and !$rss->ERROR) {
         
         // find Etag, and Last-Modified
-        foreach($resp->headers as $h) {
+        foreach ($resp->headers as $h) {
             // 2003-03-02 - Nicola Asuni (www.tecnick.com) - fixed bug "Undefined offset: 1"
             if (strpos($h, ": ")) {
                 list($field, $val) = explode(": ", $h, 2);
@@ -305,11 +305,11 @@ 
                 $val = "";
             }
             
-            if ( $field == 'ETag' ) {
+            if ($field == 'ETag') {
                 $rss->etag = $val;
             }
             
-            if ( $field == 'Last-Modified' ) {
+            if ($field == 'Last-Modified') {
                 $rss->last_modified = $val;
             }
         }
@@ -320,9 +320,9 @@ 
         $errormsg = "Failed to parse RSS file.";
         
         if ($rss) {
-            $errormsg .= " (" . $rss->ERROR . ")";
+            $errormsg .= " (".$rss->ERROR.")";
         }
-        error($errormsg,E_USER_NOTICE);
+        error($errormsg, E_USER_NOTICE);
         
         return false;
     } // end if ($rss and !$rss->error)
@@ -333,65 +333,65 @@ 
     Purpose:    setup constants with default values
                 check for user overrides
 \*=======================================================================*/
-function init () {
-    if ( defined('MAGPIE_INITALIZED') ) {
+function init() {
+    if (defined('MAGPIE_INITALIZED')) {
         return;
     }
     else {
         define('MAGPIE_INITALIZED', true);
     }
     
-    if ( !defined('MAGPIE_CACHE_ON') ) {
+    if (!defined('MAGPIE_CACHE_ON')) {
         define('MAGPIE_CACHE_ON', true);
     }
 
-    if ( !defined('MAGPIE_CACHE_DIR') ) {
+    if (!defined('MAGPIE_CACHE_DIR')) {
         define('MAGPIE_CACHE_DIR', './cache');
     }
 
-    if ( !defined('MAGPIE_CACHE_AGE') ) {
-        define('MAGPIE_CACHE_AGE', 60*60); // one hour
+    if (!defined('MAGPIE_CACHE_AGE')) {
+        define('MAGPIE_CACHE_AGE', 60 * 60); // one hour
     }
 
-    if ( !defined('MAGPIE_CACHE_FRESH_ONLY') ) {
+    if (!defined('MAGPIE_CACHE_FRESH_ONLY')) {
         define('MAGPIE_CACHE_FRESH_ONLY', false);
     }
 
-    if ( !defined('MAGPIE_OUTPUT_ENCODING') ) {
+    if (!defined('MAGPIE_OUTPUT_ENCODING')) {
         define('MAGPIE_OUTPUT_ENCODING', 'UTF-8');
     }
     
-    if ( !defined('MAGPIE_INPUT_ENCODING') ) {
+    if (!defined('MAGPIE_INPUT_ENCODING')) {
         define('MAGPIE_INPUT_ENCODING', null);
     }
     
-    if ( !defined('MAGPIE_DETECT_ENCODING') ) {
+    if (!defined('MAGPIE_DETECT_ENCODING')) {
         define('MAGPIE_DETECT_ENCODING', true);
     }
     
-    if ( !defined('MAGPIE_DEBUG') ) {
+    if (!defined('MAGPIE_DEBUG')) {
         define('MAGPIE_DEBUG', 0);
     }
     
-    if ( !defined('MAGPIE_USER_AGENT') ) {
-        $ua = 'MagpieRSS/'. MAGPIE_VERSION . ' (+http://magpierss.sf.net';
+    if (!defined('MAGPIE_USER_AGENT')) {
+        $ua = 'MagpieRSS/'.MAGPIE_VERSION.' (+http://magpierss.sf.net';
         
-        if ( MAGPIE_CACHE_ON ) {
-            $ua = $ua . ')';
+        if (MAGPIE_CACHE_ON) {
+            $ua = $ua.')';
         }
         else {
-            $ua = $ua . '; No cache)';
+            $ua = $ua.'; No cache)';
         }
         
         define('MAGPIE_USER_AGENT', $ua);
     }
     
-    if ( !defined('MAGPIE_FETCH_TIME_OUT') ) {
+    if (!defined('MAGPIE_FETCH_TIME_OUT')) {
         define('MAGPIE_FETCH_TIME_OUT', 5); // 5 second timeout
     }
     
     // use gzip encoding to fetch rss files if supported?
-    if ( !defined('MAGPIE_USE_GZIP') ) {
+    if (!defined('MAGPIE_USE_GZIP')) {
         define('MAGPIE_USE_GZIP', true);    
     }
 }
@@ -414,7 +414,7 @@ 
     Function:   is_info
     Purpose:    return true if Informational status code
 \*=======================================================================*/
-function is_info ($sc) { 
+function is_info($sc) { 
     return $sc >= 100 && $sc < 200; 
 }
 
@@ -422,7 +422,7 @@ 
     Function:   is_success
     Purpose:    return true if Successful status code
 \*=======================================================================*/
-function is_success ($sc) { 
+function is_success($sc) { 
     return $sc >= 200 && $sc < 300; 
 }
 
@@ -430,7 +430,7 @@ 
     Function:   is_redirect
     Purpose:    return true if Redirection status code
 \*=======================================================================*/
-function is_redirect ($sc) { 
+function is_redirect($sc) { 
     return $sc >= 300 && $sc < 400; 
 }
 
@@ -438,7 +438,7 @@ 
     Function:   is_error
     Purpose:    return true if Error status code
 \*=======================================================================*/
-function is_error ($sc) { 
+function is_error($sc) { 
     return $sc >= 400 && $sc < 600; 
 }
 
@@ -446,7 +446,7 @@ 
     Function:   is_client_error
     Purpose:    return true if Error status code, and its a client error
 \*=======================================================================*/
-function is_client_error ($sc) { 
+function is_client_error($sc) { 
     return $sc >= 400 && $sc < 500; 
 }
 
@@ -454,6 +454,6 @@ 
     Function:   is_client_error
     Purpose:    return true if Error status code, and its a server error
 \*=======================================================================*/
-function is_server_error ($sc) { 
+function is_server_error($sc) { 
     return $sc >= 500 && $sc < 600; 
 }

Braces +9 added lines, -18 removed lines

@@ -102,8 +102,7 @@ 
         $resp = _fetch_remote_file( $url );
         if ( is_success( $resp->status ) ) {
             return _response_to_rss( $resp );
-        }
-        else {
+        } else {
             error("Failed to fetch $url and cache is off");
             return false;
         }
@@ -172,8 +171,7 @@ 
                 // reset cache on 304 (at minutillo insistent prodding)
                 $cache->set($cache_key, $rss);
                 return $rss;
-            }
-            elseif ( is_success( $resp->status ) ) {
+            } elseif ( is_success( $resp->status ) ) {
                 $rss = _response_to_rss( $resp );
                 if ( $rss ) {
                     if (MAGPIE_DEBUG > 1) {
@@ -183,24 +181,20 @@ 
                     $cache->set( $cache_key, $rss );
                     return $rss;
                 }
-            }
-            else {
+            } else {
                 $errormsg = "Failed to fetch $url ";
                 if ( $resp->status == '-100' ) {
                     $errormsg .= "(Request timed out after " . MAGPIE_FETCH_TIME_OUT . " seconds)";
-                }
-                elseif ( $resp->error ) {
+                } elseif ( $resp->error ) {
                     # compensate for Snoopy's annoying habbit to tacking
                     # on '\n'
                     $http_error = substr($resp->error, 0, -2); 
                     $errormsg .= "(HTTP Error: $http_error)";
-                }
-                else {
+                } else {
                     $errormsg .=  "(HTTP Response: " . $resp->response_code .')';
                 }
             }
-        }
-        else {
+        } else {
             $errormsg = "Unable to retrieve RSS file for unknown reasons.";
         }
         
@@ -299,8 +293,7 @@ 
             // 2003-03-02 - Nicola Asuni (www.tecnick.com) - fixed bug "Undefined offset: 1"
             if (strpos($h, ": ")) {
                 list($field, $val) = explode(": ", $h, 2);
-            }
-            else {
+            } else {
                 $field = $h;
                 $val = "";
             }
@@ -336,8 +329,7 @@ 
 function init () {
     if ( defined('MAGPIE_INITALIZED') ) {
         return;
-    }
-    else {
+    } else {
         define('MAGPIE_INITALIZED', true);
     }
     
@@ -378,8 +370,7 @@ 
         
         if ( MAGPIE_CACHE_ON ) {
             $ua = $ua . ')';
-        }
-        else {
+        } else {
             $ua = $ua . '; No cache)';
         }
         

Indentation +1 added lines, -1 removed lines

@@ -164,7 +164,7 @@
         $resp = _fetch_remote_file( $url, $request_headers );
         
         if (isset($resp) and $resp) {
-          if ($resp->status == '304' ) {
+            if ($resp->status == '304' ) {
                 // we have the most current copy
                 if ( MAGPIE_DEBUG > 1) {
                     debug("Got 304 for $url");

Doc Comments +9 added lines

@@ -228,6 +228,9 @@ 
     Purpose:    set MAGPIE_ERROR, and trigger error
 \*=======================================================================*/
 
+/**
+ * @param integer $lvl
+ */
 function error ($errormsg, $lvl=E_USER_WARNING) {
         global $MAGPIE_ERROR;
         
@@ -242,6 +245,9 @@ 
         }
 }
 
+/**
+ * @param integer $lvl
+ */
 function debug ($debugmsg, $lvl=E_USER_NOTICE) {
     trigger_error("MagpieRSS [debug] $debugmsg", $lvl);
 }
@@ -288,6 +294,9 @@ 
     Input:      an HTTP response object (see Snoopy)
     Output:     parsed RSS object (see rss_parse)
 \*=======================================================================*/
+/**
+ * @param Snoopy $resp
+ */
 function _response_to_rss ($resp) {
     $rss = new MagpieRSS( $resp->results, MAGPIE_OUTPUT_ENCODING, MAGPIE_INPUT_ENCODING, MAGPIE_DETECT_ENCODING );