Merge branch 'MDL-53247-master' of git://github.com/danpoltawski/moodle

MDL-52651 htmlpurifier: Append rel=noreferrer to links.

Thank you to Zachary Durber for originally working on this issue.

Merge branch 'MDL-53350' of git://github.com/timhunt/moodle

Merge branch 'MDL-53353' of git://github.com/timhunt/moodle

Merge branch 'MDL-53291_ajax_fix' of https://github.com/BenKelada/moodle

MDL-52774 ajax: Require authentication when getting nav branch

When getting the navigation branch data through AJAX, require
login if forcelogin is set.

MDL-52901 mod_assign: Check due dates in external save_submission

MDL-52378 singleview: Using is_disabled instead of is_locked

MDL-52378 Singleview: Exclude Grade Attribute Permissions

Singleview users must have Manage Grade permission to modify the
Exclude grade attribute.

MDL-52808 calendar: Do not return events for hidden activities

MDL-52727 mod_data: Improve output of the form fields values

This issue mostly affects the search form fields. Submitted values for
these fields are typically obtained via optional_param() with
PARAM_NOTAGS specified as the parameter type - see parse_search_field()
methods. Such values themselves are not safe enough to be printed back
directly into the HTML as they might contain malicious code.

While working on the patch, some other places with weak protection were
detected and fixed.

In case of the itemid parameters, explicit clean_param() is added to
make sure we cast the value as an integer. That should make the s()
unnecessary but it was added anyway as an extra protection (just in case
the code flow changes or the parts of the code are re-used elsewhere).

MDL-53353 drag-drop marker quetions: fix edit form jitter

MDL-53350 drag-drop to text: layout problem on reivew

weekly release 3.1dev

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

Automatically generated installer lang files

Automatically generated installer lang files

MDL-53247 adminlib: change ordering of global search setup helper

Now put 'enable global search' as last step, thanks to David for the
idea.

MDL-53247 search: add cli indexer script

Merge branch 'MDL-30811-fix3' of git://github.com/andrewnicols/moodle

MDL-30811 output: Adjust ARIA role to alert

https://www.w3.org/TR/wai-aria-practices/#alert

MDL-30811 javascript: alert JS should be in template

By moving the JS required for an alert to the JavaScript we make it
possible for themers to use frameworks other than bootstrap if they so
wish.

MDL-30811 javascript: Lazily load core dependencies

To prevent possible race conditions, we lazily load the templates, and ajax
dependencies.

MDL-53247 search: allow search to be configured before enabled

The seach manager no longer checks if search is enabled before providing
an instance. It's up to the callers to do the access control.

MDL-48362 enrol_cohort: Correct variable naming for course sync

$course->id => $instance->courseid

MDL-53291 javascript: fix jquery ajax requests when data contains ??

jQuery ajax requests containing multiple '?' get replaced with a
string similar to jQuery152330039032_1231231, this is because it
is processing a stringified data object as jsonp and thinks its a url.
 workaround is to add contenttype
see https://github.com/jquery/jquery/issues/1799

Automatically generated installer lang files

Merge branch 'MDL-30811-fix2' of git://github.com/andrewnicols/moodle

MDL-52974 mod_assign: Include missing local library

MDL-53050 mod_forum: fix for base highlight

Merge branch 'wip-MDL-50887-master' of https://github.com/marinaglancy/moodle

MDL-50887 antivirus: clamav is standard plugin, missing strings

Merge branch 'MDL-53214-master-category-feedback' of git://github.com/cameron1729/moodle

MDL-53214 grades: Fix labels on feedback fields

MDL-30811 core: Initiate notifications on first use only

Merge branch 'MDL-52513' of https://github.com/rushi963/moodle

MDL-52136 mod_forum: fix some out-of-sync stuff

Thanks CIs!

Merge branch 'MDL-48362-master-fix3' of github.com/damyon/moodle

MDL-48362 enrol: Fixes to cohort and manual enrol plugins

These were regressions from the change to use the standard editing ui
for enrolment plugins.

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

MDL-52136 mod_forum: forum_post is templatable

Merge branch 'MDL-52397_behat2' of git://github.com/dmonllao/moodle

Merge branch 'wip-mdl-51075' of https://github.com/rajeshtaneja/moodle

MDL-51075 behat: Fixed exported question file size

MDL-52397 behat: Another behat fix, there is no student2 grade

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

MDL-52136 mod_forum: Correctly quote get_string vars in mustache

MDL-52136 core: Add support for quoting variables in mustache helpers

This is required for when helpers include json-encoded variables as arguments.
As an example, imagine a template with content:

