diff options
| author | Morris Jobke <hey@morrisjobke.de> | 2015-03-16 13:11:44 +0100 |
|---|---|---|
| committer | Morris Jobke <hey@morrisjobke.de> | 2015-03-26 20:03:27 +0100 |
| commit | 55e1226dfee2778d92e7dbaf9e907f20a80c073b (patch) | |
| tree | e73bf64198b0ba71f228b5828b98ef6e3abe9198 | |
| parent | c8c722bc6de3a58e10ba42a55a178d3ba9308bae (diff) | |
| download | nextcloud-server-55e1226dfee2778d92e7dbaf9e907f20a80c073b.tar.gz nextcloud-server-55e1226dfee2778d92e7dbaf9e907f20a80c073b.zip | |
Don't use placeholder for password in IE8 & 9
* the placeholder is in IE 8 and 9 just set as text and
styled a bit grey. If the form is then serialized without
typing something in the placeholder is sent as value
* this fixes that behaviour for the password field and
properly detects empyt passwords
* fixes #14912
| -rw-r--r-- | settings/js/personal.js | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/settings/js/personal.js b/settings/js/personal.js index e2fb6f025f1..302906702bd 100644 --- a/settings/js/personal.js +++ b/settings/js/personal.js @@ -172,7 +172,13 @@ $(document).ready(function () { $('#pass2').showPassword().keyup(); } $("#passwordbutton").click(function () { - if ($('#pass1').val() !== '' && $('#pass2').val() !== '') { + var isIE8or9 = html.hasClass('lte9'); + // FIXME - TODO - once support for IE8 and IE9 is dropped + // for IE8 and IE9 this will check additionally if the typed in password + // is different from the placeholder, because in IE8/9 the placeholder + // is simply set as the value to look like a placeholder + if ($('#pass1').val() !== '' && $('#pass2').val() !== '' + && !(isIE8or9 && $('#pass2').val() === $('#pass2').attr('placeholder'))) { // Serialize the data var post = $("#passwordform").serialize(); $('#passwordchanged').hide(); |