MDL-54778 form: Make it all js style (eslint) compliant

The changes in this commit should not be problematic, just:

- different whitespace.
- some docs.
- 1 variable to camelCase.

And then, less trivial, but safe enough IMO:

- a change to camelCase some identifiers and their calculation.

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

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

MDL-54983 docs: Improve phpdocs for create_user_key

Also fixed the docs for get_user_key.

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

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

MDL-54940 behat: No need to press save

Previous step press save button, so no need to press again

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

MDL-51789 users: Correct deprecation version

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

MDL-51789 users: Allow picture update from webservices.

useredit_update_picture as moved to user_update_picture
as it's more general.  It was also moved to user/lib.php
so it can be used by both webservices and edit without more include files.

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

Merge branch 'MDL-54894-master' of git://github.com/crazyserver/moodle

Merge branch 'MDL-54940-master' of https://github.com/LukeCarrier/moodle

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

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

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

MDL-35104 auth: Don't allow modification of username

Presently it is either unreliable, or not possible to change
the username of a user created with an authentication plugin.
In some cases it is even hard coded to fail. Ideally we would
sync the username, and the issue MDL-21928 exists to address
that. However, in the mean time we should not allow the
username of an external user to be modified.

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

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

MDL-54948 core_upgrade: fix unoconv check on the environment page

MDL-54948 core_files: make the check consistent on unoconv test page

Merge branch 'wip-MDL-53399' of https://github.com/JoeyAndres/moodle

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

MDL-54778 form: Fix lots of issues with form dependencies

The original issue here was that each loop of the named values did not
check for prototypal properties. As a result, if there were input fields
with names such as 'sort', 'valueOf', 'constructor', etc. these would
return their prototypal functions instead of a falsy value, and be treated
as though they are array - hence the 'Cannot push to Function' type error.

Following on from this I discovered that the data stores were being created
as arrays, but used as objects. This can also cause issues with some form
input names -- e.g. if they are numeric.

These two issues were resolved together by correctly storing them in
objects, and checking that those objects had real properties
(hasOwnProperty). This itself has to use the prototypal function to cater
for the potential of a field name called 'hasOwnProperty'.

I also found that the instance value stores were being initialised in the
prototype (and therefore shared), which meant that there were numerous
issues if two forms were present on the same page, or one form replaced an
existing one (e.g. forms initialised in JS).

In addition, it also became apparant that several values were being used
outside of scope, or in the wrong scope. This caused further issues when
creating multiple forms on a page.

MDL-54881 quiz: Ensure user attempts are deterministic for testing.

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

MDL-54894 comments: Get course from context if not informed

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

MDL-54590 Installer: Additional info about database name

Additional info that creating database name containing dots is not
   supported

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

MDL-54940 behat: use the fixed step

MDL-54940 behat: rectify section edit regression in MDL-53381

MDL-54939 amd: build modified files

MDL-54939 amd: fix lint issues

Mostly correcting jsdoc, some other line wrapping rules and empty
functions.

MDL-54654 mod: fix forum/scorm invalid css

MDL-54654 mod_data: fix invalid css for image gallery preset

MDL-54654 choice: remove CSS definitions

The 'inherit' for border colour was not valid and so no border was
displayed.

The display:inline does not work with floated elemenets.

MDL-54654 choice: reformat CSS

MDL-54654 book: silence csslint warning

(Important is by design for print layout)

MDL-54654 assignfeedback_editpdf: remove invalid CSS

The 'left' positioing was invalid and conflicting with the
right definition anyway, so wasn't necessary.

MDL-54652 singleview: remove invalid css decleration

margin-bottom can't be used with display: inline, so use padding
instead.

MDL-54652 grade_report: fix css lint issues

max-width: initial isn't valid css and in my testing it makes no
difference, so is removed.

We also remove invalid css on exception.

This reverts commit 72ece42858643c789d2c2f86fccdab0afefa18d0

MDL-54652 block_community: remove unused css

Fixing lint errors

MDL-54652 filter_mediaplugin: refomat css

MDL-54652 filter_mediaplugin: removed unused invalid css

It is override by core styles anyway.

MDL-54652 tool_lp: remove invalid css on comeptency menu bar

MDL-54652 tool_health: make css more specific to avoid '!important'

This admin tool is unsupported anyway..

MDL-54939 eslint: tweak rules

1) Allow jsdoc descriptions to be empty. I think that matches our phpdoc
checker, and I think its probably better than me filling the
descriptions in poorly.

2) For the captilisation rules, don't apply to object properties (this
is because jQuery has lots of violators of this rule)

