Merge branch 'MDL-70727-master' of https://github.com/NashTechOpenUniversity/moodle

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

Merge branch 'MDL-71084-master' of git://github.com/peterRd/moodle

MDL-71084 core_navigation: Move secondary_test to proper location with namespace

MDL-71084 core_navigation: Provide a header title and active node text

Merge branch 'MDL-69792-master' of git://github.com/sarjona/moodle

Merge branch 'MDL-71104-master-2' of git://github.com/mickhawkins/moodle

MDL-71104 core_completion: Fix fetching of grade completion status

We should be able to fetch the grade completion status for modules
that do not have custom completion configured. This also improves
unit testing coverage.

MDL-71104 completion: Fix completion_info::get_data() unit tests

* Fixed inverted $sameuser test data.
* Fixed caching expectation check. Caching only relies on whether the
user accessing the completion data is the same user or not.
* Fixed checking for the caching of other modules. Should have been
checking cm ID and not instance ID.
* Additional test case when whole course parameter is passed as
true, but the requesting user is different from the target user.

Merge branch 'MDL-71039-master' of git://github.com/marinaglancy/moodle

Merge branch 'MDL-70313-master' of git://github.com/peterRd/moodle

Merge branch 'MDL-69036' of git://github.com/paulholden/moodle

Merge branch 'MDL-67870-master' of git://github.com/mihailges/moodle

MDL-67870 backup: Skip value change if setting is locked by hierarchy

The given condition in the if statement did not take into account cases
when the setting is locked by hierarchy. Settings which are locked by
hierarchy should should keep their current values which were inherited
from the parent.

Merge branch 'MDL-70816-master-4' of git://github.com/rezaies/moodle

MDL-70816 mod_choice: activity_dates implementation

MDL-70816 core: Activity date details base class

* Base class for fetching an activity module's important date for a user

Merge branch 'MDL-71139-master-trademark' of git://github.com/mudrd8mz/moodle

Merge branch 'MDL-70919-master' of git://github.com/marinaglancy/moodle

MDL-71149 install: update the list of stale files for 3.11

MDL-69792 core_badges: deprecated backpack.js and BADGE_BACKPACKURL

Final deprecation for constant BADGE_BACKPACKURL (it has been removed)
and badges/backpack.js file (it has been removed too).
Besides, some methods have started the deprecation process too:
 - badges_check_backpack_accessibility
 - badges_setup_backpack_js()
 - badges_local_backpack_js()

Apart from that, string 'addbackpack' has been deprecated here
because it's not used anywhere.

weekly release 4.0dev

NOBUG: Fixed file access permissions

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

MDL-70919 core_form: non-static method can not be called statically

MDL-71139 trademark: Improve the trademark messaging

Merge branch 'MDL-70992-master' of https://github.com/NashTechOpenUniversity/moodle

Merge branch 'MDL-70924' of git://github.com/paulholden/moodle

Merge branch 'MDL-70198-master-2' of git://github.com/peterRd/moodle

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

Merge branch 'MDL-70977' of git://github.com/paulholden/moodle

Merge branch 'MDL-70978-master' of git://github.com/aanabit/moodle

Merge branch 'MDL-71068-protectusername-confirmed' of https://github.com/brendanheywood/moodle

Automatically generated installer lang files

Merge branch 'MDL-70901-master' of git://github.com/marinaglancy/moodle

Merge branch 'MDL-70896-master' of git://github.com/marinaglancy/moodle

Merge branch 'MDL-70968-master' of git://github.com/marinaglancy/moodle

Merge branch 'MDL-70893-master' of git://github.com/marinaglancy/moodle

Merge branch 'MDL-70689-master' of git://github.com/sarjona/moodle

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

MDL-71068 login: Fix edge cases with $CFG->protectusernames

MDL-70198 core_navigation: Secondary nav unit tests

MDL-70198 core_navigation: Implement secondary nav and curate nodes

