Revert "Fixed Naming convention in plugin"

This reverts commit 9fcbb2d.

Author: waleedhassan5

classes/task/cleanup_students.php

@@ -18,20 +18,20 @@
  * Scheduled task for managing student accounts lifecycle.
  *
  * This task handles the automated process of disabling and/or deleting student accounts
- * based on configured criteria within the disable_delete_students plugin.
+ * based on configured criteria within the tool_disable_delete_students plugin.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>
  */
 
-namespace disable_delete_students\task;
+namespace tool_disable_delete_students\task;
 
 /**
  * Scheduled task class for cleaning up student accounts.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  */
 class cleanup_students extends \core\task\scheduled_task {
 
@@ -42,7 +42,7 @@ class cleanup_students extends \core\task\scheduled_task {
      * @throws \coding_exception
      */
     public function get_name() {
-        return get_string('taskname', 'disable_delete_students');
+        return get_string('taskname', 'tool_disable_delete_students');
     }
 
     /**
@@ -54,6 +54,6 @@ public function get_name() {
      * @return void
      */
     public function execute() {
-        \disable_delete_students\util::process_student_accounts();
+        \tool_disable_delete_students\util::process_student_accounts();
     }
 }

classes/util.php

@@ -22,13 +22,13 @@
  * - Processing account disabling and deletion based on configured rules
  * - Handling role-based exclusions
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>
  */
 
-namespace disable_delete_students;
+namespace tool_disable_delete_students;
 
 defined('MOODLE_INTERNAL') || die();
 
@@ -82,9 +82,9 @@ public static function has_excluded_roles(int $userid): bool {
     public static function process_student_accounts() {
         global $DB, $CFG;
 
-        $disableaftercourseend = get_config('disable_delete_students', 'disable_after_course_end');
-        $disableaftercreation = get_config('disable_delete_students', 'disable_after_creation');
-        $deleteaftermonths = get_config('disable_delete_students', 'delete_after_months');
+        $disableaftercourseend = get_config('tool_disable_delete_students', 'disable_after_course_end');
+        $disableaftercreation = get_config('tool_disable_delete_students', 'disable_after_creation');
+        $deleteaftermonths = get_config('tool_disable_delete_students', 'delete_after_months');
 
         // Get all active student accounts.
         $sql = "SELECT DISTINCT u.* 

db/tasks.php

@@ -15,24 +15,24 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Scheduled task definitions for the disable_delete_students plugin.
+ * Scheduled task definitions for the tool_disable_delete_students plugin.
  *
  * This file contains the configuration for scheduled tasks related to
  * the management of student accounts, specifically for disabling accounts
  * based on defined criteria.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>
  */
 
 defined('MOODLE_INTERNAL') || die();
 
-// Define the scheduled tasks for the disable_delete_students plugin.
+// Define the scheduled tasks for the tool_disable_delete_students plugin.
 $tasks = [
     [
-        'classname' => 'disable_delete_students\task\cleanup_students',
+        'classname' => 'tool_disable_delete_students\task\cleanup_students',
         'blocking' => 0,
         'minute' => '0',
         'hour' => '0',

lang/en/disable_delete_students.php lang/en/tool_disable_delete_students.php

@@ -15,13 +15,13 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Language strings for the disable_delete_students plugin.
+ * Language strings for the tool_disable_delete_students plugin.
  *
  * This file contains the language strings used in the plugin for managing
  * student account lifecycle operations, including disabling and deleting
  * accounts based on defined criteria.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>

settings.php

@@ -15,9 +15,9 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Settings for the disable_delete_students plugin.
+ * Settings for the tool_disable_delete_students plugin.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>
@@ -26,32 +26,32 @@
 defined('MOODLE_INTERNAL') || die;
 
 if ($hassiteconfig) {
-    $settings = new admin_settingpage('disable_delete_students', get_string('pluginname', 'disable_delete_students'));
+    $settings = new admin_settingpage('tool_disable_delete_students', get_string('pluginname', 'tool_disable_delete_students'));
     $ADMIN->add('tools', $settings);
 
     // Days after course end to disable account.
     $settings->add(new admin_setting_configtext(
-        'disable_delete_students/disable_after_course_end',
-        get_string('disable_after_course_end', 'disable_delete_students'),
-        get_string('disable_after_course_end_desc', 'disable_delete_students'),
+        'tool_disable_delete_students/disable_after_course_end',
+        get_string('disable_after_course_end', 'tool_disable_delete_students'),
+        get_string('disable_after_course_end_desc', 'tool_disable_delete_students'),
         21,
         PARAM_INT
     ));
 
     // Days after creation to disable account.
     $settings->add(new admin_setting_configtext(
-        'disable_delete_students/disable_after_creation',
-        get_string('disable_after_creation', 'disable_delete_students'),
-        get_string('disable_after_creation_desc', 'disable_delete_students'),
+        'tool_disable_delete_students/disable_after_creation',
+        get_string('disable_after_creation', 'tool_disable_delete_students'),
+        get_string('disable_after_creation_desc', 'tool_disable_delete_students'),
         45,
         PARAM_INT
     ));
 
     // Months after course end to delete account.
     $settings->add(new admin_setting_configtext(
-        'disable_delete_students/delete_after_months',
-        get_string('delete_after_months', 'disable_delete_students'),
-        get_string('delete_after_months_desc', 'disable_delete_students'),
+        'tool_disable_delete_students/delete_after_months',
+        get_string('delete_after_months', 'tool_disable_delete_students'),
+        get_string('delete_after_months_desc', 'tool_disable_delete_students'),
         6,
         PARAM_INT
     ));

tests/cleanup_test.php

@@ -15,17 +15,17 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Unit tests for the disable_delete_students plugin.
+ * Unit tests for the tool_disable_delete_students plugin.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @category   test
  * @copyright  2024 Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-namespace disable_delete_students;
+namespace tool_disable_delete_students;
 
 /**
- * Unit tests for the disable_delete_students plugin.
+ * Unit tests for the tool_disable_delete_students plugin.
  *
  * The tests include:
  * - Verification of excluded roles
@@ -35,7 +35,7 @@
  * - Ensuring users with excluded roles are not affected
  * - Handling of students enrolled in multiple courses
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @category   test
  * @copyright  2024 Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
@@ -59,9 +59,9 @@ protected function setUp(): void {
         parent::setUp();
         $this->resetAfterTest(true);
         // Set plugin configurations.
-        set_config('disable_after_course_end', 21, 'disable_delete_students');
-        set_config('disable_after_creation', 45, 'disable_delete_students');
-        set_config('delete_after_months', 6, 'disable_delete_students');
+        set_config('disable_after_course_end', 21, 'tool_disable_delete_students');
+        set_config('disable_after_creation', 45, 'tool_disable_delete_students');
+        set_config('delete_after_months', 6, 'tool_disable_delete_students');
 
         // Retrieve system roles.
         $this->studentrole = $DB->get_record('role', ['shortname' => 'student']);

tests/generator/lib.php

@@ -15,12 +15,12 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Generator for the disable_delete_students plugin.
+ * Generator for the tool_disable_delete_students plugin.
  *
  * This class provides methods for creating test courses and users
  * with specific roles for testing purposes.
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>
@@ -29,10 +29,10 @@
 /**
  * Tool disable_delete_students generator
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @category   test
  */
-class disable_delete_students_generator extends testing_module_generator {
+class tool_disable_delete_students_generator extends testing_module_generator {
     /**
      * Create a test course with specific settings.
      *

version.php

@@ -15,10 +15,10 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Version information for the disable_delete_students plugin.
+ * Version information for the tool_disable_delete_students plugin.
 
  *
- * @package    disable_delete_students
+ * @package    tool_disable_delete_students
  * @copyright  2024 onwards Catalyst IT {@link http://www.catalyst-eu.net/}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  * @author     Waleed ul hassan <[email protected]>
@@ -28,6 +28,6 @@
 
 $plugin->version   = 2024021500;
 $plugin->requires  = 2022112800; // Moodle 4.1
-$plugin->component = 'disable_delete_students';
+$plugin->component = 'tool_disable_delete_students';
 $plugin->maturity  = MATURITY_STABLE;
 $plugin->release   = '1.0';