Reset event handlers after cloning (thanks to Salko)

diff --git a/adminer/static/editing.js b/adminer/static/editing.js
index fe315709421eb13eeb74cf7f91f3f55f772a6990..25d8c49ee8098144a9581e535d89431dfff5b93a 100644 (file)
--- a/adminer/static/editing.js
+++ b/adminer/static/editing.js
@@ -509,8 +509,8 @@ function dumpClick(event) {
 * @this HTMLSelectElement
 */
 function foreignAddRow() {
-       this.onchange = function () { };
        var row = cloneNode(parentTag(this, 'tr'));
+       this.onchange = function () { };
        var selects = qsa('select', row);
        for (var i=0; i < selects.length; i++) {
                selects[i].name = selects[i].name.replace(/\]/, '1$&');
@@ -525,8 +525,8 @@ function foreignAddRow() {
 * @this HTMLSelectElement
 */
 function indexesAddRow() {
-       this.onchange = function () { };
        var row = cloneNode(parentTag(this, 'tr'));
+       this.onchange = function () { };
        var selects = qsa('select', row);
        for (var i=0; i < selects.length; i++) {
                selects[i].name = selects[i].name.replace(/indexes\[\d+/, '$&1');

diff --git a/changes.txt b/changes.txt
index af1990edd62533503b5fe22f5feca3435f1c115d..38fd81ebf9d6ed1e8902e267e97d6c75acb5d3bd 100644 (file)
--- a/changes.txt
+++ b/changes.txt
@@ -1,5 +1,6 @@
 Adminer 4.6.4-dev:
 Fix inline editing of empty cells (regression from 4.6.3)
+Allow adding more than two indexes and forign key columns at a time (regression from 4.4.0)
 
 Adminer 4.6.3 (released 2018-06-28):
 Disallow using password-less databases

diff --git a/editor/static/editing.js b/editor/static/editing.js
index 5b3a7b93e1833908ee4c7322647f1989ad33aadc..09ba40a8a26c3b57059e7c592789095206151e42 100644 (file)
--- a/editor/static/editing.js
+++ b/editor/static/editing.js
@@ -55,8 +55,8 @@ function whisperClick(event) {
 * @this HTMLInputElement
 */
 function emailFileChange() {
-       this.onchange = function () { };
        var el = this.cloneNode(true);
+       this.onchange = function () { };
        el.value = '';
        this.parentNode.appendChild(el);
 }