weekly release 5.0.1+

NOBUG: Add upgrade notes

MDL-85721 qtype_multianswer: Update find_duplicated_subquestions for Oracle compatibility

MDL-84695 mod_qbank: Fix illegal whitespace

MDL-84695 mod_qbank: fix unit test for MS SQL server

MDL-85721 qtype_multianswer: Fix version bump

MDL-85689 mod_quiz: Behat for adding and deleting page from quiz

MDL-85939 qbank_columnsortorder: improve preview page table alignment.

MDL-84901 auth: specific session start error language string.

MDL-85697 mod_lesson: Behat for lesson group and user overrides

MDL-85697 mod_assign: Behat for assignment group and user overrides

MDL-85697 mod_quiz: Behat for quiz group and user overrides

MDL-84255 quiz: Show quiz title in heading in accessrule_seb layout.

MDL-85947 ci: Bump GHA to Ubuntu 24.04

Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>

MDL-85863 user: Defer new-password email until after transaction commit

MDL-85659 theme_boost: Fix editor generated tables styles

MDL-81702 core_message: Add focustrap and backdrop

* Add focustrap and backdrop to message drawer.

MDL-81702 theme_boost: Add focus lock to drawers

* Add focus lock to drawers for small screens

MDL-81702 theme_boost: Remove site home link from drawers

* The site-home-link was added to all menu drawers and should
only be added to the main menu drawer (mobile version)

MDL-84695 mod_qbank: More precise get_qbank_ids_of_type_in_course query.

weekly release 5.0.1+

NOBUG: Add upgrade notes

MDL-85037 block_html: correct access checks for plugin file serving.

MDL-41924 quiz: Annotate questions for backups

Find question bank entries for all questions referenced by a question
slot, and for all questions matching a question bank referenced used for
a random question, and record their IDs so they can be included in
backups.

MDL-41924 backup: Reduce questions included in backups

Historically, backups of an activity using questions would include all
questions for the entire context, where any question had been used. This
could lead to very large backup files, and exacerbated problems with
question duplication.

This changes the process for annotating categories to include in the
backup to find specific categories, rather than whole contexts. It
includes 3 groups of categories:
1. Those that belong to the backed up activity's context.
2. Those that contain a question used directly in a question_reference
belonging to the activity.
3. Those that contain a question used via a question_set_reference
belonging to the activity.
It also adds in any parent categories that are not included by the
above, so we have a complete hierarchy.

For group 1, the backup will include all questions belonging to these
categories, as before. For groups 2 and 3, it will only include
questions for question bank entries that have been annotated by the
activity.

MDL-85069 qbank_bulkmove: Load banks and categories dynamically

The bulk move question dialogue presents 2 autocomplete fields for
selection of the target bank and category to move questions to. This was
loading all banks and categories the user could access, and filtering
categories based on the selected bank.

On large sites, this was not usable as there were too many banks and
categories to load leading to timeouts.

This refactors the dialogue to load just the current bank and cagtegory
initially, then load other options on demand via AJAX.

MDL-85069 question: Services for question bank and category search

This updates the core_question_search_shared_banks external function
with additional parameters to make it applicable outside of the "Switch
bank" dialogue in the quiz, with the corresponding changes to the
question_banks_datasource Javascript module, and unit test coverage.

It also adds a new \core_question\output\question_category_selector
class for rendering a select list of question categories, migrating
the logic used in qbank_managecategories to a core namespace. There
is a fragment callback for rendering this via AJAX.

MDL-85666 phpunit: add general plugin tests

This defines new group which can be used
to limit execution of these tests to one plugin only.

For example to execute all tests for Label module use:

phpunit --testsuite=mod_label_testsuite
phpunit --group=plugin_checks --filter=mod_label

or

phpunit --filter=mod_label

Automatically generated installer lang files

MDL-85721 qtype_numerical: Prevent duplication on restore

If question_numerical record for a question has been deleted, when its
options are loaded its answers will not have a tolerance property. This
can lead to duplication.

Also, older questions may not have a question_numerical_options record.

To avoid this, we use 0 as a default tolerance for all answers if there
is no matching record, and do the same when reading the question
structure from backups. We also generate default values for the rest of
the options.

MDL-85721 qtype: Cope with missing options records

If we restore a question (or any other) which has had its
qtype_xxx_options record deleted, we get a notification output when we
try to build the options.

This may be called from an AJAX request (such as when we duplicate a
quiz), and outputting the notification breaks the AJAX response.
Returning false also means we don't get the answers attached to the
questiondata options, so the structure doesn't match the restored data,
and we get duplication.

This emits the errors via debugging instead, which allows it to be
supressed or logged, and allows get_question_options() to continue
running.

MDL-85721 qtype_multianswer: Clean up duplicate subquestions

This adds an ad-hoc task to clean up duplicate subquestions created by
the bug.

