[Phpmyadmin-git] [SCM] phpMyAdmin branch, master, updated. RELEASE_3_3_7-10379-g79c5181

Marc Delisle lem9 at users.sourceforge.net
Sat Sep 25 14:50:49 CEST 2010


The branch, master has been updated
       via  79c5181d2a20646c1ad091955c95d9035eae7c04 (commit)
      from  1bf483ef8c7733c2c6f6936e93d242de27b59ab9 (commit)


- Log -----------------------------------------------------------------
commit 79c5181d2a20646c1ad091955c95d9035eae7c04
Author: Marc Delisle <marc at infomarc.info>
Date:   Sat Sep 25 08:50:39 2010 -0400

    bug #3060972 Password generation not available with AJAX

-----------------------------------------------------------------------

Summary of changes:
 js/functions.js           |   37 +++++++++++++++++++++++++++++++++++++
 js/password_generation.js |   10 ----------
 js/server_privileges.js   |   31 ++++---------------------------
 server_privileges.php     |    1 -
 user_password.php         |    1 -
 5 files changed, 41 insertions(+), 39 deletions(-)
 delete mode 100644 js/password_generation.js

diff --git a/js/functions.js b/js/functions.js
index 6efa1e0..bd3a2b8 100644
--- a/js/functions.js
+++ b/js/functions.js
@@ -21,6 +21,42 @@ var only_once_elements = new Array();
 var ajax_message_init = false;
 
 /**
+ * Generate a new password and copy it to the password input areas
+ *
+ * @param   object   the form that holds the password fields
+ *
+ * @return  boolean  always true
+ */
+function suggestPassword(passwd_form) {
+    // restrict the password to just letters and numbers to avoid problems:
+    // "editors and viewers regard the password as multiple words and
+    // things like double click no longer work"
+    var pwchars = "abcdefhjmnpqrstuvwxyz23456789ABCDEFGHJKLMNPQRSTUVWYXZ";
+    var passwordlength = 16;    // do we want that to be dynamic?  no, keep it simple :)
+    var passwd = passwd_form.generated_pw;
+    passwd.value = '';
+
+    for ( i = 0; i < passwordlength; i++ ) {
+        passwd.value += pwchars.charAt( Math.floor( Math.random() * pwchars.length ) )
+    }
+    passwd_form.text_pma_pw.value = passwd.value;
+    passwd_form.text_pma_pw2.value = passwd.value;
+    return true;
+}
+
+/**
+ * for libraries/display_change_password.lib.php 
+ *     libraries/user_password.php
+ *
+ */
+
+function displayPasswordGenerateButton() {
+    $('#tr_element_before_generate_password').parent().append('<tr><td>' + PMA_messages['strGeneratePassword'] + '</td><td><input type="button" id="button_generate_password" value="' + PMA_messages['strGenerate'] + '" onclick="suggestPassword(this.form)" /><input type="text" name="generated_pw" id="generated_pw" /></td></tr>');
+    $('#div_element_before_generate_password').parent().append('<div class="item"><label for="button_generate_password">' + PMA_messages['strGeneratePassword'] + ':</label><span class="options"><input type="button" id="button_generate_password" value="' + PMA_messages['strGenerate'] + '" onclick="suggestPassword(this.form)" /></span><input type="text" name="generated_pw" id="generated_pw" /></div>');
+}
+
+
+/**
  * selects the content of a given object, f.e. a textarea
  *
  * @param   object  element     element of which the content will be selected
@@ -2281,6 +2317,7 @@ $(document).ready(function() {
                 buttons : button_options
             })
             .append(data);
+            displayPasswordGenerateButton();
         }) // end $.get()
     }) // end handler for change password anchor
 
diff --git a/js/password_generation.js b/js/password_generation.js
deleted file mode 100644
index 9153cc9..0000000
--- a/js/password_generation.js
+++ /dev/null
@@ -1,10 +0,0 @@
-/* vim: set expandtab sw=4 ts=4 sts=4: */
-/**
- * for libraries/display_change_password.lib.php 
- *
- */
-
-$(document).ready(function() {
-    $('#tr_element_before_generate_password').parent().append('<tr><td>' + PMA_messages['strGeneratePassword'] + '</td><td><input type="button" id="button_generate_password" value="' + PMA_messages['strGenerate'] + '" onclick="suggestPassword(this.form)" /><input type="text" name="generated_pw" id="generated_pw" /></td></tr>');
-    $('#div_element_before_generate_password').parent().append('<div class="item"><label for="button_generate_password">' + PMA_messages['strGeneratePassword'] + ':</label><span class="options"><input type="button" id="button_generate_password" value="' + PMA_messages['strGenerate'] + '" onclick="suggestPassword(this.form)" /></span><input type="text" name="generated_pw" id="generated_pw" /></div>');
-});
diff --git a/js/server_privileges.js b/js/server_privileges.js
index a3c6385..519f9f9 100644
--- a/js/server_privileges.js
+++ b/js/server_privileges.js
@@ -73,31 +73,6 @@ function checkAddUser(the_form)
     return checkPassword(the_form);
 } // end of the 'checkAddUser()' function
 
