remove socket listeners in admin/users/search

public/src/forum/admin/users.js

@@ -1,84 +1,82 @@
 
+(function() {
 
+	function initUsers() {
 
-function initUsers() {
+		function isUserAdmin(element) {
-	
+			var parent = $(element).parents('.users-box');
-
+			return (parent.attr('data-admin') !== "0");
-	function isUserAdmin(element) {
+		}
-		var parent = $(element).parents('.users-box');
-		return (parent.attr('data-admin') !== "0");
-	}
 
-	function getUID(element) {
+		function getUID(element) {
-		var parent = $(element).parents('.users-box');
+			var parent = $(element).parents('.users-box');
-		return parent.attr('data-uid');	
+			return parent.attr('data-uid');	
-	}
+		}
 
-	jQuery('.admin-btn').each(function(index, element) {
+		jQuery('.admin-btn').each(function(index, element) {
-		var adminBtn = $(element);
+			var adminBtn = $(element);
-		var isAdmin = isUserAdmin(adminBtn);
+			var isAdmin = isUserAdmin(adminBtn);
-		
+			
-		if(isAdmin)
+			if(isAdmin)
-			adminBtn.addClass('btn-success');
+				adminBtn.addClass('btn-success');
-		else
+			else
-			adminBtn.removeClass('btn-success');
+				adminBtn.removeClass('btn-success');
 
-	});
+		});
 
-	jQuery('.delete-btn').each(function(index, element) {
+		jQuery('.delete-btn').each(function(index, element) {
-		var deleteBtn = $(element);
+			var deleteBtn = $(element);
-		var isAdmin = isUserAdmin(deleteBtn);
+			var isAdmin = isUserAdmin(deleteBtn);
-		
+			
-		if(isAdmin)
+			if(isAdmin)
-			deleteBtn.addClass('disabled');
+				deleteBtn.addClass('disabled');
-		else
+			else
-			deleteBtn.show();
+				deleteBtn.show();
-	});
+		});
 
-	jQuery('.admin-btn').on('click', function() {
+		jQuery('.admin-btn').on('click', function() {
-		var adminBtn = $(this);
+			var adminBtn = $(this);
-		var isAdmin = isUserAdmin(adminBtn);
+			var isAdmin = isUserAdmin(adminBtn);
-		var parent = adminBtn.parents('.users-box');
+			var parent = adminBtn.parents('.users-box');
-		var uid = getUID(adminBtn);
+			var uid = getUID(adminBtn);
-
+
-		if(isAdmin) {
+			if(isAdmin) {
-			socket.emit('api:admin.user.removeAdmin', uid);		
+				socket.emit('api:admin.user.removeAdmin', uid);		
-			adminBtn.removeClass('btn-success');
+				adminBtn.removeClass('btn-success');
-			parent.find('.delete-btn').removeClass('disabled');
+				parent.find('.delete-btn').removeClass('disabled');
-			parent.attr('data-admin', 0);
+				parent.attr('data-admin', 0);
-		}
+			}
-		else {
+			else {
-			bootbox.confirm('Do you really want to make "' + parent.attr('data-username') +'" an admin?', function(confirm) {
+				bootbox.confirm('Do you really want to make "' + parent.attr('data-username') +'" an admin?', function(confirm) {
-				if(confirm) {
+					if(confirm) {
-					socket.emit('api:admin.user.makeAdmin', uid);
+						socket.emit('api:admin.user.makeAdmin', uid);
-					adminBtn.addClass('btn-success');
+						adminBtn.addClass('btn-success');
-					parent.find('.delete-btn').addClass('disabled');
+						parent.find('.delete-btn').addClass('disabled');
-					parent.attr('data-admin', 1);
+						parent.attr('data-admin', 1);
-				}
+					}
-			});
+				});
-		}
+			}
-		
+			
-		return false;
+			return false;
-	});
+		});
 
-	jQuery('.delete-btn').on('click', function() {
+		jQuery('.delete-btn').on('click', function() {
-		var deleteBtn = $(this);
+			var deleteBtn = $(this);
-		var isAdmin = isUserAdmin(deleteBtn);
+			var isAdmin = isUserAdmin(deleteBtn);
-		var parent = deleteBtn.parents('.users-box');
+			var parent = deleteBtn.parents('.users-box');
-		var uid = getUID(deleteBtn);
+			var uid = getUID(deleteBtn);
 
-		if(!isAdmin) {
+			if(!isAdmin) {
-			bootbox.confirm('Do you really want to delete "' + parent.attr('data-username') +'"?', function(confirm) {
+				bootbox.confirm('Do you really want to delete "' + parent.attr('data-username') +'"?', function(confirm) {
-				socket.emit('api:admin.user.deleteUser', uid);		
+					socket.emit('api:admin.user.deleteUser', uid);		
-			});
+				});
-		}
+			}
-		
+			
-		return false;
+			return false;
-	});
+		});
-}
+	}
 
 
-(function() {
 	jQuery('document').ready(function() {
 
 		var yourid = templates.get('yourid');
@@ -103,13 +101,43 @@ function initUsers() {
 			}
 
 			timeoutId = setTimeout(function() {
+				var username = $('#search-user').val();
+				
 				jQuery('.icon-spinner').removeClass('none');
-				socket.emit('api:admin.user.search', $('#search-user').val());
+				socket.emit('api:admin.user.search', username);
+				
 			}, 250);
 		});
 		
 		initUsers();
 		
+		socket.removeAllListeners('api:admin.user.search');
+		
+		socket.on('api:admin.user.search', function(data) {
+
+			var	html = templates.prepare(templates['admin/users'].blocks['users']).parse({
+					users: data
+				}),
+				userListEl = document.querySelector('.users');
+
+			userListEl.innerHTML = html;
+			jQuery('.icon-spinner').addClass('none');				
+
+			if(data && data.length === 0) {
+				$('#user-notfound-notify').html('User not found!')
+					.show()
+					.addClass('label-important')
+					.removeClass('label-success');
+			}
+			else {
+				$('#user-notfound-notify').html(data.length + ' user'+(data.length>1?'s':'') + ' found!')
+					.show()
+					.addClass('label-success')
+					.removeClass('label-important');
+			}
+			
+			initUsers();
+		});
 	
 	});
 	

public/templates/admin/index.tpl

@@ -42,30 +42,6 @@
 	app.enter_room('admin');
 	socket.emit('api:get_all_rooms');
 
-	socket.on('api:admin.user.search', function(data) {
+	
-		console.log('move this into user.js but it should execute only once');
-		var	html = templates.prepare(templates['admin/users'].blocks['users']).parse({
-				users: data
-			}),
-			userListEl = document.querySelector('.users');
-
-		userListEl.innerHTML = html;
-		jQuery('.icon-spinner').addClass('none');				
-
-		if(data && data.length === 0) {
-			$('#user-notfound-notify').html('User not found!')
-				.show()
-				.addClass('label-important')
-				.removeClass('label-success');
-		}
-		else {
-			$('#user-notfound-notify').html(data.length + ' user'+(data.length>1?'s':'') + ' found!')
-				.show()
-				.addClass('label-success')
-				.removeClass('label-important');
-		}
-		
-		user.initUsers();
-	});
 
 </script>

src/user.js

@@ -254,6 +254,11 @@ var utils = require('./../public/src/utils.js'),
 	}
 
 	User.search = function(username, callback) {
+		if(!username) {
+			callback([]);
+			return;
+		}
+
 		RDB.keys('username:*'+ username + '*:uid', function(err, keys) {
 			if(err === null) {
 				if(keys && keys.length) {