MDL-80838 core: Add PSR-20/Clock support

This commit adds the PSR-20 ClockInterface to core, with a
moodle-specific extension to the Interface at `\core\clock`, and a
standard clock at `\core\system_clock`.

Further clocks are provided as `\incrementing_clock` and `\frozen_clock`
which are available to unit tests using:

- `$this->mock_clock_with_incrementing(?int $starttime = null);`
- `$this->mock_clock_with_frozen(?int $time = null);`

For the incrementing clock, every call to fetch the time will bump the
current time by one second.

For the frozen clock the time will not change, but can be modified with:
- `$clock->set_to(int $time);`; and
- `$clock->bump(int $seconds = 1);`

on-demand release 4.4dev+

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

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

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

Merge branch 'MDL-78551-main' of https://github.com/safatshahin/moodle

MDL-80384 courseformat: Set activity icon alt text on editing mode

From a warning using WAVE:
* The title attribute value for images that lack an alt attribute value
will be presented to screen reader users. However, providing image
content in the alt attribute typically provides better accessibility,
and should be used in most cases. The title attribute will generate a
mouse hover tooltip which may or may not be desired - this tooltip will
not be presented to touch screen or keyboard users.

MDL-78551 core: Version bump

MDL-78551 core: Register hook callbacks and dispatchers

MDL-78551 core_communication: Add hook listeners

MDL-78551 core_access: Add support for hooks api

MDL-78551 core_enrol: Add support for hooks api

MDL-78551 core_course: Add hooks api for course updates

MDL-78551 core_user: Add hooks api for user updates

MDL-78551 core_group: Add hooks api for group and membership

Merge branch 'MDL-81266' of https://github.com/stronk7/moodle

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

Merge branch 'MDL-81246-main' of https://github.com/HuongNV13/moodle

Merge branch 'MDL-78885-main' of https://github.com/rezaies/moodle

MDL-81266 phpunit: Document the changes in lib/upgrade.txt

MDL-81266 phpunit: Remove expect(Deprecation|Notice|Warning|Error)

PHPUnit 9.6 has deprecated all the expect(Deprecation|Notice|Warning|Error)
assertions, so we have to move away from them.

In core, we only had 2 cases, one easily fixed by getting rid of it,
because, for php >= 80 it's an assert-able exception.

And the other replaced with code that, temporarily, sets a custom
error handler that converts any specified E_ to an asset-able
exception.

Note that tests playing with error handlers should, always, be
run in separate process, to avoid problems or conflicts with
PHPUnit / Moodle own error handlers.

MDL-81281 phpunit: assertObjectHasAttribute is deprecated

To be integrated as part of MDL-81266

When running PHPUnit 9.6 we get the following deprecation warnings:
"assertObjectHasAttribute() is deprecated and will be removed in PHPUnit
10. Refactor your test to use assertObjectHasProperty() instead."

So we replace all instances of assertObjectHasAttribute with
assertObjectHasProperty.

PHPUnit justifies the change with:
> PHPUnit currently refers to "fields" (see above) as "attributes". This
> is (or will become) confusing considering the introduction of
> attributes in PHP 8 and their support in PHPUnit.  PHPUnit will be
> changed to use the term "property" instead of "attribute" where "field"
> is meant.

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

MDL-81281 phpunit: assertObjectNotHasAttribute is deprecated

To be integrated as part of MDL-81266

When running PHPUnit 9.6 we get the following deprecation warnings:
"assertObjectNotHasAttribute() is deprecated and will be removed in PHPUnit
10. Refactor your test to use assertObjectNotHasProperty() instead."

So we replace all instances of assertObjectNotHasAttribute with
assertObjectNotHasProperty.

PHPUnit justifies the change with:
> PHPUnit currently refers to "fields" (see above) as "attributes". This
> is (or will become) confusing considering the introduction of
> attributes in PHP 8 and their support in PHPUnit.  PHPUnit will be
> changed to use the term "property" instead of "attribute" where "field"
> is meant.

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

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

MDL-81266 composer: PHPUnit 9.6, Behat to 3.14 and other deps

Bump to PHPUnit 9.6 towards warning in advance about
stuff that will be removed for PHPUnit 10, namely:
 - expect[Deprecation|Error|Notice|Warning] methods.
 - assert[Object|Class][Not]Has[Static]Attribute methods.
 - withConsecutive()
 - TestCase::getMockClass()
 - "Test" suffix for abstract test case classes.

The behat bump is only applied to 402_STABLE and up, all them
requiring PHP 8.0 as a minimum. And generated following the
instructions in the dev docs.

MDL-80384 core_courseformat: Remove activty icon link

* Remove the link over the image icon for an activity and use
the title attribute for the tooltip

MDL-81277 lib: Update atto codemirror to 5.65.16

