From: Jakub Vrana <jakub@vrana.cz>
Date: Mon, 8 Jul 2013 16:59:43 +0000 (-0700)
Subject: Require length when changing type from char to varchar
X-Git-Tag: v4.0.0~167
X-Git-Url: https://git.joonet.de/?a=commitdiff_plain;h=219a3c51c851a18692495faed26aadf5d37243ed;p=adminer.git

Require length when changing type from char to varchar
---

diff --git a/adminer/static/editing.js b/adminer/static/editing.js
index 111b5b38..17ff1893 100644
--- a/adminer/static/editing.js
+++ b/adminer/static/editing.js
@@ -355,12 +355,14 @@ function editingTypeChange(type) {
 	var text = selectValue(type);
 	for (var i=0; i < type.form.elements.length; i++) {
 		var el = type.form.elements[i];
-		if (el.name == name + '[length]' && !(
-			(/(char|binary)$/.test(lastType) && /(char|binary)$/.test(text))
-			|| (/(enum|set)$/.test(lastType) && /(enum|set)$/.test(text))
-		)) {
+		if (el.name == name + '[length]') {
 			el.required = /var(char|binary)$/.test(text);
-			el.value = '';
+			if (!(
+				(/(char|binary)$/.test(lastType) && /(char|binary)$/.test(text))
+				|| (/(enum|set)$/.test(lastType) && /(enum|set)$/.test(text))
+			)) {
+				el.value = '';
+			}
 		}
 		if (lastType == 'timestamp' && el.name == name + '[has_default]' && /timestamp/i.test(formField(type.form, name + '[default]').value)) {
 			el.checked = false;