When the upgrade is run, it will run a query to check if any
subquestions have been duplicated (if there are multiple questions with
the same parent, text and stamp), and queue the cleanup task if there
are.

For each stamp identified, the task will find instances of the question
that are not referred to in their parent's sequence field, confirm that
they have 0 usages, and delete them.

MDL-85721 qtype_multichoice: Generate default options on restore

qtype_multichoice::get_question_options() will create a default options
object if no qtype_multichoice_options record exists. This means if we
restore a backup containing a multichoice question without any options
(see previous commit), it will always create a duplicate.

This change generates a default set of options for the backupdata if
none exists, so the identity hash will match if the same question exists
in the database without options, and we dont get duplicates.

MDL-85721 qtype_multianswer: Don't delete qtype data for subquestions

Historically, multianswer would re-use question IDs from subquestions,
so it was necessary to delete qtype-specific data if the qtype changed.
Since versioning was introduced, editing creates new versions of the
qtype and subquestions. Deleting the data means the original versions of
the subquestions cannot function correctly, so we shouldn't do that
anymore.

MDL-85069 question: Fix return URL parameters after moving questions

When calling the core_question_move_questions external function, the
returned $returnurl paramater kept the original `cat`, `category` and
`lastchanged` parameters.

If the question was moved to another question bank, this meant the user
would be redirected to that bank, but the URL would still refer to the
category in the original bank. Opening the bulk move dialogue again
would then show the original bank and category selected by default, not
the current one.

This change ensures the cat parameter is updated to match the new
category, unsets the `category` parameter as it is redundant. If a single
question is moved, "lastchanged" is updated to that question, otherwise
it is unset.

MDL-85069 qbank: Add tool_generator callback

This adds a callback for tool_generator and the required lang strings to
allow qbanks to be generated as part of test courses, by passing the
--additionalmodules=qbank argument to maketestcourse.php.

MDL-85563 phpunit: Clock API should always use Moodle system time

When using `clock::now()` the frozen cloxk and incrementing clock will now
always return a `DateTimeImmutable` that uses the configured Moodle system
time. In unit tests this will be Australia/Perth.

Before this change, it would sometimes be UCT.

MDL-85881 questions: Handle multi-byte names during migration

We were using strlen() instead of core_text::strlen() when checking the
length of a question bank name created during the migration. This meant
that multibyte strings were counted as being too long, even though they
were within the character limit.

MDL-85779 block_timeline: fix/restore sort toggle dropdown behaviour.

Changes during Bootstrap upgrade, in conjunction with both 8a8c8ab2 and
0c586b41, caused the layout to deviate from that expected elsewhere for
managing of Bootstrap dropdown elements. Thus the dynamically chosen
item was no longer being shown as selected. Fix that here.

weekly release 5.0.1+

MDL-85769 AI: Trap focus within AI drawer on smaller screens

Co-authored-by: Laurent David <laurent.david@moodle.com>

MDL-84487 mod_quiz: Use modal form for editing random questions

Creating and editing random questions were using 2 different forms - one
dynamic form in a modal, and the other a regular form on a standalone
page. The "Switch bank" UI only works in a modal, so while it was
displayed on the edit page, it didn't function.

This combines the editing functionality with the existing modal form, so
the same UI can be used for both operations, and the "Switch bank"
functionality works.

MDL-85665 mod_quiz: Behat to build quiz in single activity format course

MDL-83752 workshop: Handle empty feedbackreviewer

Causes PHP warnings when null on PHP 8.1.

Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>

MDL-85734 calendar: support multilang group names.

MDL-85466 course: space between number and course icon.

weekly release 5.0.1+

MDL-85210 mod_qbank: Update question set references during upgrade

When moving question categories during the migration to mod_qbank, set
references using questions in those categories were not updated with the
new context.

Furthermore, since top categories from system, category and course
contexts are deleted and not moved, and set references that filtered
based on a top category were left pointing to a non-existant category.

This change updates set references using the top category to point to
the new top category where its subcategories are moved, and updates set
references for all subcategories to set the new context.

MDL-85210 question: Update move_question_set_references

move_question_set_references expected the set reference to use the old
filter format.

This change converts the filter to the new format if required, before
updating the category and context ids.

MDL-75947 ltiservice_gradebookservices: Fix unit test

Fixes the test_sequential_score_posts() unit test in lineitem_test
by removing dynamic class property assignments, which were triggering
deprecation notices.

MDL-76414 navigation: Show 'Calendar' link in primary nav to guests

MDL-76342 core_output: Wrap buttons below heading on small screens

MDL-85514 core_form: support 'in' rule for multiselects via subset of

