From a7c6686c7293d1235f8983c1fe9cc0d1f16eb9fe Mon Sep 17 00:00:00 2001
From: John Tranier <john.tranier@isae-supaero.fr>
Date: Mon, 4 Mar 2024 12:00:25 +0100
Subject: [PATCH 1/3] Restore <?php tags

---
 hybridmeter/classes/cache_manager.php                            | 1 +
 hybridmeter/classes/configurator.php                             | 1 +
 hybridmeter/classes/data/general_data.php                        | 1 +
 hybridmeter/classes/data_provider.php                            | 1 +
 hybridmeter/classes/exporter.php                                 | 1 +
 hybridmeter/classes/formatter.php                                | 1 +
 hybridmeter/classes/logger.php                                   | 1 +
 hybridmeter/classes/processing.php                               | 1 +
 hybridmeter/classes/task/cron_processing.php                     | 1 +
 hybridmeter/classes/task/cron_scheduler.php                      | 1 +
 hybridmeter/classes/task/processing.php                          | 1 +
 hybridmeter/classes/test_context.php                             | 1 +
 hybridmeter/classes/test_scenario.php                            | 1 +
 hybridmeter/classes/test_scenario_course.php                     | 1 +
 hybridmeter/classes/tests/ND/inconsistent_nd.php                 | 1 +
 hybridmeter/classes/tests/NU/inconsistent_nu.php                 | 1 +
 hybridmeter/classes/tests/blacklist_scenario.php                 | 1 +
 .../classes/tests/course_count/inconsistent_active_students.php  | 1 +
 .../course_count/inconsistent_registered_active_students.php     | 1 +
 .../tests/course_count/inconsistent_registered_students.php      | 1 +
 hybridmeter/classes/tests/course_count_abstract.php              | 1 +
 hybridmeter/classes/tests/indicator_abstract.php                 | 1 +
 hybridmeter/classes/utils.php                                    | 1 +
 hybridmeter/constants.php                                        | 1 +
 hybridmeter/db/access.php                                        | 1 +
 hybridmeter/db/install.php                                       | 1 +
 hybridmeter/db/tasks.php                                         | 1 +
 hybridmeter/db/uninstall.php                                     | 1 +
 hybridmeter/db/upgrade.php                                       | 1 +
 hybridmeter/indicators.php                                       | 1 +
 hybridmeter/output/index_page.php                                | 1 +
 hybridmeter/output/renderer.php                                  | 1 +
 hybridmeter/version.php                                          | 1 +
 33 files changed, 33 insertions(+)

