MDL-49329 admin: Improve plugins overview page rendering

Provides a bit more compact layout. Finally cleans up the relevant LESS
files and makes available updates info boxes consistent across all
screens where they are displayed.

MDL-49329 admin: Add core_plugin_manager::available_updates() method

The new method is going to be used to support the "Update all plugins"
feature.

MDL-49329 admin: Get rid of mdeploy and \core\update\deployer class

The mdeploy.php standalone script (used to download and unzip plugins
into the dirroot) and the \core\update\deployer class (as a
communication bridge between the core and the mdeploy.php) was
originally designed and implemented with the assumption that it would be
eventually used for updating the Moodle core itself, too. Therefore it
was written as standalone utility without dependency on the Moodle core
libraries.

However, it never happened and there is no real demand for that. So now
there is no need to have and maintain a completely parallel solution for
common things like fetching and unzipping plugin ZIPs.

Additional reasoning for mdeploy.php was that the core is not very
reliable during the core upgrade and we could run into various troubles.
This does not seem to be that bad. We rely on a lot of core
functionality (such as output rendering, DB access etc) and plugins
deployment seems to work well (and better) with common core libraries.

So long mdeploy, and thanks for all the hard work you did for us.

MDL-49329 admin: Fix plugins check page if no plugins require attention

A legacy undefined variable was used here and there was no way to get to
the full list mode view.

MDL-49329 admin: Clean up core_plugin unit tests

As a first step for removing the \core\update\deployer and mdeploy.php,
this patch fixes existing tests.

MDL-49329 admin: If unable to install a plugin, display the reason

MDL-49329 admin: Highlight missing/downgrading plugins

MDL-49329 admin: Fix report on required core version in missing plugins

If the plugin was missing from disk, the required core version was
displayed as "Moodle {$a}" because the core dependency resolver always
returns an object.

MDL-49329 admin: Fix the API for getting remote plugin info

The previous version of the plugin manager's method
get_remote_plugin_info() was suitable for installing missing
dependencies only. To make use of for installing new plugins and/or
available updates, it must be clear that we are requesting information
for the particular plugin version only, not "given or higher" version.

MDL-49329 admin: Validate pluginfo service response

The new client's method makes sure that the returned data structure has
expected format so that the plugin manager can safely rely on it.

MDL-49329 admin: Improve Plugins check page

Better inform about the number of plugins requiring attention vs total
plugins. Clean up the renderer.

MDL-49329 admin: Improve the UI for installing dependencies

MDL-49329 admin: Add ability to cancel installation of a new plugin

The plugins check screen now provides buttons to cancel installation of
a plugin. Available only for new installations (not upgrades) and for
additional plugins (not standard), given that the web server process has
write access to the plugin folder.

This has also been reported as MDL-48535.

As a part of the patch, there is improved processing of page URLs during
the upgrade. All this dancing around $reload URL is not needed once the
$PAGE->url is properly set to guide the admin on the correct page during
the upgrade process.

MDL-49329 admin: Introduce new \core\update\validator class

This is not really new class. Originally, the validator was implemented
in the scope of the tool_installaddon plugin. But we need its features
in other parts at the core level (e.g. during the upgrade to validate
available missing dependencies or available updates). Looking back, it
was a mistake to have it implemented at the plugin level rather than the
core level.

So this patch moves the tool_installaddon_validator class to
\core\update\validator class, together with associated unit tests,
fixture files and strings. There were changes made in the
tool_installaddon to make use of this new class instead of the old one.

This is a great step towards the bright future, hopefully there won't be
many regressions.