The 'in' rule currently behaves the same way that 'eq' does when used
with multi-selects, which is not correct. It should be modelled by
checking whether the selected values are a subset of the rule values
(i.e. selected values are 'in' the rule values). This patch implements
that for core form rules as well as for the admin settings show/hide.
Several behat features are updated as well, because these were verifying
the incorrect behaviour for multi-select in rules.

MDL-85773 forms: Read-only selects must have static labels

This is to prevent labels with an invalid for attribute, referencing non-existing elements.

MDL-85742 block_timeline: Fix active item indicator spacing

MDL-85678 mod_feedback: a11y Behat coverage for Manage templates page

MDL-85678 mod_feedback: Fix a11y issues in the Manage templates page

- Fix the heading hierarchy.
- Add a visually-hidden column header for the actions column.

MDL-75947 ltiservice_gradebookservices: score->timestamp as dategraded

If this is not passed in to grade_update(), then it's not passed on to
update_raw_grade(), resulting in time() being used instead, which isn't
correct. The grade timemodified value should be set to the timestamp
provided by the tool in the score post JSON, not the time of the score
post itself.

Co-authored-by: Jayce Birrell <jayce.birrell@moodle.com>

MDL-75947 ltiservice_gradebookservices: test sequential score posting

This tests highlights a problem with the score handling logic and
will currently fail. On score post, the current time is used for
grade->timemodified instead of the score->timestamp provided by the
tool in the POST JSON.

Automatically generated installer lang files

weekly release 5.0.1+

NOBUG: Add upgrade notes

MDL-84531 core: default deprecated attribute `$replacement` to null.

MDL-84752 qtype_essay: create accessible min/max words form elements.

Because of historical requirements involving fieldsets in question
type forms (see 958e7671), we should use ARIA label attribute on
grouped elements rather than normal labels in order to make them
accessible.

Co-authored-by: Jun Pataleta <jun@moodle.com>

MDL-72123 message: Add upgrade note

MDL-85700 theme_boost: Fix singleselect spacing

- Fix single_select template adding gap between elements.

MDL-82797 question engine: gift import fix

MDL-84611 blocks: Respect instance_allow_multiple()

MDL-84611 block_html: Change tests to respect capabilities

MDL-85274 calendar: Duration limits added to forms

MDL-85316 admin: Add environment check for AWS Aurora database

MDL-85350 questions: skip restore test for certain 3rd party plugins

The test_restore_course_with_same_stamp_questions() test assumes
that all question types using answers will also use the table
"question_answers" to store said answers. However, some 3rd party
qtypes might use custom tables and for those, the test would fail.

Moodle release 5.0.1

NOBUG: Add upgrade notes

MDL-84976 tool_policy: noclean param was not set

MDL-72123 message: Hide add to contacts button if user not allowed

MDL-85683 enrol: use formatted course name for notification context.

weekly release 5.0+

NOBUG: Add upgrade notes

MDL-83762 core_files: Bind resolve IPs and ports to cURL calls

MDL-84518 course: improved course visibility state handling

MDL-80380 core_auth: Add upgrade note

MDL-80380 admin: Support Microsoft XOAUTH2 for outgoing email

MDL-85488 mathjax: refactor loader to enhance configuration and loading

MDL-85323 core: Don't cache the login page

MDL-85375 libraries: Escaping a query parameter on pg_insert_id()

MDL-84706 bigbluebuttonbn: improved authorization and mitigate CSRF risks

MDL-79993 course: Fix PHPUnit failure

MDL-79993 course: Improve user access to recent courses service

MDL-84497 badges: fix missing action sesskey checks.

MDL-85645 mod_quiz: Behat to display questionbank name of quiz question.

MDL-85239 mod_quiz: Use visually-hidden instead of deprecated sr-only

MDL-84804 output: amend page title when navigating dynamic tabs.

Prepend the current tab name to the page title, to ensure we can
easily identify the current page/tab and that it is both unique
and accessible.

MDL-84804 javascript: new module for manipulating the page title.

MDL-85041 core: Make methods calling `announceChanges` async as well

MDL-85041 core: Improve autocomplete/tag a11y for assistive tech

- Updated autocomplete behavior to announce dynamic changes
via ARIA live regions
- Fixed focus management for keyboard and screen reader navigation
- Added proper role/aria tags for interactive tag components

MDL-85200 mod_quiz: Improve efficience of populate_question_sources

This uses the contexts from the categories of questions in the quiz to
narrow down the list of course modules we are querying to just those we
will need.

MDL-85200 mod_quiz: Expand behat test to cover shared bank display

This extends the existing behat test for adding a question from a shared
bank to confirm that the source bank is displayed with the question on
the Questions page.

The bank name is already passed through format_string() before being
passed to the template, so additional encoding by the template is not
necessary, hence the template has been changed to use triple braces.

MDL-85571 mod_assign: Improve PHPUnit tests

MDL-85571 mod_assign: Add visibility check to due digest notification