diff --git a/hybridmeter/classes/cache_manager.php b/hybridmeter/classes/cache_manager.php
index 2eba14f..f219d91 100644
--- a/hybridmeter/classes/cache_manager.php
+++ b/hybridmeter/classes/cache_manager.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/configurator.php b/hybridmeter/classes/configurator.php
index ede91cc..9f5d68b 100644
--- a/hybridmeter/classes/configurator.php
+++ b/hybridmeter/classes/configurator.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/data/general_data.php b/hybridmeter/classes/data/general_data.php
index 3cc94f9..39116fb 100644
--- a/hybridmeter/classes/data/general_data.php
+++ b/hybridmeter/classes/data/general_data.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/data_provider.php b/hybridmeter/classes/data_provider.php
index 5ee81eb..5703283 100644
--- a/hybridmeter/classes/data_provider.php
+++ b/hybridmeter/classes/data_provider.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/exporter.php b/hybridmeter/classes/exporter.php
index 20a6b1e..30eb300 100644
--- a/hybridmeter/classes/exporter.php
+++ b/hybridmeter/classes/exporter.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/formatter.php b/hybridmeter/classes/formatter.php
index 7f03465..9745382 100644
--- a/hybridmeter/classes/formatter.php
+++ b/hybridmeter/classes/formatter.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/logger.php b/hybridmeter/classes/logger.php
index 971a8c9..eaebabe 100644
--- a/hybridmeter/classes/logger.php
+++ b/hybridmeter/classes/logger.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/processing.php b/hybridmeter/classes/processing.php
index d2766eb..fde28eb 100644
--- a/hybridmeter/classes/processing.php
+++ b/hybridmeter/classes/processing.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/task/cron_processing.php b/hybridmeter/classes/task/cron_processing.php
index e114dc6..1f06bdb 100644
--- a/hybridmeter/classes/task/cron_processing.php
+++ b/hybridmeter/classes/task/cron_processing.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/task/cron_scheduler.php b/hybridmeter/classes/task/cron_scheduler.php
index eec946f..453efa5 100644
--- a/hybridmeter/classes/task/cron_scheduler.php
+++ b/hybridmeter/classes/task/cron_scheduler.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/task/processing.php b/hybridmeter/classes/task/processing.php
index 54a6f5f..83d53a1 100644
--- a/hybridmeter/classes/task/processing.php
+++ b/hybridmeter/classes/task/processing.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/test_context.php b/hybridmeter/classes/test_context.php
index 8e8e214..3fb72d6 100644
--- a/hybridmeter/classes/test_context.php
+++ b/hybridmeter/classes/test_context.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/test_scenario.php b/hybridmeter/classes/test_scenario.php
index 7c9fb63..089b032 100644
--- a/hybridmeter/classes/test_scenario.php
+++ b/hybridmeter/classes/test_scenario.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/test_scenario_course.php b/hybridmeter/classes/test_scenario_course.php
index f254f2b..a7e675f 100644
--- a/hybridmeter/classes/test_scenario_course.php
+++ b/hybridmeter/classes/test_scenario_course.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/ND/inconsistent_nd.php b/hybridmeter/classes/tests/ND/inconsistent_nd.php
index 8207342..160aa4a 100644
--- a/hybridmeter/classes/tests/ND/inconsistent_nd.php
+++ b/hybridmeter/classes/tests/ND/inconsistent_nd.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/NU/inconsistent_nu.php b/hybridmeter/classes/tests/NU/inconsistent_nu.php
index 90da4d8..449b885 100644
--- a/hybridmeter/classes/tests/NU/inconsistent_nu.php
+++ b/hybridmeter/classes/tests/NU/inconsistent_nu.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/blacklist_scenario.php b/hybridmeter/classes/tests/blacklist_scenario.php
index 42580ff..14c8532 100644
--- a/hybridmeter/classes/tests/blacklist_scenario.php
+++ b/hybridmeter/classes/tests/blacklist_scenario.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/course_count/inconsistent_active_students.php b/hybridmeter/classes/tests/course_count/inconsistent_active_students.php
index 5485950..f0c39e6 100644
--- a/hybridmeter/classes/tests/course_count/inconsistent_active_students.php
+++ b/hybridmeter/classes/tests/course_count/inconsistent_active_students.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/course_count/inconsistent_registered_active_students.php b/hybridmeter/classes/tests/course_count/inconsistent_registered_active_students.php
index 4ee09f3..90372a2 100644
--- a/hybridmeter/classes/tests/course_count/inconsistent_registered_active_students.php
+++ b/hybridmeter/classes/tests/course_count/inconsistent_registered_active_students.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/course_count/inconsistent_registered_students.php b/hybridmeter/classes/tests/course_count/inconsistent_registered_students.php
index 6802619..a067e75 100644
--- a/hybridmeter/classes/tests/course_count/inconsistent_registered_students.php
+++ b/hybridmeter/classes/tests/course_count/inconsistent_registered_students.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/course_count_abstract.php b/hybridmeter/classes/tests/course_count_abstract.php
index 4f3711e..da647a9 100644
--- a/hybridmeter/classes/tests/course_count_abstract.php
+++ b/hybridmeter/classes/tests/course_count_abstract.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/tests/indicator_abstract.php b/hybridmeter/classes/tests/indicator_abstract.php
index bfc3a72..2ddcea5 100644
--- a/hybridmeter/classes/tests/indicator_abstract.php
+++ b/hybridmeter/classes/tests/indicator_abstract.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/classes/utils.php b/hybridmeter/classes/utils.php
index c0c7571..2de932b 100644
--- a/hybridmeter/classes/utils.php
+++ b/hybridmeter/classes/utils.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/constants.php b/hybridmeter/constants.php
index b0c61fb..dd399b5 100644
--- a/hybridmeter/constants.php
+++ b/hybridmeter/constants.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/db/access.php b/hybridmeter/db/access.php
index c291e50..f726acb 100644
--- a/hybridmeter/db/access.php
+++ b/hybridmeter/db/access.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/db/install.php b/hybridmeter/db/install.php
index 63b97fa..425a6f4 100644
--- a/hybridmeter/db/install.php
+++ b/hybridmeter/db/install.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/db/tasks.php b/hybridmeter/db/tasks.php
index 4bf4d06..dcd0cc2 100644
--- a/hybridmeter/db/tasks.php
+++ b/hybridmeter/db/tasks.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/db/uninstall.php b/hybridmeter/db/uninstall.php
index 5373496..6c15008 100644
--- a/hybridmeter/db/uninstall.php
+++ b/hybridmeter/db/uninstall.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/db/upgrade.php b/hybridmeter/db/upgrade.php
index 11829b7..d8c0cf5 100644
--- a/hybridmeter/db/upgrade.php
+++ b/hybridmeter/db/upgrade.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/indicators.php b/hybridmeter/indicators.php
index b5b223f..c9773cc 100644
--- a/hybridmeter/indicators.php
+++ b/hybridmeter/indicators.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/output/index_page.php b/hybridmeter/output/index_page.php
index d711d76..9a7f031 100644
--- a/hybridmeter/output/index_page.php
+++ b/hybridmeter/output/index_page.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/output/renderer.php b/hybridmeter/output/renderer.php
index a216095..f8a6fbf 100644
--- a/hybridmeter/output/renderer.php
+++ b/hybridmeter/output/renderer.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
diff --git a/hybridmeter/version.php b/hybridmeter/version.php
index d190d29..c2c081b 100644
--- a/hybridmeter/version.php
+++ b/hybridmeter/version.php
@@ -1,3 +1,4 @@
+<?php
 // This file is part of Moodle - http://moodle.org
 //
 // Moodle is free software: you can redistribute it and/or modify
