Merge branch 'MDL-68991-master' of git://github.com/rezaies/moodle
authorJake Dallimore <jake@moodle.com>
Thu, 11 Jun 2020 04:27:16 +0000 (12:27 +0800)
committerJake Dallimore <jake@moodle.com>
Thu, 11 Jun 2020 04:27:16 +0000 (12:27 +0800)
lib/amd/build/userfeedback.min.js
lib/amd/build/userfeedback.min.js.map
lib/amd/src/userfeedback.js
lib/classes/external/record_userfeedback_action.php [moved from lib/classes/external/userfeedback/record_action.php with 97% similarity]
lib/classes/external/userfeedback/generate_url.php [deleted file]
lib/classes/userfeedback.php
lib/db/services.php
lib/tests/behat/userfeedback.feature
lib/tests/external/record_userfeedback_action_test.php [moved from lib/tests/external/userfeedback/record_action_test.php with 77% similarity]
lib/tests/external/userfeedback/generate_url_test.php [deleted file]
version.php

index 3eee54b..03f8faf 100644 (file)
Binary files a/lib/amd/build/userfeedback.min.js and b/lib/amd/build/userfeedback.min.js differ
index 2a4225f..71fb5b6 100644 (file)
Binary files a/lib/amd/build/userfeedback.min.js.map and b/lib/amd/build/userfeedback.min.js.map differ
index c1b1a7f..28a77ed 100644 (file)
@@ -42,8 +42,12 @@ export const registerEventListeners = () => {
         if (giveAction) {
             e.preventDefault();
 
-            giveFeedback()
-                .then(() => hideRoot(giveAction))
+            if (!window.open(giveAction.href)) {
+                throw new Error('Unable to open popup');
+            }
+
+            Promise.resolve(giveAction)
+                .then(hideRoot)
                 .then(recordAction)
                 .catch(Notification.exception);
         }
@@ -60,26 +64,6 @@ export const registerEventListeners = () => {
     });
 };
 