{{# str }} somekey, someidentifier, { "fullname": "{{ fullname }}" } {{/ str }}

If the fullname variable were to include the double-quote character (e.g.
John "Trevor" Doe) because of the way in which mustache renders content, it
would become:

{{# str }} somekey, someidentifier, { "fullname": "John "Trevor" Doe" } {{/ str }}

This results in an invalid JSON structure.

To work around this issue, the quote characters in the passed variable
must be escaped:

{{# str }} somekey, someidentifier, { "fullname": "John \"Trevor\" Doe" } {{/ str }}

Unfortunately, Mustache provides no way of doing so natively.

With this function, we can quote the text as appropriate:

{{# str }} somekey, someidentifier, { "fullname": {{# quote }}{{ fullname }}{{/ quote }} } {{/ str }}

This also handles the case where the quoted content includes the Mustache
delimeter ({{ or }}).

For example:
fullname = 'John "}}Trevor{{" Doe'

Ordinarily this would be rendered as:
{{# str }} somekey, someidentifier, { "fullname": "John "}}Trevor{{" Doe" } {{/ str }}

This rendering is both a JSON error, and also a mustache syntax error because of the mustache delimeters.

The quote helper also escapes these by wrapping them in change delimeter
tags:

{{# str }} somekey, someidentifier, { "fullname": "John "{{=<% %>=}}}}<%={{ }}=%>Trevor{{=<% %>=}}{{{{=<% %>=}}" Doe" } {{/ str }}

Merge branch 'MDL-36652' of https://github.com/rushi963/moodle

Merge branch 'wip-mdl-53126' of https://github.com/rajeshtaneja/moodle

Merge branch 'MDL-53200-master-enfix' of git://github.com/mudrd8mz/moodle

MDL-50887 clam: upgrade step with wrong version

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

Merge branch 'MDL-50887-master' of https://github.com/lucisgit/moodle

Merge branch 'MDL-52397_fix-behat' of git://github.com/dmonllao/moodle

Merge branch 'MDL-52397_fix-oracle' of git://github.com/dmonllao/moodle

Merge branch 'MDL-48362-master-fix2' of github.com/damyon/moodle

MDL-52397 mod_assign: Fix behat test

MDL-48362 enrol_cohort: Fix unit test by manually triggering sync

Sync is now automatically triggered by add/update instance. We need
to manually trigger sync before each part of the test to clean up
role assignments from the previous test.

MDL-50546 mod_quiz: Correct typos in WS docs

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

MDL-52489 mod_assign: mentioned changes in upgrade.txt

Merge branch 'MDL-53077' of git://github.com/Dave-B/moodle

Merge branch 'MDL-53166' of https://github.com/eugeneventer/moodle-fixes

Merge branch 'MDL-48362-master-fix1' of github.com/damyon/moodle

Merge branch 'MDL-53263-forum-reply-deeplink' of https://github.com/brendanheywood/moodle

Merge branch 'MDL-46424_master' of https://github.com/dmonllao/moodle

Merge branch 'wip-MDL-44952-master' of https://github.com/marinaglancy/moodle

MDL-52397 editpdf: Fix draft vs nondraft comparison

We can not rely on the db engine returning results in the same order
if no order is specified in get_comments and get_annotations.

Also fixing a related unit test.

MDL-48362 enrol_cohort: Perform immediate sync on add/update instance

This restores backwards compatible behaviour.

MDL-51687 mod_folder: Trigger event when downloading all files.

Merge branch 'MDL-30811-fix' of git://github.com/andrewnicols/moodle

Merge branch 'MDL-52809-master' of git://github.com/dpalou/moodle

MDL-44952 mod_feedback: set value for hidenoselect if disabled

MDL-30811 form: Remove continue from test.

MDL-52974 mod_assign: Improve tests for view_submission_status

External tests only

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

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

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

Merge branch 'MDL-53027-master' of git://github.com/cameron1729/moodle

Merge branch 'MDL-53050-forum-target' of https://github.com/brendanheywood/moodle

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

Merge branch 'MDL-52936_master' of https://github.com/marxjohnson/moodle

Merge branch 'MDL-53214-master' of git://github.com/cameron1729/moodle

Merge branch 'wip-MDL-53172-master' of https://github.com/marinaglancy/moodle

Merge branch 'MDL-50175-master' of git://github.com/danpoltawski/moodle

MDL-50887 tests: Increment number of plugins in core_component_testcase.

We have got one more core plugin now.

MDL-50887 antivirus_clamav: Add unit tests for scanner class.

We are using "stubbing" method to replace real component (clamav) control
points with doubles in order to avoid direct calls to clamav. For more
details, please refer to https://phpunit.de/manual/4.8/en/test-doubles.html

Notice, that while it is possible to use setExpectedException in the tests,
this does not seem play correctly with consecutive assertions in the same
test after exception is thrown (they seem omitted). Explicit try/catch
construction has been used in testing instead.

To run just this test, use:
vendor/bin/phpunit antivirus_clamav_scanner_testcase lib/antivirus/clamav/tests/scanner_test.php

MDL-50887 antivirus: Add unit tests for manager class.

To run just this test, use:
vendor/bin/phpunit core_antivirus_testcase lib/tests/antivirus_test.php

MDL-50887 antivirus_clamav: Access config properties using getter.

MDL-50887 antivirus: Add config getter to base class.

Improving architecture to make it testable.

MDL-50887 antivirus_clamav: Split scanning logic and results processing.

This refactoring will make possible to assert scan results processing
behaviour in unit testing.

MDL-50887 antivirus: Replace 'antiviruses' with 'antivirus plugins'.

Language string chnages as per Helen Foster's comments in the ticket.

MDL-50887 antivirus: Refactor adminlib to use html_writer.

MDL-50887 antivirus: Use namespace referenced classes.

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

Merge branch 'MDL-50794-master-workshop-restrict-filetypes' of git://github.com/mudrd8mz/moodle

Merge branch 'MDL-53213' of https://github.com/mr-russ/moodle

Merge branch 'MDL-52965_master' of git://github.com/dmonllao/moodle

MDL-48362 enrol: Correct singleselect use in behat test