Merge branch 'MDL-79368-main' of https://github.com/roland04/moodle

MDL-80875 uploadcourse: Fix params validation

MDL-80300 mod_quiz: Integration coding fixups

Merge branch 'mdl-80300-main-refactor' of https://github.com/matthewhilton/moodle

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

MDL-63128 core_cache: Add new is_cluster_available helper

The ::is_cluster_available() will return true if the PHP support redis cluster.

MDL-63128 core: Add support for use redis cluster as session handler

Co-authored-by: Avi Levy <avi@sysbind.co.il>

MDL-63128 cachestore_redis: Support Redis cluster as cache store

Co-authored-by: Daniel Thee Roperto <daniel@theeroperto.com>
Co-authored-by: Avi Levy <avi@sysbind.co.il>

Merge branch 'MDL-79239-master' of https://github.com/ssj365/moodle

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

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

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

MDL-81074 core: Simplify strftime tests

MDL-81074 core: align core_date::strftime results for numeric and string

MDL-81074 core: Add assertTimeStringMatches PHPUnit Assertion

Also raised as https://github.com/sebastianbergmann/phpunit/issues/5757

on-demand release 4.4dev+

NOBUG: Fixed file access permissions

NOBUG: Fixed SVG browser compatibility

Merge branch 'MDL-81132-master' of https://github.com/jleyva/moodle

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

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

Merge branch 'MDL-80099_main' of https://github.com/marxjohnson/moodle

Merge branch 'MDL-81319' of https://github.com/stronk7/moodle

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

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

MDL-78885 comboboxsearch: Points from review

MDL-78885 comboboxsearch: Deprecate trigger

MDL-78885 gradereport_grader: minor fixes to the collapse columns

MDL-78885 comboboxsearch: Some small fixes

- Remove aria-busy
- Remove aria-label from the listbox element. A listbox that is part of
  a combobox does not need to be labelled.
- No need for the aria-label for the select all checkbox because it is
  already enclosed in the <label> tag.

- Having "view results for ..." as aria-label of listbox options is not
  helpful and only creates noise.
- Deprecated (viewresults,core) as it was not used anymore
- aria-hidden="true" should not be used on .d-none elements
- 'noresults' was falsely documented as a required context variable for
  resultset.mustache. Not only it was not required, it was not even
  provided!

- Remove duplicate code from user_selector.mustache
- Remove the redundant aria-label from search_input_auto. It already has
  a label.
- Fix placeholder missing from one case

MDL-78885 comboboxsearch: Show x button when user search is not empty

MDL-78885 comboboxsearch: Comboboxes should have a meaningful label

Also, used the combobox's label as the dialogue's label as well

MDL-78885 comboboxsearch: Remove the redundant rtl context variable

MDL-78885 aria: Add support for data-short-text

MDL-78885 aria: Improve focus back

To support the case when a combobox is inside another dropdown

MDL-78885 comboboxsearch: Improve user search and keyboard handling

- Prevent rerendering of the menu when it's already open to preserve the
  selected/highlighted option.
- Allow ArrowUp and ArrowDown to trigger the user search dropdown for
  improved keyboard navigation.

This commit ensures that the menu is not re-rendered when it's already
open, preserving the selected option to enhance the user experience.
Additionally, ArrowUp and ArrowDown keys now trigger the dropdown,
improving keyboard navigation and user interactions.

MDL-78885 comboboxsearch: Reset status of filters to a zero state

Most of this is automatically handled by aria.js for other comboboxes.
But the search combobox inside the grade and group dropdowns are special
cases because they are not the same as the toggle element of the
dropdown.

MDL-78885 comboboxsearch: Switch event listener from 'click' to 'change'

Move from listening to 'click' to listening to 'change'

MDL-78885 comboboxsearch: .dropdown() calls should be on toggle element

Introduced the toggle property.

MDL-78885 comboboxesearch: user search should have value as well

MDL-78885 comboboxesearch: Comboboxes should have a name and value

- core_grades_renderer::group_selector() was changed previously, but it
  was providing redundant context data to
core_group/comboboxsearch/group_selector and on the other hand, it was
not providing 'name' and 'value' for core/comboboxsearch
- The $groupactionbaseurl parameter of
  core_grades_renderer::group_selector() is now deprecated as it was not
used.
- a bug is fixed: if only a lastname initial was set, the clear search
  button was not being displayed.

MDL-78885 comboboxsearch: remove aria-activedecendant on options change

MDL-78885 comboboxsearch: Enable aria.js for auto handling

- Added the `dropdown` class to ensure toggle element focus when the
  dropdown is closed.
- Improved keyboard handling by adding the `dropdown` class.
  aria.js will automatically handle keyboard interactions.
- Removed redundant keyboard handling.
- The "view all results" option is just a normal option in a combobox.
  It should not be treated as the default action for a combobox.
