weekly release 4.0.1+

MDL-73212 gha: Allow parameters to be passed via secret

We are disabling the -v (verbose) option, used by default. Still,
now we support a secret (that can be added to everyone's repository)
to accept any PHPUnit's command line options and run the tests with
them.

Some examples:

phpunit_options = -v (to keep the verbose option enabled)
phpunit_options = --testdox (to print information about every test...)

... (basically anything supported by the CLI)

Also, we are raising here the environment from Ubuntu 18.04 to 20.04

(self tested by GHA, heh)

Automatically generated installer lang files

MDL-74715 reportbuilder: switch event for adding filters/conditions.

Switch the select element event listener to that provided by custom
events helper, for adding report filters and conditions. This improves
cross-platform support.

MDL-69939 webservice: correct AJAX bool value in services definition.

MDL-74358 availability: Avoid php8 problem with array_values(null)

weekly release 4.0.1+

MDL-72324 core: Add upgrade.txt note about $plugin->incompatible

MDL-73156 user: Custom user profile field names now filtered.

MDL-74694 reportbuilder: unit test coverage of external exporters.

MDL-74713 admin: format custom profile fields in identify config.

MDL-72555 block_site_main_menu: Fix activity chooser issue

* When adding the block_site_main_menu on every page, the activity chooser generated
for this entire page is set with courseid = 1. So any activity is then added to the
home page instead of a course.

MDL-74597 themes: remove pix icon references from custom menu items.

Support for them was removed in 8c336899, however existing config
remained that would lead to bogus URLs in custom user menu items.

MDL-74565 gradereport_grader: define capability required for editing.

MDL-74526 theme: remove icon from empty categories in RTL languages

MDL-74449 gradebook: Protect flatten_dependencies_array() a little bit

It has been detected that the flatten_dependencies_array() was fragile
and leading to wrong results when some incorrect data was passed to it.

This includes:

- Missing elements.
- Null dependencies.
- Non array dependencies.

While the existing behaviour (testing-wise) has been preserved, now the
situations above are better controlled and the function ignores all
those incorrect cases that shouldn't happen ever.

That implies that a good number of notices/warnings/errors aren't
happening anymore. That was impacting both results (when the problems
were  only notices and warnings) and execution (when the problems
were errors).

Covered with tests.

MDL-74296 mod_bigbluebuttonbn: Fix completion indicator

* Completion view should work out of the box
* Patch the _user_outline and _user_complete so it works with completion view, while
implementing something better in MDL-74468

weekly release 4.0.1+

MDL-69756 nextcloud: copy-overwrite handling

MDL-74802 core: Remove tooltip on the notification close button

Reverting the addition of a Boostrap tooltip on the alert notification's
close button. It introduced a bug where the tooltip remains after
dismissing the notification alert. We can manually toggle the state of
the tooltip via JS, but I don't think it's worth the effort. The tooltip
is meant as an added bonus for sighted users to see what the close
button is about.

MDL-74802 core_user: Use button label to close notification alert

With the notification alert's close button now properly labelled, it is
better for Behat to interact on it using its label.

MDL-73013 filelib: Fix code lint

MDL-73013 filelib: Address coding style violations

MDL-74649 course: render scale information using page popup layout.

MDL-74718 webdav: correct the loop when reading chunks

Adjusted the condition while reading the streams
in chunks. Read till the end to make sure we have got all
the data. Else we might not get all the data to show
in file picker.

MDL-73466 forumreport_summary: use formatted course module name.

MDL-74182 core_task: Obey CFG debug settings

MDL-72324 upgrade: Fix incompatible check in upgrade to match docs

Automatically generated installer lang files

MDL-74628 forumreport_summary: remove left margin of filter popover.

MDL-74529 theme_boost: accessible autolink styles from filters.

The `autolink` class is used by various filter types to create
hyperlinks within text. Extend existing `aalink` class so they
behave in the same way.

MDL-73774 timeline: Prevent ID duplication

Automatically generated installer lang files

MDL-74871 user: account for absense of identity fields during export.

MDL-72983 tool_dataprivacy: respect capability to export own data.

MDL-72377 grade: fix category form hidden/until fields.

Ensure the saved values for the hidden and hidden until fields are
populated when editing a grade category. Fix behaviour on enabling
the hidden until field.

Re-enable tests commented out in eba1d32c.

MDL-72377 grade: increase size of category hidden fields.

As well as storing 1/0 to indicate hidden state, they are also used
to store a timestamp indicating hidden until date. Increasing field
size allows these values to be stored without triggering exceptions.

MDL-74573 mod_assign: convert view other attempt link to a button

MDL-70223 h5p: fix invalid id when adding h5p activity

MDL-69970 icons: flip help icon direction in RTL languages

weekly release 4.0.1+

Automatically generated installer lang files

MDL-71301 qtype_gapselect: Make blank answer box labels more meaningful

MDL-67966 mod_workshop: Use condition to verify form is using editor

This adds a condition to check if Online text is used
for the submission. The functions file_postupdate_standard_editor
and file_postupdate_standard_filemanager needs the property
content_editor of the formdata.

MDL-74752 qtype_match: fix regrading logic - take 2

MDL-74752 qtype_multianswer: implement regrading hooks

MDL-74752 qtypes gapselect & ddwtos: implement regrading hooks

MDL-74752 qtype_match: implement regrading hooks

As much as they can be before MDL-5560 is addressed.

MDL-74752 qtype_multichoice: fix regrading logic

The implements the new regrade-related hooks, and also has
tests for the changes to the core system, now that we have a question
type we can use for them.

MDL-74752 question regrading: implement the required hooks

This commit implements the necessary core hooks to ensure we only
allow a regrade of a quetion attempt to take place if the new and old
versions of the question are sufficiently similar.

It will be followed by commits to each question type where the
new method needs to be implemented.

Automated tests will be included in the first of those (mulitple choice)
becuse we need a question type that implements the hooks to test
the core changes.

MDL-74630 core_contenbank: Add contextid to URL parameters

Automatically generated installer lang files

Automatically generated installer lang files

Automatically generated installer lang files

MDL-74250 core: Improve indicator that link opens in new window

MDL-74764 core_course: Activity is not in alphabetical order.

weekly release 4.0.1+

MDL-74310 mod_lti: fix ltilinkid on lineitems import

MDL-74816 mod_resource: Allow specify default file name in generator

MDL-74790 mod_resource: better detection of empty module intro.

MDL-74533 core_completion: Update using new grade pass for completion

MDL-74802 core: Improve the accessibility of alert notifications

* Make the ordering of the message and the close button logical on
  the DOM.
* Hide the times HTML entity
* Add sr-only label on the close button for non-sighted users
* Add tooltip on the close button sighted users

MDL-73386 mod_quiz: Use plain text for quiz navigation fake block title

* No need to enclose the title of the quiz navigation fake block in
a span tag. The block title also gets appended to the skip link which
means that a block title with a span tag assigned with an identifier
can cause duplicate ID errors on the DOM.

MDL-73225 mod_bigbluebuttonbn: Improve error message when joining

    Display more friendly messages when:
    * When displaying activity and the server is not available
    * When we join a meeting and the server is not available or an error is detected

MDL-74577 core: Reset login_failed_count_since_success after login

When the user is logged in successfully then reset the value
of login_failed_count_since_success to zero, if the value of
login_failed_count_since_success is greater than zero.

weekly release 4.0.1+

MDL-73801 backup: set predictably ordered backup next start time.

Prior to this change, the testcase would assert ordering based on
identical fields which could lead to random failures. Set distinct
next start time of course backup, and also remove random ordering
by sorting on defined course order (all other fields being equal).

MDL-74377 reportbuilder: Fix dimmed text for dropdown items

To show inactive/unavailable rows '.dimmed_text' class usages have been replaced with '.text-muted'.

MDL-73966 grade: fetch required user name fields of unenrolled users.

Automatically generated installer lang files

MDL-71276 message_email: include alternate email in privacy export.

MDL-74678 reportbuilder: Allow null in category description callback

Since category description is a nullable column, we need to allow null
value in category description column callback in category entity

MDL-74344 comment: consistent capability check for accessing report.

Co-authored-by: David Wipperfurth <dwipperfurth@webcourseworks.com>

Automatically generated installer lang files

MDL-73013 calendar: fix serving of files added to category events.

weekly release 4.0.1+

MDL-69951 filter_tidy: Replace drop-font-tags

The option drop-font-tags was removed in HTML Tidy version 5.5.14.  It
had been deprecated since 5.1.25, the docs said "Use the clean option
instead" which is what this commit does (see
https://api.html-tidy.org/tidy/quickref_5.1.25.html#drop-font-tags).

MDL-73163 behat: Changed most to must

MDL-74416 phpunit: fix generator counters reset

MDL-74699 enrol_lti: fix clientid retrieval from aud during launches

MDL-74553 blocks: do not refer to form fields by id

This is not compatible with the dynamic forms and random ids

MDL-74726 mod_bigbluebuttonbn:  Fix muteonstart

Added missing muteonstart_editable in mod_form  block condition

MDL-73898 user: exclude guest user from participants table.

MDL-74710 mod_bigbluebuttonbn: Fix settings url.

MDL-74413 phpunit: Move more tests to use correct names and namespaces

Applied the following changes to various testcase classes:

- Namespaced with component[\level2-API]
- Moved to level2-API subdirectory when required.
- Fixed incorrect use statements with leading backslash.
- Remove file phpdoc block
- Remove MOODLE_INTERNAL if not needed.
- Changed code to point to global scope when needed.
- Fix some relative paths and comments here and there.
- All them passing individually.
- Complete runs passing too.

Special mention to:

- Moved to the level2 "privacy" namespace:
  - \mod_assign\privacy\feedback_legacy_polyfill_test
  - \mod_assign\privacy\submission_legacy_polyfill_test

- Moved to the level2 "task" namespace:
  - \core_message\task\migrate_message_data_test
  - \ltiservice_gradebookservices\task\cleanup_test
  - \message_email\task\send_email_test
  - \mod_lti\task\clean_access_tokens_test
  - \mod_workshop\task\cron_task_test

- Moved to the level2 "event" namespace:
  - \core_h5p\event\deleted_test
  - \core_h5p\event\viewed_test

- Renamed to a better name:
  - backup_forum_activity_task_test.php (missing "task")

Moodle release 4.0.1

MDL-73971 phpunit: Move more tests to use correct names and namespaces

Applied the following changes to various testcase classes:

- Namespaced with component[\level2-API]
- Moved to level2-API subdirectory when required.
- Fixed incorrect use statements with leading backslash.
- Remove file phpdoc block
- Remove MOODLE_INTERNAL if not needed.
- Changed code to point to global scope when needed.
- Fix some relative paths and comments here and there.
- All them passing individually.
- Complete runs passing too.

Special mention to:

- Some fixtures, initially defined in the test files have been
  moved to new files in fixtures subdirectory, leaving the unit
  test files clearer:
  - moodle2_course_format_test.php
- Rename wrong named test:
  - baseoptiogroup_test = baseoptigroup_test

weekly release 4.0+

MDL-57065 theme_boost: fix debug messages

The first debug message is covered by the top navbar header. This patch
will move the body's first debug message down below the navbar.

MDL-74602 mod_assign: fix js error with assign grader on classic

MDL-74584 mod_bigbluebuttonbn: Update behat tests

MDL-74584 mod_bigbluebuttonbn: Update unit tests

MDL-74584 course: New optional parameter in get_module_types_names()

Adds new parameter in get_module_types_names() to optionally reset
the $modnames static cache.

MDL-74584 mod_bigbluebuttonbn: Add install script and new upgrade step

Adds a new install script for the BBB module and a new upgrade step.
The BBB module now will be disabled by default for new installations
and also it will be disabled for the existing sites that use the
default BBB server configuration. Admistrators will be required to
confirm the acceptance of the related DPA prior to (re)enabling the
plugin once again.

MDL-74584 mod_bigbluebuttonbn: Add DPA info in the general setting page

Adds a note in the BBB general settings page to raise awareness about
the legal obligations related to the data processing agreement with the
service provider which is currently configured.

MDL-74584 mod_bigbluebuttonbn: Implement 'pre_enable_plugin_actions'

Implements a function for the 'pre_enable_plugin_actions' callback.
When the default BBB server settings are used while the administrator
attempts to enable the BBB plugin, the function will prevent this
action and trigger a dynamic form requesting the admin to conifrm
their acceptance of the related data processing agreement. The process
of enabling the plugin will continue as usual in cases where the
admin has already confirmed the acceptance of the dpa or a different
BBB server configuration is used other than the default.

MDL-74584 admin: Add callback executed prior to enabling a module

Adds a callback xxx_pre_enable_plugin_actions in admin/modules.php
which plugins can use to force additional actions before enabling the
plugin. The return value (bool) from the plugin callback method
specifies whether the process of enabling the plugin should continue
after the added actions or not.

MDL-74511 core: Add phpcs default config

MDL-41498 course: courses participant page

MDL-74567 core: re-factor flickr libraries to use shared user-agent.