Make sure there are not empty options on dst select to avoid creating empty user or group. This issue was introduced by b4e9a4da

1 files changed, 6 insertions, 0 deletions

diff --git a/usr/local/www/system_groupmanager.php b/usr/local/www/system_groupmanager.php
index b2e65f7..cd6cf9b 100644
--- a/usr/local/www/system_groupmanager.php
+++ b/usr/local/www/system_groupmanager.php
@@ -210,6 +210,12 @@ function remove_selected(id) {
 function copy_selected(srcid, dstid) {
 	src_selbox = document.getElementById(srcid);
 	dst_selbox = document.getElementById(dstid);
+	count = dst_selbox.options.length;
+	for (index = count - 1; index >= 0; index--) {
+		if (dst_selbox.options[index].value == '') {
+			dst_selbox.remove(index);
+		}
+	}
 	count = src_selbox.options.length;
 	for (index = 0; index < count; index++) {
 		if (src_selbox.options[index].selected) {