From bca4465edfd6ed70a301ac88c171ae02e43ceb0e Mon Sep 17 00:00:00 2001 From: Mark Jaquith Date: Wed, 29 Jul 2015 04:30:24 +0000 Subject: [PATCH] Do not re-generate the password after pressing "cancel". Persist the state. Also removes pass2 code not needed since we're generating from one field. fixes #33164 Built from https://develop.svn.wordpress.org/trunk@33475 git-svn-id: http://core.svn.wordpress.org/trunk@33442 1a063a9b-81f0-0310-95a4-ce76da25c4cd --- wp-admin/js/user-profile.js | 8 +++++--- wp-admin/js/user-profile.min.js | 2 +- wp-includes/version.php | 2 +- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/wp-admin/js/user-profile.js b/wp-admin/js/user-profile.js index 4f78fbf068..799ee97276 100644 --- a/wp-admin/js/user-profile.js +++ b/wp-admin/js/user-profile.js @@ -178,7 +178,9 @@ $generateButton.hide(); $passwordWrapper.show(); - generatePassword(); + if ( $pass1Text.val().length === 0 ) { + generatePassword(); + } _.defer( function() { $pass1Text.focus(); @@ -205,7 +207,7 @@ } function check_pass_strength() { - var pass1 = $('#pass1').val(), pass2 = $('#pass2').val(), strength; + var pass1 = $('#pass1').val(), strength; $('#pass-strength-result').removeClass('short bad good strong'); if ( ! pass1 ) { @@ -213,7 +215,7 @@ return; } - strength = wp.passwordStrength.meter( pass1, wp.passwordStrength.userInputBlacklist(), pass2 ); + strength = wp.passwordStrength.meter( pass1, wp.passwordStrength.userInputBlacklist(), pass1 ); switch ( strength ) { case 2: diff --git a/wp-admin/js/user-profile.min.js b/wp-admin/js/user-profile.min.js index b4da63a1cf..27c35eb86a 100644 --- a/wp-admin/js/user-profile.min.js +++ b/wp-admin/js/user-profile.min.js @@ -1 +1 @@ -!function(a){function b(){"function"!=typeof zxcvbn?setTimeout(b,50):(i.val(i.data("pw")),i.trigger("pwupdate"),h.addClass("show-password"))}function c(){var c=a("#pass-strength-result")[0];p=i.val(),h=i.parent(),j=a('').attr({id:"pass1-text",name:"pass1-text",autocomplete:"off"}).addClass(i[0].className).data("pw",i.data("pw")).val(i.val()).on("input propertychange",function(){j.val()!==p&&(k.val(j.val()),i.val(j.val()).trigger("pwupdate"),p=j.val())}),i.after(j),1===parseInt(i.data("reveal"),10)&&b(),i.on("input propertychange pwupdate",function(){i.val()!==p&&(p=i.val(),j.val(p),i.add(j).removeClass("short bad good strong"),c.className&&(i.add(j).addClass(c.className),"short"===c.className||"bad"===c.className?(m.prop("checked")||n.prop("disabled",!0),l.show()):(n.prop("disabled",!1),l.hide())))})}function d(){var a=g.find(".wp-hide-pw");a.show().on("click",function(){1===parseInt(a.data("toggle"),10)?(h.addClass("show-password"),a.data("toggle",0).attr({"aria-label":userProfileL10n.ariaHide}).find(".text").text(userProfileL10n.hide).end().find(".dashicons").removeClass("dashicons-visibility").addClass("dashicons-hidden"),j.focus(),_.isUndefined(j[0].setSelectionRange)||j[0].setSelectionRange(0,100)):(h.removeClass("show-password"),a.data("toggle",1).attr({"aria-label":userProfileL10n.ariaShow}).find(".text").text(userProfileL10n.show).end().find(".dashicons").removeClass("dashicons-hidden").addClass("dashicons-visibility"),i.focus(),_.isUndefined(i[0].setSelectionRange)||i[0].setSelectionRange(0,100))})}function e(){var e,f,r;g=a(".user-pass1-wrap"),a(".user-pass2-wrap").hide(),o=a("#submit").on("click",function(){q=!1}),n=o.add(" #createusersub"),l=a(".pw-weak"),m=l.find(".pw-checkbox"),m.change(function(){n.prop("disabled",!m.prop("checked"))}),i=a("#pass1"),i.length&&c(),k=a("#pass2").on("input propertychange",function(){k.val().length>0&&(i.val(k.val()),k.val(""),p="",i.trigger("pwupdate"))}),e=g.find(".wp-pwd").hide(),d(),f=g.find("button.wp-generate-pw").show(),f.on("click",function(){q=!0,f.hide(),e.show(),b(),_.defer(function(){j.focus(),_.isUndefined(j[0].setSelectionRange)||j[0].setSelectionRange(0,100)},0)}),r=g.find("button.wp-cancel-pw"),r.on("click",function(){q=!1,f.show(),e.hide()}),g.closest("form").on("submit",function(){q=!1,k.val(i.val()),h.removeClass("show-password")})}function f(){var b,c=a("#pass1").val(),d=a("#pass2").val();if(a("#pass-strength-result").removeClass("short bad good strong"),!c)return void a("#pass-strength-result").html(" ");switch(b=wp.passwordStrength.meter(c,wp.passwordStrength.userInputBlacklist(),d)){case 2:a("#pass-strength-result").addClass("bad").html(pwsL10n.bad);break;case 3:a("#pass-strength-result").addClass("good").html(pwsL10n.good);break;case 4:a("#pass-strength-result").addClass("strong").html(pwsL10n.strong);break;case 5:a("#pass-strength-result").addClass("short").html(pwsL10n.mismatch);break;default:a("#pass-strength-result").addClass("short").html(pwsL10n["short"])}}var g,h,i,j,k,l,m,n,o,p,q=!1;a(document).ready(function(){var b,c,d,g,h=a("#display_name");a("#pass1").val("").on("input propertychange pwupdate",f),a("#pass-strength-result").show(),a(".color-palette").click(function(){a(this).siblings('input[name="admin_color"]').prop("checked",!0)}),h.length&&a("#first_name, #last_name, #nickname").bind("blur.user_profile",function(){var b=[],c={display_nickname:a("#nickname").val()||"",display_username:a("#user_login").val()||"",display_firstname:a("#first_name").val()||"",display_lastname:a("#last_name").val()||""};c.display_firstname&&c.display_lastname&&(c.display_firstlast=c.display_firstname+" "+c.display_lastname,c.display_lastfirst=c.display_lastname+" "+c.display_firstname),a.each(a("option",h),function(a,c){b.push(c.value)}),a.each(c,function(d,e){if(e){var f=e.replace(/<\/?[a-z][^>]*>/gi,"");c[d].length&&-1===a.inArray(f,b)&&(b.push(f),a("