-- 
GitLab


From 1f41a7c8b192b5ec20be48af20b52c153b53fe5f Mon Sep 17 00:00:00 2001
From: John Tranier <john.tranier@isae-supaero.fr>
Date: Mon, 4 Mar 2024 13:21:01 +0100
Subject: [PATCH 2/3] PHPCS : Restore class fields

---
 hybridmeter/classes/configurator.php      |  4 ++--
 hybridmeter/classes/data/general_data.php | 16 ++++++++--------
 hybridmeter/classes/exporter.php          |  2 +-
 hybridmeter/classes/processing.php        |  4 ++--
 hybridmeter/styles.css                    | 18 ------------------
 5 files changed, 13 insertions(+), 31 deletions(-)

diff --git a/hybridmeter/classes/configurator.php b/hybridmeter/classes/configurator.php
index 9f5d68b..7f1deae 100644
--- a/hybridmeter/classes/configurator.php
+++ b/hybridmeter/classes/configurator.php
@@ -39,9 +39,9 @@ class configurator {
 
     protected $data;
 
-    protected $begindate;
+    protected $begin_date;
 
-    protected $enddate;
+    protected $end_date;
 
     protected static $instance = null;
 
diff --git a/hybridmeter/classes/data/general_data.php b/hybridmeter/classes/data/general_data.php
index 39116fb..60753bb 100644
--- a/hybridmeter/classes/data/general_data.php
+++ b/hybridmeter/classes/data/general_data.php
@@ -27,18 +27,18 @@ use report_hybridmeter\classes\data_provider as data_provider;
 
 class general_data {
 
-    protected $begintimestamp;
-    protected $endtimestamp;
+    protected $begin_timestamp;
+    protected $end_timestamp;
 
     protected $courses;
-    protected $usedcourses;
-    protected $digitalisedcourses;
+    protected $used_courses;
+    protected $digitalised_courses;
 
-    protected $nbstudentsconcerneddigitalised;
-    protected $nbstudentsconcernedused;
+    protected $nb_students_concerned_digitalised;
+    protected $nb_students_concerned_used;
 
-    protected $nbstudentsconcerneddigitalisedactive;
-    protected $nbstudentsconcernedusedactive;
+    protected $nb_students_concerned_digitalised_active;
+    protected $nb_students_concerned_used_active;
 
     public function __construct($courses, $configurator) {
         $dataprovider = data_provider::get_instance();
diff --git a/hybridmeter/classes/exporter.php b/hybridmeter/classes/exporter.php
index 30eb300..fb12ef1 100644
--- a/hybridmeter/classes/exporter.php
+++ b/hybridmeter/classes/exporter.php
@@ -57,7 +57,7 @@ class exporter {
      * to deal with double variable processed as integer.
      *
      */
-    protected $fieldstype;
+    protected $fields_type;
 
     /* Human-readable strings associated with fields (and displayed in CSV)
      * If an alias exists then the alias is displayed, otherwise this is the raw field name
diff --git a/hybridmeter/classes/processing.php b/hybridmeter/classes/processing.php
index fde28eb..10adbe1 100644
--- a/hybridmeter/classes/processing.php
+++ b/hybridmeter/classes/processing.php
@@ -46,8 +46,8 @@ class processing {
 
     protected $formatter;
     protected $exporter;
-    protected $begindate;
-    protected $enddate;
+    protected $begin_date;
+    protected $end_date;
 
     function __construct() {
         logger::log("# Processing: initializing");
diff --git a/hybridmeter/styles.css b/hybridmeter/styles.css
index d68686e..7261360 100644
--- a/hybridmeter/styles.css
+++ b/hybridmeter/styles.css
@@ -1,26 +1,8 @@
-/*
- * This file is part of Moodle - http://moodle.org/
- *
- * Moodle is free software: you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation, either version 3 of the License, or
- * (at your option) any later version.
- *
- * Moodle is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
- */
-
 /**
  * @author Nassim Bennouar
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @copyright (C) 2020  ISAE-SUPAERO (https://www.isae-supaero.fr/)
  */
-
 #hybridmeter-app > h3 {
   margin-bottom: 20px;
   margin-left: 2px;
-- 
GitLab


From 2385818e118d1a784045f09ec5edef853326c78b Mon Sep 17 00:00:00 2001
From: John Tranier <john.tranier@isae-supaero.fr>
Date: Mon, 4 Mar 2024 13:26:40 +0100
Subject: [PATCH 3/3] Release 1.1.9

---
 CHANGES.md              | 5 +++++
 hybridmeter/version.php | 4 ++--
 2 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/CHANGES.md b/CHANGES.md
index 4017cdf..e297aa5 100644
--- a/CHANGES.md
+++ b/CHANGES.md
@@ -6,6 +6,11 @@ _Ce document décrit les modifications du plugin HybridMeter pour chaque version
 
 [Site HybridMeter](https://online.isae-supaero.fr/hybridmeter)
 
+## Hybridmeter version 1.1.9 (2024-03-04)
+### Chore
+- Fix regressions introduced by code cleaning
+
+
 ## Hybridmeter version 1.1.8 (2024-03-04)
 ### Chore
 - Fix errors raised by PHPCS
diff --git a/hybridmeter/version.php b/hybridmeter/version.php
index c2c081b..4ba4587 100644
--- a/hybridmeter/version.php
+++ b/hybridmeter/version.php
@@ -21,7 +21,7 @@
  * @package
  */
 $plugin->component = "report_hybridmeter";
-$plugin->version = 2024030400;
+$plugin->version = 2024030401;
 $plugin->requires = 2018051700; // Moodle 3.5.7 minimum
-$plugin->release = "1.1.8";
+$plugin->release = "1.1.9";
 $plugin->maturity = MATURITY_RC;
-- 
GitLab