MDL-54939 amd: Fix lint issues fixable with eslint --fix

weekly release 3.2dev

NOBUG: Fixed SVG browser compatibility

Merge branch 'MDL-52127-eloy' of https://github.com/danpoltawski/moodle

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

Merge branch 'MDL-45762-master' of https://github.com/xow/moodle

Merge branch 'MDL-53788_modlti_master' of git://github.com/nhoobin/moodle

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

Merge branch 'MDL-54559-int-master' of https://github.com/merrill-oakland/moodle

MDL-54559 theme: Recenter search box

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

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

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

Merge branch 'MASTER_MDL-53306' of https://github.com/jacac/moodle

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

Merge branch 'MDL-54036_master' of https://github.com/liorgil/moodle

Merge branch 'MDL-54622_m32v1' of https://github.com/sbourget/moodle

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

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

Merge branch 'MDL-54797' of git://github.com/aolley/moodle

Merge branch 'MDL-54787_master' of https://github.com/snake/moodle

Merge branch 'MDL-54576_themeSelector' of https://github.com/mrmark/moodle

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

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

Merge branch 'MDL-51978-master' of https://github.com/jtc/moodle

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

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

MDL-52127 grunt: add .eslingingore file, keep task

Eloy prefers we keep the ignorefiles grunt task

MDL-45762 course: Error when accessing a conditionally hidden section

Show a nice error when the user accidentally views a section
that is conditionally hidden.  Otherwise, continue to show
original error when section is just hidden to them.

MDL-53788 multilangupgrade: fix aggressive course cache rebuilding

MDL-53788 user: fix aggressive course cache rebuilding

MDL-53788 mod_lti: fix aggressive course cache rebuilding

MDL-54920 Behat: Allow viewport size to be set consistently

Provides a new Behat step

And I set viewport size to "800x600"

MDL-52127 grunt: fix coding style issues in gruntfile

MDL-52127 grunt: be aware of third party paths

1) Parse thirdpartylibs.xml and generate an array of third party
file paths to use in grunt tasks
2) In the lint tasks, we filter third party files from being linted
3) We add a new task to generate ignore files - currently for eslint,
but will be potentially useful for other things in the future
4) Remove .eslintignore from source control

Why have the ability to generate a .eslintignore file? For tooling
integration - by having the eslintignore file people can use other
eslint tools without having to just use grunt (e.g. editor
integrations).

MDL-52127 js: switch to a more YUI tolerant eslint config

Instead of having the stricter configuration for all files, use
the less strict default config and switch to the stricter config
for building AMD modules.

This means that the eslint commandline/editor inspections will work
better for all files and not generate false positives when using editor
integrations. But since grunt is required to build AMD modules we still
get the stricter checking for those files on build.

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

MDL-51505 tool_templatelibrary: Prevent jumping when selecting template

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

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

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

MDL-53292 core: deprecate callback delete_course

MDL-51163 course: Remove course/category.php

MDL-37554 activities select all option

MDL-54772 mod_assign: Fixes for PDF editor

* Render the annotations and comments within the drawing canvas, not in
the drawing region.
* Remove edit-related event bindings for stamp annotations.

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

MDL-54899 behat: Tag for MOODLE 32 version

MDL-52127 js: check YUI modules with eslint

Because we don't have the entire YUI rollup, we can't apply some of the
rules safely, so we ignore some rules.

At this point I am not 100% certain this configuration is robust enough,
to deal with the lack fo rollup knowledge although its a good sign that
there are no errors on our entire yui codebase.

MDL-52127 js: check amd files with eslint grunt task

I have spent quite a lot of time working through the current list of
eslint options and configuring them for Moodle style and I think this is
a very good basis to start us at (as well as taking some of out jshint
options out with https://www.npmjs.com/package/polyjuice ). Thanks to
Andrew Nicols, Mark Johnson and Frédéric Massart for some refinements.

With this configuration the grunt build will fail if errors are present
in the js (though you can of course tell jshint to ignore some errors,
as I have done in admin/tool/lp/amd/src/competency_rule_points.js and
defining the Y global in lib/amd/src/yui.js ).

The grunt task will not report warnings by default, but a new
--show-lint-warnings flag will help achieve that. Editor
integrations/stanadalone eslint tool will surely be a better way of
getting eslint errors rather than using the grunt task.

MDL-54787 mod_quiz: Fix notification error on regrade

MDL-54890 course: remove erranous duplicated switch label

Detected by eslint rule 'no-duplicate-case'