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

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

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

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

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

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

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

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-83896 mod_lesson: Add course overview integration

- Implement course overview integration
- Redirect index.php to course overview

MDL-83896 mod_lesson: Add new methods to lesson class

Add new methods to the lesson class that will be needed by
the overview page.

Merge branch 'MDL-76342-main' of https://github.com/yusufwib01/moodle

Merge branch 'MDL-85769-main' of https://github.com/meirzamoodle/moodle

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

Merge branch 'MDL-85725' of https://github.com/paulholden/moodle into main

MDL-83896 mod_lesson: Add generators for lesson submissions

MDL-83896 mod_lesson: Update strings to improve consistency

Update how the word ‘attempts’ is used to  make the terminology more consistent
within Lesson and better aligned with how ‘attempts’ is used in other activities.

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.

Merge branch 'MDL-83891-main' of https://github.com/aanabit/moodle

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' of https://github.com/paulholden/moodle

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

MDL-84795 reportbuilder: consistent/performant loading of context data.

Ensure we select only context related fields as provided by the helper,
and avoid empty joins by loading instances with the helper-specified ID
field `{$tablealias}.ctxid`.

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-85880 reportbuilder: replace jQuery usage for parsing Javascript.

Merge branch 'MDL-85533-main-v03' of https://github.com/ferranrecio/moodle

MDL-85533 course: activity chooser full refactor

MDL-85533 course: remove activitychoosertabmode setting

Merge branch 'MDL-79319-main' of https://github.com/laurentdavid/moodle

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.1dev

NOBUG: Add upgrade notes

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

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

MDL-85855 calendar: format course shortname attribute correctly.

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-83891 mod_data: Redirect index.php to activities overview page

MDL-83891 mod_data: Add overview page

MDL-85825 behat: Exclude fieldsets

Fieldsets were getting incorrectly picked up from the 'form_row' selector
which was causing the i_expand_the_autocomplete() function to target the wrong element

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

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

Merge branch 'MDL-83365-main' of https://github.com/andrewnicols/moodle

MDL-83365 upgradenotes: Add release notes link generator

Merge branch 'MDL-76414-main' of https://github.com/yusufwib01/moodle

Merge branch 'MDL-85734-main' of https://github.com/lucaboesch/moodle

Merge branch 'MDL-85466-main' of https://github.com/lucaboesch/moodle

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

MDL-83895 mod_h5pactivity: Fix Overview column name

MDL-60799 enrol: update_instance() needs separate objects passed

Passing the same object means enrol_cohort does not recognise a change
in role when an edit or batch course upload operation is done.

 * enrol/editinstance.php was modifying $instance before passing it to
   enrol_plugin::update_instance() so the latter never sees a difference
   versus $data. This is redundant since update_instance() is doing the
   same thing a second time.
 * admin/tool/uploadcourse/classes/course.php is passing the same object
   as both arguments to enrol_plugin::update_instance().

There is an expectation that update_enrol_plugin_data() is to mutate its
$instance argument and then return it based on how derived classes in
enrol_guest and enrol_self behave, so cloning $instance within
update_enrol_plugin_data() before mutating it isn't an option.

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' of https://github.com/paulholden/moodle

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

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

Merge branch 'MDL-83895-main' of https://github.com/aanabit/moodle

Merge branch 'MDL-85436-main' of https://github.com/sarjona/moodle

MDL-83891 mod_data: Add approved option to entries generator

MDL-85734 calendar: support multilang group names.

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

MDL-85436 lib: Deprecate unused icons

MDL-85825 core_form: enable hideIf() on header element

weekly release 5.1dev

NOBUG: Add upgrade notes

Merge branch 'install_main' of https://git.in.moodle.com/amosbot/moodle-install

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

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-85307 filter_glossary: Do not filter concept

Filtering the concept here causes a recursive loop, disabling it here will prevent this.

Merge branch 'MDL-85399' of https://github.com/paulholden/moodle

Merge branch 'MDL-85120' of https://github.com/paulholden/moodle

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

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

Merge branch 'MDL-79052_final-deprecation-of-removing-the-legacy-theme-settings' of https://github.com/ziegenberg/moodle

Merge branch 'MDL-78375_final-deprecation-of-device-related-theme-functions' of https://github.com/ziegenberg/moodle

Merge branch 'MDL-85678-main' of https://github.com/junpataleta/moodle

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

Merge branch 'MDL-85159-main' of https://github.com/stevandoMoodle/moodle

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

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.

Merge branch 'MDL-83904-master' of https://github.com/ngtrdai/moodle

Merge branch 'MDL-85463-main' of https://github.com/sarjona/moodle

MDL-79052 core: Final deprecation of removing the legacy theme settings

MDL-78375 core: Final deprecation of device related theme functions

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

Merge branch 'MDL-85158' of https://github.com/pedrojordao/moodle into main

Merge branch 'MDL-42263-main' of https://github.com/Jayce0808/moodle into main

MDL-73003 core_grades: Don't use 'scale' to determine grading method.

When a forum activity used a rubrics with a scale for grading, it was not treated as an advanced grading method, causing
the code to mishandle the grading object and not return any grades.

MDL-83904 badges: Add namespace to badges/classes/observer.php

Merge branch 'MDL-85742-main' of https://github.com/lucaboesch/moodle