Moodle release 3.9.5

Teachers can see user idnumber fields only when they are configured
identity fields. Students cannot.

.eslintignore

@@ -0,0 +1,92 @@
+# Generated by "grunt ignorefiles"
+*/**/yui/src/*/meta/
+*/**/build/
+node_modules/
+vendor/
+admin/tool/policy/amd/src/jquery-eu-cookie-law-popup.js
+auth/cas/CAS/
+cache/stores/mongodb/MongoDB/
+enrol/lti/ims-blti/
+filter/algebra/AlgParser.pm
+filter/tex/mimetex.*
+h5p/h5plib/v124/joubel/core/
+h5p/h5plib/v124/joubel/editor/
+lib/editor/atto/plugins/html/yui/src/codemirror/
+lib/editor/atto/plugins/html/yui/src/beautify/
+lib/editor/atto/yui/src/rangy/js/*.*
+lib/editor/tinymce/plugins/pdw/tinymce/
+lib/editor/tinymce/plugins/spellchecker/rpc.php
+lib/editor/tinymce/tiny_mce/
+lib/mlbackend/php/phpml/
+lib/adodb/
+lib/bennu/
+lib/evalmath/
+lib/phpspreadsheet/
+lib/google/
+lib/htmlpurifier/
+lib/jabber/
+lib/minify/matthiasmullie-minify/
+lib/minify/matthiasmullie-pathconverter/
+lib/pear/HTML/Common.php
+lib/pear/HTML/QuickForm.php
+lib/pear/HTML/QuickForm/
+lib/pear/PEAR.php
+lib/phpmailer/
+lib/simplepie/
+lib/tcpdf/
+lib/typo3/
+lib/yuilib/
+lib/yuilib/gallery/
+lib/jquery/
+lib/html2text/
+lib/markdown/
+lib/xhprof/
+lib/horde/
+lib/requirejs/
+lib/amd/src/loglevel.js
+lib/mustache/
+lib/amd/src/mustache.js
+lib/graphlib.php
+lib/php-css-parser/
+lib/rtlcss/
+lib/scssphp/
+lib/spout/
+lib/amd/src/chartjs-lazy.js
+lib/maxmind/GeoIp2/
+lib/maxmind/MaxMind/
+lib/ltiprovider/
+lib/amd/src/truncate.js
+lib/fonts/
+lib/amd/src/adapter.js
+lib/validateurlsyntax.php
+lib/amd/src/popper.js
+lib/geopattern-php/
+lib/php-jwt/
+lib/babel-polyfill/
+lib/polyfills/
+lib/emoji-data/
+lib/plist/
+media/player/videojs/amd/src/video-lazy.js
+media/player/videojs/amd/src/Youtube-lazy.js
+media/player/videojs/videojs/
+media/player/videojs/amd/src/videojs-flash-lazy.js
+media/player/videojs/videojs/video-js.swf
+mod/assign/feedback/editpdf/fpdi/
+repository/s3/S3.php
+theme/boost/scss/bootstrap/
+theme/boost/amd/src/bootstrap/alert.js
+theme/boost/amd/src/bootstrap/button.js
+theme/boost/amd/src/bootstrap/carousel.js
+theme/boost/amd/src/bootstrap/collapse.js
+theme/boost/amd/src/bootstrap/dropdown.js
+theme/boost/amd/src/bootstrap/index.js
+theme/boost/amd/src/bootstrap/modal.js
+theme/boost/amd/src/bootstrap/popover.js
+theme/boost/amd/src/bootstrap/tools/sanitizer.js
+theme/boost/amd/src/bootstrap/scrollspy.js
+theme/boost/amd/src/bootstrap/tab.js
+theme/boost/amd/src/bootstrap/toast.js
+theme/boost/amd/src/bootstrap/tooltip.js
+theme/boost/amd/src/bootstrap/util.js
+theme/boost/amd/src/tether.js
+theme/boost/scss/fontawesome/

.eslintrc

@@ -204,7 +204,7 @@
       }
     },
     {
-      files: ["**/amd/src/*.js", "**/amd/src/**/*.js", "Gruntfile*.js", "babel-plugin-add-module-to-define.js", "jsdoc.conf.js"],
+      files: ["**/amd/src/*.js", "**/amd/src/**/*.js", "Gruntfile*.js", "babel-plugin-add-module-to-define.js"],
       // We support es6 now. Woot!
       env: {
         es6: true

.github/workflows/push.yml

@@ -1,12 +1,6 @@
 name: Core
 
-on:
-  push:
-    branches-ignore:
-      - master
-      - MOODLE_[0-9]+_STABLE
-    tags-ignore:
-      - v[0-9]+.[0-9]+.[0-9]+*
+on: [push]
 
 env:
   php: 7.4

.gitignore

@@ -26,7 +26,6 @@
 /cscope.*
 /.patches/
 /.idea/
-.phpstorm.*
 /nbproject/
 CVS
 .DS_Store
@@ -34,7 +33,6 @@ CVS
 /.project
 /.buildpath
 /.cache
-.phpunit.result.cache
 phpunit.xml
 # Composer support. Do not ignore composer.json, or composer.lock. These should be shipped by us.
 composer.phar
@@ -46,8 +44,3 @@ composer.phar
 /lib/yuilib/*/*/*-coverage.js
 atlassian-ide-plugin.xml
 /node_modules/
-/.vscode/
-moodle-plugin-ci.phar
-.eslintignore
-.stylelintignore
-/jsdoc

.grunt/jsdoc/README.md

@@ -1,19 +0,0 @@
-# Moodle JavaScript Documentation
-
-```
-                                 .-..-.
-   _____                         | || |
-  /____/-.---_  .---.  .---.  .-.| || | .---.
-  | |  _   _  |/  _  \/  _  \/  _  || |/  __ \
-  * | | | | | || |_| || |_| || |_| || || |___/
-    |_| |_| |_|\_____/\_____/\_____||_|\_____)
-
-Moodle - the world's open source learning platform
-
-```
-
-## About
-This generated documentation includes API documentation for JavaScript written in the AMD and ES2015 module formats within Moodle.
-
-## Related information
-See [https://docs.moodle.org/dev](https://docs.moodle.org/dev) for other related Developer Documentation.

.grunt/jsdoc/jsdoc.conf.js

@@ -1,131 +0,0 @@
-// 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/>.
-
-/**
- * Helper functions for working with Moodle component names, directories, and sources.
- *
- * @copyright  2019 Andrew Nicols <andrew@nicols.co.uk>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-"use strict";
-/* eslint-env node */
-
-// Do not include any plugins as stanard.
-const plugins = [];
-
-plugins.push('plugins/markdown');
-
-/**
- * Get the source configuration.
- *
- * @return {Object}
- */
-const getSource = () => {
-    const glob = require('glob');
-    const path = require('path');
-    const ComponentList = require(path.resolve('GruntfileComponents.js'));
-    const thirdPartyPaths = ComponentList.getThirdPartyPaths();
-
-    const source = {
-        include: [],
-        includePattern: ".+\\.js$",
-    };
-
-    let includeList = [];
-
-    ComponentList.getAmdSrcGlobList().forEach(async pattern => {
-        includeList.push(...glob.sync(pattern));
-    });
-
-    const cwdLength = process.cwd().length + 1;
-    includeList.forEach(path => {
-        if (source.include.indexOf(path) !== -1) {
-            // Ensure no duplicates.
-            return;
-        }
-
-        const relPath = path.substring(cwdLength);
-        if (thirdPartyPaths.indexOf(relPath) !== -1) {
-            return;
-        }
-
-        source.include.push(path);
-    });
-
-    source.include.push('.grunt/jsdoc/README.md');
-    return source;
-};
-
-const tags = {
-    // Allow the use of unknown tags.
-    // We have a lot of legacy uses of these.
-    allowUnknownTags: true,
-
-    // We make use of jsdoc and closure dictionaries as standard.
-    dictionaries: [
-        'jsdoc',
-        'closure',
-    ],
-};
-
-// Template configuraiton.
-const templates = {
-    cleverLinks: false,
-    monospaceLinks: false,
-};
-
-module.exports = {
-    opts: {
-        destination: "./jsdoc/",
-        template: "node_modules/docdash",
-    },
-    plugins,
-    recurseDepth: 10,
-    source: getSource(),
-    sourceType: 'module',
-    tags,
-    templates,
-    docdash: {
-        collapse: true,
-        search: true,
-        sort: true,
-        sectionOrder: [
-            "Namespaces",
-            "Modules",
-            "Events",
-            "Classes",
-            "Externals",
-            "Mixins",
-            "Tutorials",
-            "Interfaces"
-        ],
-        "menu": {
-            "Developer Docs": {
-                href: "https://docs.moodle.org/dev",
-                target: "_blank",
-                "class": "menu-item",
-                id: "devdocs"
-            },
-            "MDN Docs": {
-                href: "https://developer.mozilla.org/en-US/docs/Web/JavaScript",
-                target: "_blank",
-                "class": "menu-item",
-                id: "mdndocs",
-            },
-        },
-        typedefs: true,
-    },
-};

.stylelintignore

@@ -0,0 +1,93 @@
+# Generated by "grunt ignorefiles"
+**/yui/build/*
+theme/boost/style/moodle.css
+theme/classic/style/moodle.css
+node_modules/
+vendor/
+admin/tool/policy/amd/src/jquery-eu-cookie-law-popup.js
+auth/cas/CAS/
+cache/stores/mongodb/MongoDB/
+enrol/lti/ims-blti/
+filter/algebra/AlgParser.pm
+filter/tex/mimetex.*
+h5p/h5plib/v124/joubel/core/
+h5p/h5plib/v124/joubel/editor/
+lib/editor/atto/plugins/html/yui/src/codemirror/
+lib/editor/atto/plugins/html/yui/src/beautify/
+lib/editor/atto/yui/src/rangy/js/*.*
+lib/editor/tinymce/plugins/pdw/tinymce/
+lib/editor/tinymce/plugins/spellchecker/rpc.php
+lib/editor/tinymce/tiny_mce/
+lib/mlbackend/php/phpml/
+lib/adodb/
+lib/bennu/
+lib/evalmath/
+lib/phpspreadsheet/
+lib/google/
+lib/htmlpurifier/
+lib/jabber/
+lib/minify/matthiasmullie-minify/
+lib/minify/matthiasmullie-pathconverter/
+lib/pear/HTML/Common.php
+lib/pear/HTML/QuickForm.php
+lib/pear/HTML/QuickForm/
+lib/pear/PEAR.php
+lib/phpmailer/
+lib/simplepie/
+lib/tcpdf/
+lib/typo3/
+lib/yuilib/
+lib/yuilib/gallery/
+lib/jquery/
+lib/html2text/
+lib/markdown/
+lib/xhprof/
+lib/horde/
+lib/requirejs/
+lib/amd/src/loglevel.js
+lib/mustache/
+lib/amd/src/mustache.js
+lib/graphlib.php
+lib/php-css-parser/
+lib/rtlcss/
+lib/scssphp/
+lib/spout/
+lib/amd/src/chartjs-lazy.js
+lib/maxmind/GeoIp2/
+lib/maxmind/MaxMind/
+lib/ltiprovider/
+lib/amd/src/truncate.js
+lib/fonts/
+lib/amd/src/adapter.js
+lib/validateurlsyntax.php
+lib/amd/src/popper.js
+lib/geopattern-php/
+lib/php-jwt/
+lib/babel-polyfill/
+lib/polyfills/
+lib/emoji-data/
+lib/plist/
+media/player/videojs/amd/src/video-lazy.js
+media/player/videojs/amd/src/Youtube-lazy.js
+media/player/videojs/videojs/
+media/player/videojs/amd/src/videojs-flash-lazy.js
+media/player/videojs/videojs/video-js.swf
+mod/assign/feedback/editpdf/fpdi/
+repository/s3/S3.php
+theme/boost/scss/bootstrap/
+theme/boost/amd/src/bootstrap/alert.js
+theme/boost/amd/src/bootstrap/button.js
+theme/boost/amd/src/bootstrap/carousel.js
+theme/boost/amd/src/bootstrap/collapse.js
+theme/boost/amd/src/bootstrap/dropdown.js
+theme/boost/amd/src/bootstrap/index.js
+theme/boost/amd/src/bootstrap/modal.js
+theme/boost/amd/src/bootstrap/popover.js
+theme/boost/amd/src/bootstrap/tools/sanitizer.js
+theme/boost/amd/src/bootstrap/scrollspy.js
+theme/boost/amd/src/bootstrap/tab.js
+theme/boost/amd/src/bootstrap/toast.js
+theme/boost/amd/src/bootstrap/tooltip.js
+theme/boost/amd/src/bootstrap/util.js
+theme/boost/amd/src/tether.js
+theme/boost/scss/fontawesome/

.travis.yml

@@ -10,7 +10,7 @@ language: php
 
 os: linux
 
-dist: bionic
+dist: xenial
 
 services:
     - mysql
@@ -19,12 +19,6 @@ services:
 addons:
   postgresql: "9.6"
 
-branches:
-  except:
-    - master
-    - /MOODLE_[0-9]+_STABLE/
-    - /^v[0-9]+\.[0-9]+\.[0-9]+.*/
-
 jobs:
     # Enable fast finish.
     # This will fail the build if a single job fails (except those in allow_failures).
@@ -220,7 +214,7 @@ before_script:
         # We need the official upstream for comparison
         git remote add upstream https://github.com/moodle/moodle.git;
 
-        git fetch upstream MOODLE_310_STABLE;
+        git fetch upstream MOODLE_39_STABLE;
         export GIT_PREVIOUS_COMMIT="`git merge-base FETCH_HEAD $TRAVIS_COMMIT`";
         export GIT_COMMIT="$TRAVIS_COMMIT";
         export UPSTREAM_FETCH_HEAD=`git rev-parse FETCH_HEAD`

Gruntfile.js

@@ -122,10 +122,9 @@ const registerStyleLintTasks = (grunt, files, fullRunDir) => {
     if (hasScss) {
         scssTasks.unshift('stylelint:scss');
     }
-    scssTasks.unshift('ignorefiles');
     grunt.registerTask('scss', scssTasks);
 
-    const cssTasks = ['ignorefiles'];
+    const cssTasks = [];
     if (hasCss) {
         cssTasks.push('stylelint:css');
     }
@@ -352,13 +351,6 @@ module.exports = function(grunt) {
                 }]
             }
         },