Implement the new secondary nav and curate the nodes from flatnav and settingsnav

MDL-70198 core_navigation: Create new base view class

MDL-70992 Folder: Unnecessary tab stops to the left of files

Merge branch 'MDL-70565' of git://github.com/paulholden/moodle

Merge branch 'master-MDL-71044_v3' of https://github.com/golenkovm/moodle

Merge branch 'MDL-67748-master-managewebservices' of git://github.com/mudrd8mz/moodle

MDL-67748 admin: Add Behat tests for tokens filtering features

Credit goes to Andrew for Behat data generators for webservices.

Co-authored-by: Andrew Nicols <andrew@nicols.co.uk>

Merge branch 'MDL-44613-integration-master' of git://github.com/mihailges/moodle

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

Merge branch 'MDL-70780-master' of git://github.com/andrewnicols/moodle

MDL-44613 mod_assign: Fix behat failure in classic

The link to the calendar page is missing in the navigation block in
classic which causes a behat failure in the 'Set availibility dates
for an assignment' feature. The fix includes adding the calendar block
to the Course page which can be used to navigate to the calendar page.

MDL-69474 accessibility: page->url can be not set, protect against that

Both in unit tests, but also in other areas (say blocks, for example)
it's possible that page->url has not been set yet.

So, let's change current code to always check if we have something
(page->url) to compare.

Note the use of has_set_url() is needed, because page->url is magic
method and it doesn't work as expected. Hate magics!

MDL-67748 admin: Do not show missing capabilities for mobile app service

We used to display capabilities like "Manage any calendar entries",
"Delete evidence", "Manage competency frameworks", "View hidden courses"
and others as "Missing capabilities" for the mobile app service tokens.

This gave dangerous impression that the app will not work for students
without these capabilities granted. There are known cases of admins who
started to grant all these caps to the Authenticated user role because
they were afraid the app would not work for them.

The problem here is that the official mobile app service includes some
functions that have these capabilities declared as required. But they
are not really required to use the app. Either the app makes its own
clever checks of capabilities before calling the functions, or sometimes
the capabilities are not even correctly declared.

It is safer for everybody to display this information for custom
services only where the risk of the falsely missing caps is lower and
the information is more accurate.

Also, the help text has been improved so it does not suggest that these
capabilities must be always added. We do not know why the service has
them declared. In some cases, a service has capabilities declared just
because it makes use of them in the if-then fashion.

Additionally, the patch also displays the service short name because it
is actually needed to know.

MDL-67748 admin: Show user identity fields on the manage tokens page

On sites with many users, the fullname itself may not be unique
identifier of users. Display the user identity fields below the token
owner's name.

MDL-67748 admin: Add filter to the WS tokens management page

The patch adds ability to filter the list of token by the token value,
the user and the service. Also the button to create a new token is made
more prominent and easier to spot.

MDL-67748 admin: Improve the display of tokens the user cannot see

MDL-67748 admin: Refactor tokens management as an external page

No need to have a dedicated admin_setting_managewebservicetokens admin
setting subclass, and yet do not use almost any of its features. So the
patch merges the list and the forms handling into a single external
page.

Similarly, it feels like overkill to have a renderer method for a
simple confirmation widget - especially given that the URLs were
hard-coded in it. So that one was dropped.

MDL-67748 admin: Improve the display of missing capabilities

The patch improves the display of the missing capabilities on the Manage
tokens page and the Select authorised users page. The list of
capabilities is rendered in a collapsible area and does not occupy too
much space by default.

Additionally on the later page, the patch improves the display of the
list of existing authorised users so that it respects the configured
identity fields.

MDL-67748 admin: Improve get_missing_capabilities_by_users()

The previous implementation falsely reported all implicit capabilities
inherited from the authenticated user archetype. That caused a lot of
capabilities reported as missing, even if they were correctly granted.