-
-/**
- * Generate a new password and copy it to the password input areas
- *
- * @param   object   the form that holds the password fields
- *
- * @return  boolean  always true
- */
-function suggestPassword(passwd_form) {
-    // restrict the password to just letters and numbers to avoid problems:
-    // "editors and viewers regard the password as multiple words and
-    // things like double click no longer work"
-    var pwchars = "abcdefhjmnpqrstuvwxyz23456789ABCDEFGHJKLMNPQRSTUVWYXZ";
-    var passwordlength = 16;    // do we want that to be dynamic?  no, keep it simple :)
-    var passwd = passwd_form.generated_pw;
-    passwd.value = '';
-
-    for ( i = 0; i < passwordlength; i++ ) {
-        passwd.value += pwchars.charAt( Math.floor( Math.random() * pwchars.length ) )
-    }
-    passwd_form.text_pma_pw.value = passwd.value;
-    passwd_form.text_pma_pw2.value = passwd.value;
-    return true;
-}
-
 /**
  * When a new user is created and retrieved over Ajax, append the user's row to
  * the user's table
@@ -238,6 +213,7 @@ $(document).ready(function() {
                 modal: true,
                 buttons: button_options
             }); //dialog options end
+            displayPasswordGenerateButton();
         }); // end $.get()
 
     });//end of Add New User AJAX event handler
@@ -339,7 +315,8 @@ $(document).ready(function() {
             .dialog({
                 width: 900,
                 buttons: button_options
-            })
+            }); //dialog options end
+            displayPasswordGenerateButton();
         }) // end $.get()
     })
 
@@ -467,4 +444,4 @@ $(document).ready(function() {
 
 }, 'top.frame_content'); //end $(document).ready()
 
-/**#@- */
\ No newline at end of file
+/**#@- */
diff --git a/server_privileges.php b/server_privileges.php
index 710151e..abb65cb 100644
--- a/server_privileges.php
+++ b/server_privileges.php
@@ -17,7 +17,6 @@ $GLOBALS['js_include'][] = 'server_privileges.js';
 $GLOBALS['js_include'][] = 'functions.js';
 $GLOBALS['js_include'][] = 'jquery/jquery-ui-1.8.custom.js';
 
-$GLOBALS['js_include'][] = 'password_generation.js';
 require './libraries/server_common.inc.php';
 
 /**
diff --git a/user_password.php b/user_password.php
index 58f9cfc..d3b577b 100644
--- a/user_password.php
+++ b/user_password.php
@@ -35,7 +35,6 @@ if (! defined('PMA_NO_VARIABLES_IMPORT')) {
 require_once './libraries/common.inc.php';
 
 $GLOBALS['js_include'][] = 'server_privileges.js';
-$GLOBALS['js_include'][] = 'password_generation.js';
 
 /**
  * Displays an error message and exits if the user isn't allowed to use this


hooks/post-receive
-- 
phpMyAdmin




More information about the Git mailing list