MDL-53048 core_form: passwordunmask must exist on page for validation
authorAndrew Nicols <andrew@nicols.co.uk>
Wed, 26 Oct 2016 05:33:08 +0000 (13:33 +0800)
committerAndrew Nicols <andrew@nicols.co.uk>
Wed, 26 Oct 2016 05:33:10 +0000 (13:33 +0800)
commit3778a9226f05a6c0203b291a95378dd8a56c6516
treee4b9e15f5ed914eb57340651acf921b5b4e025f4
parentf34d743dd4d1401f475943bd8ad98eb7ad7ed15d
MDL-53048 core_form: passwordunmask must exist on page for validation

The initial approach I took was to provide a noscript area containing the
standard input element, and to move it to a hidden input element as part of
the passwordunmask setup.

This allowed behat tests to pass both with and without JS enabled (i.e.
with Goutte and real browsers), and also ensured that the real input did
not appear on screen.

However, the standard formslib validation occurs before other page JS is
run and hooks into the elements on page during its setup.
Since noscript elements are not accessible to scripts as standard elements,
the form validation would fail for _client_ side validation and work for
server-side validation.

This change creates creates the hidden input element in the template
instead, as well as the password element in the noscript tag.

Doing this means that when JS is disabled, the form has two elements of the
same name - one hidden, and the second a visible password field.
Since the latter element will always override the former one on form
submission, and the noscript variant is last, the noscript variant wins
when JS is disabled and tests continue to pass.

When JS is enabled, the noscript variant is not a valid part of the DOM
(the browser sees it as a hidden text field essentially). The password is
not automatically entered into the hidden field, so the autocomplete
prevention continues to work as expected.
lib/form/amd/build/passwordunmask.min.js
lib/form/amd/src/passwordunmask.js
lib/form/templates/element-passwordunmask.mustache
theme/boost/templates/core_form/element-passwordunmask.mustache