Merge branch 'MDL-61858_34' of git://github.com/markn86/moodle into MOODLE_34_STABLE

Author: David Monllao

admin/tool/phpunit/classes/privacy/provider.php

@@ -0,0 +1,46 @@
+<?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 tool_phpunit.
+ *
+ * @package    tool_phpunit
+ * @copyright  2018 Mark Nelson <markn@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+
+namespace tool_phpunit\privacy;
+
+defined('MOODLE_INTERNAL') || die();
+
+/**
+ * Privacy Subsystem for tool_phpunit implementing null_provider.
+ *
+ * @copyright  2018 Mark Nelson <markn@moodle.com>
+ * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
+ */
+class provider implements \core_privacy\local\metadata\null_provider {
+
+    /**
+     * Get the language string identifier with the component's language
+     * file to explain why this plugin stores no data.
+     *
+     * @return  string
+     */
+    public static function get_reason() : string {
+        return 'privacy:metadata';
+    }
+}

grade/grading/form/guide/classes/privacy/provider.php

@@ -37,6 +37,7 @@
  * @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 {
 
     /**

portfolio/boxnet/classes/privacy/provider.php

@@ -36,6 +36,7 @@
 class provider implements
     // This portfolio plugin does not store any data itself.
     // It has no database tables, and it purely acts as a conduit, sending data externally.
+    \core_privacy\local\metadata\provider,
     \core_portfolio\privacy\portfolio_provider {
 
     /**

portfolio/flickr/classes/privacy/provider.php

@@ -36,6 +36,7 @@
 class provider implements
         // This portfolio plugin does not store any data itself.
         // It has no database tables, and it purely acts as a conduit, sending data externally.
+        \core_privacy\local\metadata\provider,
         \core_portfolio\privacy\portfolio_provider {
 
     /**

portfolio/googledocs/classes/privacy/provider.php

@@ -36,6 +36,7 @@
 class provider implements
         // This portfolio plugin does not store any data itself.
         // It has no database tables, and it purely acts as a conduit, sending data externally.
+        \core_privacy\local\metadata\provider,
         \core_portfolio\privacy\portfolio_provider {
 
     /**

portfolio/mahara/classes/privacy/provider.php

@@ -36,6 +36,7 @@
 class provider implements
         // This portfolio plugin does not store any data itself.
         // It has no database tables, and it purely acts as a conduit, sending data externally.
+        \core_privacy\local\metadata\provider,
         \core_portfolio\privacy\portfolio_provider {
 
     /**

portfolio/picasa/classes/privacy/provider.php

@@ -36,6 +36,7 @@
 class provider implements
         // This portfolio plugin does not store any data itself.
         // It has no database tables, and it purely acts as a conduit, sending data externally.
+        \core_privacy\local\metadata\provider,
         \core_portfolio\privacy\portfolio_provider {
 
     /**

privacy/tests/provider_test.php

@@ -163,6 +163,18 @@ public function test_metadata_provider($component, $classname) {
         }
     }
 
+    /**
+     * Test that all providers implement some form of compliant provider.
+     *
+     * @dataProvider get_component_list
+     * @param string $component frankenstyle component name, e.g. 'mod_assign'
+     * @param string $classname the fully qualified provider classname
+     */
+    public function test_all_providers_compliant($component, $classname) {
+        $manager = new manager();
+        $this->assertTrue($manager->component_is_compliant($component));
+    }
+
     /**
      * Data provider for the metadata\provider tests.
      *