This new implementation uses a different logic. Instead of seeking for
explicitly assigned capabilities, it searches for capabilities that are
not assigned to any of the user's role across the system.

Please refer to the inline documentation. This should be still used for
informative reports only, not for actual permissions evaluation. The
context has been ignored here, as well as all the overrides etc. This
patch just makes it a lesser evil.

MDL-67748 admin: Simplify the webservice token form

We can now use the autocomplete element to select the user. No need to
for the legacy behaviour.

MDL-67748 admin: Respect the viewfullnames cap at manage tokens table

MDL-67748 user: Add new core_user/form_user_selector JS module

The module provides AJAX support for the autocomplete form element.

MDL-67748 user: Add a new core_user_search_identity external function

The purpose of this external function is to provide data for
asynchronous user selectors and similar widgets. It allows to search
users matching the given query in their name or other available identity
fields.

MDL-67748 user: Fix @link docs tag usage

Not related to this issue, just spotted while working on it.

MDL-67748 admin: Move the web service token form to its own file

MDL-67748 admin: Move the web services under the server administration

I suppose the original reasoning was that the web services protocols are
pluggable and that plugins can provide external function. But those are
not good reasons to put the whole Web services section under plugins.
Web services are not a plugin type and they always felt very
counter-intuitive there.

From the server administrtaion perspective, web services are similar to
scheduled tasks and therefore, they should live in the same section.

Merge branch 'MDL-69474-master' of https://github.com/NashTechOpenUniversity/moodle

Merge branch 'MDL-44613-master-1' of git://github.com/mihailges/moodle

Merge branch 'MDL-65974-master-iteration03' of git://github.com/ferranrecio/moodle

MDL-70977 mod_forum: fix empty equality check updating post content.

Using the external method for updating posts, the check for empty
subject/message content wasn't correct (disallowed the string '0'),
in addition to being impossible to set a posts message format
property to FORMAT_MOODLE (integer 0).

MDL-70924 course: ensure required fields present when getting courses.

MDL-65974 course: fix return section in modchooser

Merge branch 'MDL-66667-cache-course-image-master' of https://github.com/dmitriim/moodle

Merge branch 'MDL-65974-master-iteration03' of git://github.com/ferranrecio/moodle

Merge branch 'MDL-70306-master' of git://github.com/peterRd/moodle

MDL-70306 auth: Update the thirdpartylib.xml

MDL-70313 core: Update the thirdpartylib.xml

MDL-70313 core: Rebuilt module files

MDL-70313 core: Update codemirror to 5.59.4

MDL-66667 course: add course image cache

MDL-44613 mod_assign: Automate testing of the assignment availability

MDL-44613 tool_behat: Update datetime format usages to strftime

Updates all existing datetime format usages to strftime in the
'time to timestamp' transformation in tool_behat behat tests.

MDL-44613 core_message: Update datetime format usages to strftime

Updates all existing datetime format usages to strftime in the
'time to timestamp' transformation in core_message behat tests.

MDL-44613 behat: strftime format in time to timestamp transformation

Modifies the 'string time to timestamp' behat trasformation to use
userdate() instead of date(). The userdate() method is generally used
throught Moodle for constructing formatted date strings and this change
will provide more consistency and prevent any potential behat failures.
Therefore, if the date format is defined in the given trasformation, it
has to be strftime compatible. Example:
'I should see "##tomorrow noon##%A, %d %B %Y, %I:%M %p##"'

Automatically generated installer lang files

MDL-70978 core_contentbank: Increasing contrast for unlisted content

MDL-70689 oauth2: unit tests for new IMS OBv2.1

Apart from adding new unit tests for covering new IMS OBv2.1 issuer,
some extra tests have been added to test existing issuers and confirm
they are still working as expected after the changes.

MDL-70689 oauth2: deprecate unused strings

MDL-70689 oauth2: self-register IMS OBv2.1 services