-/**
- * The action function that is called when users choose to give feedback.
- *
- * @returns {Promise<void>}
- */
-const giveFeedback = () => {
-    return Ajax.call([{
-        methodname: 'core_get_userfeedback_url',
-        args: {
-            contextid: M.cfg.contextid,
-        }
-    }])[0]
-        .then(url => {
-            if (!window.open(url)) {
-                throw new Error('Unable to open popup');
-            }
-            return;
-        });
-};
-
 /**
  * Record the action that the user took.
  *
@@ -22,7 +22,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-namespace core\external\userfeedback;
+namespace core\external;
 
 defined('MOODLE_INTERNAL') || die();
 
@@ -38,7 +38,7 @@ use external_value;
  * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-class record_action extends external_api {
+class record_userfeedback_action extends external_api {
     /**
      * Returns description of parameters.
      *
diff --git a/lib/classes/external/userfeedback/generate_url.php b/lib/classes/external/userfeedback/generate_url.php
deleted file mode 100644 (file)
index 307bbc2..0000000
+++ /dev/null
@@ -1,81 +0,0 @@
-<?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/>.
-
-/**
- * External API to generate and return the URL of the feedback site.
- *
- * @package    core
- * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace core\external\userfeedback;
-
-defined('MOODLE_INTERNAL') || die();
-
-require_once("$CFG->libdir/externallib.php");
-
-use external_api;
-use external_function_parameters;
-use external_value;
-use external_single_structure;
-use external_multiple_structure;
-
-/**
- * The external API to generate and return the feedback url.
- *
- * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class generate_url extends external_api {
-    /**
-     * Returns description of parameters.
-     *
-     * @return external_function_parameters
-     */
-    public static function execute_parameters() {
-        return new external_function_parameters([
-            'contextid' => new external_value(PARAM_INT, 'The context id of the page the user is in'),
-        ]);
-    }
-
-    /**
-     * Prepare and return the URL of the feedback site
-     *
-     * @param int $contextid The context id
-     * @return \stdClass
-     */
-    public static function execute(int $contextid) {
-        global $PAGE;
-
-        external_api::validate_parameters(self::execute_parameters(), ['contextid' => $contextid]);
-
-        $context = \context::instance_by_id($contextid);
-        self::validate_context($context);
-        $PAGE->set_context($context);
-
-        return \core_userfeedback::make_link()->out(false);
-    }
-
-    /**
-     * Returns description of method result value
-     *
-     * @return external_value
-     */
-    public static function execute_returns() {
-        return new external_value(PARAM_URL, 'Feedback site\'s URL');
-    }
-}
index 8128336..af2d7f8 100644 (file)
@@ -57,9 +57,9 @@ class core_userfeedback {
         $actions = [
             [
                 'title' => get_string('calltofeedback_give'),
-                'url' => '#',
+                'url' => static::make_link()->out(false),
                 'data' => [
-                        'action' => 'give',
+                    'action' => 'give',
                     'record' => 1,
                     'hide' => 1,
                 ],
@@ -103,13 +103,13 @@ class core_userfeedback {
             $lastactiontime = max($give ?: 0, $remind ?: 0);
 
             switch ($CFG->userfeedback_nextreminder) {
-                case self::REMIND_AFTER_UPGRADE:
-                    $lastupgrade = self::last_major_upgrade_time();
+                case static::REMIND_AFTER_UPGRADE:
+                    $lastupgrade = static::last_major_upgrade_time();
                     if ($lastupgrade >= $lastactiontime) {
                         return $lastupgrade + ($CFG->userfeedback_remindafter * DAYSECS) < time();
                     }
                     break;
-                case self::REMIND_PERIODICALLY:
+                case static::REMIND_PERIODICALLY:
                     return $lastactiontime + ($CFG->userfeedback_remindafter * DAYSECS) < time();
                     break;
             }
@@ -153,7 +153,7 @@ class core_userfeedback {
     public static function can_give_feedback(): bool {
         global $CFG;
 
-        return $CFG->enableuserfeedback && isloggedin() && !isguestuser();
+        return !empty($CFG->enableuserfeedback) && isloggedin() && !isguestuser();
     }
 
     /**
index 6b71aff..57038ef 100644 (file)
@@ -2809,7 +2809,7 @@ $functions = array(
         'capabilities'  => 'moodle/contentbank:manageowncontent',
     ],
     'core_create_userfeedback_action_record' => [
-        'classname'     => 'core\external\userfeedback\record_action',
+        'classname'     => 'core\external\record_userfeedback_action',
         'methodname'    => 'execute',
         'classpath'     => '',
         'description'   => 'Record the action that the user takes in the user feedback notification for future use.',
@@ -2817,16 +2817,6 @@ $functions = array(
         'ajax'          => 'true',
         'capabilities'  => '',
     ],
-    'core_get_userfeedback_url' => [
-        'classname'     => 'core\external\userfeedback\generate_url',
-        'methodname'    => 'execute',
-        'classpath'     => '',
-        'description'   => 'Generate a dynamic URL for the external user feedback site.' .
-                           ' The URL includes some parameters to pre-fill the user feedback form.',
-        'type'          => 'read',
-        'ajax'          => 'true',
-        'capabilities'  => '',
-    ],
 );
 
 $services = array(
index b090f80..6cf33d6 100644 (file)
@@ -34,3 +34,17 @@ Feature: Gathering user feedback
     And I reload the page
     Then I should not see "Give feedback" in the "region-main" "region"
     And I should not see "Remind me later" in the "region-main" "region"
+
+  @javascript
+  Scenario: Users should not see the notification after they click on the give feedback link
+    Given the following config values are set as admin:
+      | enableuserfeedback        | 1   |
+      | userfeedback_nextreminder | 2   |
+      | userfeedback_remindafter  | 90  |
+    When I log in as "admin"
+    And I follow "Dashboard" in the user menu
+    And I click on "Give feedback" "link"
+    And I close all opened windows
+    And I reload the page
+    Then I should not see "Give feedback" in the "region-main" "region"
+    And I should not see "Remind me later" in the "region-main" "region"
@@ -15,7 +15,7 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * External functions test for record_action.
+ * External functions test for record_feedback_action.
  *
  * @package    core
  * @category   test
@@ -23,7 +23,7 @@
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-namespace core\external\userfeedback;
+namespace core\external;
 
 defined('MOODLE_INTERNAL') || die();
 
@@ -35,20 +35,20 @@ global $CFG;
 require_once($CFG->dirroot . '/webservice/tests/helpers.php');
 
 /**
- * Class record_action_testcase
+ * Class record_userfeedback_action_testcase
  *
  * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @coversDefaultClass record_action
+ * @coversDefaultClass record_userfeedback_action
  */
-class record_action_testcase extends externallib_advanced_testcase {
+class record_userfeedback_action_testcase extends externallib_advanced_testcase {
 
     /**
-     * Data provider for test_record_action.
+     * Data provider for test_record_userfeedback_action.
      *
      * @return  array
      */
-    public function record_action_provider() {
+    public function record_userfeedback_action_provider() {
         return [
             'give action' => ['give'],
             'remind action' => ['remind'],
@@ -56,14 +56,14 @@ class record_action_testcase extends externallib_advanced_testcase {
     }
 
     /**
-     * Test the behaviour of record_action().
+     * Test the behaviour of record_userfeedback_action().
      *
-     * @dataProvider record_action_provider
+     * @dataProvider record_userfeedback_action_provider
      * @param string $action The action taken by the user
      *
      * @covers ::execute
      */
-    public function test_record_action(string $action) {
+    public function test_record_userfeedback_action(string $action) {
         $this->resetAfterTest();
 
         $context = context_system::instance();
@@ -74,7 +74,7 @@ class record_action_testcase extends externallib_advanced_testcase {
         $now = time();
 
         // Call the WS and check the action is recorded as expected.
-        $result = record_action::execute($action, $context->id);
+        $result = record_userfeedback_action::execute($action, $context->id);
         $this->assertNull($result);
 
         $preference = get_user_preferences('core_userfeedback_' . $action);
diff --git a/lib/tests/external/userfeedback/generate_url_test.php b/lib/tests/external/userfeedback/generate_url_test.php
deleted file mode 100644 (file)
index c205038..0000000
+++ /dev/null
@@ -1,118 +0,0 @@
-<?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/>.
-
-/**
- * External functions test for generate_url.
- *
- * @package    core
- * @category   test
- * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace core\external\userfeedback;
-
-defined('MOODLE_INTERNAL') || die();
-
-use externallib_advanced_testcase;
-use context_system;
-use context_course;
-use external_api;
-
-global $CFG;
-
-require_once($CFG->dirroot . '/webservice/tests/helpers.php');
-
-/**
- * Class generate_url_testcase
- *
- * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @coversDefaultClass generate_url
- */
-class generate_url_testcase extends externallib_advanced_testcase {
-
-    /**
-     * Test the behaviour of generate_url().
-     *
-     * @covers ::execute
-     */
-    public function test_record_action_system() {
-        $this->resetAfterTest();
-
-        $course = $this->getDataGenerator()->create_course();
-        $user = $this->getDataGenerator()->create_and_enrol($course, 'student');
-        $context = context_system::instance();
-
-        $this->setUser($user);
-
-        // Call the WS and check the requested data is returned as expected.
-        $result = generate_url::execute($context->id);
-        $result = external_api::clean_returnvalue(generate_url::execute_returns(), $result);
-
-        $this->assertStringStartsWith('https://feedback.moodle.org/lms', $result);
-        $this->assertStringContainsString('?lang=en', $result);
-        $this->assertStringContainsString('&moodle_url=https%3A%2F%2Fwww.example.com%2Fmoodle', $result);
-        $this->assertStringContainsString('&theme=boost', $result);
-    }
-
-    /**
-     * Test the behaviour of generate_url() in a course with a course theme.
-     *
-     * @covers ::execute
-     */
-    public function test_record_action_course_theme() {
-        $this->resetAfterTest();
-
-        // Enable course themes.
-        set_config('allowcoursethemes', 1);
-
-        $course = $this->getDataGenerator()->create_course(['theme' => 'classic']);
-        $user = $this->getDataGenerator()->create_and_enrol($course, 'student');
-        $context = context_course::instance($course->id);
-
-        $this->setUser($user);
-
-        // Call the WS and check the requested data is returned as expected.
-        $result = generate_url::execute($context->id);
-        $result = external_api::clean_returnvalue(generate_url::execute_returns(), $result);
-
-        $this->assertStringContainsString('&theme=classic', $result);
-    }
-
-    /**
-     * Test the behaviour of generate_url() when a custom feedback url is set.
-     *
-     * @covers ::execute
-     */
-    public function test_record_action_custom_feedback_url() {
-        $this->resetAfterTest();
-
-        // Enable course themes.
-        set_config('userfeedback_url', 'https://feedback.moodle.org/abc');
-
-        $user = $this->getDataGenerator()->create_user();
-        $context = context_system::instance();
-
-        $this->setUser($user);
-
-        // Call the WS and check the requested data is returned as expected.
-        $result = generate_url::execute($context->id);
-        $result = external_api::clean_returnvalue(generate_url::execute_returns(), $result);
-
-        $this->assertStringStartsWith('https://feedback.moodle.org/abc', $result);
-    }
-}
index f61484f..2c8a25c 100644 (file)
@@ -29,7 +29,7 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$version  = 2020061000.00;              // YYYYMMDD      = weekly release date of this DEV branch.
+$version  = 2020061000.01;              // YYYYMMDD      = weekly release date of this DEV branch.
                                         //         RR    = release increments - 00 in DEV branches.
                                         //           .XX = incremental changes.
 $release  = '3.9rc1 (Build: 20200609)'; // Human-friendly version name