MDL-61957 gradereport_user: Implement privacy API
authorSara Arjona <sara@moodle.com>
Thu, 12 Apr 2018 08:38:01 +0000 (10:38 +0200)
committerSara Arjona <sara@moodle.com>
Tue, 17 Apr 2018 12:31:54 +0000 (14:31 +0200)
grade/report/user/classes/privacy/provider.php [new file with mode: 0644]
grade/report/user/lang/en/gradereport_user.php
grade/report/user/tests/privacy_test.php [new file with mode: 0644]

diff --git a/grade/report/user/classes/privacy/provider.php b/grade/report/user/classes/privacy/provider.php
new file mode 100644 (file)
index 0000000..d569dc6
--- /dev/null
@@ -0,0 +1,76 @@
+<?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/>.
+
+/**
+ * Privacy Subsystem implementation for gradereport_user.
+ *
+ * @package    gradereport_user
+ * @copyright  2018 Sara Arjona <sara@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+namespace gradereport_user\privacy;
+
+defined('MOODLE_INTERNAL') || die();
+
+use \core_privacy\local\metadata\collection;
+use \core_privacy\local\request\transform;
+use \core_privacy\local\request\writer;
+
+require_once $CFG->libdir.'/grade/constants.php';
+
+
+/**
+ * Privacy Subsystem for gradereport_user implementing null_provider.
+ *
+ * @copyright  2018 Sara Arjona <sara@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class provider implements
+    \core_privacy\local\metadata\provider,
+    \core_privacy\local\request\user_preference_provider {
+
+    /**
+     * Returns meta data about this system.
+     *
+     * @param   collection     $itemcollection The initialised item collection to add items to.
+     * @return  collection     A listing of user data stored through this system.
+     */
+    public static function get_metadata(collection $items) : collection {
+        // User preferences (shared between different courses).
+        $items->add_user_preference('gradereport_user_view_user', 'privacy:metadata:preference:gradereport_user_view_user');
+
+        return $items;
+    }
+
+    /**
+     * Store all user preferences for the plugin.
+     *
+     * @param   int         $userid The userid of the user whose data is to be exported.
+     */
+    public static function export_user_preferences(int $userid) {
+        $prefvalue = get_user_preferences('gradereport_user_view_user', null, $userid);
+        if ($prefvalue !== null) {
+            $transformedvalue = transform::yesno($prefvalue);
+            writer::export_user_preference(
+                'gradereport_user',
+                'gradereport_user_view_user',
+                $transformedvalue,
+                get_string('privacy:metadata:preference:gradereport_user_view_user', 'gradereport_user')
+            );
+        }
+    }
+}
index 953b1e7..e30cbde 100644 (file)
@@ -27,5 +27,6 @@ $string['pluginname'] = 'User report';
 $string['user:view'] = 'View your own grade report';
 $string['myself'] = 'Myself';
 $string['otheruser'] = 'User';
+$string['privacy:metadata:preference:gradereport_user_view_user'] = 'Whether to view report as current user or another user in the gradebook reports';
 $string['tablesummary'] = 'The table is arranged as a list of graded items including categories of graded items. When items are in a category they will be indicated as such.';
 $string['viewas'] = 'View report as';
diff --git a/grade/report/user/tests/privacy_test.php b/grade/report/user/tests/privacy_test.php
new file mode 100644 (file)
index 0000000..b44f8bf
--- /dev/null
@@ -0,0 +1,81 @@
+<?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/>.
+
+/**
+ * Unit tests for the gradereport_user implementation of the privacy API.
+ *
+ * @package    gradereport_user
+ * @category   test
+ * @copyright  2018 Sara Arjona <sara@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+defined('MOODLE_INTERNAL') || die();
+
+use \core_privacy\local\metadata\collection;
+use \core_privacy\local\request\writer;
+use \gradereport_user\privacy\provider;
+
+/**
+ * Unit tests for the gradereport_user implementation of the privacy API.
+ *
+ * @copyright  2018 Sara Arjona <sara@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class gradereport_user_privacy_testcase extends \core_privacy\tests\provider_testcase {
+
+    /**
+     * Basic setup for these tests.
+     */
+    public function setUp() {
+        $this->resetAfterTest(true);
+    }
+
+    /**
+     * Ensure that export_user_preferences returns no data if the user has no data.
+     */
+    public function test_export_user_preferences_not_defined() {
+        $user = \core_user::get_user_by_username('admin');
+        provider::export_user_preferences($user->id);
+
+        $writer = writer::with_context(\context_system::instance());
+        $this->assertFalse($writer->has_any_data());
+    }
+
+    /**
+     * Ensure that export_user_preferences returns single preferences.
+     * These preferences can be set on each course, but the value is shared in the whole site.
+     */
+    public function test_export_user_preferences_single() {
+        // Define a user preference.
+        $user = $this->getDataGenerator()->create_user();
+        $this->setUser($user);
+        set_user_preference('gradereport_user_view_user', 1, $user);
+
+        // Validate exported data.
+        provider::export_user_preferences($user->id);
+        $context = context_user::instance($user->id);
+        $writer = writer::with_context($context);
+        $this->assertTrue($writer->has_any_data());
+        $prefs = $writer->get_user_preferences('gradereport_user');
+        $this->assertCount(1, (array) $prefs);
+        $this->assertEquals(
+            get_string('privacy:metadata:preference:gradereport_user_view_user', 'gradereport_user'),
+            $prefs->gradereport_user_view_user->description
+        );
+        $this->assertEquals(get_string('yes'), $prefs->gradereport_user_view_user->value);
+    }
+}