Merge branch 'MDL-85817-500' of https://github.com/davewoloszyn/moodle into MOODLE_500_STABLE

weekly release 5.0.1+

NOBUG: Add upgrade notes

Merge branch 'MDL-85863-500' of https://github.com/yusufwib01/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85689-500' of https://github.com/andelacruz/moodle into MOODLE_500_STABLE

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

Merge branch 'MDL-85262-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-84901-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85939-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-84255-500' of https://github.com/lucaboesch/moodle into MOODLE_500_STABLE

MDL-85721 qtype_multianswer: Fix version bump

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

Merge branch 'MDL-85721_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85697-500' of https://github.com/andelacruz/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85779-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

MDL-85939 qbank_columnsortorder: improve preview page table alignment.

Merge branch 'MDL-85659-500' of https://github.com/roland04/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85069_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

Merge branch 'MDL-84695-500' of https://github.com/lucaboesch/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85881_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

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

Merge branch 'MDL-84976-500' of https://github.com/rjnl/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85037-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85947-M500_bump-GHA-ubuntu-24-04' of https://github.com/ziegenberg/moodle into MOODLE_500_STABLE

Merge branch 'MDL-41924_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

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-85817 mod_assign: Update assignment SMS strings

MDL-84695 mod_qbank: More precise get_qbank_ids_of_type_in_course query.

weekly release 5.0.1+

NOBUG: Add upgrade notes

Merge branch 'install_500_STABLE' of https://git.in.moodle.com/amosbot/moodle-install into MOODLE_500_STABLE

Merge branch 'MDL-85350_500' of https://github.com/PhilippImhof/moodle into MOODLE_500_STABLE

Merge branch 'MDL-84487_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85200_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85563-m500' of https://github.com/NeillM/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85210_500_STABLE' of https://github.com/marxjohnson/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85432-500' of https://github.com/sh-csg/moodle into MOODLE_500_STABLE

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.

Merge branch 'MDL-76342-500' of https://github.com/yusufwib01/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85769-500' of https://github.com/meirzamoodle/moodle into MOODLE_500_STABLE

Merge branch 'patch/MDL-85666-500' of https://github.com/skodak/moodle into MOODLE_500_STABLE

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.

Merge branch 'MDL-84531-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

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.

Merge branch 'MDL-85665-500' of https://github.com/andelacruz/moodle into MOODLE_500_STABLE

Merge branch 'MDL-83752-M500' of https://github.com/ziegenberg/moodle into MOODLE_500_STABLE

Merge branch 'MDL-76414-500' of https://github.com/yusufwib01/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85734-500' of https://github.com/lucaboesch/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85466-500' of https://github.com/lucaboesch/moodle into MOODLE_500_STABLE

Merge branch 'MDL-82109-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

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>

Merge branch 'MDL-85683-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85644-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85407-50' of https://github.com/durenadev/moodle into MOODLE_500_STABLE

MDL-85734 calendar: support multilang group names.

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

weekly release 5.0.1+

Merge branch 'install_500_STABLE' of https://git.in.moodle.com/amosbot/moodle-install into MOODLE_500_STABLE

Merge branch 'MDL-75947-integration-500-fix' of https://github.com/mihailges/moodle into MOODLE_500_STABLE

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

Merge branch 'MDL-85223-500' of https://github.com/paulholden/moodle into MOODLE_500_STABLE

Merge branch 'MDL-75947-500' of https://github.com/Jayce0808/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85678-500' of https://github.com/junpataleta/moodle into MOODLE_500_STABLE

Merge branch 'MDL-83487-500' of https://github.com/muhammadarnaldo/moodle into MOODLE_500_STABLE

Merge branch 'MDL-85159-500' of https://github.com/stevandoMoodle/moodle into MOODLE_500_STABLE

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

Merge branch 'MDL-85514-500' of https://github.com/snake/moodle into MOODLE_500_STABLE

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.