AMOS BEGIN
 MOV [validationmsg_componentmatch,tool_installaddon],[validationmsg_componentmatch,core_plugin]
 MOV [validationmsg_componentmismatchname,tool_installaddon],[validationmsg_componentmismatchname,core_plugin]
 MOV [validationmsg_componentmismatchname_help,tool_installaddon],[validationmsg_componentmismatchname_help,core_plugin]
 MOV [validationmsg_componentmismatchname_info,tool_installaddon],[validationmsg_componentmismatchname_info,core_plugin]
 MOV [validationmsg_componentmismatchtype,tool_installaddon],[validationmsg_componentmismatchtype,core_plugin]
 MOV [validationmsg_componentmismatchtype_info,tool_installaddon],[validationmsg_componentmismatchtype_info,core_plugin]
 MOV [validationmsg_filenotexists,tool_installaddon],[validationmsg_filenotexists,core_plugin]
 MOV [validationmsg_filesnumber,tool_installaddon],[validationmsg_filesnumber,core_plugin]
 MOV [validationmsg_filestatus,tool_installaddon],[validationmsg_filestatus,core_plugin]
 MOV [validationmsg_filestatus_info,tool_installaddon],[validationmsg_filestatus_info,core_plugin]
 MOV [validationmsg_foundlangfile,tool_installaddon],[validationmsg_foundlangfile,core_plugin]
 MOV [validationmsg_maturity,tool_installaddon],[validationmsg_maturity,core_plugin]
 MOV [validationmsg_maturity_help,tool_installaddon],[validationmsg_maturity_help,core_plugin]
 MOV [validationmsg_missingcomponent,tool_installaddon],[validationmsg_missingcomponent,core_plugin]
 MOV [validationmsg_missingcomponent_help,tool_installaddon],[validationmsg_missingcomponent_help,core_plugin]
 MOV [validationmsg_missingcomponent_link,tool_installaddon],[validationmsg_missingcomponent_link,core_plugin]
 MOV [validationmsg_missingexpectedlangenfile,tool_installaddon],[validationmsg_missingexpectedlangenfile,core_plugin]
 MOV [validationmsg_missingexpectedlangenfile_info,tool_installaddon],[validationmsg_missingexpectedlangenfile_info,core_plugin]
 MOV [validationmsg_missinglangenfile,tool_installaddon],[validationmsg_missinglangenfile,core_plugin]
 MOV [validationmsg_missinglangenfolder,tool_installaddon],[validationmsg_missinglangenfolder,core_plugin]
 MOV [validationmsg_missingversion,tool_installaddon],[validationmsg_missingversion,core_plugin]
 MOV [validationmsg_missingversionphp,tool_installaddon],[validationmsg_missingversionphp,core_plugin]
 MOV [validationmsg_multiplelangenfiles,tool_installaddon],[validationmsg_multiplelangenfiles,core_plugin]
 MOV [validationmsg_onedir,tool_installaddon],[validationmsg_onedir,core_plugin]
 MOV [validationmsg_onedir_help,tool_installaddon],[validationmsg_onedir_help,core_plugin]
 MOV [validationmsg_pathwritable,tool_installaddon],[validationmsg_pathwritable,core_plugin]
 MOV [validationmsg_pluginversion,tool_installaddon],[validationmsg_pluginversion,core_plugin]
 MOV [validationmsg_release,tool_installaddon],[validationmsg_release,core_plugin]
 MOV [validationmsg_requiresmoodle,tool_installaddon],[validationmsg_requiresmoodle,core_plugin]
 MOV [validationmsg_rootdir,tool_installaddon],[validationmsg_rootdir,core_plugin]
 MOV [validationmsg_rootdir_help,tool_installaddon],[validationmsg_rootdir_help,core_plugin]
 MOV [validationmsg_rootdirinvalid,tool_installaddon],[validationmsg_rootdirinvalid,core_plugin]
 MOV [validationmsg_rootdirinvalid_help,tool_installaddon],[validationmsg_rootdirinvalid_help,core_plugin]
 MOV [validationmsg_targetexists,tool_installaddon],[validationmsg_targetexists,core_plugin]
 MOV [validationmsg_targetexists_help,tool_installaddon],[validationmsg_targetexists_help,core_plugin]
 MOV [validationmsg_unknowntype,tool_installaddon],[validationmsg_unknowntype,core_plugin]
 MOV [validationmsg_versionphpsyntax,tool_installaddon],[validationmsg_versionphpsyntax,core_plugin]
 MOV [validationmsglevel_debug,tool_installaddon],[validationmsglevel_debug,core_plugin]
 MOV [validationmsglevel_error,tool_installaddon],[validationmsglevel_error,core_plugin]
 MOV [validationmsglevel_info,tool_installaddon],[validationmsglevel_info,core_plugin]
 MOV [validationmsglevel_warning,tool_installaddon],[validationmsglevel_warning,core_plugin]
