Fixed disk usage data issue

Author: santoshndev

classes/output/main.php

@@ -94,12 +94,7 @@ public function get_disk_usage() {
             return get_string('notcalculated', 'block_site_stats');
         }
 
-        $unit = get_string('sizemb', 'block_site_stats');
-        if ($diskusagecache > 1024) {
-            $unit = get_string('sizegb', 'block_site_stats');
-        }
-
-        return $diskusagecache . $unit;
+        return $diskusagecache;
     }
     /**
      * Export the data for template

classes/task/diskusage.php

@@ -44,7 +44,7 @@ public function execute() {
         global $CFG;
         $cache = cache::make('block_site_stats', 'sitestats');
         $diskusage = get_directory_size($CFG->dataroot);
-        $disksize = number_format(ceil($diskusage / 1048576));
+        $disksize = display_size($diskusage);
         $cache->set('diskusage', $disksize);
         return true;
     }

lang/en/block_site_stats.php

@@ -31,6 +31,4 @@
 $string['privacy:metadata'] = 'The Site Statistics block only shows data stored in other locations.';
 $string['site_stats:myaddinstance'] = 'Add a new Site Statistics block to Dashboard';
 $string['site_stats:view'] = 'View Site Statistics block content';
-$string['sizegb'] = ' GB';
-$string['sizemb'] = ' MB';
 $string['users'] = 'Users';

lang/hi/block_site_stats.php

@@ -31,6 +31,4 @@
 $string['privacy:metadata'] = 'साइट आँकड़े ब्लॉक केवल अन्य स्थानों में संग्रहित डेटा दिखाता है।';
 $string['site_stats:myaddinstance'] = 'डैशबोर्ड पर नया साइट आँकड़ा ब्लॉक जोड़ें';
 $string['site_stats:view'] = 'साइट आँकड़े ब्लॉक की सामग्री देखें';
-$string['sizegb'] = ' जीबी';
-$string['sizemb'] = ' एमबी';
 $string['users'] = 'उपयोगकर्ता';

lang/mr/block_site_stats.php

@@ -31,6 +31,4 @@
 $string['privacy:metadata'] = 'साइट सांख्यिकी ब्लॉक फक्त इतर स्थानांतील डेटा दर्शवितो.';
 $string['site_stats:myaddinstance'] = 'डॅशबोर्डवर नविन साइट सांख्यिकी ब्लॉक जोडा';
 $string['site_stats:view'] = 'साइट सांख्यिकी ब्लॉक सामग्री पहा';
-$string['sizegb'] = ' जीबी';
-$string['sizemb'] = ' एमबी';
 $string['users'] = 'वापरकर्ते';

tests/output/main_test.php

@@ -95,20 +95,17 @@ public function test_get_activities_count(): void {
      * @return void
      */
     public function test_get_disk_usage(): void {
-        global $CFG;
         // Test when the diskusage scheduled task is not run.
         $diskusage = $this->main->get_disk_usage();
         $expected = get_string('notcalculated', 'block_site_stats');
         $this->assertEquals($expected, $diskusage);
 
         // Test when the diskusage scheduled task is run.
-        $diskusage = get_directory_size($CFG->dataroot);
-        $disksize = number_format(ceil($diskusage / 1048576));
         $task = new diskusage();
         $status = $task->execute();
         if ($status) {
             $diskusage = $this->main->get_disk_usage();
-            $this->assertEquals($disksize . ' MB', $diskusage);
+            $this->assertNotEquals($expected, $diskusage);
         }
     }
 }

version.php

@@ -1,31 +1,31 @@
-<?php
-// 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/>.
-
-/**
- * Version details
- *
- * @package    block_site_stats
- * @copyright  2024 Santosh Nagargoje <santosh.nag2217@gmail.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-defined('MOODLE_INTERNAL') || die();
-
-$plugin->version   = 2024050403;        // The current plugin version (Date: YYYYMMDDXX).
-$plugin->release = '1.0.1';
-$plugin->requires  = 2022111800;        // Requires this Moodle version.
-$plugin->component = 'block_site_stats'; // Full name of the plugin (used for diagnostics).
-$plugin->maturity = MATURITY_STABLE;
+<?php
+// 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/>.
+
+/**
+ * Version details
+ *
+ * @package    block_site_stats
+ * @copyright  2024 Santosh Nagargoje <santosh.nag2217@gmail.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+defined('MOODLE_INTERNAL') || die();
+
+$plugin->version   = 2024050404;        // The current plugin version (Date: YYYYMMDDXX).
+$plugin->release = '1.0.2';
+$plugin->requires  = 2022111800;        // Requires this Moodle version.
+$plugin->component = 'block_site_stats'; // Full name of the plugin (used for diagnostics).
+$plugin->maturity = MATURITY_STABLE;