IMS OBv2.1 services have a registration endpoint to get client id
and secret.
This patch adds and implements the "register" method for the IMS
Open Badges Connect discovery system, to get the proper client id
and secret values.

MDL-70689 oauth2: add support to IMS OBv2.1 service

In order to make easier to create and maintain new OAuth2 services,
a couple of classes have been added:

- discovery\* contains methods related to the discovery system. Until
now, only OpenID Connect was supported by Moodle so all the code was
centralised in api.php.
With this patch, as IMS OBv2.1 has a different discovery system, a new
abstract class (base_definition) has been added to be called and let
every discovery system (OpenID Connect, IMS Badge Connect...)
implement their own methods.

- service\* . Instead of keep adding methods to the api, the service
namespace has been created to store all the standard issuer services.
An interface (issuer_interface) has been created and all the services
should implement it.
This patch creates the "IMS OBv2.1" and "Custom" services and moves
the methods for "Google" service because it uses the OpenID connect
discovery system.

MDL-70565 user: filter course participants by country.

MDL-71036 phpunit: Coding style changes, 99% whitespace only

I've gone over a few of the mofified files (those
which were showing warnings and errors to CiBoT. Some of them
have been fixed completely, while others only have fixed
for the lines belonging to this issue (lib/tests/moodlelib_test.php)
for example.

MDL-71036 phpunit: Add all the noticeable changes to upgrade notes

MDL-71036 phpunit: xml config - switch coverage info to new includes

This applies the "whitelist" => "include" changes to all the core
phpunit_coverage_info occurrences, so core won't emit any deprecation
warning (see previous commit).

At the same time, modified a bunch of comments in coverage files
to be more readable/understandable.

MDL-71036 phpunit: xml config - deprecate whitelist from coverage

Following the changes in the schema, from "whitelists" to "includes",
we are deprecating these two properties from phpunit_coverage_info:

- whitelistfolders => includelistfolders
- whitelistfiles   => includelistfiles

They will continue working over the deprecation period but the init/util
scripts will throw some warnings about them being deprecated for 3.11 and
the way to replace them.

Standard 2y deprecation applies with final removed to happen @ MDL-71067

MDL-71036 phpunit: xml config - change dist file to spread them to final

Basically switch to 2-space indenting and small tweaks to comments,
so both the main phpunit.xml file and also the individual components
ones, all them based in the dist one will be consistent.

MDL-71036 phpunit: xml config - change generation code to follow xsd

This performs all the changes needed in the util generator to
produce XML files compliant with the new schema (see previous
commit for description of changes).

- All the occurrences in code of filter => coverage.
- All the occurrences in code of whitelist => include.
- Apply all the changes to comply with the new schema.

- Remove processUncoveredFilesFromWhitelist attribute, useless (false
    is its default value, and now have another name).
- Move from 4-spaces indented XML to 2-spaces indented.
- Small linefeed tweaks to generate better-looking XML.

MDL-71036 phpunit: xml config - add new (9.5) validation xsd

PHPUnit 9 comes with various changes in the XML
configuration file, namely:

  - the old "filter" section has been renamed to "coverage".
  - "whitelist" has been renamed to "include"
  - "exclude" is not a child of "include" anymore, but of "coverage".
  - "include" cannot have configuration attributes anymore, only
    "coverage" can"

Visually it means that the old section (invented example):
```
<filter>
  <whitelist attributes_may_go_here="xx">
    <directory suffix=".php">classes</directory>
    <directory suffix=".php">externallib.php</directory>
    <exclude>
      <directory suffix="_test.php">.</directory>
    </exclude>
  </whitelist>
</filter>
```
Now looks like:
```
<coverage attributes_may_go_here="xx">
  <include>
    <directory suffix=".php">classes</directory>
    <directory suffix=".php">externallib.php</directory>
  </include>
  <exclude>
    <directory suffix="_test.php">.</directory>
  </exclude>
</filter>
```

So, switching to the new xsd so we can validate the remaining changes.