AMOS END

MDL-49329 admin: Introduce new \core\update\core_manager tool

The plan is to have a single tool looking after all operations with
plugin ZIP packages (downloading, unzipping, moving to the dirroot and
back). For legacy reasons, we have that logic currently duplicated in
mdeploy and tool_installaddon. I would like to unify and simplify the
whole machinery to use the same code for available updates, manual
installation and plugin dependencies.

MDL-49329 admin: Display missing dependencies on plugins check screen

The patch improves the dependencies resolution in the plugin manager so
that the information about availability of the missing dependency is
included and can be displayed at the Plugins check screen during the
upgrade.

MDL-49329 admin: Introduce new \core\update\api client class

The purpose of this class is to provide a general client for all APIs
available at https://download.moodle.org/api/ (e.g. available updates,
plugin info, plugins list etc). Currently, fetching data from this API
is done separately at several places. This leads to code duplication and
harder maintenance (I know it well).

Additionally, the existing client was implemented as
tool_installaddon_pluginfo_client in the admin/tool/installaddon/ scope.
I will soon need to use the same functionality in the
core_plugin_manager and it would hurt my karma if the core was depending
on a class provided by a admin tool plugin (even if it is standard one).

So, there is new \core\update\api client implementing the version 1.3 of
the pluginfo API. There is a TODO note left for remaining services.

MDL-49329 admin: Move requirements resolving to the plugin manager

The patch moves the resolving logic from the renderer (where it should
not really be) to the plugin manager (controller). This is needed
because we will need the very same logic to be used at other places.

MDL-49329 admin: Improve loading of available updates info

The actual loading of available updates info objects is moved back to
the plugin manager class. As we can now mockup the manager in unit
tests, this allows us to bypass the real \core\update\checker and have
unit tests for \core\plugininfo\base::available_updates().

MDL-49329 admin: Keep reference to plugin manager in plugininfo objects

Plugin info objects are owned by the plugin manager (composition
pattern). Even if the plugin manager is a singleton, we need to keep
explicit reference to the plugin manager that owns the plugin info so
that we can mock up things in unit tests.

Therefore this patch introduces a new property of plugin info objects
that holds the reference to the instance of the plugin manager that made
(and hence owns) the given info instance.

The only trouble here is with static methods of plugin info classes such
as \core\plugininfo\base::get_enabled_plugins(). In these cases, the
code keeps using the core_plugin_manager singleton. The solution would
be to pass the plugin manager instance as a parameter but that is not
worth of change for now, IMHO.

MDL-49329 admin: Make core_plugin_manager better suited for unit testing

We can now override the plugin manager's methods in the testable
subclass while still keeping the singleton behaviour of it. The change
makes use of late static binding.

MDL-49329 admin: Do not always load info about available updates

Before this patch, whenever core_plugin_manager::get_plugins() was
called, it always attached info about available updates. But this is
needed only in quite rare cases, such as when the admin is looking at
the Plugins overview and Plugins check screens. There is no need to load
this on other places and for non-admin users.

The patch removes the loading from the method
core_plugin_manager::get_plugins_of_type() and implements lazy loading
directly in the plugininfo classes so that it is loaded only when
\core\plugininfo\base::available_updates() is actually called.

MDL-49329 admin: Fix \core\update\checker::enabled() logic

The method should check against $CFG->disableupdatenotifications and not
$CFG->disableupdateautodeploy. This had to be a copy&paste mistake from
the \core\update\deployer::enabled().

While looking at it, I also fixed couple of places where this method
should and could be used.

MDL-49329 admin: Improve the styling of the Plugins check screen

The intention here is to make the table a little bit more compact (using
less columns) and to make use of the bootstrap based label elements if
available.

weekly release 3.0dev

NOBUG: Fixed file access permissions

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

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

MDL-51623 core_event: Updating error messages

