MDL-61859 core_completion: fix unit tests expecting exceptions twice

We can't expectException twice in the same test, so split this out into
another test.

weekly release 3.5dev

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

Merge branch 'MDL-53226-master-fix' of git://github.com/junpataleta/moodle

MDL-53226 search_simpledb: Fix table name in query

Merge branch 'MDL-36941_master-fix' of git://github.com/markn86/moodle

MDL-36941 message_popup: do not remove message_popup table

This is a partial revert of commit 123631ad4718141627d77027dbef0448bae22ac8.

Most of the logic was NOT reverted, just the part deleting the table was.

Note - this fix was done on integration.git and as a result no
upgrade step was added to re-introduce the table, sorry! This
means for those on integration.git you will have to re-install
your Moodle site so the 'message_popup' table is present.

Please see tracker for comments as to why we need to keep it! :)

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

Merge branch 'MDL-61364-master' of git://github.com/lameze/moodle

Merge branch 'MDL-61364-master' of git://github.com/lameze/moodle

MDL-25617 backup: backup/restore using extra_question_fields

Merge branch 'MDL-39060-master' of git://github.com/zig-moodle/moodle

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

MDL-60736 tool_mobile: Add missing VALUE_OPTIONAL to new returned info

Merge branch 'MDL-60736-master' of git://github.com/jleyva/moodle

Merge branch 'MDL-61321-master' of git://github.com/jleyva/moodle

Merge branch 'MDL-61718-master' of git://github.com/junpataleta/moodle

Merge branch 'MDL-60762-master' of git://github.com/junpataleta/moodle

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

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

Merge branch 'MDL-61656' of git://github.com/stronk7/moodle

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

Merge branch 'MDL-61248' of https://github.com/ak4t0sh/moodle

Merge branch 'MDL-61576-master' of git://github.com/cescobedo/moodle

Merge branch 'MDL-61691-master' of https://github.com/snake/moodle

Merge branch 'MDL-55532-master' of git://github.com/lameze/moodle

Merge branch 'MDL-61733-master' of https://github.com/MartinGauk/moodle

Merge branch 'MDL-60119-master' of https://github.com/lucaboesch/moodle

MDL-61770 iplookup: Check types rather than values

MDL-61364 core: fix cibot coding issues

MDL-61364 core_question: tests for question_has_capability_on exception

MDL-61364 question: Add unit tests for question_has_capability_on

MDL-61364 core: remove param from question_has_capability_on calls

MDL-61364 question: adjust code and fix phpunit errors

MDL-61364 question: refactor question_has_capability_on

MDL-60762 tool_usertours: Respect $CFG->admin in manager.php

MDL-53226 search_simpledb: Remove unnecessary object clone

MDL-61691 core_plagiarism: fix namespace in privacy provider interface

MDL-61257 reports: Report logs link to unavailable completion report

MDL-55532 gradebook: fix grade export to include category names

MDL-61364 question: unit tests for submit_tags_form

MDL-61364 question: add support for course tags on modal

MDL-61328 tool_usertours: do not fix tour sortorder after step editing

otherwise it breaks when we try to modify the step sortorder

MDL-39060 backup: fix typo backup destroy() function

Automatically generated installer lang files

MDL-61248 admin: fix incorrectly initialized property

MDL-61733 atto_table: fix creating new table

If Atto is embedded in an HTML table (e.g. in the database activity
on the template pages), you could not create a new table.

MDL-36941 core: Remove default for convhash field in upgrade.php

Merge branch 'MDL-36941_master' of git://github.com/markn86/moodle

MDL-36941 core_message: dont query DB unnecessarily checking read status

MDL-36941 core_message: fixed minor CiBoT complaints

MDL-36941 core_message: improved performance of helper::get_messages()

Improved the query to use the 'convhash' field as well as adding an index.

Also fixed issue where 'timeread' was hardcoded as 0.

MDL-36941 core_message: added details to upgrade.txt

MDL-36941 core: added index to 'message_user_actions' table

MDL-36941 core_message: deprecated 'api::mark_all_read_for_user'

MDL-36941 core: removed 'message_working' table

MDL-36941 message_popup: removed 'message_popup' table

MDL-36941 core: added 'convhash' field to quickly get conversations

MDL-36941 core: final deprecation of functions

This was done earlier than the usual deprecation
process since we have changed the table structure.
It would take time to change the logic in these
functions, but as they have been deprecated for
some time it was decided to forego this effort.