- Ensured correct markup for 'Esc' key handling. aria.js automatically
  focuses on the toggle element if the dropdown's toggle and the
  dropdown menu are wrapped within a .dropdown element.
- Implemented menu closure for outside clicks and when leaving the edit
  box.
- Manually focused on the user search element when opening the search
 dropdown due to a focusLock issue.
- Fix the issue of another dropdown staying open
- Clicking on the clearSearchButton should not close the dropdown

MDL-78885 theme_boost: Revert 3a8eeedf of MDL-75274.

These changes that I'm undoing were not only wrong, but also not used
and were left over code.

MDL-78885 aria: Fix pressing Home and End on editable comboboxes

MDL-78885 aria: Automatically trap focus inside dialog dropdowns

- there is no need to set the focus after a dropdown is opened. It is
automatically set as the result of trapping focus.

MDL-78885 core: combobox should have a valid aria-controls

- The aria-controls attribute cannot refer to an element that doesn't
  exist
- Also update filterDataset so it returns [] when searchTerm is empty

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

MDL-78885 gradereport_singleview: Behat updates after a11y fixes

MDL-78885 core_user: A11y fixes for user combobox search

Co-authored-by: Shamim Rezaie <shamim@moodle.com>

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

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

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

MDL-78885 gradereport_grader: Collapsing a11y fixes

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

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

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

Merge branch 'MDL-69688' of https://github.com/stronk7/moodle

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

MDL-80245 reportbuilder: implement interface elements for report tags.

Allow them to be edited during report creation/updating, display in
report listings table with filtering, and implement tag callback to
allow them to be discoverable across the site.

Merge branch 'MDL-81294-main' of https://github.com/roland04/moodle

MDL-81023 core_course: Move deprecated behat steps to course directory

MDL-81319 install: update the list of stale files for 4.4

MDL-71734 lang: final removal of deprecated 400 strings.

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

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

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

MDL-80835 auth_lti: fix bad cast breaking samesite LTI usage

MDL-80835 enrol_lti: add partitioning support for OIDC state cookie

Adds the property that is required by Chrome to opt-in to its 3rd party
cookie partitioning solution, CHIPS. This specific change ensures the
'state' cookie, used in the OIDC handshake, has partitioning support.
This cookie can be partitioned unconditionally, since it's a cookie
controlled by the library and one we don't expect to be set without
partitioning elsewhere.

MDL-80835 auth_lti: add partitioning to post-auth MoodleSession cookie

Adds the property that is required by Chrome to opt-in to its 3rd party
cookie partitioning solution, CHIPS. This specific change to auth_lti is
to ensure the MoodleSession Set-Cookie header resulting from
complete_user_login() calls (in auth.php) have this property set.

MDL-80835 enrol_lti: add cookies required notice to auth login endpoint

This will be displayed if the cookie checks fail, which currently occurs
in Safari only.

MDL-80835 enrol_lti: add partitioning support to MoodleSession cookie

Adds the property that is required by Chrome to opt-in to its 3rd party
cookie partitioning solution, CHIPS. This specific change deals with the
cookie that is set when the user is not yet auth'd with the site and is
necessary to facilitate OIDC nonce retrieval and validation.

MDL-80835 auth_lti: add cookie helper facilitating CHIPS opt-in

To opt a cookie in to Chrome's 3rd party cookie partitioning solution,
CHIPS, the property 'Partitioned;' needs to be set. This adds a helper
class supporting this, for a given cookie(s).

Note also, PHP's native
cookie APIs (setcookie, etc) don't support this cookie property yet -
(https://github.com/php/php-src/issues/12646).

Since this class is intended to allow existing Set-Cookie headers to be
modified before being sent (e.g. allowing clients to set a property on a
cookie set elsewhere in code), it deals with the headers directly anyway
but it means that new cookies must also use this helper to opt-in,
instead of relying on setcookie(). E.g. where the intent is to add
partitioning support to a new cookie, that cookie must first be set
(setcookie) and then it may opt-in to partitioning via this helper;
partitioning support cannot be achieved directly through setcookie and
friends yet.

MDL-69688 phpunit: Improve test_context_not_used

This adds a few changes to the old test_context_not_used test:

- Move it to become an advanced_test, because its mission
  is to verify that the assertEventContextNotUsed() assertion
  works as expected.
- For consistency, also move the fixtures to own phpunit fixtures.
- Add proper coverage tags, to verify that the assertion is being
  covered.
- Add a data provider to provide all the current cases and ease
  any future case that may be needed in the future. One by one
  because previously there was code never executed with the
  warning expectation causing the test to stop.
- Run them in isolation, while this is not strictly required, it's
  including external fixtures and, we'll need that isolation soon
  (for changes coming when moving the test to PHPUnit 9.6 in MDL-81266).