Merge branch 'MDL-51623_parent_event' of git://github.com/moodlerooms/moodle

MDL-51285 mod_forum: Posts all need to have different stamps

Merge branch 'MDL-51341_enrolled_users_filter' of git://github.com/davosmith/moodle

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

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

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

MDL-51341 enrolment: fix 'No group' enrolled user filter

Merge branch 'MDL-51531-master' of git://github.com/damyon/moodle

MDL-51624 mod_lti: New Web Service mod_lti_view_lti

MDL-51624 mod_lti: Rename function lti_view  to lti_launch_tool

MDL-51566 mod_choice: New WS mod_choice_delete_choice_responses

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

Merge branch 'MDL-50945-master' of git://github.com/xow/moodle

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

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

Merge branch 'MDL-51632_echo_url' of git://github.com/moodlerooms/moodle

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

MDL-50524 behat: Fixed multiple when then and use of generator

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

Conflicts:
lib/upgrade.txt

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

MDL-51526 mail: Import PHPMailer 5.2.13

MDL-51579 course: Bump version to update mobile service

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

Conflicts:
version.php

Merge branch 'wip-mdl-51285' of git://github.com/rajeshtaneja/moodle

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

MDL-50945 course: Add form-submit class to course edit buttons.

With contributions from Mary Evans.

MDL-51250 course: Add validation for section name

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

MDL-41042 coursecat: reset course contacts cache more often

MDL-41042 coursecat: better handle changes in coursecontact

MDL-49028 mod_wiki: Fix for reseting pages and tags

This will ensure that both pages and tags will be
removed when all the reset options for the wiki are
selected.

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

Merge branch 'MDL-51525-master' of git://github.com/ryanwyllie/moodle

Merge branch 'wip-MDL-51315-master' of git://github.com/abgreeve/moodle

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

Merge branch 'MDL-48438' of https://github.com/andrewhancox/moodle

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

Automatically generated installer lang files

MDL-50526 mod_assign: Highlight error message when user has no group

If I set the "Require group to make submission" setting in an assignment
activity to yes, students who are not in any group are prevented from submitting
their assignment and are presented a warning instead that they should contact
their teacher. This message is now presented in an alert box.

MDL-50524 assign: improve method visibility and upgrade.txt

Merge branch 'MDL-50524' of https://github.com/andrewhancox/moodle

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

Merge branch 'MDL-51529-master' of git://github.com/damyon/moodle

Merge branch 'MDL-51250_master-empty-return-fix' of git://github.com/dmonllao/moodle

MDL-51250 course: No function returns in empty()

Merge branch 'MDL-51530-master' of git://github.com/ryanwyllie/moodle

Merge branch 'MDL-51532-master' of git://github.com/damyon/moodle

Merge branch 'MDL-51676-master-ltiservice_memberships' of git://github.com/mudrd8mz/moodle

Merge branch 'MDL-51533-master' of git://github.com/damyon/moodle

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

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

Merge branch 'MDL-51538-master' of git://github.com/damyon/moodle

Merge branch 'MDL-51539-master' of git://github.com/damyon/moodle

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

Merge branch 'MDL-51588-master' of git://github.com/FMCorz/moodle

Merge branch 'MDL-51578' of git://github.com/blaky/moodle

Merge branch 'MDL-51434_master-fix-unit-test' of git://github.com/dmonllao/moodle

MDL-51434 mod_scorm: Including protectpackagedownloads in unit tests

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

Merge branch 'MDL-51228-master' of https://github.com/dthies/moodle

MDL-51461 javascript: Fix whitespace issue

Merge branch 'MDL-51461' of https://github.com/andrewhancox/moodle

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

Merge branch 'MDL-50805_master' of https://github.com/crazyserver/moodle

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

MDL-49811 mod_choice: Removing skipresultgraph link

The target of the link is not available after MDL-30816.

Merge branch 'MDL-49811_master' of git://github.com/jinhofer/moodle

Merge branch 'master-MDL-49750' of git://github.com/jojoob/moodle

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

MDL-51676 mod_lti: Mark LTI Memberships service as a standard plugin

MDL-51564 webservice: Review the use of external format_string