Merge pull request #1604 from sistemasases/fix_2019_reports

Moved function fix from grade_categories to general use, applied fix …
sistemasases · Jul 15, 2019 · 7280120 · 7280120
2 parents 6b4848c + 25266fd
commit 7280120
Show file tree

Hide file tree

Showing 5 changed files with 63 additions and 43 deletions.
diff --git a/managers/course_and_teacher_report/course_and_teacher_report_lib.php b/managers/course_and_teacher_report/course_and_teacher_report_lib.php
@@ -45,15 +45,9 @@ class ItemReporteCursoProfesores {};
 
 function get_reporte_curso_profesores($id_instancia) {
     global $DB;
-    $semestre_object = get_current_semester();
-    $sem = $semestre_object->nombre;
-    $año = substr($sem,0,4);
+    $semestre = get_current_semester_processed();
 
-    if(substr($sem,4,1) == 'A'){
-        $semestre = $año.'02';
-    }else if(substr($sem,4,1) == 'B'){
-        $semestre = $año.'08';
-    }
+    //print_r($semestre);
 
     $sql = <<<SQL
  SELECT DISTINCT ON ( moodle_course.curso_id ) 

diff --git a/managers/grade_categories/grade_categories_lib.php b/managers/grade_categories/grade_categories_lib.php
@@ -43,17 +43,7 @@
 ///*** Get info grade_categories methods ***///
 ///******************************************///
 
-    /**
-     * Función que retorna la fecha de inicio del semestre actual
-     * @see get_current_semester()
-     * @return cadena de texto que representa la fecha de inicio del semestre actual
-     */
-function get_current_semester_start(){
-    global $DB;
-    $sql_query = "SELECT fecha_inicio AS fecha FROM {talentospilos_semestre} WHERE id = (SELECT MAX(id) FROM {talentospilos_semestre})";
-    $current_semester = $DB->get_record_sql($sql_query);
-    return $current_semester;
-}
+
     /**
      * Obtains all courses organized by their teacher where there are students from an instance
      *
@@ -65,22 +55,8 @@ function get_current_semester_start(){
 function get_courses_pilos($instanceid){
     global $DB;
 
-    $sem = get_current_semester_start();
-    $semestre = $sem->fecha;
-
-    //print_r("raw: ");
-    print_r($semestre);
-
-    $año = substr($semestre,0,4);
-    $mes = substr($semestre,5,2);
-    //print_r("año: ");
-    //print_r($año);
-    //print_r("mes: ");
-    //print_r($mes);
-
-    $semestre = $año.$mes;
+    $semestre = get_current_semester_processed();
 
-    //print_r("semestre: ");
     //print_r($semestre);
     $query_courses = "
         SELECT DISTINCT curso.id,

diff --git a/managers/periods_management/periods_lib.php b/managers/periods_management/periods_lib.php
@@ -91,6 +91,35 @@ function get_current_semester(){
      return $current_semester;
  }
 
+/**
+ * Función que retorna la fecha de inicio del semestre actual
+ * @see get_current_semester()
+ * @return cadena de texto que representa la fecha de inicio del semestre actual
+ */
+function get_current_semester_start(){
+    global $DB;
+    $sql_query = "SELECT fecha_inicio AS fecha FROM {talentospilos_semestre} WHERE id = (SELECT MAX(id) FROM {talentospilos_semestre})";
+    $current_semester = $DB->get_record_sql($sql_query);
+    return $current_semester;
+}
+
+/**
+ * Función que retorna la fecha de inicio del semestre actual en el formato #AÑO#MES
+ * @see get_current_semester_start()
+ * @return cadena de texto que representa la fecha de inicio del semestre actual con formato procesado
+ */
+function get_current_semester_processed(){
+    $sem = get_current_semester_start();
+    $semestre = $sem->fecha;
+
+    $año = substr($semestre,0,4);
+    $mes = substr($semestre,5,2);
+
+    $semestre = $año.$mes;
+
+    return $semestre;
+}
+
  /**
  * Function that returns the current semester.
  * @author Jeison Cardona Gómez <[email protected]>.

diff --git a/managers/student_grades/student_item_grades_report_lib.php b/managers/student_grades/student_item_grades_report_lib.php
@@ -63,15 +63,7 @@ function get_losed_and_aproved_item_grades($id_instancia, $semestre = null) {
     global $DB;
 
     if(!$semestre) {
-        $semestre_object = get_current_semester();
-        $sem = $semestre_object->nombre;
-        $anio = substr($sem,0,4);
-
-        if(substr($sem,4,1) == 'A'){
-            $semestre = $anio.'02';
-        }else if(substr($sem,4,1) == 'B'){
-            $semestre = $anio.'08';
-        }
+        $semestre = get_current_semester_processed();
     }
 
     $sql = <<<SQL

diff --git a/managers/view_management/validate_profile_action.php b/managers/view_management/validate_profile_action.php
@@ -50,6 +50,35 @@ function get_current_semester(){
         $current_semester = $DB->get_record_sql($sql_query);
         return $current_semester;
     }
+
+    /**
+     * Función que retorna la fecha de inicio del semestre actual
+     * @see get_current_semester()
+     * @return cadena de texto que representa la fecha de inicio del semestre actual
+     */
+    function get_current_semester_start(){
+        global $DB;
+        $sql_query = "SELECT fecha_inicio AS fecha FROM {talentospilos_semestre} WHERE id = (SELECT MAX(id) FROM {talentospilos_semestre})";
+        $current_semester = $DB->get_record_sql($sql_query);
+        return $current_semester;
+    }
+
+    /**
+     * Función que retorna la fecha de inicio del semestre actual en el formato #AÑO#MES
+     * @see get_current_semester_start()
+     * @return cadena de texto que representa la fecha de inicio del semestre actual con formato procesado
+     */
+    function get_current_semester_processed(){
+        $sem = get_current_semester_start();
+        $semestre = $sem->fecha;
+
+        $año = substr($semestre,0,4);
+        $mes = substr($semestre,5,2);
+
+        $semestre = $año.$mes;
+
+        return $semestre;
+    }
 
     /**
      * Retorna el id de talentos a partir del id de moodle