-        jsdoc: {
-            dist: {
-                options: {
-                    configure: ".grunt/jsdoc/jsdoc.conf.js",
-                },
-            },
-        },
         sass: {
             dist: {
                 files: {
@@ -432,7 +424,7 @@ module.exports = function(grunt) {
             '*/**/yui/src/*/meta/',
             '*/**/build/',
         ].concat(thirdPartyPaths);
-        grunt.file.write('.eslintignore', eslintIgnores.join('\n') + '\n');
+        grunt.file.write('.eslintignore', eslintIgnores.join('\n'));
 
         // Generate .stylelintignore.
         const stylelintIgnores = [
@@ -441,7 +433,7 @@ module.exports = function(grunt) {
             'theme/boost/style/moodle.css',
             'theme/classic/style/moodle.css',
         ].concat(thirdPartyPaths);
-        grunt.file.write('.stylelintignore', stylelintIgnores.join('\n') + '\n');
+        grunt.file.write('.stylelintignore', stylelintIgnores.join('\n'));
     };
 
     /**
@@ -843,7 +835,6 @@ module.exports = function(grunt) {
     grunt.loadNpmTasks('grunt-eslint');
     grunt.loadNpmTasks('grunt-stylelint');
     grunt.loadNpmTasks('grunt-babel');
-    grunt.loadNpmTasks('grunt-jsdoc');
 
     // Rename the grunt-contrib-watch "watch" task because we're going to wrap it.
     grunt.renameTask('watch', 'watch-grunt');
@@ -854,7 +845,7 @@ module.exports = function(grunt) {
     grunt.registerTask('ignorefiles', 'Generate ignore files for linters', tasks.ignorefiles);
     grunt.registerTask('watch', 'Run tasks on file changes', tasks.watch);
     grunt.registerTask('yui', ['eslint:yui', 'shifter']);
-    grunt.registerTask('amd', ['ignorefiles', 'eslint:amd', 'babel']);
+    grunt.registerTask('amd', ['eslint:amd', 'babel']);
     grunt.registerTask('js', ['amd', 'yui']);
 
     // Register CSS tasks.

GruntfileComponents.js

@@ -140,50 +140,6 @@ const getThirdPartyLibsList = relativeTo => {
         .sort();
 };
 
-/**
- * Get the list of thirdparty library paths.
- *
- * @returns {array}
- */
-const getThirdPartyPaths = () => {
-    const DOMParser = require('xmldom').DOMParser;
-    const fs = require('fs');
-    const path = require('path');
-    const xpath = require('xpath');
-
-    const thirdpartyfiles = getThirdPartyLibsList(fs.realpathSync('./'));
-    const libs = ['node_modules/', 'vendor/'];
-
-    const addLibToList = lib => {
-        if (!lib.match('\\*') && fs.statSync(lib).isDirectory()) {
-            // Ensure trailing slash on dirs.
-            lib = lib.replace(/\/?$/, '/');
-        }
-
-        // Look for duplicate paths before adding to array.
-        if (libs.indexOf(lib) === -1) {
-            libs.push(lib);
-        }
-    };
-
-    thirdpartyfiles.forEach(function(file) {
-        const dirname = path.dirname(file);
-
-        const xmlContent = fs.readFileSync(file, 'utf8');
-        const doc = new DOMParser().parseFromString(xmlContent);
-        const nodes = xpath.select("/libraries/library/location/text()", doc);
-
-        nodes.forEach(function(node) {
-            let lib = path.posix.join(dirname, node.toString());
-            addLibToList(lib);
-        });
-    });
-
-    return libs;
-
-};
-
-
 /**
  * Find the name of the component matching the specified path.
  *
@@ -229,5 +185,4 @@ module.exports = {
     getOwningComponentDirectory,
     getYuiSrcGlobList,
     getThirdPartyLibsList,
-    getThirdPartyPaths,
 };

TODO

@@ -1 +0,0 @@
-mod/forum/tests/behat/forum_subscriptions_default.feature

TRADEMARK.txt

@@ -2,7 +2,7 @@ MOODLE TRADEMARKS POLICY
 ========================
 
 The word "Moodle" is trademarked in many countries around the world. The word
-was originally coined as an acronym: Modular Object-Oriented Dynamic Learning
+was originally an acronym: Modular Object-Oriented Dynamic Learning
 Environment. The trademark is owned by Martin Dougiamas, Founder and Director
 of Moodle.
 

admin/admin_settings_search_form.php

@@ -15,41 +15,34 @@
 // along with Moodle.  If not, see <http://www.gnu.org/licenses/>.
 
 /**
- * Clientside filter base.
+ * Admin settings search form
  *
- * @package    tool_usertours
- * @copyright  2020 The Open University
+ * @package    admin
+ * @copyright  2016 Damyon Wiese
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-namespace tool_usertours\local\clientside_filter;
-
 defined('MOODLE_INTERNAL') || die();
 
-use stdClass;
-use tool_usertours\local\filter\base;
-use tool_usertours\tour;
+require_once $CFG->libdir.'/formslib.php';
 
 /**
- * Clientside filter base.
+ * Admin settings search form
  *
- * @copyright  2020 The Open University
+ * @package    admin
+ * @copyright  2016 Damyon Wiese
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
-abstract class clientside_filter extends base {
-    /**
-     * Returns the filter values needed for client side filtering.
-     *
-     * @param   tour            $tour       The tour to find the filter values for
-     * @return  stdClass
-     */
-    public static function get_client_side_values(tour $tour): stdClass {
-        $data = (object) [];
+class admin_settings_search_form extends moodleform {
+    function definition () {
+        $mform = $this->_form;
 
-        if (is_a(static::class, clientside_filter::class, true)) {
-            $data->filterdata = $tour->get_filter_values(static::get_filter_name());
-        }
-
-        return $data;
+        //$mform->addElement('header', 'settingsheader', get_string('search', 'admin'));
+        $elements = [];
+        $elements[] = $mform->createElement('text', 'query', get_string('query', 'admin'));
+        $elements[] = $mform->createElement('submit', 'search', get_string('search'));
+        $mform->addGroup($elements);
+        $mform->setType('query', PARAM_RAW);
+        $mform->setDefault('query', optional_param('query', '', PARAM_RAW));
     }
-}
+}

admin/category.php

@@ -158,9 +158,4 @@ echo $outputhtml;
 echo html_writer::end_tag('fieldset');
 echo html_writer::end_tag('form');
 
-$PAGE->requires->yui_module('moodle-core-formchangechecker', 'M.core_formchangechecker.init', [[
-    'formid' => 'adminsettings'
-]]);
-$PAGE->requires->string_for_js('changesmadereallygoaway', 'moodle');
-
 echo $OUTPUT->footer();

admin/classes/form/testoutgoingmailconf_form.php

@@ -43,7 +43,7 @@ class testoutgoingmailconf_form extends \moodleform {
         $mform = $this->_form;
 
         // Recipient.
-        $options = ['maxlength' => '100', 'size' => '25', 'autocomplete' => 'email'];
+        $options = ['maxlength' => '100', 'size' => '25'];
         $mform->addElement('text', 'recipient', get_string('testoutgoingmailconf_toemail', 'admin'), $options);
         $mform->setType('recipient', PARAM_EMAIL);
         $mform->addRule('recipient', get_string('required'), 'required');

admin/classes/local/externalpage/accesscallback.php

@@ -1,71 +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 admin page class that allows a callback to be provided to determine whether page can be accessed
- *
- * @package     core_admin
- * @copyright   2019 Marina Glancy
- * @license     http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace core_admin\local\externalpage;
-
-use admin_externalpage;
-
-defined('MOODLE_INTERNAL') || die();
-
-require_once("{$CFG->libdir}/adminlib.php");
-
-/**
- * Admin externalpage class
- *
- * @package     core_admin
- * @copyright   2019 Marina Glancy
- * @license     http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class accesscallback extends admin_externalpage {
-
-    /** @var callable $accesscheckcallback */
-    protected $accesscheckcallback;
-
-    /**
-     * Class constructor
-     *
-     * @param string $name
-     * @param string $visiblename
-     * @param string $url
-     * @param callable $accesscheckcallback The callback method that will be executed to check whether user has access to
-     *     this page. The setting instance ($this) is passed as an argument to the callback. Should return boolean value
-     * @param bool $hidden
-     */
-    public function __construct(string $name, string $visiblename, string $url, callable $accesscheckcallback,
-            bool $hidden = false) {
-
-        $this->accesscheckcallback = $accesscheckcallback;
-
-        parent::__construct($name, $visiblename, $url, [], $hidden);
-    }
-
-    /**
-     * Determines if the current user has access to this external page based on access callback
-     *
-     * @return bool
-     */
-    public function check_access() {
-        return ($this->accesscheckcallback)($this);
-    }
-}

admin/classes/local/settings/autocomplete.php

@@ -1,206 +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/>.
-
-/**
- * Auto complete admin setting.
- *
- * @package    core_admin
- * @copyright  2020 The Open University
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace core_admin\local\settings;
-
-defined('MOODLE_INTERNAL') || die();
-
-require_once($CFG->libdir . '/adminlib.php');
-/**
- * Auto complete setting class.
- *
- * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- * @copyright 2020 The Open University
- */
-class autocomplete extends \admin_setting_configmultiselect {
-    /** @var boolean Should we allow typing new entries to the field? */
-    protected $tags = false;
-    /** @var string Name of an AMD module to send/process ajax requests. */
-    protected $ajax = '';
-    /** @var string Placeholder text for an empty list. */
-    protected $placeholder = '';
-    /** @var bool Whether the search has to be case-sensitive. */
-    protected $casesensitive = false;
-    /** @var bool Show suggestions by default - but this can be turned off. */
-    protected $showsuggestions = true;
-    /** @var string String that is shown when there are no selections. */
-    protected $noselectionstring = '';
-    /** @var string Delimiter to store values in database. */
-    protected $delimiter = ',';
-    /** @var string Should be multiple choices? */
-    protected $multiple = true;
-    /** @var string The link to manage choices. */
-    protected $manageurl = true;
-    /** @var string The text to display in manage link. */
-    protected $managetext = true;
-
-    /**
-     * Constructor
-     *
-     * @param string $name unique ascii name, either 'mysetting' for settings that in config, or 'myplugin/mysetting'
-     * for ones in config_plugins.
-     * @param string $visiblename localised
-     * @param string $description long localised info
-     * @param array $defaultsetting array of selected items
-     * @param array $choices options for autocomplete field
-     * @param array $attributes settings for autocomplete field
-     */
-    public function __construct($name, $visiblename, $description, $defaultsetting, $choices, $attributes = null) {
-
-        if ($attributes === null) {
-            $attributes = [];
-        }
-
-        $this->placeholder = get_string('search');
-        $this->noselectionstring = get_string('noselection', 'form');
-        $defaultattributes = [
-                'tags',
-                'showsuggestions',
-                'placeholder',
-                'noselectionstring',
-                'ajax',
-                'casesensitive',
-                'delimiter',
-                'multiple',
-                'manageurl',
-                'managetext'
-        ];
-
-        foreach ($defaultattributes as $attributename) {
-            if (isset($attributes[$attributename])) {
-                $this->$attributename = $attributes[$attributename];
-            }
-        }
-
-        parent::__construct($name, $visiblename, $description, $defaultsetting, $choices);
-    }
-
-    /**
-     * Returns the select setting(s)
-     *
-     * @return mixed null or array. Null if no settings else array of setting(s)
-     */
-    public function get_setting() {
-        $result = $this->config_read($this->name);
-        if (is_null($result)) {
-            return null;
-        }
-        if ($result === '') {
-            return [];
-        }
-        return explode($this->delimiter, $result);
-    }
-
-    /**
-     * Saves setting(s) provided through $data
-     *
-     * @param array $data
-     */
-    public function write_setting($data) {
-        if (!is_array($data)) {
-            return ''; // Ignore it.
-        }
-        if (!$this->load_choices() || empty($this->choices)) {
-            return '';
-        }
-
-        unset($data['xxxxx']);
-
-        $save = [];
-        foreach ($data as $value) {
-            if (!array_key_exists($value, $this->choices)) {
-                continue; // Ignore it.
-            }
-            $save[] = $value;
-        }
-
-        return ($this->config_write($this->name, implode($this->delimiter, $save)) ? '' : get_string('errorsetting', 'admin'));
-    }
-
-    /**
-     * Returns XHTML autocomplete field
-     *
-     * @param array $data Array of values to select by default
-     * @param string $query
-     * @return string XHTML autocomplete field
-     */
-    public function output_html($data, $query = '') {
-        global $OUTPUT;
-
-        if (!$this->load_choices() or empty($this->choices)) {
-            return '';
-        }
-
-        $default = $this->get_defaultsetting();
-        if (empty($default)) {
-            $default = [];
-        }
-
-        if (is_null($data)) {
-            $data = [];
-        }
-
-        $context = [
-                'id' => $this->get_id(),
-                'name' => $this->get_full_name()
-        ];
-
-        $defaults = [];
-        $options = [];
-        $template = 'core_admin/local/settings/autocomplete';
-
-        foreach ($this->choices as $value => $name) {
-            if (in_array($value, $default)) {
-                $defaults[] = $name;
-            }
-            $options[] = [
-                    'value' => $value,
-                    'text' => $name,
-                    'selected' => in_array($value, $data),
-                    'disabled' => false
-            ];
-        }
-
-        $context['options'] = $options;
-        $context['tags'] = $this->tags;
-        $context['placeholder'] = $this->placeholder;
-        $context['casesensitive'] = $this->casesensitive;
-        $context['multiple'] = $this->multiple;
-        $context['showsuggestions'] = $this->showsuggestions;
-        $context['manageurl'] = $this->manageurl;
-        $context['managetext'] = $this->managetext;
-
-        if (is_null($default)) {
-            $defaultinfo = null;
-        } if (!empty($defaults)) {
-            $defaultinfo = implode(', ', $defaults);
-        } else {
-            $defaultinfo = get_string('none');
-        }
-
-        $element = $OUTPUT->render_from_template($template, $context);
-
-        return format_admin_setting($this, $this->visiblename, $element, $this->description, true, '', $defaultinfo, $query);
-    }
-}

admin/classes/local/settings/manage_payment_gateway_plugins.php

@@ -1,74 +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/>.
-
-/**
- * Payment gateway admin setting.
- *
- * @package    core_admin
- * @copyright  2020 Shamim Rezaie <shamim@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace core_admin\local\settings;
-
-/**
- * Generic class for managing plugins in a table that allows re-ordering and enable/disable of each plugin.
- */
-class manage_payment_gateway_plugins extends \admin_setting_manage_plugins {
-    /**
-     * Get the admin settings section title (use get_string).
-     *
-     * @return string
-     */
-    public function get_section_title() {
-        return get_string('type_paygw_plural', 'plugin');
-    }
-
-    /**
-     * Get the type of plugin to manage.
-     *
-     * @return string
-     */
-    public function get_plugin_type() {
-        return 'paygw';
-    }
-
-    /**
-     * Get the name of the second column.
-     *
-     * @return string
-     */
-    public function get_info_column_name() {
-        return get_string('supportedcurrencies', 'core_payment');
-    }
-
-    /**
-     * Get the type of plugin to manage.
-     *
-     * @param plugininfo The plugin info class.
-     * @return string
-     */
-    public function get_info_column($plugininfo) {
-        $codes = $plugininfo->get_supported_currencies();
-
-        $currencies = [];
-        foreach ($codes as $c) {
-            $currencies[$c] = new \lang_string($c, 'core_currencies');
-        }
-
-        return implode(get_string('listsep', 'langconfig') . ' ', $currencies);
-    }
-}

admin/classes/task_log_table.php

@@ -57,8 +57,6 @@ class task_log_table extends \table_sql {
             'userid'     => get_string('user', 'admin'),
             'timestart'  => get_string('task_starttime', 'admin'),
             'duration'   => get_string('task_duration', 'admin'),
-            'hostname'   => get_string('hostname', 'tool_task'),
-            'pid'        => get_string('pid', 'tool_task'),
             'db'         => get_string('task_dbstats', 'admin'),
             'result'     => get_string('task_result', 'admin'),
             'actions'    => '',
@@ -134,7 +132,6 @@ class task_log_table extends \table_sql {
 
         $sql = "SELECT
                     tl.id, tl.type, tl.component, tl.classname, tl.userid, tl.timestart, tl.timeend,
-                    tl.hostname, tl.pid,
                     tl.dbreads, tl.dbwrites, tl.result,
                     tl.dbreads + tl.dbwrites AS db,
                     tl.timeend - tl.timestart AS duration,

admin/cli/adhoc_task.php

@@ -37,13 +37,11 @@ list($options, $unrecognized) = cli_get_params(
         'showsql' => false,
         'showdebugging' => false,
         'ignorelimits' => false,
-        'force' => false,
     ], [
         'h' => 'help',
         'e' => 'execute',
         'k' => 'keep-alive',
         'i' => 'ignorelimits',
-        'f' => 'force',
     ]
 );
 
@@ -63,7 +61,6 @@ Options:
  -e, --execute             Run all queued adhoc tasks
  -k, --keep-alive=N        Keep this script alive for N seconds and poll for new adhoc tasks
  -i  --ignorelimits        Ignore task_adhoc_concurrency_limit and task_adhoc_max_runtime limits
- -f, --force               Run even if cron is disabled
 
 Example:
 \$sudo -u www-data /usr/bin/php admin/cli/adhoc_task.php --execute
@@ -95,12 +92,6 @@ if (moodle_needs_upgrading()) {
 if (empty($options['execute'])) {
     exit(0);
 }
-
-if (!get_config('core', 'cron_enabled') && !$options['force']) {
-    mtrace('Cron is disabled. Use --force to override.');
-    exit(1);
-}
-
 if (empty($options['keep-alive'])) {
     $options['keep-alive'] = 0;
 }

admin/cli/cron.php

@@ -36,23 +36,14 @@ require_once($CFG->libdir.'/cronlib.php');
 
 // now get cli options
 list($options, $unrecognized) = cli_get_params(
-    [
+    array(
         'help' => false,
         'stop' => false,
-        'list' => false,
-        'force' => false,
-        'enable' => false,
-        'disable' => false,
-        'disable-wait' => false,
-    ], [
+    ),
+    array(
         'h' => 'help',
         's' => 'stop',
-        'l' => 'list',
-        'f' => 'force',
-        'e' => 'enable',
-        'd' => 'disable',
-        'w' => 'disable-wait',
-    ]
+    )
 );
 
 if ($unrecognized) {
@@ -65,13 +56,8 @@ if ($options['help']) {
 "Execute periodic cron actions.
 
 Options:
--h, --help               Print out this help
--s, --stop               Notify all other running cron processes to stop after the current task
--l, --list               Show the list of currently running tasks and how long they have been running
--f, --force              Execute task even if cron is disabled
--e, --enable             Enable cron
--d, --disable            Disable cron
--w, --disable-wait=600   Disable cron and wait until all tasks finished or fail after N seconds (optional param)
+-h, --help            Print out this help
+-s, --stop            Notify all other running cron processes to stop after the current task
 
 Example:
 \$sudo -u www-data /usr/bin/php admin/cli/cron.php
@@ -88,91 +74,6 @@ if ($options['stop']) {
     die;
 }
 
-if ($options['enable']) {
-    set_config('cron_enabled', 1);
-    mtrace('Cron has been enabled for the site.');
-    exit(0);
-}
-
-if ($options['disable']) {
-    set_config('cron_enabled', 0);
-    \core\task\manager::clear_static_caches();
-    mtrace('Cron has been disabled for the site.');
-    exit(0);
-}
-
-if ($options['list']) {
-    $tasks = \core\task\manager::get_running_tasks();
-    mtrace('The list of currently running tasks:');
-    $format = "%7s %-12s %-9s %-20s %-52s\n";
-    printf ($format,
-        'PID',
-        'HOST',
-        'TYPE',
-        'TIME',
-        'CLASSNAME'
-    );
-    foreach ($tasks as $task) {
-        printf ($format,
-            $task->pid,
-            substr($task->hostname, 0, 12),
-            $task->type,
-            format_time(time() - $task->timestarted),
-            substr($task->classname, 0, 52)
-        );
-    }
-    exit(0);
-}
-
-if ($wait = $options['disable-wait']) {
-    $started = time();
-    if (true === $wait) {
-        // Default waiting time.
-        $waitsec = 600;
-    } else {
-        $waitsec = $wait;
-        $wait = true;
-    }
-
-    set_config('cron_enabled', 0);
-    \core\task\manager::clear_static_caches();
-    mtrace('Cron has been disabled for the site.');
-    mtrace('Allocating '. format_time($waitsec) . ' for the tasks to finish.');
-
-    $lastcount = 0;
-    while ($wait) {
-        $tasks = \core\task\manager::get_running_tasks();
-
-        if (count($tasks) == 0) {
-            mtrace('');
-            mtrace('All scheduled and adhoc tasks finished.');
-            exit(0);
-        }
-
-        if (time() - $started >= $waitsec) {
-            mtrace('');
-            mtrace('Wait time ('. format_time($waitsec) . ') elapsed, but ' . count($tasks) . ' task(s) still running.');
-            mtrace('Exiting with code 1.');
-            exit(1);
-        }
-
-        if (count($tasks) !== $lastcount) {
-            mtrace('');
-            mtrace(count($tasks) . " tasks currently running.", '');
-            $lastcount = count($tasks);
-        } else {
-            mtrace('.', '');
-        }
-
-        sleep(1);
-    }
-}
-
-if (!get_config('core', 'cron_enabled') && !$options['force']) {
-    mtrace('Cron is disabled. Use --force to override.');
-    exit(1);
-}
-
 \core\local\cli\shutdown::script_supports_graceful_exit();
 
 cron_run();

admin/cli/fix_orphaned_calendar_events.php

@@ -1,138 +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/>.
-
-/**
- * Fix orphaned calendar events that were broken by MDL-67494.
- *
- * This script will look for all the calendar events which userids
- * where broken by a wrong upgrade step, affecting to Moodle 3.9.5
- * and up.
- *
- * It performs checks to both:
- *    a) Detect if the site was affected (ran the wrong upgrade step).
- *    b) Look for orphaned calendar events, categorising them as:
- *       - standard: site / category / course / group / user events
- *       - subscription: events created via subscriptions.
- *       - action: normal action events, created to show common important dates.
- *       - override: user and group override events, particular, that some activities support.
- *       - custom: other events, not being any of the above, common or particular.
- * By specifying it (--fix) try to recover as many broken events (missing userid) as
- * possible. Standard, subscription, action, override events in core are fully supported but
- * override or custom events should be fixed by each plugin as far as there isn't any standard
- * API (plugin-wise) to launch a rebuild of the calendar events.
- *
- * @package core
- * @copyright 2021 onwards Simey Lameze <simey@moodle.com>
- * @license    https://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-define('CLI_SCRIPT', true);
-
-require_once(__DIR__ . '/../../config.php');
-require_once($CFG->libdir . "/clilib.php");
-require_once($CFG->libdir . '/db/upgradelib.php');
-
-// Supported options.
-$long = ['fix'  => false, 'help' => false];
-$short = ['f' => 'fix', 'h' => 'help'];
-
-// CLI options.
-[$options, $unrecognized] = cli_get_params($long, $short);
-
-if ($unrecognized) {
-    $unrecognized = implode("\n  ", $unrecognized);
-    cli_error(get_string('cliunknowoption', 'admin', $unrecognized));
-}
-
-if ($options['help']) {
-    $help = <<<EOT
-Fix orphaned calendar events.
-
-  This script detects calendar events that have had their
-  userid lost. By default it will perform various checks
-  and report them, showing the site status in an easy way.
-
-  Also, optionally (--fix), it wil try to recover as many
-  lost userids as possible from different sources. Note that
-  this script aims to process well-know events in core,
-  leaving custom events in 3rd part plugins mostly unmodified
-  because there isn't any consistent way to regenerate them.
-
-  For more details:  https://tracker.moodle.org/browse/MDL-71156
-
-Options:
-  -h, --help    Print out this help.
-  -f, --fix     Fix the orphaned calendar events in the DB.
-                If not specified only check and report problems to STDERR.
-
-Usage:
-  - Only report:    \$ sudo -u www-data /usr/bin/php admin/cli/fix_orphaned_calendar_events.php
-  - Report and fix: \$ sudo -u www-data /usr/bin/php admin/cli/fix_orphaned_calendar_events.php -f
-EOT;
-
-    cli_writeln($help);
-    die;
-}
-
-// Check various usual pre-requisites.
-if (empty($CFG->version)) {
-    cli_error('Database is not yet installed.');
-}
-
-$admin = get_admin();
-if (!$admin) {
-    cli_error('Error: No admin account was found.');
-}
-
-if (moodle_needs_upgrading()) {
-    cli_error('Moodle upgrade pending, script execution suspended.');
-}
-
-// Do everything as admin by default.
-\core\session\manager::set_user($admin);
-
-// Report current site status.
-cli_heading('Checking the site status');
-$needsfix = upgrade_calendar_site_status();
-
-// Report current calendar events status.
-cli_heading('Checking the calendar events status');
-$info = upgrade_calendar_events_status();
-$hasbadevents = $info['total']->bad > 0 || $info['total']->bad != $info['other']->bad;
-$needsfix = $needsfix || $hasbadevents;
-
-// If, selected, fix as many calendar events as possible.
-if ($options['fix']) {
-
-    // If the report has told us that the fix was not needed... ask for confirmation!
-    if (!$needsfix) {
-        cli_writeln("This site DOES NOT NEED to run the calendar events fix.");
-        $input = cli_input('Are you completely sure that you want to run the fix? (y/N)', 'N', ['y', 'Y', 'n', 'N']);
-        if (strtolower($input) != 'y') {
-            exit(0);
-        }
-        cli_writeln("");
-    }
-    cli_heading('Fixing as many as possible calendar events');
-    upgrade_calendar_events_fix_remaining($info);
-    // Report current (after fix) calendar events status.
-    cli_heading('Checking the calendar events status (after fix)');
-    upgrade_calendar_events_status();
-} else if ($needsfix) {
-    // Fix option was not provided but problem events have been found. Notify the user and provide info how to fix these events.
-    cli_writeln("This site NEEDS to run the calendar events fix!");
-    cli_writeln("To fix the calendar events, re-run this script with the --fix option.");
-}

admin/cli/mysql_collation.php

@@ -119,7 +119,7 @@ if (!empty($options['collation'])) {
     if ($dbcollation->value !== $collation || $dbcharset->value !== $charset) {
         // Try to convert the DB.
         echo "Converting database to '$collation' for $CFG->wwwroot:\n";
-        $sql = "ALTER DATABASE `$CFG->dbname` DEFAULT CHARACTER SET $charset DEFAULT COLLATE = $collation";
+        $sql = "ALTER DATABASE $CFG->dbname DEFAULT CHARACTER SET $charset DEFAULT COLLATE = $collation";
         try {
             $DB->change_database_structure($sql);
         } catch (exception $e) {
@@ -146,7 +146,7 @@ if (!empty($options['collation'])) {
 
         } else {
             try {
-                $DB->change_database_structure("ALTER TABLE `$table->name` CONVERT TO CHARACTER SET $charset COLLATE $collation");
+                $DB->change_database_structure("ALTER TABLE $table->name CONVERT TO CHARACTER SET $charset COLLATE $collation");
                 echo "CONVERTED\n";
                 $converted++;
             } catch (ddl_exception $e) {
@@ -163,7 +163,7 @@ if (!empty($options['collation'])) {
             }
         }
 
-        $sql = "SHOW FULL COLUMNS FROM `$table->name` WHERE collation IS NOT NULL";
+        $sql = "SHOW FULL COLUMNS FROM $table->name WHERE collation IS NOT NULL";
         $rs2 = $DB->get_recordset_sql($sql);
         foreach ($rs2 as $column) {
             $column = (object)array_change_key_case((array)$column, CASE_LOWER);
@@ -181,7 +181,7 @@ if (!empty($options['collation'])) {
                 $notnull = ($column->null === 'NO') ? 'NOT NULL' : 'NULL';
                 $default = (!is_null($column->default) and $column->default !== '') ? "DEFAULT '$column->default'" : '';
                 // primary, unique and inc are not supported for texts
-                $sql = "ALTER TABLE `$table->name`
+                $sql = "ALTER TABLE $table->name
                         MODIFY COLUMN $column->field $column->type
                         CHARACTER SET $charset
                         COLLATE $collation $notnull $default";
@@ -192,11 +192,11 @@ if (!empty($options['collation'])) {
                 $default = !is_null($column->default) ? "DEFAULT '$column->default'" : '';
 
                 if ($rowformat != '') {
-                    $sql = "ALTER TABLE `$table->name` $rowformat";
+                    $sql = "ALTER TABLE $table->name $rowformat";
                     $DB->change_database_structure($sql);
                 }
 
-                $sql = "ALTER TABLE `$table->name`
+                $sql = "ALTER TABLE $table->name
                         MODIFY COLUMN $column->field $column->type
                         CHARACTER SET $charset
                         COLLATE $collation $notnull $default";
@@ -315,8 +315,8 @@ function mysql_set_row_format($tablename, $charset, $collation, $engine) {
         if ($rs->row_format == 'Compact' || $rs->row_format == 'Redundant') {
             $rowformat = $DB->get_row_format_sql($engine, $collation);
             // Try to convert to compressed format and then try updating the collation again.
-            $DB->change_database_structure("ALTER TABLE `$tablename` $rowformat");
-            $DB->change_database_structure("ALTER TABLE `$tablename` CONVERT TO CHARACTER SET $charset COLLATE $collation");
+            $DB->change_database_structure("ALTER TABLE $tablename $rowformat");
+            $DB->change_database_structure("ALTER TABLE $tablename CONVERT TO CHARACTER SET $charset COLLATE $collation");
         } else {
             // Row format may not be the problem. Can not diagnose problem. Send fail reply.
             return false;

admin/cli/mysql_compressed_rows.php

@@ -158,7 +158,7 @@ if (!empty($options['info'])) {
     }
 
     foreach ($fixtables as $table) {
-        $DB->change_database_structure("ALTER TABLE `{$prefix}$table` ROW_FORMAT=Compressed");
+        $DB->change_database_structure("ALTER TABLE {$prefix}$table ROW_FORMAT=Compressed");
         echo str_pad($prefix . $table, 32, ' ', STR_PAD_RIGHT) . " ... Compressed\n";
     }
 
@@ -189,7 +189,7 @@ if (!empty($options['info'])) {
     echo "SET GLOBAL innodb_file_per_table=1;\n";
     echo "SET GLOBAL innodb_file_format=Barracuda;\n";
     foreach ($fixtables as $table) {
-        echo "ALTER TABLE `{$prefix}$table` ROW_FORMAT=Compressed;\n";
+        echo "ALTER TABLE {$prefix}$table ROW_FORMAT=Compressed;\n";
     }
     echo "\n";
     exit(0);

admin/cli/restore_backup.php

@@ -1,109 +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/>.
-
-/**
- * This script allows to restore a course from CLI.
- *
- * @package    core
- * @subpackage cli
- * @copyright  2020 Catalyst IT
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-define('CLI_SCRIPT', 1);
-
-require(__DIR__ . '/../../config.php');
-require_once($CFG->libdir . '/clilib.php');
-require_once($CFG->dirroot . "/backup/util/includes/restore_includes.php");
-
-list($options, $unrecognized) = cli_get_params([
-    'file' => '',
-    'categoryid' => '',
-    'showdebugging' => false,
-    'help' => false,
-], [
-    'f' => 'file',
-    'c' => 'categoryid',
-    's' => 'showdebugging',
-    'h' => 'help',
-]);
-
-if ($unrecognized) {
-    $unrecognized = implode("\n  ", $unrecognized);
-    cli_error(get_string('cliunknowoption', 'admin', $unrecognized));
-}
-
-if ($options['help'] || !($options['file']) || !($options['categoryid'])) {
-    $help = <<<EOL
-Restore backup into provided category.
-
-Options:
--f, --file=STRING           Path to the backup file.
--c, --categoryid=INT        ID of the category to restore too.
--s, --showdebugging         Show developer level debugging information
--h, --help                  Print out this help.
-
-Example:
-\$sudo -u www-data /usr/bin/php admin/cli/restore_backup.php --file=/path/to/backup/file.mbz --categoryid=1\n
-EOL;
-
-    echo $help;
-    exit(0);
-}
-
-if ($options['showdebugging']) {
-    set_debugging(DEBUG_DEVELOPER, true);
-}
-
-if (!$admin = get_admin()) {
-    print_error('noadmins');
-}
-
-if (!file_exists($options['file'])) {
-    print_error('filenotfound');
-}
-
-if (!$category = $DB->get_record('course_categories', ['id' => $options['categoryid']], 'id')) {
-    print_error('invalidcategoryid');
-}
-
-$backupdir = "restore_" . uniqid();
-$path = $CFG->tempdir . DIRECTORY_SEPARATOR . "backup" . DIRECTORY_SEPARATOR . $backupdir;
-
-cli_heading(get_string('extractingbackupfileto', 'backup', $path));
-$fp = get_file_packer('application/vnd.moodle.backup');
-$fp->extract_to_pathname($options['file'], $path);
-
-cli_heading(get_string('preprocessingbackupfile'));
-try {
-    list($fullname, $shortname) = restore_dbops::calculate_course_names(0, get_string('restoringcourse', 'backup'),
-        get_string('restoringcourseshortname', 'backup'));
-
-    $courseid = restore_dbops::create_new_course($fullname, $shortname, $category->id);
-
-    $rc = new restore_controller($backupdir, $courseid, backup::INTERACTIVE_NO,
-        backup::MODE_GENERAL, $admin->id, backup::TARGET_NEW_COURSE);
-    $rc->execute_precheck();
-    $rc->execute_plan();
-    $rc->destroy();
-} catch (Exception $e) {
-    cli_heading(get_string('cleaningtempdata'));
-    fulldelete($path);
-    print_error('generalexceptionmessage', 'error', '', $e->getMessage());
-}
-
-cli_heading(get_string('restoredcourseid', 'backup', $courseid));
-exit(0);

admin/cli/scheduled_task.php

@@ -30,17 +30,8 @@ require_once("$CFG->libdir/clilib.php");
 require_once("$CFG->libdir/cronlib.php");
 
 list($options, $unrecognized) = cli_get_params(
-    [
-        'help' => false,
-        'list' => false,
-        'execute' => false,
-        'showsql' => false,
-        'showdebugging' => false,
-        'force' => false,
-    ], [
-        'h' => 'help',
-        'f' => 'force',
-    ]
+    array('help' => false, 'list' => false, 'execute' => false, 'showsql' => false, 'showdebugging' => false),
+    array('h' => 'help')
 );
 
 if ($unrecognized) {
@@ -58,7 +49,6 @@ if ($options['help'] or (!$options['list'] and !$options['execute'])) {
     --showsql             Show sql queries before they are executed
     --showdebugging       Show developer level debugging information
     -h, --help            Print out this help
-    -f, --force           Execute task even if cron is disabled
 
     Example:
     \$sudo -u www-data /usr/bin/php admin/cli/scheduled_task.php --execute=\\core\\task\\session_cleanup_task
@@ -131,13 +121,6 @@ if ($execute = $options['execute']) {
         exit(1);
     }
 
-    if (!get_config('core', 'cron_enabled') && !$options['force']) {
-        mtrace('Cron is disabled. Use --force to override.');
-        exit(1);
-    }
-
-    \core\task\manager::scheduled_task_starting($task);
-
     // Increase memory limit.
     raise_memory_limit(MEMORY_EXTRA);
 

admin/cli/svgtool.php

@@ -38,17 +38,17 @@ if ($unrecognized) {
 }
 
 // If necessary add files that should be ignored - such as in 3rd party plugins.
-$ignorelist = array();
+$blacklist = array();
 $path = $options['path'];
 if (!file_exists($path)) {
     cli_error("Invalid path $path");
 }
 
 if ($options['ie9fix']) {
-    core_admin_recurse_svgs($path, '', 'core_admin_svgtool_ie9fix', $ignorelist);
+    core_admin_recurse_svgs($path, '', 'core_admin_svgtool_ie9fix', $blacklist);
 
 } else if ($options['noaspectratio']) {
-    core_admin_recurse_svgs($path, '', 'core_admin_svgtool_noaspectratio', $ignorelist);
+    core_admin_recurse_svgs($path, '', 'core_admin_svgtool_noaspectratio', $blacklist);
 
 } else {
     $help =
@@ -153,9 +153,9 @@ function core_admin_svgtool_noaspectratio($file) {
  * @param string $base
  * @param string $sub
  * @param string $filecallback
- * @param array $ignorelist List of files to be ignored and skipped.
+ * @param array $blacklist
  */
-function core_admin_recurse_svgs($base, $sub, $filecallback, $ignorelist) {
+function core_admin_recurse_svgs($base, $sub, $filecallback, $blacklist) {
     if (is_dir("$base/$sub")) {
         $items = new DirectoryIterator("$base/$sub");
         foreach ($items as $item) {
@@ -163,7 +163,7 @@ function core_admin_recurse_svgs($base, $sub, $filecallback, $ignorelist) {
                 continue;
             }
             $file = $item->getFilename();
-            core_admin_recurse_svgs("$base/$sub", $file, $filecallback, $ignorelist);
+            core_admin_recurse_svgs("$base/$sub", $file, $filecallback, $blacklist);
         }
         unset($item);
         unset($items);
@@ -174,7 +174,7 @@ function core_admin_recurse_svgs($base, $sub, $filecallback, $ignorelist) {
             return;
         }
         $file = realpath("$base/$sub");
-        if (in_array($file, $ignorelist)) {
+        if (in_array($file, $blacklist)) {
             return;
         }
         $filecallback($file);

admin/environment.xml

@@ -3447,7 +3447,7 @@
       </PHP_EXTENSION>
       <PHP_EXTENSION name="hash" level="required"/>
       <PHP_EXTENSION name="fileinfo" level="required"/>
-      <PHP_EXTENSION name="sodium" level="optional"/>
+      <PHP_EXTENSION name="sodium" level="required"/>
     </PHP_EXTENSIONS>
     <PHP_SETTINGS>
       <PHP_SETTING name="memory_limit" value="96M" level="required">
@@ -3527,193 +3527,6 @@
           <ON_CHECK message="sixtyfourbitswarning" />
         </FEEDBACK>
       </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_max_input_vars" level="optional">
-      </CUSTOM_CHECK>
-    </CUSTOM_CHECKS>
-  </MOODLE>
-  <MOODLE version="4.0" requires="3.6">
-    <UNICODE level="required">
-      <FEEDBACK>
-        <ON_ERROR message="unicoderequired" />
-      </FEEDBACK>
-    </UNICODE>
-    <DATABASE level="required">
-      <VENDOR name="mariadb" version="10.2.29" />
-      <VENDOR name="mysql" version="5.7" />
-      <VENDOR name="postgres" version="10" />
-      <VENDOR name="mssql" version="14.0" />
-      <VENDOR name="oracle" version="11.2" />
-    </DATABASE>
-    <PHP version="7.3.0" level="required">
-    </PHP>
-    <PCREUNICODE level="optional">
-      <FEEDBACK>
-        <ON_CHECK message="pcreunicodewarning" />
-      </FEEDBACK>
-    </PCREUNICODE>
-    <PHP_EXTENSIONS>
-      <PHP_EXTENSION name="iconv" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="iconvrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="mbstring" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="mbstringrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="curl" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="curlrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="openssl" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="opensslrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="tokenizer" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="tokenizerrecommended" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="xmlrpc" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="xmlrpcrecommended" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="soap" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="soaprecommended" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="ctype" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="ctyperequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="zip" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="ziprequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="zlib" level="required">
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="gd" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="gdrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="simplexml" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="simplexmlrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="spl" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="splrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="pcre" level="required">
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="dom" level="required">
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="xml" level="required">
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="xmlreader" level="required">
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="intl" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="intlrequired" />
-        </FEEDBACK>
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="json" level="required">
-      </PHP_EXTENSION>
-      <PHP_EXTENSION name="hash" level="required"/>
-      <PHP_EXTENSION name="fileinfo" level="required"/>
-      <PHP_EXTENSION name="sodium" level="optional"/>
-      <PHP_EXTENSION name="exif" level="optional"/>
-    </PHP_EXTENSIONS>
-    <PHP_SETTINGS>
-      <PHP_SETTING name="memory_limit" value="96M" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="settingmemorylimit" />
-        </FEEDBACK>
-      </PHP_SETTING>
-      <PHP_SETTING name="file_uploads" value="1" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="settingfileuploads" />
-        </FEEDBACK>
-      </PHP_SETTING>
-      <PHP_SETTING name="opcache.enable" value="1" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="opcacherecommended" />
-        </FEEDBACK>
-      </PHP_SETTING>
-    </PHP_SETTINGS>
-    <CUSTOM_CHECKS>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_database_storage_engine" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="unsupporteddbstorageengine" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="question/engine/upgrade/upgradelib.php" function="quiz_attempts_upgraded" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="quizattemptsupgradedmessage" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_slasharguments" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="slashargumentswarning" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_database_tables_row_format" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="unsupporteddbtablerowformat" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_unoconv_version" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="unoconvwarning" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_libcurl_version" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="libcurlwarning" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_mysql_file_format" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="unsupporteddbfileformat" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_mysql_file_per_table" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="unsupporteddbfilepertable" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_mysql_large_prefix" level="required">
-        <FEEDBACK>
-          <ON_ERROR message="unsupporteddblargeprefix" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_is_https" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="ishttpswarning" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_mysql_incomplete_unicode_support" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="incompleteunicodesupport" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_sixtyfour_bits" level="optional">
-        <FEEDBACK>
-          <ON_CHECK message="sixtyfourbitswarning" />
-        </FEEDBACK>
-      </CUSTOM_CHECK>
-      <CUSTOM_CHECK file="lib/upgradelib.php" function="check_max_input_vars" level="optional">
-      </CUSTOM_CHECK>
     </CUSTOM_CHECKS>
   </MOODLE>
 </COMPATIBILITY_MATRIX>

admin/roles/tests/privacy_test.php

@@ -536,7 +536,7 @@ class core_role_privacy_testcase extends provider_testcase {
             $user1->id,
             $admin->id
         ];
-        $this->assertEqualsCanonicalizing($expected, $userlist2->get_userids());
+        $this->assertEquals($expected, $userlist2->get_userids(), '', 0.0, 10, true);
 
         // The user list for coursecontext1 should user1, user2 and admin (role creator).
         $userlist3 = new \core_privacy\local\request\userlist($coursecontext1, $component);
@@ -547,7 +547,7 @@ class core_role_privacy_testcase extends provider_testcase {
             $user2->id,
             $admin->id
         ];
-        $this->assertEqualsCanonicalizing($expected, $userlist3->get_userids());
+        $this->assertEquals($expected, $userlist3->get_userids(), '', 0.0, 10, true);
 
         // The user list for coursecatcontext should user2 and admin (role creator).
         $userlist4 = new \core_privacy\local\request\userlist($coursecatcontext, $component);
@@ -557,7 +557,7 @@ class core_role_privacy_testcase extends provider_testcase {
             $user2->id,
             $admin->id
         ];
-        $this->assertEqualsCanonicalizing($expected, $userlist4->get_userids());
+        $this->assertEquals($expected, $userlist4->get_userids(), '', 0.0, 10, true);
 
         // The user list for systemcontext should user1 and admin (role creator).
         $userlist6 = new \core_privacy\local\request\userlist($systemcontext, $component);
@@ -567,7 +567,7 @@ class core_role_privacy_testcase extends provider_testcase {
             $user1->id,
             $admin->id
         ];
-        $this->assertEqualsCanonicalizing($expected, $userlist6->get_userids());
+        $this->assertEquals($expected, $userlist6->get_userids(), '', 0.0, 10, true);
 
         // The user list for cmcontext should user1, user2 and admin (role creator).
         $userlist7 = new \core_privacy\local\request\userlist($cmcontext, $component);
@@ -578,7 +578,7 @@ class core_role_privacy_testcase extends provider_testcase {
             $user2->id,
             $admin->id
         ];
-        $this->assertEqualsCanonicalizing($expected, $userlist7->get_userids());
+        $this->assertEquals($expected, $userlist7->get_userids(), '', 0.0, 10, true);
 
         // The user list for blockcontext should user1 and admin (role creator).
         $userlist8 = new \core_privacy\local\request\userlist($blockcontext, $component);
@@ -588,7 +588,7 @@ class core_role_privacy_testcase extends provider_testcase {
             $user1->id,
             $admin->id
         ];
-        $this->assertEqualsCanonicalizing($expected, $userlist8->get_userids());
+        $this->assertEquals($expected, $userlist8->get_userids(), '', 0.0, 10, true);
     }
 
     /**
@@ -744,4 +744,4 @@ class core_role_privacy_testcase extends provider_testcase {
         }
         return $rolesnames;
     }
-}
+}

admin/search.php

@@ -68,16 +68,9 @@ if ($errormsg !== '') {
 $showsettingslinks = true;
 
 if ($hassiteconfig) {
-    $data = [
-        'action' => new moodle_url('/admin/search.php'),
-        'btnclass' => 'btn-primary',
-        'inputname' => 'query',
-        'searchstring' => get_string('search'),
-        'query' => $query,
-        'extraclasses' => 'd-flex justify-content-center'
-    ];
-    echo $OUTPUT->render_from_template('core/search_input', $data);
-
+    require_once("admin_settings_search_form.php");
+    $form = new admin_settings_search_form();
+    $form->display();
     echo '<hr>';
     if ($query) {
         echo admin_search_settings_html($query);

admin/settings/courses.php

@@ -22,8 +22,6 @@
  * @license http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */
 
-use core_admin\local\settings\filesize;
-
 $capabilities = array(
     'moodle/backup:backupcourse',
     'moodle/category:manage',
@@ -77,17 +75,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
     $temp->add(new admin_setting_configselect('moodlecourse/visible', new lang_string('visible'), new lang_string('visible_help'),
         1, $choices));
 
-    // Enable/disable download course content.
-    $choices = [
-        DOWNLOAD_COURSE_CONTENT_DISABLED => new lang_string('no'),
-        DOWNLOAD_COURSE_CONTENT_ENABLED => new lang_string('yes'),
-    ];
-    $downloadcontentsitedefault = new admin_setting_configselect('moodlecourse/downloadcontentsitedefault',
-            new lang_string('enabledownloadcoursecontent', 'course'),
-            new lang_string('downloadcoursecontent_help', 'course'), 0, $choices);
-    $downloadcontentsitedefault->add_dependent_on('downloadcoursecontentallowed');
-    $temp->add($downloadcontentsitedefault);
-
     // Course format.
     $temp->add(new admin_setting_heading('courseformathdr', new lang_string('type_format', 'plugin'), ''));
 
@@ -174,21 +161,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
 
     $ADMIN->add('courses', $temp);
 
-    // Download course content.
-    $downloadcoursedefaulturl = new moodle_url('/admin/settings.php', ['section' => 'coursesettings']);
-    $temp = new admin_settingpage('downloadcoursecontent', new lang_string('downloadcoursecontent', 'course'));
-    $temp->add(new admin_setting_configcheckbox('downloadcoursecontentallowed',
-            new lang_string('downloadcoursecontentallowed', 'admin'),
-            new lang_string('downloadcoursecontentallowed_desc', 'admin', $downloadcoursedefaulturl->out()), 0));
-
-    // 50MB default maximum size per file when downloading course content.
-    $defaultmaxdownloadsize = 50 * filesize::UNIT_MB;
-    $temp->add(new filesize('maxsizeperdownloadcoursefile', new lang_string('maxsizeperdownloadcoursefile', 'admin'),
-            new lang_string('maxsizeperdownloadcoursefile_desc', 'admin'), $defaultmaxdownloadsize, filesize::UNIT_MB));
-    $temp->hide_if('maxsizeperdownloadcoursefile', 'downloadcoursecontentallowed');
-
-    $ADMIN->add('courses', $temp);
-
     // "courserequests" settingpage.
     $temp = new admin_settingpage('courserequest', new lang_string('courserequest'));
     $temp->add(new admin_setting_configcheckbox('enablecourserequests',
@@ -212,7 +184,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
     // Add a category for the Activity Chooser.
     $ADMIN->add('courses', new admin_category('activitychooser', new lang_string('activitychoosercategory', 'course')));
     $temp = new admin_settingpage('activitychoosersettings', new lang_string('activitychoosersettings', 'course'));
-    // Tab mode for the activity chooser.
     $temp->add(
         new admin_setting_configselect(
             'activitychoosertabmode',
@@ -226,31 +197,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
             ]
         )
     );
-
-    // Build a list of plugins that use the footer callback.
-    $pluginswithfunction = get_plugins_with_function('custom_chooser_footer', 'lib.php');
-    $pluginsoptions = [];
-    $pluginsoptions[COURSE_CHOOSER_FOOTER_NONE] = get_string('activitychooserhidefooter', 'course');
-    if ($pluginswithfunction) {
-        foreach ($pluginswithfunction as $plugintype => $plugins) {
-            foreach ($plugins as $pluginname => $pluginfunction) {
-                $plugin = $plugintype.'_'.$pluginname;
-                $pluginsoptions[$plugin] = get_string('pluginname', $plugin);
-            }
-        }
-    }
-
-    // Select what plugin to show in the footer.
-    $temp->add(
-        new admin_setting_configselect(
-            'activitychooseractivefooter',
-            new lang_string('activitychooseractivefooter', 'course'),
-            new lang_string('activitychooseractivefooter_desc', 'course'),
-            COURSE_CHOOSER_FOOTER_NONE,
-            $pluginsoptions
-        )
-    );
-
     $ADMIN->add('activitychooser', $temp);
     $ADMIN->add('activitychooser',
         new admin_externalpage('activitychooserrecommended', new lang_string('activitychooserrecommendations', 'course'),
@@ -313,9 +259,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
         ['value' => 1, 'locked' => 0])
     );
 
-    $temp->add(new admin_setting_configcheckbox_with_lock('backup/backup_general_legacyfiles',
-        new lang_string('generallegacyfiles', 'backup'),
-        new lang_string('configlegacyfiles', 'backup'), array('value' => 1, 'locked' => 0)));
     $ADMIN->add('backups', $temp);
 
     // Create a page for general import configuration and defaults.
@@ -342,9 +285,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
         new lang_string('configgeneralcontentbankcontent', 'backup'),
         ['value' => 1, 'locked' => 0])
     );
-    $temp->add(new admin_setting_configcheckbox_with_lock('backup/backup_import_legacyfiles',
-        new lang_string('generallegacyfiles', 'backup'),
-        new lang_string('configlegacyfiles', 'backup'), array('value' => 1, 'locked' => 0)));
 
     $ADMIN->add('backups', $temp);
 
@@ -471,10 +411,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
         1)
     );
 
-    $temp->add(new admin_setting_configcheckbox('backup/backup_auto_legacyfiles',
-        new lang_string('generallegacyfiles', 'backup'),
-        new lang_string('configlegacyfiles', 'backup'), 1));
-
     //$temp->add(new admin_setting_configcheckbox('backup/backup_auto_messages', new lang_string('messages', 'message'), new lang_string('backupmessageshelp','message'), 0));
     //$temp->add(new admin_setting_configcheckbox('backup/backup_auto_blogs', new lang_string('blogs', 'blog'), new lang_string('backupblogshelp','blog'), 0));
 
@@ -537,9 +473,6 @@ if ($hassiteconfig or has_any_capability($capabilities, $systemcontext)) {
     $temp->add(new admin_setting_configcheckbox_with_lock('restore/restore_general_contentbankcontent',
         new lang_string('generalcontentbankcontent', 'backup'),
         new lang_string('configrestorecontentbankcontent', 'backup'), array('value' => 1, 'locked' => 0)));
-    $temp->add(new admin_setting_configcheckbox_with_lock('restore/restore_general_legacyfiles',
-        new lang_string('generallegacyfiles', 'backup'),
-        new lang_string('configlegacyfiles', 'backup'), array('value' => 1, 'locked' => 0)));
 
     // Restore defaults when merging into another course.
     $temp->add(new admin_setting_heading('mergerestoredefaults', new lang_string('mergerestoredefaults', 'backup'), ''));

admin/settings/development.php

@@ -38,17 +38,9 @@ if ($hassiteconfig) { // speedup for non-admins, add all caps used on this page
     // "debugging" settingpage
     $temp = new admin_settingpage('debugging', new lang_string('debugging', 'admin'));
     $temp->add(new admin_setting_special_debug());
-    $temp->add(new admin_setting_configcheckbox('debugdisplay', new lang_string('debugdisplay', 'admin'),
-        new lang_string('configdebugdisplay', 'admin'), 0));
+    $temp->add(new admin_setting_configcheckbox('debugdisplay', new lang_string('debugdisplay', 'admin'), new lang_string('configdebugdisplay', 'admin'), ini_get_bool('display_errors')));
     $temp->add(new admin_setting_configcheckbox('perfdebug', new lang_string('perfdebug', 'admin'), new lang_string('configperfdebug', 'admin'), '7', '15', '7'));
     $temp->add(new admin_setting_configcheckbox('debugstringids', new lang_string('debugstringids', 'admin'), new lang_string('debugstringids_desc', 'admin'), 0));
-    $temp->add(new admin_setting_configselect('debugsqltrace',
-            new lang_string('debugsqltrace', 'admin'),
-            new lang_string('debugsqltrace_desc', 'admin'), 0, array(
-               0 => new lang_string('disabled', 'admin'),
-               1 => new lang_string('debugsqltrace1', 'admin'),
-               2 => new lang_string('debugsqltrace2', 'admin'),
-             100 => new lang_string('debugsqltrace100', 'admin'))));
     $temp->add(new admin_setting_configcheckbox('debugvalidators', new lang_string('debugvalidators', 'admin'), new lang_string('configdebugvalidators', 'admin'), 0));
     $temp->add(new admin_setting_configcheckbox('debugpageinfo', new lang_string('debugpageinfo', 'admin'), new lang_string('configdebugpageinfo', 'admin'), 0));
     $ADMIN->add('development', $temp);

admin/settings/language.php

@@ -8,8 +8,6 @@ if ($hassiteconfig) {
     $temp = new admin_settingpage('langsettings', new lang_string('languagesettings', 'admin'));
     $temp->add(new admin_setting_configcheckbox('autolang', new lang_string('autolang', 'admin'), new lang_string('configautolang', 'admin'), 1));
     $temp->add(new admin_setting_configselect('lang', new lang_string('lang', 'admin'), new lang_string('configlang', 'admin'), current_language(), get_string_manager()->get_list_of_translations())); // $CFG->lang might be set in installer already, default en is in setup.php
-    $temp->add(new admin_setting_configcheckbox('autolangusercreation', new lang_string('autolangusercreation', 'admin'),
-        new lang_string('configautolangusercreation', 'admin'), 1));
     $temp->add(new admin_setting_configcheckbox('langmenu', new lang_string('langmenu', 'admin'), new lang_string('configlangmenu', 'admin'), 1));
     $temp->add(new admin_setting_langlist());
     $temp->add(new admin_setting_configcheckbox('langcache', new lang_string('langcache', 'admin'), new lang_string('langcache_desc', 'admin'), 1));

admin/settings/payment.php

@@ -1,29 +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/>.
-
-/**
- * Adds payments links to the admin tree
- *
- * @package    core
- * @copyright  2020 Marina Glancy
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-$ADMIN->add('payment', new admin_externalpage(
-    'paymentaccounts',
-    new lang_string('paymentaccounts', 'payment'),
-    new moodle_url("/payment/accounts.php"),
-    ['moodle/payment:manageaccounts', 'moodle/payment:viewpayments']));

admin/settings/plugins.php

@@ -167,42 +167,6 @@ if ($hassiteconfig) {
     $ADMIN->add('modules', new admin_category('antivirussettings', new lang_string('antiviruses', 'antivirus')));
     $temp = new admin_settingpage('manageantiviruses', new lang_string('antivirussettings', 'antivirus'));
     $temp->add(new admin_setting_manageantiviruses());
-
-    // Common settings.
-    $temp->add(new admin_setting_heading('antiviruscommonsettings', new lang_string('antiviruscommonsettings', 'antivirus'), ''));
-
-    // Alert email.
-    $temp->add(
-        new admin_setting_configtext(
-            'antivirus/notifyemail',
-            new lang_string('notifyemail', 'antivirus'),
-            new lang_string('notifyemail_help', 'antivirus'),
-            '',
-            PARAM_EMAIL
-        )
-    );
-
-    // Enable quarantine.
-    $temp->add(
-        new admin_setting_configcheckbox(
-            'antivirus/enablequarantine',
-            new lang_string('enablequarantine', 'antivirus'),
-            new lang_string('enablequarantine_help', 'antivirus',
-            \core\antivirus\quarantine::DEFAULT_QUARANTINE_FOLDER),
-            0
-        )
-    );
-
-    // Quarantine time.
-    $temp->add(
-        new admin_setting_configduration(
-            'antivirus/quarantinetime',
-            new lang_string('quarantinetime', 'antivirus'),
-            new lang_string('quarantinetime_desc', 'antivirus'),
-            \core\antivirus\quarantine::DEFAULT_QUARANTINE_TIME
-        )
-    );
-
     $ADMIN->add('antivirussettings', $temp);
     $plugins = core_plugin_manager::instance()->get_plugins_of_type('antivirus');
     core_collator::asort_objects_by_property($plugins, 'displayname');
@@ -280,25 +244,6 @@ if ($hassiteconfig) {
         $plugin->load_settings($ADMIN, 'mediaplayers', $hassiteconfig);
     }
 
-    // Payment gateway plugins.
-    $ADMIN->add('modules', new admin_category('paymentgateways', new lang_string('type_paygw_plural', 'plugin')));
-    $temp = new admin_settingpage('managepaymentgateways', new lang_string('type_paygwmanage', 'plugin'));
-    $temp->add(new \core_admin\local\settings\manage_payment_gateway_plugins());
-    $temp->add(new admin_setting_description(
-        'managepaymentgatewayspostfix',
-        '',
-        new lang_string('gotopaymentaccounts', 'payment',
-            html_writer::link(new moodle_url('/payment/accounts.php'), get_string('paymentaccounts', 'payment')))
-    ));
-    $ADMIN->add('paymentgateways', $temp);
-
-    $plugins = core_plugin_manager::instance()->get_plugins_of_type('paygw');
-    core_collator::asort_objects_by_property($plugins, 'displayname');
-    foreach ($plugins as $plugin) {
-        /** @var \core\plugininfo\paygw $plugin */
-        $plugin->load_settings($ADMIN, 'paymentgateways', $hassiteconfig);
-    }
-
     // Data format settings.
     $ADMIN->add('modules', new admin_category('dataformatsettings', new lang_string('dataformats')));
     $temp = new admin_settingpage('managedataformats', new lang_string('managedataformats'));
@@ -604,19 +549,8 @@ if ($hassiteconfig) {
 
     // Search engine selection.
     $temp->add(new admin_setting_heading('searchengineheading', new lang_string('searchengine', 'admin'), ''));
-    $searchengineselect = new admin_setting_configselect('searchengine',
-            new lang_string('selectsearchengine', 'admin'), '', 'simpledb', $engines);
-    $searchengineselect->set_validate_function(function(string $value): string {
-        global $CFG;
-
-        // Check nobody's setting the indexing and query-only server to the same one.
-        if (isset($CFG->searchenginequeryonly) && $CFG->searchenginequeryonly === $value) {
-            return get_string('searchenginequeryonlysame', 'admin');
-        } else {
-            return '';
-        }
-    });
-    $temp->add($searchengineselect);
+    $temp->add(new admin_setting_configselect('searchengine',
+                                new lang_string('selectsearchengine', 'admin'), '', 'simpledb', $engines));
     $temp->add(new admin_setting_heading('searchoptionsheading', new lang_string('searchoptions', 'admin'), ''));
     $temp->add(new admin_setting_configcheckbox('searchindexwhendisabled',
             new lang_string('searchindexwhendisabled', 'admin'), new lang_string('searchindexwhendisabled_desc', 'admin'),
@@ -656,43 +590,6 @@ if ($hassiteconfig) {
         new lang_string('searchhideallcategory_desc', 'admin'),
         0));
 
-    $temp->add(new admin_setting_heading('searchmanagement', new lang_string('searchmanagement', 'admin'),
-            new lang_string('searchmanagement_desc', 'admin')));
-
-    // Get list of search engines including those with alternate settings.
-    $searchenginequeryonlyselect = new admin_setting_configselect('searchenginequeryonly',
-            new lang_string('searchenginequeryonly', 'admin'),
-            new lang_string('searchenginequeryonly_desc', 'admin'), '', function() use($engines) {
-                $options = ['' => new lang_string('searchenginequeryonly_none', 'admin')];
-                foreach ($engines as $name => $display) {
-                    $options[$name] = $display;
-
-                    $classname = '\search_' . $name . '\engine';
-                    $engine = new $classname;
-                    if ($engine->has_alternate_configuration()) {
-                        $options[$name . '-alternate'] =
-                                new lang_string('searchenginealternatesettings', 'admin', $display);
-                    }
-                }
-                return $options;
-            });
-    $searchenginequeryonlyselect->set_validate_function(function(string $value): string {
-        global $CFG;
-
-        // Check nobody's setting the indexing and query-only server to the same one.
-        if (isset($CFG->searchengine) && $CFG->searchengine === $value) {
-            return get_string('searchenginequeryonlysame', 'admin');
-        } else {
-            return '';
-        }
-    });
-    $temp->add($searchenginequeryonlyselect);
-    $temp->add(new admin_setting_configcheckbox('searchbannerenable',
-            new lang_string('searchbannerenable', 'admin'), new lang_string('searchbannerenable_desc', 'admin'),
-            0));
-    $temp->add(new admin_setting_confightmleditor('searchbanner',
-            new lang_string('searchbanner', 'admin'), '', ''));
-
     $ADMIN->add('searchplugins', $temp);
     $ADMIN->add('searchplugins', new admin_externalpage('searchareas', new lang_string('searchareas', 'admin'),
         new moodle_url('/admin/searchareas.php')));

admin/settings/security.php

@@ -160,23 +160,6 @@ if ($hassiteconfig) { // speedup for non-admins, add all caps used on this page
     $temp->add(new admin_setting_configportlist('curlsecurityallowedport',
                new lang_string('curlsecurityallowedport', 'admin'),
                new lang_string('curlsecurityallowedportsyntax', 'admin'), ""));
-
-    // HTTP Header referrer policy settings.
-    $referreroptions = [
-        'default' => get_string('referrernone', 'admin'),
-        'no-referrer' => 'no-referrer',
-        'no-referrer-when-downgrade' => 'no-referrer-when-downgrade',
-        'origin' => 'origin',
-        'origin-when-cross-origin' => 'origin-when-cross-origin',
-        'same-origin' => 'same-origin',
-        'strict-origin' => 'strict-origin',
-        'strict-origin-when-cross-origin' => 'strict-origin-when-cross-origin',
-        'unsafe-url' => 'unsafe-url',
-    ];
-    $temp->add(new admin_setting_configselect('referrerpolicy',
-            new lang_string('referrerpolicy', 'admin'),
-            new lang_string('referrerpolicydesc', 'admin'), 'default', $referreroptions));
-
     $ADMIN->add('security', $temp);
 
     // "notifications" settingpage

admin/settings/server.php

@@ -1,225 +1,162 @@
 <?php
-// This file is part of Moodle - https://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/>.
 
-/**
- * Defines settingpages and externalpages under the "server" category.
- *
- * @package     core
- * @category    admin
- * @copyright   2006 Martin Dougiamas
- * @license     http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
+// This file defines settingpages and externalpages under the "server" category
 
-defined('MOODLE_INTERNAL') || die();
+if ($hassiteconfig) { // speedup for non-admins, add all caps used on this page
 
-if ($hassiteconfig) {
-    // System paths.
-    $temp = new admin_settingpage('systempaths', new lang_string('systempaths', 'admin'));
-    $temp->add(new admin_setting_configexecutable('pathtophp', new lang_string('pathtophp', 'admin'),
-        new lang_string('configpathtophp', 'admin'), ''));
-    $temp->add(new admin_setting_configexecutable('pathtodu', new lang_string('pathtodu', 'admin'),
-        new lang_string('configpathtodu', 'admin'), ''));
-    $temp->add(new admin_setting_configexecutable('aspellpath', new lang_string('aspellpath', 'admin'),
-        new lang_string('edhelpaspellpath'), ''));
-    $temp->add(new admin_setting_configexecutable('pathtodot', new lang_string('pathtodot', 'admin'),
-        new lang_string('pathtodot_help', 'admin'), ''));
-    $temp->add(new admin_setting_configexecutable('pathtogs', new lang_string('pathtogs', 'admin'),
-        new lang_string('pathtogs_help', 'admin'), '/usr/bin/gs'));
-    $temp->add(new admin_setting_configexecutable('pathtopython', new lang_string('pathtopython', 'admin'),
-        new lang_string('pathtopythondesc', 'admin'), ''));
-    $ADMIN->add('server', $temp);
+// "systempaths" settingpage
+$temp = new admin_settingpage('systempaths', new lang_string('systempaths','admin'));
+$temp->add(new admin_setting_configexecutable('pathtophp', new lang_string('pathtophp', 'admin'),
+    new lang_string('configpathtophp', 'admin'), ''));
+$temp->add(new admin_setting_configexecutable('pathtodu', new lang_string('pathtodu', 'admin'), new lang_string('configpathtodu', 'admin'), ''));
+$temp->add(new admin_setting_configexecutable('aspellpath', new lang_string('aspellpath', 'admin'), new lang_string('edhelpaspellpath'), ''));
+$temp->add(new admin_setting_configexecutable('pathtodot', new lang_string('pathtodot', 'admin'), new lang_string('pathtodot_help', 'admin'), ''));
+$temp->add(new admin_setting_configexecutable('pathtogs', new lang_string('pathtogs', 'admin'), new lang_string('pathtogs_help', 'admin'), '/usr/bin/gs'));
+$temp->add(new admin_setting_configexecutable('pathtopython', new lang_string('pathtopython', 'admin'),
+    new lang_string('pathtopythondesc', 'admin'), ''));
+$ADMIN->add('server', $temp);
 
-    // Support contact.
-    $temp = new admin_settingpage('supportcontact', new lang_string('supportcontact', 'admin'));
-    $primaryadmin = get_admin();
-    if ($primaryadmin) {
-        $primaryadminemail = $primaryadmin->email;
-        $primaryadminname = fullname($primaryadmin, true);
-    } else {
-        // No defaults during installation - admin user must be created first.
-        $primaryadminemail = null;
-        $primaryadminname = null;
-    }
-    $temp->add(new admin_setting_configtext('supportname', new lang_string('supportname', 'admin'),
-        new lang_string('configsupportname', 'admin'), $primaryadminname, PARAM_NOTAGS));
-    $setting = new admin_setting_configtext('supportemail', new lang_string('supportemail', 'admin'),
-        new lang_string('configsupportemail', 'admin'), $primaryadminemail, PARAM_EMAIL);
-    $setting->set_force_ltr(true);
-    $temp->add($setting);
-    $temp->add(new admin_setting_configtext('supportpage', new lang_string('supportpage', 'admin'),
-        new lang_string('configsupportpage', 'admin'), '', PARAM_URL));
-    $ADMIN->add('server', $temp);
 
-    // Session handling.
-    $temp = new admin_settingpage('sessionhandling', new lang_string('sessionhandling', 'admin'));
-    if (empty($CFG->session_handler_class) and $DB->session_lock_supported()) {
-        $temp->add(new admin_setting_configcheckbox('dbsessions', new lang_string('dbsessions', 'admin'),
-            new lang_string('configdbsessions', 'admin'), 0));
-    }
 
-    $temp->add(new admin_setting_configduration('sessiontimeout', new lang_string('sessiontimeout', 'admin'),
-        new lang_string('configsessiontimeout', 'admin'), 8 * 60 * 60));
+// "supportcontact" settingpage
+$temp = new admin_settingpage('supportcontact', new lang_string('supportcontact','admin'));
+$primaryadmin = get_admin();
+if ($primaryadmin) {
+    $primaryadminemail = $primaryadmin->email;
+    $primaryadminname  = fullname($primaryadmin, true);
+} else {
+    // no defaults during installation - admin user must be created first
+    $primaryadminemail = NULL;
+    $primaryadminname  = NULL;
+}
+$temp->add(new admin_setting_configtext('supportname', new lang_string('supportname', 'admin'),
+    new lang_string('configsupportname', 'admin'), $primaryadminname, PARAM_NOTAGS));
+$setting = new admin_setting_configtext('supportemail', new lang_string('supportemail', 'admin'),
+    new lang_string('configsupportemail', 'admin'), $primaryadminemail, PARAM_EMAIL);
+$setting->set_force_ltr(true);
+$temp->add($setting);
+$temp->add(new admin_setting_configtext('supportpage', new lang_string('supportpage', 'admin'), new lang_string('configsupportpage', 'admin'), '', PARAM_URL));
+$ADMIN->add('server', $temp);
 
-    $temp->add(new admin_setting_configtext('sessioncookie', new lang_string('sessioncookie', 'admin'),
-        new lang_string('configsessioncookie', 'admin'), '', PARAM_ALPHANUM));
-    $temp->add(new admin_setting_configtext('sessioncookiepath', new lang_string('sessioncookiepath', 'admin'),
-        new lang_string('configsessioncookiepath', 'admin'), '', PARAM_RAW));
-    $temp->add(new admin_setting_configtext('sessioncookiedomain', new lang_string('sessioncookiedomain', 'admin'),
-        new lang_string('configsessioncookiedomain', 'admin'), '', PARAM_RAW, 50));
-    $ADMIN->add('server', $temp);
 
-    // Statistics.
-    $temp = new admin_settingpage('stats', new lang_string('stats'), 'moodle/site:config', empty($CFG->enablestats));
-    $temp->add(new admin_setting_configselect('statsfirstrun', new lang_string('statsfirstrun', 'admin'),
-        new lang_string('configstatsfirstrun', 'admin'), 'none',
-        [
-            'none' => new lang_string('none'),
-            60 * 60 * 24 * 7 => new lang_string('numweeks', 'moodle', 1),
-            60 * 60 * 24 * 14 => new lang_string('numweeks', 'moodle', 2),
-            60 * 60 * 24 * 21 => new lang_string('numweeks', 'moodle', 3),
-            60 * 60 * 24 * 28 => new lang_string('nummonths', 'moodle', 1),
-            60 * 60 * 24 * 56 => new lang_string('nummonths', 'moodle', 2),
-            60 * 60 * 24 * 84 => new lang_string('nummonths', 'moodle', 3),
-            60 * 60 * 24 * 112 => new lang_string('nummonths', 'moodle', 4),
-            60 * 60 * 24 * 140 => new lang_string('nummonths', 'moodle', 5),
-            60 * 60 * 24 * 168 => new lang_string('nummonths', 'moodle', 6),
-            'all' => new lang_string('all')
-        ]
-    ));
-    $temp->add(new admin_setting_configselect('statsmaxruntime', new lang_string('statsmaxruntime', 'admin'),
-        new lang_string('configstatsmaxruntime3', 'admin'), 0,
-        [
-            0 => new lang_string('untilcomplete'),
-            60 * 30 => '10 ' . new lang_string('minutes'),
-            60 * 30 => '30 ' . new lang_string('minutes'),
-            60 * 60 => '1 ' . new lang_string('hour'),
-            60 * 60 * 2 => '2 ' . new lang_string('hours'),
-            60 * 60 * 3 => '3 ' . new lang_string('hours'),
-            60 * 60 * 4 => '4 ' . new lang_string('hours'),
-            60 * 60 * 5 => '5 ' . new lang_string('hours'),
-            60 * 60 * 6 => '6 ' . new lang_string('hours'),
-            60 * 60 * 7 => '7 ' . new lang_string('hours'),
-            60 * 60 * 8 => '8 ' . new lang_string('hours'),
-        ]
-    ));
-    $temp->add(new admin_setting_configtext('statsruntimedays', new lang_string('statsruntimedays', 'admin'),
-        new lang_string('configstatsruntimedays', 'admin'), 31, PARAM_INT));
-    $temp->add(new admin_setting_configtext('statsuserthreshold', new lang_string('statsuserthreshold', 'admin'),
-        new lang_string('configstatsuserthreshold', 'admin'), 0, PARAM_INT));
-    $ADMIN->add('server', $temp);
+// "sessionhandling" settingpage
+$temp = new admin_settingpage('sessionhandling', new lang_string('sessionhandling', 'admin'));
+if (empty($CFG->session_handler_class) and $DB->session_lock_supported()) {
+    $temp->add(new admin_setting_configcheckbox('dbsessions', new lang_string('dbsessions', 'admin'), new lang_string('configdbsessions', 'admin'), 0));
+}
 
-    // HTTP.
-    $temp = new admin_settingpage('http', new lang_string('http', 'admin'));
-    $temp->add(new admin_setting_configcheckbox('slasharguments', new lang_string('slasharguments', 'admin'),
-        new lang_string('configslasharguments', 'admin'), 1));
-    $temp->add(new admin_setting_heading('reverseproxy', new lang_string('reverseproxy', 'admin'), '', ''));
-    $options = [
-        0 => 'HTTP_CLIENT_IP, HTTP_X_FORWARDED_FOR, REMOTE_ADDR',
-        GETREMOTEADDR_SKIP_HTTP_CLIENT_IP => 'HTTP_X_FORWARDED_FOR, REMOTE_ADDR',
-        GETREMOTEADDR_SKIP_HTTP_X_FORWARDED_FOR => 'HTTP_CLIENT, REMOTE_ADDR',
-        GETREMOTEADDR_SKIP_HTTP_X_FORWARDED_FOR | GETREMOTEADDR_SKIP_HTTP_CLIENT_IP => 'REMOTE_ADDR'
-    ];
-    $temp->add(new admin_setting_configselect('getremoteaddrconf', new lang_string('getremoteaddrconf', 'admin'),
-        new lang_string('configgetremoteaddrconf', 'admin'),
-        GETREMOTEADDR_SKIP_HTTP_X_FORWARDED_FOR | GETREMOTEADDR_SKIP_HTTP_CLIENT_IP, $options));
-    $temp->add(new admin_setting_configtext('reverseproxyignore', new lang_string('reverseproxyignore', 'admin'),
-        new lang_string('configreverseproxyignore', 'admin'), ''));
+$temp->add(new admin_setting_configduration('sessiontimeout', new lang_string('sessiontimeout', 'admin'),
+    new lang_string('configsessiontimeout', 'admin'), 8 * 60 * 60));
 
-    $temp->add(new admin_setting_heading('webproxy', new lang_string('webproxy', 'admin'),
-        new lang_string('webproxyinfo', 'admin')));
-    $temp->add(new admin_setting_configtext('proxyhost', new lang_string('proxyhost', 'admin'),
-        new lang_string('configproxyhost', 'admin'), '', PARAM_HOST));
-    $temp->add(new admin_setting_configtext('proxyport', new lang_string('proxyport', 'admin'),
-        new lang_string('configproxyport', 'admin'), 0, PARAM_INT));
-    $options = ['HTTP' => 'HTTP'];
-    if (defined('CURLPROXY_SOCKS5')) {
-        $options['SOCKS5'] = 'SOCKS5';
-    }
-    $temp->add(new admin_setting_configselect('proxytype', new lang_string('proxytype', 'admin'),
-        new lang_string('configproxytype', 'admin'), 'HTTP', $options));
-    $temp->add(new admin_setting_configtext('proxyuser', new lang_string('proxyuser', 'admin'),
-        new lang_string('configproxyuser', 'admin'), ''));
-    $temp->add(new admin_setting_configpasswordunmask('proxypassword', new lang_string('proxypassword', 'admin'),
-        new lang_string('configproxypassword', 'admin'), ''));
-    $temp->add(new admin_setting_configtext('proxybypass', new lang_string('proxybypass', 'admin'),
-        new lang_string('configproxybypass', 'admin'), 'localhost, 127.0.0.1'));
-    $ADMIN->add('server', $temp);
+$temp->add(new admin_setting_configtext('sessioncookie', new lang_string('sessioncookie', 'admin'), new lang_string('configsessioncookie', 'admin'), '', PARAM_ALPHANUM));
+$temp->add(new admin_setting_configtext('sessioncookiepath', new lang_string('sessioncookiepath', 'admin'), new lang_string('configsessioncookiepath', 'admin'), '', PARAM_RAW));
+$temp->add(new admin_setting_configtext('sessioncookiedomain', new lang_string('sessioncookiedomain', 'admin'), new lang_string('configsessioncookiedomain', 'admin'), '', PARAM_RAW, 50));
+$ADMIN->add('server', $temp);
 
-    $temp = new admin_settingpage('maintenancemode', new lang_string('sitemaintenancemode', 'admin'));
-    $options = [0 => new lang_string('disable'), 1 => new lang_string('enable')];
-    $temp->add(new admin_setting_configselect('maintenance_enabled', new lang_string('sitemaintenancemode', 'admin'),
-        new lang_string('helpsitemaintenance', 'admin'), 0, $options));
-    $temp->add(new admin_setting_confightmleditor('maintenance_message', new lang_string('optionalmaintenancemessage', 'admin'),
-        '', ''));
-    $ADMIN->add('server', $temp);
 
-    // Cleanup.
-    $temp = new admin_settingpage('cleanup', new lang_string('cleanup', 'admin'));
-    $temp->add(new admin_setting_configselect('deleteunconfirmed', new lang_string('deleteunconfirmed', 'admin'),
-        new lang_string('configdeleteunconfirmed', 'admin'), 168,
-        [
-            0 => new lang_string('never'),
-            168 => new lang_string('numdays', '', 7),
-            144 => new lang_string('numdays', '', 6),
-            120 => new lang_string('numdays', '', 5),
-            96 => new lang_string('numdays', '', 4),
-            72 => new lang_string('numdays', '', 3),
-            48 => new lang_string('numdays', '', 2),
-            24 => new lang_string('numdays', '', 1),
-            12 => new lang_string('numhours', '', 12),
-            6 => new lang_string('numhours', '', 6),
-            1 => new lang_string('numhours', '', 1),
-        ]
-    ));
+// "stats" settingpage
+$temp = new admin_settingpage('stats', new lang_string('stats'), 'moodle/site:config', empty($CFG->enablestats));
+$temp->add(new admin_setting_configselect('statsfirstrun', new lang_string('statsfirstrun', 'admin'), new lang_string('configstatsfirstrun', 'admin'), 'none', array('none' => new lang_string('none'),
+                                                                                                                                                           60*60*24*7 => new lang_string('numweeks','moodle',1),
+                                                                                                                                                           60*60*24*14 => new lang_string('numweeks','moodle',2),
+                                                                                                                                                           60*60*24*21 => new lang_string('numweeks','moodle',3),
+                                                                                                                                                           60*60*24*28 => new lang_string('nummonths','moodle',1),
+                                                                                                                                                           60*60*24*56 => new lang_string('nummonths','moodle',2),
+                                                                                                                                                           60*60*24*84 => new lang_string('nummonths','moodle',3),
+                                                                                                                                                           60*60*24*112 => new lang_string('nummonths','moodle',4),
+                                                                                                                                                           60*60*24*140 => new lang_string('nummonths','moodle',5),
+                                                                                                                                                           60*60*24*168 => new lang_string('nummonths','moodle',6),
+                                                                                                                                                           'all' => new lang_string('all') )));
+$temp->add(new admin_setting_configselect('statsmaxruntime', new lang_string('statsmaxruntime', 'admin'), new lang_string('configstatsmaxruntime3', 'admin'), 0, array(0 => new lang_string('untilcomplete'),
+                                                                                                                                                            60*30 => '10 '.new lang_string('minutes'),
+                                                                                                                                                            60*30 => '30 '.new lang_string('minutes'),
+                                                                                                                                                            60*60 => '1 '.new lang_string('hour'),
+                                                                                                                                                            60*60*2 => '2 '.new lang_string('hours'),
+                                                                                                                                                            60*60*3 => '3 '.new lang_string('hours'),
+                                                                                                                                                            60*60*4 => '4 '.new lang_string('hours'),
+                                                                                                                                                            60*60*5 => '5 '.new lang_string('hours'),
+                                                                                                                                                            60*60*6 => '6 '.new lang_string('hours'),
+                                                                                                                                                            60*60*7 => '7 '.new lang_string('hours'),
+                                                                                                                                                            60*60*8 => '8 '.new lang_string('hours') )));
+$temp->add(new admin_setting_configtext('statsruntimedays', new lang_string('statsruntimedays', 'admin'), new lang_string('configstatsruntimedays', 'admin'), 31, PARAM_INT));
+$temp->add(new admin_setting_configtext('statsuserthreshold', new lang_string('statsuserthreshold', 'admin'), new lang_string('configstatsuserthreshold', 'admin'), 0, PARAM_INT));
+$ADMIN->add('server', $temp);
 
-    $temp->add(new admin_setting_configselect('deleteincompleteusers', new lang_string('deleteincompleteusers', 'admin'),
-        new lang_string('configdeleteincompleteusers', 'admin'), 0,
-        [
-            0 => new lang_string('never'),
-            168 => new lang_string('numdays', '', 7),
-            144 => new lang_string('numdays', '', 6),
-            120 => new lang_string('numdays', '', 5),
-            96 => new lang_string('numdays', '', 4),
-            72 => new lang_string('numdays', '', 3),
-            48 => new lang_string('numdays', '', 2),
-            24 => new lang_string('numdays', '', 1),
-        ]
-    ));
 
-    $temp->add(new admin_setting_configcheckbox('disablegradehistory', new lang_string('disablegradehistory', 'grades'),
-        new lang_string('disablegradehistory_help', 'grades'), 0));
+// "http" settingpage
+$temp = new admin_settingpage('http', new lang_string('http', 'admin'));
+$temp->add(new admin_setting_configcheckbox('slasharguments', new lang_string('slasharguments', 'admin'), new lang_string('configslasharguments', 'admin'), 1));
+$temp->add(new admin_setting_heading('reverseproxy', new lang_string('reverseproxy', 'admin'), '', ''));
+$options = array(
+    0 => 'HTTP_CLIENT_IP, HTTP_X_FORWARDED_FOR, REMOTE_ADDR',
+    GETREMOTEADDR_SKIP_HTTP_CLIENT_IP => 'HTTP_X_FORWARDED_FOR, REMOTE_ADDR',
+    GETREMOTEADDR_SKIP_HTTP_X_FORWARDED_FOR => 'HTTP_CLIENT, REMOTE_ADDR',
+    GETREMOTEADDR_SKIP_HTTP_X_FORWARDED_FOR|GETREMOTEADDR_SKIP_HTTP_CLIENT_IP => 'REMOTE_ADDR');
+$temp->add(new admin_setting_configselect('getremoteaddrconf', new lang_string('getremoteaddrconf', 'admin'),
+    new lang_string('configgetremoteaddrconf', 'admin'),
+    GETREMOTEADDR_SKIP_DEFAULT, $options));
+$temp->add(new admin_setting_configtext('reverseproxyignore', new lang_string('reverseproxyignore', 'admin'), new lang_string('configreverseproxyignore', 'admin'), ''));
 
-    $temp->add(new admin_setting_configselect('gradehistorylifetime', new lang_string('gradehistorylifetime', 'grades'),
-        new lang_string('gradehistorylifetime_help', 'grades'), 0,
-        [
-            0 => new lang_string('neverdeletehistory', 'grades'),
-            1000 => new lang_string('numdays', '', 1000),
-            365 => new lang_string('numdays', '', 365),
-            180 => new lang_string('numdays', '', 180),
-            150 => new lang_string('numdays', '', 150),
-            120 => new lang_string('numdays', '', 120),
-            90 => new lang_string('numdays', '', 90),
-            60 => new lang_string('numdays', '', 60),
-            30 => new lang_string('numdays', '', 30),
-        ]
-    ));
+$temp->add(new admin_setting_heading('webproxy', new lang_string('webproxy', 'admin'), new lang_string('webproxyinfo', 'admin')));
+$temp->add(new admin_setting_configtext('proxyhost', new lang_string('proxyhost', 'admin'), new lang_string('configproxyhost', 'admin'), '', PARAM_HOST));
+$temp->add(new admin_setting_configtext('proxyport', new lang_string('proxyport', 'admin'), new lang_string('configproxyport', 'admin'), 0, PARAM_INT));
+$options = array('HTTP'=>'HTTP');
+if (defined('CURLPROXY_SOCKS5')) {
+    $options['SOCKS5'] = 'SOCKS5';
+}
+$temp->add(new admin_setting_configselect('proxytype', new lang_string('proxytype', 'admin'), new lang_string('configproxytype','admin'), 'HTTP', $options));
+$temp->add(new admin_setting_configtext('proxyuser', new lang_string('proxyuser', 'admin'), new lang_string('configproxyuser', 'admin'), ''));
+$temp->add(new admin_setting_configpasswordunmask('proxypassword', new lang_string('proxypassword', 'admin'), new lang_string('configproxypassword', 'admin'), ''));
+$temp->add(new admin_setting_configtext('proxybypass', new lang_string('proxybypass', 'admin'), new lang_string('configproxybypass', 'admin'), 'localhost, 127.0.0.1'));
+$ADMIN->add('server', $temp);
 
-    $temp->add(new admin_setting_configselect('tempdatafoldercleanup', new lang_string('tempdatafoldercleanup', 'admin'),
-        new lang_string('configtempdatafoldercleanup', 'admin'), 168,
-        [
+$temp = new admin_settingpage('maintenancemode', new lang_string('sitemaintenancemode', 'admin'));
+$options = array(0=>new lang_string('disable'), 1=>new lang_string('enable'));
+$temp->add(new admin_setting_configselect('maintenance_enabled', new lang_string('sitemaintenancemode', 'admin'),
+                                          new lang_string('helpsitemaintenance', 'admin'), 0, $options));
+$temp->add(new admin_setting_confightmleditor('maintenance_message', new lang_string('optionalmaintenancemessage', 'admin'),
+                                              '', ''));
+$ADMIN->add('server', $temp);
+
+$temp = new admin_settingpage('cleanup', new lang_string('cleanup', 'admin'));
+$temp->add(new admin_setting_configselect('deleteunconfirmed', new lang_string('deleteunconfirmed', 'admin'), new lang_string('configdeleteunconfirmed', 'admin'), 168, array(0 => new lang_string('never'),
+                                                                                                                                                                    168 => new lang_string('numdays', '', 7),
+                                                                                                                                                                    144 => new lang_string('numdays', '', 6),
+                                                                                                                                                                    120 => new lang_string('numdays', '', 5),
+                                                                                                                                                                    96 => new lang_string('numdays', '', 4),
+                                                                                                                                                                    72 => new lang_string('numdays', '', 3),
+                                                                                                                                                                    48 => new lang_string('numdays', '', 2),
+                                                                                                                                                                    24 => new lang_string('numdays', '', 1),
+                                                                                                                                                                    12 => new lang_string('numhours', '', 12),
+                                                                                                                                                                    6 => new lang_string('numhours', '', 6),
+                                                                                                                                                                    1 => new lang_string('numhours', '', 1))));
+
+$temp->add(new admin_setting_configselect('deleteincompleteusers', new lang_string('deleteincompleteusers', 'admin'), new lang_string('configdeleteincompleteusers', 'admin'), 0, array(0 => new lang_string('never'),
+                                                                                                                                                                    168 => new lang_string('numdays', '', 7),
+                                                                                                                                                                    144 => new lang_string('numdays', '', 6),
+                                                                                                                                                                    120 => new lang_string('numdays', '', 5),
+                                                                                                                                                                    96 => new lang_string('numdays', '', 4),
+                                                                                                                                                                    72 => new lang_string('numdays', '', 3),
+                                                                                                                                                                    48 => new lang_string('numdays', '', 2),
+                                                                                                                                                                    24 => new lang_string('numdays', '', 1))));
+
+
+$temp->add(new admin_setting_configcheckbox('disablegradehistory', new lang_string('disablegradehistory', 'grades'),
+                                            new lang_string('disablegradehistory_help', 'grades'), 0));
+
+$temp->add(new admin_setting_configselect('gradehistorylifetime', new lang_string('gradehistorylifetime', 'grades'),
+                                          new lang_string('gradehistorylifetime_help', 'grades'), 0, array(0 => new lang_string('neverdeletehistory', 'grades'),
+                                                                                                   1000 => new lang_string('numdays', '', 1000),
+                                                                                                    365 => new lang_string('numdays', '', 365),
+                                                                                                    180 => new lang_string('numdays', '', 180),
+                                                                                                    150 => new lang_string('numdays', '', 150),
+                                                                                                    120 => new lang_string('numdays', '', 120),
+                                                                                                     90 => new lang_string('numdays', '', 90),
+                                                                                                     60 => new lang_string('numdays', '', 60),
+                                                                                                     30 => new lang_string('numdays', '', 30))));
+
+$temp->add(new admin_setting_configselect('tempdatafoldercleanup', new lang_string('tempdatafoldercleanup', 'admin'),
+        new lang_string('configtempdatafoldercleanup', 'admin'), 168, array(
             1 => new lang_string('numhours', '', 1),
             3 => new lang_string('numhours', '', 3),
             6 => new lang_string('numhours', '', 6),
@@ -229,303 +166,224 @@ if ($hassiteconfig) {
             24 => new lang_string('numhours', '', 24),
             48 => new lang_string('numdays', '', 2),
             168 => new lang_string('numdays', '', 7),
-        ]
-    ));
+)));
 
-    $ADMIN->add('server', $temp);
+$ADMIN->add('server', $temp);
 
     $temp->add(new admin_setting_configduration('filescleanupperiod',
         new lang_string('filescleanupperiod', 'admin'),
         new lang_string('filescleanupperiod_help', 'admin'),
         86400));
 
-    // Environment.
-    $ADMIN->add('server', new admin_externalpage('environment', new lang_string('environment', 'admin'),
-        "{$CFG->wwwroot}/{$CFG->admin}/environment.php"));
+$ADMIN->add('server', new admin_externalpage('environment', new lang_string('environment','admin'), "$CFG->wwwroot/$CFG->admin/environment.php"));
+$ADMIN->add('server', new admin_externalpage('phpinfo', new lang_string('phpinfo'), "$CFG->wwwroot/$CFG->admin/phpinfo.php"));
+$ADMIN->add('server', new admin_externalpage('testoutgoingmailconf', new lang_string('testoutgoingmailconf', 'admin'),
+            new moodle_url("$CFG->wwwroot/$CFG->admin/testoutgoingmailconf.php"), 'moodle/site:config', true));
 
-    // PHP info.
-    $ADMIN->add('server', new admin_externalpage('phpinfo', new lang_string('phpinfo'),
-        "{$CFG->wwwroot}/{$CFG->admin}/phpinfo.php"));
 
-    // Test outgoing mail configuration (hidden, accessed via direct link from the settings page).
-    $ADMIN->add('server', new admin_externalpage('testoutgoingmailconf', new lang_string('testoutgoingmailconf', 'admin'),
-        new moodle_url('/admin/testoutgoingmailconf.php'), 'moodle/site:config', true));
+// "performance" settingpage
+$temp = new admin_settingpage('performance', new lang_string('performance', 'admin'));
 
-    // Performance.
-    $temp = new admin_settingpage('performance', new lang_string('performance', 'admin'));
+// Memory limit options for large administration tasks.
+$memoryoptions = array(
+    '64M' => '64M',
+    '128M' => '128M',
+    '256M' => '256M',
+    '512M' => '512M',
+    '1024M' => '1024M',
+    '2048M' => '2048M');
 
-    // Memory limit options for large administration tasks.
-    $memoryoptions = [
-        '64M' => '64M',
-        '128M' => '128M',
-        '256M' => '256M',
-        '512M' => '512M',
-        '1024M' => '1024M',
-        '2048M' => '2048M',
-    ];
+// Allow larger memory usage for 64-bit sites only.
+if (PHP_INT_SIZE === 8) {
+    $memoryoptions['3072M'] = '3072M';
+    $memoryoptions['4096M'] = '4096M';
+}
 
-    // Allow larger memory usage for 64-bit sites only.
-    if (PHP_INT_SIZE === 8) {
-        $memoryoptions['3072M'] = '3072M';
-        $memoryoptions['4096M'] = '4096M';
-    }
-
-    $temp->add(new admin_setting_configselect('extramemorylimit', new lang_string('extramemorylimit', 'admin'),
-        new lang_string('configextramemorylimit', 'admin'), '512M', $memoryoptions));
-
-    $temp->add(new admin_setting_configtext('maxtimelimit', new lang_string('maxtimelimit', 'admin'),
+$temp->add(new admin_setting_configselect('extramemorylimit', new lang_string('extramemorylimit', 'admin'),
+                                          new lang_string('configextramemorylimit', 'admin'), '512M',
+                                          $memoryoptions));
+$temp->add(new admin_setting_configtext('maxtimelimit', new lang_string('maxtimelimit', 'admin'),
         new lang_string('maxtimelimit_desc', 'admin'), 0, PARAM_INT));
 
-    $temp->add(new admin_setting_configtext('curlcache', new lang_string('curlcache', 'admin'),
-        new lang_string('configcurlcache', 'admin'), 120, PARAM_INT));
+$temp->add(new admin_setting_configtext('curlcache', new lang_string('curlcache', 'admin'),
+                                        new lang_string('configcurlcache', 'admin'), 120, PARAM_INT));
 
-    $temp->add(new admin_setting_configtext('curltimeoutkbitrate', new lang_string('curltimeoutkbitrate', 'admin'),
-        new lang_string('curltimeoutkbitrate_help', 'admin'), 56, PARAM_INT));
+$temp->add(new admin_setting_configtext('curltimeoutkbitrate', new lang_string('curltimeoutkbitrate', 'admin'),
+                                        new lang_string('curltimeoutkbitrate_help', 'admin'), 56, PARAM_INT));
 
-    $ADMIN->add('server', $temp);
+$ADMIN->add('server', $temp);
 
-    // Tasks.
-    $ADMIN->add('server', new admin_category('taskconfig', new lang_string('taskadmintitle', 'admin')));
 
-    // Task processing.
-    $temp = new admin_settingpage('taskprocessing', new lang_string('taskprocessing', 'admin'));
+$ADMIN->add('server', new admin_category('taskconfig', new lang_string('taskadmintitle', 'admin')));
+$temp = new admin_settingpage('taskprocessing', new lang_string('taskprocessing','admin'));
+$temp->add(
+    new admin_setting_configtext(
+        'task_scheduled_concurrency_limit',
+        new lang_string('task_scheduled_concurrency_limit', 'admin'),
+        new lang_string('task_scheduled_concurrency_limit_desc', 'admin'),
+        3,
+        PARAM_INT
+    )
+);
 
-    $setting = new admin_setting_configcheckbox(
-        'cron_enabled',
-        new lang_string('cron_enabled', 'admin'),
-        new lang_string('cron_enabled_desc', 'admin'),
+$temp->add(
+    new admin_setting_configduration(
+        'task_scheduled_max_runtime',
+        new lang_string('task_scheduled_max_runtime', 'admin'),
+        new lang_string('task_scheduled_max_runtime_desc', 'admin'),
+        30 * MINSECS
+    )
+);
+
+$temp->add(
+    new admin_setting_configtext(
+        'task_adhoc_concurrency_limit',
+        new lang_string('task_adhoc_concurrency_limit', 'admin'),
+        new lang_string('task_adhoc_concurrency_limit_desc', 'admin'),
+        3,
+        PARAM_INT
+    )
+);
+
+$temp->add(
+    new admin_setting_configduration(
+        'task_adhoc_max_runtime',
+        new lang_string('task_adhoc_max_runtime', 'admin'),
+        new lang_string('task_adhoc_max_runtime_desc', 'admin'),
+        30 * MINSECS
+    )
+);
+$ADMIN->add('taskconfig', $temp);
+
+$temp = new admin_settingpage('tasklogging', new lang_string('tasklogging','admin'));
+$temp->add(
+    new admin_setting_configselect(
+        'task_logmode',
+        new lang_string('task_logmode', 'admin'),
+        new lang_string('task_logmode_desc', 'admin'),
+        \core\task\logmanager::MODE_ALL,
+        [
+            \core\task\logmanager::MODE_ALL => new lang_string('task_logmode_all', 'admin'),
+            \core\task\logmanager::MODE_FAILONLY => new lang_string('task_logmode_failonly', 'admin'),
+            \core\task\logmanager::MODE_NONE => new lang_string('task_logmode_none', 'admin'),
+        ]
+    )
+);
+$temp->add(
+    new admin_setting_configcheckbox(
+        'task_logtostdout',
+        new lang_string('task_logtostdout', 'admin'),
+        new lang_string('task_logtostdout_desc', 'admin'),
         1
-    );
-    $setting->set_updatedcallback('theme_reset_static_caches');
-    $temp->add($setting);
-
-    $temp->add(
-        new admin_setting_configtext(
-            'task_scheduled_concurrency_limit',
-            new lang_string('task_scheduled_concurrency_limit', 'admin'),
-            new lang_string('task_scheduled_concurrency_limit_desc', 'admin'),
-            3,
-            PARAM_INT
-        )
-    );
+    )
+);
 
+if (\core\task\logmanager::uses_standard_settings()) {
     $temp->add(
         new admin_setting_configduration(
-            'task_scheduled_max_runtime',
-            new lang_string('task_scheduled_max_runtime', 'admin'),
-            new lang_string('task_scheduled_max_runtime_desc', 'admin'),
-            30 * MINSECS
+            'task_logretention',
+            new \lang_string('task_logretention', 'admin'),
+            new \lang_string('task_logretention_desc', 'admin'),
+            28 * DAYSECS
         )
     );
 
     $temp->add(
         new admin_setting_configtext(
-            'task_adhoc_concurrency_limit',
-            new lang_string('task_adhoc_concurrency_limit', 'admin'),
-            new lang_string('task_adhoc_concurrency_limit_desc', 'admin'),
-            3,
+            'task_logretainruns',
+            new \lang_string('task_logretainruns', 'admin'),
+            new \lang_string('task_logretainruns_desc', 'admin'),
+            20,
             PARAM_INT
         )
     );
+}
+$ADMIN->add('taskconfig', $temp);
 
-    $temp->add(
-        new admin_setting_configduration(
-            'task_adhoc_max_runtime',
-            new lang_string('task_adhoc_max_runtime', 'admin'),
-            new lang_string('task_adhoc_max_runtime_desc', 'admin'),
-            30 * MINSECS
-        )
-    );
-    $ADMIN->add('taskconfig', $temp);
+if (\core\task\logmanager::uses_standard_settings()) {
+    $ADMIN->add('taskconfig', new admin_externalpage(
+        'tasklogs',
+        new lang_string('tasklogs','admin'),
+        "{$CFG->wwwroot}/{$CFG->admin}/tasklogs.php"
+    ));
+}
 
-    // Task log configuration.
-    $temp = new admin_settingpage('tasklogging', new lang_string('tasklogging', 'admin'));
-    $temp->add(
-        new admin_setting_configselect(
-            'task_logmode',
-            new lang_string('task_logmode', 'admin'),
-            new lang_string('task_logmode_desc', 'admin'),
-            \core\task\logmanager::MODE_ALL,
-            [
-                \core\task\logmanager::MODE_ALL => new lang_string('task_logmode_all', 'admin'),
-                \core\task\logmanager::MODE_FAILONLY => new lang_string('task_logmode_failonly', 'admin'),
-                \core\task\logmanager::MODE_NONE => new lang_string('task_logmode_none', 'admin'),
-            ]
-        )
-    );
-    $temp->add(
-        new admin_setting_configcheckbox(
-            'task_logtostdout',
-            new lang_string('task_logtostdout', 'admin'),
-            new lang_string('task_logtostdout_desc', 'admin'),
-            1
-        )
-    );
+// E-mail settings.
+$ADMIN->add('server', new admin_category('email', new lang_string('categoryemail', 'admin')));
 
-    if (\core\task\logmanager::uses_standard_settings()) {
-        $temp->add(
-            new admin_setting_configduration(
-                'task_logretention',
-                new \lang_string('task_logretention', 'admin'),
-                new \lang_string('task_logretention_desc', 'admin'),
-                28 * DAYSECS
-            )
-        );
+$temp = new admin_settingpage('outgoingmailconfig', new lang_string('outgoingmailconfig', 'admin'));
 
-        $temp->add(
-            new admin_setting_configtext(
-                'task_logretainruns',
-                new \lang_string('task_logretainruns', 'admin'),
-                new \lang_string('task_logretainruns_desc', 'admin'),
-                20,
-                PARAM_INT
-            )
-        );
-    }
-    $ADMIN->add('taskconfig', $temp);
-
-    // Task logs.
-    if (\core\task\logmanager::uses_standard_settings()) {
-        $ADMIN->add('taskconfig', new admin_externalpage(
-            'tasklogs',
-            new lang_string('tasklogs', 'admin'),
-            "{$CFG->wwwroot}/{$CFG->admin}/tasklogs.php"
-        ));
-    }
-
-    // Email.
-    $ADMIN->add('server', new admin_category('email', new lang_string('categoryemail', 'admin')));
-
-    // Outgoing mail configuration.
-    $temp = new admin_settingpage('outgoingmailconfig', new lang_string('outgoingmailconfig', 'admin'));
-
-    $temp->add(new admin_setting_heading('smtpheading', new lang_string('smtp', 'admin'),
-        new lang_string('smtpdetail', 'admin')));
-
-    $temp->add(new admin_setting_configtext('smtphosts', new lang_string('smtphosts', 'admin'),
-        new lang_string('configsmtphosts', 'admin'), '', PARAM_RAW));
-
-    $options = [
-        '' => new lang_string('none', 'admin'),
-        'ssl' => 'SSL',
-        'tls' => 'TLS',
-    ];
-
-    $temp->add(new admin_setting_configselect('smtpsecure', new lang_string('smtpsecure', 'admin'),
-        new lang_string('configsmtpsecure', 'admin'), '', $options));
-
-    $authtypeoptions = [
-        'LOGIN' => 'LOGIN',
-        'PLAIN' => 'PLAIN',
-        'NTLM' => 'NTLM',
-        'CRAM-MD5' => 'CRAM-MD5',
-    ];
-
-    $temp->add(new admin_setting_configselect('smtpauthtype', new lang_string('smtpauthtype', 'admin'),
-        new lang_string('configsmtpauthtype', 'admin'), 'LOGIN', $authtypeoptions));
-
-    $temp->add(new admin_setting_configtext('smtpuser', new lang_string('smtpuser', 'admin'),
-        new lang_string('configsmtpuser', 'admin'), '', PARAM_NOTAGS));
-
-    $temp->add(new admin_setting_configpasswordunmask('smtppass', new lang_string('smtppass', 'admin'),
-        new lang_string('configsmtpuser', 'admin'), ''));
-
-    $temp->add(new admin_setting_configtext('smtpmaxbulk', new lang_string('smtpmaxbulk', 'admin'),
-        new lang_string('configsmtpmaxbulk', 'admin'), 1, PARAM_INT));
-
-    $temp->add(new admin_setting_heading('noreplydomainheading', new lang_string('noreplydomain', 'admin'),
+$temp->add(new admin_setting_heading('smtpheading', new lang_string('smtp', 'admin'),
+            new lang_string('smtpdetail', 'admin')));
+$temp->add(new admin_setting_configtext('smtphosts', new lang_string('smtphosts', 'admin'),
+            new lang_string('configsmtphosts', 'admin'), '', PARAM_RAW));
+$options = array('' => new lang_string('none', 'admin'), 'ssl' => 'SSL', 'tls' => 'TLS');
+$temp->add(new admin_setting_configselect('smtpsecure', new lang_string('smtpsecure', 'admin'),
+            new lang_string('configsmtpsecure', 'admin'), '', $options));
+$authtypeoptions = array('LOGIN' => 'LOGIN', 'PLAIN' => 'PLAIN', 'NTLM' => 'NTLM', 'CRAM-MD5' => 'CRAM-MD5');
+$temp->add(new admin_setting_configselect('smtpauthtype', new lang_string('smtpauthtype', 'admin'),
+            new lang_string('configsmtpauthtype', 'admin'), 'LOGIN', $authtypeoptions));
+$temp->add(new admin_setting_configtext('smtpuser', new lang_string('smtpuser', 'admin'),
+            new lang_string('configsmtpuser', 'admin'), '', PARAM_NOTAGS));
+$temp->add(new admin_setting_configpasswordunmask('smtppass', new lang_string('smtppass', 'admin'),
+            new lang_string('configsmtpuser', 'admin'), ''));
+$temp->add(new admin_setting_configtext('smtpmaxbulk', new lang_string('smtpmaxbulk', 'admin'),
+           new lang_string('configsmtpmaxbulk', 'admin'), 1, PARAM_INT));
+$temp->add(new admin_setting_heading('noreplydomainheading', new lang_string('noreplydomain', 'admin'),
         new lang_string('noreplydomaindetail', 'admin')));
-
-    $temp->add(new admin_setting_configtext('noreplyaddress', new lang_string('noreplyaddress', 'admin'),
-        new lang_string('confignoreplyaddress', 'admin'), 'noreply@' . get_host_from_url($CFG->wwwroot), PARAM_EMAIL));
-
-    $temp->add(new admin_setting_configtextarea('allowedemaildomains',
+$temp->add(new admin_setting_configtext('noreplyaddress', new lang_string('noreplyaddress', 'admin'),
+          new lang_string('confignoreplyaddress', 'admin'), 'noreply@' . get_host_from_url($CFG->wwwroot), PARAM_EMAIL));
+$temp->add(new admin_setting_configtextarea('allowedemaildomains',
         new lang_string('allowedemaildomains', 'admin'),
         new lang_string('configallowedemaildomains', 'admin'),
         ''));
-
-    $temp->add(new admin_setting_heading('divertallemailsheading', new lang_string('divertallemails', 'admin'),
-        new lang_string('divertallemailsdetail', 'admin')));
-    $temp->add(new admin_setting_configtext('divertallemailsto',
-        new lang_string('divertallemailsto', 'admin'),
-        new lang_string('divertallemailsto_desc', 'admin'),
-        ''));
-    $temp->add(new admin_setting_configtextarea('divertallemailsexcept',
-        new lang_string('divertallemailsexcept', 'admin'),
-        new lang_string('divertallemailsexcept_desc', 'admin'),
-        '', PARAM_RAW, '50', '4'));
-
-    $noreplyaddress = isset($CFG->noreplyaddress) ? $CFG->noreplyaddress : 'noreply@example.com';
-    $dkimdomain = substr(strrchr($noreplyaddress, "@"), 1);
-    $dkimselector = empty($CFG->emaildkimselector) ? '[selector]' : $CFG->emaildkimselector;
-    $pempath = "\$CFG->dataroot/dkim/{$dkimdomain}/{$dkimselector}.private";
-    $temp->add(new admin_setting_heading('emaildkim', new lang_string('emaildkim', 'admin'),
-        new lang_string('emaildkiminfo', 'admin', ['path' => $pempath, 'docs' => \get_docs_url('Mail_configuration#DKIM')])));
-    $temp->add(new admin_setting_configtext('emaildkimselector', new lang_string('emaildkimselector', 'admin'),
-        new lang_string('configemaildkimselector', 'admin'), '', PARAM_FILE));
-
-    $url = new moodle_url('/admin/testoutgoingmailconf.php');
-    $link = html_writer::link($url, get_string('testoutgoingmailconf', 'admin'));
-    $temp->add(new admin_setting_heading('testoutgoinmailc', new lang_string('testoutgoingmailconf', 'admin'),
+$url = new moodle_url('/admin/testoutgoingmailconf.php');
+$link = html_writer::link($url, get_string('testoutgoingmailconf', 'admin'));
+$temp->add(new admin_setting_heading('testoutgoinmailc', new lang_string('testoutgoingmailconf', 'admin'),
         new lang_string('testoutgoingmaildetail', 'admin', $link)));
-
-    $temp->add(new admin_setting_heading('emaildoesnotfit', new lang_string('doesnotfit', 'admin'),
+$temp->add(new admin_setting_heading('emaildoesnotfit', new lang_string('doesnotfit', 'admin'),
         new lang_string('doesnotfitdetail', 'admin')));
+$charsets = get_list_of_charsets();
+unset($charsets['UTF-8']); // Not needed here.
+$options = array();
+$options['0'] = 'UTF-8';
+$options = array_merge($options, $charsets);
+$temp->add(new admin_setting_configselect('sitemailcharset', new lang_string('sitemailcharset', 'admin'),
+          new lang_string('configsitemailcharset','admin'), '0', $options));
+$temp->add(new admin_setting_configcheckbox('allowusermailcharset', new lang_string('allowusermailcharset', 'admin'),
+          new lang_string('configallowusermailcharset', 'admin'), 0));
+$temp->add(new admin_setting_configcheckbox('allowattachments', new lang_string('allowattachments', 'admin'),
+          new lang_string('configallowattachments', 'admin'), 1));
+$options = array('LF' => 'LF', 'CRLF' => 'CRLF');
+$temp->add(new admin_setting_configselect('mailnewline', new lang_string('mailnewline', 'admin'),
+          new lang_string('configmailnewline', 'admin'), 'LF', $options));
 
-    $charsets = get_list_of_charsets();
-    unset($charsets['UTF-8']);
-    $options = [
-        '0' => 'UTF-8',
-    ];
-    $options = array_merge($options, $charsets);
-    $temp->add(new admin_setting_configselect('sitemailcharset', new lang_string('sitemailcharset', 'admin'),
-        new lang_string('configsitemailcharset', 'admin'), '0', $options));
-
-    $temp->add(new admin_setting_configcheckbox('allowusermailcharset', new lang_string('allowusermailcharset', 'admin'),
-        new lang_string('configallowusermailcharset', 'admin'), 0));
-
-    $temp->add(new admin_setting_configcheckbox('allowattachments', new lang_string('allowattachments', 'admin'),
-        new lang_string('configallowattachments', 'admin'), 1));
-
-    $options = [
-        'LF' => 'LF',
-        'CRLF' => 'CRLF',
-    ];
-    $temp->add(new admin_setting_configselect('mailnewline', new lang_string('mailnewline', 'admin'),
-        new lang_string('configmailnewline', 'admin'), 'LF', $options));
-
-    $choices = [
-        new lang_string('never', 'admin'),
-        new lang_string('always', 'admin'),
-        new lang_string('onlynoreply', 'admin'),
-    ];
-    $temp->add(new admin_setting_configselect('emailfromvia', new lang_string('emailfromvia', 'admin'),
-        new lang_string('configemailfromvia', 'admin'), 1, $choices));
-
+$choices = array(new lang_string('never', 'admin'),
+                 new lang_string('always', 'admin'),
+                 new lang_string('onlynoreply', 'admin'));
+$temp->add(new admin_setting_configselect('emailfromvia', new lang_string('emailfromvia', 'admin'),
+          new lang_string('configemailfromvia', 'admin'), 1, $choices));
     $temp->add(new admin_setting_configtext('emailsubjectprefix', new lang_string('emailsubjectprefix', 'admin'),
         new lang_string('configemailsubjectprefix', 'admin'), '', PARAM_RAW));
 
-    $temp->add(new admin_setting_configtextarea('emailheaders', new lang_string('emailheaders', 'admin'),
-        new lang_string('configemailheaders', 'admin'), '', PARAM_RAW, '50', '3'));
+$ADMIN->add('email', $temp);
 
-    $ADMIN->add('email', $temp);
-
-    // Update notifications.
-    if (empty($CFG->disableupdatenotifications)) {
-        $temp = new admin_settingpage('updatenotifications', new lang_string('updatenotifications', 'core_admin'));
-        $temp->add(new admin_setting_configcheckbox('updateautocheck', new lang_string('updateautocheck', 'core_admin'),
-            new lang_string('updateautocheck_desc', 'core_admin'), 1));
-        $temp->add(new admin_setting_configselect('updateminmaturity', new lang_string('updateminmaturity', 'core_admin'),
-            new lang_string('updateminmaturity_desc', 'core_admin'), MATURITY_STABLE,
-            [
-                MATURITY_ALPHA => new lang_string('maturity'.MATURITY_ALPHA, 'core_admin'),
-                MATURITY_BETA => new lang_string('maturity'.MATURITY_BETA, 'core_admin'),
-                MATURITY_RC => new lang_string('maturity'.MATURITY_RC, 'core_admin'),
-                MATURITY_STABLE => new lang_string('maturity'.MATURITY_STABLE, 'core_admin'),
-            ]
-        ));
-        $temp->add(new admin_setting_configcheckbox('updatenotifybuilds', new lang_string('updatenotifybuilds', 'core_admin'),
-            new lang_string('updatenotifybuilds_desc', 'core_admin'), 0));
-        $ADMIN->add('server', $temp);
-    }
+// "update notifications" settingpage
+if (empty($CFG->disableupdatenotifications)) {
+    $temp = new admin_settingpage('updatenotifications', new lang_string('updatenotifications', 'core_admin'));
+    $temp->add(new admin_setting_configcheckbox('updateautocheck', new lang_string('updateautocheck', 'core_admin'),
+                                                new lang_string('updateautocheck_desc', 'core_admin'), 1));
+    $temp->add(new admin_setting_configselect('updateminmaturity', new lang_string('updateminmaturity', 'core_admin'),
+                                              new lang_string('updateminmaturity_desc', 'core_admin'), MATURITY_STABLE,
+                                              array(
+                                                  MATURITY_ALPHA  => new lang_string('maturity'.MATURITY_ALPHA, 'core_admin'),
+                                                  MATURITY_BETA   => new lang_string('maturity'.MATURITY_BETA, 'core_admin'),
+                                                  MATURITY_RC     => new lang_string('maturity'.MATURITY_RC, 'core_admin'),
+                                                  MATURITY_STABLE => new lang_string('maturity'.MATURITY_STABLE, 'core_admin'),
+                                              )));
+    $temp->add(new admin_setting_configcheckbox('updatenotifybuilds', new lang_string('updatenotifybuilds', 'core_admin'),
+                                                new lang_string('updatenotifybuilds_desc', 'core_admin'), 0));
+    $ADMIN->add('server', $temp);
 }
+
+} // end of speedup

admin/settings/top.php

@@ -39,7 +39,6 @@ $ADMIN->add('root', new admin_category('license', new lang_string('license')));
 $ADMIN->add('root', new admin_category('location', new lang_string('location','admin')));
 $ADMIN->add('root', new admin_category('language', new lang_string('language')));
 $ADMIN->add('root', new admin_category('messaging', new lang_string('messagingcategory', 'admin')));
-$ADMIN->add('root', new admin_category('payment', new lang_string('payments', 'payment')));
 $ADMIN->add('root', new admin_category('modules', new lang_string('plugins', 'admin')));
 $ADMIN->add('root', new admin_category('security', new lang_string('security','admin')));
 $ADMIN->add('root', new admin_category('appearance', new lang_string('appearance','admin')));

admin/templates/local/settings/autocomplete.mustache

@@ -1,87 +0,0 @@
-{{!
-    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/>.
-}}
-{{!
-    @template core_admin\local\settings\autocomplete
-
-    Admin setting auto complete.
-
-    Context variables required for this template:
-    * name - element name
-    * id - element id
-    * options - choices
-    * multiple - is multiple choices?
-    * manageurl - url to manage choices
-    * managetext - string to display manage url
-    * tags - Should we allow typing new entries to the field?
-    * ajax - Name of an AMD module to send/process ajax requests
-    * placeholder - Placeholder text for an empty list
-    * casesensitive - Whether the search has to be case-sensitive
-    * showsuggestions - Show suggestions by default - but this can be turned off
-    * noselectionstring - String that is shown when there are no selections
-
-
-    Example context (json):
-    {
-        "name": "name0",
-        "id": "id0",
-        "options": [{
-            "value": "1",
-            "text": "option 1",
-            "selected": true,
-            "disabled": false
-        }],
-        "multiple": true,
-        "manageurl": "",
-        "managetext": "",
-        "tags": false,
-        "ajax": "",
-        "placeholder": "",
-        "casesensitive": false,
-        "showsuggestions": true,
-        "noselectionstring": ""
-    }
-}}
-{{!
-    Setting autocomplete.
-}}
-
-<div class="form-select defaultsnext">
-    <input type="hidden" name="{{name}}[xxxxx]" value="_qf__force_multiselect_submission">
-    <select class="custom-select" name="{{name}}[]"
-            id="{{id}}"
-            {{#multiple}}multiple{{/multiple}}>
-        {{#options}}
-            <option value="{{value}}" {{#selected}}selected{{/selected}}>{{{text}}}</option>
-        {{/options}}
-    </select>
-    {{#manageurl}}
-        <a href="{{manageurl}}">{{managetext}}</a>
-    {{/manageurl}}
-</div>
-<br/>
-
-{{#js}}
-    require(['core/form-autocomplete'], function(module) {
-        module.enhance({{#quote}}#{{id}}{{/quote}},
-        {{tags}},
-        {{#quote}}{{ajax}}{{/quote}},
-        {{#quote}}{{placeholder}}{{/quote}},
-        {{casesensitive}},
-        {{showsuggestions}},
-        {{#quote}}{{noselectionstring}}{{/quote}});
-    });
-{{/js}}

admin/tests/behat/behat_admin.php

@@ -56,7 +56,7 @@ class behat_admin extends behat_base {
             $this->execute('behat_navigation::i_select_from_flat_navigation_drawer', [get_string('administrationsite')]);
 
             // Search by label.
-            $this->execute('behat_forms::i_set_the_field_to', [get_string('search'), $label]);
+            $this->execute('behat_forms::i_set_the_field_to', [get_string('query', 'admin'), $label]);
             $this->execute("behat_forms::press_button", get_string('search', 'admin'));
 
             // Admin settings does not use the same DOM structure than other moodle forms

admin/tool/analytics/amd/build/log_info.min.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../src/log_info.js"],"names":["define","$","str","ModalFactory","Notification","loadInfo","id","info","link","get_string","then","langString","bodyInfo","forEach","item","append","create","title","body","html","large","catch","exception"],"mappings":"AAsBAA,OAAM,2BAAC,CAAC,QAAD,CAAW,UAAX,CAAuB,oBAAvB,CAA6C,mBAA7C,CAAD,CAAoE,SAASC,CAAT,CAAYC,CAAZ,CAAiBC,CAAjB,CAA+BC,CAA/B,CAA6C,CAEnH,MAAoD,CAShDC,QAAQ,CAAE,kBAASC,CAAT,CAAaC,CAAb,CAAmB,CAEzB,GAAIC,CAAAA,CAAI,CAAGP,CAAC,CAAC,wBAAyBK,CAAzB,CAA8B,KAA/B,CAAZ,CACAJ,CAAG,CAACO,UAAJ,CAAe,SAAf,CAA0B,gBAA1B,EAA4CC,IAA5C,CAAiD,SAASC,CAAT,CAAqB,CAElE,GAAIC,CAAAA,CAAQ,CAAGX,CAAC,CAAC,MAAD,CAAhB,CACAM,CAAI,CAACM,OAAL,CAAa,SAASC,CAAT,CAAe,CACxBF,CAAQ,CAACG,MAAT,CAAgB,OAASD,CAAT,CAAgB,OAAhC,CACH,CAFD,EAGAF,CAAQ,CAACG,MAAT,CAAgB,OAAhB,EAEA,MAAOZ,CAAAA,CAAY,CAACa,MAAb,CAAoB,CACvBC,KAAK,CAAEN,CADgB,CAEvBO,IAAI,CAAEN,CAAQ,CAACO,IAAT,EAFiB,CAGvBC,KAAK,GAHkB,CAApB,CAIJZ,CAJI,CAMV,CAdD,EAcGa,KAdH,CAcSjB,CAAY,CAACkB,SAdtB,CAeH,CA3B+C,CA6BvD,CA/BK,CAAN","sourcesContent":["// This file is part of Moodle - http://moodle.org/\n//\n// Moodle is free software: you can redistribute it and/or modify\n// it under the terms of the GNU General Public License as published by\n// the Free Software Foundation, either version 3 of the License, or\n// (at your option) any later version.\n//\n// Moodle is distributed in the hope that it will be useful,\n// but WITHOUT ANY WARRANTY; without even the implied warranty of\n// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n// GNU General Public License for more details.\n//\n// You should have received a copy of the GNU General Public License\n// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.\n\n/**\n * Shows a dialogue with info about this logs.\n *\n * @module     tool_analytics/log_info\n * @copyright  2017 David Monllao {@link http://www.davidmonllao.com}\n * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later\n */\ndefine(['jquery', 'core/str', 'core/modal_factory', 'core/notification'], function($, str, ModalFactory, Notification) {\n\n    return /** @alias module:tool_analytics/log_info */ {\n\n        /**\n         * Prepares a modal info for a log's results.\n         *\n         * @method loadInfo\n         * @param {int} id\n         * @param {string[]} info\n         */\n        loadInfo: function(id, info) {\n\n            var link = $('[data-model-log-id=\"' + id + '\"]');\n            str.get_string('loginfo', 'tool_analytics').then(function(langString) {\n\n                var bodyInfo = $(\"<ul>\");\n                info.forEach(function(item) {\n                    bodyInfo.append('<li>' + item + '</li>');\n                });\n                bodyInfo.append(\"</ul>\");\n\n                return ModalFactory.create({\n                    title: langString,\n                    body: bodyInfo.html(),\n                    large: true,\n                }, link);\n\n            }).catch(Notification.exception);\n        }\n    };\n});\n"],"file":"log_info.min.js"}
+{"version":3,"sources":["../src/log_info.js"],"names":["define","$","str","ModalFactory","Notification","loadInfo","id","info","link","get_string","then","langString","bodyInfo","forEach","item","append","create","title","body","html","large","catch","exception"],"mappings":"AAwBAA,OAAM,2BAAC,CAAC,QAAD,CAAW,UAAX,CAAuB,oBAAvB,CAA6C,mBAA7C,CAAD,CAAoE,SAASC,CAAT,CAAYC,CAAZ,CAAiBC,CAAjB,CAA+BC,CAA/B,CAA6C,CAEnH,MAAoD,CAShDC,QAAQ,CAAE,kBAASC,CAAT,CAAaC,CAAb,CAAmB,CAEzB,GAAIC,CAAAA,CAAI,CAAGP,CAAC,CAAC,wBAAyBK,CAAzB,CAA8B,KAA/B,CAAZ,CACAJ,CAAG,CAACO,UAAJ,CAAe,SAAf,CAA0B,gBAA1B,EAA4CC,IAA5C,CAAiD,SAASC,CAAT,CAAqB,CAElE,GAAIC,CAAAA,CAAQ,CAAGX,CAAC,CAAC,MAAD,CAAhB,CACAM,CAAI,CAACM,OAAL,CAAa,SAASC,CAAT,CAAe,CACxBF,CAAQ,CAACG,MAAT,CAAgB,OAASD,CAAT,CAAgB,OAAhC,CACH,CAFD,EAGAF,CAAQ,CAACG,MAAT,CAAgB,OAAhB,EAEA,MAAOZ,CAAAA,CAAY,CAACa,MAAb,CAAoB,CACvBC,KAAK,CAAEN,CADgB,CAEvBO,IAAI,CAAEN,CAAQ,CAACO,IAAT,EAFiB,CAGvBC,KAAK,GAHkB,CAApB,CAIJZ,CAJI,CAMV,CAdD,EAcGa,KAdH,CAcSjB,CAAY,CAACkB,SAdtB,CAeH,CA3B+C,CA6BvD,CA/BK,CAAN","sourcesContent":["// This file is part of Moodle - http://moodle.org/\n//\n// Moodle is free software: you can redistribute it and/or modify\n// it under the terms of the GNU General Public License as published by\n// the Free Software Foundation, either version 3 of the License, or\n// (at your option) any later version.\n//\n// Moodle is distributed in the hope that it will be useful,\n// but WITHOUT ANY WARRANTY; without even the implied warranty of\n// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n// GNU General Public License for more details.\n//\n// You should have received a copy of the GNU General Public License\n// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.\n\n/**\n * Shows a dialogue with info about this logs.\n *\n * @module     tool_analytics/log_info\n * @class      log_info\n * @package    tool_analytics\n * @copyright  2017 David Monllao {@link http://www.davidmonllao.com}\n * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later\n */\ndefine(['jquery', 'core/str', 'core/modal_factory', 'core/notification'], function($, str, ModalFactory, Notification) {\n\n    return /** @alias module:tool_analytics/log_info */ {\n\n        /**\n         * Prepares a modal info for a log's results.\n         *\n         * @method loadInfo\n         * @param {int} id\n         * @param {string[]} info\n         */\n        loadInfo: function(id, info) {\n\n            var link = $('[data-model-log-id=\"' + id + '\"]');\n            str.get_string('loginfo', 'tool_analytics').then(function(langString) {\n\n                var bodyInfo = $(\"<ul>\");\n                info.forEach(function(item) {\n                    bodyInfo.append('<li>' + item + '</li>');\n                });\n                bodyInfo.append(\"</ul>\");\n\n                return ModalFactory.create({\n                    title: langString,\n                    body: bodyInfo.html(),\n                    large: true,\n                }, link);\n\n            }).catch(Notification.exception);\n        }\n    };\n});\n"],"file":"log_info.min.js"}

admin/tool/analytics/amd/build/potential-contexts.min.js.map

@@ -1 +1 @@
-{"version":3,"sources":["../src/potential-contexts.js"],"names":["define","$","Ajax","processResults","selector","results","contexts","isArray","each","index","context","push","value","id","label","name","transport","query","success","failure","promise","modelid","attr","call","methodname","args","then","fail"],"mappings":"AAuBAA,OAAM,qCAAC,CAAC,QAAD,CAAW,WAAX,CAAD,CAA0B,SAASC,CAAT,CAAYC,CAAZ,CAAkB,CAE9C,MAA8D,CAE1DC,cAAc,CAAE,wBAASC,CAAT,CAAmBC,CAAnB,CAA4B,CACxC,GAAIC,CAAAA,CAAQ,CAAG,EAAf,CACA,GAAIL,CAAC,CAACM,OAAF,CAAUF,CAAV,CAAJ,CAAwB,CACpBJ,CAAC,CAACO,IAAF,CAAOH,CAAP,CAAgB,SAASI,CAAT,CAAgBC,CAAhB,CAAyB,CACrCJ,CAAQ,CAACK,IAAT,CAAc,CACVC,KAAK,CAAEF,CAAO,CAACG,EADL,CAEVC,KAAK,CAAEJ,CAAO,CAACK,IAFL,CAAd,CAIH,CALD,EAMA,MAAOT,CAAAA,CAEV,CATD,IASO,CACH,MAAOD,CAAAA,CACV,CACJ,CAhByD,CAkB1DW,SAAS,CAAE,mBAASZ,CAAT,CAAmBa,CAAnB,CAA0BC,CAA1B,CAAmCC,CAAnC,CAA4C,IAC/CC,CAAAA,CAD+C,CAG/CC,CAAO,CAAGpB,CAAC,CAACG,CAAD,CAAD,CAAYkB,IAAZ,CAAiB,SAAjB,GAA+B,IAHM,CAInDF,CAAO,CAAGlB,CAAI,CAACqB,IAAL,CAAU,CAAC,CACjBC,UAAU,CAAE,mCADK,CAEjBC,IAAI,CAAE,CACFR,KAAK,CAAEA,CADL,CAEFI,OAAO,CAAEA,CAFP,CAFW,CAAD,CAAV,CAAV,CAQAD,CAAO,CAAC,CAAD,CAAP,CAAWM,IAAX,CAAgBR,CAAhB,EAAyBS,IAAzB,CAA8BR,CAA9B,CACH,CA/ByD,CAmCjE,CArCK,CAAN","sourcesContent":["// This file is part of Moodle - http://moodle.org/\n//\n// Moodle is free software: you can redistribute it and/or modify\n// it under the terms of the GNU General Public License as published by\n// the Free Software Foundation, either version 3 of the License, or\n// (at your option) any later version.\n//\n// Moodle is distributed in the hope that it will be useful,\n// but WITHOUT ANY WARRANTY; without even the implied warranty of\n// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n// GNU General Public License for more details.\n//\n// You should have received a copy of the GNU General Public License\n// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.\n\n/**\n * Potential contexts selector module.\n *\n * @module     tool_analytics/potential-contexts\n * @copyright  2019 David Monllao\n * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later\n */\n\ndefine(['jquery', 'core/ajax'], function($, Ajax) {\n\n    return /** @alias module:tool_analytics/potential-contexts */ {\n\n        processResults: function(selector, results) {\n            var contexts = [];\n            if ($.isArray(results)) {\n                $.each(results, function(index, context) {\n                    contexts.push({\n                        value: context.id,\n                        label: context.name\n                    });\n                });\n                return contexts;\n\n            } else {\n                return results;\n            }\n        },\n\n        transport: function(selector, query, success, failure) {\n            var promise;\n\n            let modelid = $(selector).attr('modelid') || null;\n            promise = Ajax.call([{\n                methodname: 'tool_analytics_potential_contexts',\n                args: {\n                    query: query,\n                    modelid: modelid\n                }\n            }]);\n\n            promise[0].then(success).fail(failure);\n        }\n\n    };\n\n});\n"],"file":"potential-contexts.min.js"}
+{"version":3,"sources":["../src/potential-contexts.js"],"names":["define","$","Ajax","processResults","selector","results","contexts","isArray","each","index","context","push","value","id","label","name","transport","query","success","failure","promise","modelid","attr","call","methodname","args","then","fail"],"mappings":"AAyBAA,OAAM,qCAAC,CAAC,QAAD,CAAW,WAAX,CAAD,CAA0B,SAASC,CAAT,CAAYC,CAAZ,CAAkB,CAE9C,MAA8D,CAE1DC,cAAc,CAAE,wBAASC,CAAT,CAAmBC,CAAnB,CAA4B,CACxC,GAAIC,CAAAA,CAAQ,CAAG,EAAf,CACA,GAAIL,CAAC,CAACM,OAAF,CAAUF,CAAV,CAAJ,CAAwB,CACpBJ,CAAC,CAACO,IAAF,CAAOH,CAAP,CAAgB,SAASI,CAAT,CAAgBC,CAAhB,CAAyB,CACrCJ,CAAQ,CAACK,IAAT,CAAc,CACVC,KAAK,CAAEF,CAAO,CAACG,EADL,CAEVC,KAAK,CAAEJ,CAAO,CAACK,IAFL,CAAd,CAIH,CALD,EAMA,MAAOT,CAAAA,CAEV,CATD,IASO,CACH,MAAOD,CAAAA,CACV,CACJ,CAhByD,CAkB1DW,SAAS,CAAE,mBAASZ,CAAT,CAAmBa,CAAnB,CAA0BC,CAA1B,CAAmCC,CAAnC,CAA4C,IAC/CC,CAAAA,CAD+C,CAG/CC,CAAO,CAAGpB,CAAC,CAACG,CAAD,CAAD,CAAYkB,IAAZ,CAAiB,SAAjB,GAA+B,IAHM,CAInDF,CAAO,CAAGlB,CAAI,CAACqB,IAAL,CAAU,CAAC,CACjBC,UAAU,CAAE,mCADK,CAEjBC,IAAI,CAAE,CACFR,KAAK,CAAEA,CADL,CAEFI,OAAO,CAAEA,CAFP,CAFW,CAAD,CAAV,CAAV,CAQAD,CAAO,CAAC,CAAD,CAAP,CAAWM,IAAX,CAAgBR,CAAhB,EAAyBS,IAAzB,CAA8BR,CAA9B,CACH,CA/ByD,CAmCjE,CArCK,CAAN","sourcesContent":["// This file is part of Moodle - http://moodle.org/\n//\n// Moodle is free software: you can redistribute it and/or modify\n// it under the terms of the GNU General Public License as published by\n// the Free Software Foundation, either version 3 of the License, or\n// (at your option) any later version.\n//\n// Moodle is distributed in the hope that it will be useful,\n// but WITHOUT ANY WARRANTY; without even the implied warranty of\n// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the\n// GNU General Public License for more details.\n//\n// You should have received a copy of the GNU General Public License\n// along with Moodle.  If not, see <http://www.gnu.org/licenses/>.\n\n/**\n * Potential contexts selector module.\n *\n * @module     tool_analytics/potential-contexts\n * @class      potential-contexts\n * @package    tool_analytics\n * @copyright  2019 David Monllao\n * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later\n */\n\ndefine(['jquery', 'core/ajax'], function($, Ajax) {\n\n    return /** @alias module:tool_analytics/potential-contexts */ {\n\n        processResults: function(selector, results) {\n            var contexts = [];\n            if ($.isArray(results)) {\n                $.each(results, function(index, context) {\n                    contexts.push({\n                        value: context.id,\n                        label: context.name\n                    });\n                });\n                return contexts;\n\n            } else {\n                return results;\n            }\n        },\n\n        transport: function(selector, query, success, failure) {\n            var promise;\n\n            let modelid = $(selector).attr('modelid') || null;\n            promise = Ajax.call([{\n                methodname: 'tool_analytics_potential_contexts',\n                args: {\n                    query: query,\n                    modelid: modelid\n                }\n            }]);\n\n            promise[0].then(success).fail(failure);\n        }\n\n    };\n\n});\n"],"file":"potential-contexts.min.js"}

admin/tool/analytics/amd/src/log_info.js

@@ -17,6 +17,8 @@
  * Shows a dialogue with info about this logs.
  *
  * @module     tool_analytics/log_info
+ * @class      log_info
+ * @package    tool_analytics
  * @copyright  2017 David Monllao {@link http://www.davidmonllao.com}
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */

admin/tool/analytics/amd/src/potential-contexts.js

@@ -17,6 +17,8 @@
  * Potential contexts selector module.
  *
  * @module     tool_analytics/potential-contexts
+ * @class      potential-contexts
+ * @package    tool_analytics
  * @copyright  2019 David Monllao
  * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
  */

admin/tool/analytics/classes/output/models_list.php

@@ -64,7 +64,7 @@ class models_list implements \renderable, \templatable {
         $data = new \stdClass();
 
         $newmodelmenu = new \action_menu();
-        $newmodelmenu->set_menu_trigger(get_string('newmodel', 'tool_analytics'), 'btn btn-secondary');
+        $newmodelmenu->set_menu_trigger(get_string('newmodel', 'tool_analytics'), 'btn btn-default');
         $newmodelmenu->set_alignment(\action_menu::TL, \action_menu::BL);
 
         $newmodelmenu->add(new \action_menu_link(

admin/tool/analytics/templates/evaluation_options.mustache

@@ -57,10 +57,10 @@
 {{/trainedexternally}}
 
 {{! Hidden by default if #trainedexternally as the default option is trainedmodel in this case.}}
-<div id="id-evaluation-timesplitting-container" class="mt-3 {{#trainedexternally}}hidden{{/trainedexternally}}">
+<div id="id-evaluation-timesplitting-container" class="m-t-1 {{#trainedexternally}}hidden{{/trainedexternally}}">
     {{#str}} selecttimesplittingforevaluation, tool_analytics {{/str}}
     <div>
-        <select id="id-evaluation-timesplitting" name="timesplitting" class="custom-select mt-3">
+        <select id="id-evaluation-timesplitting" name="timesplitting" class="custom-select m-t-1">
             {{#timesplittingmethods}}
                 <option value="{{id}}">{{text}}</option>
             {{/timesplittingmethods}}

admin/tool/analytics/templates/export_options.mustache

@@ -39,7 +39,7 @@
     <input class="custom-control-input" type="radio" name="exportoption" id="id-mode-exportmodel" value="exportmodel" checked>
     <label class="custom-control-label" for="id-mode-exportmodel">{{#str}} exportmodel, tool_analytics {{/str}}</label>
 </div>
-<div class="custom-control custom-checkbox ml-5" id="id-includeweights-container">
+<div class="custom-control custom-checkbox m-l-2" id="id-includeweights-container">
   <input class="custom-control-input" type="checkbox" id="id-includeweights" value="1" checked>
   <label class="custom-control-label" for="id-includeweights">{{#str}} exportincludeweights, tool_analytics {{/str}}</label>
 </div>

admin/tool/analytics/tests/external_test.php

@@ -66,6 +66,8 @@ class tool_analytics_external_testcase extends externallib_advanced_testcase {
 
     /**
      * test_potential_contexts description
+     *
+     * @expectedException required_capability_exception
      */
     public function test_potential_contexts_no_manager() {
         $this->resetAfterTest();
@@ -73,7 +75,6 @@ class tool_analytics_external_testcase extends externallib_advanced_testcase {
         $user = $this->getDataGenerator()->create_user();
         $this->setUser($user);
 
-        $this->expectException(required_capability_exception::class);
         $this->assertCount(2, \tool_analytics\external::potential_contexts());
     }
 }

admin/tool/analytics/version.php

@@ -24,6 +24,6 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version   = 2020110900; // The current plugin version (Date: YYYYMMDDXX).
-$plugin->requires  = 2020110300; // Requires this Moodle version.
+$plugin->version   = 2020061500; // The current plugin version (Date: YYYYMMDDXX).
+$plugin->requires  = 2020060900; // Requires this Moodle version.
 $plugin->component = 'tool_analytics'; // Full name of the plugin (used for diagnostics).

admin/tool/availabilityconditions/version.php

@@ -24,6 +24,6 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version = 2020110900;
-$plugin->requires = 2020110300;
+$plugin->version = 2020061500;
+$plugin->requires = 2020060900;
 $plugin->component = 'tool_availabilityconditions';

admin/tool/behat/cli/init.php

@@ -49,10 +49,6 @@ list($options, $unrecognized) = cli_get_params(
         'torun'    => 0,
         'optimize-runs' => '',
         'add-core-features-to-theme' => false,
-        'axe'      => false,
-        'disable-composer' => false,
-        'composer-upgrade' => true,
-        'composer-self-update' => true,
     ),
     array(
         'j' => 'parallel',
@@ -68,36 +64,18 @@ $help = "
 Behat utilities to initialise behat tests
 
 Usage:
-  php init.php      [--parallel=value [--maxruns=value] [--fromrun=value --torun=value]]
-                    [--axe] [-o | --optimize-runs] [-a | --add-core-features-to-theme]
-                    [--no-composer-self-update] [--no-composer-upgrade]
-                    [--disable-composer]
-                    [--help]
+  php init.php [--parallel=value [--maxruns=value] [--fromrun=value --torun=value]] [--help]
 
 Options:
--j, --parallel      Number of parallel behat run to initialise
--m, --maxruns       Max parallel processes to be executed at one time
---fromrun           Execute run starting from (Used for parallel runs on different vms)
---torun             Execute run till (Used for parallel runs on different vms)
---axe               Include axe accessibility tests
+-j, --parallel   Number of parallel behat run to initialise
+-m, --maxruns    Max parallel processes to be executed at one time.
+--fromrun        Execute run starting from (Used for parallel runs on different vms)
+--torun          Execute run till (Used for parallel runs on different vms)
 
--o, --optimize-runs
-                    Split features with specified tags in all parallel runs.
+-o, --optimize-runs Split features with specified tags in all parallel runs.
+-a, --add-core-features-to-theme Add all core features to specified theme's
 
--a, --add-core-features-to-theme
-                    Add all core features to specified theme's
-
---no-composer-self-update
-                    Prevent upgrade of the composer utility using its self-update command
-
---no-composer-upgrade
-                    Prevent update development dependencies using composer
-
---disable-composer
-                    A shortcut to disable composer self-update and dependency update
-                    Note: Installation of composer and/or dependencies will still happen as required
-
--h, --help          Print out this help
+-h, --help     Print out this help
 
 Example from Moodle root directory:
 \$ php admin/tool/behat/cli/init.php --parallel=2
@@ -124,10 +102,7 @@ if ($options['parallel'] && $options['parallel'] > 1) {
     }
 } else {
     // Only sanitize options for single run.
-    $cmdoptionsforsinglerun = [
-        'add-core-features-to-theme',
-        'axe',
-    ];
+    $cmdoptionsforsinglerun = array('add-core-features-to-theme');
 
     foreach ($cmdoptionsforsinglerun as $option) {
         if (!empty($options[$option])) {
@@ -140,15 +115,8 @@ if ($options['parallel'] && $options['parallel'] > 1) {
 $cwd = getcwd();
 $output = null;
 
-if ($options['disable-composer']) {
-    // Disable self-update and upgrade easily.
-    // Note: Installation will still occur regardless of this setting.
-    $options['composer-self-update'] = false;
-    $options['composer-upgrade'] = false;
-}
-
-// Install and update composer and dependencies as required.
-testing_update_composer_dependencies($options['composer-self-update'], $options['composer-upgrade']);
+// If behat dependencies not downloaded then do it first, else symfony/process can't be used.
+testing_update_composer_dependencies();
 
 // Check whether the behat test environment needs to be updated.
 chdir(__DIR__);

admin/tool/behat/cli/run.php

@@ -45,7 +45,7 @@ list($options, $unrecognised) = cli_get_params(
     array(
         'stop-on-failure' => 0,
         'verbose'  => false,
-        'replace'  => '',
+        'replace'  => false,
         'help'     => false,
         'tags'     => '',
         'profile'  => '',
@@ -70,7 +70,7 @@ $help = "
 Behat utilities to run behat tests in parallel
 
 Usage:
-  php run.php [--BEHAT_OPTION=\"value\"] [--feature=\"value\"] [--replace=\"{run}\"] [--fromrun=value --torun=value] [--help]
+  php run.php [--BEHAT_OPTION=\"value\"] [--feature=\"value\"] [--replace] [--fromrun=value --torun=value] [--help]
 
 Options:
 --BEHAT_OPTION     Any combination of behat option specified in http://behat.readthedocs.org/en/v2.5/guides/6.cli.html
@@ -144,10 +144,9 @@ $extraopts = $unrecognised;
 if ($options['profile']) {
     $profile = $options['profile'];
 
-    // If profile passed is not set, then exit (note we skip if the 'replace' option is found within the 'profile' value).
+    // If profile passed is not set, then exit.
     if (!isset($CFG->behat_config[$profile]) && !isset($CFG->behat_profiles[$profile]) &&
-            !($options['replace'] && (strpos($profile, (string) $options['replace']) !== false))) {
-
+        !(isset($options['replace']) && (strpos($options['profile'], $options['replace']) >= 0 ))) {
         echo "Invalid profile passed: " . $profile . PHP_EOL;
         exit(1);
     }

admin/tool/behat/cli/util.php

@@ -54,7 +54,6 @@ list($options, $unrecognized) = cli_get_params(
         'torun'       => 0,
         'optimize-runs' => '',
         'add-core-features-to-theme' => false,
-        'axe'         => false,
     ),
     array(
         'h' => 'help',
@@ -70,7 +69,7 @@ $help = "
 Behat utilities to manage the test environment
 
 Usage:
-  php util.php [--install|--drop|--enable|--disable|--diag|--updatesteps|--axe|--help] [--parallel=value [--maxruns=value]]
+  php util.php [--install|--drop|--enable|--disable|--diag|--updatesteps|--help] [--parallel=value [--maxruns=value]]
 
 Options:
 --install      Installs the test environment for acceptance tests
@@ -79,7 +78,6 @@ Options:
 --disable      Disables test environment
 --diag         Get behat test environment status code
 --updatesteps  Update feature step file.
---axe          Include axe accessibility tests
 
 -j, --parallel Number of parallel behat run operation
 -m, --maxruns Max parallel processes to be executed at one time.

admin/tool/behat/cli/util_single_run.php

@@ -49,7 +49,6 @@ list($options, $unrecognized) = cli_get_params(
         'updatesteps' => false,
         'optimize-runs' => '',
         'add-core-features-to-theme' => false,
-        'axe'         => false,
     ),
     array(
         'h' => 'help',
@@ -76,7 +75,6 @@ Options:
 --disable        Disables test environment
 --diag           Get behat test environment status code
 --updatesteps    Update feature step file.
---axe            Include axe accessibility tests
 
 -o, --optimize-runs Split features with specified tags in all parallel runs.
 -a, --add-core-features-to-theme Add all core features to specified theme's
@@ -183,17 +181,11 @@ if ($options['install']) {
         behat_config_manager::set_behat_run_config_value('behatsiteenabled', 1);
     }
 
-    // Define whether to run Behat with axe tests.
-    behat_config_manager::set_behat_run_config_value('axe', $options['axe']);
-
     // Enable test mode.
-    $timestart = microtime(true);
-    mtrace('Creating Behat configuration ...', '');
     behat_util::start_test_mode($options['add-core-features-to-theme'], $options['optimize-runs'], $parallel, $run);
-    mtrace(' done in ' . round(microtime(true) - $timestart, 2) . ' seconds.');
 
     // Themes are only built in the 'enable' command.
-    behat_util::build_themes(true);
+    behat_util::build_themes();
     mtrace("Testing environment themes built");
 
     // This is only displayed once for parallel install.

admin/tool/behat/tests/behat/datetime_strings.feature

@@ -21,5 +21,5 @@ Feature: Transform date time string arguments
     And I log in as "teacher1"
     And I am on "Course 1" course homepage
     And I follow "Test assignment name"
-    And I should see "##yesterday##%A, %d %B %Y##"
+    And I should see "##yesterday##l, j F Y##"
     And I log out

admin/tool/behat/tests/behat/edit_permissions.feature

@@ -75,9 +75,9 @@ Feature: Edit capabilities
       | Your word for 'Student'             | Studier & 'learner' |
     And I press "Save and display"
     And I navigate to course participants
-    Then I should see "Teacher >= editing (Teacher)" in the "Teacher 1" "table_row"
-    And I should see "Teacher < \"editing\" (Non-editing teacher)" in the "Teaching Assistant" "table_row"
-    And I should see "Studier & 'learner' (Student)" in the "Student One" "table_row"
+    Then I should see "Teacher >= editing" in the "Teacher 1" "table_row"
+    And I should see "Teacher < \"editing\"" in the "Teaching Assistant" "table_row"
+    And I should see "Studier & 'learner'" in the "Student One" "table_row"
     And I navigate to "Users > Permissions" in current page administration
     And I should see "Teacher >= editing" in the "mod/forum:replypost" "table_row"
     And I should see "Teacher < \"editing\"" in the "mod/forum:replypost" "table_row"

admin/tool/behat/tests/behat/get_and_set_fields.feature

@@ -48,7 +48,7 @@ Feature: Verify that all form fields values can be get and set
     And the "Unenrol users" select box should not contain "Baker"
     And the "Unenrol users" select box should not contain "President, Baker"
     And I am on "Course 1" course homepage with editing mode on
-    And I am on the "Test this one" "wiki activity" page
+    And I follow "Test this one"
     And I press "Create page"
     # Text (textarea & editor) & Select (multi-select) - Checking "I set the following fields to these values".
     When I set the following fields to these values:
@@ -61,7 +61,8 @@ Feature: Verify that all form fields values can be get and set
     # Checkbox - Checking "I set the field" and "The field matches value" ticked.
     And I set the field "Force format" to "1"
     And I press "Save and return to course"
-    And I am on the "Test this one" "wiki activity" page
+    And I should see "Test this one"
+    And I follow "Test this one"
     And I navigate to "Edit settings" in current page administration
     And I expand all fieldsets
     And the field "Force format" matches value "1"
@@ -69,7 +70,8 @@ Feature: Verify that all form fields values can be get and set
     # Checkbox - Checking "I set the field" and "The field matches value" unticked.
     And I set the field "Force format" to ""
     And I press "Save and return to course"
-    And I am on the "Test this one" "wiki activity" page
+    And I should see "Test this one"
+    And I follow "Test this one"
     And I navigate to "Edit settings" in current page administration
     And I expand all fieldsets
     And the field "Force format" matches value ""
@@ -78,7 +80,8 @@ Feature: Verify that all form fields values can be get and set
     And I set the following fields to these values:
       | Force format | 1 |
     And I press "Save and return to course"
-    And I am on the "Test this one" "wiki activity" page
+    And I should see "Test this one"
+    And I follow "Test this one"
     And I navigate to "Edit settings" in current page administration
     And I expand all fieldsets
     And the following fields match these values:
@@ -89,7 +92,8 @@ Feature: Verify that all form fields values can be get and set
     And I set the following fields to these values:
       | Force format | |
     And I press "Save and return to course"
-    And I am on the "Test this one" "wiki activity" page
+    And I should see "Test this one"
+    And I follow "Test this one"
     And I navigate to "Edit settings" in current page administration
     And I expand all fieldsets
     And the following fields match these values:
@@ -111,21 +115,20 @@ Feature: Verify that all form fields values can be get and set
       | Wiki name | Test this one baby |
       | Default format | HTML |
     And I press "Cancel"
+    And I am on "Course 1" course homepage
     # Radio - Checking "I set the field" and "the field matches value".
-    And the following "activity" exists:
-      | activity         | choice                       |
-      | course           | C1                           |
-      | idnumber         | choice1                      |
-      | intro            | Test choice description      |
-      | name             | Test choice name             |
-      | option           | Option 1, Option 2, Option 3 |
-      | section          | 1                            |
-      | allowupdate      | 1                            |
-    And I am on the "Test choice name" "choice activity" page
-    And I set the field "Option 1" to "1"
+    And I add a "Choice" to section "1" and I fill the form with:
+      | Choice name | Test choice name |
+      | Description | Test choice description |
+      | Allow choice to be updated | Yes |
+      | Option 1 | one |
+      | Option 2 | two |
+      | Option 3 | three |
+    And I follow "Test choice name"
+    And I set the field "one" to "1"
     And I press "Save my choice"
-    And the field "Option 1" matches value "1"
-    And the field "Option 2" matches value ""
+    And the field "one" matches value "1"
+    And the field "two" matches value ""
     # Check if field xpath set/match works.
     And I am on "Course 1" course homepage
     And I navigate to "Edit settings" in current page administration
@@ -171,23 +174,31 @@ Feature: Verify that all form fields values can be get and set
     And I press "Cancel"
     # Advanced checkbox requires real browser to allow uncheck to work. MDL-58681. MDL-55386.
     # Advanced checkbox - Checking "I set the field" and "The field matches value" ticked.
-    And I am on the "Test choice name" "choice activity editing" page
+    And I am on "Course 1" course homepage
+    And I follow "Test this one"
+    And I navigate to "Edit settings" in current page administration
     And I set the field "Display description on course page" to "1"
     And I press "Save and return to course"
-    And I am on the "Test choice name" "choice activity editing" page
+    And I should see "Test this one"
+    And I follow "Test this one"
+    And I navigate to "Edit settings" in current page administration
     And the field "Display description on course page" matches value "1"
     And the field "Display description on course page" does not match value ""
     # Advanced checkbox - Checking "I set the field" and "The field matches value" unticked.
     And I set the field "Display description on course page" to ""
     And I press "Save and return to course"
-    And I am on the "Test choice name" "choice activity editing" page
+    And I should see "Test this one"
+    And I follow "Test this one"
+    And I navigate to "Edit settings" in current page administration
     And the field "Display description on course page" matches value ""
     And the field "Display description on course page" does not match value "1"
     # Advanced checkbox - Checking "I set the following fields to these values:" and "The following fields match these values" ticked.
     And I set the following fields to these values:
       | Display description on course page | 1 |
     And I press "Save and return to course"
-    And I am on the "Test choice name" "choice activity editing" page
+    And I should see "Test this one"
+    And I follow "Test this one"
+    And I navigate to "Edit settings" in current page administration
     And the following fields match these values:
       | Display description on course page | 1 |
     And the following fields do not match these values:
@@ -196,7 +207,9 @@ Feature: Verify that all form fields values can be get and set
     And I set the following fields to these values:
       | Display description on course page | |
     And I press "Save and return to course"
-    And I am on the "Test choice name" "choice activity editing" page
+    And I should see "Test this one"
+    And I follow "Test this one"
+    And I navigate to "Edit settings" in current page administration
     And the following fields match these values:
       | Display description on course page | |
     And the following fields do not match these values:

admin/tool/behat/tests/behat/i_am_on_page.feature

@@ -1,85 +0,0 @@
-@tool @tool_behat
-Feature: Use core page resolvers for the I am on the page steps
-  In order to write tests correctly
-  As a developer
-  I need to have steps which take me straight to a page
-
-  Scenario Outline: When I am on an instance
-    Given the following "categories" exist:
-      | name  | category | idnumber |
-      | Cat 1 | 0        | CAT1     |
-    And the following "course" exists:
-      | fullname  | Economics 101 |
-      | shortname | ECON101       |
-      | idnumber  | 2021-econ101  |
-    And the following "activity" exists:
-      | course   | ECON101                   |
-      | activity | forum                     |
-      | name     | Fundamentals of Economics |
-      | idnumber | fundamentalsofeconomics   |
-    And I log in as "admin"
-    When I am on the <identifier> <type> page
-    Then I should see "<shouldsee>"
-
-    Examples:
-      | description              | identifier                  | type                         | shouldsee                  |
-      | Course Category idnumber | CAT1                        | category                     | Add a new course           |
-      | Course Category name     | "Cat 1"                     | Category                     | Add a new course           |
-      | Course Full name         | "Economics 101"             | course                       | Fundamentals of Economics  |
-      | Course Short name        | ECON101                     | COURSE                       | Fundamentals of Economics  |
-      | Course idnumber          | "2021-econ101"              | Course                       | Fundamentals of Economics  |
-      | Forum idnumber           | fundamentalsofeconomics     | Activity                     | Add a new discussion       |
-      | Generic activity editing | fundamentalsofeconomics     | "Activity editing"           | Updating: Forum            |
-      | Forum name               | "Fundamentals of Economics" | "Forum activity"             | Add a new discussion       |
-      | Forum name editing       | "Fundamentals of Economics" | "Forum activity editing"     | Updating: Forum            |
-      | Forum name permissions   | "Fundamentals of Economics" | "Forum activity permissions" | Permissions in Forum: Fun  |
-      | Forum name roles         | "Fundamentals of Economics" | "Forum activity roles"       | Assign roles in Forum: Fun |
-
-  Scenario Outline: When I am on an instance logged in as
-    Given the following "categories" exist:
-      | name  | category | idnumber |
-      | Cat 1 | 0        | CAT1     |
-    And the following "course" exists:
-      | fullname  | Economics 101 |
-      | shortname | ECON101       |
-      | idnumber  | 2021-econ101  |
-    And the following "activity" exists:
-      | course   | ECON101                   |
-      | activity | forum                     |
-      | name     | Fundamentals of Economics |
-      | idnumber | fundamentalsofeconomics   |
-    When I am on the <identifier> <type> page logged in as admin
-    Then I should see "<shouldsee>"
-
-    Examples:
-      | description              | identifier                  | type                         | shouldsee                  |
-      | Course Category idnumber | CAT1                        | category                     | Add a new course           |
-      | Course Category name     | "Cat 1"                     | Category                     | Add a new course           |
-      | Course Full name         | "Economics 101"             | course                       | Fundamentals of Economics  |
-      | Course Short name        | ECON101                     | COURSE                       | Fundamentals of Economics  |
-      | Course idnumber          | "2021-econ101"              | Course                       | Fundamentals of Economics  |
-      | Forum idnumber           | fundamentalsofeconomics     | Activity                     | Add a new discussion       |
-      | Generic activity editing | fundamentalsofeconomics     | "Activity editing"           | Updating: Forum            |
-      | Forum name               | "Fundamentals of Economics" | "Forum activity"             | Add a new discussion       |
-      | Forum name editing       | "Fundamentals of Economics" | "Forum activity editing"     | Updating: Forum            |
-      | Forum name permissions   | "Fundamentals of Economics" | "Forum activity permissions" | Permissions in Forum: Fun  |
-      | Forum name roles         | "Fundamentals of Economics" | "Forum activity roles"       | Assign roles in Forum: Fun |
-
-  Scenario Outline: When I am on a named page
-    Given I log in as "admin"
-    When I am on the <identifier> page
-    Then I should see "<shouldsee>"
-
-    Examples:
-      | description | identifier | shouldsee                   |
-      | Admin page  | "Admin notifications"      | Check for available updates |
-      | Home page   | Homepage   | Recently accessed courses   |
-
-  Scenario Outline: When I am on a named page logged in as
-    When I am on the <identifier> page logged in as admin
-    Then I should see "<shouldsee>"
-
-    Examples:
-      | description | identifier            | shouldsee                   |
-      | Admin page  | "Admin notifications" | Check for available updates |
-      | Home page   | Homepage              | Recently accessed courses   |

admin/tool/behat/tests/manager_util_test.php

@@ -87,7 +87,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
     /**
      * Setup test.
      */
-    public function setUp(): void {
+    public function setup() {
         global $CFG;
 
         $this->resetAfterTest();
@@ -165,7 +165,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
 
@@ -219,7 +219,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
 
@@ -264,7 +264,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
 
@@ -295,7 +295,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -324,7 +324,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -368,7 +368,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -399,7 +399,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -428,7 +428,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -532,7 +532,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -611,7 +611,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
         // Check contexts.
@@ -646,7 +646,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
 
@@ -662,7 +662,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
 
@@ -678,7 +678,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
 
@@ -693,7 +693,7 @@ class tool_behat_manager_util_testcase extends advanced_testcase {
             $this->assertCount(count($paths), $suites[$themename]['paths']);
 
             foreach ($paths as $key => $feature) {
-                $this->assertStringContainsString($feature, $suites[$themename]['paths'][$key]);
+                $this->assertContains($feature, $suites[$themename]['paths'][$key]);
             }
         }
     }

admin/tool/behat/version.php

@@ -24,6 +24,6 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version   = 2020110900;   // The current plugin version (Date: YYYYMMDDXX)
-$plugin->requires  = 2020110300;   // Requires this Moodle version
+$plugin->version   = 2020061500;   // The current plugin version (Date: YYYYMMDDXX)
+$plugin->requires  = 2020060900;   // Requires this Moodle version
 $plugin->component = 'tool_behat'; // Full name of the plugin (used for diagnostics)

admin/tool/capability/tests/events_test.php

@@ -36,7 +36,7 @@ class tool_capability_events_testcase extends advanced_testcase {
     /**
      * Setup testcase.
      */
-    public function setUp(): void {
+    public function setUp() {
         $this->setAdminUser();
         $this->resetAfterTest();
     }

admin/tool/capability/version.php

@@ -24,6 +24,6 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version   = 2020110900; // The current plugin version (Date: YYYYMMDDXX).
-$plugin->requires  = 2020110300; // Requires this Moodle version.
+$plugin->version   = 2020061500; // The current plugin version (Date: YYYYMMDDXX).
+$plugin->requires  = 2020060900; // Requires this Moodle version.
 $plugin->component = 'tool_capability'; // Full name of the plugin (used for diagnostics).

admin/tool/capability/yui/build/moodle-tool_capability-search/moodle-tool_capability-search-debug.js

@@ -53,13 +53,6 @@ SEARCH.prototype = {
      * @protected
      */
     button: null,
-    /**
-     * The cancel button for the form.
-     * @property button
-     * @type Node
-     * @protected
-     */
-    cancel: null,
     /**
      * The last search node if there is one.
      * If there is a last search node then the last search term will be persisted between requests.
@@ -83,28 +76,17 @@ SEARCH.prototype = {
         this.button = this.form.all('input[type=submit]');
         this.lastsearch = this.form.one('input[name=search]');
 
-        var div = Y.Node.create('<div id="capabilitysearchui" class="input-group simplesearchform mb-2"' +
-            'data-fieldtype="text"></div>'),
-            label = Y.Node.create('<label for="capabilitysearch"><span class="sr-only"' +
-                this.get('strsearch') + '</span></label>');
-        this.cancel = Y.Node.create('<a class="btn btn-clear d-none icon-no-margin">' +
-                '<i class="icon fa fa-times fa-fw " aria-hidden="true"></i>' +
-                '</a>');
-        this.input = Y.Node.create('<input type="text" class="form-control withclear" placeholder="' +
-            this.get('strsearch') + '"id="capabilitysearch" />');
+        var div = Y.Node.create('<div id="capabilitysearchui" data-fieldtype="text"></div>'),
+            label = Y.Node.create('<label for="capabilitysearch">' + this.get('strsearch') + '</label>');
+        this.input = Y.Node.create('<input type="text" id="capabilitysearch" />');
 
-        div.append(label).append(this.input).append(this.cancel);
+        div.append(label).append(this.input);
 
         this.select.insert(div, 'before');
 
         this.input.on('keyup', this.typed, this);
         this.select.on('change', this.validate, this);
 
-        this.cancel.on('click', function() {
-            this.input.set('value', '');
-            this.typed();
-        }, this);
-
         if (this.lastsearch) {
             this.input.set('value', this.lastsearch.get('value'));
             this.typed();
@@ -151,11 +133,6 @@ SEARCH.prototype = {
                 last.set('selected', true);
             }
         }
-        if (search !== '') {
-            this.cancel.removeClass("d-none");
-        } else {
-            this.cancel.addClass("d-none");
-        }
         this.validate();
     }
 };

admin/tool/capability/yui/build/moodle-tool_capability-search/moodle-tool_capability-search-min.js

@@ -1 +1 @@
-YUI.add("moodle-tool_capability-search",function(s,t){var e=function(){e.superclass.constructor.apply(this,arguments)};e.prototype={form:null,select:null,selectoptions:{},input:null,button:null,cancel:null,lastsearch:null,initializer:function(){this.form=s.one("#capability-overview-form"),this.select=this.form.one("select[data-search=capability]"),this.select.setStyle("minWidth",this.select.get("offsetWidth")),this.select.get("options").each(function(t){var e=t.get("value");this.selectoptions[e]=t},this),this.button=this.form.all("input[type=submit]"),this.lastsearch=this.form.one("input[name=search]");var t=s.Node.create('<div id="capabilitysearchui" class="input-group simplesearchform mb-2"data-fieldtype="text"></div>'),e=s.Node.create('<label for="capabilitysearch"><span class="sr-only"'+this.get("strsearch")+"</span></label>");this.cancel=s.Node.create('<a class="btn btn-clear d-none icon-no-margin"><i class="icon fa fa-times fa-fw " aria-hidden="true"></i></a>'),this.input=s.Node.create('<input type="text" class="form-control withclear" placeholder="'+this.get("strsearch")+'"id="capabilitysearch" />'),t.append(e).append(this.input).append(this.cancel),this.select.insert(t,"before"),this.input.on("keyup",this.typed,this),this.select.on("change",this.validate,this),this.cancel.on("click",function(){this.input.set("value",""),this.typed()},this),this.lastsearch&&(this.input.set("value",this.lastsearch.get("value")),this.typed(),this.select.one("option[selected]")&&this.select.set("scrollTop",this.select.one("option[selected]").get("getX"))),this.validate()},validate:function(){this.button.set("disabled",""===this.select.get("value"))},typed:function(){var t,e=this.input.get("value"),s=0,i=null;for(t in this.lastsearch&&this.lastsearch.set("value",e),this.select.all("option").remove(),this.selectoptions)0<=t.indexOf(e)&&(s++,i=this.selectoptions[t],this.select.append(this.selectoptions[t]));0===s?this.input.addClass("error"):(this.input.removeClass("error"),1===s&&i.set("selected",!0)),""!==e?this.cancel.removeClass("d-none"):this.cancel.addClass("d-none"),this.validate()}},s.extend(e,s.Base,e.prototype,{NAME:"tool_capability-search",ATTRS:{strsearch:{}}}),M.tool_capability=M.tool_capability||{},M.tool_capability.init_capability_search=function(t){new e(t)}},"@VERSION@",{requires:["base","node"]});
+YUI.add("moodle-tool_capability-search",function(i,t){var e=function(){e.superclass.constructor.apply(this,arguments)};e.prototype={form:null,select:null,selectoptions:{},input:null,button:null,lastsearch:null,initializer:function(){this.form=i.one("#capability-overview-form"),this.select=this.form.one("select[data-search=capability]"),this.select.setStyle("minWidth",this.select.get("offsetWidth")),this.select.get("options").each(function(t){var e=t.get("value");this.selectoptions[e]=t},this),this.button=this.form.all("input[type=submit]"),this.lastsearch=this.form.one("input[name=search]");var t=i.Node.create('<div id="capabilitysearchui" data-fieldtype="text"></div>'),e=i.Node.create('<label for="capabilitysearch">'+this.get("strsearch")+"</label>");this.input=i.Node.create('<input type="text" id="capabilitysearch" />'),t.append(e).append(this.input),this.select.insert(t,"before"),this.input.on("keyup",this.typed,this),this.select.on("change",this.validate,this),this.lastsearch&&(this.input.set("value",this.lastsearch.get("value")),this.typed(),this.select.one("option[selected]")&&this.select.set("scrollTop",this.select.one("option[selected]").get("getX"))),this.validate()},validate:function(){this.button.set("disabled",""===this.select.get("value"))},typed:function(){var t,e=this.input.get("value"),i=0,s=null;for(t in this.lastsearch&&this.lastsearch.set("value",e),this.select.all("option").remove(),this.selectoptions)0<=t.indexOf(e)&&(i++,s=this.selectoptions[t],this.select.append(this.selectoptions[t]));0===i?this.input.addClass("error"):(this.input.removeClass("error"),1===i&&s.set("selected",!0)),this.validate()}},i.extend(e,i.Base,e.prototype,{NAME:"tool_capability-search",ATTRS:{strsearch:{}}}),M.tool_capability=M.tool_capability||{},M.tool_capability.init_capability_search=function(t){new e(t)}},"@VERSION@",{requires:["base","node"]});

admin/tool/capability/yui/build/moodle-tool_capability-search/moodle-tool_capability-search.js

@@ -53,13 +53,6 @@ SEARCH.prototype = {
      * @protected
      */
     button: null,
-    /**
-     * The cancel button for the form.
-     * @property button
-     * @type Node
-     * @protected
-     */
-    cancel: null,
     /**
      * The last search node if there is one.
      * If there is a last search node then the last search term will be persisted between requests.
@@ -83,28 +76,17 @@ SEARCH.prototype = {
         this.button = this.form.all('input[type=submit]');
         this.lastsearch = this.form.one('input[name=search]');
 
-        var div = Y.Node.create('<div id="capabilitysearchui" class="input-group simplesearchform mb-2"' +
-            'data-fieldtype="text"></div>'),
-            label = Y.Node.create('<label for="capabilitysearch"><span class="sr-only"' +
-                this.get('strsearch') + '</span></label>');
-        this.cancel = Y.Node.create('<a class="btn btn-clear d-none icon-no-margin">' +
-                '<i class="icon fa fa-times fa-fw " aria-hidden="true"></i>' +
-                '</a>');
-        this.input = Y.Node.create('<input type="text" class="form-control withclear" placeholder="' +
-            this.get('strsearch') + '"id="capabilitysearch" />');
+        var div = Y.Node.create('<div id="capabilitysearchui" data-fieldtype="text"></div>'),
+            label = Y.Node.create('<label for="capabilitysearch">' + this.get('strsearch') + '</label>');
+        this.input = Y.Node.create('<input type="text" id="capabilitysearch" />');
 
-        div.append(label).append(this.input).append(this.cancel);
+        div.append(label).append(this.input);
 
         this.select.insert(div, 'before');
 
         this.input.on('keyup', this.typed, this);
         this.select.on('change', this.validate, this);
 
-        this.cancel.on('click', function() {
-            this.input.set('value', '');
-            this.typed();
-        }, this);
-
         if (this.lastsearch) {
             this.input.set('value', this.lastsearch.get('value'));
             this.typed();
@@ -151,11 +133,6 @@ SEARCH.prototype = {
                 last.set('selected', true);
             }
         }
-        if (search !== '') {
-            this.cancel.removeClass("d-none");
-        } else {
-            this.cancel.addClass("d-none");
-        }
         this.validate();
     }
 };

admin/tool/capability/yui/src/search/js/search.js

@@ -51,13 +51,6 @@ SEARCH.prototype = {
      * @protected
      */
     button: null,
-    /**
-     * The cancel button for the form.
-     * @property button
-     * @type Node
-     * @protected
-     */
-    cancel: null,
     /**
      * The last search node if there is one.
      * If there is a last search node then the last search term will be persisted between requests.
@@ -81,28 +74,17 @@ SEARCH.prototype = {
         this.button = this.form.all('input[type=submit]');
         this.lastsearch = this.form.one('input[name=search]');
 
-        var div = Y.Node.create('<div id="capabilitysearchui" class="input-group simplesearchform mb-2"' +
-            'data-fieldtype="text"></div>'),
-            label = Y.Node.create('<label for="capabilitysearch"><span class="sr-only"' +
-                this.get('strsearch') + '</span></label>');
-        this.cancel = Y.Node.create('<a class="btn btn-clear d-none icon-no-margin">' +
-                '<i class="icon fa fa-times fa-fw " aria-hidden="true"></i>' +
-                '</a>');
-        this.input = Y.Node.create('<input type="text" class="form-control withclear" placeholder="' +
-            this.get('strsearch') + '"id="capabilitysearch" />');
+        var div = Y.Node.create('<div id="capabilitysearchui" data-fieldtype="text"></div>'),
+            label = Y.Node.create('<label for="capabilitysearch">' + this.get('strsearch') + '</label>');
+        this.input = Y.Node.create('<input type="text" id="capabilitysearch" />');
 
-        div.append(label).append(this.input).append(this.cancel);
+        div.append(label).append(this.input);
 
         this.select.insert(div, 'before');
 
         this.input.on('keyup', this.typed, this);
         this.select.on('change', this.validate, this);
 
-        this.cancel.on('click', function() {
-            this.input.set('value', '');
-            this.typed();
-        }, this);
-
         if (this.lastsearch) {
             this.input.set('value', this.lastsearch.get('value'));
             this.typed();
@@ -149,11 +131,6 @@ SEARCH.prototype = {
                 last.set('selected', true);
             }
         }
-        if (search !== '') {
-            this.cancel.removeClass("d-none");
-        } else {
-            this.cancel.addClass("d-none");
-        }
         this.validate();
     }
 };

admin/tool/cohortroles/classes/output/cohort_role_assignments_table.php

@@ -189,12 +189,7 @@ class cohort_role_assignments_table extends table_sql {
                    FROM {tool_cohortroles} uca
                    JOIN {user} u ON u.id = uca.userid
                    JOIN {cohort} c ON c.id = uca.cohortid";
-
-        // Check if any additional filtering is required.
-        [$sqlwhere, $params] = $this->get_sql_where();
-        if ($sqlwhere) {
-            $sql .= " WHERE {$sqlwhere}";
-        }
+        $params = array();
 
         // Add order by if needed.
         if (!$count && $sqlsort = $this->get_sql_sort()) {

admin/tool/cohortroles/db/upgrade.php

@@ -44,8 +44,5 @@ function xmldb_tool_cohortroles_upgrade($oldversion) {
     // Automatically generated Moodle v3.9.0 release upgrade line.
     // Put any upgrade step following this.
 
-    // Automatically generated Moodle v3.10.0 release upgrade line.
-    // Put any upgrade step following this.
-
     return true;
 }

admin/tool/cohortroles/index.php

@@ -86,8 +86,8 @@ if ($removeid) {
 
     $title = get_string('existingcohortroles', 'tool_cohortroles');
     echo $output->heading($title);
-
-    $table = new tool_cohortroles\output\cohort_role_assignments_table('cohort-role-assignments', $pageurl);
+    $url = new moodle_url('/admin/tool/cohortroles/index.php');
+    $table = new tool_cohortroles\output\cohort_role_assignments_table(uniqid(), $url);
     echo $table->out(50, true);
 
     echo $output->spacer();

admin/tool/cohortroles/tests/api_test.php

@@ -49,7 +49,7 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
     /**
      * Setup function- we will create a course and add an assign instance to it.
      */
-    protected function setUp(): void {
+    protected function setUp() {
         $this->resetAfterTest(true);
 
         // Create some users.
@@ -60,6 +60,9 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
         cohort_add_member($this->cohort->id, $this->userassignover->id);
     }
 
+    /**
+     * @expectedException required_capability_exception
+     */
     public function test_create_cohort_role_assignment_without_permission() {
         $this->setUser($this->userassignto);
         $params = (object) array(
@@ -67,10 +70,12 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
             'roleid' => $this->roleid,
             'cohortid' => $this->cohort->id
         );
-        $this->expectException(required_capability_exception::class);
         api::create_cohort_role_assignment($params);
     }
 
+    /**
+     * @expectedException core_competency\invalid_persistent_exception
+     */
     public function test_create_cohort_role_assignment_with_invalid_data() {
         $this->setAdminUser();
         $params = (object) array(
@@ -78,7 +83,6 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
             'roleid' => -8,
             'cohortid' => $this->cohort->id
         );
-        $this->expectException(\core_competency\invalid_persistent_exception::class);
         api::create_cohort_role_assignment($params);
     }
 
@@ -96,6 +100,9 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
         $this->assertEquals($result->get('cohortid'), $this->cohort->id);
     }
 
+    /**
+     * @expectedException required_capability_exception
+     */
     public function test_delete_cohort_role_assignment_without_permission() {
         $this->setAdminUser();
         $params = (object) array(
@@ -105,10 +112,12 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
         );
         $result = api::create_cohort_role_assignment($params);
         $this->setUser($this->userassignto);
-        $this->expectException(required_capability_exception::class);
         api::delete_cohort_role_assignment($result->get('id'));
     }
 
+    /**
+     * @expectedException dml_missing_record_exception
+     */
     public function test_delete_cohort_role_assignment_with_invalid_data() {
         $this->setAdminUser();
         $params = (object) array(
@@ -117,7 +126,6 @@ class tool_cohortroles_api_testcase extends advanced_testcase {
             'cohortid' => $this->cohort->id
         );
         $result = api::create_cohort_role_assignment($params);
-        $this->expectException(dml_missing_record_exception::class);
         api::delete_cohort_role_assignment($result->get('id') + 1);
     }
 

admin/tool/cohortroles/tests/privacy_test.php

@@ -43,7 +43,7 @@ class tool_cohortroles_privacy_testcase extends \core_privacy\tests\provider_tes
     /**
      * Overriding setUp() function to always reset after tests.
      */
-    public function setUp(): void {
+    public function setUp() {
         $this->resetAfterTest(true);
     }
 
@@ -87,7 +87,7 @@ class tool_cohortroles_privacy_testcase extends \core_privacy\tests\provider_tes
             CONTEXT_COURSECAT
         ];
         // Test the User's contexts equal the system and course category context.
-        $this->assertEqualsCanonicalizing($expected, $contextlevels);
+        $this->assertEquals($expected, $contextlevels, '', 0, 10, true);
     }
 
     /**

admin/tool/cohortroles/version.php

@@ -25,8 +25,8 @@
 defined('MOODLE_INTERNAL') || die();
 
 
-$plugin->version   = 2020110900; // The current plugin version (Date: YYYYMMDDXX).
-$plugin->requires  = 2020110300; // Requires this Moodle version.
+$plugin->version   = 2020061500; // The current plugin version (Date: YYYYMMDDXX).
+$plugin->requires  = 2020060900; // Requires this Moodle version.
 $plugin->component = 'tool_cohortroles'; // Full name of the plugin (used for diagnostics).
 
 $plugin->dependencies = array(

admin/tool/customlang/classes/form/export.php

@@ -1,68 +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/>.
-
-/**
- * Creates Formular for customlang file export
- *
- * @package    tool_customlang
- * @copyright  2020 Thomas Wedekind <Thomas.Wedekind@univie.ac.at>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-namespace tool_customlang\form;
-
-use tool_customlang_utils;
-
-/**
- * Formular for customlang file export
- *
- * @copyright  2020 Thomas Wedekind <Thomas.Wedekind@univie.ac.at>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class export extends \moodleform {
-
-    /**
-     * Add elements to form
-     */
-    public function definition() {
-        $lng = $this->_customdata['lng'];
-        $mform = $this->_form;
-
-        $langdir = tool_customlang_utils::get_localpack_location($lng);
-
-        // The export button only appears if a local lang is present.
-        if (!check_dir_exists($langdir) || !count(glob("$langdir/*"))) {
-            print_error('nolocallang', 'tool_customlang');
-        }
-
-        $langfiles = scandir($langdir);
-        $fileoptions = [];
-        foreach ($langfiles as $file) {
-            if (substr($file, 0, 1) != '.') {
-                $fileoptions[$file] = $file;
-            }
-        }
-
-        $mform->addElement('hidden', 'lng', $lng);
-        $mform->setType('lng', PARAM_LANG);
-
-        $select = $mform->addElement('select', 'files', get_string('exportfilter', 'tool_customlang'), $fileoptions);
-        $select->setMultiple(true);
-        $mform->addRule('files', get_string('required'), 'required', null, 'client');
-        $mform->setDefault('files', $fileoptions);
-
-        $this->add_action_buttons(true, get_string('export', 'tool_customlang'));
-    }
-}

admin/tool/customlang/classes/form/import.php

@@ -1,68 +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/>.
-
-/**
- * Upload a zip of custom lang php files.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace tool_customlang\form;
-
-use tool_customlang\local\importer;
-
-/**
- * Upload a zip/php of custom lang php files.
- *
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class import extends \moodleform {
-
-    /**
-     * Form definition.
-     */
-    public function definition() {
-        $mform = $this->_form;
-        $mform->addElement('header', 'settingsheader', get_string('import', 'tool_customlang'));
-
-        $mform->addElement('hidden', 'lng');
-        $mform->setType('lng', PARAM_LANG);
-        $mform->setDefault('lng', $this->_customdata['lng']);
-
-        $filemanageroptions = array(
-            'accepted_types' => array('.php', '.zip'),
-            'maxbytes' => 0,
-            'maxfiles' => 1,
-            'subdirs' => 0
-        );
-
-        $mform->addElement('filepicker', 'pack', get_string('langpack', 'tool_customlang'),
-            null, $filemanageroptions);
-        $mform->addRule('pack', null, 'required');
-
-        $modes = [
-            importer::IMPORTALL => get_string('import_all', 'tool_customlang'),
-            importer::IMPORTUPDATE => get_string('import_update', 'tool_customlang'),
-            importer::IMPORTNEW => get_string('import_new', 'tool_customlang'),
-        ];
-        $mform->addElement('select', 'importmode', get_string('import_mode', 'tool_customlang'), $modes);
-
-        $mform->addElement('submit', 'importcustomstrings', get_string('importfile', 'tool_customlang'));
-    }
-}

admin/tool/customlang/classes/local/importer.php

@@ -1,268 +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/>.
-
-/**
- * Custom lang importer.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace tool_customlang\local;
-
-use tool_customlang\local\mlang\phpparser;
-use tool_customlang\local\mlang\logstatus;
-use tool_customlang\local\mlang\langstring;
-use core\output\notification;
-use stored_file;
-use coding_exception;
-use moodle_exception;
-use core_component;
-use stdClass;
-
-/**
- * Class containing tha custom lang importer
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class importer {
-
-    /** @var int imports will only create new customizations */
-    public const IMPORTNEW = 1;
-    /** @var int imports will only update the current customizations */
-    public const IMPORTUPDATE = 2;
-    /** @var int imports all strings */
-    public const IMPORTALL = 3;
-
-    /**
-     * @var string the language name
-     */
-    protected $lng;
-
-    /**
-     * @var int the importation mode (new, update, all)
-     */
-    protected $importmode;
-
-    /**
-     * @var string request folder path
-     */
-    private $folder;
-
-    /**
-     * @var array import log messages
-     */
-    private $log;
-
-    /**
-     * Constructor for the importer class.
-     *
-     * @param string $lng the current language to import.
-     * @param int $importmode the import method (IMPORTALL, IMPORTNEW, IMPORTUPDATE).
-     */
-    public function __construct(string $lng, int $importmode = self::IMPORTALL) {
-        $this->lng = $lng;
-        $this->importmode = $importmode;
-        $this->log = [];
-    }
-
-    /**
-     * Returns the last parse log.
-     *
-     * @return logstatus[] mlang logstatus with the messages
-     */
-    public function get_log(): array {
-        return $this->log;
-    }
-
-    /**
-     * Import customlang files.
-     *
-     * @param stored_file[] $files array of files to import
-     */
-    public function import(array $files): void {
-        // Create a temporal folder to store the files.
-        $this->folder = make_request_directory(false);
-
-        $langfiles = $this->deploy_files($files);
-
-        $this->process_files($langfiles);
-    }
-
-    /**
-     * Deploy all files into a request folder.
-     *
-     * @param stored_file[] $files array of files to deploy
-     * @return string[] of file paths
-     */
-    private function deploy_files(array $files): array {
-        $result = [];
-        // Desploy all files.
-        foreach ($files as $file) {
-            if ($file->get_mimetype() == 'application/zip') {
-                $result = array_merge($result, $this->unzip_file($file));
-            } else {
-                $path = $this->folder.'/'.$file->get_filename();
-                $file->copy_content_to($path);
-                $result = array_merge($result, [$path]);
-            }
-        }
-        return $result;
-    }
-
-    /**
-     * Unzip a file into the request folder.
-     *
-     * @param stored_file $file the zip file to unzip
-     * @return string[] of zip content paths
-     */
-    private function unzip_file(stored_file $file): array {
-        $fp = get_file_packer('application/zip');
-        $zipcontents = $fp->extract_to_pathname($file, $this->folder);
-        if (!$zipcontents) {
-            throw new moodle_exception("Error Unzipping file", 1);
-        }
-        $result = [];
-        foreach ($zipcontents as $contentname => $success) {
-            if ($success) {
-                $result[] = $this->folder.'/'.$contentname;
-            }
-        }
-        return $result;
-    }
-
-    /**
-     * Import strings from a list of langfiles.
-     *
-     * @param string[] $langfiles an array with file paths
-     */
-    private function process_files(array $langfiles): void {
-        $parser = phpparser::get_instance();
-        foreach ($langfiles as $filepath) {
-            $component = $this->component_from_filepath($filepath);
-            if ($component) {
-                $strings = $parser->parse(file_get_contents($filepath));
-                $this->import_strings($strings, $component);
-            }
-        }
-    }
-
-    /**
-     * Try to get the component from a filepath.
-     *
-     * @param string $filepath the filepath
-     * @return stdCalss|null the DB record of that component
-     */
-    private function component_from_filepath(string $filepath) {
-        global $DB;
-
-        // Get component from filename.
-        $pathparts = pathinfo($filepath);
-        if (empty($pathparts['filename'])) {
-            throw new coding_exception("Cannot get filename from $filepath", 1);
-        }
-        $filename = $pathparts['filename'];
-
-        $normalized = core_component::normalize_component($filename);
-        if (count($normalized) == 1 || empty($normalized[1])) {
-            $componentname = $normalized[0];
-        } else {
-            $componentname = implode('_', $normalized);
-        }
-
-        $result = $DB->get_record('tool_customlang_components', ['name' => $componentname]);
-
-        if (!$result) {
-            $this->log[] = new logstatus('notice_missingcomponent', notification::NOTIFY_ERROR, null, $componentname);
-            return null;
-        }
-        return $result;
-    }
-
-    /**
-     * Import an array of strings into the customlang tables.
-     *
-     * @param langstring[] $strings the langstring to set
-     * @param stdClass $component the target component
-     */
-    private function import_strings(array $strings, stdClass $component): void {
-        global $DB;
-
-        foreach ($strings as $newstring) {
-            // Check current DB entry.
-            $customlang = $DB->get_record('tool_customlang', [
-                'componentid' => $component->id,
-                'stringid' => $newstring->id,
-                'lang' => $this->lng,
-            ]);
-            if (!$customlang) {
-                $customlang = null;
-            }
-
-            if ($this->can_save_string($customlang, $newstring, $component)) {
-                $customlang->local = $newstring->text;
-                $customlang->timecustomized = $newstring->timemodified;
-                $customlang->outdated = 0;
-                $customlang->modified = 1;
-                $DB->update_record('tool_customlang', $customlang);
-            }
-        }
-    }
-
-    /**
-     * Determine if a specific string can be saved based on the current importmode.
-     *
-     * @param stdClass $customlang customlang original record
-     * @param langstring $newstring the new strign to store
-     * @param stdClass $component the component target
-     * @return bool if the string can be stored
-     */
-    private function can_save_string(?stdClass $customlang, langstring $newstring, stdClass $component): bool {
-        $result = false;
-        $message = 'notice_success';
-        if (empty($customlang)) {
-            $message = 'notice_inexitentstring';
-            $this->log[] = new logstatus($message, notification::NOTIFY_ERROR, null, $component->name, $newstring);
-            return $result;
-        }
-
-        switch ($this->importmode) {
-            case self::IMPORTNEW:
-                $result = empty($customlang->local);
-                $warningmessage = 'notice_ignoreupdate';
-                break;
-            case self::IMPORTUPDATE:
-                $result = !empty($customlang->local);
-                $warningmessage = 'notice_ignorenew';
-                break;
-            case self::IMPORTALL:
-                $result = true;
-                break;
-        }
-        if ($result) {
-            $errorlevel = notification::NOTIFY_SUCCESS;
-        } else {
-            $errorlevel = notification::NOTIFY_ERROR;
-            $message = $warningmessage;
-        }
-        $this->log[] = new logstatus($message, $errorlevel, null, $component->name, $newstring);
-
-        return $result;
-    }
-}

admin/tool/customlang/classes/local/mlang/langstring.php

@@ -1,177 +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/>.
-
-/**
- * Language string based on David Mudrak langstring from local_amos.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace tool_customlang\local\mlang;
-
-use moodle_exception;
-use stdclass;
-
-/**
- * Class containing a lang string cleaned.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-/**
- * Represents a single string
- */
-class langstring {
-
-    /** @var string identifier */
-    public $id = null;
-
-    /** @var string */
-    public $text = '';
-
-    /** @var int the time stamp when this string was saved */
-    public $timemodified = null;
-
-    /** @var bool is deleted */
-    public $deleted = false;
-
-    /** @var stdclass extra information about the string */
-    public $extra = null;
-
-    /**
-     * Class constructor.
-     *
-     * @param string $id string identifier
-     * @param string $text string text
-     * @param int $timemodified
-     * @param int $deleted
-     * @param stdclass $extra
-     */
-    public function __construct(string $id, string $text = '', int $timemodified = null,
-            int $deleted = 0, stdclass $extra = null) {
-
-        if (is_null($timemodified)) {
-            $timemodified = time();
-        }
-        $this->id           = $id;
-        $this->text         = $text;
-        $this->timemodified = $timemodified;
-        $this->deleted      = $deleted;
-        $this->extra        = $extra;
-    }
-
-    /**
-     * Given a string text, returns it being formatted properly for storing in AMOS repository.
-     *
-     * Note: This method is taken directly from local_amos as it is highly tested and robust.
-     * The Moodle 1.x part is keep on puspose to make it easier the copy paste from both codes.
-     * This could change in the future when AMOS stop suporting the 1.x langstrings.
-     *
-     * We need to know for what branch the string should be prepared due to internal changes in
-     * format required by get_string()
-     * - for get_string() in Moodle 1.6 - 1.9 use $format == 1
-     * - for get_string() in Moodle 2.0 and higher use $format == 2
-     *
-     * Typical usages of this methods:
-     *  $t = langstring::fix_syntax($t);          // sanity new translations of 2.x strings
-     *  $t = langstring::fix_syntax($t, 1);       // sanity legacy 1.x strings
-     *  $t = langstring::fix_syntax($t, 2, 1);    // convert format of 1.x strings into 2.x
-     *
-     * Backward converting 2.x format into 1.x is not supported
-     *
-     * @param string $text string text to be fixed
-     * @param int $format target get_string() format version
-     * @param int $from which format version does the text come from, defaults to the same as $format
-     * @return string
-     */
-    public static function fix_syntax(string $text, int $format = 2, ?int $from = null): string {
-        if (is_null($from)) {
-            $from = $format;
-        }
-
-        // Common filter.
-        $clean = trim($text);
-        $search = [
-            // Remove \r if it is part of \r\n.
-            '/\r(?=\n)/',
-
-            // Control characters to be replaced with \n
-            // LINE TABULATION, FORM FEED, CARRIAGE RETURN, END OF TRANSMISSION BLOCK,
-            // END OF MEDIUM, SUBSTITUTE, BREAK PERMITTED HERE, NEXT LINE, START OF STRING,
-            // STRING TERMINATOR and Unicode character categorys Zl and Zp.
-            '/[\x{0B}-\r\x{17}\x{19}\x{1A}\x{82}\x{85}\x{98}\x{9C}\p{Zl}\p{Zp}]/u',
-
-            // Control characters to be removed
-            // NULL, ENQUIRY, ACKNOWLEDGE, BELL, SHIFT {OUT,IN}, DATA LINK ESCAPE,
-            // DEVICE CONTROL {ONE,TWO,THREE,FOUR}, NEGATIVE ACKNOWLEDGE, SYNCHRONOUS IDLE, ESCAPE,
-            // DELETE, PADDING CHARACTER, HIGH OCTET PRESET, NO BREAK HERE, INDEX,
-            // {START,END} OF SELECTED AREA, CHARACTER TABULATION {SET,WITH JUSTIFICATION},
-            // LINE TABULATION SET, PARTIAL LINE {FORWARD,BACKWARD}, REVERSE LINE FEED,
-            // SINGLE SHIFT {TWO,THREE}, DEVICE CONTROL STRING, PRIVATE USE {ONE,TWO},
-            // SET TRANSMIT STATE, MESSAGE WAITING, {START,END} OF GUARDED AREA,
-            // {SINGLE {GRAPHIC,} CHARACTER,CONTROL SEQUENCE} INTRODUCER, OPERATING SYSTEM COMMAND,
-            // PRIVACY MESSAGE, APPLICATION PROGRAM COMMAND, ZERO WIDTH {,NO-BREAK} SPACE,
-            // REPLACEMENT CHARACTER.
-            '/[\0\x{05}-\x{07}\x{0E}-\x{16}\x{1B}\x{7F}\x{80}\x{81}\x{83}\x{84}\x{86}-\x{93}\x{95}-\x{97}\x{99}-\x{9B}\x{9D}-\x{9F}\x{200B}\x{FEFF}\x{FFFD}]++/u',
-
-            // Remove trailing whitespace at the end of lines in a multiline string.
-            '/[ \t]+(?=\n)/',
-        ];
-        $replace = [
-            '',
-            "\n",
-            '',
-            '',
-        ];
-        $clean = preg_replace($search, $replace, $clean);
-
-        if (($format === 2) && ($from === 2)) {
-            // Sanity translations of 2.x strings.
-            $clean = preg_replace("/\n{3,}/", "\n\n\n", $clean); // Collapse runs of blank lines.
-
-        } else if (($format === 2) && ($from === 1)) {
-            // Convert 1.x string into 2.x format.
-            $clean = preg_replace("/\n{3,}/", "\n\n\n", $clean); // Collapse runs of blank lines.
-            $clean = preg_replace('/%+/', '%', $clean); // Collapse % characters.
-            $clean = str_replace('\$', '@@@___XXX_ESCAPED_DOLLAR__@@@', $clean); // Remember for later.
-            $clean = str_replace("\\", '', $clean); // Delete all slashes.
-            $clean = preg_replace('/(^|[^{])\$a\b(\->[a-zA-Z0-9_]+)?/', '\\1{$a\\2}', $clean); // Wrap placeholders.
-            $clean = str_replace('@@@___XXX_ESCAPED_DOLLAR__@@@', '$', $clean);
-            $clean = str_replace('&#36;', '$', $clean);
-
-        } else if (($format === 1) && ($from === 1)) {
-            // Sanity legacy 1.x strings.
-            $clean = preg_replace("/\n{3,}/", "\n\n", $clean); // Collapse runs of blank lines.
-            $clean = str_replace('\$', '@@@___XXX_ESCAPED_DOLLAR__@@@', $clean);
-            $clean = str_replace("\\", '', $clean); // Delete all slashes.
-            $clean = str_replace('$', '\$', $clean); // Escape all embedded variables.
-            // Unescape placeholders: only $a and $a->something are allowed. All other $variables are left escaped.
-            $clean = preg_replace('/\\\\\$a\b(\->[a-zA-Z0-9_]+)?/', '$a\\1', $clean); // Unescape placeholders.
-            $clean = str_replace('@@@___XXX_ESCAPED_DOLLAR__@@@', '\$', $clean);
-            $clean = str_replace('"', "\\\"", $clean); // Add slashes for ".
-            $clean = preg_replace('/%+/', '%', $clean); // Collapse % characters.
-            $clean = str_replace('%', '%%', $clean); // Duplicate %.
-
-        } else {
-            throw new moodle_exception('Unknown get_string() format version');
-        }
-        return $clean;
-    }
-}

admin/tool/customlang/classes/local/mlang/logstatus.php

@@ -1,92 +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/>.
-
-/**
- * Language string based on David Mudrak langstring from local_amos.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace tool_customlang\local\mlang;
-
-use moodle_exception;
-use stdclass;
-
-/**
- * Class containing a lang string cleaned.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-/**
- * Represents a single string
- */
-class logstatus {
-
-    /** @var langstring the current string */
-    public $langstring = null;
-
-    /** @var string the component */
-    public $component = null;
-
-    /** @var string the string ID */
-    public $stringid = null;
-
-    /** @var string the original filename */
-    public $filename = null;
-
-    /** @var int the error level */
-    public $errorlevel = null;
-
-    /** @var string the message identifier */
-    private $message;
-
-    /**
-     * Class creator.
-     *
-     * @param string $message the message identifier to display
-     * @param string $errorlevel the notice level
-     * @param string|null $filename the filename of this log
-     * @param string|null $component the component of this log
-     * @param langstring|null $langstring the langstring of this log
-     */
-    public function __construct(string $message, string $errorlevel, ?string $filename = null,
-            ?string $component = null, ?langstring $langstring = null) {
-
-        $this->filename = $filename;
-        $this->component = $component;
-        $this->langstring = $langstring;
-        $this->message = $message;
-        $this->errorlevel = $errorlevel;
-
-        if ($langstring) {
-            $this->stringid = $langstring->id;
-        }
-    }
-
-    /**
-     * Get the log message.
-     *
-     * @return string the log message.
-     */
-    public function get_message(): string {
-        return get_string($this->message, 'tool_customlang', $this);
-    }
-}

admin/tool/customlang/classes/local/mlang/phpparser.php

@@ -1,260 +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/>.
-
-/**
- * Mlang PHP based on David Mudrak phpparser for local_amos.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace tool_customlang\local\mlang;
-
-use coding_exception;
-use moodle_exception;
-
-/**
- * Parser of Moodle strings defined as associative array.
- *
- * Moodle core just includes this file format directly as normal PHP code. However
- * for security reasons, we must not do this for files uploaded by anonymous users.
- * This parser reconstructs the associative $string array without actually including
- * the file.
- */
-class phpparser {
-
-    /** @var holds the singleton instance of self */
-    private static $instance = null;
-
-    /**
-     * Prevents direct creation of object
-     */
-    private function __construct() {
-    }
-
-    /**
-     * Prevent from cloning the instance
-     */
-    public function __clone() {
-        throw new coding_exception('Cloning os singleton is not allowed');
-    }
-
-    /**
-     * Get the singleton instance fo this class
-     *
-     * @return phpparser singleton instance of phpparser
-     */
-    public static function get_instance(): phpparser {
-        if (is_null(self::$instance)) {
-            self::$instance = new phpparser();
-        }
-        return self::$instance;
-    }
-
-    /**
-     * Parses the given data in Moodle PHP string format
-     *
-     * Note: This method is adapted from local_amos as it is highly tested and robust.
-     * The priority is keeping it similar to the original one to make it easier to mantain.
-     *
-     * @param string $data definition of the associative array
-     * @param int $format the data format on the input, defaults to the one used since 2.0
-     * @return langstring[] array of langstrings of this file
-     */
-    public function parse(string $data, int $format = 2): array {
-        $result = [];
-        $strings = $this->extract_strings($data);
-        foreach ($strings as $id => $text) {
-            $cleaned = clean_param($id, PARAM_STRINGID);
-            if ($cleaned !== $id) {
-                continue;
-            }
-            $text = langstring::fix_syntax($text, 2, $format);
-            $result[] = new langstring($id, $text);
-        }
-        return $result;
-    }
-
-    /**
-     * Low level parsing method
-     *
-     * Note: This method is adapted from local_amos as it is highly tested and robust.
-     * The priority is keeping it similar to the original one to make it easier to mantain.
-     *
-     * @param string $data
-     * @return string[] the data strings
-     */
-    protected function extract_strings(string $data): array {
-
-        $strings = []; // To be returned.
-
-        if (empty($data)) {
-            return $strings;
-        }
-
-        // Tokenize data - we expect valid PHP code.
-        $tokens = token_get_all($data);
-
-        // Get rid of all non-relevant tokens.
-        $rubbish = [T_WHITESPACE, T_INLINE_HTML, T_COMMENT, T_DOC_COMMENT, T_OPEN_TAG, T_CLOSE_TAG];
-        foreach ($tokens as $i => $token) {
-            if (is_array($token)) {
-                if (in_array($token[0], $rubbish)) {
-                    unset($tokens[$i]);
-                }
-            }
-        }
-
-        $id = null;
-        $text = null;
-        $line = 0;
-        $expect = 'STRING_VAR'; // The first expected token is '$string'.
-
-        // Iterate over tokens and look for valid $string array assignment patterns.
-        foreach ($tokens as $token) {
-            $foundtype = null;
-            $founddata = null;
-            if (is_array($token)) {
-                $foundtype = $token[0];
-                $founddata = $token[1];
-                if (!empty($token[2])) {
-                    $line = $token[2];
-                }
-
-            } else {
-                $foundtype = 'char';
-                $founddata = $token;
-            }
-
-            if ($expect == 'STRING_VAR') {
-                if ($foundtype === T_VARIABLE and $founddata === '$string') {
-                    $expect = 'LEFT_BRACKET';
-                    continue;
-                } else {
-                    // Allow other code at the global level.
-                    continue;
-                }
-            }
-
-            if ($expect == 'LEFT_BRACKET') {
-                if ($foundtype === 'char' and $founddata === '[') {
-                    $expect = 'STRING_ID';
-                    continue;
-                } else {
-                    throw new moodle_exception('Parsing error. Expected character [ at line '.$line);
-                }
-            }
-
-            if ($expect == 'STRING_ID') {
-                if ($foundtype === T_CONSTANT_ENCAPSED_STRING) {
-                    $id = $this->decapsulate($founddata);
-                    $expect = 'RIGHT_BRACKET';
-                    continue;
-                } else {
-                    throw new moodle_exception('Parsing error. Expected T_CONSTANT_ENCAPSED_STRING array key at line '.$line);
-                }
-            }
-
-            if ($expect == 'RIGHT_BRACKET') {
-                if ($foundtype === 'char' and $founddata === ']') {
-                    $expect = 'ASSIGNMENT';
-                    continue;
-                } else {
-                    throw new moodle_exception('Parsing error. Expected character ] at line '.$line);
-                }
-            }
-
-            if ($expect == 'ASSIGNMENT') {
-                if ($foundtype === 'char' and $founddata === '=') {
-                    $expect = 'STRING_TEXT';
-                    continue;
-                } else {
-                    throw new moodle_exception('Parsing error. Expected character = at line '.$line);
-                }
-            }
-
-            if ($expect == 'STRING_TEXT') {
-                if ($foundtype === T_CONSTANT_ENCAPSED_STRING) {
-                    $text = $this->decapsulate($founddata);
-                    $expect = 'SEMICOLON';
-                    continue;
-                } else {
-                    throw new moodle_exception(
-                        'Parsing error. Expected T_CONSTANT_ENCAPSED_STRING array item value at line '.$line
-                    );
-                }
-            }
-
-            if ($expect == 'SEMICOLON') {
-                if (is_null($id) or is_null($text)) {
-                    throw new moodle_exception('Parsing error. NULL string id or value at line '.$line);
-                }
-                if ($foundtype === 'char' and $founddata === ';') {
-                    if (!empty($id)) {
-                        $strings[$id] = $text;
-                    }
-                    $id = null;
-                    $text = null;
-                    $expect = 'STRING_VAR';
-                    continue;
-                } else {
-                    throw new moodle_exception('Parsing error. Expected character ; at line '.$line);
-                }
-            }
-
-        }
-
-        return $strings;
-    }
-
-    /**
-     * Given one T_CONSTANT_ENCAPSED_STRING, return its value without quotes
-     *
-     * Also processes escaped quotes inside the text.
-     *
-     * Note: This method is taken directly from local_amos as it is highly tested and robust.
-     *
-     * @param string $text value obtained by token_get_all()
-     * @return string value without quotes
-     */
-    protected function decapsulate(string $text): string {
-
-        if (strlen($text) < 2) {
-            throw new moodle_exception('Parsing error. Expected T_CONSTANT_ENCAPSED_STRING in decapsulate()');
-        }
-
-        if (substr($text, 0, 1) == "'" and substr($text, -1) == "'") {
-            // Single quoted string.
-            $text = trim($text, "'");
-            $text = str_replace("\'", "'", $text);
-            $text = str_replace('\\\\', '\\', $text);
-            return $text;
-
-        } else if (substr($text, 0, 1) == '"' and substr($text, -1) == '"') {
-            // Double quoted string.
-            $text = trim($text, '"');
-            $text = str_replace('\"', '"', $text);
-            $text = str_replace('\\\\', '\\', $text);
-            return $text;
-
-        } else {
-            throw new moodle_exception(
-                'Parsing error. Unexpected quotation in T_CONSTANT_ENCAPSED_STRING in decapsulate(): '.$text
-            );
-        }
-    }
-}

admin/tool/customlang/classes/output/renderer.php

@@ -57,9 +57,8 @@ class renderer extends \plugin_renderer_base {
     protected function render_tool_customlang_menu(\tool_customlang_menu $menu) {
         $output = '';
         foreach ($menu->get_items() as $item) {
-            $button = $this->single_button($item->url, $item->title, $item->method);
-            $output .= $this->box($button, 'menu');
+            $output .= $this->single_button($item->url, $item->title, $item->method);
         }
-        return $output;
+        return $this->box($output, 'menu');
     }
 }

admin/tool/customlang/cli/export.php

@@ -1,135 +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/>.
-
-/**
- * Export custom language strings to zip files.
- *
- * @package    tool_customlang
- * @subpackage customlang
- * @copyright  2020 Thomas Wedekind <thomas.wedekind@univie.ac.at>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-define('CLI_SCRIPT', true);
-
-require(__DIR__ . '/../../../../config.php');
-require_once("$CFG->libdir/clilib.php");
-require_once("$CFG->dirroot/$CFG->admin/tool/customlang/locallib.php");
-
-$usage = <<<EOF
-"Export custom language files to a target folder.
-Useful for uploading custom langstrings to AMOS or importing or syncing them to other moodle instances.
-
-Options:
--l, --lang              Comma seperated language ids to export, default: all
--c, --components        Comma seperated components to export, default: all
--t, --target            Target to copy the zip files to, default: $CFG->tempdir/customlang
--o, --overwrite         Overwrite existing files in the target folder.
-                            Note: If the target is not set, the files are always overwritten!
--h, --help              Print out this help
-
-Examples:
-Export all custom language files to the default folder:
-\$ sudo -u www-data /usr/bin/php admin/tool/customlang/cli/export.php
-
-Export just the english files of moodle core and the activity 'quiz' in a subfolder in my home folder:
-\$ sudo -u www-data /usr/bin/php admin/tool/customlang/cli/export.php --lang='en' --components='moodle,quiz' --target='~/customdir'
-
-EOF;
-
-$dafaulttarget = "$CFG->tempdir/customlang/";
-
-// Now get cli options.
-list($options, $unrecognized) = cli_get_params(
-    [
-        'lang' => '',
-        'components' => '',
-        'target' => $dafaulttarget,
-        'overwrite' => false,
-        'help' => false,
-    ],
-    ['h' => 'help', 'c' => 'components', 't' => 'target', 'o' => 'overwrite']
-);
-
-if ($unrecognized) {
-    $unrecognized = implode("\n  ", $unrecognized);
-    cli_error(get_string('cliunknowoption', 'admin', $unrecognized));
-}
-
-if ($options['help']) {
-    echo $usage;
-    die;
-}
-if (!file_exists($options['target'])) {
-    mkdir($options['target'], 0777, true);
-}
-
-cli_writeln(get_string('cliexportheading', 'tool_customlang'));
-$langs = [];
-if ($options['lang']) {
-    $langs = explode(',', $options['lang']);
-} else {
-    // No language set. We export all installed languages.
-    $langs = array_keys(get_string_manager()->get_list_of_translations(true));
-}
-
-foreach ($langs as $lang) {
-    $filename = $options['target'] . get_string('exportzipfilename', 'tool_customlang', ['lang' => $lang]);
-    // If the file exists and we are not using the temp folder it requires an ovewrite.
-    if ($options['target'] != $dafaulttarget && file_exists($filename) && !$options['overwrite']) {
-        cli_problem(get_string('cliexportfileexists', 'tool_customlang', $lang));
-        continue;
-    }
-    cli_heading(get_string('cliexportstartexport', 'tool_customlang', $lang));
-    $langdir = tool_customlang_utils::get_localpack_location($lang);
-    if (!file_exists($langdir)) {
-        // No custom files set for this language set.
-        cli_writeln(get_string('cliexportnofilefoundforlang', 'tool_customlang', ['lang' => $lang]));
-        continue;
-    }
-    $zipper = get_file_packer();
-    $tempzip = tempnam($CFG->tempdir . '/', 'tool_customlang_export');
-    $filelist = [];
-    if ($options['components']) {
-        $components = explode(',', $options['components']);
-        foreach ($components as $component) {
-            $filepath = "$langdir/$component.php";
-            if (file_exists($filepath)) {
-                $filelist["$component.php"] = $filepath;
-            } else {
-                cli_problem(
-                    get_string('cliexportfilenotfoundforcomponent', 'tool_customlang', ['lang' => $lang, 'file' => $filepath])
-                );
-            }
-        }
-    } else {
-        $langfiles = scandir($langdir);
-        foreach ($langfiles as $file) {
-            if (substr($file, 0, 1) != '.') {
-                $filelist[$file] = "$langdir/$file";
-            }
-        }
-    }
-    if (empty($filelist)) {
-        cli_problem(get_string('cliexportnofilefoundforlang', 'tool_customlang', ['lang' => $lang]));
-        continue;
-    }
-    if ($zipper->archive_to_pathname($filelist, $filename)) {
-        cli_writeln(get_string('cliexportzipdone', 'tool_customlang', $filename));
-    } else {
-        cli_problem(get_string('cliexportzipfail', 'tool_customlang', $filename));
-    }
-}

admin/tool/customlang/cli/import.php

@@ -1,204 +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/>.
-
-/**
- * CLI customlang import tool.
- *
- * @package    tool_customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-use tool_customlang\local\importer;
-use core\output\notification;
-
-define('CLI_SCRIPT', true);
-
-require(__DIR__ . '/../../../../config.php');
-require_once($CFG->dirroot.'/'.$CFG->admin.'/tool/customlang/locallib.php');
-require_once("$CFG->libdir/clilib.php");
-
-$usage =
-"Import lang customization.
-
-It can get a single file or a folder.
-If no lang is provided it will try to infere from the filename
-
-Options:
---lang                  The target language (will get from filename if not provided)
---source=path           File or folder of the custom lang files (zip or php files)
---mode                  What string should be imported. Options are:
-                            - all: all string will be imported (default)
-                            - new: only string with no previous customisation
-                            - update: only strings already modified
---checkin               Save strings to the language pack
--h, --help              Print out this help
-
-Examples:
-\$ sudo -u www-data /usr/bin/php admin/tool/customlang/cli/import.php --lang=en --source=customlangs.zip
-
-\$ sudo -u www-data /usr/bin/php admin/tool/customlang/cli/import.php --source=/tmp/customlangs --checkin
-
-\$ sudo -u www-data /usr/bin/php admin/tool/customlang/cli/import.php --lang=en --source=/tmp/customlangs
-
-";
-
-list($options, $unrecognized) = cli_get_params(
-    [
-        'help' => false,
-        'lang' => false,
-        'source' => false,
-        'mode' => 'all',
-        'checkin' => false,
-    ],
-    ['h' => 'help']
-);
-
-if ($unrecognized) {
-    $unrecognized = implode("\n  ", $unrecognized);
-    cli_error(get_string('cliunknowoption', 'admin', $unrecognized));
-}
-
-if ($options['help']) {
-    cli_write($usage);
-    exit(0);
-}
-
-$source = $options['source'] ?? null;
-$lang = $options['lang'] ?? null;
-$modeparam = $options['mode'] ?? 'all';
-$checkin = $options['checkin'] ?? false;
-
-$modes = [
-    'all' => importer::IMPORTALL,
-    'update' => importer::IMPORTUPDATE,
-    'new' => importer::IMPORTNEW,
-];
-if (!isset($modes[$modeparam])) {
-    cli_error(get_string('climissingmode', 'tool_customlang'));
-}
-$mode = $modes[$modeparam];
-
-if (empty($source)) {
-    $source = $CFG->dataroot.'/temp/customlang';
-}
-
-if (!file_exists($source)) {
-    cli_error(get_string('climissingsource', 'tool_customlang'));
-}
-
-// Emulate normal session - we use admin account by default.
-cron_setup_user();
-
-// Get the file list.
-$files = [];
-$langfiles = [];
-
-if (is_file($source)) {
-    $files[] = $source;
-}
-if (is_dir($source)) {
-    $filelist = glob("$source/*");
-    foreach ($filelist as $filename) {
-        $files[] = "$filename";
-    }
-}
-
-$countfiles = 0;
-foreach ($files as $filepath) {
-    // Try to get the lang.
-    $filelang = $lang;
-    // Get component from filename.
-    $pathparts = pathinfo($filepath);
-    $filename = $pathparts['filename'];
-    $extension = $pathparts['extension'];
-    if ($extension == 'zip') {
-        if (!$filelang) {
-            // Try to get the lang from the filename.
-            if (strrpos($filename, 'customlang_') === 0) {
-                $parts = explode('_', $filename);
-                if (!empty($parts[1])) {
-                    $filelang = $parts[1];
-                }
-            }
-        }
-    } else if ($extension != 'php') {
-        // Ignore any other file extension.
-        continue;
-    }
-    if (empty($filelang)) {
-        cli_error(get_string('climissinglang', 'tool_customlang'));
-    }
-    if (!isset($langfiles[$filelang])) {
-        $langfiles[$filelang] = [];
-    }
-    $langfiles[$filelang][] = $filepath;
-    $countfiles ++;
-}
-
-if (!$countfiles) {
-    cli_error(get_string('climissingfiles', 'tool_customlang'));
-}
-
-foreach ($langfiles as $lng => $files) {
-    $importer = new importer($lng, $mode);
-    $storedfiles = [];
-    $fs = get_file_storage();
-
-    cli_heading(get_string('clifiles', 'tool_customlang', $lng));
-
-    foreach ($files as $file) {
-        // Generate a valid stored_file from this file.
-        $record = (object)[
-            'filearea' => 'draft',
-            'component' => 'user',
-            'filepath' => '/',
-            'itemid'   => file_get_unused_draft_itemid(),
-            'license'  => $CFG->sitedefaultlicense,
-            'author'   => '',
-            'filename' => clean_param(basename($file), PARAM_FILE),
-            'contextid' => \context_user::instance($USER->id)->id,
-            'userid' => $USER->id,
-        ];
-        cli_writeln($file);
-        $storedfiles[] = $fs->create_file_from_pathname($record, $file);
-    }
-    cli_writeln("");
-
-    // Import files.
-    cli_heading(get_string('cliimporting', 'tool_customlang', $modeparam));
-    $importer->import($storedfiles);
-    // Display logs.
-    $log = $importer->get_log();
-    if (empty($log)) {
-        cli_problem(get_string('clinolog', 'tool_customlang', $lng));
-    }
-    foreach ($log as $message) {
-        if ($message->errorlevel == notification::NOTIFY_ERROR) {
-            cli_problem($message->get_message());
-        } else {
-            cli_writeln($message->get_message());
-        }
-    }
-    // Do the checkin if necessary.
-    if ($checkin) {
-        tool_customlang_utils::checkin($lng);
-        cli_writeln(get_string('savecheckin', 'tool_customlang'));
-    }
-    cli_writeln("");
-}
-
-exit(0);

admin/tool/customlang/db/access.php

@@ -46,14 +46,5 @@ $capabilities = array(
             'manager' => CAP_ALLOW
         ),
     ),
-    /* allows the user to export the current language customization */
-    'tool/customlang:export' => array(
-        'riskbitmask' => RISK_CONFIG,
-        'captype' => 'read',
-        'contextlevel' => CONTEXT_SYSTEM,
-        'archetypes' => array(
-            'manager' => CAP_ALLOW
-        ),
-    ),
 
 );

admin/tool/customlang/db/upgrade.php

@@ -44,8 +44,5 @@ function xmldb_tool_customlang_upgrade($oldversion) {
     // Automatically generated Moodle v3.9.0 release upgrade line.
     // Put any upgrade step following this.
 
-    // Automatically generated Moodle v3.10.0 release upgrade line.
-    // Put any upgrade step following this.
-
     return true;
 }

admin/tool/customlang/export.php

@@ -1,68 +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/>.
-
-/**
- * Performs the custom lang export.
- *
- * @package    tool_customlang
- * @subpackage customlang
- * @copyright  2020 Thomas Wedekind <thomas.wedekind@univie.ac.at>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-require(__DIR__ . '/../../../config.php');
-require_once($CFG->dirroot.'/'.$CFG->admin.'/tool/customlang/locallib.php');
-require_once($CFG->libdir.'/adminlib.php');
-
-global $PAGE, $CFG;
-
-require_login(SITEID, false);
-require_capability('tool/customlang:export', context_system::instance());
-
-$lng = required_param('lng', PARAM_LANG);
-
-admin_externalpage_setup('toolcustomlang', '', null,
-    new moodle_url('/admin/tool/customlang/import.php', ['lng' => $lng]));
-
-$form = new \tool_customlang\form\export(null, ['lng' => $lng]);
-
-if ($form->is_cancelled()) {
-    redirect('index.php');
-    die();
-} else if ($formdata = $form->get_data()) {
-    $tempzip = tempnam($CFG->tempdir . '/', 'tool_customlang_export');
-    $filelist = [];
-    foreach ($formdata->files as $file) {
-        $filepath = tool_customlang_utils::get_localpack_location($lng). '/' . $file;
-        if (file_exists($filepath)) {
-            $filelist[$file] = $filepath;
-        }
-    }
-    $zipper = new zip_packer();
-
-    if (!empty($filelist) && $zipper->archive_to_pathname($filelist, $tempzip)) {
-        // Filename include the lang name so the file can be imported with automatic language detection.
-        send_temp_file($tempzip, "customlang_$lng.zip");
-        die();
-    }
-}
-
-$output = $PAGE->get_renderer('tool_customlang');
-
-echo $output->header();
-echo $output->heading(get_string('pluginname', 'tool_customlang'));
-$form->display();
-echo $OUTPUT->footer();

admin/tool/customlang/filter_form.php

@@ -32,6 +32,7 @@ class tool_customlang_filter_form extends moodleform {
 
     function definition() {
         $mform = $this->_form;
+        $current = $this->_customdata['current'];
 
         $mform->addElement('header', 'filtersettings', get_string('filter', 'tool_customlang'));
 

admin/tool/customlang/import.php

@@ -1,74 +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/>.
-
-/**
- * Import custom lang files.
- *
- * @package    tool_customlang
- * @subpackage customlang
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-use tool_customlang\form\import;
-use tool_customlang\local\importer;
-
-require(__DIR__ . '/../../../config.php');
-require_once($CFG->dirroot.'/'.$CFG->admin.'/tool/customlang/locallib.php');
-require_once($CFG->libdir.'/adminlib.php');
-
-require_login(SITEID, false);
-require_capability('tool/customlang:edit', context_system::instance());
-
-$lng = required_param('lng', PARAM_LANG);
-
-admin_externalpage_setup('toolcustomlang', '', null,
-    new moodle_url('/admin/tool/customlang/import.php', ['lng' => $lng]));
-
-$output = $PAGE->get_renderer('tool_customlang');
-
-$form = new import(null, ['lng' => $lng]);
-if ($data = $form->get_data()) {
-    require_sesskey();
-
-    // Get the file from the users draft area.
-    $usercontext = context_user::instance($USER->id);
-    $fs = get_file_storage();
-    $files = $fs->get_area_files($usercontext->id, 'user', 'draft', $data->pack, 'id',
-        false);
-
-    // Send files to the importer.
-    $importer = new importer($data->lng, $data->importmode);
-    $importer->import($files);
-
-    echo $output->header();
-
-    // Display logs.
-    $log = $importer->get_log();
-    foreach ($log as $message) {
-        echo $output->notification($message->get_message(), $message->errorlevel);
-    }
-
-    // Show continue button.
-    echo $output->continue_button(new moodle_url('index.php', array('lng' => $lng)));
-
-} else {
-    echo $output->header();
-
-    $form->display();
-}
-
-echo $OUTPUT->footer();

admin/tool/customlang/index.php

@@ -35,7 +35,6 @@ require_capability('tool/customlang:view', context_system::instance());
 $action  = optional_param('action', '', PARAM_ALPHA);
 $confirm = optional_param('confirm', false, PARAM_BOOL);
 $lng     = optional_param('lng', '', PARAM_LANG);
-$next     = optional_param('next', 'edit', PARAM_ALPHA);
 
 admin_externalpage_setup('toolcustomlang');
 $langs = get_string_manager()->get_list_of_translations();
@@ -60,10 +59,11 @@ if ($action === 'checkout') {
     raise_memory_limit(MEMORY_EXTRA);
     tool_customlang_utils::checkout($lng, $progressbar);
 
-    echo $output->continue_button(new moodle_url("/admin/tool/customlang/{$next}.php", array('lng' => $lng)), 'get');
+    echo $output->continue_button(new moodle_url('/admin/tool/customlang/edit.php', array('lng' => $lng)), 'get');
     echo $output->footer();
     exit;
 }
+
 if ($action === 'checkin') {
     require_sesskey();
     require_capability('tool/customlang:edit', context_system::instance());
@@ -132,21 +132,6 @@ if (has_capability('tool/customlang:edit', context_system::instance())) {
             'method'    => 'post',
         );
     }
-    $menu['import'] = array(
-        'title'     => get_string('import', 'tool_customlang'),
-        'url'       => new moodle_url($PAGE->url, ['action' => 'checkout', 'lng' => $lng, 'next' => 'import']),
-        'method'    => 'post',
-    );
-}
-if (has_capability('tool/customlang:export', context_system::instance())) {
-    $langdir = tool_customlang_utils::get_localpack_location($lng);
-    if (check_dir_exists(dirname($langdir)) && count(glob("$langdir/*"))) {
-        $menu['export'] = [
-            'title'     => get_string('export', 'tool_customlang'),
-            'url'       => new moodle_url("/admin/tool/customlang/export.php", ['lng' => $lng]),
-            'method'    => 'post',
-        ];
-    }
 }
 echo $output->render(new tool_customlang_menu($menu));
 

admin/tool/customlang/lang/en/tool_customlang.php

@@ -30,26 +30,9 @@ $string['checkin'] = 'Save strings to language pack';
 $string['checkout'] = 'Open language pack for editing';
 $string['checkoutdone'] = 'Language pack loaded';
 $string['checkoutinprogress'] = 'Loading language pack';
-$string['cliexportfileexists'] = 'File for {$a->lang} already exists, skipping. If you want to overwrite add the --override=true option.';
-$string['cliexportheading'] = 'Starting to export lang files.';
-$string['cliexportnofilefoundforlang'] = 'No file found to export. Skipping export for this language.';
-$string['cliexportfilenotfoundforcomponent'] = 'File {$a->filepath} not found for language {$a->lang}. Skipping this file.';
-$string['cliexportstartexport'] = 'Exporting language {$a}';
-$string['cliexportzipdone'] = 'Zip created: {$a}';
-$string['cliexportzipfail'] = 'Cannot create zip {$a}';
-$string['clifiles'] = 'Files to import into {$a}';
-$string['cliimporting'] = 'Import files string (mode {$a})';
-$string['clinolog'] = 'Nothing to import into {$a}';
-$string['climissinglang'] = 'Missing language';
-$string['climissingfiles'] = 'Missing valid files';
-$string['climissingmode'] = 'Missing or invalid mode (valid is all, new or update)';
-$string['climissingsource'] = 'Missing file or folder';
 $string['confirmcheckin'] = 'You are about to save modifications to your local language pack. This will export the customised strings from the translator into your site data directory and your site will start using the modified strings. Press \'Continue\' to proceed with saving.';
 $string['customlang:edit'] = 'Edit local translation';
-$string['customlang:export'] = 'Export local translation';
 $string['customlang:view'] = 'View local translation';
-$string['export'] = 'Export custom strings';
-$string['exportfilter'] = 'Select component(s) to export';
 $string['filter'] = 'Filter strings';
 $string['filtercomponent'] = 'Show strings of these components';
 $string['filtercustomized'] = 'Customised only';
@@ -62,24 +45,11 @@ $string['headingcomponent'] = 'Component';
 $string['headinglocal'] = 'Local customisation';
 $string['headingstandard'] = 'Standard text';
 $string['headingstringid'] = 'String';
-$string['import'] = 'Import custom strings';
-$string['import_mode'] = 'Import mode';
-$string['import_new'] = 'Create only strings without local customisation';
-$string['import_update'] = 'Update only strings with local customisation';
-$string['import_all'] = 'Create or update all strings from the component(s)';
-$string['importfile'] = 'Import file';
-$string['langpack'] = 'Language component(s)';
 $string['markinguptodate'] = 'Marking the customisation as up-to-date';
 $string['markinguptodate_help'] = 'The customised translation may get outdated if either the English original or the master translation has modified since the string was customised on your site. Review the customised translation. If you find it up-to-date, click the checkbox. Edit it otherwise.';
 $string['markuptodate'] = 'mark as up-to-date';
 $string['modifiedno'] = 'There are no modified strings to save.';
 $string['modifiednum'] = 'There are {$a} modified strings. Do you wish to save these changes to your local language pack?';
-$string['nolocallang'] = 'No local strings found.';
-$string['notice_ignorenew'] = 'Ignoring string {$a->component}/{$a->stringid} because it is not customised.';
-$string['notice_ignoreupdate'] = 'Ignoring string {$a->component}/{$a->stringid} because it is already defined.';
-$string['notice_inexitentstring'] = 'String {$a->component}/{$a->stringid} not found.';
-$string['notice_missingcomponent'] = 'Missing component {$a->component}.';
-$string['notice_success'] = 'String {$a->component}/{$a->stringid} updated successfully.';
 $string['nostringsfound'] = 'No strings found, please modify the filter settings';
 $string['placeholder'] = 'Placeholders';
 $string['placeholder_help'] = 'Placeholders are special statements like `{$a}` or `{$a->something}` within the string. They are replaced with a value when the string is actually printed.

admin/tool/customlang/locallib.php

@@ -249,7 +249,7 @@ class tool_customlang_utils {
      * @param string $lang language code
      * @return string full path
      */
-    public static function get_localpack_location($lang) {
+    protected static function get_localpack_location($lang) {
         global $CFG;
 
         return $CFG->langlocalroot.'/'.$lang.'_local';

admin/tool/customlang/templates/translator.mustache

@@ -59,7 +59,7 @@
     <input type="hidden" name="sesskey" value="{{{ sesskey }}}">
     <input type="hidden" name="p" value="{{ currentpage }}">
 
-    <fieldset class="m-3">
+    <fieldset class="m-a-1 m-3">
         <button type="submit" name="savecontinue" class="btn btn-secondary">
             {{#str}}savecontinue, tool_customlang{{/str}}
         </button>
@@ -70,7 +70,7 @@
 
     <div class="list-group">
         <div class="container-fluid d-none d-md-block list-group-item border-bottom-0">
-            <div class="row">
+            <div class="row-fluid">
                 <div class="col-sm-4 col-md-2">
                     <strong>{{#str}}headingcomponent, tool_customlang{{/str}}</strong>
                 </div>
@@ -81,7 +81,7 @@
                     <strong>{{#str}}headingstandard, tool_customlang{{/str}}</strong>
                 </div>
                 <div class="col-sm-12 col-md-6">
-                    <span class="pl-3">
+                    <span class="p-l-1 pl-3">
                         <strong>{{#str}}headinglocal, tool_customlang{{/str}}</strong>
                     </span>
                 </div>
@@ -96,7 +96,7 @@
                 {{#outdated}}list-group-item-warning{{/outdated}}
                 {{#modified}}list-group-item-info{{/modified}}"
             >
-            <div class="row">
+            <div class="row-fluid ">
                 <div class="col-sm-4 col-md-2">
                     <div class="d-md-none">
                         <strong>{{#str}}headingcomponent, tool_customlang{{/str}}</strong>
@@ -129,8 +129,8 @@
                         <strong>{{#str}}headinglocal, tool_customlang{{/str}}</strong>
                     </div>
                     <div class="py-2 py-md-0 px-md-3">
-                        <label for="{{id}}" class="sr-only sr-only-focusable">{{{ component }}}/{{{ stringid }}}</label>
-                        <textarea class="form-control w-100 border-box" id="{{id}}" name="cust[{{id}}]" cols="40" rows="3">{{{ local }}}</textarea>
+                        <textarea class="form-control w-100 border-box" name="cust[{{id}}]" cols="40" rows="3">{{{ local }}}</textarea>
+
                         {{#checkupdated}}
                         <div class="uptodatewrapper">
                             <div class="form-check">
@@ -147,7 +147,7 @@
     {{/strings}}
     </div>
 
-    <fieldset class="m-3">
+    <fieldset class="m-a-1 m-3">
         <button type="submit" name="savecontinue" class="btn btn-secondary">
             {{#str}}savecontinue, tool_customlang{{/str}}
         </button>

admin/tool/customlang/tests/behat/customisation_create.feature

@@ -1,33 +0,0 @@
-@tool @tool_customlang
-Feature: Within a moodle instance, an administrator should be able to modify langstrings for the entire Moodle installation.
-  In order to change langstrings in the adminsettings of the instance,
-  As an admin
-  I need to be able to access and change values in the the language customisation of the language pack.
-
-  Background:
-    # This is a very slow running test and on slow databases can take minutes to complete.
-    Given I mark this test as slow setting a timeout factor of 4
-
-    And I log in as "admin"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    And I press "Open language pack for editing"
-    And I press "Continue"
-    And I set the field "Show strings of these components" to "moodle.php"
-    And I set the field "String identifier" to "administrationsite"
-    And I press "Show strings"
-    And I set the field "core/administrationsite" to "Custom string example"
-
-  @javascript
-  Scenario: Edit an string but don't save it to lang pack.
-    When I press "Apply changes and continue editing"
-    Then I should see "Site administration" in the "page-header" "region"
-    And I should not see "Custom string example" in the "page-header" "region"
-
-  @javascript
-  Scenario: Customize an string as admin and save it to lang pack.
-    Given I press "Save changes to the language pack"
-    And I should see "There are 1 modified strings."
-    When I click on "Continue" "button"
-    Then I should see "Custom string example" in the "page-header" "region"
-    And I should not see "Site administration" in the "page-header" "region"

admin/tool/customlang/tests/behat/export.feature

@@ -1,53 +0,0 @@
-@tool @tool_customlang
-Feature: Within a moodle instance, an administrator should be able to export modified langstrings.
-  In order to export modified langstrings in the adminsettings of the instance,
-  As an admin
-  I need to be able to export the php-files of the language customisation of a language.
-
-  Background:
-    # This is a very slow running feature and on slow databases can take minutes to complete.
-    Given I mark this test as slow setting a timeout factor of 4
-
-  @javascript
-  Scenario: Export button should not appear if no customization is made
-    Given I log in as "admin"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    Then I should see "Open language pack for editing"
-    And I should not see "Export custom strings"
-
-  @javascript
-  Scenario: Export button should not appear if no customization is saved into langpack
-    Given I log in as "admin"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    And I press "Open language pack for editing"
-    And I press "Continue"
-    And I set the field "Show strings of these components" to "moodle.php"
-    And I set the field "String identifier" to "accept"
-    And I press "Show strings"
-    And I set the field "core/accept" to "Accept-custom_export"
-    When I press "Apply changes and continue editing"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    Then I should see "Open language pack for editing"
-    And I should see "There are 1 modified strings."
-    And I should not see "Export custom strings"
-
-  @javascript
-  Scenario: Export the php-file including a customised langstring.
-    Given I log in as "admin"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    And I press "Open language pack for editing"
-    And I press "Continue"
-    And I set the field "Show strings of these components" to "moodle.php"
-    And I set the field "String identifier" to "accept"
-    And I press "Show strings"
-    And I set the field "core/accept" to "Accept-custom_export"
-    When I press "Save changes to the language pack"
-    And I should see "There are 1 modified strings."
-    And I click on "Continue" "button"
-    Then I set the field "lng" to "en"
-    And I click on "Export custom strings" "button"
-    And I set the field "Select component(s) to export" to "moodle.php"

admin/tool/customlang/tests/behat/import_files.feature

@@ -1,51 +0,0 @@
-@tool @tool_customlang @_file_upload
-Feature: Within a moodle instance, an administrator should be able to import modified langstrings.
-  In order to import modified langstrings in the adminsettings from one to another instance,
-  As an admin
-  I need to be able to import the zips and php files of the language customisation of a language.
-
-  Background:
-    # This is a very slow running test and on slow databases can take minutes to complete.
-    Given I mark this test as slow setting a timeout factor of 4
-
-    And I log in as "admin"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    And I click on "Import custom strings" "button"
-    And I press "Continue"
-
-  @javascript
-  Scenario: Import a PHP file to add a new core lang customization
-    When I upload "admin/tool/customlang/tests/fixtures/tool_customlang.php" file to "Language component(s)" filemanager
-    And I press "Import file"
-    Then I should see "String tool_customlang/pluginname updated successfully."
-    And I should see "String tool_customlang/nonexistentinvetedstring not found."
-    And I click on "Continue" "button"
-    And I should see "There are 1 modified strings."
-    And I click on "Save strings to language pack" "button"
-    And I click on "Continue" "button"
-    And I should see "An amazing import feature" in the "page-header" "region"
-
-  @javascript
-  Scenario: Try to import a PHP file from a non existent component
-    When I upload "admin/tool/customlang/tests/fixtures/mod_fakecomponent.php" file to "Language component(s)" filemanager
-    And I press "Import file"
-    Then I should see "Missing component mod_fakecomponent."
-
-  @javascript
-  Scenario: Import a zip file with some PHP files in it.
-    When I upload "admin/tool/customlang/tests/fixtures/customlang.zip" file to "Language component(s)" filemanager
-    And I press "Import file"
-    Then I should see "String core/administrationsite updated successfully."
-    And I should see "String core/language updated successfully."
-    And I should see "String core/nonexistentinvetedstring not found."
-    And I should see "String tool_customlang/pluginname updated successfully."
-    And I should see "String tool_customlang/nonexistentinvetedstring not found."
-    And I should see "Missing component mod_fakecomponent."
-    And I click on "Continue" "button"
-    And I should see "There are 3 modified strings."
-    And I click on "Save strings to language pack" "button"
-    And I click on "Continue" "button"
-    And I should see "Uploaded custom string" in the "page-header" "region"
-    And I should see "Another Uploaded string" in the "page-header" "region"
-    And I should see "An amazing import feature" in the "page-header" "region"

admin/tool/customlang/tests/behat/import_mode.feature

@@ -1,84 +0,0 @@
-@tool @tool_customlang @_file_upload
-Feature: Within a moodle instance, an administrator should be able to import langstrings with several modes.
-  In order to import modified langstrings in the adminsettings from one to another instance,
-  As an admin
-  I need to be able to import only some language customisation strings depending on some conditions.
-
-  Background:
-    # This is a very slow running feature and on slow databases can take minutes to complete.
-    Given I mark this test as slow setting a timeout factor of 4
-
-    # Add one customization.
-    And I log in as "admin"
-    And I navigate to "Language > Language customisation" in site administration
-    And I set the field "lng" to "en"
-    And I press "Open language pack for editing"
-    And I press "Continue"
-    And I set the field "Show strings of these components" to "moodle.php"
-    And I set the field "String identifier" to "administrationsite"
-    And I press "Show strings"
-    And I set the field "core/administrationsite" to "Custom string example"
-    And I press "Save changes to the language pack"
-    And I should see "There are 1 modified strings."
-    And I click on "Continue" "button"
-    And I should see "Custom string example" in the "page-header" "region"
-
-  @javascript
-  Scenario: Update only customized strings
-    When I set the field "lng" to "en"
-    And I click on "Import custom strings" "button"
-    And I press "Continue"
-    And I upload "admin/tool/customlang/tests/fixtures/moodle.php" file to "Language component(s)" filemanager
-    And I set the field "Import mode" to "Update only strings with local customisation"
-    And I press "Import file"
-    Then I should see "String core/administrationsite updated successfully."
-    And I should see "Ignoring string core/language because it is not customised."
-    And I should see "String core/nonexistentinvetedstring not found."
-    And I click on "Continue" "button"
-    And I should see "There are 1 modified strings."
-    And I should not see "Uploaded custom string" in the "page-header" "region"
-    And I click on "Save strings to language pack" "button"
-    And I click on "Continue" "button"
-    And I should not see "Custom string example" in the "page-header" "region"
-    And I should see "Uploaded custom string" in the "page-header" "region"
-    And I should not see "Another Uploaded string" in the "page-header" "region"
-
-  @javascript
-  Scenario: Create only new strings
-    When I set the field "lng" to "en"
-    And I click on "Import custom strings" "button"
-    And I press "Continue"
-    And I upload "admin/tool/customlang/tests/fixtures/moodle.php" file to "Language component(s)" filemanager
-    And I set the field "Import mode" to "Create only strings without local customisation"
-    And I press "Import file"
-    Then I should see "Ignoring string core/administrationsite because it is already defined."
-    And I should see "String core/language updated successfully."
-    And I should see "String core/nonexistentinvetedstring not found."
-    And I click on "Continue" "button"
-    And I should see "There are 1 modified strings."
-    And I should not see "Uploaded custom string" in the "page-header" "region"
-    And I click on "Save strings to language pack" "button"
-    And I click on "Continue" "button"
-    And I should see "Custom string example" in the "page-header" "region"
-    And I should not see "Uploaded custom string" in the "page-header" "region"
-    And I should see "Another Uploaded string" in the "page-header" "region"
-
-  @javascript
-  Scenario: Import all strings
-    When I set the field "lng" to "en"
-    And I click on "Import custom strings" "button"
-    And I press "Continue"
-    And I upload "admin/tool/customlang/tests/fixtures/moodle.php" file to "Language component(s)" filemanager
-    And I set the field "Import mode" to "Create or update all strings from the component(s)"
-    And I press "Import file"
-    Then I should see "String core/administrationsite updated successfully."
-    And I should see "String core/language updated successfully."
-    And I should see "String core/nonexistentinvetedstring not found."
-    And I click on "Continue" "button"
-    And I should see "There are 2 modified strings."
-    And I should not see "Uploaded custom string" in the "page-header" "region"
-    And I click on "Save strings to language pack" "button"
-    And I click on "Continue" "button"
-    And I should not see "Custom string example" in the "page-header" "region"
-    And I should see "Uploaded custom string" in the "page-header" "region"
-    And I should see "Another Uploaded string" in the "page-header" "region"

admin/tool/customlang/tests/fixtures/mod_fakecomponent.php

@@ -1,29 +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/>.
-
-/**
- * Local language pack.
- *
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-defined('MOODLE_INTERNAL') || die();
-
-
-$string['administrationsite'] = 'Uploaded custom string';
-$string['language'] = 'Another Uploaded string';
-$string['nonexistentinvetedstring'] = 'This should not be imported';

admin/tool/customlang/tests/fixtures/moodle.php

@@ -1,28 +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/>.
-
-/**
- * Local language pack.
- *
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-defined('MOODLE_INTERNAL') || die();
-
-$string['administrationsite'] = 'Uploaded custom string';
-$string['language'] = 'Another Uploaded string';
-$string['nonexistentinvetedstring'] = 'This should not be imported';

admin/tool/customlang/tests/fixtures/tool_customlang.php

@@ -1,28 +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/>.
-
-/**
- * Local language pack from http://localhost/m/MDL-69583
- *
- * @copyright  2020 Ferran Recio <ferran@moodle.com>
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-defined('MOODLE_INTERNAL') || die();
-
-
-$string['pluginname'] = 'An amazing import feature';
-$string['nonexistentinvetedstring'] = 'This string should not be imported';

admin/tool/customlang/tests/local/mlang/phpparser_test.php

@@ -1,207 +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/>.
-
-/**
- * PHP lang parser test.
- *
- * @package    tool_customlang
- * @copyright  2015 Damyon Wiese
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-
-namespace tool_customlang\local\mlang;
-
-use advanced_testcase;
-use moodle_exception;
-
-/**
- * PHP lang parser test class.
- *
- * @package    tool_customlang
- * @copyright  2015 Damyon Wiese
- * @license    http://www.gnu.org/copyleft/gpl.html GNU GPL v3 or later
- */
-class phpparser_testcase extends advanced_testcase {
-
-
-    /**
-     * Test get instance static method.
-     *
-     */
-    public function test_get_instance(): void {
-
-        $instance = phpparser::get_instance();
-
-        $this->assertInstanceOf('tool_customlang\local\mlang\phpparser', $instance);
-        $this->assertEquals($instance, phpparser::get_instance());
-    }
-
-    /**
-     * Test get instance parse method.
-     *
-     * @dataProvider parse_provider
-     * @param string $phpcode PHP code to test
-     * @param array $expected Expected result
-     * @param bool $exception if an exception is expected
-     */
-    public function test_parse(string $phpcode, array $expected, bool $exception): void {
-
-        $instance = phpparser::get_instance();
-
-        if ($exception) {
-            $this->expectException(moodle_exception::class);
-        }
-
-        $strings = $instance->parse($phpcode);
-
-        $this->assertEquals(count($expected), count($strings));
-        foreach ($strings as $key => $langstring) {
-            $this->assertEquals($expected[$key][0], $langstring->id);
-            $this->assertEquals($expected[$key][1], $langstring->text);
-        }
-    }
-
-    /**
-     * Data provider for the test_parse.
-     *
-     * @return  array
-     */
-    public function parse_provider() : array {
-        return [
-            'Invalid PHP code' => [
-                'No PHP code', [], false
-            ],
-            'No PHP open tag' => [
-                "\$string['example'] = 'text';\n", [], false
-            ],
-            'One string code' => [
-                "<?php \$string['example'] = 'text';\n", [['example', 'text']], false
-            ],
-            'Extra spaces' => [
-                "<?php \$string['example']   =   'text';\n", [['example', 'text']], false
-            ],
-            'Extra tabs' => [
-                "<?php \$string['example']\t=\t'text';\n", [['example', 'text']], false
-            ],
-            'Double quote string' => [
-                "<?php
-                    \$string['example'] = \"text\";
-                    \$string[\"example2\"] = 'text2';
-                    \$string[\"example3\"] = \"text3\";
-                ", [
-                    ['example', 'text'],
-                    ['example2', 'text2'],
-                    ['example3', 'text3'],
-                ], false
-            ],
-            'Multiple lines strings' => [
-                "<?php
-                    \$string['example'] = 'First line\nsecondline';
-                    \$string['example2'] = \"First line\nsecondline2\";
-                ", [
-                    ['example', "First line\nsecondline"],
-                    ['example2', "First line\nsecondline2"],
-                ], false
-            ],
-            'Two strings code' => [
-                "<?php
-                    \$string['example'] = 'text';
-                    \$string['example2'] = 'text2';
-                ", [
-                    ['example', 'text'],
-                    ['example2', 'text2'],
-                ], false
-            ],
-            'Scaped characters' => [
-                "<?php
-                    \$string['example'] = 'Thos are \\' quotes \" 1';
-                    \$string['example2'] = \"Thos are ' quotes \\\" 2\";
-                ", [
-                    ['example', "Thos are ' quotes \" 1"],
-                    ['example2', "Thos are ' quotes \" 2"],
-                ], false
-            ],
-            'PHP with single line comments' => [
-                "<?php
-                    // This is a comment.
-                    \$string['example'] = 'text';
-                    // This is another commment.
-                ", [
-                    ['example', 'text'],
-                ], false
-            ],
-            'PHP with block comments' => [
-                "<?php
-                    /* This is a block comment. */
-                    \$string['example'] = 'text';
-                    /* This is another
-                    block comment. */
-                ", [
-                    ['example', 'text'],
-                ], false
-            ],
-            'Wrong variable name' => [
-                "<?php
-                    \$stringwrong['example'] = 'text';
-                    \$wringstring['example'] = 'text';
-                ", [], false
-            ],
-            'Single line commented valid line' => [
-                "<?php
-                    // \$string['example'] = 'text';
-                ", [], false
-            ],
-            'Block commented valid line' => [
-                "<?php
-                    /*
-                    \$string['example'] = 'text';
-                    */
-                ", [], false
-            ],
-            'Syntax error 1 (double assignation)' => [
-                "<?php
-                    \$string['example'] = 'text' = 'wrong';
-                ", [], true
-            ],
-            'Syntax error 2 (no closing string)' => [
-                "<?php
-                    \$string['example'] = 'wrong;
-                ", [], true
-            ],
-            'Syntax error 3 (Array without key)' => [
-                "<?php
-                    \$string[] = 'wrong';
-                ", [], true
-            ],
-            'Syntax error 4 (Array not open)' => [
-                "<?php
-                    \$string'example'] = 'wrong';
-                ", [], true
-            ],
-            'Syntax error 5 (Array not closed)' => [
-                "<?php
-                    \$string['example' = 'wrong';
-                ", [], true
-            ],
-            'Syntax error 6 (Missing assignment)' => [
-                "<?php
-                    \$string['example'] 'wrong';
-                ", [], true
-            ],
-        ];
-    }
-
-}