MDL-36941 message_popup: use new WS for marking notifications as read

Also changed the observer to listen for the notification_viewed event,
rather than the message_viewed event.

MDL-36941 core: added new mark notification as read webservice

MDL-36941 core: added events for notifications

MDL-36941 core_message: mark notification/messages takes stdClass

This avoid unnecessary DB calls.

MDL-36941 core: convert existing api to use new table structure

Also deprecated the following functions -

1. message_move_userfrom_unread2read - It is not necessary
   for us to mark a message as read on user deletion.
2. message_get_blocked_users - Horrible logic used to
   determine if a user is blocked via reference on some
   randomly chosen 'isblocked' variable.
3. message_get_contacts - The same as above. This can be
   done in a much nicer way.
4. message_mark_message_read - We want two functions to do
   this to avoid confusing messages and notifications.
5. message_can_delete_message - This assumed the variable
   $message contained the 'useridto' property, which
   was present in the old table structure. We do not want
   future usages where a query is done on the new table
   and is simply passed as this won't contain this property.
6. message_delete_message - Same as above.

MDL-36941 core: alter events to use new table structure

MDL-36941 core_message: removed unit test that will always fail

We are changing how the API behaves so the old tables will never
be populated. This unit test is testing a deprecated function
which we will not be refactoring to use the new table structure.

MDL-36941 core: added new messaging and notification tables

MDL-60119 mod_feedback: Suppress choice weight in parentheses.

weekly release 3.5dev

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

MDL-61576 mod_lti: remove allowinstructorcustom in lib/locallib.php

MDL-51362 libraries: Remove deprecated admin tool unittest

MDL-61718 admin: Make sure city and country columns are not duplicated

* Since City/town and Country have been added in $CFG->showuseridentity
since 3.4, we need to exclude the required city and country columns
when fetching the extra columns using get_extra_user_fields().

Merge branch 'MDL-61678' of git://github.com/stronk7/moodle

Merge branch 'MDL-58179-master-fix' of https://github.com/andrewnicols/moodle

MDL-58179 mod_lesson: Add missing : in labels

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

MDL-61530 mod_glossary: import_entries.feature needs @_file_upload tag

Merge branch 'MDL-52989-master' of git://github.com/junpataleta/moodle

MDL-52989 mod_lesson: Fix typo in feature description

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

MDL-61514 Quiz: Fix restore error

Include mod/quiz/locallib.php to fix the restore error.

Merge branch 'MDL-61658-master-allcountrycodes' of git://github.com/mudrd8mz/moodle

MDL-61658 user: Fix displaying of user's country name

In these cases we want to use the method get_list_of_countries() to
obtain translated names of all potential country codes. The country code
is already there in the database. So it does not matter if the country
is among those whitelisted in $CFG->allcountrycodes - we want to see its
name anyway.

MDL-61658 user: Fix parsing of the agedigitalconsentmap setting

When parsing and validating the value of the Digital age of consent
(agedigitalconsentmap) field, we want to load all country codes. Not
just the ones whitelisted for drop down menus that users can choose
from.

MDL-60736 tool_mobile: Support session lang in WS

MDL-60736 tool_mobile: Return language settings in get_public_config WS

MDL-61410 question: Check we have tags to set to avoid notices

Merge branch 'MDL-59434-master' of https://github.com/sammarshallou/moodle

Merge branch 'MDL-61519-master' of https://github.com/andrewnicols/moodle

MDL-61519 coursecat: Reset the coursecat cache on enrolment

Merge branch 'MDL-61519-master' of https://github.com/MartinGauk/moodle

Automatically generated installer lang files

Merge branch 'MDL-61605' of https://github.com/timhunt/moodle

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

Merge branch 'MDL-61410-master' of git://github.com/lameze/moodle

MDL-53226 search_simpledb: Refine the patch

- Clumsy fallback only when there is no full-text search support
- Mimic solr tests
- pgsql tokenization using simple configuration
- workaround for mysql '*' search issue
- total results proper calculation
- SQL server FTS support
- Standarize dml full-text search checkings
- Upgrade note about the new dml method
- Set search_simpledb as default engine if no solr config

MDL-53226 search_simpledb: Use databases full-text search capabilities

MDL-53226 search_simpledb: Adding clumsy search

MDL-61519 coursecat: do not cache get_all_children_ids